peddler 0.1.3 → 0.2.0

data/History.txt

@@ -5,4 +5,7 @@
 == 0.1.2 / 2009-08-17
 	* Minor stuff.
 == 0.1.3 / 2009-11-06
-	* Nothing major, Tom.
+	* Nothing major.
+== 0.2.0 / 2010-01-10
+	* Switched to ActiveSupport under the hood.
+	* Added a purge_inventory short-hand method to the client class.

data/README.rdoc

@@ -115,4 +115,4 @@ Great sales. For a change, let's download something different from Amazon. Here'
 
 Run rdoc and check the source for more detailed info.
 
-Copyright © 2009 Hakan Senol Ensari, released under the MIT license
+Copyright © 2009 Hakan Şenol Ensari, released under the MIT license

data/Rakefile

@@ -1,18 +1,18 @@
-require "rubygems"
-require "rake"
+require 'rubygems'
+require 'rake'
 begin
-  require "jeweler"
+  require 'jeweler'
   Jeweler::Tasks.new do |s|
-    s.name = "peddler"
-    s.summary = "A Ruby wrapper to the Amazon Inventory Management API"
-    s.email = "hakan.ensari@papercavalier.com"
-    s.homepage = "http://snl.github.com/peddler"
-    s.description = "Peddler is a Ruby wrapper to the Amazon Inventory Management API."
-    s.authors = ["Hakan Senol Ensari"]
-    s.add_dependency "xml-simple"
-    s.add_development_dependency "rspec"
+    s.name = 'peddler'
+    s.summary = 'A Ruby wrapper to the Amazon Inventory Management API'
+    s.email = 'hakan.ensari@papercavalier.com'
+    s.homepage = 'http://snl.github.com/peddler'
+    s.description = 'Peddler is a Ruby wrapper to the Amazon Inventory Management API.'
+    s.authors = ['Hakan Şenol Ensari']
+    s.add_dependency 'activesupport', '>= 2.3.5'
+    s.add_development_dependency 'rspec', '>= 1.2.9'
   end
   Jeweler::GemcutterTasks.new
 rescue LoadError
-  puts "Jeweler (or a dependency) not available. Install it with: sudo gem install jeweler"
+  puts 'Jeweler (or a dependency) not available. Install it with: sudo gem install jeweler'
 end

data/VERSION.yml

@@ -1,4 +1,4 @@
 --- 
 :major: 0
-:minor: 1
-:patch: 3
+:minor: 2
+:patch: 0

data/lib/peddler/client.rb

@@ -1,33 +1,38 @@
 # = Peddler
-# Peddler is a Ruby wrapper to the Amazon Inventory management API.
+# Peddler is a Ruby wrapper to the {Amazon Inventory management API}[https://images-na.ssl-images-amazon.com/images/G/01/Amazon_AIM/Amazon_AIM.pdf].
 # 
-# Peddler::Client has some detailed explanation and examples of usage.
+# Peddler::Client contains some detailed explanation and examples of usage.
 module Peddler
   # This is the public interface of the Peddler library.
   class Client
     # Creates a client instance.
     #
-    #    client = Peddler::Client.new :username => "foo@bar.com",
-    #                                 :password => "secret",
-    #                                 :region => "us"
+    #    client = Peddler::Client.new(
+    #      :username => 'foo@bar.com',
+    #      :password => 'secret',
+    #      :region   => 'us'
+    #    )
     #
     def initialize(params={})
-      params.each_pair { |key, value| self.send("#{key}=", value) }
+      params.each_pair { |key, value| self.send("#{key}=", value.to_s) }
     end
     
+    # Sets username.
     def username=(username)
       self.transport.username = username
     end
     
+    # Sets password.
     def password=(password)
       self.transport.password = password
     end
     
     # Sets Amazon region.
     #
-    # Possible regions: ["us", "uk", "de", "ca", "fr", "jp"]
+    #     client.region = :jp
+    #
     def region=(region)
-      self.transport.region = region
+      self.transport.region = region.to_s
     end
     
     # Creates an inventory batch.
@@ -40,6 +45,9 @@ module Peddler
     #      :quantity   => 1)
     #    batch << book
     #    batch.upload
+    #
+    # Once the batch is processed, you may view the report:
+    #
     #    report = client.new_report :upload, :id => batch.id
     #    p report.body
     #    => "Feed Processing Summary:\n\tNumber of records processed\t\t1\n\tNumber of records successful\t\t1\n\n"
@@ -48,8 +56,18 @@ module Peddler
       Peddler::Inventory::Batch.new(self.transport.dup)
     end
     
+    # A short-hand method to purge inventory.
+    #
+    #    client.purge_inventory
+    #
+    def purge_inventory
+      empty_batch = Peddler::Inventory::Batch.new(self.transport.dup)
+      empty_batch.upload(:method => 'purge')
+      
+    end
+    
     # Creates an inventory item. Parameter keys are lowercased and underscored but otherwise the same as
-    # Amazon's colum titles in their tab-delimited templates.
+    # the colum titles in the tab-delimited upload templates provided by Amazon.
     def new_inventory_item(params={})
       Peddler::Inventory::Item.new(params)
     end
@@ -140,13 +158,17 @@ module Peddler
       Peddler::Refunds::Item.new(params)
     end
     
-    # Creates an instance for an already-generated report. Works only with what I call legacy reports, that is,
-    # anything that comes before section 7 in the API docs.
+    # Creates an instance for an already-generated report. Works only with 
+    # legacy reports, meaning anything that comes before section 7 in the API
+    # docs.
     #
-    # Possible report names: [:upload, :order, :preorder, :batch_refund, :open_listings, :open_listings_lite, :open_listings_liter]
+    # Possible report names: [:upload, :order, :preorder, :batch_refund,
+    # :open_listings, :open_listings_lite, :open_listings_liter]
     #
-    # You can download a specific report by using its ID. Otherwise, the instance will fetch the latest available report. One
-    # oddball exception: upload reports do require an ID and will return nil if you don't provide one.
+    # You can download a specific report by using its ID. Otherwise, the 
+    # instance will fetch the latest available report. One oddball exception: 
+    # Upload reports do require an ID and will return nil if you don't provide
+    # one.
     # 
     #    orders_report = client.new_report :order
     #    orders = client.detab(orders_report.body)
@@ -171,8 +193,7 @@ module Peddler
     #
     #    client.generate_report :order, :number_of_days => 15
     #
