jro_apps/redmine Commit - r1909:925ef8f4f038

Fixed: the default status is lost when reordering issue statuses (#1955)....

Jean-Philippe Lang -

r1909:925ef8f4f038

parent child

Context file:

r1909:925ef8f4f038

app/models/issue_status.rb +2 -2

                validates_length_of :name, :maximum => 30
                validates_format_of :name, :with => /^[\w\s\'\-]*$/i
-               def before_save
-                 IssueStatus.update_all "is_default=#{connection.quoted_false}" if self.is_default?
+               def after_save
+                 IssueStatus.update_all("is_default=#{connection.quoted_false}", ['id <> ?', id]) if self.is_default?
                end
                # Returns the default status for new issues

test/unit/issue_status_test.rb +7 0

                  assert_equal status, IssueStatus.default
                  assert !IssueStatus.find(1).is_default
                end
+               def test_reorder_should_not_clear_default_status
+                 status = IssueStatus.default
+                 status.move_to_bottom
+                 status.reload
+                 assert status.is_default?
+               end
              end

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages