stripe 3.0.0 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 390fa649e902fb6c709e2d803c92ffbf136a2384
-  data.tar.gz: 837bf7e6916a55a1feeaffcd41c4e713eb5ce92f
+  metadata.gz: 79dc89b3485d9c9077874530b9483ba3a048b1d8
+  data.tar.gz: a73e14de59f52f2282196f1a30537a7d6c353497
 SHA512:
-  metadata.gz: 635ac7d16f11b9ab531faf3f26f1d35e68947e18b92e5680656ce0963f99c910f5d8383f784895ae205a060740ac1540ab5b932da6ad2dc46cd99f8cb8c53163
-  data.tar.gz: 47eb2e2fcaf5cb7637456663229c7a28349eac275518d3749967d33c3025ac9a4f4df1e8eea570e7808b1def4ce89045b941e3a910ab64813128483b7677a4ef
+  metadata.gz: 59133df8f5c29d44597f9a8145c77f0becf8705fe1b507b4612fb57930eb0b09afa2ef05e1ebe10c52e0749484bbcc957dbf998a2eb50df3d3d9879a9bcb2004
+  data.tar.gz: e5b4737ca902571ed0866df6b1401c5c3a2823d70e8a8da224dc5f2266b50cb454cffade965b5746098ce07e8ce0dd6eeedfe6c0b8c444a275d935eb9e008a42

data/History.txt

@@ -1,3 +1,8 @@
+=== 3.0.1 2017-07-11
+
+* Properties set with an API resource will now serialize that resource's ID if possible
+* API resources will throw an ArgumentError on save if a property has been with an API resource that cannot be serialized
+
 === 3.0.0 2017-06-27
 
 * `#pay` on invoice now takes params as well as opts

data/VERSION

@@ -1 +1 @@
-3.0.0
+3.0.1

data/lib/stripe/stripe_object.rb

@@ -176,7 +176,7 @@ module Stripe
         unsaved = @unsaved_values.include?(k)
         if options[:force] || unsaved || v.is_a?(StripeObject)
           update_hash[k.to_sym] =
-            serialize_params_value(@values[k], @original_values[k], unsaved, options[:force])
+            serialize_params_value(@values[k], @original_values[k], unsaved, options[:force], key: k)
         end
       end
 
@@ -337,7 +337,7 @@ module Stripe
       self
     end
 
-    def serialize_params_value(value, original, unsaved, force)
+    def serialize_params_value(value, original, unsaved, force, key: nil)
       case true
       when value == nil
         ''
@@ -348,13 +348,32 @@ module Stripe
       # be updated from their proper endpoints, and therefore they are not
       # included when serializing even if they've been modified.
       #
-      # There are _some_ known exceptions though. For example, to save on API
-      # calls it's sometimes desirable to update a customer's default source by
-      # setting a new card (or other) object with `#source=` and then saving
-      # the customer. The `#save_with_parent` flag to override the default
-      # behavior allows us to handle these exceptions.
+      # There are _some_ known exceptions though.
+      #
+      # For example, if the value is unsaved (meaning the user has set it), and
+      # it looks like the API resource is persisted with an ID, then we include
+      # the object so that parameters are serialized with a reference to its
+      # ID.
+      #
+      # Another example is that on save API calls it's sometimes desirable to
+      # update a customer's default source by setting a new card (or other)
+      # object with `#source=` and then saving the customer. The
+      # `#save_with_parent` flag to override the default behavior allows us to
+      # handle these exceptions.
+      #
+      # We throw an error if a property was set explicitly but we can't do
+      # anything with it because the integration is probably not working as the
+      # user intended it to.
       when value.is_a?(APIResource) && !value.save_with_parent
-        nil
+        if !unsaved
+          nil
+        elsif value.respond_to?(:id) && value.id != nil
+          value
+        else
+          raise ArgumentError, "Cannot save property `#{key}` containing " \
+            "an API resource. It doesn't appear to be persisted and is " \
+            "not marked as `save_with_parent`."
+        end
 
       when value.is_a?(Array)
         update = value.map { |v| serialize_params_value(v, nil, true, force) }

data/lib/stripe/version.rb

@@ -1,3 +1,3 @@
 module Stripe
-  VERSION = '3.0.0'
+  VERSION = '3.0.1'
 end

data/test/stripe/stripe_object_test.rb

@@ -266,16 +266,29 @@ module Stripe
       assert_equal([{ :foo => "bar" }], serialized[:metadata])
     end
 
-    should "#serialize_params and remove embedded APIResources" do
+    should "#serialize_params and embed an API resource that's been set and has an ID" do
+      customer = Customer.construct_from({ :id => "cus_123" })
+      obj = Stripe::StripeObject.construct_from({})
+
+      # the key here is that the property is set explicitly (and therefore
+      # marked as unsaved), which is why it gets included below
+      obj.customer = customer
+
+      serialized = obj.serialize_params
+      assert_equal({ :customer => customer }, serialized)
+    end
+
+    should "#serialize_params and not include API resources that have not been set" do
+      customer = Customer.construct_from({ :id => "cus_123" })
       obj = Stripe::StripeObject.construct_from({
-        :customer => Customer.construct_from({})
+        :customer => customer
       })
 
       serialized = obj.serialize_params
       assert_equal({}, serialized)
     end
 
-    should "#serialize_params and remove embedded APIResources unless flagged with save_with_parent" do
+    should "#serialize_params serializes API resources flagged with save_with_parent" do
       c = Customer.construct_from({})
       c.save_with_parent = true
 
@@ -287,6 +300,23 @@ module Stripe
       assert_equal({ :customer => {} }, serialized)
     end
 
+    should "#serialize_params should raise an error on other embedded API resources" do
+      # This customer doesn't have an ID and therefore the library doesn't know
+      # what to do with it and throws an ArgumentError because it's probably
+      # not what the user expected to happen.
+      customer = Customer.construct_from({})
+
+      obj = Stripe::StripeObject.construct_from({})
+      obj.customer = customer
+
+      e = assert_raises(ArgumentError) do
+        obj.serialize_params
+      end
+      assert_equal "Cannot save property `customer` containing " \
+        "an API resource. It doesn't appear to be persisted and is " \
+        "not marked as `save_with_parent`.", e.message
+    end
+
     should "#serialize_params takes a force option" do
       obj = Stripe::StripeObject.construct_from({
         :id => 'id',

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: stripe
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.0.1
 platform: ruby
 authors:
 - Stripe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-06-27 00:00:00.000000000 Z
+date: 2017-07-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -233,3 +233,4 @@ test_files:
 - test/stripe_test.rb
 - test/test_data.rb
 - test/test_helper.rb
+has_rdoc: