Optimize the N+1 query in watcher_recipients. #5415

Contributed by Holger Just git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3803 e93f8b46-1217-0410-a6f0-8f06a7374b81

Optimize the N+1 query in watcher_recipients. #5415
Contributed by Holger Just git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@3803 e93f8b46-1217-0410-a6f0-8f06a7374b81
d2f8feb7 · Eric Davis · 116c7a79 · d2f8feb7
Commit d2f8feb7 authored Jun 20, 2010 by Eric Davis
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

acts_as_watchable.rb vendor/plugins/acts_as_watchable/lib/acts_as_watchable.rb +2 -1

No files found.
--- a/vendor/plugins/acts_as_watchable/lib/acts_as_watchable.rb
+++ b/vendor/plugins/acts_as_watchable/lib/acts_as_watchable.rb
@@ -57,7 +57,8 @@ module Redmine
        
        # Returns an array of watchers' email addresses
        def watcher_recipients
-          notified = watchers.collect(&:user).select(&:active?)
+          notified = watcher_users.active
+
          if respond_to?(:visible?)
            notified.reject! {|user| !visible?(user)}
          end