From 5ce2f4f81c3315287f8725b423a05f6b71fd2ba8 2012-09-22 09:48:38
From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: 2012-09-22 09:48:38
Subject: [PATCH] Use eager loaded #principal association instead of #user (#11904).

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

---

diff --git a/app/models/project.rb b/app/models/project.rb
index f30832e..71ffdc8 100644
--- 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.user}
+    members.select {|m| m.mail_notification? || m.user.mail_notification == 'all'}.collect {|m| m.principal}
   end
 
   # Returns an array of all custom fields enabled for project issues