| @@ -0,0 +1,9 | |||
|
|
1 | class FixMessagesStickyNull < ActiveRecord::Migration | |
|
|
2 | def self.up | |
|
|
3 | Message.update_all('sticky = 0', 'sticky IS NULL') | |
|
|
4 | end | |
|
|
5 | ||
|
|
6 | def self.down | |
|
|
7 | # nothing to do | |
|
|
8 | end | |
|
|
9 | end | |
| @@ -66,6 +66,10 class Message < ActiveRecord::Base | |||
|
|
66 | 66 | board.reset_counters! |
|
|
67 | 67 | end |
|
|
68 | 68 | |
|
|
69 | def sticky=(arg) | |
|
|
70 | write_attribute :sticky, (arg == true || arg.to_s == '1' ? 1 : 0) | |
|
|
71 | end | |
|
|
72 | ||
|
|
69 | 73 | def sticky? |
|
|
70 | 74 | sticky == 1 |
|
|
71 | 75 | end |
| @@ -128,4 +128,19 class MessageTest < Test::Unit::TestCase | |||
|
|
128 | 128 | author.roles_for_project(message.project).first.remove_permission!(:delete_own_messages) |
|
|
129 | 129 | assert !message.reload.destroyable_by?(author.reload) |
|
|
130 | 130 | end |
|
|
131 | ||
|
|
132 | def test_set_sticky | |
|
|
133 | message = Message.new | |
|
|
134 | assert_equal 0, message.sticky | |
|
|
135 | message.sticky = nil | |
|
|
136 | assert_equal 0, message.sticky | |
|
|
137 | message.sticky = false | |
|
|
138 | assert_equal 0, message.sticky | |
|
|
139 | message.sticky = true | |
|
|
140 | assert_equal 1, message.sticky | |
|
|
141 | message.sticky = '0' | |
|
|
142 | assert_equal 0, message.sticky | |
|
|
143 | message.sticky = '1' | |
|
|
144 | assert_equal 1, message.sticky | |
|
|
145 | end | |
|
|
131 | 146 | end |
General Comments 0
You need to be logged in to leave comments.
Login now