Some safe transformations from codegen

lib/stripe/apple_pay_domain.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
+# Domains registered for Apple Pay on the Web
 module Stripe
-  # Domains registered for Apple Pay on the Web
   class ApplePayDomain < APIResource
     extend Stripe::APIOperations::Create
     include Stripe::APIOperations::Delete

lib/stripe/bitcoin_receiver.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
+# Directly creating or retrieving BitcoinReceivers is deprecated. Please use
+# the Sources API instead: https://stripe.com/docs/sources/bitcoin
 module Stripe
   class BitcoinReceiver < APIResource
-    # Directly creating or retrieving BitcoinReceivers is deprecated. Please use
-    # the Sources API instead: https://stripe.com/docs/sources/bitcoin
     extend Stripe::APIOperations::List
 
     OBJECT_NAME = "bitcoin_receiver".freeze
@@ -14,8 +14,7 @@ module Stripe
 
     def resource_url
       if respond_to?(:customer) && !customer.nil? && customer != ""
-        "#{Customer.resource_url}/#{CGI.escape(customer)}/sources" \
+        "#{Customer.resource_url}/#{CGI.escape(customer)}/sources/#{CGI.escape(id)}"
-        "/#{CGI.escape(id)}"
       else
         "#{self.class.resource_url}/#{CGI.escape(id)}"
       end

lib/stripe/card.rb

@@ -10,14 +10,11 @@ module Stripe
 
     def resource_url
       if respond_to?(:recipient) && !recipient.nil? && !recipient.empty?
-        "#{Recipient.resource_url}/#{CGI.escape(recipient)}/cards" \
+        "#{Recipient.resource_url}/#{CGI.escape(recipient)}/cards/#{CGI.escape(id)}"
-        "/#{CGI.escape(id)}"
       elsif respond_to?(:customer) && !customer.nil? && !customer.empty?
-        "#{Customer.resource_url}/#{CGI.escape(customer)}/sources" \
+        "#{Customer.resource_url}/#{CGI.escape(customer)}/sources/#{CGI.escape(id)}"
-        "/#{CGI.escape(id)}"
       elsif respond_to?(:account) && !account.nil? && !account.empty?
-        "#{Account.resource_url}/#{CGI.escape(account)}/external_accounts" \
+        "#{Account.resource_url}/#{CGI.escape(account)}/external_accounts/#{CGI.escape(id)}"
-        "/#{CGI.escape(id)}"
       end
     end
 

lib/stripe/invoice.rb

@@ -21,8 +21,7 @@ module Stripe
     end
 
     def mark_uncollectible(params = {}, opts = {})
-      resp, opts = request(:post, resource_url + "/mark_uncollectible", params,
+      resp, opts = request(:post, resource_url + "/mark_uncollectible", params, opts)
-                           opts)
       initialize_from(resp.data, opts)
     end
 

lib/stripe/issuing/card.rb

@@ -13,7 +13,7 @@ module Stripe
 
       def details(params = {}, opts = {})
         resp, opts = request(:get, resource_url + "/details", params, opts)
-        Util.convert_to_stripe_object(resp.data, opts)
+        initialize_from(resp.data, opts)
       end
     end
   end

lib/stripe/order.rb

@@ -12,21 +12,13 @@ module Stripe
     custom_method :return_order, http_verb: :post, http_path: "returns"
 
     def pay(params = {}, opts = {})
-      resp, opts = request(:post, pay_url, params, opts)
+      resp, opts = request(:post, resource_url + "/pay", params, opts)
       initialize_from(resp.data, opts)
     end
 
-    def return_order(params, opts = {})
+    def return_order(params = {}, opts = {})
-      resp, opts = request(:post, returns_url, params, opts)
+      resp, opts = request(:post, resource_url + "/returns", params, opts)
-      Util.convert_to_stripe_object(resp.data, opts)
+      initialize_from(resp.data, opts)
-    end
-
-    private def pay_url
-      resource_url + "/pay"
-    end
-
-    private def returns_url
-      resource_url + "/returns"
     end
   end
 end

lib/stripe/recipient.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
+# Recipients objects are deprecated. Please use Stripe Connect instead.
 module Stripe
-  # Recipients objects are deprecated. Please use Stripe Connect instead.
   class Recipient < APIResource
     extend Stripe::APIOperations::Create
     include Stripe::APIOperations::Delete

lib/stripe/subscription_item.rb

@@ -10,8 +10,7 @@ module Stripe
     OBJECT_NAME = "subscription_item".freeze
 
     def usage_record_summaries(params = {}, opts = {})
-      resp, opts = request(:get, resource_url + "/usage_record_summaries",
+      resp, opts = request(:get, resource_url + "/usage_record_summaries", params, opts)
-                           params, opts)
       Util.convert_to_stripe_object(resp.data, opts)
     end
   end