[PATCH] BZ803106: add pagination to images list on pool_families show

Tzu-Mainn Chen tzumainn at redhat.com
Mon Jul 2 20:07:13 UTC 2012


https://bugzilla.redhat.com/show_bug.cgi?id=803106
---
 src/app/controllers/pool_families_controller.rb |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/src/app/controllers/pool_families_controller.rb b/src/app/controllers/pool_families_controller.rb
index dcbc59f..0ec70ed 100644
--- a/src/app/controllers/pool_families_controller.rb
+++ b/src/app/controllers/pool_families_controller.rb
@@ -77,7 +77,9 @@ class PoolFamiliesController < ApplicationController
     @title = @pool_family.name
     save_breadcrumb(pool_family_path(@pool_family), @pool_family.name)
     require_privilege(Privilege::VIEW, @pool_family)
-    @images = Aeolus::Image::Warehouse::Image.by_environment(@pool_family.name)
+    @all_images = Aeolus::Image::Warehouse::Image.by_environment(@pool_family.name)
+    @images = paginate_collection(@all_images, params[:page], PER_PAGE)
+
     load_pool_family_tabs
 
     respond_to do |format|
@@ -216,7 +218,7 @@ class PoolFamiliesController < ApplicationController
   def load_pool_family_tabs
     @tabs = [{:name => t('pools.pools'),:view => 'pools', :id => 'pools', :count => @pool_family.pools.count},
              {:name => t('accounts'), :view => 'provider_accounts', :id => 'provider_accounts', :count => @pool_family.provider_accounts.count},
-             {:name => t('images.index.images'), :view => 'images', :id => 'images', :count => @images.count},
+             {:name => t('images.index.images'), :view => 'images', :id => 'images', :count => @all_images.count},
     ]
     add_permissions_tab(@pool_family)
     details_tab_name = params[:details_tab].blank? ? 'pools' : params[:details_tab]
-- 
1.7.6.5




More information about the aeolus-devel mailing list