From: Jean-Philippe Lang <jp_lang@yahoo.fr>
Date: Sun, 12 Jun 2016 12:21:57 +0000 (+0000)
Subject: Preload principals.
X-Git-Tag: 3.4.0~904
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=10c22b5384a4bc2c7977675716a2372e72def8da;p=redmine.git

Preload principals.

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

diff --git a/app/models/project.rb b/app/models/project.rb
index dee2e6dfd..e9b2647d4 100644
--- a/app/models/project.rb
+++ b/app/models/project.rb
@@ -524,7 +524,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.principal.present? && (m.mail_notification? || m.principal.mail_notification == 'all')}.collect {|m| m.principal}
+    members.preload(:principal).select {|m| m.principal.present? && (m.mail_notification? || m.principal.mail_notification == 'all')}.collect {|m| m.principal}
   end
 
   # Returns a scope of all custom fields enabled for project issues