-    # A word of caution. Open listings may crap up with larger inventories. I will have to migrate to a cURL-based
-    # HTTP client to get that working again.
+    # A word of caution. Open listings may crap up with larger inventories.
     def generate_report(name,params={})
       Peddler::LegacyReports.generate(self.transport, name, params)
     end
@@ -207,7 +228,8 @@ module Peddler
       Peddler::Handlers::TabDelimitedHandler.decode_response(msg)
     end
     
-  protected
+    private
+    
     def transport #:nodoc:all
       @transport ||= Peddler::Transport.new
     end

data/lib/peddler/feeds.rb

@@ -1,34 +1,38 @@
 module Peddler
-  # This module includes functionality to handle the charge-when-ship-related feeds Amazon added to the API
-  # in its latest incarnation in 2009.
+  
+  # This module includes functionality to handle the charge-when-ship-related 
+  # feeds Amazon added to the API in 2009.
   module Feeds
-    # Downloadable file. The processing report in case of feeds. Outputs Amazon's response verbatim.
-    # Will add functionality to parse the response some time down the road.
+    
+    # This is a downloadable file. Outputs Amazon's response verbatim for now.
     class Download
       attr_accessor :id, :type, :related_reference_id, :available_at, :acknowledged
       
+      MAPPED_PARAMS = {
+        'DownloadId'          => 'id',
+        'DownloadType'        => 'type',
+        'RelatedReferenceId'  => 'related_reference_id',
+        'AvailableDate'       => 'available_at',
+        'Acknowledged'        => 'acknowledged'}
+      
       def initialize(transport, params={})
-        @mapped_params = {
-          "DownloadId"          => "id",
-          "DownloadType"        => "type",
-          "RelatedReferenceId"  => "related_reference_id",
-          "AvailableDate"       => "available_at",
-          "Acknowledged"        => "acknowledged"}
         @transport = transport
-        params.each_pair{ |key, value| self.send "#{@mapped_params[key]}=", value }
+        params.each_pair{ |k, v| self.send "#{MAPPED_PARAMS[k]}=", v }
       end
       
-      # Retrieves and returns report
+      # Retrieves and returns report.
       def to_s
         @body ||= download_report
       end
-    private
+      
+      private
+      
       def download_report
         return nil if @id.nil?
         @transport.modernize_request
         @transport.query_params.merge!({
-          "Action"      => "download",
-          "downloadId"  => @id})
+          'Action'      => 'download',
+          'downloadId'  => @id})
         @transport.execute_request
       end
     end
@@ -38,21 +42,22 @@ module Peddler
       attr_writer   :file_content
       attr_accessor :batch, :download, :status, :type, :id, :submitted_at, :started_processing_at, :completed_processing_at, :messages_processed, :messages_successful, :messages_with_errors, :messages_with_warnings
       
+      MAPPED_PARAMS = {
+        'UploadStatus'              => 'status',
+        'UploadType'                => 'type',
+        'UploadId'                  => 'id',
+        'SubmittedDate'             => 'submitted_at',
+        'StartedProcessingDate'     => 'started_processing_at',
+        'CompletedProcessingDate'   => 'completed_processing_at',
+        'CompletedProcesssingDate'  => 'completed_processing_at',
+        'MessagesProcessed'         => 'messages_processed',
+        'MessagesSuccessful'        => 'messages_successful',
+        'MessagesWithErrors'        => 'messages_with_errors',
+        'MessagesWithWarnings'      => 'messages_with_warnings'}
+        
       def initialize(transport)
         @transport = transport
         @batch = []
-        @mapped_params = {
-          "UploadStatus"              => "status",
-          "UploadType"                => "type",
-          "UploadId"                  => "id",
-          "SubmittedDate"             => "submitted_at",
-          "StartedProcessingDate"     => "started_processing_at",
-          "CompletedProcessingDate"   => "completed_processing_at",
-          "CompletedProcesssingDate"  => "completed_processing_at",
-          "MessagesProcessed"         => "messages_processed",
-          "MessagesSuccessful"        => "messages_successful",
-          "MessagesWithErrors"        => "messages_with_errors",
-          "MessagesWithWarnings"      => "messages_with_warnings"}
       end
       
       # Returns content of the upload file.
@@ -72,11 +77,11 @@ module Peddler
       
       # Uploads batch.
       def upload
-        raise PeddlerError.new("Batch already uploaded") unless @id.nil?
+        raise PeddlerError.new('Batch already uploaded') unless @id.nil?
         @transport.modernize_request
         @transport.query_params.merge!({
-          "Action"     => "upload",
-          "uploadType" => @type})
+          'Action'     => 'upload',
+          'uploadType' => @type})
         @transport.body = file_content
         res = @transport.execute_request
         process_response(res)
@@ -87,27 +92,25 @@ module Peddler
       def <<(item)
         @batch << item
       end
-    private
+      
+      private
+      
       def refresh_status
         @transport.modernize_request
         @transport.query_params.merge!({
-          "Action"   => "uploadStatus",
-          "uploadId" => @id})
+          'Action'   => 'uploadStatus',
+          'uploadId' => @id})
         res = @transport.execute_request
         process_response(res)
       end
       
       def process_response(res)
-        xml = Peddler::Handlers::XMLHandler.decode_response(res)
-        params = Peddler::Handlers::XMLHandler.parse(:upload, xml)
-        if params[0]
-          params[0].each_pair do |key, value|
-            if key == "RelatedDownloadsList"
-              params = Peddler::Handlers::XMLHandler.parse(:download, value)
-              @download = Peddler::Feeds::Download.new(@transport, params[0])
-            else
-              self.send "#{@mapped_params[key]}=", value
-            end
+        upload = Hash.from_xml(res)['Response']['Upload'] || Hash.from_xml(res)['Response']['UploadsStatusList']['Upload']
+        upload.each_pair do |key, value|
+          if key == 'RelatedDownloadsList'
+            @download = Peddler::Feeds::Download.new(@transport, value['Download'])
+          else
+            self.send "#{MAPPED_PARAMS[key]}=", value
           end
         end
       end
@@ -119,7 +122,7 @@ module Peddler
       class Batch < Peddler::Feeds::Feed
         def initialize(transport)
           @file_header = "order-id\torder-item-id\tquantity\tship-date\tcarrier-code\tcarrier-name\ttracking-number\tship-method\r\n"
