jnunemaker-httparty 0.1.1 → 0.1.4

data/History.txt

@@ -1,3 +1,20 @@
+== 0.1.4 2008-11-08
+* 2 major enhancements:
+  * Removed some cruft
+  * Added ability to follow redirects automatically and turn that off (Alex Vollmer)
+
+== 0.1.3 2008-08-22
+
+* 3 major enhancements:
+	* Added http_proxy key for setting proxy server and port (francxk@gmail.com)
+	* Now raises exception when http error occurs (francxk@gmail.com)
+	* Changed auto format detection from file extension to response content type (Jay Pignata)
+
+== 0.1.2 2008-08-09
+
+* 1 major enhancement:
+	* default_params were not being appended to query string if option[:query] was blank
+
 == 0.1.1 2008-07-30
 
 * 2 major enhancement:

data/Manifest.txt

@@ -8,12 +8,11 @@ config/hoe.rb
 config/requirements.rb
 examples/aaws.rb
 examples/delicious.rb
+examples/google.rb
 examples/twitter.rb
 examples/whoismyrep.rb
 httparty.gemspec
 lib/httparty.rb
-lib/httparty/core_ext.rb
-lib/httparty/core_ext/hash.rb
 lib/httparty/version.rb
 script/console
 script/destroy

data/README.txt

@@ -9,7 +9,7 @@ Makes http fun again!
 * Easy get, post, put, delete requests
 * Basic http authentication
 * Default request query string parameters (ie: for api keys that are needed on each request)
-* Automatic parsing of JSON and XML into ruby hashes
+* Automatic parsing of JSON and XML into ruby hashes based on response content-type
 
 == SYNOPSIS:
 
@@ -30,18 +30,33 @@ That works and all but what if you don't want to embed your username and passwor
 	class Twitter
 	  include HTTParty
 	  base_uri 'twitter.com'
-  
-	  def initialize(user, pass)
-	    self.class.basic_auth user, pass
+
+	  def initialize(u, p)
+	    @auth = {:username => u, :password => p}
 	  end
-  
+
 	  def post(text)
-	    self.class.post('/statuses/update.json', :query => {:status => text})
+	    options = { :query => {:status => text}, :basic_auth => @auth }
+	    self.class.post('/statuses/update.json', options)
 	  end
 	end
 	
 	Twitter.new('username', 'password').post("It's an HTTParty and everyone is invited!")
 
+=== REQUEST OPTIONS
+
+Each of the HTTP method (get, post, put and delete) each take a hash of options.
+The following keys can be specified in the options:
+
+headers::       A <tt>Hash</tt> of header key/value pairs
+query::         A <tt>Hash</tt> of query key/value pairs
+body::          The body of the request. If it's a <tt>Hash</tt>, it is
+                converted into query-string format, otherwise it is sent
+                as-is.
+basic_auth::    A <tt>Hash</tt> containing keys for <tt>:username</tt> and
+                <tt>:password</tt>.
+no_follow::     Turns off automatic redirect following
+
 == REQUIREMENTS:
 
 * Active Support >= 2.1

data/Rakefile

@@ -1,4 +1,11 @@
 require 'config/requirements'
 require 'config/hoe' # setup Hoe + all gem configuration
+require "spec/rake/spectask"
 
-Dir['tasks/**/*.rake'].each { |rake| load rake }
+Dir['tasks/**/*.rake'].each { |rake| load rake }
+
+task :default => :spec
+
+Spec::Rake::SpecTask.new do |t|
+  t.spec_files = FileList["spec/**/*_spec.rb"]
+end

data/examples/aaws.rb

@@ -8,7 +8,6 @@ module AAWS
     include HTTParty
     base_uri 'http://ecs.amazonaws.com'
     default_params :Service => 'AWSECommerceService', :Operation => 'ItemSearch', :SearchIndex => 'Books'
-    format :xml
     
     def initialize(key)
       self.class.default_params :AWSAccessKeyId => key

data/examples/delicious.rb

@@ -6,7 +6,6 @@ config = YAML::load(File.read(File.join(ENV['HOME'], '.delicious')))
 class Delicious
   include HTTParty
   base_uri 'https://api.del.icio.us/v1'
-  format :xml
   
   def initialize(u, p)
     @auth = {:username => u, :password => p}

data/examples/google.rb

