ruby-on-rails - Rails - update_attributes 遇到验证

Question

所以我有一个用户模型，包括登录名、电子邮件地址、密码、密码确认、姓名、头像(图片)等。前 5 个有验证，基本上说明所有 5 个都需要存在才能创建一个新模式。

但是，这会给我带来更新方面的问题。

我有一个编辑页面，用户只能在其中编辑他们的名字和头像。我目前不打算让他们更改登录名，我希望从不同的页面更改电子邮件和密码。

所以编辑表单看起来像这样:

<% form_for @user, :html => { :multipart => true } do |u| %>
 <p>
  <label>Name:</label>
  <%= u.text_field :name %>
 </p>
 <p>
  <label>Avatar:</label>
  <%= display_user_avatar %>
  <%= u.file_field :avatar%>
 </p>
 <p>
  <%= submit_tag %>
 </p>
<% end %>

如果我尝试做一个 @user.update_attributes(params[:user]) ，然后因为只有 2 个参数是 name和 avatar ，更新失败，因为需要密码、密码确认、电子邮件等内容来验证条目，而它们根本不存在于该表单中。

我可以通过做 @user.update_attribute(:name, params[:user][:name]) 来解决这个问题，但随后我担心避免验证是否是一件好事™。特别是关于密码更新之类的事情，我确实需要验证新密码。

还有其他方法吗？

如果我只是使用 update_attribute for :name 来做到这一点和 :avatar ，我将如何去做？

这行得通吗？

params[:user].each do |attribute|
  @user.update_attribute(attribute, params[:user][attribute])
end

这是一种可以接受的方式来做到这一点......？

--编辑为后续-
好的，我按照你的建议尝试并做了

  def update
    @user = User.find_by_login(params[:id])
    if @user.update_attributes!(params[:user])
      redirect_to edit_user_path(@user)
    else
      flash[:notice] = @user.errors
      redirect_to edit_user_path(@user)
    end
  end

所以它正在做 !版本，在浏览器中捕获并显示的异常是:

Validation failed: Password is too short (minimum is 5 characters)

服务器日志中的信息是:

Processing UsersController#update (for 127.0.0.1 at 2010-07-18 11:56:59) [PUT]
  Parameters: {"user"=>{"name"=>"testeeeeee"}, "commit"=>"Save changes", "action"=>"update", "_method"=>"put", "authenticity_token"=>"BMEGRW/pmIJVs1zlVH2TtZX2TQW8soeCXmMx4kquzMA=", "id"=>"tester", "controller"=>"users"}

嗯。看这个，才发现是在提交 "id"=>"tester" .现在，我设置了我的路由，以便它显示用户登录名，而不是 user_id ......这可能是为什么？它正在尝试使用 user_id == tester 查找用户的更新。 ，但由于它不存在，它会尝试创建一个？
由于路线，我实际上做错了什么吗？

嗯... rake 路线告诉我路线是:

edit_user GET    /users/:id/edit(.:format)                             {:action=>"edit", :controller=>"users"}
          PUT    /users/:id(.:format)                                  {:action=>"update", :controller=>"users"}

我在 user.rb中设置了这样的路线文件:

  def to_param
    "#{login}"
  end

但它肯定一直在显示 login而不是 id一直以来。但是我在更新操作开始时也做对了 @user = User.find_by_login(params[:id]) ，然后更新 @user .

我很困惑。 >.<

第二次更新:

我的 User.rb验证内容如下:

  validates_length_of :login, :within => 3..20
  validates_length_of :password, :within => 5..20
  validates_presence_of :login, :email, :password, :password_confirmation, :salt, :name, :on => :create
  validates_uniqueness_of :login, :case_sensitive => false
  validates_confirmation_of :password
  validates_format_of :email, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i, :message => "format is invalid."
  attr_accessor :password, :password_confirmation

hashed_pa​​ssword 部分在这里:

  def password=(pass)
    @password = pass
    self.salt = User.random_string(10) if !self.salt?
    self.hashed_password = User.encrypt(@password, self.salt)
  end

u.attributes给我

>> u.attributes
=> {"salt"=>"NHpH5glxsU", "name"=>"test er", "avatar_updated_at"=>nil, "updated_at"=>Sat Jul 17 07:04:24 UTC 2010, "avatar_file_size"=>nil, "avatar_file_name"=>nil, "hashed_password"=>"84f8675c1ed43ef7f8645a375ea9f867c9a25c83", "id"=>1, "avatar_content_type"=>nil, "login"=>"tester", "email"=>"tester@tester.com", "created_at"=>Fri May 07 10:09:37 UTC 2010}

呃……好吧，就是你说的，关于虚属性 password其实是不存在的……
那么我该如何解决呢？
Bugger，在这里我以为我在摆弄我自己的身份验证代码......

更改为这些身份验证插件之一有多容易？我需要创建一个新的 User 模型吗？或者该插件应该能够与我当前的插件一起使用吗？

感谢到目前为止的所有帮助，顺便说一句! :D

Answer 1

我已经通过 update_attributes 检查了这个和仅 2 个属性的部分更新工作正常。所有其他属性都保留其先前的值，这意味着验证不应失败。有几件事可以尝试:

在您的 Controller 操作中，您是否通过 User.find 加载用户?即您是否从有效模型开始。

您确定更新因验证错误而失败吗？尝试更换 update_attributes与 update_attributes! .如果由于验证更新失败，后者将抛出异常。或查询 @user.errors在尝试更新以确认哪个验证失败之后。

更新

如 User.find_by_login没有找到匹配的记录，它会返回 nil并且不会为您创建新记录。有没有可能是数据库中的tester用户密码太短了？也许该用户是在您将验证放入代码之前创建的？在保存记录之前，您是否使用任何类型的插件或回调来加密用户密码？是 password实际上是一个未保存的虚拟属性，实际密码位于 encrypted_password 之类的字段中。 ?

从 script/console 试试这个(使用与您正在测试应用程序相同的环境 - 开发或生产)

> user = User.find_by_login 'tester'
> user.valid?
> user.attributes

user.valid?将返回 true的 false并且会在您尝试更新之前告诉您用户是否有效。

更新 2(修复验证)

在修复自己的代码方面，您可以在 User 中添加如下所示的方法。模型:

def password_validation_required?
  hashed_password.blank? || !@password.blank?
end

然后更新所有与密码相关的验证规则，以便它们仅在此方法返回 true 时才适用。例如

validates_length_of :password, :within => 5..20, 
  :if => :password_validation_required?

这意味着只有在我们还没有 hashed_password 时才执行密码验证规则。 (例如在新用户上)或者如果通过 password= 指定了新的纯文本密码.如果用户已经有密码并且保持不变，则跳过密码验证。

不过，您考虑使用插件是正确的。编写自己的身份验证代码可能是一项有趣的练习，如果您有一些不寻常的要求，则可能需要这样做。不利的一面是可能存在您没有想到的安全问题。改造类似 restful_authentication到您的应用程序应该不会太糟糕。您可能只需要重命名 User 上的一两个字段。模型。