Commit c940767f2a5b693ff33904ad0fc7974441ed0b75
1 parent
6a0defc3
Exists in
master
and in
4 other branches
Notification:: small refactoring
Showing
1 changed file
with
2 additions
and
4 deletions
Show diff stats
app/models/mailer_observer.rb
| @@ -52,8 +52,7 @@ class MailerObserver < ActiveRecord::Observer | @@ -52,8 +52,7 @@ class MailerObserver < ActiveRecord::Observer | ||
| 52 | 52 | ||
| 53 | def changed_merge_request(merge_request) | 53 | def changed_merge_request(merge_request) |
| 54 | if merge_request.assignee_id_changed? | 54 | if merge_request.assignee_id_changed? |
| 55 | - recipients_ids = [merge_request.assignee_id_was] | ||
| 56 | - recipients_ids << merge_request.assignee_id | 55 | + recipients_ids = merge_request.assignee_id_was, merge_request.assignee_id |
| 57 | recipients_ids.delete current_user.id | 56 | recipients_ids.delete current_user.id |
| 58 | 57 | ||
| 59 | User.find(recipients_ids).each do |user| | 58 | User.find(recipients_ids).each do |user| |
| @@ -64,8 +63,7 @@ class MailerObserver < ActiveRecord::Observer | @@ -64,8 +63,7 @@ class MailerObserver < ActiveRecord::Observer | ||
| 64 | 63 | ||
| 65 | def changed_issue(issue) | 64 | def changed_issue(issue) |
| 66 | if issue.assignee_id_changed? | 65 | if issue.assignee_id_changed? |
| 67 | - recipients_ids = [issue.assignee_id_was] | ||
| 68 | - recipients_ids << issue.assignee_id | 66 | + recipients_ids = issue.assignee_id_was, issue.assignee_id |
| 69 | recipients_ids.delete current_user.id | 67 | recipients_ids.delete current_user.id |
| 70 | 68 | ||
| 71 | User.find(recipients_ids).each do |user| | 69 | User.find(recipients_ids).each do |user| |