-          @type = "_POST_FLAT_FILE_FULFILLMENT_DATA_"
+          @type = '_POST_FLAT_FILE_FULFILLMENT_DATA_'
           super(transport)
         end
       end
@@ -152,7 +155,7 @@ module Peddler
         def initialize(transport)
           @file_header = "TemplateType=OrderCancellation  Version=1.0/1.0.3 This row for Amazon.com use only.  Do not modify or delete.\r\n" +
                          "order-id\tcancellation-reason-code\tamazon-order-item-code\r\n"
-          @type = "_POST_FLAT_FILE_ORDER_ACKNOWLEDGEMENT_DATA_"
+          @type = '_POST_FLAT_FILE_ORDER_ACKNOWLEDGEMENT_DATA_'
           super(transport)
         end
       end
@@ -174,7 +177,7 @@ module Peddler
         # Outputs a formatted line for the tab-delimited upload file.
         def to_s
           if @cancellation_reason_code.nil? != @amazon_order_item_code.nil?
-            raise PeddlerError.new("Provide codes for both cancellation reason and Amazon order item (or omit both).")
+            raise PeddlerError.new('Provide codes for both cancellation reason and Amazon order item (or omit both).')
           end
           "#{@order_id}\t#{@cancellation_reason_code}\t#{@amazon_order_item_code}\r\n"
         end

data/lib/peddler/handlers.rb

@@ -1,48 +1,24 @@
 module Peddler
   module Handlers
     class XMLHandler
-      # Decodes an XML response.
-      def self.decode_response(res)
-        XmlSimple.xml_in(res)
-      end
-      # Parses responses to uploads and status queries for feeds in Section 7 of the docs. Walks
-      # through lists and returns an array of hashes.
-      def self.parse(name, xml)
-        name = name.to_s.capitalize
-        list = xml["#{name}sStatusList"] || xml["#{name}sList"]
-        if list
-          list.collect { |s| parse_status(name, s) }
-        else
-          [ parse_status(name, xml) ]
-        end
-      end
       
       # Parses legacy responses to queries on statuses of generated reports and inventory uploads.
-      def self.parse_legacy(xml)
-        if xml["Batch"]
-          xml["Batch"].collect  { |input| Peddler::LegacyReports::UploadStatus.new(input) }
-        elsif xml["Report"]
-          xml["Report"].collect { |input| Peddler::LegacyReports::ReportStatus.new(input) }
-        end
-      end
-    protected
-      def self.parse_status(name, xml)
-        if xml[name]
-          xml[name][0].inject({}) do |memo, pair|
-            key, value = pair
-            value[0] = Time.parse(value[0]) if key =~ /Date$/
-            memo.merge!({ key => value[0] })
-          end
+      def self.parse_legacy(hash)
+        if hash['Batches']
+          hash['Batches']['Batch'].collect  { |input| Peddler::LegacyReports::UploadStatus.new(input) }
+        elsif hash['Reports']
+          hash['Reports']['Report'].collect { |input| Peddler::LegacyReports::ReportStatus.new(input) }
         end
       end
     end
     
     class TabDelimitedHandler
-      # Decodes tab-delimited content into an array of OpenStruct objects.
+      # Decodes tab-delimited content into an array of OpenStruct objects. It
+      # assumes first line contains parameter names.
       def self.decode_response(res)
         lines = res.split("\n")
         if lines.size > 1
-          params = lines[0].split("\t").collect{ |value| value.gsub(/-/, "_") }
+          params = lines[0].split("\t").collect{ |value| value.gsub(/-/, '_') }
           params_size = params.size
           (1..(lines.size - 1)).collect do |line_key|
             values = lines[line_key].split("\t")

data/lib/peddler/inventory.rb

@@ -4,9 +4,13 @@ module Peddler
       # Returns number of inventory uploads queued at Amazon
       def self.count(transport)
         transport.legacize_request
-        transport.path << "manual-reports/get-pending-uploads-count"
+        transport.path << 'manual-reports/get-pending-uploads-count'
         res = transport.execute_request
-        Peddler::Handlers::XMLHandler.decode_response(res).to_i
+        if res =~ /^<PendingUploadsCount>(.*)<\/PendingUploadsCount>$/
+          $1.to_i
+        else
+          nil
+        end
       end
     end
     
@@ -22,24 +26,27 @@ module Peddler
       
       # Uploads batch to Amazon.
       def upload(params={})
-        raise PeddlerError.new("Batch already uploaded") unless @id.nil?
+        raise PeddlerError.new('Batch already uploaded') unless @id.nil?
         @transport.legacize_request
-        @transport.path << "catalog-upload/"
+        @transport.path << 'catalog-upload/'
+        
         case params[:method].to_s
-        when "modify"
-          @transport.path << "modify-only"
+        when 'modify'
+          @transport.path << 'modify-only'
           @transport.body = file_content(:short)
-        when "purge"
-          @transport.path << "purge-replace"
+        when 'purge'
+          @transport.path << 'purge-replace'
           @transport.body = file_content
         else
-          @transport.path << "add-modify-delete"
+          @transport.path << 'add-modify-delete'
           @transport.body = file_content
         end
         params.delete(:method)
+        
         params = defaultize(params)
         @transport.headers.merge!(params)
         res = @transport.execute_request
+        
         if res =~ /^<BatchID>(.*)<\/BatchID>$/
           @id = $1
         else
@@ -48,26 +55,8 @@ module Peddler
         true
       end
       
-      # Reformats parameters and mixes in some defaults.
-      def defaultize(params)
-        { :upload_for         => "Marketplace",
-          :file_format        =>"TabDelimited",
-          :asin_match_create  => "Y",
-          :asinate            => "Y",
-          :batch_id           => "Y",
-          :email              => "Y"}.each_pair{ |key, value| params[key] = value unless params[key] }
-        # Some Amazon dimwit figured he'd spell this differently
-        if params[:enable_expedited_shipping]
-          params["enable-expedited-shipping"] = params[:enable_expedited_shipping]
-          params.delete(:enable_expedited_shipping)
-        else
-          params["enable-expedited-shipping"] = "Y"
-        end
-        params
-      end
-    
+      # Returns upload file string.
       def file_content(type=:long)
-        return @file_content if @file_content 
         case type
         when :long
           out = "product-id\tproduct-id-type\titem-condition\tprice\tsku\tquantity\tadd-delete\twill-ship-internationally\texpedited-shipping\titem-note\titem-is-marketplace\tfulfillment-center-id\titem-name\titem-description\tcategory1\timage-url\tshipping-fee\tbrowse-path\tstorefront-feature\tboldface\tasin1\tasin2\tasin3\r\n"
