Permalink
Browse files

wasn't paginating correctly

  • Loading branch information...
1 parent 131112d commit ee67e4d378b7c6a95627af60c87afcc2ffaddcdd @michaelglass michaelglass committed Aug 31, 2011
Showing with 5 additions and 2 deletions.
  1. +5 −2 lib/shopify-api-limits/shopify_api/base.rb
@@ -1,10 +1,11 @@
+require 'active_resource'
##
# Redefines #find_every to automatically compose resultsets from multiple ShopifyAPI queries due to API limit of 250 records / request.
# Seemlessly stitches all requests to #all, #find(:all), etc, as if there were no LIMIT.
# @see http://wiki.shopify.com/Retrieving_more_than_250_Products%2C_Orders_etc.
#
module ShopifyAPI
- class Base
+ class Base < ActiveResource::Base
SHOPIFY_API_MAX_LIMIT = 250
class << self
@@ -28,9 +29,11 @@ class << self
raise ShopifyAPI::Limits::Error.new if ShopifyAPI.credit_maxed?
# Iterate from 1 -> max-pages and call the original #find_every, capturing the responses into one list
+
rs = []
1.upto(pages) {|page|
- rs.concat find_every.bind(self).call(options[:params].update(:page => page))
+ options[:params].update(page: page)
+ rs.concat find_every.bind(self).call(options)
}
rs
end

0 comments on commit ee67e4d

Please sign in to comment.