cafepress_api 0.2.1 → 0.2.2

data/lib/cafepress_api/version.rb

@@ -15,5 +15,5 @@
 # along with CafePressAPI.  If not, see <http://www.gnu.org/licenses/>.
 
 module CafePressAPI
-  VERSION = "0.2.1"
+  VERSION = "0.2.2"
 end

data/lib/cafepress_api.rb

@@ -69,7 +69,9 @@ Or better yet, add the mapping yourself in product_genders.rb and submit it back
 
       # The CafePress API (at the time of this code) doesn't correctly return
       # product images only for the colors which are available for the product.
-      # as such, I am filtering out "invalid" colors from the productUrl(s)
+      # as such, I am filtering out "invalid" colors from the productUrl(s).
+      # Except in the case where the API returns NO colors... then I am accepting
+      # all images with a color id of "0", whatever that means...
       valid_color_ids = []
       product.get_elements('color').each do |color|
         color_id = color.attributes['id']
@@ -88,7 +90,7 @@ Or better yet, add the mapping yourself in product_colors.rb and submit it back
       image_urls = []
       product.get_elements("productImage").each do |product_image|
         # only parse if it is a product image for an available color
-        if valid_color_ids.include?(product_image.attributes['colorId'])
+        if valid_color_ids.include?(product_image.attributes['colorId']) || valid_color_ids.length == 0
           if product_image.attributes['productUrl'].include?('_Front')
             image_urls << {:color_id => product_image.attributes['colorId'], :url => product_image.attributes['productUrl'], :view => FRONT_PRODUCT_VIEW, :size => product_image.attributes['imageSize']}
           elsif product_image.attributes['productUrl'].include?('_Back')
@@ -100,14 +102,18 @@ Or better yet, add the mapping yourself in product_colors.rb and submit it back
         end
       end
 
-      # for whatever reason, some products don't have a default color
-      begin
-        default_color_id = product.get_elements("color[@default='true']").first.attributes['id']
-      rescue
+      # For some reason, there are some products without colors...
+      if valid_color_ids.length == 0
         default_color_id = nil
+      else
+        begin
+          default_color_id = product.get_elements("color[@default='true']").first.attributes['id']
+        rescue
+          # Some products HAVE colors, but don't have any set to default=true
+          default_color_id = nil
+        end
       end
 
-
       products << {
         :name => product.attributes['name'],
         :default_caption => product.attributes['defaultCaption'],

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: cafepress_api
 version: !ruby/object:Gem::Version 
-  hash: 21
+  hash: 19
   prerelease: false
   segments: 
   - 0
   - 2
-  - 1
-  version: 0.2.1
+  - 2
+  version: 0.2.2
 platform: ruby
 authors: 
 - Benjamin Lee Smith