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

Jason Guiditta jguiditt at redhat.com
Mon Jul 2 20:32:36 UTC 2012


On 02/07/12 16:07 -0400, Tzu-Mainn Chen wrote:
>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
>

ACK, pushed.

-j



More information about the aeolus-devel mailing list