| @@ -819,12 +819,12 class Issue < ActiveRecord::Base | |||||
| 819 | end |
|
819 | end | |
| 820 | end |
|
820 | end | |
| 821 |
|
821 | |||
| 822 | # Returns the previous assignee if changed |
|
822 | # Returns the previous assignee (user or group) if changed | |
| 823 | def assigned_to_was |
|
823 | def assigned_to_was | |
| 824 | # assigned_to_id_was is reset before after_save callbacks |
|
824 | # assigned_to_id_was is reset before after_save callbacks | |
| 825 | user_id = @previous_assigned_to_id || assigned_to_id_was |
|
825 | user_id = @previous_assigned_to_id || assigned_to_id_was | |
| 826 | if user_id && user_id != assigned_to_id |
|
826 | if user_id && user_id != assigned_to_id | |
| 827 |
@assigned_to_was ||= |
|
827 | @assigned_to_was ||= Principal.find_by_id(user_id) | |
| 828 | end |
|
828 | end | |
| 829 | end |
|
829 | end | |
| 830 |
|
830 | |||
| @@ -2591,4 +2591,12 class IssueTest < ActiveSupport::TestCase | |||||
| 2591 | issue.tracker = Tracker.find(2) |
|
2591 | issue.tracker = Tracker.find(2) | |
| 2592 | assert_equal IssueStatus.find(3), issue.status |
|
2592 | assert_equal IssueStatus.find(3), issue.status | |
| 2593 | end |
|
2593 | end | |
|
|
2594 | ||||
|
|
2595 | def test_assigned_to_was_with_a_group | |||
|
|
2596 | group = Group.find(10) | |||
|
|
2597 | ||||
|
|
2598 | issue = Issue.generate!(:assigned_to => group) | |||
|
|
2599 | issue.reload.assigned_to = nil | |||
|
|
2600 | assert_equal group, issue.assigned_to_was | |||
|
|
2601 | end | |||
| 2594 | end |
|
2602 | end | |
General Comments 0
You need to be logged in to leave comments.
Login now