Merge pull request activeadmin#3879 from knu/pagination_params

timoschilling · timoschilling · commit c82d7327adff · 2015-04-07T11:23:58.000+02:00
Add a `:params` option to PaginatedCollection.
diff --git a/lib/active_admin/views/components/paginated_collection.rb b/lib/active_admin/views/components/paginated_collection.rb
@@ -32,11 +32,13 @@ class PaginatedCollection < ActiveAdmin::Component
       # collection => A paginated collection from kaminari
       # options    => These options will be passed to `page_entries_info`
       #   entry_name     => The name to display for this resource collection
+      #   params         => Extra parameters for pagination (e.g. { anchor: 'details' })
       #   param_name     => Parameter name for page number in the links (:page by default)
       #   download_links => Download links override (false or [:csv, :pdf])
       #
       def build(collection, options = {})
         @collection     = collection
+        @params         = options.delete(:params)
         @param_name     = options.delete(:param_name)
         @download_links = options.delete(:download_links)
         @display_total  = options.delete(:pagination_total) { true }
@@ -95,6 +97,7 @@ def build_per_page_select
 
       def build_pagination
         options = {}
+        options[:params]     = @params     if @params
         options[:param_name] = @param_name if @param_name
 
         text_node paginate collection, options
diff --git a/spec/unit/views/components/paginated_collection_spec.rb b/spec/unit/views/components/paginated_collection_spec.rb
@@ -65,6 +65,19 @@ def paginated_collection(*args)
       end
     end
 
+    context "when specifying :params option" do
+      let(:collection) do
+        posts = 10.times.map{ Post.new }
+        Kaminari.paginate_array(posts).page(1).per(5)
+      end
+
+      let(:pagination) { paginated_collection(collection, param_name: :post_page, params: { anchor: 'here' }) }
+
+      it "should pass it through to Kaminari" do
+        expect(pagination.children.last.content).to match(/\/admin\/posts\?post_page=2#here/)
+      end
+    end
+
     context "when specifying download_links: false option" do
       let(:collection) do
         posts = 10.times.map{ Post.new }
