| @@ -44,7 +44,7 class UserPreference < ActiveRecord::Base | |||
|
|
44 | 44 | if attribute_present? attr_name |
|
|
45 | 45 | super |
|
|
46 | 46 | else |
|
|
47 |
h = read_attribute(:others) |
|
|
|
47 | h = (read_attribute(:others) || {}).dup | |
|
|
48 | 48 | h.update(attr_name => value) |
|
|
49 | 49 | write_attribute(:others, h) |
|
|
50 | 50 | value |
| @@ -54,4 +54,19 class UserPreferenceTest < ActiveSupport::TestCase | |||
|
|
54 | 54 | assert up.save |
|
|
55 | 55 | assert_kind_of Hash, up.others |
|
|
56 | 56 | end |
|
|
57 | ||
|
|
58 | def test_reading_value_from_nil_others_hash | |
|
|
59 | up = UserPreference.new(:user => User.new) | |
|
|
60 | up.others = nil | |
|
|
61 | assert_nil up.others | |
|
|
62 | assert_nil up[:foo] | |
|
|
63 | end | |
|
|
64 | ||
|
|
65 | def test_writing_value_to_nil_others_hash | |
|
|
66 | up = UserPreference.new(:user => User.new) | |
|
|
67 | up.others = nil | |
|
|
68 | assert_nil up.others | |
|
|
69 | up[:foo] = 'bar' | |
|
|
70 | assert_equal 'bar', up[:foo] | |
|
|
71 | end | |
|
|
57 | 72 | end |
General Comments 0
You need to be logged in to leave comments.
Login now