meineerde/redmine
1
0
Fork 0
mirror of https://github.com/meineerde/redmine.git synced 2025-12-24 09:21:12 +00:00
Code Issues Releases Wiki Activity

Missing notification if previous assignee was a group (#19197).

Browse Source 
git-svn-id: http://svn.redmine.org/redmine/trunk@14049 e93f8b46-1217-0410-a6f0-8f06a7374b81
This commit is contained in:
Jean-Philippe Lang 2015-03-07 08:20:18 +00:00
parent f821eaf722
commit 3789a8539e
2 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/models/issue.rb
View File

@ -830,12 +830,12 @@ class Issue < ActiveRecord::Base
end
end
# Returns the previous assignee if changed
# Returns the previous assignee (user or group) if changed
def assigned_to_was
# assigned_to_id_was is reset before after_save callbacks
user_id = @previous_assigned_to_id || assigned_to_id_was
if user_id && user_id != assigned_to_id
@assigned_to_was ||= User.find_by_id(user_id)
@assigned_to_was ||= Principal.find_by_id(user_id)
end
end

8
test/unit/issue_test.rb
View File

@ -2614,4 +2614,12 @@ class IssueTest < ActiveSupport::TestCase
issue.tracker = Tracker.find(2)
assert_equal IssueStatus.find(3), issue.status
end
def test_assigned_to_was_with_a_group
group = Group.find(10)
issue = Issue.generate!(:assigned_to => group)
issue.reload.assigned_to = nil
assert_equal group, issue.assigned_to_was
end
end