@@ -76,20 +65,63 @@ module Peddler
           out = "sku\tprice\tquantity\r\n"
           @batch.each{ |item| out << item.to_s(:short) }
         end
-        @file_content = out
-      end
-    
-      def file_content=(file_content)
-        @file_content = file_content
+        out
       end
-    
+      
+      # Adds an item to inventory.
       def <<(item)
         @batch << item
       end
+      
+      private
+      
+      def defaultize(params)
+        params = {
+          :upload_for         => 'Marketplace',
+          :file_format        => 'TabDelimited',
+          :asin_match_create  => 'Y',
+          :asinate            => 'Y',
+          :batch_id           => 'Y',
+          :email              => 'Y'
+        }.merge(params)
+        
+        # Some Amazon dimwit figured he'd rather not camelize this one
+        if params[:enable_expedited_shipping]
+          params['enable-expedited-shipping'] = params[:enable_expedited_shipping]
+          params.delete(:enable_expedited_shipping)
+        else
+          params['enable-expedited-shipping'] = 'Y'
+        end
+        
+        params
+      end
     end
     
+    # This is an inventory item.
     class Item
-      attr_accessor :product_id, :product_id_type, :item_condition, :price, :sku, :quantity, :add_delete, :will_ship_internationally, :expedited_shipping, :item_note, :item_is_marketplace, :fulfillment_center_id, :item_name, :item_description, :category1, :image_url, :shipping_fee, :browse_path, :storefront_feature, :boldface, :asin1, :asin2, :asin3
+      attr_accessor :product_id,
+                    :product_id_type,
+                    :item_condition,
+                    :price,
+                    :sku,
+                    :quantity,
+                    :add_delete,
+                    :will_ship_internationally,
+                    :expedited_shipping,
+                    :item_note,
+                    :item_is_marketplace,
+                    :fulfillment_center_id,
+                    :item_name,
+                    :item_description,
+                    :category1,
+                    :image_url,
+                    :shipping_fee,
+                    :browse_path,
+                    :storefront_feature,
+                    :boldface,
+                    :asin1,
+                    :asin2,
+                    :asin3
       
       def initialize(params={})
         params.each_pair{ |key, value| send("#{key.to_s}=", value) }

data/lib/peddler/legacy_reports.rb

@@ -1,6 +1,8 @@
 module Peddler
+  
   # This module contains methods to manage legacy reports -- anything that comes before section 7 in the API docs.
   module LegacyReports
+    
     # Returns statuses of most recent reports in an array of OpenStructs.
     def self.latest(transport,name,params={})
       transport.legacize_request
@@ -13,8 +15,7 @@ module Peddler
         transport.headers[:number_of_reports] = params[:count] if params[:count]
       end
       res = transport.execute_request
-      xml = Peddler::Handlers::XMLHandler.decode_response(res)
-      Peddler::Handlers::XMLHandler.parse_legacy(xml)
+      Peddler::Handlers::XMLHandler.parse_legacy(Hash.from_xml(res))
     end
     
     # Requests a report to be generated and returns the report instance if request is successful.

data/lib/peddler/reports.rb

@@ -1,28 +1,32 @@
 module Peddler
+  
   # This module generates and downloads unshipped order reports.
-  # I decided to keep this out of Peddler::LegacyReports because the API is quite different.
+  # I decided to keep this out of Peddler::LegacyReports because the API is
+  # quite different.
   module Reports
-    # This is an unshipped orders report. It's very similar to the feed objects so I'm just porting over the class.
-    # It will have a few stray attributes, but whatever.
+    
+    # This is an unshipped orders report. It is very similar to the feed 
+    # objects, so I'm just porting over the class.
     class UnshippedOrdersReport < Peddler::Feeds::Feed
       alias         :unshipped_orders :batch
       attr_accessor :starts_at, :ends_at, :scheduled
       
-      # Creates new unshipped order report. It literally sends a request to Amazon to generate the
-      # report if the report ID is not already set.
+      MAPPED_PARAMS = {
+        "ReportID"                  => "id",
+        "StartDate"                 => "starts_at",
+        "EndDate"                   => "ends_at",
+        "DownloadType"              => "type",
+        "Scheduled"                 => "scheduled",
+        "ReportStatus"              => "status",
+        "SubmittedDate"             => "submitted_at",
+        "StartedProcessingDate"     => "started_processing_at",
+        "CompletedProcessingDate"   => "completed_processing_at",
+        "CompletedProcesssingDate"  => "completed_processing_at"}
+      
+      # Creates new unshipped order report. It will send a request to 
+      # Amazon to generate the report if the report ID is not already set.
       def initialize(transport, params={})
         super(transport)
-        @mapped_params = {
-          "ReportID"                  => "id",
-          "StartDate"                 => "starts_at",
-          "EndDate"                   => "ends_at",
-          "DownloadType"              => "type",
-          "Scheduled"                 => "scheduled",
-          "ReportStatus"              => "status",
-          "SubmittedDate"             => "submitted_at",
-          "StartedProcessingDate"     => "started_processing_at",
-          "CompletedProcessingDate"   => "completed_processing_at",
-          "CompletedProcesssingDate"  => "completed_processing_at"}
         params.each_pair{ |key, value| self.send "#{key}=", value }
         @starts_at ||= (Date.today - 7).strftime("%Y-%m-%dT00:00:00-00:00")
         @ends_at ||= (Date.today + 1).strftime("%Y-%m-%dT00:00:00-00:00")
@@ -33,7 +37,9 @@ module Peddler
         end
         self
       end
