diff options
| author | Valery Sizov <vsv2711@gmail.com> | 2011-12-18 16:36:30 +0200 |
|---|---|---|
| committer | Valery Sizov <vsv2711@gmail.com> | 2011-12-18 16:36:30 +0200 |
| commit | c940767f2a5b693ff33904ad0fc7974441ed0b75 (patch) | |
| tree | 29ae0bbaa5f75e8d66ab0abb88b93208f69e1b90 | |
| parent | 6a0defc3ca2edf40fc58db4662a40f22514c88f6 (diff) | |
| download | gitlab-ce-c940767f2a5b693ff33904ad0fc7974441ed0b75.tar.gz | |
Notification:: small refactoring
| -rw-r--r-- | app/models/mailer_observer.rb | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/app/models/mailer_observer.rb b/app/models/mailer_observer.rb index 2106e9c4810..159f655494c 100644 --- a/app/models/mailer_observer.rb +++ b/app/models/mailer_observer.rb @@ -52,8 +52,7 @@ class MailerObserver < ActiveRecord::Observer def changed_merge_request(merge_request) if merge_request.assignee_id_changed? - recipients_ids = [merge_request.assignee_id_was] - recipients_ids << merge_request.assignee_id + recipients_ids = merge_request.assignee_id_was, merge_request.assignee_id recipients_ids.delete current_user.id User.find(recipients_ids).each do |user| @@ -64,8 +63,7 @@ class MailerObserver < ActiveRecord::Observer def changed_issue(issue) if issue.assignee_id_changed? - recipients_ids = [issue.assignee_id_was] - recipients_ids << issue.assignee_id + recipients_ids = issue.assignee_id_was, issue.assignee_id recipients_ids.delete current_user.id User.find(recipients_ids).each do |user| |