Fixed: editing a message may cause sticky attribute to be NULL (#3356).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2787 e93f8b46-1217-0410-a6f0-8f06a7374b81

Fixed: editing a message may cause sticky attribute to be NULL (#3356).
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@2787 e93f8b46-1217-0410-a6f0-8f06a7374b81
7642b5a9 · Jean-Philippe Lang · 5e760402 · 7642b5a9 · 7642b5a9 · 7642b5a9
Commit 7642b5a9 authored Jun 14, 2009 by Jean-Philippe Lang
Showing with 28 additions and 0 deletions

message.rb app/models/message.rb +4 -0

20090614091200_fix_messages_sticky_null.rb db/migrate/20090614091200_fix_messages_sticky_null.rb +9 -0

message_test.rb test/unit/message_test.rb +15 -0

No files found.
--- a/app/models/message.rb
+++ b/app/models/message.rb
@@ -66,6 +66,10 @@ class Message < ActiveRecord::Base
    board.reset_counters!
  end
  
+  def sticky=(arg)
+    write_attribute :sticky, (arg == true || arg.to_s == '1' ? 1 : 0)
+  end
+  
  def sticky?
    sticky == 1
  end

--- a/db/migrate/20090614091200_fix_messages_sticky_null.rb
+++ b/db/migrate/20090614091200_fix_messages_sticky_null.rb
+class FixMessagesStickyNull < ActiveRecord::Migration
+  def self.up
+    Message.update_all('sticky = 0', 'sticky IS NULL')
+  end
+
+  def self.down
+    # nothing to do
+  end
+end
--- a/test/unit/message_test.rb
+++ b/test/unit/message_test.rb
@@ -128,4 +128,19 @@ class MessageTest < Test::Unit::TestCase
    author.roles_for_project(message.project).first.remove_permission!(:delete_own_messages)
    assert !message.reload.destroyable_by?(author.reload)
  end
+  
+  def test_set_sticky
+    message = Message.new
+    assert_equal 0, message.sticky
+    message.sticky = nil
+    assert_equal 0, message.sticky
+    message.sticky = false
+    assert_equal 0, message.sticky
+    message.sticky = true
+    assert_equal 1, message.sticky
+    message.sticky = '0'
+    assert_equal 0, message.sticky
+    message.sticky = '1'
+    assert_equal 1, message.sticky
+  end
 end