-    private
+      
+      private
+      
       def refresh_status
         @transport.modernize_request
         @transport.query_params.merge!({
@@ -55,17 +61,14 @@ module Peddler
       end
       
       def process_response(res)
-        xml = Peddler::Handlers::XMLHandler.decode_response(res)
-        params = Peddler::Handlers::XMLHandler.parse(:report, xml)
-        if params[0]
-          params[0].each_pair do |key, value|
-            if key == "ListOfDownloads"
-              params = Peddler::Handlers::XMLHandler.parse(:download, value)
-              @download = Peddler::Feeds::Download.new(@transport, params[0])
-              @batch = Peddler::Handlers::TabDelimitedHandler.decode_response(@download.to_s)
-            else
-              self.send "#{@mapped_params[key]}=", value
-            end
+        hash = Hash.from_xml(res)
+        hash['Response']['Report'].each_pair do |key, value|
+          if key == "ListOfDownloads"
+            params = Peddler::Handlers::XMLHandler.parse(:download, value)
+            @download = Peddler::Feeds::Download.new(@transport, value['Download'])
+            @batch = Peddler::Handlers::TabDelimitedHandler.decode_response(@download.to_s)
+          else
+            self.send "#{MAPPED_PARAMS[key]}=", value
           end
         end
       end

data/lib/peddler.rb

@@ -1,25 +1,19 @@
 # Peddler is a Ruby wrapper to the Amazon Inventory management API.
 module Peddler
-  VERSION = "0.1"  
+  VERSION = '0.2.0'
 end
 
-class String #:nodoc: all
-  def camelize
-    self.gsub(/(^|_)(.)/) { $2.upcase }
-  end
-end
-
-require "net/https"
-require "ostruct"
-require "tempfile"
-require "time"
-require "xmlsimple"
+require 'net/https'
+require 'ostruct'
+require 'tempfile'
+require 'time'
+require 'active_support'
 
-require File.dirname(__FILE__) + "/peddler/client"
-require File.dirname(__FILE__) + "/peddler/handlers"
-require File.dirname(__FILE__) + "/peddler/feeds"
-require File.dirname(__FILE__) + "/peddler/inventory"
-require File.dirname(__FILE__) + "/peddler/legacy_reports"
-require File.dirname(__FILE__) + "/peddler/refunds"
-require File.dirname(__FILE__) + "/peddler/reports"
-require File.dirname(__FILE__) + "/peddler/transport"
+require File.dirname(__FILE__) + '/peddler/client'
+require File.dirname(__FILE__) + '/peddler/handlers'
+require File.dirname(__FILE__) + '/peddler/feeds'
+require File.dirname(__FILE__) + '/peddler/inventory'
+require File.dirname(__FILE__) + '/peddler/legacy_reports'
+require File.dirname(__FILE__) + '/peddler/refunds'
+require File.dirname(__FILE__) + '/peddler/reports'
+require File.dirname(__FILE__) + '/peddler/transport'

data/peddler.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{peddler}
-  s.version = "0.1.3"
+  s.version = "0.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Hakan Senol Ensari"]
-  s.date = %q{2009-11-06}
+  s.authors = ["Hakan \305\236enol Ensari"]
+  s.date = %q{2010-01-11}
   s.description = %q{Peddler is a Ruby wrapper to the Amazon Inventory Management API.}
   s.email = %q{hakan.ensari@papercavalier.com}
   s.extra_rdoc_files = [
@@ -66,15 +66,15 @@ Gem::Specification.new do |s|
     s.specification_version = 3
 
     if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
-      s.add_runtime_dependency(%q<xml-simple>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, [">= 0"])
+      s.add_runtime_dependency(%q<activesupport>, [">= 2.3.5"])
+      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
     else
-      s.add_dependency(%q<xml-simple>, [">= 0"])
-      s.add_dependency(%q<rspec>, [">= 0"])
+      s.add_dependency(%q<activesupport>, [">= 2.3.5"])
+      s.add_dependency(%q<rspec>, [">= 1.2.9"])
     end
   else
-    s.add_dependency(%q<xml-simple>, [">= 0"])
-    s.add_dependency(%q<rspec>, [">= 0"])
+    s.add_dependency(%q<activesupport>, [">= 2.3.5"])
+    s.add_dependency(%q<rspec>, [">= 1.2.9"])
   end
 end
 

data/spec/peddler/feeds_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), "/../spec_helper") 
+require File.join(File.dirname(__FILE__), '/../spec_helper') 
 
 module Peddler
   module Feeds
@@ -8,39 +8,39 @@ module Peddler
           @transport = Peddler::Transport.new
           @transport.modernize_request
           @feed = Peddler::Feeds::OrderFulfillment::Batch.new(@transport)
-          @fulfilled_order = Peddler::Feeds::OrderFulfillment::Item.new :order_id  => "123-1234567-1234567",
-                                                                 :ship_date => Date.parse("2009-08-11").to_s
+          @fulfilled_order = Peddler::Feeds::OrderFulfillment::Item.new :order_id  => '123-1234567-1234567',
+                                                                 :ship_date => Date.parse('2009-08-11').to_s
         end
       
-        it "should add items to batch" do
+        it 'should add items to batch' do
           @feed.batch.size.should == 0
           @feed << @fulfilled_order
           @feed.batch.size.should == 1
         end
     
-        it "should create content for upload" do
+        it 'should create content for upload' do
           @feed << @fulfilled_order
           @feed.file_content.should == "order-id\torder-item-id\tquantity\tship-date\tcarrier-code\tcarrier-name\ttracking-number\tship-method\r\n123-1234567-1234567\t\t\t2009-08-11\t\t\t\t\r\n"
         end
   
-        it "should upload" do
+        it 'should upload' do
           @feed.id.should == nil
-          @transport.stub!(:execute_request).and_return('<Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><Upload> <UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_SUBMITTED_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate></Upload></Response>')
+          @transport.stub!(:execute_request).and_return('<?xml version="1.0" encoding="UTF-8"?><Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><Upload> <UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_SUBMITTED_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate></Upload></Response>')
           @feed.upload
-          @feed.id.should == "130895733"
-          @feed.status.should == "_SUBMITTED_"
+          @feed.id.should == '130895733'
+          @feed.status.should == '_SUBMITTED_'
         end
         
-        it "should refresh status" do
-          @transport.stub!(:execute_request).and_return('<Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><Upload> <UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_SUBMITTED_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate></Upload></Response>')
+        it 'should refresh status' do
+          @transport.stub!(:execute_request).and_return('<?xml version="1.0" encoding="UTF-8"?><Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><Upload> <UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_SUBMITTED_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate></Upload></Response>')
           @feed.upload
