a2z 0.0.4 → 0.1.0

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+## A2z 0.1.0 (Jan 3, 2012)
+
+* Add `editorial_reviews` accessor on `Item`, populating them when present in
+  an item lookup or item search API response. *Matt Huggins*
+
+* Fix issue with browse node response data being empty. *Matt Huggins*
+
 ## A2z 0.0.4 (Jan 2, 2012)
 
 * Implement Amazon's BrowseNodeLookup operation, accessible through

data/README.md

@@ -116,7 +116,7 @@ would make the following calls possible.
     response.total_pages                   # => 44
     response.items.size                    # => 10
     response.items.first                   # => #<A2z::Responses::Item ...>
-    response.more_search_results_url       # => "http://www.amazon.com/gp/redirect.html?camp=2025&creative=386001&location=..."
+    response.more_search_results_url       # => "http://www.amazon.com/gp/redirect.html?camp=2025&creative=386001..."
     response.operation_request             # => #<A2z::Responses::OperationRequest ...>
 
 For more information on interacting with `A2z::Responses::Item` and
@@ -226,7 +226,9 @@ For example:
     item.artist                            # => "Dave Matthews Band"
     item.manufacturer                      # => "RCA"
     item.product_group                     # => "Music"
-    item.detail_page_url                   # => ""http://www.amazon.com/Away-From-World-Deluxe-Version/dp/B008FERRFO..."
+    item.editorial_reviews.size            # => 1
+    item.editorial_reviews.first           # => #<A2z::Responses::EditorialReview ...>
+    item.detail_page_url                   # => "http://www.amazon.com/Away-From-World-Deluxe-Version/dp/B008FERRFO..."
 
 When an item lookup or item search includes the "Images" response group, items
 will also have their `small_image`, `medium_image`, `large_image`, and
@@ -290,6 +292,16 @@ For example:
     image.width                            # => 297
     image.height                           # => 500
 
+### Editorial Reviews
+
+Editorial reviews are accessible on items when the "EditorialResponse" response
+group is included as part of an item lookup or item search request.  Editorial
+response objects only include the source and the content.
+
+    review = item.editorial_reviews.first  # => #<A2z::Responses::EditorialReview ...>
+    review.source                          # => "Product Description"
+    review.content                         # => "Breakout pop sensation Carly Rae Jepsen releases her..."
+
 ### Operation Requests
 
 Item lookup and item search response objects include an

data/lib/a2z/helpers.rb

@@ -14,5 +14,9 @@ module A2z
         word.downcase!
       end
     end
+    
+    def array_wrap(value)
+      value.kind_of?(Array) ? value : [value].compact
+    end
   end
 end

data/lib/a2z/responses.rb

@@ -1,5 +1,6 @@
 require 'a2z/responses/browse_node'
 require 'a2z/responses/browse_node_lookup'
+require 'a2z/responses/editorial_review'
 require 'a2z/responses/image'
 require 'a2z/responses/image_set'
 require 'a2z/responses/item'

data/lib/a2z/responses/browse_node.rb

@@ -1,6 +1,8 @@
 module A2z
   module Responses
     class BrowseNode
+      include Helpers
+      
       attr_accessor :id, :name, :ancestors, :children
       
       def initialize
@@ -23,20 +25,20 @@ module A2z
       
       def self.from_response(data)
         new.tap do |browse_node|
-          browse_node.id   = data['BrowseNodeId']
-          browse_node.name = data['Name']
-          browse_node.root = data['IsCategoryRoot'] == '1' rescue false
-          
-          if data['Children'] && data['Children']['BrowseNode']
-            children = data['Children']['BrowseNode']
-            children = [children] unless children.kind_of?(Array)
-            browse_node.children = children.collect { |child| BrowseNode.from_response(child) }
-          end
-          
-          if data['Ancestors'] && data['Ancestors']['BrowseNode']
-            ancestors = data['Ancestors']['BrowseNode']
-            ancestors = [ancestors] unless ancestors.kind_of?(Array)
-            browse_node.ancestors = ancestors.collect { |ancestor| BrowseNode.from_response(ancestor) }
+          if data
+            browse_node.id   = data['BrowseNodeId']
+            browse_node.name = data['Name']
+            browse_node.root = data['IsCategoryRoot'] == '1' rescue false
+            
+            if data['Children'] && data['Children']['BrowseNode']
+              children = array_wrap(data['Children']['BrowseNode'])
+              browse_node.children = children.collect { |child| BrowseNode.from_response(child) }
+            end
+            
+            if data['Ancestors'] && data['Ancestors']['BrowseNode']
+              ancestors = array_wrap(data['Ancestors']['BrowseNode'])
+              browse_node.ancestors = ancestors.collect { |ancestor| BrowseNode.from_response(ancestor) }
+            end
           end
           
           browse_node.freeze

