fix: modify methods

stripe · Sep 8, 2023 · 09b4781 · 09b4781
1 parent ef87efd
commit 09b4781
Show file tree

Hide file tree

Showing 42 changed files with 154 additions and 1,177 deletions.
diff --git a/stripe/api_resources/billing_portal/configuration.py b/stripe/api_resources/billing_portal/configuration.py
@@ -2,7 +2,6 @@
 # File generated from our OpenAPI spec
 from __future__ import absolute_import, division, print_function
 
-from stripe import util
 from stripe.api_resources.abstract import (
     CreateableAPIResource,
     ListableAPIResource,
@@ -13,6 +12,7 @@
 from stripe.stripe_object import StripeObject
 from typing import Any, Dict, Optional, cast
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 
 class Configuration(
@@ -83,35 +83,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        configuration,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/billing_portal/configurations/{configuration}".format(
-                configuration=util.sanitize_id(configuration)
-            ),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/billing_portal/configurations/{configuration}".format(
-                configuration=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "Configuration":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> "Configuration":

diff --git a/stripe/api_resources/charge.py b/stripe/api_resources/charge.py
@@ -14,6 +14,7 @@
 from stripe.stripe_object import StripeObject
 from typing import Any, Dict, Optional, cast
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 from typing_extensions import TYPE_CHECKING
 
@@ -169,33 +170,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        charge,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/charges/{charge}".format(charge=util.sanitize_id(charge)),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/charges/{charge}".format(
-                charge=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "Charge":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> "Charge":

diff --git a/stripe/api_resources/coupon.py b/stripe/api_resources/coupon.py
@@ -107,33 +107,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        coupon,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/coupons/{coupon}".format(coupon=util.sanitize_id(coupon)),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/coupons/{coupon}".format(
-                coupon=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "Coupon":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> "Coupon":

diff --git a/stripe/api_resources/credit_note.py b/stripe/api_resources/credit_note.py
@@ -14,6 +14,7 @@
 from stripe.stripe_object import StripeObject
 from typing import Any, Dict, List, Optional, cast
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 from typing_extensions import TYPE_CHECKING
 
@@ -119,33 +120,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        id,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/credit_notes/{id}".format(id=util.sanitize_id(id)),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/credit_notes/{id}".format(
-                id=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "CreditNote":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def preview(

diff --git a/stripe/api_resources/customer.py b/stripe/api_resources/customer.py
@@ -241,35 +241,9 @@ def list_payment_methods(self, idempotency_key=None, **params):
         )
 
     @classmethod
-    def _cls_modify(
-        cls,
-        customer,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/customers/{customer}".format(
-                customer=util.sanitize_id(customer)
-            ),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/customers/{customer}".format(
-                customer=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "Customer":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> Any:

diff --git a/stripe/api_resources/dispute.py b/stripe/api_resources/dispute.py
@@ -12,6 +12,7 @@
 from stripe.stripe_object import StripeObject
 from typing import Dict, List, Optional
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 from typing_extensions import TYPE_CHECKING
 
@@ -106,33 +107,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        dispute,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/disputes/{dispute}".format(dispute=util.sanitize_id(dispute)),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/disputes/{dispute}".format(
-                dispute=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "Dispute":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> "Dispute":

diff --git a/stripe/api_resources/file_link.py b/stripe/api_resources/file_link.py
@@ -2,7 +2,6 @@
 # File generated from our OpenAPI spec
 from __future__ import absolute_import, division, print_function
 
-from stripe import util
 from stripe.api_resources.abstract import (
     CreateableAPIResource,
     ListableAPIResource,
@@ -12,6 +11,7 @@
 from stripe.api_resources.list_object import ListObject
 from typing import Dict, Optional, cast
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 from typing_extensions import TYPE_CHECKING
 
@@ -85,33 +85,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        link,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/file_links/{link}".format(link=util.sanitize_id(link)),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/file_links/{link}".format(
-                link=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "FileLink":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def retrieve(cls, id, api_key=None, **params) -> "FileLink":

diff --git a/stripe/api_resources/identity/verification_session.py b/stripe/api_resources/identity/verification_session.py
@@ -13,6 +13,7 @@
 from stripe.stripe_object import StripeObject
 from typing import Dict, Optional, cast
 from typing_extensions import Literal
+from urllib.parse import quote_plus
 
 from typing_extensions import TYPE_CHECKING
 
@@ -132,35 +133,9 @@ def list(
         return result
 
     @classmethod
-    def _cls_modify(
-        cls,
-        session,
-        api_key=None,
-        stripe_version=None,
-        stripe_account=None,
-        **params
-    ):
-        return cls._static_request(
-            "post",
-            "/v1/identity/verification_sessions/{session}".format(
-                session=util.sanitize_id(session)
-            ),
-            api_key=api_key,
-            stripe_version=stripe_version,
-            stripe_account=stripe_account,
-            params=params,
-        )
-
-    @util.class_method_variant("_cls_modify")
-    def modify(self, idempotency_key=None, **params):
-        return self._request(
-            "post",
-            "/v1/identity/verification_sessions/{session}".format(
-                session=util.sanitize_id(self.get("id"))
-            ),
-            idempotency_key=idempotency_key,
-            params=params,
-        )
+    def modify(cls, id, **params) -> "VerificationSession":
+        url = "%s/%s" % (cls.class_url(), quote_plus(id))
+        return cls._static_request("post", url, params=params)
 
     @classmethod
     def _cls_redact(