-          @transport.stub!(:execute_request).and_return('<Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><UploadsStatusList><Upload><UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_IN_PROGRESS_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate><StartedProcessingDate>2007-04-05T00:39:00+0000</StartedProcessingDate></Upload></UploadsStatusList></Response>')
-          @feed.status.should == "_SUBMITTED_"
-          @feed.status!.should == "_IN_PROGRESS_"
-          @transport.stub!(:execute_request).and_return('<Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><UploadsStatusList><Upload><UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_DONE_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate><StartedProcessingDate>2007-04-05T00:39:00+0000</StartedProcessingDate><CompletedProcessingDate>2007-04-05T01:02:00+0000</CompletedProcessingDate><MessagesProcessed>2</MessagesProcessed><MessagesSuccessful>2</MessagesSuccessful><MessagesWithErrors>0</MessagesWithErrors><MessagesWithWarnings>0</MessagesWithWarnings><RelatedDownloadsList><Download><DownloadId>3404021</DownloadId><DownloadType>FeedSummaryReport</DownloadType><RelatedReferenceId>4307285844</RelatedReferenceId><AvailableDate>2009-04-24T23:47:24+00:00</AvailableDate><Acknowledged>FALSE</Acknowledged></Download></RelatedDownloadsList></Upload></UploadsStatusList></Response>')
-          @feed.status!.should == "_DONE_"
-          @feed.download.id.should == "3404021"
-          @feed.download.available_at.to_s.should == "Sat Apr 25 00:47:24 +0100 2009"
+          @transport.stub!(:execute_request).and_return('<?xml version="1.0" encoding="UTF-8"?><Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><UploadsStatusList><Upload><UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_IN_PROGRESS_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate><StartedProcessingDate>2007-04-05T00:39:00+0000</StartedProcessingDate></Upload></UploadsStatusList></Response>')
+          @feed.status.should == '_SUBMITTED_'
+          @feed.status!.should == '_IN_PROGRESS_'
+          @transport.stub!(:execute_request).and_return('<?xml version="1.0" encoding="UTF-8"?><Response xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://merchant-query.amazon.com/query/schema/MerchantQueryResponses.xsd"><UploadsStatusList><Upload><UploadId>130895733</UploadId><UploadType>_POST_FLAT_FILE_FULFILLMENT_DATA_</UploadType><UploadStatus>_DONE_</UploadStatus><SubmittedDate>2007-04-05T00:34:00+0000</SubmittedDate><StartedProcessingDate>2007-04-05T00:39:00+0000</StartedProcessingDate><CompletedProcessingDate>2007-04-05T01:02:00+0000</CompletedProcessingDate><MessagesProcessed>2</MessagesProcessed><MessagesSuccessful>2</MessagesSuccessful><MessagesWithErrors>0</MessagesWithErrors><MessagesWithWarnings>0</MessagesWithWarnings><RelatedDownloadsList><Download><DownloadId>3404021</DownloadId><DownloadType>FeedSummaryReport</DownloadType><RelatedReferenceId>4307285844</RelatedReferenceId><AvailableDate>2009-04-24T23:47:24+00:00</AvailableDate><Acknowledged>FALSE</Acknowledged></Download></RelatedDownloadsList></Upload></UploadsStatusList></Response>')
+          @feed.status!.should == '_DONE_'
+          @feed.download.id.should == '3404021'
+          @feed.download.available_at.should == '2009-04-24T23:47:24+00:00'
         end
       end
     end
@@ -51,27 +51,27 @@ module Peddler
           @transport = Peddler::Transport.new
           @transport.modernize_request
           @feed = Peddler::Feeds::OrderCancellation::Batch.new(@transport)
-          @cancelled_order = Peddler::Feeds::OrderCancellation::Item.new :order_id  => "123-1234567-1234567"
+          @cancelled_order = Peddler::Feeds::OrderCancellation::Item.new :order_id  => '123-1234567-1234567'
         end
         
-        it "should add items to batch" do
+        it 'should add items to batch' do
           @feed.batch.size.should == 0
           @feed << @cancelled_order
           @feed.batch.size.should == 1
         end
         
-        it "should create content for upload" do
+        it 'should create content for upload' do
           @feed << @cancelled_order
           @feed.file_content.should == "TemplateType=OrderCancellation  Version=1.0/1.0.3 This row for Amazon.com use only.  Do not modify or delete.\r\norder-id\tcancellation-reason-code\tamazon-order-item-code\r\n123-1234567-1234567\t\t\r\n"
         end
         
-        it "should raise error if cancellation reason code is given but Amazon order item code is missing" do
-          @cancelled_order.cancellation_reason_code = "BuyerCanceled"
+        it 'should raise error if cancellation reason code is given but Amazon order item code is missing' do
+          @cancelled_order.cancellation_reason_code = 'BuyerCanceled'
           lambda { @cancelled_order.to_s }.should raise_error(PeddlerError)
         end
         
-        it "should raise error if Amazon order item code is given but cancellation reason code is missing" do
-          @cancelled_order.amazon_order_item_code = "111111111111"
+        it 'should raise error if Amazon order item code is given but cancellation reason code is missing' do
+          @cancelled_order.amazon_order_item_code = '111111111111'
           lambda { @cancelled_order.to_s }.should raise_error(PeddlerError)
         end
       end

data/spec/peddler/handlers_spec.rb

@@ -1,18 +1,11 @@
 require File.join(File.dirname(__FILE__), "/../spec_helper") 
+
 module Peddler
-  module Handlers
-    describe XMLHandler do
-      it "should parse XML" do
-        xml = Peddler::Handlers::XMLHandler.decode_response("<Reports> <Report>reportstarttime=08-13-2009:06-16-06 reportendtime=12-31-1969:16-00-00 reportid=2599221990 </Report></Reports>")
-        statuses = Peddler::Handlers::XMLHandler.parse_legacy(xml)
-        statuses[0].id.should == "2599221990"
-      end
-    end
-    
+  module Handlers    
     describe TabDelimitedHandler do
       it "should parse tab-delimited text" do
         text = Peddler::Handlers::TabDelimitedHandler.decode_response("title\tautor\nA Thousand Plateaus\tGilles Deleuze\n")
-        text[0].title.should == "A Thousand Plateaus"
+        text[0].title.should == 'A Thousand Plateaus'
       end
     end
   end

data/spec/peddler/inventory_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), "/../spec_helper") 
+require File.join(File.dirname(__FILE__), '/../spec_helper') 
 
 module Peddler
   
@@ -6,10 +6,10 @@ module Peddler
     
     describe Queue do
 
-      it "should show queue count" do
+      it 'should show queue count' do
         transport = Peddler::Transport.new
         transport.legacize_request
-        transport.stub!(:execute_request).and_return("<PendingUploadsCount>1</PendingUploadsCount>")
+        transport.stub!(:execute_request).and_return('<PendingUploadsCount>1</PendingUploadsCount>')
         Peddler::Inventory::Queue.count(transport).should == 1
       end
 
