amazon-ruby 0.0.1 → 0.0.2

data/lib/amazon-ruby/client.rb

@@ -1,44 +1,89 @@
-require "amazon-ruby/cart"
-require "amazon-ruby/product"
 require "amazon-ruby/response"
+require "amazon-ruby/client/itemtype"
+require "amazon-ruby/client/operation"
+require "amazon-ruby/client/responsegroup"
+require "amazon-ruby/client/searchindex"
+require "amazon-ruby/client/countrycode"
 
 module AmazonRuby
 	class Client
 
-		include Cart
-
-		include Product
-
 		API_VERSION = '2011-08-01'
 		API_SERVICE = 'AWSECommerceService'
 
-    def initialize(country, amazon_key, amazon_secret, amazon_associate_tag)
+		def initialize(country, amazon_key, amazon_secret, amazon_associate_tag)
 	    @locale = Locale.new(country);
 	    @amazon_key = amazon_key
 			@amazon_secret = amazon_secret
 			@amazon_associate_tag = amazon_associate_tag
 		end
 
+		#Amazon access key id
 		def amazon_key
 			@amazon_key
 		end
 
+		#Amazon secret access key
 		def amazon_secret
 			@amazon_secret
 		end
 
+		#Amazon associate tag
 		def amazon_associate_tag
 			@amazon_associate_tag
 		end
 
+		#Locale
 		def locale
 			@locale
 		end
 
-		def request_params
-			@request_params
+		def add_to_cart()
+			raise UnsupportedFeature
+		end
+
+		def clear_cart()
+			raise UnsupportedFeature
+		end
+
+		def create_cart()
+			raise UnsupportedFeature
+		end
+
+		def get_cart()
+			raise UnsupportedFeature
+		end
+
+		def modify_cart()
+			raise UnsupportedFeature
+		end
+
+		def itemSearch()
+			raise UnsupportedFeature
+		end
+
+		def item_lookup(id, item_type)
+			request_params = {
+					'Operation' => AmazonRuby::Client::Operation::ITEM_LOOKUP,
+					'ItemId' => id,
+					'IdType' => AmazonRuby::Client::ItemType::UPC,
+					'SearchIndex' => AmazonRuby::Client::SearchIndex::ALL,
+					'ResponseGroup' => AmazonRuby::Client::ResponseGroup::ITEM_ATTRIBUTES
+			}
+			get(request_params)
+		end
+
+		def browse_node_lookup()
+			raise UnsupportedFeature
 		end
 
+		def similar_lookup()
+			raise UnsupportedFeature
+		end
+
+		private
+
+		#Base parameters that should be merged with request parameters
 		def base_params
 			{
 				'AWSAccessKeyId' => @amazon_key,
@@ -49,28 +94,15 @@ module AmazonRuby
 			}
 		end
 
-		#Get all parameters
-		def all_params
-			base_params.merge(@request_params)
-    end
-
-		#Do the request
-		def get(sent_params)
+		#Perform request
+		def get(request_params)
 			raise KeyNotFound unless @amazon_key
 			raise TagNotFound unless @amazon_associate_tag
 			raise SecretNotFound unless @amazon_secret
 
-			#Reset parameters to new parameters for new request
-			@request_params = sent_params
-
 			#Do request
-			resp = Net::HTTP.get_response(url)
-			Response.new(resp.body, resp.code)
-		end
-
-		#Build query string from parameter hash
-		def query_string
-			all_params.sort.map { |k, v| "#{k}=" + escape(v) }.join('&')
+			response = Net::HTTP.get_response(url(request_params))
+			Response.new(response)
 		end
 
 		# Adds a signature to a query
@@ -83,8 +115,11 @@ module AmazonRuby
 		  "#{unsigned_query}&Signature=#{signature}"
 		end
 
