]> source.dussan.org Git - redmine.git/commitdiff
Replaces find(:all) calls in classic_pagination.
authorJean-Philippe Lang <jp_lang@yahoo.fr>
Sun, 2 Dec 2012 21:21:28 +0000 (21:21 +0000)
committerJean-Philippe Lang <jp_lang@yahoo.fr>
Sun, 2 Dec 2012 21:21:28 +0000 (21:21 +0000)
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@10924 e93f8b46-1217-0410-a6f0-8f06a7374b81

lib/plugins/classic_pagination/lib/pagination.rb

index 6a3e1a97b0f661cb63937fc8bbcc6b0b6c533a38..0354d5aa659b1a0b1db4f62d1ab1947451551481 100644 (file)
@@ -173,21 +173,28 @@ module ActionController
     # the +model+ and given +conditions+. Override this method to implement a
     # custom counter.
     def count_collection_for_pagination(model, options)
-      model.count(:conditions => options[:conditions],
-                  :joins => options[:join] || options[:joins],
-                  :include => options[:include],
-                  :select => (options[:group] ? "DISTINCT #{options[:group]}" : options[:count]))
+      model.
+        where(options[:conditions]).
+        joins(options[:join] || options[:joins]).
+        includes(options[:include]).
+        select(options[:group] ? "DISTINCT #{options[:group]}" : options[:count]).
+        count
     end
     
     # Returns a collection of items for the given +model+ and +options[conditions]+,
     # ordered by +options[order]+, for the current page in the given +paginator+.
     # Override this method to implement a custom finder.
     def find_collection_for_pagination(model, options, paginator)
-      model.find(:all, :conditions => options[:conditions],
-                 :order => options[:order_by] || options[:order],
-                 :joins => options[:join] || options[:joins], :include => options[:include],
-                 :select => options[:select], :limit => options[:per_page],
-                 :group => options[:group], :offset => paginator.current.offset)
+      model.
+        where(options[:conditions]).
+        order(options[:order_by] || options[:order]).
+        joins(options[:join] || options[:joins]).
+        includes(options[:include]).
+        select(options[:select]).
+        limit(options[:per_page]).
+        group(options[:group]).
+        offset(paginator.current.offset).
+        all
     end
   
     protected :create_paginators_and_retrieve_collections,