@@ -21,47 +21,49 @@ module Peddler
         @transport = Peddler::Transport.new
         @transport.legacize_request
         @inventory = Peddler::Inventory::Batch.new(@transport)
-        @item = Peddler::Inventory::Item.new :product_id => "1234567890",
+        @item = Peddler::Inventory::Item.new :product_id => '1234567890',
                                            :price => 100.00,
-                                           :sku => "FOO-SKU",
+                                           :sku => 'FOO-SKU',
                                            :quantity => 10
       end
       
-      it "should add items to batch" do
+      it 'should add items to batch' do
         @inventory.batch.size.should == 0
         @inventory << @item
         @inventory.batch.size.should == 1
       end
     
-      it "should generate an upload file" do
+      it 'should generate an upload file' do
         @inventory << @item
         @inventory.file_content.should == "product-id\tproduct-id-type\titem-condition\tprice\tsku\tquantity\tadd-delete\twill-ship-internationally\texpedited-shipping\titem-note\titem-is-marketplace\tfulfillment-center-id\titem-name\titem-description\tcategory1\timage-url\tshipping-fee\tbrowse-path\tstorefront-feature\tboldface\tasin1\tasin2\tasin3\r\n1234567890\t\t\t100.0\tFOO-SKU\t10\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\t\r\n"
       end
     
-      it "should generate a modify-only upload file" do
+      it 'should generate a modify-only upload file' do
         @inventory << @item
         @inventory.file_content(:short).should == "sku\tprice\tquantity\r\nFOO-SKU\t100.0\t10\r\n"
       end
     
-      it "should generate default headers for uploading" do
-        params = @inventory.defaultize :file_format => "UIEE",
-                                       :enable_expedited_shipping => "N"
+      it 'should generate default headers for uploading' do
+        params = @inventory.send(:defaultize,
+          :file_format => 'UIEE',
+          :enable_expedited_shipping => 'N'
+        )
         params[:method].should be_nil
-        params[:upload_for].should == "Marketplace"
-        params[:email].should == "Y"
-        params[:file_format].should == "UIEE"
-        params["enable-expedited-shipping"].should == "N"
+        params[:upload_for].should == 'Marketplace'
+        params[:email].should == 'Y'
+        params[:file_format].should == 'UIEE'
+        params['enable-expedited-shipping'].should == 'N'
         params[:enable_expedited_shipping].should be_nil
       end
     
-      it "should upload batch" do
-        @transport.stub!(:execute_request).and_return("<BatchID>2585199250</BatchID>")
+      it 'should upload batch' do
+        @transport.stub!(:execute_request).and_return('<BatchID>2585199250</BatchID>')
         @inventory.upload.should == true
-        @inventory.id.should == "2585199250"
+        @inventory.id.should == '2585199250'
       end
       
-      it "should raise error if a subsequent upload is attempted" do
-        @transport.stub!(:execute_request).and_return("<BatchID>2585199250</BatchID>")
+      it 'should raise error if a subsequent upload is attempted' do
+        @transport.stub!(:execute_request).and_return('<BatchID>2585199250</BatchID>')
         @inventory.upload.should == true
         @inventory.id.should_not == nil
         lambda { @inventory.upload }.should raise_error(PeddlerError)

data/spec/peddler/legacy_reports_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.dirname(__FILE__), "/../spec_helper") 
+require File.join(File.dirname(__FILE__), '/../spec_helper') 
 
 module Peddler
   describe LegacyReports do
@@ -7,28 +7,28 @@ module Peddler
       @transport.legacize_request
     end
     
-    it "should generate an order report" do  
-      @transport.stub!(:execute_request).and_return("<Status>SUCCESS</Status>")
+    it 'should generate an order report' do  
+      @transport.stub!(:execute_request).and_return('<Status>SUCCESS</Status>')
       res = Peddler::LegacyReports.generate(@transport, :order, :number_of_days => 30)
       res.should be_an_instance_of(Peddler::LegacyReports::Report)
     end
         
-    it "should show most recent order reports" do
-      @transport.stub!(:execute_request).and_return("<Reports> <Report>reportstarttime=07-23-2009:08-33-48 reportendtime=08-07-2009:08-33-48 reportid=1234567890 </Report></Reports>")
+    it 'should show most recent order reports' do
+      @transport.stub!(:execute_request).and_return('<Reports> <Report>reportstarttime=12-26-2009:19-00-02 reportendtime=01-10-2010:19-00-02 reportid=2893717072 </Report><Report>reportstarttime=01-10-2010:14-00-00 reportendtime=01-10-2010:18-00-00 reportid=2893636754 </Report><Report>reportstarttime=12-26-2009:18-00-01 reportendtime=01-10-2010:18-00-01 reportid=2893629784 </Report><Report>reportstarttime=12-26-2009:17-00-07 reportendtime=01-10-2010:17-00-07 reportid=2893548356 </Report><Report>reportstarttime=12-26-2009:16-00-02 reportendtime=01-10-2010:16-00-02 reportid=2893461740 </Report><Report>reportstarttime=12-26-2009:15-00-02 reportendtime=01-10-2010:15-00-02 reportid=2893375352 </Report><Report>reportstarttime=01-10-2010:10-00-00 reportendtime=01-10-2010:14-00-00 reportid=2893291418 </Report><Report>reportstarttime=12-26-2009:14-00-02 reportendtime=01-10-2010:14-00-02 reportid=2893283538 </Report><Report>reportstarttime=12-26-2009:13-00-02 reportendtime=01-10-2010:13-00-02 reportid=2893197830 </Report><Report>reportstarttime=12-26-2009:12-00-02 reportendtime=01-10-2010:12-00-02 reportid=2893112598 </Report></Reports>')
       report_status = Peddler::LegacyReports.latest(@transport, :order, :count => 1)[0]
-      report_status.starts_at.should == "07-23-2009:08-33-48"
-      report_status.id.should == "1234567890"
+      report_status.starts_at.should == '12-26-2009:19-00-02'
+      report_status.id.should == '2893717072'
     end
     
