Rails - update_attributes遇到验证

时间:2010-07-17 15:46:14

标签: ruby-on-rails validation update-attributes

所以我有一个用户模型,包括登录,电子邮件地址,密码,密码确认,姓名,头像(图片)等。前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个参数nameavatar,更新失败,因为密码,密码确认,电子邮件等内容都是需要验证条目,并且它们根本不存在。

我可以通过@user.update_attribute(:name, params[:user][:name])解决这个问题,但后来我担心避免验证是否是Good Thing™。特别是在密码更新方面,我需要验证新密码。

还有其他办法吗?

如果我 只是使用:name:avatar的update_attribute来执行此操作,我将如何进行此操作?

这会有用吗?

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

所以它正在执行!版本,并且发现了异常&amp;浏览器中显示的是:

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"}

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

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

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

我很困惑。 &GT;。&LT;

<小时/> 第二次更新:

我的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}

Urmmm ......好的,这就是你所说的,虚拟属性password实际上是不存在的...... 那么如何绕过 ? Bugger,在这里,我以为我正在聪明地摆弄我自己的身份验证代码......

更改为其中一个身份验证插件有多容易?我是否需要创建新的用户模型?或者该插件是否能够与我当前的插件一起使用?

感谢迄今为止的所有帮助,顺便说一下! :d

1 个答案:

答案 0 :(得分:10)

我已经检查了这一点,通过update_attributes仅对2个属性进行了部分更新。所有其他属性都保留其先前的值,这意味着验证不应失败。有几件事要尝试:

  • 在您的控制器操作中,您是通过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(例如新用户)或者通过{{1}指定了新的纯文本密码,那么只会执行密码验证规则}。如果用户已经拥有密码并且保持不变,则跳过密码验证。

你考虑使用插件是正确的。编写自己的身份验证代码可能是一项有趣的练习,如果您有一些不寻常的要求,则可能需要这些代码。缺点是可能存在您没有想到的安全问题。将restful_authentication之类的内容改装到您的应用中应该不会太糟糕。您可能只需要重命名password=模型上的一个或两个字段。