fixed: no recipient if one of the members of the project has disabled mail notifications

git-svn-id: http://redmine.rubyforge.org/svn/trunk@145 e93f8b46-1217-0410-a6f0-8f06a7374b81

fixed: no recipient if one of the members of the project has disabled mail notifications
git-svn-id: http://redmine.rubyforge.org/svn/trunk@145 e93f8b46-1217-0410-a6f0-8f06a7374b81
e1ef9a5c · Jean-Philippe Lang · f3babef5 · e1ef9a5c
Commit e1ef9a5c authored Jan 03, 2007 by Jean-Philippe Lang
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

mailer.rb app/models/mailer.rb +2 -2

No files found.
--- a/app/models/mailer.rb
+++ b/app/models/mailer.rb
@@ -21,7 +21,7 @@ class Mailer < ActionMailer::Base

  def issue_add(issue)
    # Sends to all project members
-    @recipients     = issue.project.members.collect { |m| m.user.mail if m.user.mail_notification }
+    @recipients     = issue.project.members.collect { |m| m.user.mail if m.user.mail_notification }.compact
    @from           = $RDM_MAIL_FROM
    @subject        = "[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}] #{issue.status.name} - #{issue.subject}"
    @body['issue']  = issue
@@ -30,7 +30,7 @@ class Mailer < ActionMailer::Base
  def issue_edit(journal)
    # Sends to all project members
    issue = journal.journalized
-    @recipients     = issue.project.members.collect { |m| m.user.mail if m.user.mail_notification }
+    @recipients     = issue.project.members.collect { |m| m.user.mail if m.user.mail_notification }.compact
    @from           = $RDM_MAIL_FROM
    @subject        = "[#{issue.project.name} - #{issue.tracker.name} ##{issue.id}] #{issue.status.name} - #{issue.subject}"
    @body['issue']  = issue