-    it "should show most recent uploads" do
-      @transport.stub!(:execute_request).and_return("<Batches> <Batch>batchid=2586376232 status=Done dateandtime=08/07/2009 01:48:23 PDT activateditems=1020 numberofwarnings=0 itemsnotacivated=0 </Batch></Batches>")
+    it 'should show most recent uploads' do
+      @transport.stub!(:execute_request).and_return('<Batches> <Batch>batchid=2586376232 status=Done dateandtime=08/07/2009 01:48:23 PDT activateditems=1020 numberofwarnings=0 itemsnotacivated=0 </Batch></Batches>')
       upload = Peddler::LegacyReports.latest(@transport, :upload, :count => 1)[0]
-      upload.id.should == "2586376232"
-      upload.status.should == "Done"
-      upload.datetime.should == "08/07/2009 01:48:23 PDT"
-      upload.activated_items.should == "1020"
-      upload.number_of_warnings.should == "0"
-      upload.items_not_activated.should == "0"
+      upload.id.should == '2586376232'
+      upload.status.should == 'Done'
+      upload.datetime.should == '08/07/2009 01:48:23 PDT'
+      upload.activated_items.should == '1020'
+      upload.number_of_warnings.should == '0'
+      upload.items_not_activated.should == '0'
     end
   end
   
@@ -40,46 +40,46 @@ module Peddler
         @report = Peddler::LegacyReports::Report.new(@transport)
       end
       
-      it "should download an inventory upload error log" do 
+      it 'should download an inventory upload error log' do 
         @transport.stub!(:execute_request).and_return("Feed Processing Summary:\n\tNumber of records processed\t\t82\n\tNumber of records successful\t\t81\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n27\t1111111111\t8026\tError\tSeller is not authorized to list products in this category. For more details, see http://sellercentral.amazon.com/gp/errorcode/8026\n")
         
         @report.name = :upload
-        @report.id = "1234567890"
+        @report.id = '1234567890'
       end
       
-      it "should download an inventory upload error log" do
+      it 'should download an inventory upload error log' do
         @transport.stub!(:execute_request).and_return("Feed Processing Summary:\n\tNumber of records processed\t\t82\n\tNumber of records successful\t\t81\n\noriginal-record-number\tsku\terror-code\terror-type\terror-message\n27\t1111111111\t8026\tError\tSeller is not authorized to list products in this category. For more details, see http://sellercentral.amazon.com/gp/errorcode/8026\n")
         @report.name = :upload
-        @report.id = "1234567890"
+        @report.id = '1234567890'
         @report.body.should =~ /Feed Processing Summary/
       end
       
-      it "should retrieve an order report by id" do
+      it 'should retrieve an order report by id' do
         @transport.stub!(:execute_request).and_return("payments-status\torder-id\torder-item-id\tpayments-date\tpayments-transaction-id\titem-name\tlisting-id\tsku\tprice\tshipping-fee\tquantity-purchased\ttotal-price\tpurchase-date\tbatch-id\tbuyer-email\tbuyer-name\trecipient-name\tship-address-1\tship-address-2\tship-city\tship-state\tship-zip\tship-country\tspecial-comments\tupc\tship-method\n\t001-1234567-1234567\t12345678901234\t2009-07-23 08:59:03 PST\t\tFoo Bar\t000000000000\t1234567890\t10.00\t3.99\t1\t13.99\t2009-07-23 08:59:03 PST\t\tfoo@bar.com\tJOHN DOE\tJohn Doe\t1 MAIN ST\t\tNEW YORK\tNY\t10001-1000\tUS\t\t\tstandard\n")
         @report.name = :order
-        @report.id = "1234567890"
+        @report.id = '1234567890'
         res = Peddler::Handlers::TabDelimitedHandler.decode_response(@report.body)
         res.size.should == 1
-        res[0].order_id.should == "001-1234567-1234567"
-        res[0].buyer_name.should == "JOHN DOE"
+        res[0].order_id.should == '001-1234567-1234567'
+        res[0].buyer_name.should == 'JOHN DOE'
       end
     
-      it "should retrieve an order report by report name" do
+      it 'should retrieve an order report by report name' do
         @transport.stub!(:execute_request).and_return("payments-status\torder-id\torder-item-id\tpayments-date\tpayments-transaction-id\titem-name\tlisting-id\tsku\tprice\tshipping-fee\tquantity-purchased\ttotal-price\tpurchase-date\tbatch-id\tbuyer-email\tbuyer-name\trecipient-name\tship-address-1\tship-address-2\tship-city\tship-state\tship-zip\tship-country\tspecial-comments\tupc\tship-method\n\t001-1234567-1234567\t12345678901234\t2009-07-23 08:59:03 PST\t\tFoo Bar\t000000000000\t1234567890\t10.00\t3.99\t1\t13.99\t2009-07-23 08:59:03 PST\t\tfoo@bar.com\tJOHN DOE\tJohn Doe\t1 MAIN ST\t\tNEW YORK\tNY\t10001-1000\tUS\t\t\tstandard\n")
         @report.name = :order
         res = Peddler::Handlers::TabDelimitedHandler.decode_response(@report.body)
         res.size.should == 1
-        res[0].order_id.should == "001-1234567-1234567"
-        res[0].buyer_name.should == "JOHN DOE"
+        res[0].order_id.should == '001-1234567-1234567'
+        res[0].buyer_name.should == 'JOHN DOE'
       end
       
-      it "should retrieve an open listings liter report by id" do
+      it 'should retrieve an open listings liter report by id' do
         @transport.stub!(:execute_request).and_return("seller-sku\tquantity\nSKU-FOO\t14\n")
         @report.name = :open_listings_liter
-        @report.id = "1234567890"
+        @report.id = '1234567890'
         res = Peddler::Handlers::TabDelimitedHandler.decode_response(@report.body)
         res.size.should == 1
-        res[0].seller_sku.should == "SKU-FOO"
+        res[0].seller_sku.should == 'SKU-FOO'
       end
     end
   end

metadata

@@ -1,26 +1,26 @@
 --- !ruby/object:Gem::Specification 
 name: peddler
 version: !ruby/object:Gem::Version 
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors: 
-- Hakan Senol Ensari
+- "Hakan \xC5\x9Eenol Ensari"
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2009-11-06 00:00:00 +00:00
+date: 2010-01-11 00:00:00 +00:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: xml-simple
+  name: activesupport
   type: :runtime
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 2.3.5
     version: 
 - !ruby/object:Gem::Dependency 
   name: rspec
@@ -30,7 +30,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 1.2.9
     version: 
 description: Peddler is a Ruby wrapper to the Amazon Inventory Management API.
 email: hakan.ensari@papercavalier.com