-    # The Amazon URL.
-    def url
+		# The Amazon URL.
+		def url(request_params)
+			combined_params = base_params.merge(request_params)
+			query_string = combined_params.sort.map { |k, v| "#{k}=" + escape(v) }.join('&')
+
 			URI::HTTP.build(
 					:host => @locale.host,
 					:path => '/onca/xml',

data/lib/amazon-ruby/client/countrycode.rb

@@ -0,0 +1,15 @@
+module AmazonRuby
+	class Client
+		module CountryCode
+			CANADA = 'ca'
+			CHINA = 'cn'
+			GERMANY = 'de'
+			FRANCE = 'fr'
+			ITALY = 'it'
+			JAPAN = 'jp'
+			UNITED_STATES = 'us'
+			UNITED_KINGDOM = 'uk'
+		end
+	end
+end
+

data/lib/amazon-ruby/client/itemtype.rb

@@ -0,0 +1,11 @@
+module AmazonRuby
+	class Client
+		module ItemType
+			UPC = 'UPC'
+			EAN = 'EAN'
+			ISBN = 'ISBN'
+			SKU = 'SKU'
+		end
+	end
+end
+

data/lib/amazon-ruby/client/operation.rb

@@ -0,0 +1,25 @@
+module AmazonRuby
+	class Client
+		module Operation
+			BROWSE_NODE_LOOKUP = 'BrowseNodeLookup'
+			ITEM_LOOKUP = 'ItemLookup'
+			ITEM_SEARCH = 'ItemSearch'
+			SIMILARITY_LOOKUP = 'SimilarityLookup'
+
+			#Not Yet Implemented
+			CUSTOMER_CONTENT_LOOKUP = 'CustomerContentLookup'
+			CUSTOMER_CONTENT_SEARCH = 'CustomerContentSearch'
+			HELP = 'Help'
+			LIST_LOOKUP = 'ListLookup'
+			LIST_SEARCH = 'ListSearch'
+			SELLER_LISTING_LOOKUP = 'SellerListingLookup'
+			SELLER_LISTING_SEARCH = 'SellerListingSearch'
+			SELLER_LOOKUP = 'SellerLookup'
+			TAG_LOOKUP = 'TagLookup'
+			TRANSACTION_LOOKUP = 'TransactionLookup'
+			VEHICLE_PART_LOOKUP = 'VehiclePartLookup'
+			VEHICLE_PART_SEARCH = 'VehiclePartSearch'
+			VEHICLE_SEARCH = 'VehicleSearch'
+		end
+	end
+end

data/lib/amazon-ruby/client/responsegroup.rb

@@ -0,0 +1,7 @@
+module AmazonRuby
+	class Client
+		module ResponseGroup
+			ITEM_ATTRIBUTES = 'ItemAttributes'
+		end
+	end
+end

data/lib/amazon-ruby/client/searchindex.rb

@@ -0,0 +1,7 @@
+module AmazonRuby
+	class Client
+			module SearchIndex
+				ALL = 'All'
+			end
+		end
+end

data/lib/amazon-ruby/exception.rb

@@ -15,4 +15,8 @@ module AmazonRuby
   #Invalid HTTP Client
 	class InvalidHttpClient < ArgumentError; end
 
+  #Unsupported Feature
+	class UnsupportedFeature < ArgumentError; end
+
+
 end

data/lib/amazon-ruby/locale.rb

@@ -1,3 +1,5 @@
+require "amazon-ruby/client/countrycode"
+
 module AmazonRuby
 	class Locale
 

data/lib/amazon-ruby/response.rb

@@ -1,35 +1,32 @@
 module AmazonRuby
 	class Response
 
+		def initialize(response)
+			@response_body = response.body
+			@response_code = response.code.to_i
+		end
+
+		#Body of response
 		def response_body
 			@response_body
 		end
 
+		#Response code
 		def response_code
 			@response_code
 		end
 
-		def initialize(response_body, response_code)
-			@response_body = response_body
-			@response_code = response_code.to_i
-			@response_xml = nil
-		end
-
-		def errors
-			find('Error')
+		#Was the response valid?
+		def is_valid
+			@response_code == 200 ? true : false;
 		end
 
+		#Return array of values
+		#This is pretty generic.  This should be a LOT more robust.
 		def find(attribute)
-			xml.xpath("//xmlns:#{attribute}").map { |e| parse_xml(e) }
-		end
-		alias [] find
-
-		def error_count
-			errors.count
-		end
-
-		def valid_response
-			response_body == 200 ? true : false;
+			result = Array.new
+			xml.xpath("//xmlns:#{attribute}").map do |e| result << e.text end
+			result
 		end
 
 		# The XML document.
@@ -37,44 +34,5 @@ module AmazonRuby
 			Nokogiri::XML(@response_body)
 		end
 
-		#Such an ugly hacky mess...
-		def parse_xml(xml_input)
-      case xml_input
-      when Nokogiri::XML::Document
-        parse_xml(xml_input.root)
-      when Nokogiri::XML::Element
-        hsh = {}
-
-        xml_input.attributes.each_pair do |key, attr|
-          hsh[key] = attr.value
-        end
-
-        xml_input.children.each do |child|
-          result = parse_xml(child)
-
-          if child.name == 'text'
-            if hsh.empty?
-              return result
-            else
-              hsh['__content__'] = result
-            end
-          elsif hsh[child.name]
-            case hsh[child.name]
-            when Array
-              hsh[child.name] << result
-            else
-              hsh[child.name] = [hsh[child.name]] << result
-            end
-          else
-            hsh[child.name] = result
-          end
-        end
-
-        hsh
-      else
-        xml_input.content.to_s
-      end
-		end
-
 	end
 end

data/lib/amazon-ruby/version.rb

@@ -1,3 +1,3 @@
 module AmazonRuby
-	VERSION = "0.0.1"
+	VERSION = "0.0.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: amazon-ruby
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-12-13 00:00:00.000000000Z
+date: 2011-12-14 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &70348662472500 !ruby/object:Gem::Requirement
+  requirement: &70205913570620 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,7 +21,7 @@ dependencies:
         version: '1.4'
   type: :runtime
   prerelease: false
-  version_requirements: *70348662472500
+  version_requirements: *70205913570620
 description: A Ruby wrapper for the Amazon Advertising API
 email:
 - rhoppes@zappos.com
@@ -29,15 +29,18 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/amazon-ruby/cart.rb
+- lib/amazon-ruby/client/countrycode.rb
+- lib/amazon-ruby/client/itemtype.rb
+- lib/amazon-ruby/client/operation.rb
+- lib/amazon-ruby/client/responsegroup.rb
+- lib/amazon-ruby/client/searchindex.rb
 - lib/amazon-ruby/client.rb
 - lib/amazon-ruby/exception.rb
 - lib/amazon-ruby/locale.rb
-- lib/amazon-ruby/product.rb
 - lib/amazon-ruby/response.rb
 - lib/amazon-ruby/version.rb
 - lib/amazon-ruby.rb
-homepage: ''
+homepage: http://www.zappos.com
 licenses: []
 post_install_message: 
 rdoc_options: []

data/lib/amazon-ruby/cart.rb

@@ -1,25 +0,0 @@
-module AmazonRuby
-	module Cart
-
-		def add_to_cart()
-			puts "ADD TO CART"
-		end
-
-		def clear_cart()
-			puts "CLEAR CART"
-		end
-
-		def create_cart()
-			puts "CREATE CART"
-		end
-
-		def get_cart()
-			puts "GET CART"
-		end
-
-		def modify_cart()
-			puts "MODIFY CART"
-		end
-
-	end
-end

data/lib/amazon-ruby/product.rb

@@ -1,34 +0,0 @@
-module AmazonRuby
-	module Product
-
-		#Hacky mess
-		OPERATION_ITEM_LOOKUP = 'ItemLookup'
-
-		ID_TYPE_UPC = 'UPC'
-		ID_TYPE_EAN = 'EAN'
-		ID_TYPE_ISBN = 'ISBN'
-		ID_TYPE_SKU = 'SKU'
-
-		SEARCH_INDEX_ALL = 'All'
-
-		RESPONSE_GROUP_ITEM_ATTRIBUTES = 'ItemAttributes'
-
-		def itemSearch()
-			puts "DO ITEM SEARCH"
-		end
-
-		def item_lookup(id,item_type)
-			request_params = {'Operation' => OPERATION_ITEM_LOOKUP,'ItemId' => id,'IdType' => ID_TYPE_UPC, 'SearchIndex' => SEARCH_INDEX_ALL, 'ResponseGroup' => RESPONSE_GROUP_ITEM_ATTRIBUTES }
-			get(request_params)
-		end
-
-		def browse_node_lookup()
-			puts "DO BROWSE NODE LOOKUP"
-		end
-
-		def similar_lookup()
-			puts "DO SIMILAR LOOKUP"
-		end
-
-	end
-end