jro_apps/redmine Commit - r10259:18d1c62ca8a8

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

Jean-Philippe Lang -

r10259:18d1c62ca8a8

parent child

Context file:

r10259:18d1c62ca8a8

app/models/project.rb +1 -1

                # 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

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages