用户属性不会保存

Question

我的用户模型有一个名为“points”的属性，当我尝试在另一个模型控制器中更新它（递减点）时，该属性将不会保存（即使将其添加到 attr_accessible 之后）。

我的 Venture Controller 代码中的方法：

def upvote
@venture = Venture.find(params[:id])
if current_user.points < UPVOTE_AMOUNT
  flash[:error] = "Not enough points!"
else
  flash[:success] = "Vote submitted!"
  current_user.vote_for(@venture)
  decremented = current_user.points - UPVOTE_AMOUNT
  current_user.points = decremented
  current_user.save
  redirect_to :back
end

我什至尝试过使用 update_attributes 方法，但无济于事。

我用闪存添加了一个快速的小测试，看看它是否正在保存：

if current_user.save
    flash[:success] = "Yay"
else
    flash[:error] = "No"
end 

并且返回了错误。

current_user 来自我的会话助手：

def current_user
   @current_user ||= user_from_remember_token
end

提前感谢。

我的用户模型：

class User < ActiveRecord::Base
attr_accessor :password, :points
attr_accessible :name, :email, :password, :password_confirmation, :points

STARTING_POINTS = 50

acts_as_voter
has_karma :ventures

has_many :ventures, :dependent => :destroy

email_regex = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i

validates :name, :presence => true,
                 :length   => { :maximum => 50 }
validates :email, :presence => true,
          :format => { :with => email_regex },
          :uniqueness => { :case_sensitive => false }
validates :password, :presence     => true,
                     :confirmation => true,
                     :length       => { :within => 6..40 }

before_save :encrypt_password
after_initialize :initialize_points

def has_password?(submitted_password)
  password_digest == encrypt(submitted_password)
end

def self.authenticate(email, submitted_password)
  user = find_by_email(email)
  return nil  if user.nil?
  return user if user.has_password?(submitted_password)
end

def self.authenticate_with_salt(id, cookie_salt)
  user = find_by_id(id)
  (user && user.salt == cookie_salt) ? user : nil
end

private

def initialize_points
  self.points = STARTING_POINTS
end  

def encrypt_password
  self.salt = make_salt if new_record?
  self.password_digest = encrypt(password)
end

def encrypt(string)
  secure_hash("#{salt}--#{string}")
end

def make_salt
  secure_hash("#{Time.now.utc}--#{password}")
end

def secure_hash(string)
  Digest::SHA2.hexdigest(string)
end
end

这是我打印 <%= debug current_user %> 后得到的结果

--- !ruby/object:User 
attributes: 
id: 1
name: Test User
email: [email protected]
created_at: 2011-08-27 21:03:01.391918
updated_at: 2011-08-27 21:03:01.418370
password_digest: 40d5ed415df384adaa5182a5fe59964625f9e65a688bb3cc9e30b4eef2a0614b
salt: ac7a332f5d63bc6ad0f61ceacb66bc154e1cad1164fcaed6189d8cea2b55ffe4
admin: t
points: 50
longitude_user: 
latitude_user: 
attributes_cache: {}

changed_attributes: {}

destroyed: false
errors: !omap []

marked_for_destruction: false
new_record: false
points: 50
previously_changed: {}

readonly: false

Answer 1

您要求每次保存用户时都提供用户密码。提交赞成票时，密码不存在，因此验证未通过。

Answer 2

这表明某种验证失败。规避此问题的一个简单方法是使用 update_attribute。这将更新单个属性并保存而不运行验证。

所以改为写

current_user.update_attribute :points, current_user.points - UPVOTE_AMOUNT

这应该有效。

这并不能解决为什么保存现有用户可能会失败的问题，因此您仍然需要检查验证和 before_save 操作。

希望这有帮助。

哈。的确。 update_attribute 会跳过验证，但 before_save 不会。
因此，如果 before_save 是问题所在，您只想在密码更改时触发，因此您可以执行类似的操作

def encrypt_password
  self.salt = make_salt if new_record?      
  self.password_digest = encrypt(password) if self.password_changed?
end

但这仅在 password 是实际的情况下才有效你的模型的属性，这似乎不太可能。为什么要以明文形式存储哈希（出于安全原因）和密码。所以...我猜你只有一个 password_digest 字段，然后它应该变成类似这样的内容：

def encrypt_password
  self.salt = make_salt if new_record?      
  self.password_digest = encrypt(password) if password.present?
end

仅当给出密码时，才尝试重新创建摘要。