Use eager loaded #principal association instead of #user (#11904).

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10442 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/models/project.rb b/app/models/project.rb
index 71ffdc8f52e2470d553b4ea6e1b83f2ca616a62e..9a65bdd2ee5bb7a4135f377584713cb1a1994be0 100644 (file)
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -472,7 +472,7 @@ class Project < ActiveRecord::Base
   # Returns the users that should be notified on project events
   def notified_users
     # TODO: User part should be extracted to User#notify_about?
-    members.select {|m| m.mail_notification? || m.user.mail_notification == 'all'}.collect {|m| m.principal}
+    members.select {|m| m.principal.present? && (m.mail_notification? || m.principal.mail_notification == 'all')}.collect {|m| m.principal}
   end
 
   # Returns an array of all custom fields enabled for project issues