tcg-player-sdk 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 30c00bca03558c4b683d20e3ffec48dc5e843fdd5c7170c3410014ad3e2da619
-  data.tar.gz: f5a805adc5ea41c1d09d1d0c9e62c72ad6fa5c8bd147e2cb1a24d2eeca1084f4
+  metadata.gz: 1f6eeb012e245a03c621cfbb4eb55882bed72fd12fd6f19e5aeaff128bf4b158
+  data.tar.gz: 5584cb03b502ad8450de6afe6e4517313acde7dc2259b8d0eac51c1df6bff061
 SHA512:
-  metadata.gz: 6b24bdcb6d5a30ba9e1ec9bb682af8456331b0d74e418d734237699fbce048ffee172b2d15511f85c34b791fde8435b9ff84b37c5a11a2d6dea279f7a57e8ed1
-  data.tar.gz: d77e338ace8d2605deb9c3c170e75078afea999cdf04f16c7548963d92ef63316b3060089b2540a33fb37c9875f3a991d51a886b6bcd121295993842cb44efe8
+  metadata.gz: 76338f3a710643a18677707569fbc84992e022d82f6bed3b0e9ebbb57b1e0543eda8a6866f0ca682396029e56dc7cdf65c15df8bf3f180fbe29bd594a681cd8e
+  data.tar.gz: 572e74efdc7619649ec6ce3201e3405be8c87f7c72f57ab7e088615dd11456cdde860fc62af9570075de274b0aaf5f80e6dbe1f438d6416450e5f2082f95fe82

data/lib/tcg-player-sdk/product_price_list.rb

@@ -1,18 +1,7 @@
 class TCGPlayerSDK::ProductPriceList < TCGPlayerSDK::ResponseStruct
 
-  ##
-  # Group prices by product ID.  This will set
-  # prices to an empty hash if there was an error.  Check response.errors for details about any errors.
-  #
-  # @param _response[TCGPlayerSDK::ResponseStruct] the result of calling TCGPlayerAPI#product_pricing
   def initialize(hash = {})
     super
-
-    if(self.success && self.results)
-      @prices = self.results.map{|r| TCGPlayerSDK::ProductPrice.new(r)}.group_by{|r| r.productId}
-    else
-      @prices = {}
-    end
   end
 
   ##
@@ -42,6 +31,12 @@ class TCGPlayerSDK::ProductPriceList < TCGPlayerSDK::ResponseStruct
   #
   # @return [Hash<Integer, Array<TCGPlayerSDK::ProductPrice>>]
   def prices
+    if(self.success && self.results)
+      @prices ||= self.results.map{|r| TCGPlayerSDK::ProductPrice.new(r)}.group_by{|r| r.productId}
+    else
+      @prices = {}
+    end
+
     @prices
   end
 
@@ -50,13 +45,15 @@ class TCGPlayerSDK::ProductPriceList < TCGPlayerSDK::ResponseStruct
   #
   # @return [Hash<Integer, Array<TCGPlayerSDK::ProductPrice>>]
   def valid_prices
-    valid_prices = {}
-    @prices.each do |id, prl|
-      valid_prices[id] ||= []
-      valid_prices[id] = prl.select{|pr| pr.has_valid_prices?}
+    if(@valid_prices.nil?)
+      @valid_prices = {}
+      @prices.each do |id, prl|
+        @valid_prices[id] ||= []
+        @valid_prices[id] = prl.select{|pr| pr.has_valid_prices?}
+      end
     end
 
-    return valid_prices
+    return @valid_prices
   end
 end
 
@@ -80,4 +77,14 @@ class TCGPlayerSDK::ProductPrice < TCGPlayerSDK::ResponseStruct
   def has_valid_prices?
     return !(lowPrice.nil? && midPrice.nil? && highPrice.nil? && directLowPrice.nil? && marketPrice.nil?)
   end
+
+  ##
+  # Returns "price points" described by this price object.  i.e. "midPrice" and "highPrice".  These are also accessors,
+  # so access away
+  #   price.points.each{|pp| puts price.send(pp)}
+  #
+  # @return Array<Symbol>
+  def points
+    self.keys.select{|k| k.to_s =~ /Price/i}
+  end
 end

data/lib/tcg-player-sdk/tcg_player_sdk.rb

@@ -155,18 +155,21 @@ class TCGPlayerSDK
   # https://docs.tcgplayer.com/reference/catalog_getproduct-1
   #
   def product_details(_ids, params = {})
-    ids = id_list(_ids)
-    query("#{CATALOG_URL}/products/#{ids}", params)
+    slice_ids(_ids) do |slice|
+      query("#{CATALOG_URL}/products/#{id_list(slice)}", params)
+    end
   end
 
   ##
   # Accessor to https://docs.tcgplayer.com/reference/pricing_getproductprices-1
+  # Automatically handles arbitrarily large number of _ids and provides one merged response
   #
   # @param _ids An array of product IDs to query
   # @return [TCGPlayerSDK::ProductPriceList]
   def product_pricing(_ids)
-    ids = id_list(_ids)
-    TCGPlayerSDK::ProductPriceList.new(query("#{PRICING_URL}/product/#{ids}"))
+    slice_ids(_ids) do |slice|
+      TCGPlayerSDK::ProductPriceList.new(query("#{PRICING_URL}/product/#{id_list(slice)}"))
+    end
   end
 
   private
@@ -179,4 +182,23 @@ class TCGPlayerSDK
   def id_list(value)
     value.is_a?(Array) ? value.join(',') : value
   end
+
+  ##
+  # Slice the input ids into smaller pieces, and execute the given block with each slice.  Merge the results
+  def slice_ids(_ids, slice_size=200)
+    ids = _ids.is_a?(Array) ? _ids : [_ids]
+    resp = nil
+
+    ids.each_slice(slice_size) do |slice|
+      response = yield(slice)
+      if(resp.nil?)
+        resp = response
+      else
+        resp.results += response.results if(response.results && resp.results)
+      end
+    end
+
+    return resp
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tcg-player-sdk
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Carl Svensson
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-02-25 00:00:00.000000000 Z
+date: 2022-02-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: http