添加username字段后无法用Active Record update更新单属性：验证还是回调问题？

核心问题根源 #

更新失败的本质是模型全局验证规则触发了不必要的校验，具体有两个关键原因：

• 旧用户的username字段为空：你添加username字段后，给模型加了presence: true的全局验证（默认在create和update阶段都会触发），但旧用户（添加字段前创建的）的username值为NULL，调用update时直接触发验证失败，导致数据库回滚。
• password验证触发无效校验：模型里的password存在性验证是全局生效的，而password是has_secure_password提供的虚拟属性——更新时你并没有传入password参数，模型会判定password为空，触发验证失败。

至于邮件会发送，是因为你的代码逻辑没有判断update是否成功，不管更新结果如何都会执行发送邮件的代码。

具体解决方案 #

1. 限定验证规则的触发时机 #

把不需要在更新阶段执行的验证，限定为仅在创建用户时生效：

# 修改User模型的验证规则
validates :email, presence: true, uniqueness: true, format: { with: /\A[\w+\-.]+@[a-z\d\-]+(\.[a-z]+)*\.[a-z]+\z/i, message: "is not a valid email address"}
# 仅创建用户时校验密码
validates :password, presence: true, length: { minimum: 8}, on: :create
# 仅创建用户时校验用户名存在性，更新阶段允许修改（若有需要）
validates :username, presence: true, uniqueness: true, length: { minimum: 3, maximum: 20, message: "must be between 3 and 20 characters"}, on: :create

2. 补全旧用户的username值 #

如果要求所有用户必须有username，可以通过批量操作给旧用户补全，比如用邮箱前缀作为默认用户名：

# 运行rails runner执行这段代码
User.where(username: nil).find_each do |user|
  user.update_column(:username, user.email.split('@').first)
end

3. 修正控制器逻辑，判断更新结果 #

确保只有更新成功后才发送邮件：

def request_reset_password
  @user = User.find_by(email: params[:email])
  if @user && @user.confirmed && @user.confirmation_token.nil?
    @user.confirmation_token = SecureRandom.hex(4)
    # 先判断更新是否成功
    if @user.update(confirmation_token: @user.confirmation_token)
      UserMailer.with(user: @user).request_password_reset.deliver_now
      redirect_to root_path, notice: "Email sent successfully."
    else
      # 可打印错误日志排查问题
      # Rails.logger.error @user.errors.full_messages
      redirect_to root_path, alert: "Failed to reset password."
    end
  else
    redirect_to root_path, alert: "Email doesn't exist or is not confirmed yet."
  end
end

额外优化：简化查询逻辑 #

原代码里@user_email = User.find_by(email: params[:email])再@user = User.find(@user_email.id)完全多余，直接用User.find_by(email: params[:email])即可，避免@user_email为nil时触发异常。

内容的提问来源于stack exchange，提问作者Mercurie