sucker 1.3.0.pre.3 → 1.3.0

data/README.md

@@ -3,17 +3,9 @@ Sucker
 
 Sucker is a Nokogiri-based Ruby wrapper to the [Amazon Product Advertising API](https://affiliate-program.amazon.co.uk/gp/advertising/api/detail/main.html).
 
-It's minimalist and fast. It supports __the entire API__.
+Sucker is fast and supports __the entire Amazon API__.
 
-![Electrolux](https://github.com/papercavalier/sucker/raw/master/electrolux.jpg)
-
-Trim fat
---------
-1.3.0.pre has major changes under the hood.
-
-Active Support and Curb are no more.
-
-I edited out some nonessential methods. Check [here](http://rdoc.info/github/papercavalier/sucker/master/frames) to see what's left.
+![Hoover](https://github.com/papercavalier/sucker/raw/master/hoover.jpg)
 
 Usage
 -----
@@ -28,32 +20,29 @@ Set up.
 Build a request.
 
     worker << {
-      "Operation"     => 'ItemLookup',
-      "IdType"        => 'ASIN',
-      "ItemId"        => '0816614024',
-      "ResponseGroup" => 'ItemAttributes' }
+      'Operation'     => 'ItemLookup',
+      'IdType'        => 'ASIN',
+      'ItemId'        => '0816614024',
+      'ResponseGroup' => 'ItemAttributes' }
 
-Literally, get a response.
+Get a response.
 
     response = worker.get
 
-Time for some business logic.
+Do something with it.
 
-    items = response['Item'] if response.valid?
+    items = response['Item'] if response.valid? # and so on
 
 Repeat ad infinitum.
 
-[Check the features](http://relishapp.com/papercavalier/sucker).
-
-[Read the API.](https://affiliate-program.amazon.co.uk/gp/advertising/api/detail/main.html)
+Read further [here](http://rdoc.info/github/papercavalier/sucker/master/frames) and [here](http://relishapp.com/papercavalier/sucker).
 
-
-Monkey-patch that Net::HTTP
----------------------------
+Multiple IPs
+------------
 
 Amazon limits calls to a venue to one per second per IP address.
 
-If your server has multiple local interfaces, do the following:
+If your server has multiple local interfaces, use them simultaneously like so:
 
     your_ips.each do |ip|
       Thread.new do
@@ -62,36 +51,17 @@ If your server has multiple local interfaces, do the following:
       end
     end
 
-Throttle calls
-----------------
-
-Use [Throttler](https://github.com/papercavalier/throttler) to throttle calls to one per second per IP address. Let me know if you figure out a more elegant solution.
-
-More concise syntax
--------------------
-
-If you are on Ruby 1.9, do:
-
-    worker << {
-      operation: 'ItemLookup',
-      id_type:   'ASIN',
-      item_id:   '0816614024' }
-
-Stub
-----
+Stubbing in Tests
+-----------------
 
 Use [VCR](http://github.com/myronmarston/vcr).
 
-Check out [this](http://github.com/papercavalier/sucker/blob/master/spec/support/vcr.rb) and [this](https://github.com/papercavalier/sucker/blob/master/features/step_definitions/sucker_steps.rb).
-
 Compatibility
 -------------
 
 Specs pass against Ruby 1.8.7, Ruby 1.9.2, JRuby 1.5.6, and Rubinius 1.2.1.
 
-Morale(s) of the story
--------------------
+Moral of the story
+------------------
 
 Don't overabstract a spaghetti API.
-
-Fancy a DSL? Write your own on top of this.

data/lib/sucker/hash.rb

@@ -0,0 +1,44 @@
+module Sucker
+  class Hash < ::Hash
+    class << self
+
+      # Based on https://gist.github.com/335286
+      def from_xml(xml)
+        case xml
+        when Nokogiri::XML::Document
+          new[xml.root.name] = from_xml(xml.root)
+        when Nokogiri::XML::Element
+          result_hash = new
+
+          xml.attributes.each_pair do |key, attribute|
+            result_hash[key] = attribute.value
+          end
+
+          xml.children.each do |child|
+            result = from_xml(child)
+
+            if child.name == 'text'
+              if result_hash.empty?
+                return result
+              else
+                result_hash['__content__'] = result
+              end
+            elsif result_hash[child.name]
+              if result_hash[child.name].is_a? Array
+                result_hash[child.name] << result
+              else
+                result_hash[child.name] = [result_hash[child.name]] << result
+              end
+            else
+              result_hash[child.name] = result
+            end
+          end
+
+          result_hash
+        else
+          xml.content.to_s
+        end
+      end
+    end
+  end
+end