Actions
Defect #13008
closedUsage of attribute_present? in UserPreference
Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Description
Hi,
when I enter User.new.pref
in the Rails console, user_id
is stored in the others
hash.
>> User.new.pref #<UserPreference id: nil, user_id: nil, others: {"user_id"=>nil}, hide_mail: false, time_zone: nil>
Solution: In UserPreference#[]
and UserPreference#[]=
attribute_present?
should be replaced by has_attribute?
.
The current implementation also works because UsersController#create
saves the user before creating the preference, so attribute_present? :user_id
returns true
.
Best regards,
Thomas
Actions