Comment 7 for bug 1871784

Revision history for this message
kuangpeiling (kuangpeiling) wrote :

i can't understand why not fix this bug,but add doc?I see it is easy to fix it by seting create_user function resource_options_ref_to_mapper before update password while update_user function is right
such as
    def create_user(self, user_id, user):
        with sql.session_for_write() as session:
            password = user.pop("password", None)
            user_ref = model.User.from_dict(user)
            # Set resource options passed on creation
            resource_options.resource_options_ref_to_mapper(
                user_ref, model.UserOption)
            if password:
                user_ref.password = password
                if self._change_password_required(user_ref):
                    user_ref.password_ref.expires_at = \
                        datetime.datetime.utcnow()
            user_ref.created_at = datetime.datetime.utcnow()
            session.add(user_ref)
            return base.filter_user(user_ref.to_dict())