Commit 5b4032d1759f6ac666fd102a7eed689c8dc110ea
Exists in
master
and in
4 other branches
Merge pull request #3118 from 2called-chaos/dont_send_mails_twice
Do not send issue status mail twice if author = assignee
Showing
1 changed file
with
1 additions
and
1 deletions
Show diff stats
app/observers/issue_observer.rb
| @@ -27,7 +27,7 @@ class IssueObserver < ActiveRecord::Observer | @@ -27,7 +27,7 @@ class IssueObserver < ActiveRecord::Observer | ||
| 27 | 27 | ||
| 28 | def create_note(issue) | 28 | def create_note(issue) |
| 29 | Note.create_status_change_note(issue, current_user, issue.state) | 29 | Note.create_status_change_note(issue, current_user, issue.state) |
| 30 | - [issue.author, issue.assignee].compact.each do |recipient| | 30 | + [issue.author, issue.assignee].compact.uniq.each do |recipient| |
| 31 | Notify.delay.issue_status_changed_email(recipient.id, issue.id, issue.state, current_user.id) | 31 | Notify.delay.issue_status_changed_email(recipient.id, issue.id, issue.state, current_user.id) |
| 32 | end | 32 | end |
| 33 | end | 33 | end |