Merge pull request #44 from Masa331/array_query_parameters

Return array query parameters same as received

Author: davidcelis

lib/grape/pagination.rb

@@ -15,7 +15,7 @@ def paginate(collection)
           pages = ApiPagination.pages_from(collection)
 
           pages.each do |k, v|
-            old_params = Rack::Utils.parse_query(request.query_string)
+            old_params = Rack::Utils.parse_nested_query(request.query_string)
             new_params = old_params.merge('page' => v)
             links << %(<#{url}?#{new_params.to_param}>; rel="#{k}")
           end

spec/grape_spec.rb

@@ -99,5 +99,16 @@
         expect(per_page).to eq(10)
       end
     end
+
+    context 'with query string including array parameter' do
+      before do
+        get '/numbers', { count: 100, parity: ['odd', 'even']}
+      end
+
+      it 'returns links with with same received parameters' do
+        expect(links).to include('<http://example.org/numbers?count=100&page=10&parity%5B%5D=odd&parity%5B%5D=even>; rel="last"')
+        expect(links).to include('<http://example.org/numbers?count=100&page=2&parity%5B%5D=odd&parity%5B%5D=even>; rel="next"')
+      end
+    end
   end
 end