refactored the Page.aggregate to instanciate a lot less AR objects. Its actually instanciating only the ones being displayed. Still over 150 but still a massive improvement compared to over 1500 before.

1 files changed, 9 insertions, 7 deletions

diff --git a/app/models/page.rb b/app/models/page.rb
index 00832bd..37b221e 100644
--- a/app/models/page.rb
+++ b/app/models/page.rb
@@ -62,15 +62,17 @@ class Page < ActiveRecord::Base
     
     options = defaults.merge options
     
-    Page.heads.find_tagged_with(
-      options[:tags].gsub(/\s/, ", "), 
-      :match_all => true,
-      :order => "#{options[:order_by]} #{options[:order_direction]}"
-    ).paginate(:page=>page, :per_page => options[:limit])
-    
+    Page.heads.paginate(
+      find_options_for_find_tagged_with(
+        options[:tags].gsub(/\s/, ","), :match_all => true
+      ).merge(
+        :page     => page,
+        :per_page => options[:limit],
+        :order    => "#{options[:order_by]} #{options[:order_direction]}"
+      )
+    )
   end
 
-  
   def self.custom_templates
     files = Dir.entries(FULL_PUBLIC_TEMPLATE_PATH).select do |x| 
       x if x.gsub!(".html.erb", "")