Simplification for later.

Signed-off-by: David Celis <[email protected]>

Author: davidcelis

README.md

@@ -1,6 +1,6 @@
 # api-pagination [![Build Status](https://travis-ci.org/davidcelis/api-pagination.png)](https://travis-ci.org/davidcelis/api-pagination)
 
-Put pagination info for your Rails API in your Link headers, not your response body.
+Put pagination info for your API in Link headers, not the response body.
 
 ## Installation
 

lib/api-pagination.rb

@@ -4,11 +4,9 @@
 module ApiPagination
   protected
     def paginate(scope)
-      query_params = request.query_parameters
-      scope        = instance_variable_get(:"@#{scope}")
-      url          = request.original_url.sub(/\?.*$/, '')
-      pages        = {}
-      links        = []
+      scope = instance_variable_get(:"@#{scope}")
+      url   = request.original_url.sub(/\?.*$/, '')
+      pages = {}
 
       unless scope.first_page?
         pages[:first] = 1
@@ -20,9 +18,9 @@ def paginate(scope)
         pages[:next] = scope.current_page + 1
       end
 
-      pages.each do |k, v|
-        new_params = query_params.merge({ :page => v })
-        links << %(<#{url}?#{new_params.to_param}>; rel="#{k}")
+      links = pages.map do |k, v|
+        new_params = request.query_parameters.merge({ :page => v })
+        %(<#{url}?#{new_params.to_param}>; rel="#{k}")
       end
 
       headers['Link'] = links.join(', ')