data/lib/a2z/responses/editorial_review.rb

@@ -0,0 +1,15 @@
+module A2z
+  module Responses
+    class EditorialReview
+      attr_accessor :source, :content
+      
+      def self.from_response(data)
+        new.tap do |editorial_review|
+          editorial_review.source = data['Source']
+          editorial_review.content = data['Content']
+          editorial_review.freeze
+        end
+      end
+    end
+  end
+end

data/lib/a2z/responses/image.rb

@@ -3,6 +3,11 @@ module A2z
     class Image
       attr_accessor :url, :width, :height
       
+      def initialize
+        @width = 0
+        @height = 0
+      end
+      
       def width=(value)
         @width = value.to_i
       end

data/lib/a2z/responses/item.rb

@@ -4,12 +4,13 @@ module A2z
       include Helpers
       
       attr_accessor :asin, :parent_asin, :detail_page_url, :links, :image_sets,
-                    :small_image, :medium_image, :large_image
+                    :small_image, :medium_image, :large_image, :editorial_reviews
       
       def initialize
         @links = []
         @attrs = {}
         @image_sets = {}
+        @editorial_reviews = []
       end
       
       def keys
@@ -23,7 +24,7 @@ module A2z
       def method_missing(name, *args, &block)
         method_name = name.to_s
         
-        if method_name.end_with?('?')  # && method_name != 'has_key?'
+        if method_name.end_with?('?')
           has_key?(method_name.sub(/\?$/, ''))
         elsif has_key?(method_name)
           self[method_name]
@@ -50,8 +51,13 @@ module A2z
           item.parent_asin = data['ParentASIN']
           item.detail_page_url = data['DetailPageURL']
           
+          item.small_image = Image.from_response(data['SmallImage']) if data['SmallImage']
+          item.medium_image = Image.from_response(data['MediumImage']) if data['MediumImage']
+          item.large_image = Image.from_response(data['LargeImage']) if data['LargeImage']
+          
           if data['ItemLinks']
-            item.links = data['ItemLinks']['ItemLink'].collect { |link| ItemLink.from_response(link) }
+            item_links = array_wrap(data['ItemLinks']['ItemLink'])
+            item.links = item_links.collect { |link| ItemLink.from_response(link) }
           end
           
           if data['ItemAttributes']
@@ -59,18 +65,17 @@ module A2z
           end
           
           if data['ImageSets'] && data['ImageSets']['ImageSet']
-            image_sets = data['ImageSets']['ImageSet']
-            image_sets = [image_sets] unless image_sets.kind_of?(Array)
-            
+            image_sets = array_wrap(data['ImageSets']['ImageSet'])
             image_sets.each do |image_set|
               image_set = ImageSet.from_response(image_set)
               item.image_sets[image_set.category.to_sym] = image_set
             end
           end
           
-          item.small_image = Image.from_response(data['SmallImage']) if data['SmallImage']
-          item.medium_image = Image.from_response(data['MediumImage']) if data['MediumImage']
-          item.large_image = Image.from_response(data['LargeImage']) if data['LargeImage']
+          if data['EditorialReviews'] && data['EditorialReviews']['EditorialReview']
+            reviews = array_wrap(data['EditorialReviews']['EditorialReview'])
+            item.editorial_reviews = reviews.collect { |review| EditorialReview.from_response(review) }
+          end
           
           item.freeze
         end

data/lib/a2z/responses/item_search.rb

@@ -1,6 +1,8 @@
 module A2z
   module Responses
     class ItemSearch
+      include Helpers
+      
       attr_accessor :operation_request, :items, :total_results, :total_pages, :more_search_results_url
       
       def initialize
@@ -35,8 +37,7 @@ module A2z
       private
       
       def self.items_from_response(data)
-        items = data['Items']['Item'] rescue []
-        items = [items].compact unless items.kind_of?(Array)
+        items = array_wrap(data['Items']['Item']) rescue []
         items.collect { |item| Item.from_response(item) }
       end
     end

