stripe 1.10.0 → 1.10.1

data/History.txt

@@ -1,6 +1,14 @@
+=== 1.10.1 2014-02-03
+
+* Fix marshaling of StripeObjects
+
 === 1.10.0 2014-01-29
 
 * Support for multiple subscriptions per customer
+* Allow #save to take opts (for :expand)
+* Testing ruby 2.1.0
+* Replace multi_json with json
+* Fix #try and #respond_to? on StripeObjects
 
 === 1.9.9 2013-12-02
 

data/README.rdoc

@@ -15,12 +15,12 @@ If you want to build the gem from source:
 
 * Ruby 1.8.7 or above. (Ruby 1.8.6 may work if you load
   ActiveSupport.)
-* rest-client, multi_json
+* rest-client, json
 
 == Mirrors
 
 The stripe gem is mirrored on Rubygems, so you should be able to
-install it via <tt>gem install stripe</tt> if desired. We recommend using 
+install it via <tt>gem install stripe</tt> if desired. We recommend using
 the https://code.stripe.com mirror so all code is fetched over SSL.
 
 Note that if you are installing via bundler, you should be sure to use the https
@@ -29,7 +29,7 @@ comprimised in transit and alter the code of gems fetched securely over https:
 
   source 'https://code.stripe.com'
   source 'https://rubygems.org'
-  
+
   gem 'rails'
   gem 'stripe'
 

data/VERSION

@@ -1 +1 @@
-1.10.0
+1.10.1

data/lib/stripe.rb

@@ -4,7 +4,7 @@ require 'cgi'
 require 'set'
 require 'openssl'
 require 'rest_client'
-require 'multi_json'
+require 'json'
 
 # Version
 require 'stripe/version'
@@ -17,7 +17,6 @@ require 'stripe/api_operations/list'
 
 # Resources
 require 'stripe/util'
-require 'stripe/json'
 require 'stripe/stripe_object'
 require 'stripe/api_resource'
 require 'stripe/singleton_api_resource'
@@ -180,7 +179,7 @@ module Stripe
     headers[:stripe_version] = api_version if api_version
 
     begin
-      headers.update(:x_stripe_client_user_agent => Stripe::JSON.dump(user_agent))
+      headers.update(:x_stripe_client_user_agent => JSON.generate(user_agent))
     rescue => e
       headers.update(:x_stripe_client_raw_user_agent => user_agent.inspect,
                      :error => "#{e} (#{e.class})")
@@ -195,8 +194,8 @@ module Stripe
     begin
       # Would use :symbolize_names => true, but apparently there is
       # some library out there that makes symbolize_names not work.
-      response = Stripe::JSON.load(response.body)
-    rescue MultiJson::DecodeError
+      response = JSON.parse(response.body)
+    rescue JSON::ParserError
       raise general_api_error(response.code, response.body)
     end
 
@@ -210,11 +209,11 @@ module Stripe
 
   def self.handle_api_error(rcode, rbody)
     begin
-      error_obj = Stripe::JSON.load(rbody)
+      error_obj = JSON.parse(rbody)
       error_obj = Util.symbolize_names(error_obj)
       error = error_obj[:error] or raise StripeError.new # escape from parsing
 
-    rescue MultiJson::DecodeError, StripeError
+    rescue JSON::ParserError, StripeError
       raise general_api_error(rcode, rbody)
     end
 

data/lib/stripe/api_operations/update.rb

@@ -1,8 +1,8 @@
 module Stripe
   module APIOperations
     module Update
-      def save
-        values = serialize_params(self)
+      def save(opts={})
+        values = serialize_params(self).merge(opts)
 
         if @values[:metadata]
           values[:metadata] = serialize_metadata

data/lib/stripe/stripe_object.rb

@@ -36,12 +36,12 @@ module Stripe
     end
 
     def to_s(*args)
-      Stripe::JSON.dump(@values, :pretty => true)
+      JSON.pretty_generate(@values)
     end
 
     def inspect()
       id_string = (self.respond_to?(:id) && !self.id.nil?) ? " id=#{self.id}" : ""
-      "#<#{self.class}:0x#{self.object_id.to_s(16)}#{id_string}> JSON: " + Stripe::JSON.dump(@values, :pretty => true)
+      "#<#{self.class}:0x#{self.object_id.to_s(16)}#{id_string}> JSON: " + JSON.pretty_generate(@values)
     end
 
     def refresh_from(values, api_key, partial=false)