@@ -0,0 +1,15 @@
+dir = File.expand_path(File.join(File.dirname(__FILE__), '..', 'lib'))
+require File.join(dir, 'httparty')
+require 'pp'
+
+class Google
+  include HTTParty
+end
+
+# google.com redirects to www.google.com so this is live test for redirection
+pp Google.get('http://google.com')
+
+puts '', '*'*70, ''
+
+# check that ssl is requesting right
+pp Google.get('https://www.google.com')

data/examples/whoismyrep.rb

@@ -4,7 +4,6 @@ require 'pp'
 
 class Rep
   include HTTParty
-  format :xml
 end
 
 puts Rep.get('http://whoismyrepresentative.com/whoismyrep.php?zip=46544').inspect

data/httparty.gemspec

@@ -1,14 +1,14 @@
 Gem::Specification.new do |s|
   s.name = %q{httparty}
-  s.version = "0.1.1"
+  s.version = "0.1.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["John Nunemaker"]
-  s.date = %q{2008-07-31}
+  s.date = %q{2008-11-08}
   s.description = %q{Makes http fun! Also, makes consuming restful web services dead easy.}
   s.email = ["nunemaker@gmail.com"]
   s.extra_rdoc_files = ["History.txt", "License.txt", "Manifest.txt", "PostInstall.txt", "README.txt"]
