Nemcio/Redmine
1
0
Fork 0
mirror of https://github.com/redmine/redmine.git synced 2026-02-07 15:20:11 +01:00
Code Issues Packages Projects Releases Activity

Merged r14988 and r14988 (#21504).

Browse Source 
git-svn-id: http://svn.redmine.org/redmine/branches/3.1-stable@15015 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Jean-Philippe Lang
2016-01-09 08:41:39 +00:00
parent 4a98b7d58f
commit 72ba180cba
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/models/issue_priority.rb
View File

@@ -19,7 +19,7 @@ class IssuePriority < Enumeration
has_many :issues, :foreign_key => 'priority_id'
after_destroy {|priority| priority.class.compute_position_names}
after_save {|priority| priority.class.compute_position_names if priority.position_changed? && priority.position}
after_save {|priority| priority.class.compute_position_names if (priority.position_changed? && priority.position) || priority.active_changed?}
OptionName = :enumeration_issue_priorities

14
test/unit/issue_priority_test.rb
View File

@@ -99,6 +99,20 @@ class IssuePriorityTest < ActiveSupport::TestCase
assert_equal %w(lowest default high4 high3 high2 highest), IssuePriority.active.to_a.sort.map(&:position_name)
end
def test_moving_a_priority_should_update_position_names
prio = IssuePriority.first
prio.move_to = 'lowest'
prio.reload
assert_equal 'highest', prio.position_name
end
def test_deactivating_a_priority_should_update_position_names
prio = IssuePriority.active.order(:position).last
prio.active = false
prio.save
assert_equal 'highest', IssuePriority.active.order(:position).last.position_name
end
def test_destroying_a_priority_should_update_position_names
IssuePriority.find_by_position_name('highest').destroy
assert_equal %w(lowest default high2 highest), IssuePriority.active.to_a.sort.map(&:position_name)