@@ -87,7 +87,7 @@ module Stripe
     end
 
     def to_json(*a)
-      Stripe::JSON.dump(@values)
+      JSON.generate(@values)
     end
 
     def as_json(*a)
@@ -102,6 +102,15 @@ module Stripe
       @values.each(&blk)
     end
 
+    def _dump(level)
+      Marshal.dump([@values, @api_key])
+    end
+
+    def self._load(args)
+      values, api_key = Marshal.load(args)
+      construct_from(values, api_key)
+    end
+
     if RUBY_VERSION < '1.9.2'
       def respond_to?(symbol)
         @values.has_key?(symbol) || super

data/lib/stripe/version.rb

@@ -1,3 +1,3 @@
 module Stripe
-  VERSION = '1.10.0'
+  VERSION = '1.10.1'
 end

data/stripe.gemspec

@@ -14,7 +14,7 @@ spec = Gem::Specification.new do |s|
 
   s.add_dependency('rest-client', '~> 1.4')
   s.add_dependency('mime-types', '~> 1.25')
-  s.add_dependency('multi_json', '>= 1.0.4', '< 2')
+  s.add_dependency('json', '~> 1.8.1')
 
   s.add_development_dependency('mocha', '~> 0.13.2')
   s.add_development_dependency('shoulda', '~> 3.4.0')

data/test/stripe/stripe_object_test.rb

@@ -3,11 +3,18 @@ require File.expand_path('../../test_helper', __FILE__)
 module Stripe
   class StripeObjectTest < Test::Unit::TestCase
     should "implement #respond_to correctly" do
-      obj = Stripe::StripeObject.construct_from({ :some_key => "something", :id => 123 })
-
+      obj = Stripe::StripeObject.construct_from({ :id => 1, :foo => 'bar' })
       assert obj.respond_to?(:id)
-      assert obj.respond_to?(:some_key)
-      assert !obj.respond_to?(:some_other_key)
+      assert obj.respond_to?(:foo)
+      assert !obj.respond_to?(:baz)
+    end
+
+    should "marshal a stripe object correctly" do
+      obj = Stripe::StripeObject.construct_from({ :id => 1, :name => 'Stripe' }, 'apikey')
+      m = Marshal.load(Marshal.dump(obj))
+      assert_equal m.id, 1
+      assert_equal m.name, 'Stripe'
+      assert_equal m.api_key, 'apikey'
     end
   end
 end

data/test/test_helper.rb

@@ -26,7 +26,7 @@ end
 def test_response(body, code=200)
   # When an exception is raised, restclient clobbers method_missing.  Hence we
   # can't just use the stubs interface.
-  body = MultiJson.dump(body) if !(body.kind_of? String)
+  body = JSON.generate(body) if !(body.kind_of? String)
   m = mock
   m.instance_variable_set('@stripe_values', { :body => body, :code => code })
   def m.body; @stripe_values[:body]; end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 1.10.0
+  version: 1.10.1
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-30 00:00:00.000000000 Z
+date: 2014-02-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -45,27 +45,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.25'
 - !ruby/object:Gem::Dependency
-  name: multi_json
+  name: json
   requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.0.4
-    - - <
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '2'
+        version: 1.8.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: 1.0.4
-    - - <
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '2'
+        version: 1.8.1
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
@@ -177,7 +171,6 @@ files:
 - lib/stripe/event.rb
 - lib/stripe/invoice.rb
 - lib/stripe/invoice_item.rb
-- lib/stripe/json.rb
 - lib/stripe/list_object.rb
 - lib/stripe/plan.rb
 - lib/stripe/recipient.rb

data/lib/stripe/json.rb

@@ -1,21 +0,0 @@
-module Stripe
-  module JSON
-    if MultiJson.respond_to?(:dump)
-      def self.dump(*args)
-        MultiJson.dump(*args)
-      end
-
-      def self.load(*args)
-        MultiJson.load(*args)
-      end
-    else
-      def self.dump(*args)
-        MultiJson.encode(*args)
-      end
-
-      def self.load(*args)
-        MultiJson.decode(*args)
-      end
-    end
-  end
-end