data/lib/a2z/responses/operation_request.rb

@@ -1,6 +1,8 @@
 module A2z
   module Responses
     class OperationRequest
+      include Helpers
+      
       attr_accessor :request_id, :request_processing_time, :headers, :arguments
       
       def initialize
@@ -13,16 +15,14 @@ module A2z
           operation_request.request_id = data['RequestId']
           operation_request.request_processing_time = data['RequestProcessingTime'].to_f
           
-          if data['HTTPHeaders']
-            headers = data['HTTPHeaders']['Header']
-            headers = [headers] unless headers.kind_of?(Array)
+          if data['HTTPHeaders'] && data['HTTPHeaders']['Header']
+            headers = array_wrap(data['HTTPHeaders']['Header'])
             headers = headers.collect { |h| [ h['Name'], h['Value'] ] }
             operation_request.headers = Hash[headers]
           end
           
-          if data['Arguments']
-            arguments = data['Arguments']['Argument']
-            arguments = [arguments] unless arguments.kind_of?(Array)
+          if data['Arguments'] && data['Arguments']['Argument']
+            arguments = array_wrap(data['Arguments']['Argument'])
             arguments = arguments.collect { |a| [ a['Name'], a['Value'] ] }
             operation_request.arguments = Hash[arguments]
           end

data/lib/a2z/version.rb

@@ -1,3 +1,3 @@
 module A2z
-  VERSION = '0.0.4'
+  VERSION = '0.1.0'
 end

data/spec/a2z/responses/editorial_review_spec.rb

@@ -0,0 +1,13 @@
+require 'spec_helper'
+
+describe A2z::Responses::EditorialReview do
+  subject do
+    A2z::Responses::EditorialReview.from_response(editorial_review_hash)
+  end
+  
+  let(:editorial_review_hash) { Hash.new }
+  
+  describe '.from_response' do
+    it 'should return an editorial review object'
+  end
+end

data/spec/a2z/version_spec.rb

@@ -3,5 +3,5 @@ require 'spec_helper'
 describe A2z do
   subject { A2z }
   
-  specify { subject::VERSION.should == '0.0.4' }
+  specify { subject::VERSION.should == '0.1.0' }
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: a2z
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-02 00:00:00.000000000 Z
+date: 2013-01-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: crack
-  requirement: &2156589140 !ruby/object:Gem::Requirement
+  requirement: &2152624760 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2156589140
+  version_requirements: *2152624760
 - !ruby/object:Gem::Dependency
   name: jeff
-  requirement: &2156588160 !ruby/object:Gem::Requirement
+  requirement: &2152623600 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2156588160
+  version_requirements: *2152623600
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &2156587300 !ruby/object:Gem::Requirement
+  requirement: &2152623080 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156587300
+  version_requirements: *2152623080
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &2156586640 !ruby/object:Gem::Requirement
+  requirement: &2152622440 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *2156586640
+  version_requirements: *2152622440
 description: Ruby DSL for Amazon Product Advertising API
 email: matt.huggins@gmail.com
 executables: []
@@ -81,6 +81,7 @@ files:
 - lib/a2z/responses.rb
 - lib/a2z/responses/browse_node.rb
 - lib/a2z/responses/browse_node_lookup.rb
+- lib/a2z/responses/editorial_review.rb
 - lib/a2z/responses/image.rb
 - lib/a2z/responses/image_set.rb
 - lib/a2z/responses/item.rb
@@ -96,6 +97,7 @@ files:
 - spec/a2z/requests/response_group_spec.rb
 - spec/a2z/responses/browse_node_lookup_spec.rb
 - spec/a2z/responses/browse_node_spec.rb
+- spec/a2z/responses/editorial_review_spec.rb
 - spec/a2z/responses/image_set_spec.rb
 - spec/a2z/responses/image_spec.rb
 - spec/a2z/responses/item_link_spec.rb
@@ -138,6 +140,7 @@ test_files:
 - spec/a2z/requests/response_group_spec.rb
 - spec/a2z/responses/browse_node_lookup_spec.rb
 - spec/a2z/responses/browse_node_spec.rb
+- spec/a2z/responses/editorial_review_spec.rb
 - spec/a2z/responses/image_set_spec.rb
 - spec/a2z/responses/image_spec.rb
 - spec/a2z/responses/item_link_spec.rb