-  s.files = ["History.txt", "License.txt", "Manifest.txt", "PostInstall.txt", "README.txt", "Rakefile", "config/hoe.rb", "config/requirements.rb", "examples/aaws.rb", "examples/delicious.rb", "examples/twitter.rb", "examples/whoismyrep.rb", "httparty.gemspec", "lib/httparty.rb", "lib/httparty/core_ext.rb", "lib/httparty/core_ext/hash.rb", "lib/httparty/version.rb", "script/console", "script/destroy", "script/generate", "script/txt2html", "setup.rb", "spec/hash_spec.rb", "spec/httparty_spec.rb", "spec/spec.opts", "spec/spec_helper.rb", "tasks/deployment.rake", "tasks/environment.rake", "tasks/website.rake", "website/css/common.css", "website/index.html"]
+  s.files = ["History.txt", "License.txt", "Manifest.txt", "PostInstall.txt", "README.txt", "Rakefile", "config/hoe.rb", "config/requirements.rb", "examples/aaws.rb", "examples/delicious.rb", "examples/google.rb", "examples/twitter.rb", "examples/whoismyrep.rb", "httparty.gemspec", "lib/httparty.rb", "lib/httparty/version.rb", "script/console", "script/destroy", "script/generate", "script/txt2html", "setup.rb", "spec/hash_spec.rb", "spec/httparty_spec.rb", "spec/spec.opts", "spec/spec_helper.rb", "tasks/deployment.rake", "tasks/environment.rake", "tasks/website.rake", "website/css/common.css", "website/index.html"]
   s.has_rdoc = true
   s.homepage = %q{http://httparty.rubyforge.org}
   s.post_install_message = %q{When you HTTParty, you must party hard!}
@@ -24,10 +24,13 @@ Gem::Specification.new do |s|
 
     if current_version >= 3 then
       s.add_runtime_dependency(%q<activesupport>, [">= 2.1"])
+      s.add_development_dependency(%q<hoe>, [">= 1.8.0"])
     else
       s.add_dependency(%q<activesupport>, [">= 2.1"])
+      s.add_dependency(%q<hoe>, [">= 1.8.0"])
     end
   else
     s.add_dependency(%q<activesupport>, [">= 2.1"])
+    s.add_dependency(%q<hoe>, [">= 1.8.0"])
   end
-end
+end

data/lib/httparty/version.rb

@@ -2,7 +2,7 @@ module HTTParty
   module VERSION #:nodoc:
     MAJOR = 0
     MINOR = 1
-    TINY  = 1
+    TINY  = 4
 
     STRING = [MAJOR, MINOR, TINY].join('.')
   end

data/lib/httparty.rb

@@ -5,26 +5,34 @@ require 'ostruct'
 require 'rubygems'
 require 'active_support'
 
-$:.unshift(File.dirname(__FILE__)) unless
-  $:.include?(File.dirname(__FILE__)) || $:.include?(File.expand_path(File.dirname(__FILE__)))
+directory = File.dirname(__FILE__)
+$:.unshift(directory) unless $:.include?(directory) || $:.include?(File.expand_path(directory))
 
-dir = File.expand_path(File.join(File.dirname(__FILE__), 'httparty'))
-require dir + '/core_ext'
-  
 module HTTParty
+  class UnsupportedFormat < StandardError; end
+  class RedirectionTooDeep < StandardError; end
+  
   def self.included(base)
     base.extend ClassMethods
   end
   
-  class UnsupportedFormat < StandardError; end
-  
-  AllowedFormats = %w[xml json]
+  AllowedFormats = {:xml => 'text/xml', :json => 'application/json'}
   
   module ClassMethods    
+    #
+    # Set an http proxy
+    #
+    #	class Twitter
+    #	  include HTTParty
+    #	  http_proxy http://myProxy, 1080
+    # ....
+    def http_proxy(addr=nil, port = nil)
+	   @http_proxyaddr = addr
+	   @http_proxyport = port
+    end
+
     def base_uri(base_uri=nil)
       return @base_uri unless base_uri
-      # don't want this to ever end with /
-      base_uri = base_uri.ends_with?('/') ? base_uri.chop : base_uri
       @base_uri = normalize_base_uri(base_uri)
     end
     
@@ -52,8 +60,7 @@ module HTTParty
     end
     
     def format(f)
-      f = f.to_s
-      raise UnsupportedFormat, "Must be one of: #{AllowedFormats.join(', ')}" unless AllowedFormats.include?(f)
+      raise UnsupportedFormat, "Must be one of: #{AllowedFormats.keys.join(', ')}" unless AllowedFormats.key?(f)
       @format = f
     end
     
@@ -78,14 +85,11 @@ module HTTParty
     end
     
     private
-      def http(uri)
-        if @http.blank?
-          @http = Net::HTTP.new(uri.host, uri.port)
-          @http.use_ssl = (uri.port == 443)
-          # so we can avoid ssl warnings
-          @http.verify_mode = OpenSSL::SSL::VERIFY_NONE
-        end
-        @http
+      def http(uri) #:nodoc:
+        http = Net::HTTP.new(uri.host, uri.port, @http_proxyaddr, @http_proxyport)
+        http.use_ssl = (uri.port == 443)
+        http.verify_mode = OpenSSL::SSL::VERIFY_NONE
+        http
       end
       
       # FIXME: this method is doing way to much and needs to be split up
@@ -94,55 +98,72 @@ module HTTParty
       #   body        => hash of keys/values or a query string (foo=bar&baz=poo)
       #   headers     => hash of headers to send request with
       #   basic_auth  => :username and :password to use as basic http authentication (overrides @auth class instance variable)
-      def send_request(method, path, options={})
+      # Raises exception Net::XXX (http error code) if an http error occured
+      def send_request(method, path, options={}) #:nodoc:
+        options = {:limit => 5}.merge(options)
+        options[:limit] = 0 if options.delete(:no_follow)
+        
+        raise HTTParty::RedirectionTooDeep, 'HTTP redirects too deep' if options[:limit].to_i <= 0
         raise ArgumentError, 'only get, post, put and delete methods are supported' unless %w[get post put delete].include?(method.to_s)
         raise ArgumentError, ':headers must be a hash' if options[:headers] && !options[:headers].is_a?(Hash)
         raise ArgumentError, ':basic_auth must be a hash' if options[:basic_auth] && !options[:basic_auth].is_a?(Hash)
-        # we always want path that begins with /
-        path           = path =~ /^(\/|https?:\/\/)/ ? path : "/#{path}"
-        @format      ||= format_from_path(path)
-        uri            = URI.parse("#{base_uri}#{path}")
+        
+        path           = URI.parse(path)
+        uri            = path.relative? ? URI.parse("#{base_uri}#{path}") : path
         existing_query = uri.query ? "#{uri.query}&" : ''
         uri.query      = if options[:query].blank?
-          existing_query
+          existing_query + default_params.to_query
         else
           existing_query + (options[:query].is_a?(Hash) ? default_params.merge(options[:query]).to_query : options[:query])
         end
+        
         klass          = Net::HTTP.const_get method.to_s.downcase.capitalize
         request        = klass.new(uri.request_uri)
         request.body   = options[:body].is_a?(Hash) ? options[:body].to_query : options[:body] unless options[:body].blank?
         basic_auth     = options.delete(:basic_auth) || @auth
         request.initialize_http_header headers.merge(options[:headers] || {})
-        # note to self: self, do not put basic auth above headers because it removes basic auth
         request.basic_auth(basic_auth[:username], basic_auth[:password]) if basic_auth
         response       = http(uri).request(request)
-        parse_response(response.body)
+        @format      ||= format_from_mimetype(response['content-type'])
+        
+        case response
+        when Net::HTTPSuccess
+          parse_response(response.body)
+        when Net::HTTPRedirection
+          options[:limit] -= 1
+          send_request(method, response['location'], options)
+        else
+          response.instance_eval { class << self; attr_accessor :body_parsed; end }
+          begin; response.body_parsed = parse_response(response.body); rescue; end
+          response.error! # raises  exception corresponding to http error Net::XXX
+        end
+
       end
       
-      def parse_response(body)
+      def parse_response(body) #:nodoc:
+        return nil if body.nil? or body.empty?
         case @format
-        when 'xml'
+        when :xml
           Hash.from_xml(body)
-        when 'json'
+        when :json
           ActiveSupport::JSON.decode(body)
         else
-          # just return the response if no format 
           body
         end
       end
     
       # Makes it so uri is sure to parse stuff like google.com with the http
-      def normalize_base_uri(str)
-        str =~ /^https?:\/\// ? str : "http#{'s' if str.include?(':443')}://#{str}"
+      def normalize_base_uri(url) #:nodoc:
+        use_ssl = (url =~ /^https/) || url.include?(':443')
+        url.chop! if url.ends_with?('/')
+        url.gsub!(/^https?:\/\//i, '')
+        "http#{'s' if use_ssl}://#{url}"
       end
       
-      # Returns a format that we can handle from the path if possible. 
-      # Just does simple pattern matching on file extention:
-      #   /foobar.xml => 'xml'
-      #   /foobar.json => 'json'
-      def format_from_path(path)
-        ext = File.extname(path)[1..-1]
-        !ext.blank? && AllowedFormats.include?(ext) ? ext : nil
+      # Uses the HTTP Content-Type header to determine the format of the response
+      # It compares the MIME type returned to the types stored in the AllowedFormats hash
+      def format_from_mimetype(mimetype) #:nodoc:
+        AllowedFormats.each { |k, v| return k if mimetype.include?(v) }
       end
   end
 end

data/spec/httparty_spec.rb

@@ -13,11 +13,11 @@ end
 describe HTTParty do
   
   describe "base uri" do
-    it "should be gettable" do
+    it "should have reader" do
       Foo.base_uri.should == 'http://api.foo.com/v1'
     end
     
-    it 'should be setable' do
+    it 'should have writer' do
       Foo.base_uri('http://api.foobar.com')
       Foo.base_uri.should == 'http://api.foobar.com'
     end
@@ -28,7 +28,13 @@ describe HTTParty do
     end
     
     it "should add https if not present for ssl requests" do
-      FooWithHttps.base_uri.should == 'https://api.foo.com/v1:443'
+      Foo.base_uri('api.foo.com/v1:443')
+      Foo.base_uri.should == 'https://api.foo.com/v1:443'
+    end
+    
+    it "should not remove https for ssl requests" do
+      Foo.base_uri('https://api.foo.com/v1:443')
+      Foo.base_uri.should == 'https://api.foo.com/v1:443'
     end
   end
   
@@ -66,12 +72,12 @@ describe HTTParty do
   describe "format" do
     it "should allow xml" do
       Foo.format :xml
-      Foo.instance_variable_get("@format").should == 'xml'
+      Foo.instance_variable_get("@format").should == :xml
     end
     
     it "should allow json" do
       Foo.format :json
-      Foo.instance_variable_get("@format").should == 'json'
+      Foo.instance_variable_get("@format").should == :json
     end
     
     it 'should not allow funky format' do
@@ -91,24 +97,6 @@ describe HTTParty do
     end
   end
   
-  describe "deriving format from path" do
-    it "should work if there is extension and extension is an allowed format" do
-      %w[xml json].each do |ext|
-        Foo.send(:format_from_path, "/foo/bar.#{ext}").should == ext
-      end
-    end
-    
-    it "should NOT work if there is extension but extention is not allow format" do
-      Foo.send(:format_from_path, '/foo/bar.php').should == nil
-    end
-    
-    it 'should NOT work if there is no extension' do
-      ['', '.'].each do |ext|
-        Foo.send(:format_from_path, "/foo/bar#{ext}").should == nil
-      end
-    end
-  end
-  
   describe 'parsing responses' do
     it 'should handle xml automatically' do
       xml = %q[<books><book><id>1234</id><name>Foo Bar!</name></book></books>]
@@ -141,5 +129,93 @@ describe HTTParty do
         Foo.send(:send_request, 'get', '/foo', :basic_auth => 'string')
       end.should raise_error(ArgumentError)
     end
+
+    it "should not attempt to parse empty responses" do
+      http = Net::HTTP.new('localhost', 80)
+      Foo.stub!(:http).and_return(http)
+      response = Net::HTTPNoContent.new("1.1", 204, "No content for you")
+      response.stub!(:body).and_return(nil)
+      http.stub!(:request).and_return(response)
+
+      Foo.headers.clear # clear out bogus settings from other specs
+      Foo.format :xml
+      Foo.send(:send_request, 'get', '/bar').should be_nil
+
+      response.stub!(:body).and_return("")
+      Foo.send(:send_request, 'get', 'bar').should be_nil
+    end
+
+    describe "that respond with redirects" do
+      def setup_http
+        @http = Net::HTTP.new('localhost', 80)
+        Foo.stub!(:http).and_return(@http)
+        @redirect = Net::HTTPFound.new("1.1", 302, "")
+        @redirect.stub!(:[]).with('location').and_return('/foo')
+        @ok = Net::HTTPOK.new("1.1", 200, "Content for you")
+        @ok.stub!(:body).and_return({"foo" => "bar"}.to_xml)
+        @http.should_receive(:request).and_return(@redirect, @ok)
+        Foo.headers.clear
+        Foo.format :xml
+      end
+
+      it "should handle redirects for GET transparently" do
+        setup_http
+        Foo.get('/foo/').should == {"hash" => {"foo" => "bar"}}
+      end
+
+      it "should handle redirects for POST transparently" do
+        setup_http
+        Foo.post('/foo/', {:foo => :bar}).should == {"hash" => {"foo" => "bar"}}
+      end
+
+      it "should handle redirects for DELETE transparently" do
+        setup_http
+        Foo.delete('/foo/').should == {"hash" => {"foo" => "bar"}}
+      end
+
+      it "should handle redirects for PUT transparently" do
+        setup_http
+        Foo.put('/foo/').should == {"hash" => {"foo" => "bar"}}
+      end
+
+      it "should prevent infinite loops" do
+        http = Net::HTTP.new('localhost', 80)
+        Foo.stub!(:http).and_return(http)
+        redirect = Net::HTTPFound.new("1.1", "302", "Look, over there!")
+        redirect.stub!(:[]).with('location').and_return('/foo')
+        http.stub!(:request).and_return(redirect)
+
+        lambda do
+          Foo.send(:send_request, 'get', '/foo')
+        end.should raise_error(HTTParty::RedirectionTooDeep)
+      end
+
+      describe "with explicit override of automatic redirect handling" do
+
+        it "should fail with redirected GET" do
+          lambda do
+            Foo.get('/foo', :no_follow => true)
+          end.should raise_error(HTTParty::RedirectionTooDeep)
+        end
+
+        it "should fail with redirected POST" do
+          lambda do
+            Foo.post('/foo', :no_follow => true)
+          end.should raise_error(HTTParty::RedirectionTooDeep)
+        end
+
+        it "should fail with redirected DELETE" do
+          lambda do
+            Foo.delete('/foo', :no_follow => true)
+          end
+        end
+
+        it "should fail with redirected PUT" do
+          lambda do
+            Foo.put('/foo', :no_follow => true)
+          end
+        end
+      end
+    end
   end
 end

data/tasks/deployment.rake

@@ -1,3 +1,12 @@
+desc 'Preps the gem for a new release'
+task :prep_for_release do
+  require 'rio'
+  Rake::Task['manifest:refresh'].invoke
+  gemspec = %x[rake debug_gem]
+  lines = gemspec.split("\n")
+  rio('httparty.gemspec') < lines[1, lines.length-1].join("\n")
+end
+
 desc 'Release the website and new gem version'
 task :deploy => [:check_version, :website, :release] do
   puts "Remember to create SVN tag:"

data/tasks/website.rake

@@ -14,4 +14,4 @@ task :website_upload do
 end
 
 desc 'Generate and upload website files'
-task :website => [:website_generate, :website_upload, :publish_docs]
+task :website => [:website_upload, :publish_docs]

data/website/index.html

@@ -15,6 +15,7 @@
 		<ul id="nav">
 			<li><a href="rdoc/">Docs</a></li> 
 			<li><a href="http://github.com/jnunemaker/httparty">Github</a></li>
+			<li><a href="http://jnunemaker.lighthouseapp.com/projects/14842-httparty/tickets">Lighthouse</a></li>
 			<li><a href="http://rubyforge.org/projects/httparty/">Rubyforge</a></li> 
 		</ul> 
 	</div> 
@@ -43,12 +44,13 @@ Twitter.post('/statuses/update.json', :query => {:status => "It's an HTTParty an
   include HTTParty
   base_uri 'twitter.com'
 
-  def initialize(user, pass)
-    self.class.basic_auth user, pass
+  def initialize(u, p)
+    @auth = {:username => u, :password => p}
   end
 
   def post(text)
-    self.class.post('/statuses/update.json', :query => {:status => text})
+    options = { :query => {:status => text}, :basic_auth => @auth }
+    self.class.post('/statuses/update.json', options)
   end
 end
 
@@ -66,12 +68,16 @@ Twitter.new('username', 'password').post("It's an HTTParty and everyone is invit
 		<p>Created by <a href="http://addictedtonew.com/about/">John Nunemaker</a></p> 
 	</div> 
 </div> 
+
+<script type="text/javascript">
+var gaJsHost = (("https:" == document.location.protocol) ? "https://ssl." : "http://www.");
+document.write(unescape("%3Cscript src='" + gaJsHost + "google-analytics.com/ga.js' type='text/javascript'%3E%3C/script%3E"));
+</script>
+<script type="text/javascript">
+var pageTracker = _gat._getTracker("UA-85301-19");
+pageTracker._initData();
+pageTracker._trackPageview();
+</script>
  
-<script src="http://www.google-analytics.com/urchin.js" type="text/javascript"></script> 
-<script type="text/javascript">_uacct = "UA-85301-9"; urchinTracker();</script> 
- 
-<!-- 103bees.com 'bee' code v1.11 - please do not make any changes! --> 
-<script type="text/javascript" src="http://103bees.com/bees/?bee=3672&amp;fid=5643"></script> 
-<!-- 103bees.com 'bee' code --> 
 </body> 
 </html>

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: jnunemaker-httparty
 version: !ruby/object:Gem::Version 
-  version: 0.1.1
+  version: 0.1.4
 platform: ruby
 authors: 
 - John Nunemaker
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-07-31 00:00:00 -07:00
+date: 2008-11-08 00:00:00 -08:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -21,6 +21,15 @@ dependencies:
       - !ruby/object:Gem::Version 
         version: "2.1"
     version: 
+- !ruby/object:Gem::Dependency 
+  name: hoe
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: 1.8.0
+    version: 
 description: Makes http fun! Also, makes consuming restful web services dead easy.
 email: 
 - nunemaker@gmail.com
@@ -45,12 +54,11 @@ files:
 - config/requirements.rb
 - examples/aaws.rb
 - examples/delicious.rb
+- examples/google.rb
 - examples/twitter.rb
 - examples/whoismyrep.rb
 - httparty.gemspec
 - lib/httparty.rb
-- lib/httparty/core_ext.rb
-- lib/httparty/core_ext/hash.rb
 - lib/httparty/version.rb
 - script/console
 - script/destroy

data/lib/httparty/core_ext/hash.rb

@@ -1,21 +0,0 @@
-module HTTParty
-  module CoreExt
-    module HashConversions
-      def to_struct
-        o = OpenStruct.new
-        self.each do |k, v|
-          # if id, we create an accessor so we don't get warning about id deprecation
-          if k.to_s == 'id'
-            o.class.class_eval "attr_accessor :id"
-            o.id = v
-          else
-            o.send("#{k}=", v.is_a?(Hash) ? v.to_struct : v)
-          end
-        end
-        o
-      end
-    end
-  end
-end
-
-Hash.send :include, HTTParty::CoreExt::HashConversions

data/lib/httparty/core_ext.rb

@@ -1,2 +0,0 @@
-dir = File.expand_path(File.join(File.dirname(__FILE__), 'core_ext'))
-require File.join(dir, 'hash')

data/spec/hash_spec.rb

@@ -1,11 +0,0 @@
-require File.join(File.dirname(__FILE__), 'spec_helper')
-
-describe HTTParty::CoreExt::HashConversions do
-  it "should convert hash to struct" do
-    {'foo' => 'bar'}.to_struct.should == OpenStruct.new(:foo => 'bar')
-  end
-  
-  it 'should convert nested hash to struct' do
-    {'foo' => {'bar' => 'baz'}}.to_struct.should == OpenStruct.new(:foo => OpenStruct.new(:bar => 'baz'))
-  end
-end