From e97eaf9963bfd398b0794f9ae06374f4ac97b024 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Thu, 14 Nov 2024 13:03:47 -0500
Subject: [PATCH 01/35] [ADD] Add recurring payments with stripe

---
 recurring_payments_stripe/__init__.py         |  3 +
 recurring_payments_stripe/__manifest__.py     | 19 +++++
 recurring_payments_stripe/models/__init__.py  |  3 +
 .../models/account_move.py                    | 64 +++++++++++++++++
 .../models/sale_subscription.py               | 44 ++++++++++++
 .../security/ir.model.access.csv              |  2 +
 .../views/sale_subscription_views.xml         | 18 +++++
 recurring_payments_stripe/wizard/__init__.py  |  2 +
 .../sale_subscription_payment_wizard.py       | 72 +++++++++++++++++++
 .../sale_subscription_payment_wizard.xml      | 37 ++++++++++
 10 files changed, 264 insertions(+)
 create mode 100644 recurring_payments_stripe/__init__.py
 create mode 100644 recurring_payments_stripe/__manifest__.py
 create mode 100644 recurring_payments_stripe/models/__init__.py
 create mode 100644 recurring_payments_stripe/models/account_move.py
 create mode 100644 recurring_payments_stripe/models/sale_subscription.py
 create mode 100644 recurring_payments_stripe/security/ir.model.access.csv
 create mode 100644 recurring_payments_stripe/views/sale_subscription_views.xml
 create mode 100644 recurring_payments_stripe/wizard/__init__.py
 create mode 100644 recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
 create mode 100644 recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml

diff --git a/recurring_payments_stripe/__init__.py b/recurring_payments_stripe/__init__.py
new file mode 100644
index 0000000000..408a6001bd
--- /dev/null
+++ b/recurring_payments_stripe/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+from . import models
+from . import wizard
diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payments_stripe/__manifest__.py
new file mode 100644
index 0000000000..d6339d88f8
--- /dev/null
+++ b/recurring_payments_stripe/__manifest__.py
@@ -0,0 +1,19 @@
+# -*- coding: utf-8 -*-
+{
+    "name": "Recurring Payments with Stripe",
+    "version": "16.0.1.0.0",
+    "summary": """ Recurring Payments with Stripe """,
+    "author": "Binhex, Odoo Community Association (OCA)",
+    "website": "https://github.com/OCA/contract",
+    "license": "AGPL-3",
+    "category": "Subscription Management",
+    "depends": ["subscription_oca", "payment_stripe", "payment"],
+    "data": [
+        "views/sale_subscription_views.xml",
+        "wizard/sale_subscription_payment_wizard.xml",
+        "security/ir.model.access.csv"
+    ],
+    "application": True,
+    "installable": True,
+    "auto_install": False,
+}
diff --git a/recurring_payments_stripe/models/__init__.py b/recurring_payments_stripe/models/__init__.py
new file mode 100644
index 0000000000..82c93204e4
--- /dev/null
+++ b/recurring_payments_stripe/models/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+from . import sale_subscription
+from . import account_move
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
new file mode 100644
index 0000000000..7a30b07c4f
--- /dev/null
+++ b/recurring_payments_stripe/models/account_move.py
@@ -0,0 +1,64 @@
+# -*- coding: utf-8 -*-
+import logging
+import stripe
+
+from odoo import models, fields, api, _
+from odoo.exceptions import UserError, ValidationError
+
+_logger = logging.getLogger(__name__)
+
+
+class AccountMove(models.Model):
+    _inherit = "account.move"
+
+    def stripe_pay_invoice(self):
+        """Paga la factura en Stripe si `is_recurrent` está activado en la suscripción."""
+        for invoice in self:
+            subscription = invoice.invoice_origin and self.env[
+                "sale.subscription"
+            ].search([("code", "=", invoice.invoice_origin)], limit=1)
+            if (
+                subscription
+                and subscription.is_recurrent
+                and subscription.stripe_customer
+            ):
+                provider = self.env["payment.provider"].search(
+                    [("code", "=", "stripe")],
+                )
+                stripe.api_key = provider.stripe_secret_key
+                try:
+                    # Crea un PaymentIntent en Stripe para el monto de la factura
+                    payment_intent = stripe.PaymentIntent.create(
+                        amount=int(
+                            invoice.amount_total * 100
+                        ),  # Stripe maneja montos en centavos
+                        currency=invoice.currency_id.name.lower(),
+                        customer=subscription.stripe_customer,
+                        payment_method="pm_1QL6VqRwXzxKyS2wEWj2Js5U",
+                        payment_method_types=["card"],
+                        description=f"Odoo Invoice {invoice.name}",
+                        metadata={"odoo_invoice_id": invoice.id},
+                    )
+
+                    # Confirmar el pago y actualizar el estado de la factura en Odoo
+                    if payment_intent["status"] == "succeeded":
+                        invoice.action_post()
+                        invoice.payment_state = "paid"
+                    else:
+                        raise Exception(
+                            f"Error en el pago de Stripe: {payment_intent['status']}"
+                        )
+
+                except stripe.error.StripeError as e:
+                    raise UserError(f"Stripe error: {e.user_message or str(e)}")
+
+    def action_post(self):
+        """Sobreescribe el método `action_post` para procesar el pago automático si `is_recurrent` está activo."""
+        res = super(AccountMove, self).action_post()
+        for invoice in self:
+            subscription = self.env["sale.subscription"].search(
+                [("code", "=", invoice.invoice_origin)], limit=1
+            )
+            if subscription and subscription.is_recurrent:
+                invoice.stripe_pay_invoice()
+        return res
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
new file mode 100644
index 0000000000..72f6750378
--- /dev/null
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -0,0 +1,44 @@
+# -*- coding: utf-8 -*-
+import logging
+import stripe
+
+from odoo import models, fields, api, _
+from odoo.exceptions import UserError, ValidationError
+
+_logger = logging.getLogger(__name__)
+
+
+class SaleSubscription(models.Model):
+    _inherit = "sale.subscription"
+
+    is_recurrent = fields.Boolean(string="Pago Automático Recurrente")
+    # ID del cliente en Stripe
+    stripe_customer = fields.Char(string="Stripe Customer ID")
+
+    def _get_client_stripe(self, secret_key):
+        client = stripe.StripeClient(secret_key)
+        return client
+
+    def create_stripe_customer(self):
+        """Crea o recupera el cliente de Stripe asociado a la suscripción."""
+        provider = self.env["payment.provider"].search(
+            [("code", "=", "stripe")],
+        )
+        stripe.api_key = provider.stripe_secret_key
+
+        client = self._get_client_stripe(provider.stripe_secret_key)
+
+        if not self.stripe_customer:
+            customer = stripe.Customer.create(
+                email=self.partner_id.email,
+                name=self.partner_id.name,
+                metadata={"odoo_subscription_id": self.id},
+            )
+            self.stripe_customer = customer["id"]
+        return self.stripe_customer
+
+    @api.onchange("is_recurrent")
+    def _onchange_is_recurrent(self):
+        for record in self:
+            if record.is_recurrent:
+                record.create_stripe_customer()
diff --git a/recurring_payments_stripe/security/ir.model.access.csv b/recurring_payments_stripe/security/ir.model.access.csv
new file mode 100644
index 0000000000..1b4b0ce2a6
--- /dev/null
+++ b/recurring_payments_stripe/security/ir.model.access.csv
@@ -0,0 +1,2 @@
+id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
+access_sale_subscription_payment_wizard_manager,sale_subscription_payment_wizard_manager,model_sale_subscription_payment_wizard,,1,1,1,1
diff --git a/recurring_payments_stripe/views/sale_subscription_views.xml b/recurring_payments_stripe/views/sale_subscription_views.xml
new file mode 100644
index 0000000000..a223bbb835
--- /dev/null
+++ b/recurring_payments_stripe/views/sale_subscription_views.xml
@@ -0,0 +1,18 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <!-- View sale.subscription form -->
+    <record id="sale_subscription_form" model="ir.ui.view">
+        <field name="name">sale.subscription.form</field>
+        <field name="model">sale.subscription</field>
+        <field name="inherit_id" ref="subscription_oca.sale_subscription_form" />
+        <field name="arch" type="xml">
+            <field name="recurring_next_date" position="after">
+                <field name="is_recurrent" widget="boolean_toggle" />
+                <field name="stripe_customer" widget="CopyClipboardChar" />
+            </field>
+
+        </field>
+    </record>
+
+</odoo>
diff --git a/recurring_payments_stripe/wizard/__init__.py b/recurring_payments_stripe/wizard/__init__.py
new file mode 100644
index 0000000000..15e249b83d
--- /dev/null
+++ b/recurring_payments_stripe/wizard/__init__.py
@@ -0,0 +1,2 @@
+# -*- coding: utf-8 -*-
+from . import sale_subscription_payment_wizard
diff --git a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
new file mode 100644
index 0000000000..7dbcf3da1a
--- /dev/null
+++ b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
@@ -0,0 +1,72 @@
+from odoo import models, fields, api
+from odoo.exceptions import UserError
+import stripe
+
+
+class SaleSubscriptionPaymentWizard(models.TransientModel):
+    _name = "sale.subscription.payment.wizard"
+    _description = "Wizard para Tokenizar Método de Pago de Suscripción"
+
+    subscription_id = fields.Many2one(
+        "sale.subscription", string="Suscripción", required=True
+    )
+    card_number = fields.Char(string="Número de Tarjeta", required=True)
+    card_exp_month = fields.Char(string="Mes de Expiración (MM)", required=True)
+    card_exp_year = fields.Char(string="Año de Expiración (YYYY)", required=True)
+    card_cvc = fields.Char(string="CVC", required=True)
+
+    @api.model
+    def default_get(self, fields):
+        res = super(SaleSubscriptionPaymentWizard, self).default_get(fields)
+        subscription_id = self.env.context.get("active_id")
+        if subscription_id:
+            res["subscription_id"] = subscription_id
+        return res
+
+    def action_create_token(self):
+        """Crea un token en Stripe usando los datos de la tarjeta y lo guarda en la suscripción."""
+        provider = self.env["payment.provider"].search(
+            [("code", "=", "stripe")],
+        )
+        stripe.api_key = provider.stripe_secret_key
+
+        # Intenta crear un token en Stripe
+        try:
+            token = stripe.Token.create(
+                card={
+                    "number": self.card_number,
+                    "exp_month": self.card_exp_month,
+                    "exp_year": self.card_exp_year,
+                    "cvc": self.card_cvc,
+                },
+            )
+
+            # Vincular el token de tarjeta con el cliente en Stripe
+            subscription = self.subscription_id
+            if not subscription.stripe_customer:
+                customer = stripe.Customer.create(
+                    email=subscription.partner_id.email,
+                    name=subscription.partner_id.name,
+                )
+                subscription.stripe_customer = customer["id"]
+
+            # Crear y vincular el método de pago a la suscripción
+            payment_method = stripe.PaymentMethod.create(
+                type="card",
+                card={"token": token.id},
+            )
+            stripe.PaymentMethod.attach(
+                payment_method.id, customer=subscription.stripe_customer
+            )
+            stripe.Customer.modify(
+                subscription.stripe_customer,
+                invoice_settings={"default_payment_method": payment_method.id},
+            )
+
+            # Guardar detalles del método de pago en la suscripción
+            subscription.stripe_payment_method_id = payment_method.id
+            subscription.card_last4 = token.card.last4
+            subscription.card_brand = token.card.brand
+
+        except stripe.error.StripeError as e:
+            raise UserError(f"Error en Stripe: {e.user_message or str(e)}")
diff --git a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml
new file mode 100644
index 0000000000..c174291520
--- /dev/null
+++ b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml
@@ -0,0 +1,37 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <!-- View sale.subscription.payment.wizard form -->
+    <record id="view_sale_subscription_payment_wizard_form" model="ir.ui.view">
+        <field name="name">sale.subscription.payment.wizard.form</field>
+        <field name="model">sale.subscription.payment.wizard</field>
+        <field name="arch" type="xml">
+            <form string="Tokenizar Método de Pago">
+                <group>
+                    <field name="subscription_id" readonly="1" />
+                    <field name="card_number" />
+                    <field name="card_exp_month" />
+                    <field name="card_exp_year" />
+                    <field name="card_cvc" />
+                </group>
+                <footer>
+                    <button string="Guardar Método de Pago" type="object"
+                        name="action_create_token" class="oe_highlight" />
+                    <button string="Cancelar" class="btn-secondary" special="cancel" />
+                </footer>
+            </form>
+        </field>
+    </record>
+
+    <!-- Action sale.subscription.payment.wizard -->
+    <record id="action_sale_subscription_payment_wizard"
+        model="ir.actions.act_window">
+        <field name="name">Sale Subscription Payment Wizard</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">sale.subscription.payment.wizard</field>
+        <field name="view_mode">form</field>
+        <field name="binding_model_id" ref="model_sale_subscription" />
+        <field name="target">new</field>
+    </record>
+
+</odoo>

From e47bd6c8fddd58f3fcd575fda57d7e0e2d8dc815 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Thu, 14 Nov 2024 16:09:48 -0500
Subject: [PATCH 02/35] [FIX] Fixed recurring payments

---
 recurring_payments_stripe/__init__.py         |   1 -
 recurring_payments_stripe/__manifest__.py     |   3 +-
 recurring_payments_stripe/data/ir_cron.xml    |  16 ++
 .../models/account_move.py                    | 163 ++++++++++++++----
 .../models/sale_subscription.py               |  12 +-
 .../security/ir.model.access.csv              |   2 -
 .../views/sale_subscription_views.xml         |   2 +-
 recurring_payments_stripe/wizard/__init__.py  |   2 -
 .../sale_subscription_payment_wizard.py       |  72 --------
 .../sale_subscription_payment_wizard.xml      |  37 ----
 10 files changed, 155 insertions(+), 155 deletions(-)
 create mode 100644 recurring_payments_stripe/data/ir_cron.xml
 delete mode 100644 recurring_payments_stripe/security/ir.model.access.csv
 delete mode 100644 recurring_payments_stripe/wizard/__init__.py
 delete mode 100644 recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
 delete mode 100644 recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml

diff --git a/recurring_payments_stripe/__init__.py b/recurring_payments_stripe/__init__.py
index 408a6001bd..a0fdc10fe1 100644
--- a/recurring_payments_stripe/__init__.py
+++ b/recurring_payments_stripe/__init__.py
@@ -1,3 +1,2 @@
 # -*- coding: utf-8 -*-
 from . import models
-from . import wizard
diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payments_stripe/__manifest__.py
index d6339d88f8..d5245e666a 100644
--- a/recurring_payments_stripe/__manifest__.py
+++ b/recurring_payments_stripe/__manifest__.py
@@ -10,8 +10,7 @@
     "depends": ["subscription_oca", "payment_stripe", "payment"],
     "data": [
         "views/sale_subscription_views.xml",
-        "wizard/sale_subscription_payment_wizard.xml",
-        "security/ir.model.access.csv"
+        "data/ir_cron.xml",
     ],
     "application": True,
     "installable": True,
diff --git a/recurring_payments_stripe/data/ir_cron.xml b/recurring_payments_stripe/data/ir_cron.xml
new file mode 100644
index 0000000000..b33eeb57bf
--- /dev/null
+++ b/recurring_payments_stripe/data/ir_cron.xml
@@ -0,0 +1,16 @@
+<odoo>
+    <data noupdate="1">
+        <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
+        <record id="ir_cron_process_due_invoices" model="ir.cron">
+            <field name="name">Procesar Facturas Vencidas para Suscripciones
+                Recurrentes</field>
+            <field name="model_id" ref="recurring_payments_stripe.model_account_move" />
+            <field name="state">code</field>
+            <field name="code">model.cron_process_due_invoices()</field>
+            <field name="interval_number">1</field>
+            <field name="interval_type">days</field>
+            <field name="numbercall">-1</field>
+            <field name="active">True</field>
+        </record>
+    </data>
+</odoo>
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index 7a30b07c4f..cc1bec6ac0 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -1,6 +1,7 @@
 # -*- coding: utf-8 -*-
 import logging
 import stripe
+from datetime import date
 
 from odoo import models, fields, api, _
 from odoo.exceptions import UserError, ValidationError
@@ -11,54 +12,158 @@
 class AccountMove(models.Model):
     _inherit = "account.move"
 
-    def stripe_pay_invoice(self):
-        """Paga la factura en Stripe si `is_recurrent` está activado en la suscripción."""
+    # def stripe_pay_invoice(self):
+    #     """Paga la factura en Stripe si `is_recurrent` está activado en la suscripción."""
+    #     for invoice in self:
+    #         subscription = invoice.invoice_date and self.env[
+    #             "sale.subscription"
+    #         ].search([("code", "=", invoice.invoice_date)], limit=1)
+    #         if (
+    #             subscription
+    #             and subscription.is_recurrent
+    #             and subscription.stripe_customer
+    #         ):
+    #             provider = self.env["payment.provider"].search(
+    #                 [("code", "=", "stripe")],
+    #             )
+    #             stripe.api_key = provider.stripe_secret_key
+    #             token = self.env["payment.token"].search(
+    #                 [("provider_id", "=", provider.id)]
+    #             )
+    #             try:
+    #                 # Crea un PaymentIntent en Stripe para el monto de la factura
+    #                 payment_intent = stripe.PaymentIntent.create(
+    #                     # Stripe maneja montos en centavos
+    #                     amount=int(invoice.amount_total * 100),
+    #                     currency=invoice.currency_id.name.lower(),
+    #                     customer=token.provider_ref,
+    #                     payment_method=token.stripe_payment_method,
+    #                     payment_method_types=["card"],
+    #                     # Para pagos automáticos sin intervención del usuario
+    #                     off_session=True,
+    #                     # Confirmar el PaymentIntent inmediatamente
+    #                     confirm=True,
+    #                     metadata={"odoo_invoice_id": invoice.id},
+    #                 )
+
+    #                 # Confirmar el pago y actualizar el estado de la factura en Odoo
+    #                 if payment_intent["status"] == "succeeded":
+    #                     invoice.action_post()
+    #                     invoice.payment_state = "paid"
+    #                 else:
+    #                     raise Exception(
+    #                         f"Error en el pago de Stripe: {payment_intent['status']}"
+    #                     )
+
+    #             except stripe.StripeError as e:
+    #                 raise UserError(f"Stripe error: {e.user_message or str(e)}")
+
+    # def action_post(self):
+    #     """Sobreescribe el método `action_post` para procesar el pago automático si `is_recurrent` está activo."""
+    #     res = super(AccountMove, self).action_post()
+    #     for invoice in self:
+    #         subscription = self.env["sale.subscription"].search(
+    #             [("code", "=", invoice.invoice_date)], limit=1
+    #         )
+    #         if subscription and subscription.is_recurrent:
+    #             invoice.stripe_pay_invoice()
+    #     return res
+
+    def action_register_payment(self):
+        """Sobreescribe `action_register_payment` para procesar automáticamente el pago con Stripe en suscripciones."""
         for invoice in self:
-            subscription = invoice.invoice_origin and self.env[
-                "sale.subscription"
-            ].search([("code", "=", invoice.invoice_origin)], limit=1)
-            if (
-                subscription
-                and subscription.is_recurrent
-                and subscription.stripe_customer
-            ):
+            # Buscar la suscripción asociada a la factura, si existe
+            subscription = self.env["sale.subscription"].search(
+                [("code", "=", invoice.invoice_origin)], limit=1
+            )
+
+            # Verificar si la suscripción es recurrente y tiene método de pago en Stripe
+            if subscription and subscription.is_recurrent:
                 provider = self.env["payment.provider"].search(
                     [("code", "=", "stripe")],
                 )
                 stripe.api_key = provider.stripe_secret_key
+                token = self.env["payment.token"].search(
+                    [("provider_id", "=", provider.id)]
+                )
                 try:
-                    # Crea un PaymentIntent en Stripe para el monto de la factura
+                    # Crear el PaymentIntent en Stripe y confirmarlo inmediatamente
                     payment_intent = stripe.PaymentIntent.create(
-                        amount=int(
-                            invoice.amount_total * 100
-                        ),  # Stripe maneja montos en centavos
+                        # Stripe usa centavos
+                        amount=int(invoice.amount_total * 100),
                         currency=invoice.currency_id.name.lower(),
-                        customer=subscription.stripe_customer,
-                        payment_method="pm_1QL6VqRwXzxKyS2wEWj2Js5U",
-                        payment_method_types=["card"],
-                        description=f"Odoo Invoice {invoice.name}",
+                        customer=token.provider_ref,
+                        payment_method=token.stripe_payment_method,
+                        # Para pagos automáticos sin intervención del usuario
+                        off_session=True,
+                        # Confirmar el PaymentIntent inmediatamente
+                        confirm=True,
                         metadata={"odoo_invoice_id": invoice.id},
                     )
 
-                    # Confirmar el pago y actualizar el estado de la factura en Odoo
+                    # Manejar el resultado del PaymentIntent
                     if payment_intent["status"] == "succeeded":
-                        invoice.action_post()
+                        # Si el pago es exitoso, registrar el pago en la factura
+                        Payment = self.env["account.payment"].sudo()
+                        payment_vals = {
+                            "journal_id": self.env["account.journal"]
+                            .search([("type", "=", "bank")], limit=1)
+                            .id,
+                            "amount": invoice.amount_total,
+                            "payment_type": "inbound",
+                            "partner_type": "customer",
+                            "partner_id": invoice.partner_id.id,
+                            "payment_method_id": self.env.ref(
+                                "account.account_payment_method_manual_in"
+                            ).id,
+                            "ref": f"Stripe PaymentIntent {payment_intent['id']}",
+                        }
+                        payment = Payment.create(payment_vals)
+                        payment.action_post()
                         invoice.payment_state = "paid"
+                    elif payment_intent["status"] == "requires_action":
+                        raise UserError(
+                            "El pago requiere autenticación adicional (3D Secure)."
+                        )
                     else:
-                        raise Exception(
+                        raise UserError(
                             f"Error en el pago de Stripe: {payment_intent['status']}"
                         )
 
-                except stripe.error.StripeError as e:
-                    raise UserError(f"Stripe error: {e.user_message or str(e)}")
+                except stripe.StripeError as e:
+                    raise UserError(f"Error de Stripe: {e.user_message or str(e)}")
 
-    def action_post(self):
-        """Sobreescribe el método `action_post` para procesar el pago automático si `is_recurrent` está activo."""
-        res = super(AccountMove, self).action_post()
-        for invoice in self:
+            else:
+                # Si no es una suscripción recurrente o no tiene método de pago en Stripe, llama al método original
+                return super(AccountMove, self).action_register_payment()
+
+    @api.model
+    def cron_process_due_invoices(self):
+        """Procesa el pago de facturas vencidas para suscripciones recurrentes."""
+        # Obtener la fecha de hoy
+        today = date.today()
+
+        # Buscar facturas de suscripciones recurrentes que vencen hoy y están abiertas
+        invoices = self.search(
+            [
+                ("state", "=", "posted"),
+                ("payment_state", "=", "not_paid"),
+            ]
+        )
+
+        for invoice in invoices:
+            # Buscar la suscripción asociada a la factura
             subscription = self.env["sale.subscription"].search(
                 [("code", "=", invoice.invoice_origin)], limit=1
             )
+
+            # Verificar si es una suscripción recurrente con Stripe
             if subscription and subscription.is_recurrent:
-                invoice.stripe_pay_invoice()
-        return res
+                try:
+                    # Intentar registrar el pago en la factura
+                    invoice.action_register_payment()
+                except Exception as e:
+                    # Capturar excepciones en caso de error en el registro de pago
+                    _logger.error(
+                        f"Error al procesar el pago de la factura {invoice.id}: {str(e)}"
+                    )
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
index 72f6750378..d03a64cb0c 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -15,10 +15,6 @@ class SaleSubscription(models.Model):
     # ID del cliente en Stripe
     stripe_customer = fields.Char(string="Stripe Customer ID")
 
-    def _get_client_stripe(self, secret_key):
-        client = stripe.StripeClient(secret_key)
-        return client
-
     def create_stripe_customer(self):
         """Crea o recupera el cliente de Stripe asociado a la suscripción."""
         provider = self.env["payment.provider"].search(
@@ -26,13 +22,11 @@ def create_stripe_customer(self):
         )
         stripe.api_key = provider.stripe_secret_key
 
-        client = self._get_client_stripe(provider.stripe_secret_key)
-
         if not self.stripe_customer:
             customer = stripe.Customer.create(
-                email=self.partner_id.email,
-                name=self.partner_id.name,
-                metadata={"odoo_subscription_id": self.id},
+                email=self.env.user.email,
+                name=self.env.user.name,
+                metadata={"odoo_subscription": self.id},
             )
             self.stripe_customer = customer["id"]
         return self.stripe_customer
diff --git a/recurring_payments_stripe/security/ir.model.access.csv b/recurring_payments_stripe/security/ir.model.access.csv
deleted file mode 100644
index 1b4b0ce2a6..0000000000
--- a/recurring_payments_stripe/security/ir.model.access.csv
+++ /dev/null
@@ -1,2 +0,0 @@
-id,name,model_id/id,group_id/id,perm_read,perm_write,perm_create,perm_unlink
-access_sale_subscription_payment_wizard_manager,sale_subscription_payment_wizard_manager,model_sale_subscription_payment_wizard,,1,1,1,1
diff --git a/recurring_payments_stripe/views/sale_subscription_views.xml b/recurring_payments_stripe/views/sale_subscription_views.xml
index a223bbb835..e00c2949fb 100644
--- a/recurring_payments_stripe/views/sale_subscription_views.xml
+++ b/recurring_payments_stripe/views/sale_subscription_views.xml
@@ -9,7 +9,7 @@
         <field name="arch" type="xml">
             <field name="recurring_next_date" position="after">
                 <field name="is_recurrent" widget="boolean_toggle" />
-                <field name="stripe_customer" widget="CopyClipboardChar" />
+                <!-- <field name="stripe_customer" widget="CopyClipboardChar" /> -->
             </field>
 
         </field>
diff --git a/recurring_payments_stripe/wizard/__init__.py b/recurring_payments_stripe/wizard/__init__.py
deleted file mode 100644
index 15e249b83d..0000000000
--- a/recurring_payments_stripe/wizard/__init__.py
+++ /dev/null
@@ -1,2 +0,0 @@
-# -*- coding: utf-8 -*-
-from . import sale_subscription_payment_wizard
diff --git a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
deleted file mode 100644
index 7dbcf3da1a..0000000000
--- a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.py
+++ /dev/null
@@ -1,72 +0,0 @@
-from odoo import models, fields, api
-from odoo.exceptions import UserError
-import stripe
-
-
-class SaleSubscriptionPaymentWizard(models.TransientModel):
-    _name = "sale.subscription.payment.wizard"
-    _description = "Wizard para Tokenizar Método de Pago de Suscripción"
-
-    subscription_id = fields.Many2one(
-        "sale.subscription", string="Suscripción", required=True
-    )
-    card_number = fields.Char(string="Número de Tarjeta", required=True)
-    card_exp_month = fields.Char(string="Mes de Expiración (MM)", required=True)
-    card_exp_year = fields.Char(string="Año de Expiración (YYYY)", required=True)
-    card_cvc = fields.Char(string="CVC", required=True)
-
-    @api.model
-    def default_get(self, fields):
-        res = super(SaleSubscriptionPaymentWizard, self).default_get(fields)
-        subscription_id = self.env.context.get("active_id")
-        if subscription_id:
-            res["subscription_id"] = subscription_id
-        return res
-
-    def action_create_token(self):
-        """Crea un token en Stripe usando los datos de la tarjeta y lo guarda en la suscripción."""
-        provider = self.env["payment.provider"].search(
-            [("code", "=", "stripe")],
-        )
-        stripe.api_key = provider.stripe_secret_key
-
-        # Intenta crear un token en Stripe
-        try:
-            token = stripe.Token.create(
-                card={
-                    "number": self.card_number,
-                    "exp_month": self.card_exp_month,
-                    "exp_year": self.card_exp_year,
-                    "cvc": self.card_cvc,
-                },
-            )
-
-            # Vincular el token de tarjeta con el cliente en Stripe
-            subscription = self.subscription_id
-            if not subscription.stripe_customer:
-                customer = stripe.Customer.create(
-                    email=subscription.partner_id.email,
-                    name=subscription.partner_id.name,
-                )
-                subscription.stripe_customer = customer["id"]
-
-            # Crear y vincular el método de pago a la suscripción
-            payment_method = stripe.PaymentMethod.create(
-                type="card",
-                card={"token": token.id},
-            )
-            stripe.PaymentMethod.attach(
-                payment_method.id, customer=subscription.stripe_customer
-            )
-            stripe.Customer.modify(
-                subscription.stripe_customer,
-                invoice_settings={"default_payment_method": payment_method.id},
-            )
-
-            # Guardar detalles del método de pago en la suscripción
-            subscription.stripe_payment_method_id = payment_method.id
-            subscription.card_last4 = token.card.last4
-            subscription.card_brand = token.card.brand
-
-        except stripe.error.StripeError as e:
-            raise UserError(f"Error en Stripe: {e.user_message or str(e)}")
diff --git a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml b/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml
deleted file mode 100644
index c174291520..0000000000
--- a/recurring_payments_stripe/wizard/sale_subscription_payment_wizard.xml
+++ /dev/null
@@ -1,37 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<odoo>
-
-    <!-- View sale.subscription.payment.wizard form -->
-    <record id="view_sale_subscription_payment_wizard_form" model="ir.ui.view">
-        <field name="name">sale.subscription.payment.wizard.form</field>
-        <field name="model">sale.subscription.payment.wizard</field>
-        <field name="arch" type="xml">
-            <form string="Tokenizar Método de Pago">
-                <group>
-                    <field name="subscription_id" readonly="1" />
-                    <field name="card_number" />
-                    <field name="card_exp_month" />
-                    <field name="card_exp_year" />
-                    <field name="card_cvc" />
-                </group>
-                <footer>
-                    <button string="Guardar Método de Pago" type="object"
-                        name="action_create_token" class="oe_highlight" />
-                    <button string="Cancelar" class="btn-secondary" special="cancel" />
-                </footer>
-            </form>
-        </field>
-    </record>
-
-    <!-- Action sale.subscription.payment.wizard -->
-    <record id="action_sale_subscription_payment_wizard"
-        model="ir.actions.act_window">
-        <field name="name">Sale Subscription Payment Wizard</field>
-        <field name="type">ir.actions.act_window</field>
-        <field name="res_model">sale.subscription.payment.wizard</field>
-        <field name="view_mode">form</field>
-        <field name="binding_model_id" ref="model_sale_subscription" />
-        <field name="target">new</field>
-    </record>
-
-</odoo>

From e72ee4b3df4f19a56222b3862449d6a8b7801e51 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Thu, 28 Nov 2024 15:00:56 -0500
Subject: [PATCH 03/35] [FIX] Resolved all suggestion

---
 recurring_payments_stripe/__manifest__.py     |   1 -
 recurring_payments_stripe/data/ir_cron.xml    |   3 +-
 .../models/account_move.py                    | 125 ++++--------------
 .../models/sale_subscription.py               |  48 ++++---
 .../views/sale_subscription_views.xml         |   6 +-
 5 files changed, 53 insertions(+), 130 deletions(-)

diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payments_stripe/__manifest__.py
index d5245e666a..bca6e36757 100644
--- a/recurring_payments_stripe/__manifest__.py
+++ b/recurring_payments_stripe/__manifest__.py
@@ -12,7 +12,6 @@
         "views/sale_subscription_views.xml",
         "data/ir_cron.xml",
     ],
-    "application": True,
     "installable": True,
     "auto_install": False,
 }
diff --git a/recurring_payments_stripe/data/ir_cron.xml b/recurring_payments_stripe/data/ir_cron.xml
index b33eeb57bf..d6580b1969 100644
--- a/recurring_payments_stripe/data/ir_cron.xml
+++ b/recurring_payments_stripe/data/ir_cron.xml
@@ -2,8 +2,7 @@
     <data noupdate="1">
         <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
         <record id="ir_cron_process_due_invoices" model="ir.cron">
-            <field name="name">Procesar Facturas Vencidas para Suscripciones
-                Recurrentes</field>
+            <field name="name">Process Overdue Invoices for Subscriptions</field>
             <field name="model_id" ref="recurring_payments_stripe.model_account_move" />
             <field name="state">code</field>
             <field name="code">model.cron_process_due_invoices()</field>
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index cc1bec6ac0..2535853585 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -1,10 +1,9 @@
 # -*- coding: utf-8 -*-
 import logging
 import stripe
-from datetime import date
 
-from odoo import models, fields, api, _
-from odoo.exceptions import UserError, ValidationError
+from odoo import models, api
+from odoo.exceptions import UserError
 
 _logger = logging.getLogger(__name__)
 
@@ -12,82 +11,24 @@
 class AccountMove(models.Model):
     _inherit = "account.move"
 
-    # def stripe_pay_invoice(self):
-    #     """Paga la factura en Stripe si `is_recurrent` está activado en la suscripción."""
-    #     for invoice in self:
-    #         subscription = invoice.invoice_date and self.env[
-    #             "sale.subscription"
-    #         ].search([("code", "=", invoice.invoice_date)], limit=1)
-    #         if (
-    #             subscription
-    #             and subscription.is_recurrent
-    #             and subscription.stripe_customer
-    #         ):
-    #             provider = self.env["payment.provider"].search(
-    #                 [("code", "=", "stripe")],
-    #             )
-    #             stripe.api_key = provider.stripe_secret_key
-    #             token = self.env["payment.token"].search(
-    #                 [("provider_id", "=", provider.id)]
-    #             )
-    #             try:
-    #                 # Crea un PaymentIntent en Stripe para el monto de la factura
-    #                 payment_intent = stripe.PaymentIntent.create(
-    #                     # Stripe maneja montos en centavos
-    #                     amount=int(invoice.amount_total * 100),
-    #                     currency=invoice.currency_id.name.lower(),
-    #                     customer=token.provider_ref,
-    #                     payment_method=token.stripe_payment_method,
-    #                     payment_method_types=["card"],
-    #                     # Para pagos automáticos sin intervención del usuario
-    #                     off_session=True,
-    #                     # Confirmar el PaymentIntent inmediatamente
-    #                     confirm=True,
-    #                     metadata={"odoo_invoice_id": invoice.id},
-    #                 )
-
-    #                 # Confirmar el pago y actualizar el estado de la factura en Odoo
-    #                 if payment_intent["status"] == "succeeded":
-    #                     invoice.action_post()
-    #                     invoice.payment_state = "paid"
-    #                 else:
-    #                     raise Exception(
-    #                         f"Error en el pago de Stripe: {payment_intent['status']}"
-    #                     )
-
-    #             except stripe.StripeError as e:
-    #                 raise UserError(f"Stripe error: {e.user_message or str(e)}")
-
-    # def action_post(self):
-    #     """Sobreescribe el método `action_post` para procesar el pago automático si `is_recurrent` está activo."""
-    #     res = super(AccountMove, self).action_post()
-    #     for invoice in self:
-    #         subscription = self.env["sale.subscription"].search(
-    #             [("code", "=", invoice.invoice_date)], limit=1
-    #         )
-    #         if subscription and subscription.is_recurrent:
-    #             invoice.stripe_pay_invoice()
-    #     return res
-
     def action_register_payment(self):
-        """Sobreescribe `action_register_payment` para procesar automáticamente el pago con Stripe en suscripciones."""
+        """
+        Override `action_register_payment` to automatically process Stripe
+        payment on subscriptions.
+        """
         for invoice in self:
-            # Buscar la suscripción asociada a la factura, si existe
-            subscription = self.env["sale.subscription"].search(
-                [("code", "=", invoice.invoice_origin)], limit=1
-            )
+            # Find the subscription associated with the invoice, if it exists
+            subscription = invoice.subscription_id
 
-            # Verificar si la suscripción es recurrente y tiene método de pago en Stripe
-            if subscription and subscription.is_recurrent:
-                provider = self.env["payment.provider"].search(
-                    [("code", "=", "stripe")],
-                )
+            # Check if the subscription is recurring and has a payment method
+            if subscription and subscription.charge_automatically:
+                provider = subscription.provider_id
                 stripe.api_key = provider.stripe_secret_key
                 token = self.env["payment.token"].search(
                     [("provider_id", "=", provider.id)]
                 )
                 try:
-                    # Crear el PaymentIntent en Stripe y confirmarlo inmediatamente
+                    # Create the PaymentIntent and confirm it immediately
                     payment_intent = stripe.PaymentIntent.create(
                         # Stripe usa centavos
                         amount=int(invoice.amount_total * 100),
@@ -98,12 +39,12 @@ def action_register_payment(self):
                         off_session=True,
                         # Confirmar el PaymentIntent inmediatamente
                         confirm=True,
-                        metadata={"odoo_invoice_id": invoice.id},
+                        metadata={"odoo_invoice_id": str(invoice.id)},
                     )
 
                     # Manejar el resultado del PaymentIntent
                     if payment_intent["status"] == "succeeded":
-                        # Si el pago es exitoso, registrar el pago en la factura
+                        # If the payment is successful, record the payment on the invoice
                         Payment = self.env["account.payment"].sudo()
                         payment_vals = {
                             "journal_id": self.env["account.journal"]
@@ -123,47 +64,31 @@ def action_register_payment(self):
                         invoice.payment_state = "paid"
                     elif payment_intent["status"] == "requires_action":
                         raise UserError(
-                            "El pago requiere autenticación adicional (3D Secure)."
+                            "Payment requires additional authentication (3D Secure)."
                         )
                     else:
                         raise UserError(
-                            f"Error en el pago de Stripe: {payment_intent['status']}"
+                            f"Stripe payment error: {payment_intent['status']}"
                         )
 
                 except stripe.StripeError as e:
-                    raise UserError(f"Error de Stripe: {e.user_message or str(e)}")
+                    raise UserError(f"Stripe error: {e.user_message or str(e)}")
 
             else:
-                # Si no es una suscripción recurrente o no tiene método de pago en Stripe, llama al método original
                 return super(AccountMove, self).action_register_payment()
 
     @api.model
     def cron_process_due_invoices(self):
-        """Procesa el pago de facturas vencidas para suscripciones recurrentes."""
-        # Obtener la fecha de hoy
-        today = date.today()
-
-        # Buscar facturas de suscripciones recurrentes que vencen hoy y están abiertas
-        invoices = self.search(
-            [
-                ("state", "=", "posted"),
-                ("payment_state", "=", "not_paid"),
-            ]
-        )
+        """Process payment of overdue invoices for recurring subscriptions."""
 
-        for invoice in invoices:
-            # Buscar la suscripción asociada a la factura
-            subscription = self.env["sale.subscription"].search(
-                [("code", "=", invoice.invoice_origin)], limit=1
-            )
+        for invoice in self:
+            # Find the subscription associated with the invoice
+            subscription = invoice.subscription_id
 
-            # Verificar si es una suscripción recurrente con Stripe
-            if subscription and subscription.is_recurrent:
+            # Check if it's a recurring subscription with Stripe
+            if subscription and subscription.charge_automatically:
                 try:
-                    # Intentar registrar el pago en la factura
+                    # Register the payment
                     invoice.action_register_payment()
                 except Exception as e:
-                    # Capturar excepciones en caso de error en el registro de pago
-                    _logger.error(
-                        f"Error al procesar el pago de la factura {invoice.id}: {str(e)}"
-                    )
+                    _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
index d03a64cb0c..30bef18952 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -1,38 +1,36 @@
 # -*- coding: utf-8 -*-
-import logging
 import stripe
 
-from odoo import models, fields, api, _
-from odoo.exceptions import UserError, ValidationError
-
-_logger = logging.getLogger(__name__)
+from odoo import models, fields, api
 
 
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
-    is_recurrent = fields.Boolean(string="Pago Automático Recurrente")
-    # ID del cliente en Stripe
+    charge_automatically = fields.Boolean(string="Charge Automatically")
     stripe_customer = fields.Char(string="Stripe Customer ID")
+    provider_id = fields.Many2one(
+        string="Provider_id",
+        domain=[("code", "=", "stripe")],
+        comodel_name="payment.provider",
+    )
 
     def create_stripe_customer(self):
-        """Crea o recupera el cliente de Stripe asociado a la suscripción."""
-        provider = self.env["payment.provider"].search(
-            [("code", "=", "stripe")],
-        )
-        stripe.api_key = provider.stripe_secret_key
-
-        if not self.stripe_customer:
-            customer = stripe.Customer.create(
-                email=self.env.user.email,
-                name=self.env.user.name,
-                metadata={"odoo_subscription": self.id},
-            )
-            self.stripe_customer = customer["id"]
-        return self.stripe_customer
-
-    @api.onchange("is_recurrent")
-    def _onchange_is_recurrent(self):
+        provider = self.provider_id
+        if provider:
+            stripe.api_key = provider.stripe_secret_key
+
+            if not self.stripe_customer:
+                customer = stripe.Customer.create(
+                    email=self.env.user.email,
+                    name=self.env.user.name,
+                    metadata={"odoo_subscription": str(self.id)},
+                )
+                self.stripe_customer = customer["id"]
+            return self.stripe_customer
+
+    @api.onchange("charge_automatically")
+    def _onchange_charge_automatically(self):
         for record in self:
-            if record.is_recurrent:
+            if record.charge_automatically:
                 record.create_stripe_customer()
diff --git a/recurring_payments_stripe/views/sale_subscription_views.xml b/recurring_payments_stripe/views/sale_subscription_views.xml
index e00c2949fb..dfca8d901b 100644
--- a/recurring_payments_stripe/views/sale_subscription_views.xml
+++ b/recurring_payments_stripe/views/sale_subscription_views.xml
@@ -8,8 +8,10 @@
         <field name="inherit_id" ref="subscription_oca.sale_subscription_form" />
         <field name="arch" type="xml">
             <field name="recurring_next_date" position="after">
-                <field name="is_recurrent" widget="boolean_toggle" />
-                <!-- <field name="stripe_customer" widget="CopyClipboardChar" /> -->
+                <field name="charge_automatically" widget="boolean_toggle" />
+                <field name="provider_id"
+                    options="{'no_create': True, 'no_open': True}"
+                    attrs="{'invisible': [('charge_automatically', '=', False)]}" />
             </field>
 
         </field>

From ea7a9b2dd1f08f17d7b199492f59657c5827cd78 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Thu, 28 Nov 2024 15:51:41 -0500
Subject: [PATCH 04/35] [ADD] Add precommit rules

---
 contract_sale/README.rst                      |   2 +-
 contract_sale/readme/CONFIGURE.rst            |   0
 contract_sale/readme/CREDITS.rst              |   0
 contract_sale/readme/DEVELOP.rst              |   0
 contract_sale/readme/HISTORY.rst              |   0
 contract_sale/readme/INSTALL.rst              |   0
 contract_sale/readme/ROADMAP.rst              |   0
 contract_sale/static/description/index.html   |  13 +-
 recurring_payments_stripe/README.rst          |  70 +++
 recurring_payments_stripe/__init__.py         |   1 -
 recurring_payments_stripe/__manifest__.py     |   1 -
 recurring_payments_stripe/data/ir_cron.xml    |  26 +-
 recurring_payments_stripe/models/__init__.py  |   1 -
 .../models/account_move.py                    |   4 +-
 .../models/sale_subscription.py               |   3 +-
 .../readme/CONFIGURE.rst                      |   0
 .../readme/CONTRIBUTORS.rst                   |   0
 recurring_payments_stripe/readme/CREDITS.rst  |   0
 .../readme/DESCRIPTION.rst                    |   0
 recurring_payments_stripe/readme/DEVELOP.rst  |   0
 recurring_payments_stripe/readme/HISTORY.rst  |   0
 recurring_payments_stripe/readme/INSTALL.rst  |   0
 recurring_payments_stripe/readme/ROADMAP.rst  |   0
 recurring_payments_stripe/readme/USAGE.rst    |   0
 .../static/description/index.html             | 415 ++++++++++++++++++
 .../views/sale_subscription_views.xml         |   8 +-
 .../odoo/addons/recurring_payments_stripe     |   1 +
 setup/recurring_payments_stripe/setup.py      |   6 +
 28 files changed, 521 insertions(+), 30 deletions(-)
 create mode 100644 contract_sale/readme/CONFIGURE.rst
 create mode 100644 contract_sale/readme/CREDITS.rst
 create mode 100644 contract_sale/readme/DEVELOP.rst
 create mode 100644 contract_sale/readme/HISTORY.rst
 create mode 100644 contract_sale/readme/INSTALL.rst
 create mode 100644 contract_sale/readme/ROADMAP.rst
 create mode 100644 recurring_payments_stripe/README.rst
 create mode 100644 recurring_payments_stripe/readme/CONFIGURE.rst
 create mode 100644 recurring_payments_stripe/readme/CONTRIBUTORS.rst
 create mode 100644 recurring_payments_stripe/readme/CREDITS.rst
 create mode 100644 recurring_payments_stripe/readme/DESCRIPTION.rst
 create mode 100644 recurring_payments_stripe/readme/DEVELOP.rst
 create mode 100644 recurring_payments_stripe/readme/HISTORY.rst
 create mode 100644 recurring_payments_stripe/readme/INSTALL.rst
 create mode 100644 recurring_payments_stripe/readme/ROADMAP.rst
 create mode 100644 recurring_payments_stripe/readme/USAGE.rst
 create mode 100644 recurring_payments_stripe/static/description/index.html
 create mode 120000 setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe
 create mode 100644 setup/recurring_payments_stripe/setup.py

diff --git a/contract_sale/README.rst b/contract_sale/README.rst
index e125136f45..2930853090 100644
--- a/contract_sale/README.rst
+++ b/contract_sale/README.rst
@@ -7,7 +7,7 @@ Contract from Sale
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:00fb3cdc565442ffcd3351e97d0516ce6f4ceb55402981b183f7e717a1e23173
+   !! source digest: sha256:c2e49dd78cebc553bbe7e5bfa2c8658e29e120878e688512e9599b43144815dd
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Production%2FStable-green.png
diff --git a/contract_sale/readme/CONFIGURE.rst b/contract_sale/readme/CONFIGURE.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/readme/CREDITS.rst b/contract_sale/readme/CREDITS.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/readme/DEVELOP.rst b/contract_sale/readme/DEVELOP.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/readme/HISTORY.rst b/contract_sale/readme/HISTORY.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/readme/INSTALL.rst b/contract_sale/readme/INSTALL.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/readme/ROADMAP.rst b/contract_sale/readme/ROADMAP.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/contract_sale/static/description/index.html b/contract_sale/static/description/index.html
index 2fec06806d..c4924509c2 100644
--- a/contract_sale/static/description/index.html
+++ b/contract_sale/static/description/index.html
@@ -8,10 +8,11 @@
 
 /*
 :Author: David Goodger (goodger@python.org)
-:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
 :Copyright: This stylesheet has been placed in the public domain.
 
 Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
 
 See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
 customize this style sheet.
@@ -274,7 +275,7 @@
   margin-left: 2em ;
   margin-right: 2em }
 
-pre.code .ln { color: grey; } /* line numbers */
+pre.code .ln { color: gray; } /* line numbers */
 pre.code, code { background-color: #eeeeee }
 pre.code .comment, code .comment { color: #5C6576 }
 pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
@@ -300,7 +301,7 @@
 span.pre {
   white-space: pre }
 
-span.problematic {
+span.problematic, pre.problematic {
   color: red }
 
 span.section-subtitle {
@@ -366,7 +367,7 @@ <h1 class="title">Contract from Sale</h1>
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:00fb3cdc565442ffcd3351e97d0516ce6f4ceb55402981b183f7e717a1e23173
+!! source digest: sha256:c2e49dd78cebc553bbe7e5bfa2c8658e29e120878e688512e9599b43144815dd
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Production/Stable" src="https://img.shields.io/badge/maturity-Production%2FStable-green.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/contract_sale"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-contract_sale"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module allows access to contracts for sale employees without account
@@ -428,7 +429,9 @@ <h2><a class="toc-backref" href="#toc-entry-5">Contributors</a></h2>
 <div class="section" id="maintainers">
 <h2><a class="toc-backref" href="#toc-entry-6">Maintainers</a></h2>
 <p>This module is maintained by the OCA.</p>
-<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<a class="reference external image-reference" href="https://odoo-community.org">
+<img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
+</a>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
diff --git a/recurring_payments_stripe/README.rst b/recurring_payments_stripe/README.rst
new file mode 100644
index 0000000000..31fcb6beb7
--- /dev/null
+++ b/recurring_payments_stripe/README.rst
@@ -0,0 +1,70 @@
+==============================
+Recurring Payments with Stripe
+==============================
+
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:7aacc5b46917b14e43404143f0ccd1519d1411d580330b8eb8a8ef608fe3a5a1
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github
+    :target: https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe
+    :alt: OCA/contract
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payments_stripe
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=16.0
+    :alt: Try me on Runboat
+
+|badge1| |badge2| |badge3| |badge4| |badge5|
+
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/contract/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/contract/issues/new?body=module:%20recurring_payments_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
+
+Credits
+=======
+
+Authors
+~~~~~~~
+
+* Binhex
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/recurring_payments_stripe/__init__.py b/recurring_payments_stripe/__init__.py
index a0fdc10fe1..0650744f6b 100644
--- a/recurring_payments_stripe/__init__.py
+++ b/recurring_payments_stripe/__init__.py
@@ -1,2 +1 @@
-# -*- coding: utf-8 -*-
 from . import models
diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payments_stripe/__manifest__.py
index bca6e36757..ca812198c9 100644
--- a/recurring_payments_stripe/__manifest__.py
+++ b/recurring_payments_stripe/__manifest__.py
@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 {
     "name": "Recurring Payments with Stripe",
     "version": "16.0.1.0.0",
diff --git a/recurring_payments_stripe/data/ir_cron.xml b/recurring_payments_stripe/data/ir_cron.xml
index d6580b1969..ff41276318 100644
--- a/recurring_payments_stripe/data/ir_cron.xml
+++ b/recurring_payments_stripe/data/ir_cron.xml
@@ -1,15 +1,13 @@
-<odoo>
-    <data noupdate="1">
-        <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
-        <record id="ir_cron_process_due_invoices" model="ir.cron">
-            <field name="name">Process Overdue Invoices for Subscriptions</field>
-            <field name="model_id" ref="recurring_payments_stripe.model_account_move" />
-            <field name="state">code</field>
-            <field name="code">model.cron_process_due_invoices()</field>
-            <field name="interval_number">1</field>
-            <field name="interval_type">days</field>
-            <field name="numbercall">-1</field>
-            <field name="active">True</field>
-        </record>
-    </data>
+<odoo noupdate="1">
+    <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
+    <record id="ir_cron_process_due_invoices" model="ir.cron">
+        <field name="name">Process Overdue Invoices for Subscriptions</field>
+        <field name="model_id" ref="recurring_payments_stripe.model_account_move" />
+        <field name="state">code</field>
+        <field name="code">model.cron_process_due_invoices()</field>
+        <field name="interval_number">1</field>
+        <field name="interval_type">days</field>
+        <field name="numbercall">-1</field>
+        <field name="active">True</field>
+    </record>
 </odoo>
diff --git a/recurring_payments_stripe/models/__init__.py b/recurring_payments_stripe/models/__init__.py
index 82c93204e4..fe83caa1a8 100644
--- a/recurring_payments_stripe/models/__init__.py
+++ b/recurring_payments_stripe/models/__init__.py
@@ -1,3 +1,2 @@
-# -*- coding: utf-8 -*-
 from . import sale_subscription
 from . import account_move
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index 2535853585..14d29e011e 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -1,8 +1,8 @@
-# -*- coding: utf-8 -*-
 import logging
+
 import stripe
 
-from odoo import models, api
+from odoo import api, models
 from odoo.exceptions import UserError
 
 _logger = logging.getLogger(__name__)
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
index 30bef18952..ccc60eaf70 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -1,7 +1,6 @@
-# -*- coding: utf-8 -*-
 import stripe
 
-from odoo import models, fields, api
+from odoo import api, fields, models
 
 
 class SaleSubscription(models.Model):
diff --git a/recurring_payments_stripe/readme/CONFIGURE.rst b/recurring_payments_stripe/readme/CONFIGURE.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/CONTRIBUTORS.rst b/recurring_payments_stripe/readme/CONTRIBUTORS.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/CREDITS.rst b/recurring_payments_stripe/readme/CREDITS.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/DESCRIPTION.rst b/recurring_payments_stripe/readme/DESCRIPTION.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/DEVELOP.rst b/recurring_payments_stripe/readme/DEVELOP.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/HISTORY.rst b/recurring_payments_stripe/readme/HISTORY.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/INSTALL.rst b/recurring_payments_stripe/readme/INSTALL.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/ROADMAP.rst b/recurring_payments_stripe/readme/ROADMAP.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/readme/USAGE.rst b/recurring_payments_stripe/readme/USAGE.rst
new file mode 100644
index 0000000000..e69de29bb2
diff --git a/recurring_payments_stripe/static/description/index.html b/recurring_payments_stripe/static/description/index.html
new file mode 100644
index 0000000000..d9629f4109
--- /dev/null
+++ b/recurring_payments_stripe/static/description/index.html
@@ -0,0 +1,415 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
+<title>Recurring Payments with Stripe</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 9511 2024-01-13 09:50:07Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+Despite the name, some widely supported CSS2 features are used.
+
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: gray; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic, pre.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="recurring-payments-with-stripe">
+<h1 class="title">Recurring Payments with Stripe</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:7aacc5b46917b14e43404143f0ccd1519d1411d580330b8eb8a8ef608fe3a5a1
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payments_stripe"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-4">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/contract/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20recurring_payments_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
+<ul class="simple">
+<li>Binhex</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org">
+<img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
+</a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe">OCA/contract</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>
diff --git a/recurring_payments_stripe/views/sale_subscription_views.xml b/recurring_payments_stripe/views/sale_subscription_views.xml
index dfca8d901b..bdfc7c87c0 100644
--- a/recurring_payments_stripe/views/sale_subscription_views.xml
+++ b/recurring_payments_stripe/views/sale_subscription_views.xml
@@ -1,4 +1,4 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
 
     <!-- View sale.subscription form -->
@@ -9,9 +9,11 @@
         <field name="arch" type="xml">
             <field name="recurring_next_date" position="after">
                 <field name="charge_automatically" widget="boolean_toggle" />
-                <field name="provider_id"
+                <field
+                    name="provider_id"
                     options="{'no_create': True, 'no_open': True}"
-                    attrs="{'invisible': [('charge_automatically', '=', False)]}" />
+                    attrs="{'invisible': [('charge_automatically', '=', False)]}"
+                />
             </field>
 
         </field>
diff --git a/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe b/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe
new file mode 120000
index 0000000000..0bdeb61a53
--- /dev/null
+++ b/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe
@@ -0,0 +1 @@
+../../../../recurring_payments_stripe
\ No newline at end of file
diff --git a/setup/recurring_payments_stripe/setup.py b/setup/recurring_payments_stripe/setup.py
new file mode 100644
index 0000000000..28c57bb640
--- /dev/null
+++ b/setup/recurring_payments_stripe/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)

From 6007122055d9a6eec5b53d89f3c2c98efe8f77bd Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 3 Dec 2024 09:02:58 -0500
Subject: [PATCH 05/35] [ADD] Add stripe library

---
 requirements.txt | 1 +
 1 file changed, 1 insertion(+)

diff --git a/requirements.txt b/requirements.txt
index 7d41f1be0f..55fb35df0a 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,2 +1,3 @@
 # generated from manifests external_dependencies
 python-dateutil
+stripe

From d21d398a484be3c48d12476e8ec68b2879cb2a67 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 3 Dec 2024 09:05:41 -0500
Subject: [PATCH 06/35] [ADD] Add stripe library

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index 19e8652da2..7e75fffc03 100644
--- a/README.md
+++ b/README.md
@@ -34,6 +34,7 @@ addon | version | maintainers | summary
 [contract_variable_quantity](contract_variable_quantity/) | 16.0.1.1.0 |  | Variable quantity in contract recurrent invoicing
 [product_contract](product_contract/) | 16.0.1.0.0 | [![sbejaoui](https://github.com/sbejaoui.png?size=30px)](https://github.com/sbejaoui) | Recurring - Product Contract
 [subscription_oca](subscription_oca/) | 16.0.1.0.0 |  | Generate recurring invoices.
+[recurring_payments_stripe](recurring_payments_stripe/) | 16.0.1.0.0 |  | Pay automatically with Stripe
 
 [//]: # (end addons)
 

From ea94dbd2bc286f778d6553395ef28e77d52ffd36 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 3 Dec 2024 10:39:15 -0500
Subject: [PATCH 07/35] [FIX] Fixed precommit error

---
 recurring_payments_stripe/README.rst                    | 8 ++++++++
 recurring_payments_stripe/__manifest__.py               | 2 ++
 recurring_payments_stripe/models/account_move.py        | 4 ++--
 recurring_payments_stripe/models/sale_subscription.py   | 4 ++--
 recurring_payments_stripe/static/description/index.html | 2 ++
 5 files changed, 16 insertions(+), 4 deletions(-)

diff --git a/recurring_payments_stripe/README.rst b/recurring_payments_stripe/README.rst
index 31fcb6beb7..e4e53e726e 100644
--- a/recurring_payments_stripe/README.rst
+++ b/recurring_payments_stripe/README.rst
@@ -65,6 +65,14 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
+.. |maintainer-mjavint| image:: https://github.com/mjavint.png?size=40px
+    :target: https://github.com/mjavint
+    :alt: mjavint
+
+Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
+
+|maintainer-mjavint| 
+
 This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payments_stripe/__manifest__.py
index ca812198c9..5fd878ff46 100644
--- a/recurring_payments_stripe/__manifest__.py
+++ b/recurring_payments_stripe/__manifest__.py
@@ -13,4 +13,6 @@
     ],
     "installable": True,
     "auto_install": False,
+    "external_dependencies": {"python": ["stripe"]},
+    "maintainers": ["mjavint"],
 }
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index 14d29e011e..cdbf348265 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -2,7 +2,7 @@
 
 import stripe
 
-from odoo import api, models
+from odoo import _, api, models
 from odoo.exceptions import UserError
 
 _logger = logging.getLogger(__name__)
@@ -64,7 +64,7 @@ def action_register_payment(self):
                         invoice.payment_state = "paid"
                     elif payment_intent["status"] == "requires_action":
                         raise UserError(
-                            "Payment requires additional authentication (3D Secure)."
+                            _("Payment requires additional authentication (3D Secure).")
                         )
                     else:
                         raise UserError(
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
index ccc60eaf70..49a7614337 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -6,8 +6,8 @@
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
-    charge_automatically = fields.Boolean(string="Charge Automatically")
-    stripe_customer = fields.Char(string="Stripe Customer ID")
+    charge_automatically = fields.Boolean("Charge Automatically")
+    stripe_customer = fields.Char("Stripe Customer ID")
     provider_id = fields.Many2one(
         string="Provider_id",
         domain=[("code", "=", "stripe")],
diff --git a/recurring_payments_stripe/static/description/index.html b/recurring_payments_stripe/static/description/index.html
index d9629f4109..77b6e81dee 100644
--- a/recurring_payments_stripe/static/description/index.html
+++ b/recurring_payments_stripe/static/description/index.html
@@ -406,6 +406,8 @@ <h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
+<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
+<p><a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
 <p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe">OCA/contract</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>

From b87b47351fa12c13beeda8ea36ad763b2f09e705 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 3 Dec 2024 10:57:48 -0500
Subject: [PATCH 08/35] [FIX] Fixed precommit error

---
 recurring_payments_stripe/models/account_move.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index cdbf348265..877392b63a 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -72,7 +72,7 @@ def action_register_payment(self):
                         )
 
                 except stripe.StripeError as e:
-                    raise UserError(f"Stripe error: {e.user_message or str(e)}")
+                    raise UserError(f"Stripe error: {e}")
 
             else:
                 return super(AccountMove, self).action_register_payment()

From e0d75a05384efe36861396fa979d27e356597c46 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 3 Dec 2024 11:38:20 -0500
Subject: [PATCH 09/35] [FIX] Fixed precommit flake error

---
 recurring_payments_stripe/models/account_move.py      | 2 +-
 recurring_payments_stripe/models/sale_subscription.py | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payments_stripe/models/account_move.py
index 877392b63a..bf3bbd78ae 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payments_stripe/models/account_move.py
@@ -72,7 +72,7 @@ def action_register_payment(self):
                         )
 
                 except stripe.StripeError as e:
-                    raise UserError(f"Stripe error: {e}")
+                    raise UserError(f"Stripe error: {e}") from e
 
             else:
                 return super(AccountMove, self).action_register_payment()
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payments_stripe/models/sale_subscription.py
index 49a7614337..34f118bf13 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payments_stripe/models/sale_subscription.py
@@ -6,7 +6,7 @@
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
-    charge_automatically = fields.Boolean("Charge Automatically")
+    charge_automatically = fields.Boolean()
     stripe_customer = fields.Char("Stripe Customer ID")
     provider_id = fields.Many2one(
         string="Provider_id",

From ec6d741946b7dd9e9f6c1b4525371c8131f00eb0 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 6 Dec 2024 10:55:26 -0500
Subject: [PATCH 10/35] [FIX] Fixed translation error in account_move.py and
 spaces inside summary

---
 .../README.rst                                       | 12 ++++++------
 .../__init__.py                                      |  0
 .../__manifest__.py                                  |  4 ++--
 .../data/ir_cron.xml                                 |  2 +-
 .../models/__init__.py                               |  0
 .../models/account_move.py                           |  2 +-
 .../models/sale_subscription.py                      |  2 +-
 .../readme/CONFIGURE.rst                             |  0
 .../readme/CONTRIBUTORS.rst                          |  0
 .../readme/CREDITS.rst                               |  0
 .../readme/DESCRIPTION.rst                           |  0
 .../readme/DEVELOP.rst                               |  0
 .../readme/HISTORY.rst                               |  0
 .../readme/INSTALL.rst                               |  0
 .../readme/ROADMAP.rst                               |  0
 .../readme/USAGE.rst                                 |  0
 .../static/description/index.html                    |  6 +++---
 .../views/sale_subscription_views.xml                |  0
 18 files changed, 14 insertions(+), 14 deletions(-)
 rename {recurring_payments_stripe => recurring_payment_stripe}/README.rst (89%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/__init__.py (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/__manifest__.py (83%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/data/ir_cron.xml (86%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/models/__init__.py (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/models/account_move.py (98%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/models/sale_subscription.py (97%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/CONFIGURE.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/CONTRIBUTORS.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/CREDITS.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/DESCRIPTION.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/DEVELOP.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/HISTORY.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/INSTALL.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/ROADMAP.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/readme/USAGE.rst (100%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/static/description/index.html (91%)
 rename {recurring_payments_stripe => recurring_payment_stripe}/views/sale_subscription_views.xml (100%)

diff --git a/recurring_payments_stripe/README.rst b/recurring_payment_stripe/README.rst
similarity index 89%
rename from recurring_payments_stripe/README.rst
rename to recurring_payment_stripe/README.rst
index e4e53e726e..7bdcdd1dd7 100644
--- a/recurring_payments_stripe/README.rst
+++ b/recurring_payment_stripe/README.rst
@@ -2,7 +2,7 @@
 Recurring Payments with Stripe
 ==============================
 
-.. 
+..
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
@@ -17,10 +17,10 @@ Recurring Payments with Stripe
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github
-    :target: https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe
+    :target: https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe
     :alt: OCA/contract
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payments_stripe
+    :target: https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payment_stripe
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
     :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=16.0
@@ -40,7 +40,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/contract/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/contract/issues/new?body=module:%20recurring_payments_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/contract/issues/new?body=module:%20recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -71,8 +71,8 @@ promote its widespread use.
 
 Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 
-|maintainer-mjavint| 
+|maintainer-mjavint|
 
-This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe>`_ project on GitHub.
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/recurring_payments_stripe/__init__.py b/recurring_payment_stripe/__init__.py
similarity index 100%
rename from recurring_payments_stripe/__init__.py
rename to recurring_payment_stripe/__init__.py
diff --git a/recurring_payments_stripe/__manifest__.py b/recurring_payment_stripe/__manifest__.py
similarity index 83%
rename from recurring_payments_stripe/__manifest__.py
rename to recurring_payment_stripe/__manifest__.py
index 5fd878ff46..3981dc5e2c 100644
--- a/recurring_payments_stripe/__manifest__.py
+++ b/recurring_payment_stripe/__manifest__.py
@@ -1,7 +1,7 @@
 {
-    "name": "Recurring Payments with Stripe",
+    "name": "Recurring Payment with Stripe",
     "version": "16.0.1.0.0",
-    "summary": """ Recurring Payments with Stripe """,
+    "summary": """Recurring Payment with Stripe""",
     "author": "Binhex, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/contract",
     "license": "AGPL-3",
diff --git a/recurring_payments_stripe/data/ir_cron.xml b/recurring_payment_stripe/data/ir_cron.xml
similarity index 86%
rename from recurring_payments_stripe/data/ir_cron.xml
rename to recurring_payment_stripe/data/ir_cron.xml
index ff41276318..dc55679997 100644
--- a/recurring_payments_stripe/data/ir_cron.xml
+++ b/recurring_payment_stripe/data/ir_cron.xml
@@ -2,7 +2,7 @@
     <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
     <record id="ir_cron_process_due_invoices" model="ir.cron">
         <field name="name">Process Overdue Invoices for Subscriptions</field>
-        <field name="model_id" ref="recurring_payments_stripe.model_account_move" />
+        <field name="model_id" ref="recurring_payment_stripe.model_account_move" />
         <field name="state">code</field>
         <field name="code">model.cron_process_due_invoices()</field>
         <field name="interval_number">1</field>
diff --git a/recurring_payments_stripe/models/__init__.py b/recurring_payment_stripe/models/__init__.py
similarity index 100%
rename from recurring_payments_stripe/models/__init__.py
rename to recurring_payment_stripe/models/__init__.py
diff --git a/recurring_payments_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
similarity index 98%
rename from recurring_payments_stripe/models/account_move.py
rename to recurring_payment_stripe/models/account_move.py
index bf3bbd78ae..f8de38d9b6 100644
--- a/recurring_payments_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -42,7 +42,7 @@ def action_register_payment(self):
                         metadata={"odoo_invoice_id": str(invoice.id)},
                     )
 
-                    # Manejar el resultado del PaymentIntent
+                    # Handling the result of the PaymentIntent
                     if payment_intent["status"] == "succeeded":
                         # If the payment is successful, record the payment on the invoice
                         Payment = self.env["account.payment"].sudo()
diff --git a/recurring_payments_stripe/models/sale_subscription.py b/recurring_payment_stripe/models/sale_subscription.py
similarity index 97%
rename from recurring_payments_stripe/models/sale_subscription.py
rename to recurring_payment_stripe/models/sale_subscription.py
index 34f118bf13..3632b137ad 100644
--- a/recurring_payments_stripe/models/sale_subscription.py
+++ b/recurring_payment_stripe/models/sale_subscription.py
@@ -9,7 +9,7 @@ class SaleSubscription(models.Model):
     charge_automatically = fields.Boolean()
     stripe_customer = fields.Char("Stripe Customer ID")
     provider_id = fields.Many2one(
-        string="Provider_id",
+        string="Provider",
         domain=[("code", "=", "stripe")],
         comodel_name="payment.provider",
     )
diff --git a/recurring_payments_stripe/readme/CONFIGURE.rst b/recurring_payment_stripe/readme/CONFIGURE.rst
similarity index 100%
rename from recurring_payments_stripe/readme/CONFIGURE.rst
rename to recurring_payment_stripe/readme/CONFIGURE.rst
diff --git a/recurring_payments_stripe/readme/CONTRIBUTORS.rst b/recurring_payment_stripe/readme/CONTRIBUTORS.rst
similarity index 100%
rename from recurring_payments_stripe/readme/CONTRIBUTORS.rst
rename to recurring_payment_stripe/readme/CONTRIBUTORS.rst
diff --git a/recurring_payments_stripe/readme/CREDITS.rst b/recurring_payment_stripe/readme/CREDITS.rst
similarity index 100%
rename from recurring_payments_stripe/readme/CREDITS.rst
rename to recurring_payment_stripe/readme/CREDITS.rst
diff --git a/recurring_payments_stripe/readme/DESCRIPTION.rst b/recurring_payment_stripe/readme/DESCRIPTION.rst
similarity index 100%
rename from recurring_payments_stripe/readme/DESCRIPTION.rst
rename to recurring_payment_stripe/readme/DESCRIPTION.rst
diff --git a/recurring_payments_stripe/readme/DEVELOP.rst b/recurring_payment_stripe/readme/DEVELOP.rst
similarity index 100%
rename from recurring_payments_stripe/readme/DEVELOP.rst
rename to recurring_payment_stripe/readme/DEVELOP.rst
diff --git a/recurring_payments_stripe/readme/HISTORY.rst b/recurring_payment_stripe/readme/HISTORY.rst
similarity index 100%
rename from recurring_payments_stripe/readme/HISTORY.rst
rename to recurring_payment_stripe/readme/HISTORY.rst
diff --git a/recurring_payments_stripe/readme/INSTALL.rst b/recurring_payment_stripe/readme/INSTALL.rst
similarity index 100%
rename from recurring_payments_stripe/readme/INSTALL.rst
rename to recurring_payment_stripe/readme/INSTALL.rst
diff --git a/recurring_payments_stripe/readme/ROADMAP.rst b/recurring_payment_stripe/readme/ROADMAP.rst
similarity index 100%
rename from recurring_payments_stripe/readme/ROADMAP.rst
rename to recurring_payment_stripe/readme/ROADMAP.rst
diff --git a/recurring_payments_stripe/readme/USAGE.rst b/recurring_payment_stripe/readme/USAGE.rst
similarity index 100%
rename from recurring_payments_stripe/readme/USAGE.rst
rename to recurring_payment_stripe/readme/USAGE.rst
diff --git a/recurring_payments_stripe/static/description/index.html b/recurring_payment_stripe/static/description/index.html
similarity index 91%
rename from recurring_payments_stripe/static/description/index.html
rename to recurring_payment_stripe/static/description/index.html
index 77b6e81dee..4c10c95f85 100644
--- a/recurring_payments_stripe/static/description/index.html
+++ b/recurring_payment_stripe/static/description/index.html
@@ -369,7 +369,7 @@ <h1 class="title">Recurring Payments with Stripe</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:7aacc5b46917b14e43404143f0ccd1519d1411d580330b8eb8a8ef608fe3a5a1
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payments_stripe"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payment_stripe"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
@@ -386,7 +386,7 @@ <h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/contract/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20recurring_payments_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -408,7 +408,7 @@ <h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/recurring_payments_stripe">OCA/contract</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe">OCA/contract</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>
diff --git a/recurring_payments_stripe/views/sale_subscription_views.xml b/recurring_payment_stripe/views/sale_subscription_views.xml
similarity index 100%
rename from recurring_payments_stripe/views/sale_subscription_views.xml
rename to recurring_payment_stripe/views/sale_subscription_views.xml

From d35998530dd36c5507ec17968cff2c746ba8e9d2 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 6 Dec 2024 11:04:33 -0500
Subject: [PATCH 11/35] [FIX] Fixed translation error in account_move.py and
 spaces inside summary

---
 recurring_payment_stripe/README.rst                    | 10 +++++-----
 recurring_payment_stripe/static/description/index.html |  6 +++---
 .../odoo/addons/recurring_payment_stripe               |  1 +
 .../setup.py                                           |  0
 .../odoo/addons/recurring_payments_stripe              |  1 -
 5 files changed, 9 insertions(+), 9 deletions(-)
 create mode 120000 setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
 rename setup/{recurring_payments_stripe => recurring_payment_stripe}/setup.py (100%)
 delete mode 120000 setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe

diff --git a/recurring_payment_stripe/README.rst b/recurring_payment_stripe/README.rst
index 7bdcdd1dd7..a209ffdc5e 100644
--- a/recurring_payment_stripe/README.rst
+++ b/recurring_payment_stripe/README.rst
@@ -1,8 +1,8 @@
-==============================
-Recurring Payments with Stripe
-==============================
+=============================
+Recurring Payment with Stripe
+=============================
 
-..
+.. 
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
@@ -71,7 +71,7 @@ promote its widespread use.
 
 Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 
-|maintainer-mjavint|
+|maintainer-mjavint| 
 
 This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe>`_ project on GitHub.
 
diff --git a/recurring_payment_stripe/static/description/index.html b/recurring_payment_stripe/static/description/index.html
index 4c10c95f85..4b7bc5e21f 100644
--- a/recurring_payment_stripe/static/description/index.html
+++ b/recurring_payment_stripe/static/description/index.html
@@ -3,7 +3,7 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
 <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
-<title>Recurring Payments with Stripe</title>
+<title>Recurring Payment with Stripe</title>
 <style type="text/css">
 
 /*
@@ -360,8 +360,8 @@
 </style>
 </head>
 <body>
-<div class="document" id="recurring-payments-with-stripe">
-<h1 class="title">Recurring Payments with Stripe</h1>
+<div class="document" id="recurring-payment-with-stripe">
+<h1 class="title">Recurring Payment with Stripe</h1>
 
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
diff --git a/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe b/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
new file mode 120000
index 0000000000..a75e72c91c
--- /dev/null
+++ b/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
@@ -0,0 +1 @@
+../../../../recurring_payment_stripe
\ No newline at end of file
diff --git a/setup/recurring_payments_stripe/setup.py b/setup/recurring_payment_stripe/setup.py
similarity index 100%
rename from setup/recurring_payments_stripe/setup.py
rename to setup/recurring_payment_stripe/setup.py
diff --git a/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe b/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe
deleted file mode 120000
index 0bdeb61a53..0000000000
--- a/setup/recurring_payments_stripe/odoo/addons/recurring_payments_stripe
+++ /dev/null
@@ -1 +0,0 @@
-../../../../recurring_payments_stripe
\ No newline at end of file

From 75ac6af14d28e48721132a742e75f8476c549550 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 27 Dec 2024 07:53:12 -0500
Subject: [PATCH 12/35] [FIX] Fixed payment transaction and design modular

---
 recurring_payment_stripe/__manifest__.py      |   3 +-
 recurring_payment_stripe/models/__init__.py   |   2 +-
 .../models/account_move.py                    |  65 ++-
 .../models/sale_subscription.py               |  27 +-
 recurring_payment_stripe/tests/__init__.py    |   1 +
 .../tests/test_payment_stripe_recurring.py    | 203 +++++++++
 .../addons/subscription_recurring_payment     |   1 +
 setup/subscription_recurring_payment/setup.py |   6 +
 subscription_recurring_payment/README.rst     |  78 ++++
 subscription_recurring_payment/__init__.py    |   1 +
 .../__manifest__.py                           |  14 +
 .../models/__init__.py                        |   1 +
 .../models/sale_subscription.py               |  11 +
 .../static/description/index.html             | 414 ++++++++++++++++++
 .../views/sale_subscription_views.xml         |   0
 15 files changed, 791 insertions(+), 36 deletions(-)
 create mode 100644 recurring_payment_stripe/tests/__init__.py
 create mode 100644 recurring_payment_stripe/tests/test_payment_stripe_recurring.py
 create mode 120000 setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
 create mode 100644 setup/subscription_recurring_payment/setup.py
 create mode 100644 subscription_recurring_payment/README.rst
 create mode 100644 subscription_recurring_payment/__init__.py
 create mode 100644 subscription_recurring_payment/__manifest__.py
 create mode 100644 subscription_recurring_payment/models/__init__.py
 create mode 100644 subscription_recurring_payment/models/sale_subscription.py
 create mode 100644 subscription_recurring_payment/static/description/index.html
 rename {recurring_payment_stripe => subscription_recurring_payment}/views/sale_subscription_views.xml (100%)

diff --git a/recurring_payment_stripe/__manifest__.py b/recurring_payment_stripe/__manifest__.py
index 3981dc5e2c..185d382589 100644
--- a/recurring_payment_stripe/__manifest__.py
+++ b/recurring_payment_stripe/__manifest__.py
@@ -6,9 +6,8 @@
     "website": "https://github.com/OCA/contract",
     "license": "AGPL-3",
     "category": "Subscription Management",
-    "depends": ["subscription_oca", "payment_stripe", "payment"],
+    "depends": ["subscription_recurring_payment", "payment_stripe"],
     "data": [
-        "views/sale_subscription_views.xml",
         "data/ir_cron.xml",
     ],
     "installable": True,
diff --git a/recurring_payment_stripe/models/__init__.py b/recurring_payment_stripe/models/__init__.py
index fe83caa1a8..0146386eec 100644
--- a/recurring_payment_stripe/models/__init__.py
+++ b/recurring_payment_stripe/models/__init__.py
@@ -1,2 +1,2 @@
-from . import sale_subscription
 from . import account_move
+from . import sale_subscription
diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index f8de38d9b6..f9e3a24f50 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,6 +17,7 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
+            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -24,20 +25,19 @@ def action_register_payment(self):
             if subscription and subscription.charge_automatically:
                 provider = subscription.provider_id
                 stripe.api_key = provider.stripe_secret_key
-                token = self.env["payment.token"].search(
-                    [("provider_id", "=", provider.id)]
-                )
+                token = self._create_token(subscription)
                 try:
                     # Create the PaymentIntent and confirm it immediately
                     payment_intent = stripe.PaymentIntent.create(
-                        # Stripe usa centavos
+                        # Stripe uses cents
                         amount=int(invoice.amount_total * 100),
                         currency=invoice.currency_id.name.lower(),
                         customer=token.provider_ref,
                         payment_method=token.stripe_payment_method,
-                        # Para pagos automáticos sin intervención del usuario
+                        automatic_payment_methods={"enabled": True},
+                        # For automatic payments without user intervention
                         off_session=True,
-                        # Confirmar el PaymentIntent inmediatamente
+                        # Confirm the PaymentIntent immediately
                         confirm=True,
                         metadata={"odoo_invoice_id": str(invoice.id)},
                     )
@@ -57,7 +57,7 @@ def action_register_payment(self):
                             "payment_method_id": self.env.ref(
                                 "account.account_payment_method_manual_in"
                             ).id,
-                            "ref": f"Stripe PaymentIntent {payment_intent['id']}",
+                            "ref": f"Stripe - {payment_intent['id']}",
                         }
                         payment = Payment.create(payment_vals)
                         payment.action_post()
@@ -75,7 +75,56 @@ def action_register_payment(self):
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return super(AccountMove, self).action_register_payment()
+                return res
+
+    def _create_token(self, subscription):
+        provider = subscription.provider_id
+        # Search for an existing payment token for the given provider and partner
+        token = self.env["payment.token"].search(
+            [
+                ("provider_id", "=", provider.id),
+                ("partner_id", "=", subscription.partner_id.id),
+            ],
+            limit=1,
+        )
+
+        # If no token exists, create a new one
+        if not token:
+            stripe.api_key = provider.stripe_secret_key
+
+            # Create a new Stripe customer
+            customer = stripe.Customer.create(
+                email=subscription.partner_id.email,
+                name=subscription.partner_id.name,
+                metadata={"odoo_subscription": str(subscription.name)},
+            )
+
+            # Create a new payment token in Odoo
+            new_token = self.env["payment.token"].create(
+                {
+                    "provider_id": provider.id,
+                    "partner_id": subscription.partner_id.id,
+                    "provider_ref": customer.id,
+                    "verified": True,
+                }
+            )
+
+            # Retrieve the default payment method for the customer,
+            # or create one if it doesn't exist
+            new_token.stripe_payment_method = (
+                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
+                .data[0]
+                .id
+                if stripe.PaymentMethod.list(
+                    customer=customer.id, type="card", limit=1
+                ).data
+                else stripe.Customer.create_source(customer.id, source="tok_visa").id
+            )
+
+            # Assign the new token to the variable
+            token = new_token
+
+        return token
 
     @api.model
     def cron_process_due_invoices(self):
diff --git a/recurring_payment_stripe/models/sale_subscription.py b/recurring_payment_stripe/models/sale_subscription.py
index 3632b137ad..f2a0874db0 100644
--- a/recurring_payment_stripe/models/sale_subscription.py
+++ b/recurring_payment_stripe/models/sale_subscription.py
@@ -1,35 +1,12 @@
-import stripe
-
-from odoo import api, fields, models
+from odoo import fields, models
 
 
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
-    charge_automatically = fields.Boolean()
-    stripe_customer = fields.Char("Stripe Customer ID")
+    charge_automatically = fields.Boolean(default=True)
     provider_id = fields.Many2one(
         string="Provider",
         domain=[("code", "=", "stripe")],
         comodel_name="payment.provider",
     )
-
-    def create_stripe_customer(self):
-        provider = self.provider_id
-        if provider:
-            stripe.api_key = provider.stripe_secret_key
-
-            if not self.stripe_customer:
-                customer = stripe.Customer.create(
-                    email=self.env.user.email,
-                    name=self.env.user.name,
-                    metadata={"odoo_subscription": str(self.id)},
-                )
-                self.stripe_customer = customer["id"]
-            return self.stripe_customer
-
-    @api.onchange("charge_automatically")
-    def _onchange_charge_automatically(self):
-        for record in self:
-            if record.charge_automatically:
-                record.create_stripe_customer()
diff --git a/recurring_payment_stripe/tests/__init__.py b/recurring_payment_stripe/tests/__init__.py
new file mode 100644
index 0000000000..3abe23bc2b
--- /dev/null
+++ b/recurring_payment_stripe/tests/__init__.py
@@ -0,0 +1 @@
+from . import test_payment_stripe_recurring
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
new file mode 100644
index 0000000000..355ecaedf8
--- /dev/null
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -0,0 +1,203 @@
+import uuid
+
+import stripe
+
+from odoo import fields
+from odoo.tests.common import TransactionCase
+
+
+class TestPaymentStripeRecurring(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.provider = cls.env["payment.provider"].create(
+            {
+                "name": "Stripe",
+                "code": "stripe",
+                "stripe_secret_key": "sk_test_4eC39HqLyjWDarjtT1zdp7dc",
+            }
+        )
+        cls.sale_journal = cls.env["account.journal"].search(
+            [
+                ("type", "=", "sale"),
+                ("company_id", "=", cls.env.ref("base.main_company").id),
+            ]
+        )[0]
+        cls.pricelist1 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test",
+            }
+        )
+        cls.partner = cls.env["res.partner"].create(
+            {
+                "name": "Test Partner",
+                "email": "test@example.com",
+                "property_product_pricelist": cls.pricelist1.id,
+            }
+        )
+        cls.tax_10pc_incl = cls.env["account.tax"].create(
+            {
+                "name": "10% Tax incl",
+                "amount_type": "percent",
+                "amount": 10,
+                "price_include": True,
+            }
+        )
+        cls.country = cls.env["res.country"].search([], limit=1)
+        cls.fiscal = cls.env["account.fiscal.position"].create(
+            {
+                "name": "Regime National",
+                "auto_apply": True,
+                "country_id": cls.country.id,
+                "vat_required": True,
+                "sequence": 10,
+            }
+        )
+        cls.product_1 = cls.env.ref("product.product_product_1")
+        cls.product_1.subscribable = True
+        cls.product_1.taxes_id = [(6, 0, cls.tax_10pc_incl.ids)]
+        cls.product_2 = cls.env.ref("product.product_product_2")
+        cls.product_2.subscribable = True
+        cls.pricelist2 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test 2",
+                "discount_policy": "with_discount",
+            }
+        )
+        cls.tmpl2 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "limited",
+                "recurring_rule_type": "days",
+            }
+        )
+        cls.tag = cls.env["sale.subscription.tag"].create(
+            {
+                "name": "Test Tag",
+            }
+        )
+        cls.stage = cls.env["sale.subscription.stage"].create(
+            {
+                "name": "Test Sub Stage",
+            }
+        )
+        cls.sub8 = cls.create_sub(
+            {
+                "partner_id": cls.partner.id,
+                "template_id": cls.tmpl2.id,
+                "pricelist_id": cls.pricelist2.id,
+                "date_start": fields.Date.today(),
+                "in_progress": True,
+                "journal_id": cls.sale_journal.id,
+                "company_id": 1,
+                "tag_ids": [(6, 0, [cls.tag.id])],
+                "stage_id": cls.stage.id,
+                "fiscal_position_id": cls.fiscal.id,
+                "charge_automatically": True,
+            }
+        )
+
+        cls.invoice = cls.env["account.move"].create(
+            {
+                "partner_id": cls.partner.id,
+                "move_type": "out_invoice",
+                "invoice_line_ids": [
+                    (
+                        0,
+                        0,
+                        {
+                            "name": "Test Product",
+                            "quantity": 1,
+                            "price_unit": 100.0,
+                        },
+                    )
+                ],
+                "subscription_id": cls.sub8.id,
+            }
+        )
+
+    @classmethod
+    def create_sub_template(cls, vals):
+        code = str(uuid.uuid4().hex)
+        default_vals = {
+            "name": "Test Template " + code,
+            "code": code,
+            "description": "Some sort of subscription terms",
+            "product_ids": [(6, 0, [cls.product_1.id, cls.product_2.id])],
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription.template"].create(default_vals)
+        return rec
+
+    @classmethod
+    def create_sub(cls, vals):
+        default_vals = {
+            "company_id": 1,
+            "partner_id": cls.partner.id,
+            "template_id": cls.tmpl2.id,
+            "tag_ids": [(6, 0, [cls.tag.id])],
+            "stage_id": cls.stage.id,
+            "pricelist_id": cls.pricelist1.id,
+            "fiscal_position_id": cls.fiscal.id,
+            "charge_automatically": True,
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription"].create(default_vals)
+        return rec
+
+    def test_action_register_payment(self):
+        # Set Stripe API key for testing
+        stripe.api_key = self.provider.stripe_secret_key
+
+        # Create a new Stripe customer
+        customer = stripe.Customer.create(
+            email=self.partner.email,
+            name=self.partner.name,
+        )
+
+        # Create a new payment method for the customer
+        payment_method = stripe.PaymentMethod.create(
+            type="card",
+            card={
+                "number": "4242424242424242",
+                "exp_month": 1,
+                "exp_year": 2025,
+                "cvc": "123",
+            },
+        )
+
+        # Attach the payment method to the customer
+        stripe.PaymentMethod.attach(
+            payment_method.id,
+            customer=customer.id,
+        )
+
+        token = self.invoice._create_token(subscription=self.sub8)
+        self.assertTrue(token, "Payment token was not created")
+
+        method_line = self.env["account.payment.method.line"].search(
+            [("name", "=", self.provider.name)], limit=1
+        )
+        self.assertTrue(method_line, "Payment method line was not found")
+        method = method_line.payment_method_id
+        self.assertTrue(method, "Payment method was not found")
+
+        # Check if the PaymentIntent was created
+        payment_intent = stripe.PaymentIntent.create(
+            amount=int(self.invoice.amount_total * 100),
+            currency=self.invoice.currency_id.name.lower(),
+            customer=token.provider_ref,
+            payment_method=token.stripe_payment_method,
+            off_session=True,
+            confirm=True,
+            metadata={"odoo_invoice_id": str(self.invoice.name)},
+        )
+        self.assertEqual(
+            payment_intent["status"],
+            "succeeded",
+            "PaymentIntent was not successful",
+        )
+        self.invoice.action_register_payment()
+        self.assertTrue(
+            self.invoice.payment_state == "paid",
+            "Invoice was not paid",
+        )
diff --git a/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment b/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
new file mode 120000
index 0000000000..c1cbc7f4aa
--- /dev/null
+++ b/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
@@ -0,0 +1 @@
+../../../../subscription_recurring_payment
\ No newline at end of file
diff --git a/setup/subscription_recurring_payment/setup.py b/setup/subscription_recurring_payment/setup.py
new file mode 100644
index 0000000000..28c57bb640
--- /dev/null
+++ b/setup/subscription_recurring_payment/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)
diff --git a/subscription_recurring_payment/README.rst b/subscription_recurring_payment/README.rst
new file mode 100644
index 0000000000..aa1f6895d0
--- /dev/null
+++ b/subscription_recurring_payment/README.rst
@@ -0,0 +1,78 @@
+==============================
+Subscription Recurring Payment
+==============================
+
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:d96984d6ce986386d955960fcce11f7536a2341990ea7190a7faff4157fbf398
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github
+    :target: https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment
+    :alt: OCA/contract
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/contract-16-0-dev/contract-16-0-dev-subscription_recurring_payment
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=16.0-dev
+    :alt: Try me on Runboat
+
+|badge1| |badge2| |badge3| |badge4| |badge5|
+
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/contract/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment%0Aversion:%2016.0-dev%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
+
+Credits
+=======
+
+Authors
+~~~~~~~
+
+* Binhex
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+.. |maintainer-mjavint| image:: https://github.com/mjavint.png?size=40px
+    :target: https://github.com/mjavint
+    :alt: mjavint
+
+Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
+
+|maintainer-mjavint| 
+
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/subscription_recurring_payment/__init__.py b/subscription_recurring_payment/__init__.py
new file mode 100644
index 0000000000..0650744f6b
--- /dev/null
+++ b/subscription_recurring_payment/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/subscription_recurring_payment/__manifest__.py b/subscription_recurring_payment/__manifest__.py
new file mode 100644
index 0000000000..06c5fa9138
--- /dev/null
+++ b/subscription_recurring_payment/__manifest__.py
@@ -0,0 +1,14 @@
+{
+    "name": "Subscription Recurring Payment",
+    "version": "16.0.1.0.0",
+    "summary": """ Subscription Recurring Payment """,
+    "author": "Binhex, Odoo Community Association (OCA)",
+    "website": "https://github.com/OCA/contract",
+    "license": "AGPL-3",
+    "category": "Subscription Management",
+    "depends": ["subscription_oca", "payment"],
+    "data": ["views/sale_subscription_views.xml"],
+    "installable": True,
+    "auto_install": False,
+    "maintainers": ["mjavint"],
+}
diff --git a/subscription_recurring_payment/models/__init__.py b/subscription_recurring_payment/models/__init__.py
new file mode 100644
index 0000000000..9119ef94dd
--- /dev/null
+++ b/subscription_recurring_payment/models/__init__.py
@@ -0,0 +1 @@
+from . import sale_subscription
diff --git a/subscription_recurring_payment/models/sale_subscription.py b/subscription_recurring_payment/models/sale_subscription.py
new file mode 100644
index 0000000000..7ee07e50be
--- /dev/null
+++ b/subscription_recurring_payment/models/sale_subscription.py
@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class SaleSubscription(models.Model):
+    _inherit = "sale.subscription"
+
+    charge_automatically = fields.Boolean(default=True)
+    provider_id = fields.Many2one(
+        string="Provider",
+        comodel_name="payment.provider",
+    )
diff --git a/subscription_recurring_payment/static/description/index.html b/subscription_recurring_payment/static/description/index.html
new file mode 100644
index 0000000000..7f5f69b694
--- /dev/null
+++ b/subscription_recurring_payment/static/description/index.html
@@ -0,0 +1,414 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
+<title>Subscription Recurring Payment</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: grey; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="subscription-recurring-payment">
+<h1 class="title">Subscription Recurring Payment</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:d96984d6ce986386d955960fcce11f7536a2341990ea7190a7faff4157fbf398
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0-dev/contract-16-0-dev-subscription_recurring_payment"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0-dev"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-4">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/contract/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment%0Aversion:%2016.0-dev%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
+<ul class="simple">
+<li>Binhex</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
+<p><a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment">OCA/contract</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>
diff --git a/recurring_payment_stripe/views/sale_subscription_views.xml b/subscription_recurring_payment/views/sale_subscription_views.xml
similarity index 100%
rename from recurring_payment_stripe/views/sale_subscription_views.xml
rename to subscription_recurring_payment/views/sale_subscription_views.xml

From ea452673b2f97e56bec3b15e04894485a7e5c2a9 Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Fri, 27 Dec 2024 07:56:26 -0500
Subject: [PATCH 13/35] [FIX] Fixed payment transaction and design modular (#6)

---
 recurring_payment_stripe/__manifest__.py      |   3 +-
 recurring_payment_stripe/models/__init__.py   |   2 +-
 .../models/account_move.py                    |  65 ++-
 .../models/sale_subscription.py               |  27 +-
 recurring_payment_stripe/tests/__init__.py    |   1 +
 .../tests/test_payment_stripe_recurring.py    | 203 +++++++++
 .../addons/subscription_recurring_payment     |   1 +
 setup/subscription_recurring_payment/setup.py |   6 +
 subscription_recurring_payment/README.rst     |  78 ++++
 subscription_recurring_payment/__init__.py    |   1 +
 .../__manifest__.py                           |  14 +
 .../models/__init__.py                        |   1 +
 .../models/sale_subscription.py               |  11 +
 .../static/description/index.html             | 414 ++++++++++++++++++
 .../views/sale_subscription_views.xml         |   0
 15 files changed, 791 insertions(+), 36 deletions(-)
 create mode 100644 recurring_payment_stripe/tests/__init__.py
 create mode 100644 recurring_payment_stripe/tests/test_payment_stripe_recurring.py
 create mode 120000 setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
 create mode 100644 setup/subscription_recurring_payment/setup.py
 create mode 100644 subscription_recurring_payment/README.rst
 create mode 100644 subscription_recurring_payment/__init__.py
 create mode 100644 subscription_recurring_payment/__manifest__.py
 create mode 100644 subscription_recurring_payment/models/__init__.py
 create mode 100644 subscription_recurring_payment/models/sale_subscription.py
 create mode 100644 subscription_recurring_payment/static/description/index.html
 rename {recurring_payment_stripe => subscription_recurring_payment}/views/sale_subscription_views.xml (100%)

diff --git a/recurring_payment_stripe/__manifest__.py b/recurring_payment_stripe/__manifest__.py
index 3981dc5e2c..185d382589 100644
--- a/recurring_payment_stripe/__manifest__.py
+++ b/recurring_payment_stripe/__manifest__.py
@@ -6,9 +6,8 @@
     "website": "https://github.com/OCA/contract",
     "license": "AGPL-3",
     "category": "Subscription Management",
-    "depends": ["subscription_oca", "payment_stripe", "payment"],
+    "depends": ["subscription_recurring_payment", "payment_stripe"],
     "data": [
-        "views/sale_subscription_views.xml",
         "data/ir_cron.xml",
     ],
     "installable": True,
diff --git a/recurring_payment_stripe/models/__init__.py b/recurring_payment_stripe/models/__init__.py
index fe83caa1a8..0146386eec 100644
--- a/recurring_payment_stripe/models/__init__.py
+++ b/recurring_payment_stripe/models/__init__.py
@@ -1,2 +1,2 @@
-from . import sale_subscription
 from . import account_move
+from . import sale_subscription
diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index f8de38d9b6..f9e3a24f50 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,6 +17,7 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
+            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -24,20 +25,19 @@ def action_register_payment(self):
             if subscription and subscription.charge_automatically:
                 provider = subscription.provider_id
                 stripe.api_key = provider.stripe_secret_key
-                token = self.env["payment.token"].search(
-                    [("provider_id", "=", provider.id)]
-                )
+                token = self._create_token(subscription)
                 try:
                     # Create the PaymentIntent and confirm it immediately
                     payment_intent = stripe.PaymentIntent.create(
-                        # Stripe usa centavos
+                        # Stripe uses cents
                         amount=int(invoice.amount_total * 100),
                         currency=invoice.currency_id.name.lower(),
                         customer=token.provider_ref,
                         payment_method=token.stripe_payment_method,
-                        # Para pagos automáticos sin intervención del usuario
+                        automatic_payment_methods={"enabled": True},
+                        # For automatic payments without user intervention
                         off_session=True,
-                        # Confirmar el PaymentIntent inmediatamente
+                        # Confirm the PaymentIntent immediately
                         confirm=True,
                         metadata={"odoo_invoice_id": str(invoice.id)},
                     )
@@ -57,7 +57,7 @@ def action_register_payment(self):
                             "payment_method_id": self.env.ref(
                                 "account.account_payment_method_manual_in"
                             ).id,
-                            "ref": f"Stripe PaymentIntent {payment_intent['id']}",
+                            "ref": f"Stripe - {payment_intent['id']}",
                         }
                         payment = Payment.create(payment_vals)
                         payment.action_post()
@@ -75,7 +75,56 @@ def action_register_payment(self):
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return super(AccountMove, self).action_register_payment()
+                return res
+
+    def _create_token(self, subscription):
+        provider = subscription.provider_id
+        # Search for an existing payment token for the given provider and partner
+        token = self.env["payment.token"].search(
+            [
+                ("provider_id", "=", provider.id),
+                ("partner_id", "=", subscription.partner_id.id),
+            ],
+            limit=1,
+        )
+
+        # If no token exists, create a new one
+        if not token:
+            stripe.api_key = provider.stripe_secret_key
+
+            # Create a new Stripe customer
+            customer = stripe.Customer.create(
+                email=subscription.partner_id.email,
+                name=subscription.partner_id.name,
+                metadata={"odoo_subscription": str(subscription.name)},
+            )
+
+            # Create a new payment token in Odoo
+            new_token = self.env["payment.token"].create(
+                {
+                    "provider_id": provider.id,
+                    "partner_id": subscription.partner_id.id,
+                    "provider_ref": customer.id,
+                    "verified": True,
+                }
+            )
+
+            # Retrieve the default payment method for the customer,
+            # or create one if it doesn't exist
+            new_token.stripe_payment_method = (
+                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
+                .data[0]
+                .id
+                if stripe.PaymentMethod.list(
+                    customer=customer.id, type="card", limit=1
+                ).data
+                else stripe.Customer.create_source(customer.id, source="tok_visa").id
+            )
+
+            # Assign the new token to the variable
+            token = new_token
+
+        return token
 
     @api.model
     def cron_process_due_invoices(self):
diff --git a/recurring_payment_stripe/models/sale_subscription.py b/recurring_payment_stripe/models/sale_subscription.py
index 3632b137ad..f2a0874db0 100644
--- a/recurring_payment_stripe/models/sale_subscription.py
+++ b/recurring_payment_stripe/models/sale_subscription.py
@@ -1,35 +1,12 @@
-import stripe
-
-from odoo import api, fields, models
+from odoo import fields, models
 
 
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
-    charge_automatically = fields.Boolean()
-    stripe_customer = fields.Char("Stripe Customer ID")
+    charge_automatically = fields.Boolean(default=True)
     provider_id = fields.Many2one(
         string="Provider",
         domain=[("code", "=", "stripe")],
         comodel_name="payment.provider",
     )
-
-    def create_stripe_customer(self):
-        provider = self.provider_id
-        if provider:
-            stripe.api_key = provider.stripe_secret_key
-
-            if not self.stripe_customer:
-                customer = stripe.Customer.create(
-                    email=self.env.user.email,
-                    name=self.env.user.name,
-                    metadata={"odoo_subscription": str(self.id)},
-                )
-                self.stripe_customer = customer["id"]
-            return self.stripe_customer
-
-    @api.onchange("charge_automatically")
-    def _onchange_charge_automatically(self):
-        for record in self:
-            if record.charge_automatically:
-                record.create_stripe_customer()
diff --git a/recurring_payment_stripe/tests/__init__.py b/recurring_payment_stripe/tests/__init__.py
new file mode 100644
index 0000000000..3abe23bc2b
--- /dev/null
+++ b/recurring_payment_stripe/tests/__init__.py
@@ -0,0 +1 @@
+from . import test_payment_stripe_recurring
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
new file mode 100644
index 0000000000..355ecaedf8
--- /dev/null
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -0,0 +1,203 @@
+import uuid
+
+import stripe
+
+from odoo import fields
+from odoo.tests.common import TransactionCase
+
+
+class TestPaymentStripeRecurring(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.provider = cls.env["payment.provider"].create(
+            {
+                "name": "Stripe",
+                "code": "stripe",
+                "stripe_secret_key": "sk_test_4eC39HqLyjWDarjtT1zdp7dc",
+            }
+        )
+        cls.sale_journal = cls.env["account.journal"].search(
+            [
+                ("type", "=", "sale"),
+                ("company_id", "=", cls.env.ref("base.main_company").id),
+            ]
+        )[0]
+        cls.pricelist1 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test",
+            }
+        )
+        cls.partner = cls.env["res.partner"].create(
+            {
+                "name": "Test Partner",
+                "email": "test@example.com",
+                "property_product_pricelist": cls.pricelist1.id,
+            }
+        )
+        cls.tax_10pc_incl = cls.env["account.tax"].create(
+            {
+                "name": "10% Tax incl",
+                "amount_type": "percent",
+                "amount": 10,
+                "price_include": True,
+            }
+        )
+        cls.country = cls.env["res.country"].search([], limit=1)
+        cls.fiscal = cls.env["account.fiscal.position"].create(
+            {
+                "name": "Regime National",
+                "auto_apply": True,
+                "country_id": cls.country.id,
+                "vat_required": True,
+                "sequence": 10,
+            }
+        )
+        cls.product_1 = cls.env.ref("product.product_product_1")
+        cls.product_1.subscribable = True
+        cls.product_1.taxes_id = [(6, 0, cls.tax_10pc_incl.ids)]
+        cls.product_2 = cls.env.ref("product.product_product_2")
+        cls.product_2.subscribable = True
+        cls.pricelist2 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test 2",
+                "discount_policy": "with_discount",
+            }
+        )
+        cls.tmpl2 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "limited",
+                "recurring_rule_type": "days",
+            }
+        )
+        cls.tag = cls.env["sale.subscription.tag"].create(
+            {
+                "name": "Test Tag",
+            }
+        )
+        cls.stage = cls.env["sale.subscription.stage"].create(
+            {
+                "name": "Test Sub Stage",
+            }
+        )
+        cls.sub8 = cls.create_sub(
+            {
+                "partner_id": cls.partner.id,
+                "template_id": cls.tmpl2.id,
+                "pricelist_id": cls.pricelist2.id,
+                "date_start": fields.Date.today(),
+                "in_progress": True,
+                "journal_id": cls.sale_journal.id,
+                "company_id": 1,
+                "tag_ids": [(6, 0, [cls.tag.id])],
+                "stage_id": cls.stage.id,
+                "fiscal_position_id": cls.fiscal.id,
+                "charge_automatically": True,
+            }
+        )
+
+        cls.invoice = cls.env["account.move"].create(
+            {
+                "partner_id": cls.partner.id,
+                "move_type": "out_invoice",
+                "invoice_line_ids": [
+                    (
+                        0,
+                        0,
+                        {
+                            "name": "Test Product",
+                            "quantity": 1,
+                            "price_unit": 100.0,
+                        },
+                    )
+                ],
+                "subscription_id": cls.sub8.id,
+            }
+        )
+
+    @classmethod
+    def create_sub_template(cls, vals):
+        code = str(uuid.uuid4().hex)
+        default_vals = {
+            "name": "Test Template " + code,
+            "code": code,
+            "description": "Some sort of subscription terms",
+            "product_ids": [(6, 0, [cls.product_1.id, cls.product_2.id])],
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription.template"].create(default_vals)
+        return rec
+
+    @classmethod
+    def create_sub(cls, vals):
+        default_vals = {
+            "company_id": 1,
+            "partner_id": cls.partner.id,
+            "template_id": cls.tmpl2.id,
+            "tag_ids": [(6, 0, [cls.tag.id])],
+            "stage_id": cls.stage.id,
+            "pricelist_id": cls.pricelist1.id,
+            "fiscal_position_id": cls.fiscal.id,
+            "charge_automatically": True,
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription"].create(default_vals)
+        return rec
+
+    def test_action_register_payment(self):
+        # Set Stripe API key for testing
+        stripe.api_key = self.provider.stripe_secret_key
+
+        # Create a new Stripe customer
+        customer = stripe.Customer.create(
+            email=self.partner.email,
+            name=self.partner.name,
+        )
+
+        # Create a new payment method for the customer
+        payment_method = stripe.PaymentMethod.create(
+            type="card",
+            card={
+                "number": "4242424242424242",
+                "exp_month": 1,
+                "exp_year": 2025,
+                "cvc": "123",
+            },
+        )
+
+        # Attach the payment method to the customer
+        stripe.PaymentMethod.attach(
+            payment_method.id,
+            customer=customer.id,
+        )
+
+        token = self.invoice._create_token(subscription=self.sub8)
+        self.assertTrue(token, "Payment token was not created")
+
+        method_line = self.env["account.payment.method.line"].search(
+            [("name", "=", self.provider.name)], limit=1
+        )
+        self.assertTrue(method_line, "Payment method line was not found")
+        method = method_line.payment_method_id
+        self.assertTrue(method, "Payment method was not found")
+
+        # Check if the PaymentIntent was created
+        payment_intent = stripe.PaymentIntent.create(
+            amount=int(self.invoice.amount_total * 100),
+            currency=self.invoice.currency_id.name.lower(),
+            customer=token.provider_ref,
+            payment_method=token.stripe_payment_method,
+            off_session=True,
+            confirm=True,
+            metadata={"odoo_invoice_id": str(self.invoice.name)},
+        )
+        self.assertEqual(
+            payment_intent["status"],
+            "succeeded",
+            "PaymentIntent was not successful",
+        )
+        self.invoice.action_register_payment()
+        self.assertTrue(
+            self.invoice.payment_state == "paid",
+            "Invoice was not paid",
+        )
diff --git a/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment b/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
new file mode 120000
index 0000000000..c1cbc7f4aa
--- /dev/null
+++ b/setup/subscription_recurring_payment/odoo/addons/subscription_recurring_payment
@@ -0,0 +1 @@
+../../../../subscription_recurring_payment
\ No newline at end of file
diff --git a/setup/subscription_recurring_payment/setup.py b/setup/subscription_recurring_payment/setup.py
new file mode 100644
index 0000000000..28c57bb640
--- /dev/null
+++ b/setup/subscription_recurring_payment/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)
diff --git a/subscription_recurring_payment/README.rst b/subscription_recurring_payment/README.rst
new file mode 100644
index 0000000000..aa1f6895d0
--- /dev/null
+++ b/subscription_recurring_payment/README.rst
@@ -0,0 +1,78 @@
+==============================
+Subscription Recurring Payment
+==============================
+
+.. 
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! This file is generated by oca-gen-addon-readme !!
+   !! changes will be overwritten.                   !!
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+   !! source digest: sha256:d96984d6ce986386d955960fcce11f7536a2341990ea7190a7faff4157fbf398
+   !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+
+.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
+    :target: https://odoo-community.org/page/development-status
+    :alt: Beta
+.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+    :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
+    :alt: License: AGPL-3
+.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github
+    :target: https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment
+    :alt: OCA/contract
+.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
+    :target: https://translation.odoo-community.org/projects/contract-16-0-dev/contract-16-0-dev-subscription_recurring_payment
+    :alt: Translate me on Weblate
+.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=16.0-dev
+    :alt: Try me on Runboat
+
+|badge1| |badge2| |badge3| |badge4| |badge5|
+
+
+**Table of contents**
+
+.. contents::
+   :local:
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues <https://github.com/OCA/contract/issues>`_.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+`feedback <https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment%0Aversion:%2016.0-dev%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+
+Do not contact contributors directly about support or help with technical issues.
+
+Credits
+=======
+
+Authors
+~~~~~~~
+
+* Binhex
+
+Maintainers
+~~~~~~~~~~~
+
+This module is maintained by the OCA.
+
+.. image:: https://odoo-community.org/logo.png
+   :alt: Odoo Community Association
+   :target: https://odoo-community.org
+
+OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.
+
+.. |maintainer-mjavint| image:: https://github.com/mjavint.png?size=40px
+    :target: https://github.com/mjavint
+    :alt: mjavint
+
+Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
+
+|maintainer-mjavint| 
+
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment>`_ project on GitHub.
+
+You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/subscription_recurring_payment/__init__.py b/subscription_recurring_payment/__init__.py
new file mode 100644
index 0000000000..0650744f6b
--- /dev/null
+++ b/subscription_recurring_payment/__init__.py
@@ -0,0 +1 @@
+from . import models
diff --git a/subscription_recurring_payment/__manifest__.py b/subscription_recurring_payment/__manifest__.py
new file mode 100644
index 0000000000..06c5fa9138
--- /dev/null
+++ b/subscription_recurring_payment/__manifest__.py
@@ -0,0 +1,14 @@
+{
+    "name": "Subscription Recurring Payment",
+    "version": "16.0.1.0.0",
+    "summary": """ Subscription Recurring Payment """,
+    "author": "Binhex, Odoo Community Association (OCA)",
+    "website": "https://github.com/OCA/contract",
+    "license": "AGPL-3",
+    "category": "Subscription Management",
+    "depends": ["subscription_oca", "payment"],
+    "data": ["views/sale_subscription_views.xml"],
+    "installable": True,
+    "auto_install": False,
+    "maintainers": ["mjavint"],
+}
diff --git a/subscription_recurring_payment/models/__init__.py b/subscription_recurring_payment/models/__init__.py
new file mode 100644
index 0000000000..9119ef94dd
--- /dev/null
+++ b/subscription_recurring_payment/models/__init__.py
@@ -0,0 +1 @@
+from . import sale_subscription
diff --git a/subscription_recurring_payment/models/sale_subscription.py b/subscription_recurring_payment/models/sale_subscription.py
new file mode 100644
index 0000000000..7ee07e50be
--- /dev/null
+++ b/subscription_recurring_payment/models/sale_subscription.py
@@ -0,0 +1,11 @@
+from odoo import fields, models
+
+
+class SaleSubscription(models.Model):
+    _inherit = "sale.subscription"
+
+    charge_automatically = fields.Boolean(default=True)
+    provider_id = fields.Many2one(
+        string="Provider",
+        comodel_name="payment.provider",
+    )
diff --git a/subscription_recurring_payment/static/description/index.html b/subscription_recurring_payment/static/description/index.html
new file mode 100644
index 0000000000..7f5f69b694
--- /dev/null
+++ b/subscription_recurring_payment/static/description/index.html
@@ -0,0 +1,414 @@
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
+<head>
+<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+<meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
+<title>Subscription Recurring Payment</title>
+<style type="text/css">
+
+/*
+:Author: David Goodger (goodger@python.org)
+:Id: $Id: html4css1.css 8954 2022-01-20 10:10:25Z milde $
+:Copyright: This stylesheet has been placed in the public domain.
+
+Default cascading style sheet for the HTML output of Docutils.
+
+See https://docutils.sourceforge.io/docs/howto/html-stylesheets.html for how to
+customize this style sheet.
+*/
+
+/* used to remove borders from tables and images */
+.borderless, table.borderless td, table.borderless th {
+  border: 0 }
+
+table.borderless td, table.borderless th {
+  /* Override padding for "table.docutils td" with "! important".
+     The right padding separates the table cells. */
+  padding: 0 0.5em 0 0 ! important }
+
+.first {
+  /* Override more specific margin styles with "! important". */
+  margin-top: 0 ! important }
+
+.last, .with-subtitle {
+  margin-bottom: 0 ! important }
+
+.hidden {
+  display: none }
+
+.subscript {
+  vertical-align: sub;
+  font-size: smaller }
+
+.superscript {
+  vertical-align: super;
+  font-size: smaller }
+
+a.toc-backref {
+  text-decoration: none ;
+  color: black }
+
+blockquote.epigraph {
+  margin: 2em 5em ; }
+
+dl.docutils dd {
+  margin-bottom: 0.5em }
+
+object[type="image/svg+xml"], object[type="application/x-shockwave-flash"] {
+  overflow: hidden;
+}
+
+/* Uncomment (and remove this text!) to get bold-faced definition list terms
+dl.docutils dt {
+  font-weight: bold }
+*/
+
+div.abstract {
+  margin: 2em 5em }
+
+div.abstract p.topic-title {
+  font-weight: bold ;
+  text-align: center }
+
+div.admonition, div.attention, div.caution, div.danger, div.error,
+div.hint, div.important, div.note, div.tip, div.warning {
+  margin: 2em ;
+  border: medium outset ;
+  padding: 1em }
+
+div.admonition p.admonition-title, div.hint p.admonition-title,
+div.important p.admonition-title, div.note p.admonition-title,
+div.tip p.admonition-title {
+  font-weight: bold ;
+  font-family: sans-serif }
+
+div.attention p.admonition-title, div.caution p.admonition-title,
+div.danger p.admonition-title, div.error p.admonition-title,
+div.warning p.admonition-title, .code .error {
+  color: red ;
+  font-weight: bold ;
+  font-family: sans-serif }
+
+/* Uncomment (and remove this text!) to get reduced vertical space in
+   compound paragraphs.
+div.compound .compound-first, div.compound .compound-middle {
+  margin-bottom: 0.5em }
+
+div.compound .compound-last, div.compound .compound-middle {
+  margin-top: 0.5em }
+*/
+
+div.dedication {
+  margin: 2em 5em ;
+  text-align: center ;
+  font-style: italic }
+
+div.dedication p.topic-title {
+  font-weight: bold ;
+  font-style: normal }
+
+div.figure {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+div.footer, div.header {
+  clear: both;
+  font-size: smaller }
+
+div.line-block {
+  display: block ;
+  margin-top: 1em ;
+  margin-bottom: 1em }
+
+div.line-block div.line-block {
+  margin-top: 0 ;
+  margin-bottom: 0 ;
+  margin-left: 1.5em }
+
+div.sidebar {
+  margin: 0 0 0.5em 1em ;
+  border: medium outset ;
+  padding: 1em ;
+  background-color: #ffffee ;
+  width: 40% ;
+  float: right ;
+  clear: right }
+
+div.sidebar p.rubric {
+  font-family: sans-serif ;
+  font-size: medium }
+
+div.system-messages {
+  margin: 5em }
+
+div.system-messages h1 {
+  color: red }
+
+div.system-message {
+  border: medium outset ;
+  padding: 1em }
+
+div.system-message p.system-message-title {
+  color: red ;
+  font-weight: bold }
+
+div.topic {
+  margin: 2em }
+
+h1.section-subtitle, h2.section-subtitle, h3.section-subtitle,
+h4.section-subtitle, h5.section-subtitle, h6.section-subtitle {
+  margin-top: 0.4em }
+
+h1.title {
+  text-align: center }
+
+h2.subtitle {
+  text-align: center }
+
+hr.docutils {
+  width: 75% }
+
+img.align-left, .figure.align-left, object.align-left, table.align-left {
+  clear: left ;
+  float: left ;
+  margin-right: 1em }
+
+img.align-right, .figure.align-right, object.align-right, table.align-right {
+  clear: right ;
+  float: right ;
+  margin-left: 1em }
+
+img.align-center, .figure.align-center, object.align-center {
+  display: block;
+  margin-left: auto;
+  margin-right: auto;
+}
+
+table.align-center {
+  margin-left: auto;
+  margin-right: auto;
+}
+
+.align-left {
+  text-align: left }
+
+.align-center {
+  clear: both ;
+  text-align: center }
+
+.align-right {
+  text-align: right }
+
+/* reset inner alignment in figures */
+div.align-right {
+  text-align: inherit }
+
+/* div.align-center * { */
+/*   text-align: left } */
+
+.align-top    {
+  vertical-align: top }
+
+.align-middle {
+  vertical-align: middle }
+
+.align-bottom {
+  vertical-align: bottom }
+
+ol.simple, ul.simple {
+  margin-bottom: 1em }
+
+ol.arabic {
+  list-style: decimal }
+
+ol.loweralpha {
+  list-style: lower-alpha }
+
+ol.upperalpha {
+  list-style: upper-alpha }
+
+ol.lowerroman {
+  list-style: lower-roman }
+
+ol.upperroman {
+  list-style: upper-roman }
+
+p.attribution {
+  text-align: right ;
+  margin-left: 50% }
+
+p.caption {
+  font-style: italic }
+
+p.credits {
+  font-style: italic ;
+  font-size: smaller }
+
+p.label {
+  white-space: nowrap }
+
+p.rubric {
+  font-weight: bold ;
+  font-size: larger ;
+  color: maroon ;
+  text-align: center }
+
+p.sidebar-title {
+  font-family: sans-serif ;
+  font-weight: bold ;
+  font-size: larger }
+
+p.sidebar-subtitle {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+p.topic-title {
+  font-weight: bold }
+
+pre.address {
+  margin-bottom: 0 ;
+  margin-top: 0 ;
+  font: inherit }
+
+pre.literal-block, pre.doctest-block, pre.math, pre.code {
+  margin-left: 2em ;
+  margin-right: 2em }
+
+pre.code .ln { color: grey; } /* line numbers */
+pre.code, code { background-color: #eeeeee }
+pre.code .comment, code .comment { color: #5C6576 }
+pre.code .keyword, code .keyword { color: #3B0D06; font-weight: bold }
+pre.code .literal.string, code .literal.string { color: #0C5404 }
+pre.code .name.builtin, code .name.builtin { color: #352B84 }
+pre.code .deleted, code .deleted { background-color: #DEB0A1}
+pre.code .inserted, code .inserted { background-color: #A3D289}
+
+span.classifier {
+  font-family: sans-serif ;
+  font-style: oblique }
+
+span.classifier-delimiter {
+  font-family: sans-serif ;
+  font-weight: bold }
+
+span.interpreted {
+  font-family: sans-serif }
+
+span.option {
+  white-space: nowrap }
+
+span.pre {
+  white-space: pre }
+
+span.problematic {
+  color: red }
+
+span.section-subtitle {
+  /* font-size relative to parent (h1..h6 element) */
+  font-size: 80% }
+
+table.citation {
+  border-left: solid 1px gray;
+  margin-left: 1px }
+
+table.docinfo {
+  margin: 2em 4em }
+
+table.docutils {
+  margin-top: 0.5em ;
+  margin-bottom: 0.5em }
+
+table.footnote {
+  border-left: solid 1px black;
+  margin-left: 1px }
+
+table.docutils td, table.docutils th,
+table.docinfo td, table.docinfo th {
+  padding-left: 0.5em ;
+  padding-right: 0.5em ;
+  vertical-align: top }
+
+table.docutils th.field-name, table.docinfo th.docinfo-name {
+  font-weight: bold ;
+  text-align: left ;
+  white-space: nowrap ;
+  padding-left: 0 }
+
+/* "booktabs" style (no vertical lines) */
+table.docutils.booktabs {
+  border: 0px;
+  border-top: 2px solid;
+  border-bottom: 2px solid;
+  border-collapse: collapse;
+}
+table.docutils.booktabs * {
+  border: 0px;
+}
+table.docutils.booktabs th {
+  border-bottom: thin solid;
+  text-align: left;
+}
+
+h1 tt.docutils, h2 tt.docutils, h3 tt.docutils,
+h4 tt.docutils, h5 tt.docutils, h6 tt.docutils {
+  font-size: 100% }
+
+ul.auto-toc {
+  list-style-type: none }
+
+</style>
+</head>
+<body>
+<div class="document" id="subscription-recurring-payment">
+<h1 class="title">Subscription Recurring Payment</h1>
+
+<!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! This file is generated by oca-gen-addon-readme !!
+!! changes will be overwritten.                   !!
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
+!! source digest: sha256:d96984d6ce986386d955960fcce11f7536a2341990ea7190a7faff4157fbf398
+!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0-dev/contract-16-0-dev-subscription_recurring_payment"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0-dev"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><strong>Table of contents</strong></p>
+<div class="contents local topic" id="contents">
+<ul class="simple">
+<li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
+<li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
+<li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-4">Maintainers</a></li>
+</ul>
+</li>
+</ul>
+</div>
+<div class="section" id="bug-tracker">
+<h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
+<p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/contract/issues">GitHub Issues</a>.
+In case of trouble, please check there if your issue has already been reported.
+If you spotted it first, help us to smash it by providing a detailed and welcomed
+<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment%0Aversion:%2016.0-dev%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<p>Do not contact contributors directly about support or help with technical issues.</p>
+</div>
+<div class="section" id="credits">
+<h1><a class="toc-backref" href="#toc-entry-2">Credits</a></h1>
+<div class="section" id="authors">
+<h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
+<ul class="simple">
+<li>Binhex</li>
+</ul>
+</div>
+<div class="section" id="maintainers">
+<h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
+<p>This module is maintained by the OCA.</p>
+<a class="reference external image-reference" href="https://odoo-community.org"><img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" /></a>
+<p>OCA, or the Odoo Community Association, is a nonprofit organization whose
+mission is to support the collaborative development of Odoo features and
+promote its widespread use.</p>
+<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
+<p><a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0-dev/subscription_recurring_payment">OCA/contract</a> project on GitHub.</p>
+<p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
+</div>
+</div>
+</div>
+</body>
+</html>
diff --git a/recurring_payment_stripe/views/sale_subscription_views.xml b/subscription_recurring_payment/views/sale_subscription_views.xml
similarity index 100%
rename from recurring_payment_stripe/views/sale_subscription_views.xml
rename to subscription_recurring_payment/views/sale_subscription_views.xml

From deb9ee4e92d223669dde8d530f2d9c7526821f98 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 14:21:08 -0500
Subject: [PATCH 14/35] [FIX] Fixed test implementation

---
 .../tests/test_payment_stripe_recurring.py    | 27 +------------------
 1 file changed, 1 insertion(+), 26 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 355ecaedf8..17be77d30a 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -93,6 +93,7 @@ def setUpClass(cls):
                 "stage_id": cls.stage.id,
                 "fiscal_position_id": cls.fiscal.id,
                 "charge_automatically": True,
+                "provider_id": cls.provider.id,
             }
         )
 
@@ -145,32 +146,6 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        # Set Stripe API key for testing
-        stripe.api_key = self.provider.stripe_secret_key
-
-        # Create a new Stripe customer
-        customer = stripe.Customer.create(
-            email=self.partner.email,
-            name=self.partner.name,
-        )
-
-        # Create a new payment method for the customer
-        payment_method = stripe.PaymentMethod.create(
-            type="card",
-            card={
-                "number": "4242424242424242",
-                "exp_month": 1,
-                "exp_year": 2025,
-                "cvc": "123",
-            },
-        )
-
-        # Attach the payment method to the customer
-        stripe.PaymentMethod.attach(
-            payment_method.id,
-            customer=customer.id,
-        )
-
         token = self.invoice._create_token(subscription=self.sub8)
         self.assertTrue(token, "Payment token was not created")
 

From a38602800caff18cd1425401b9538cee5e7d5d33 Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 14:28:08 -0500
Subject: [PATCH 15/35] 16.0 dev (#7)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation
---
 .../tests/test_payment_stripe_recurring.py    | 27 +------------------
 1 file changed, 1 insertion(+), 26 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 355ecaedf8..17be77d30a 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -93,6 +93,7 @@ def setUpClass(cls):
                 "stage_id": cls.stage.id,
                 "fiscal_position_id": cls.fiscal.id,
                 "charge_automatically": True,
+                "provider_id": cls.provider.id,
             }
         )
 
@@ -145,32 +146,6 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        # Set Stripe API key for testing
-        stripe.api_key = self.provider.stripe_secret_key
-
-        # Create a new Stripe customer
-        customer = stripe.Customer.create(
-            email=self.partner.email,
-            name=self.partner.name,
-        )
-
-        # Create a new payment method for the customer
-        payment_method = stripe.PaymentMethod.create(
-            type="card",
-            card={
-                "number": "4242424242424242",
-                "exp_month": 1,
-                "exp_year": 2025,
-                "cvc": "123",
-            },
-        )
-
-        # Attach the payment method to the customer
-        stripe.PaymentMethod.attach(
-            payment_method.id,
-            customer=customer.id,
-        )
-
         token = self.invoice._create_token(subscription=self.sub8)
         self.assertTrue(token, "Payment token was not created")
 

From 2c48daade9f3cc88bbc61da09ec591cdda3a60e3 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 14:57:25 -0500
Subject: [PATCH 16/35] [FIX] Fixed test implementation

---
 .../tests/test_payment_stripe_recurring.py                 | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 17be77d30a..9ee1d5708c 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -149,13 +149,6 @@ def test_action_register_payment(self):
         token = self.invoice._create_token(subscription=self.sub8)
         self.assertTrue(token, "Payment token was not created")
 
-        method_line = self.env["account.payment.method.line"].search(
-            [("name", "=", self.provider.name)], limit=1
-        )
-        self.assertTrue(method_line, "Payment method line was not found")
-        method = method_line.payment_method_id
-        self.assertTrue(method, "Payment method was not found")
-
         # Check if the PaymentIntent was created
         payment_intent = stripe.PaymentIntent.create(
             amount=int(self.invoice.amount_total * 100),

From 71175d721b8cdf8acc5ef83514065ea0c4cafd9c Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 14:59:18 -0500
Subject: [PATCH 17/35] 16.0 dev (#8)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation
---
 .../tests/test_payment_stripe_recurring.py                 | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 17be77d30a..9ee1d5708c 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -149,13 +149,6 @@ def test_action_register_payment(self):
         token = self.invoice._create_token(subscription=self.sub8)
         self.assertTrue(token, "Payment token was not created")
 
-        method_line = self.env["account.payment.method.line"].search(
-            [("name", "=", self.provider.name)], limit=1
-        )
-        self.assertTrue(method_line, "Payment method line was not found")
-        method = method_line.payment_method_id
-        self.assertTrue(method, "Payment method was not found")
-
         # Check if the PaymentIntent was created
         payment_intent = stripe.PaymentIntent.create(
             amount=int(self.invoice.amount_total * 100),

From 94b7070878569365bb0c994538db7625f8173bc3 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 15:19:48 -0500
Subject: [PATCH 18/35] [FIX] Fixed test implementation

---
 recurring_payment_stripe/models/account_move.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index f9e3a24f50..5be564d601 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,7 +17,6 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
-            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -75,7 +74,7 @@ def action_register_payment(self):
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return res
+                return super(AccountMove, self).action_register_payment()
 
     def _create_token(self, subscription):
         provider = subscription.provider_id

From 1f8580caa256a3b0d12eaa35797d5959104ae4ec Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 15:20:45 -0500
Subject: [PATCH 19/35] 16.0 dev (#9)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation
---
 recurring_payment_stripe/models/account_move.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index f9e3a24f50..5be564d601 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,7 +17,6 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
-            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -75,7 +74,7 @@ def action_register_payment(self):
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return res
+                return super(AccountMove, self).action_register_payment()
 
     def _create_token(self, subscription):
         provider = subscription.provider_id

From 2e59f9b214d5b89c1319ae415dcda55da115711a Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 15:40:14 -0500
Subject: [PATCH 20/35] [FIX] Fixed test implementation

---
 .../models/account_move.py                    | 62 ++++++++++---------
 .../tests/test_payment_stripe_recurring.py    | 20 +-----
 2 files changed, 35 insertions(+), 47 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index 5be564d601..beff8340a2 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -42,34 +42,32 @@ def action_register_payment(self):
                     )
 
                     # Handling the result of the PaymentIntent
-                    if payment_intent["status"] == "succeeded":
-                        # If the payment is successful, record the payment on the invoice
-                        Payment = self.env["account.payment"].sudo()
-                        payment_vals = {
-                            "journal_id": self.env["account.journal"]
-                            .search([("type", "=", "bank")], limit=1)
-                            .id,
-                            "amount": invoice.amount_total,
-                            "payment_type": "inbound",
-                            "partner_type": "customer",
-                            "partner_id": invoice.partner_id.id,
-                            "payment_method_id": self.env.ref(
-                                "account.account_payment_method_manual_in"
-                            ).id,
-                            "ref": f"Stripe - {payment_intent['id']}",
-                        }
-                        payment = Payment.create(payment_vals)
-                        payment.action_post()
-                        invoice.payment_state = "paid"
-                    elif payment_intent["status"] == "requires_action":
+                    if payment_intent["status"] != "succeeded":
                         raise UserError(
-                            _("Payment requires additional authentication (3D Secure).")
-                        )
-                    else:
-                        raise UserError(
-                            f"Stripe payment error: {payment_intent['status']}"
+                            _("Payment failed with status: %s")
+                            % payment_intent["status"]
                         )
 
+                    # If the payment is successful, record the payment on
+                    # the invoice
+                    Payment = self.env["account.payment"].sudo()
+                    payment_vals = {
+                        "journal_id": self.env["account.journal"]
+                        .search([("type", "=", "bank")], limit=1)
+                        .id,
+                        "amount": invoice.amount_total,
+                        "payment_type": "inbound",
+                        "partner_type": "customer",
+                        "partner_id": invoice.partner_id.id,
+                        "payment_method_id": self.env.ref(
+                            "account.account_payment_method_manual_in"
+                        ).id,
+                        "ref": f"Stripe - {payment_intent['id']}",
+                    }
+                    payment = Payment.create(payment_vals)
+                    payment.action_post()
+                    invoice.payment_state = "paid"
+
                 except stripe.StripeError as e:
                     raise UserError(f"Stripe error: {e}") from e
 
@@ -78,7 +76,8 @@ def action_register_payment(self):
 
     def _create_token(self, subscription):
         provider = subscription.provider_id
-        # Search for an existing payment token for the given provider and partner
+        # Search for an existing payment token for the given provider and
+        # partner
         token = self.env["payment.token"].search(
             [
                 ("provider_id", "=", provider.id),
@@ -111,13 +110,20 @@ def _create_token(self, subscription):
             # Retrieve the default payment method for the customer,
             # or create one if it doesn't exist
             new_token.stripe_payment_method = (
-                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
+                stripe.PaymentMethod.list(
+                    customer=customer.id,
+                    type="card",
+                    limit=1,
+                )
                 .data[0]
                 .id
                 if stripe.PaymentMethod.list(
                     customer=customer.id, type="card", limit=1
                 ).data
-                else stripe.Customer.create_source(customer.id, source="tok_visa").id
+                else stripe.Customer.create_source(
+                    customer.id,
+                    source="tok_visa",
+                ).id
             )
 
             # Assign the new token to the variable
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 9ee1d5708c..4c93ffb873 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -146,26 +146,8 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        token = self.invoice._create_token(subscription=self.sub8)
-        self.assertTrue(token, "Payment token was not created")
-
-        # Check if the PaymentIntent was created
-        payment_intent = stripe.PaymentIntent.create(
-            amount=int(self.invoice.amount_total * 100),
-            currency=self.invoice.currency_id.name.lower(),
-            customer=token.provider_ref,
-            payment_method=token.stripe_payment_method,
-            off_session=True,
-            confirm=True,
-            metadata={"odoo_invoice_id": str(self.invoice.name)},
-        )
-        self.assertEqual(
-            payment_intent["status"],
-            "succeeded",
-            "PaymentIntent was not successful",
-        )
         self.invoice.action_register_payment()
         self.assertTrue(
             self.invoice.payment_state == "paid",
-            "Invoice was not paid",
+            f"Invoice {self.invoice.id} should be paid",
         )

From eee35fdba2abc8a808b0be837de0d7930c7f72c5 Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 15:42:22 -0500
Subject: [PATCH 21/35] 16.0 dev (#10)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation
---
 .../models/account_move.py                    | 62 ++++++++++---------
 .../tests/test_payment_stripe_recurring.py    | 20 +-----
 2 files changed, 35 insertions(+), 47 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index 5be564d601..beff8340a2 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -42,34 +42,32 @@ def action_register_payment(self):
                     )
 
                     # Handling the result of the PaymentIntent
-                    if payment_intent["status"] == "succeeded":
-                        # If the payment is successful, record the payment on the invoice
-                        Payment = self.env["account.payment"].sudo()
-                        payment_vals = {
-                            "journal_id": self.env["account.journal"]
-                            .search([("type", "=", "bank")], limit=1)
-                            .id,
-                            "amount": invoice.amount_total,
-                            "payment_type": "inbound",
-                            "partner_type": "customer",
-                            "partner_id": invoice.partner_id.id,
-                            "payment_method_id": self.env.ref(
-                                "account.account_payment_method_manual_in"
-                            ).id,
-                            "ref": f"Stripe - {payment_intent['id']}",
-                        }
-                        payment = Payment.create(payment_vals)
-                        payment.action_post()
-                        invoice.payment_state = "paid"
-                    elif payment_intent["status"] == "requires_action":
+                    if payment_intent["status"] != "succeeded":
                         raise UserError(
-                            _("Payment requires additional authentication (3D Secure).")
-                        )
-                    else:
-                        raise UserError(
-                            f"Stripe payment error: {payment_intent['status']}"
+                            _("Payment failed with status: %s")
+                            % payment_intent["status"]
                         )
 
+                    # If the payment is successful, record the payment on
+                    # the invoice
+                    Payment = self.env["account.payment"].sudo()
+                    payment_vals = {
+                        "journal_id": self.env["account.journal"]
+                        .search([("type", "=", "bank")], limit=1)
+                        .id,
+                        "amount": invoice.amount_total,
+                        "payment_type": "inbound",
+                        "partner_type": "customer",
+                        "partner_id": invoice.partner_id.id,
+                        "payment_method_id": self.env.ref(
+                            "account.account_payment_method_manual_in"
+                        ).id,
+                        "ref": f"Stripe - {payment_intent['id']}",
+                    }
+                    payment = Payment.create(payment_vals)
+                    payment.action_post()
+                    invoice.payment_state = "paid"
+
                 except stripe.StripeError as e:
                     raise UserError(f"Stripe error: {e}") from e
 
@@ -78,7 +76,8 @@ def action_register_payment(self):
 
     def _create_token(self, subscription):
         provider = subscription.provider_id
-        # Search for an existing payment token for the given provider and partner
+        # Search for an existing payment token for the given provider and
+        # partner
         token = self.env["payment.token"].search(
             [
                 ("provider_id", "=", provider.id),
@@ -111,13 +110,20 @@ def _create_token(self, subscription):
             # Retrieve the default payment method for the customer,
             # or create one if it doesn't exist
             new_token.stripe_payment_method = (
-                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
+                stripe.PaymentMethod.list(
+                    customer=customer.id,
+                    type="card",
+                    limit=1,
+                )
                 .data[0]
                 .id
                 if stripe.PaymentMethod.list(
                     customer=customer.id, type="card", limit=1
                 ).data
-                else stripe.Customer.create_source(customer.id, source="tok_visa").id
+                else stripe.Customer.create_source(
+                    customer.id,
+                    source="tok_visa",
+                ).id
             )
 
             # Assign the new token to the variable
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 9ee1d5708c..4c93ffb873 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -146,26 +146,8 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        token = self.invoice._create_token(subscription=self.sub8)
-        self.assertTrue(token, "Payment token was not created")
-
-        # Check if the PaymentIntent was created
-        payment_intent = stripe.PaymentIntent.create(
-            amount=int(self.invoice.amount_total * 100),
-            currency=self.invoice.currency_id.name.lower(),
-            customer=token.provider_ref,
-            payment_method=token.stripe_payment_method,
-            off_session=True,
-            confirm=True,
-            metadata={"odoo_invoice_id": str(self.invoice.name)},
-        )
-        self.assertEqual(
-            payment_intent["status"],
-            "succeeded",
-            "PaymentIntent was not successful",
-        )
         self.invoice.action_register_payment()
         self.assertTrue(
             self.invoice.payment_state == "paid",
-            "Invoice was not paid",
+            f"Invoice {self.invoice.id} should be paid",
         )

From 4616bf82d95c8b0c21a27947438c0f5fb9bdfbd1 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 15:45:06 -0500
Subject: [PATCH 22/35] [FIX] Fixed precommit

---
 recurring_payment_stripe/tests/test_payment_stripe_recurring.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 4c93ffb873..582b8731b0 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -1,7 +1,5 @@
 import uuid
 
-import stripe
-
 from odoo import fields
 from odoo.tests.common import TransactionCase
 

From db7356711f181cb1c4df4ff46b079bb314242370 Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 15:47:08 -0500
Subject: [PATCH 23/35] 16.0 dev (#11)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed precommit
---
 recurring_payment_stripe/tests/test_payment_stripe_recurring.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 4c93ffb873..582b8731b0 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -1,7 +1,5 @@
 import uuid
 
-import stripe
-
 from odoo import fields
 from odoo.tests.common import TransactionCase
 

From 917df24f343734b7940bf84ac4fa878228046798 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 30 Dec 2024 16:19:10 -0500
Subject: [PATCH 24/35] [FIX] Fixed implementation

---
 recurring_payment_stripe/models/account_move.py            | 1 +
 .../tests/test_payment_stripe_recurring.py                 | 7 ++++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index beff8340a2..ee5bbc2fdc 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -143,6 +143,7 @@ def cron_process_due_invoices(self):
             if subscription and subscription.charge_automatically:
                 try:
                     # Register the payment
+                    invoice.action_post()
                     invoice.action_register_payment()
                 except Exception as e:
                     _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 582b8731b0..b83adf086a 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -144,8 +144,13 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        self.invoice.action_register_payment()
+
+        self.invoice.cron_process_due_invoices()
         self.assertTrue(
             self.invoice.payment_state == "paid",
             f"Invoice {self.invoice.id} should be paid",
         )
+        self.assertTrue(
+            self.invoice.state == "posted",
+            f"Invoice {self.invoice.id} should be posted",
+        )

From 81d188b7b24bacf85c3231572e8f2262d0352ebc Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 30 Dec 2024 16:20:20 -0500
Subject: [PATCH 25/35] 16.0 dev (#12)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed precommit

* [FIX] Fixed implementation
---
 recurring_payment_stripe/models/account_move.py            | 1 +
 .../tests/test_payment_stripe_recurring.py                 | 7 ++++++-
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index beff8340a2..ee5bbc2fdc 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -143,6 +143,7 @@ def cron_process_due_invoices(self):
             if subscription and subscription.charge_automatically:
                 try:
                     # Register the payment
+                    invoice.action_post()
                     invoice.action_register_payment()
                 except Exception as e:
                     _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 582b8731b0..b83adf086a 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -144,8 +144,13 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        self.invoice.action_register_payment()
+
+        self.invoice.cron_process_due_invoices()
         self.assertTrue(
             self.invoice.payment_state == "paid",
             f"Invoice {self.invoice.id} should be paid",
         )
+        self.assertTrue(
+            self.invoice.state == "posted",
+            f"Invoice {self.invoice.id} should be posted",
+        )

From b438fcb02af180749e87a94d2b600da00dc3adea Mon Sep 17 00:00:00 2001
From: Manuel Vinent Guilarte <53498470+mjavint@users.noreply.github.com>
Date: Mon, 6 Jan 2025 09:34:48 -0500
Subject: [PATCH 26/35] 16.0 dev (#13)

* [FIX] Fixed payment transaction and design modular

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed test implementation

* [FIX] Fixed precommit

* [FIX] Fixed implementation

From 6042cd06284470a1db9a137365f33a7f21196981 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 6 Jan 2025 10:16:34 -0500
Subject: [PATCH 27/35] [FIX] Fixed implementation tests

---
 .../tests/test_payment_stripe_recurring.py    | 54 +++++++++++++++----
 1 file changed, 45 insertions(+), 9 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index b83adf086a..547aa88bf0 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -1,6 +1,9 @@
 import uuid
 
+import stripe
+
 from odoo import fields
+from odoo.exceptions import UserError
 from odoo.tests.common import TransactionCase
 
 
@@ -102,13 +105,13 @@ def setUpClass(cls):
                 "invoice_line_ids": [
                     (
                         0,
-                        0,
+                        None,
                         {
-                            "name": "Test Product",
-                            "quantity": 1,
-                            "price_unit": 100.0,
+                            "product_id": cls.product_2.id,
+                            "quantity": 3,
+                            "price_unit": 750,
                         },
-                    )
+                    ),
                 ],
                 "subscription_id": cls.sub8.id,
             }
@@ -144,13 +147,46 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-
-        self.invoice.cron_process_due_invoices()
         self.assertTrue(
-            self.invoice.payment_state == "paid",
-            f"Invoice {self.invoice.id} should be paid",
+            self.invoice.state == "draft",
+            f"Invoice {self.invoice.id} should be in draft state",
         )
+
+        with self.assertRaises(UserError) as context:
+            self.invoice.cron_process_due_invoices()
+        self.assertIn("Payment failed with status", str(context.exception))
+
+        self.invoice.cron_process_due_invoices()
         self.assertTrue(
             self.invoice.state == "posted",
             f"Invoice {self.invoice.id} should be posted",
         )
+        self.assertTrue(
+            self.invoice.payment_state == "paid",
+            f"Invoice {self.invoice.id} should be paid",
+        )
+
+    def test_stripe_payment_intent(self):
+        stripe.api_key = self.provider.stripe_secret_key
+        provider = self.sub8.provider_id
+        stripe.api_key = provider.stripe_secret_key
+        token = self.invoice._create_token(self.sub8)
+        payment_intent: stripe.PaymentIntent = None
+        with self.assertRaises(UserError) as context:
+            payment_intent = stripe.PaymentIntent.create(
+                # Stripe uses cents
+                amount=int(self.invoice.amount_total * 100),
+                currency=self.invoice.currency_id.name.lower(),
+                customer=token.provider_ref,
+                payment_method=token.stripe_payment_method,
+                automatic_payment_methods={"enabled": True},
+                # For automatic payments without user intervention
+                off_session=True,
+                # Confirm the PaymentIntent immediately
+                confirm=True,
+                metadata={"odoo_invoice_id": str(self.invoice.id)},
+            )
+        self.assertIn("Payment failed with status", str(context.exception))
+
+        # Check if the payment was successful
+        self.assertEqual(payment_intent.status, "succeeded")

From 5329d7ce19a21faae8f541c2a587598d182b5221 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Mon, 6 Jan 2025 11:43:14 -0500
Subject: [PATCH 28/35] [FIX] Fixed implementation tests branch

---
 .../tests/test_payment_stripe_recurring.py    | 36 ++++++++-----------
 1 file changed, 15 insertions(+), 21 deletions(-)

diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 547aa88bf0..d86564edd0 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -3,7 +3,6 @@
 import stripe
 
 from odoo import fields
-from odoo.exceptions import UserError
 from odoo.tests.common import TransactionCase
 
 
@@ -152,11 +151,8 @@ def test_action_register_payment(self):
             f"Invoice {self.invoice.id} should be in draft state",
         )
 
-        with self.assertRaises(UserError) as context:
-            self.invoice.cron_process_due_invoices()
-        self.assertIn("Payment failed with status", str(context.exception))
-
         self.invoice.cron_process_due_invoices()
+
         self.assertTrue(
             self.invoice.state == "posted",
             f"Invoice {self.invoice.id} should be posted",
@@ -171,22 +167,20 @@ def test_stripe_payment_intent(self):
         provider = self.sub8.provider_id
         stripe.api_key = provider.stripe_secret_key
         token = self.invoice._create_token(self.sub8)
-        payment_intent: stripe.PaymentIntent = None
-        with self.assertRaises(UserError) as context:
-            payment_intent = stripe.PaymentIntent.create(
-                # Stripe uses cents
-                amount=int(self.invoice.amount_total * 100),
-                currency=self.invoice.currency_id.name.lower(),
-                customer=token.provider_ref,
-                payment_method=token.stripe_payment_method,
-                automatic_payment_methods={"enabled": True},
-                # For automatic payments without user intervention
-                off_session=True,
-                # Confirm the PaymentIntent immediately
-                confirm=True,
-                metadata={"odoo_invoice_id": str(self.invoice.id)},
-            )
-        self.assertIn("Payment failed with status", str(context.exception))
+
+        payment_intent = stripe.PaymentIntent.create(
+            # Stripe uses cents
+            amount=int(self.invoice.amount_total * 100),
+            currency=self.invoice.currency_id.name.lower(),
+            customer=token.provider_ref,
+            payment_method=token.stripe_payment_method,
+            automatic_payment_methods={"enabled": True},
+            # For automatic payments without user intervention
+            off_session=True,
+            # Confirm the PaymentIntent immediately
+            confirm=True,
+            metadata={"odoo_invoice_id": str(self.invoice.id)},
+        )
 
         # Check if the payment was successful
         self.assertEqual(payment_intent.status, "succeeded")

From 0b8a36e8c2edf65629fff336b07a4ced9baafc08 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Thu, 14 Nov 2024 16:09:48 -0500
Subject: [PATCH 29/35] # This is a combination of 12 commits. # This is the
 1st commit message:

[FIX] Fixed recurring payments

# This is the commit message #2:

[FIX] Resolved all suggestion

# This is the commit message #3:

[ADD] Add precommit rules

# This is the commit message #4:

[ADD] Add stripe library

# This is the commit message #5:

[ADD] Add stripe library

# This is the commit message #6:

[FIX] Fixed precommit error

# This is the commit message #7:

[FIX] Fixed precommit error

# This is the commit message #8:

[FIX] Fixed precommit flake error

# This is the commit message #9:

[FIX] Fixed translation error in account_move.py and spaces inside summary

# This is the commit message #10:

[FIX] Fixed translation error in account_move.py and spaces inside summary

# This is the commit message #11:

[FIX] Fixed payment transaction and design modular

# This is the commit message #12:

[FIX] Fixed test implementation
---
 .../models/account_move.py                    | 66 +++++++++----------
 .../tests/test_payment_stripe_recurring.py    | 56 +++++++---------
 2 files changed, 54 insertions(+), 68 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index ee5bbc2fdc..f9e3a24f50 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,6 +17,7 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
+            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -42,42 +43,43 @@ def action_register_payment(self):
                     )
 
                     # Handling the result of the PaymentIntent
-                    if payment_intent["status"] != "succeeded":
+                    if payment_intent["status"] == "succeeded":
+                        # If the payment is successful, record the payment on the invoice
+                        Payment = self.env["account.payment"].sudo()
+                        payment_vals = {
+                            "journal_id": self.env["account.journal"]
+                            .search([("type", "=", "bank")], limit=1)
+                            .id,
+                            "amount": invoice.amount_total,
+                            "payment_type": "inbound",
+                            "partner_type": "customer",
+                            "partner_id": invoice.partner_id.id,
+                            "payment_method_id": self.env.ref(
+                                "account.account_payment_method_manual_in"
+                            ).id,
+                            "ref": f"Stripe - {payment_intent['id']}",
+                        }
+                        payment = Payment.create(payment_vals)
+                        payment.action_post()
+                        invoice.payment_state = "paid"
+                    elif payment_intent["status"] == "requires_action":
                         raise UserError(
-                            _("Payment failed with status: %s")
-                            % payment_intent["status"]
+                            _("Payment requires additional authentication (3D Secure).")
+                        )
+                    else:
+                        raise UserError(
+                            f"Stripe payment error: {payment_intent['status']}"
                         )
-
-                    # If the payment is successful, record the payment on
-                    # the invoice
-                    Payment = self.env["account.payment"].sudo()
-                    payment_vals = {
-                        "journal_id": self.env["account.journal"]
-                        .search([("type", "=", "bank")], limit=1)
-                        .id,
-                        "amount": invoice.amount_total,
-                        "payment_type": "inbound",
-                        "partner_type": "customer",
-                        "partner_id": invoice.partner_id.id,
-                        "payment_method_id": self.env.ref(
-                            "account.account_payment_method_manual_in"
-                        ).id,
-                        "ref": f"Stripe - {payment_intent['id']}",
-                    }
-                    payment = Payment.create(payment_vals)
-                    payment.action_post()
-                    invoice.payment_state = "paid"
 
                 except stripe.StripeError as e:
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return super(AccountMove, self).action_register_payment()
+                return res
 
     def _create_token(self, subscription):
         provider = subscription.provider_id
-        # Search for an existing payment token for the given provider and
-        # partner
+        # Search for an existing payment token for the given provider and partner
         token = self.env["payment.token"].search(
             [
                 ("provider_id", "=", provider.id),
@@ -110,20 +112,13 @@ def _create_token(self, subscription):
             # Retrieve the default payment method for the customer,
             # or create one if it doesn't exist
             new_token.stripe_payment_method = (
-                stripe.PaymentMethod.list(
-                    customer=customer.id,
-                    type="card",
-                    limit=1,
-                )
+                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
                 .data[0]
                 .id
                 if stripe.PaymentMethod.list(
                     customer=customer.id, type="card", limit=1
                 ).data
-                else stripe.Customer.create_source(
-                    customer.id,
-                    source="tok_visa",
-                ).id
+                else stripe.Customer.create_source(customer.id, source="tok_visa").id
             )
 
             # Assign the new token to the variable
@@ -143,7 +138,6 @@ def cron_process_due_invoices(self):
             if subscription and subscription.charge_automatically:
                 try:
                     # Register the payment
-                    invoice.action_post()
                     invoice.action_register_payment()
                 except Exception as e:
                     _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index d86564edd0..17be77d30a 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -104,13 +104,13 @@ def setUpClass(cls):
                 "invoice_line_ids": [
                     (
                         0,
-                        None,
+                        0,
                         {
-                            "product_id": cls.product_2.id,
-                            "quantity": 3,
-                            "price_unit": 750,
+                            "name": "Test Product",
+                            "quantity": 1,
+                            "price_unit": 100.0,
                         },
-                    ),
+                    )
                 ],
                 "subscription_id": cls.sub8.id,
             }
@@ -146,41 +146,33 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        self.assertTrue(
-            self.invoice.state == "draft",
-            f"Invoice {self.invoice.id} should be in draft state",
-        )
-
-        self.invoice.cron_process_due_invoices()
+        token = self.invoice._create_token(subscription=self.sub8)
+        self.assertTrue(token, "Payment token was not created")
 
-        self.assertTrue(
-            self.invoice.state == "posted",
-            f"Invoice {self.invoice.id} should be posted",
-        )
-        self.assertTrue(
-            self.invoice.payment_state == "paid",
-            f"Invoice {self.invoice.id} should be paid",
+        method_line = self.env["account.payment.method.line"].search(
+            [("name", "=", self.provider.name)], limit=1
         )
+        self.assertTrue(method_line, "Payment method line was not found")
+        method = method_line.payment_method_id
+        self.assertTrue(method, "Payment method was not found")
 
-    def test_stripe_payment_intent(self):
-        stripe.api_key = self.provider.stripe_secret_key
-        provider = self.sub8.provider_id
-        stripe.api_key = provider.stripe_secret_key
-        token = self.invoice._create_token(self.sub8)
-
+        # Check if the PaymentIntent was created
         payment_intent = stripe.PaymentIntent.create(
-            # Stripe uses cents
             amount=int(self.invoice.amount_total * 100),
             currency=self.invoice.currency_id.name.lower(),
             customer=token.provider_ref,
             payment_method=token.stripe_payment_method,
-            automatic_payment_methods={"enabled": True},
-            # For automatic payments without user intervention
             off_session=True,
-            # Confirm the PaymentIntent immediately
             confirm=True,
-            metadata={"odoo_invoice_id": str(self.invoice.id)},
+            metadata={"odoo_invoice_id": str(self.invoice.name)},
+        )
+        self.assertEqual(
+            payment_intent["status"],
+            "succeeded",
+            "PaymentIntent was not successful",
+        )
+        self.invoice.action_register_payment()
+        self.assertTrue(
+            self.invoice.payment_state == "paid",
+            "Invoice was not paid",
         )
-
-        # Check if the payment was successful
-        self.assertEqual(payment_intent.status, "succeeded")

From 4a833660c6805e82c22b96b9545420c6c129f96a Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Tue, 21 Jan 2025 10:41:11 -0500
Subject: [PATCH 30/35] [FIX] Fixed and squash old commit

---
 .../models/account_move.py                    | 66 ++++++++++---------
 .../tests/test_payment_stripe_recurring.py    | 56 +++++++++-------
 2 files changed, 68 insertions(+), 54 deletions(-)

diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
index f9e3a24f50..ee5bbc2fdc 100644
--- a/recurring_payment_stripe/models/account_move.py
+++ b/recurring_payment_stripe/models/account_move.py
@@ -17,7 +17,6 @@ def action_register_payment(self):
         payment on subscriptions.
         """
         for invoice in self:
-            res = super(AccountMove, self).action_register_payment()
             # Find the subscription associated with the invoice, if it exists
             subscription = invoice.subscription_id
 
@@ -43,43 +42,42 @@ def action_register_payment(self):
                     )
 
                     # Handling the result of the PaymentIntent
-                    if payment_intent["status"] == "succeeded":
-                        # If the payment is successful, record the payment on the invoice
-                        Payment = self.env["account.payment"].sudo()
-                        payment_vals = {
-                            "journal_id": self.env["account.journal"]
-                            .search([("type", "=", "bank")], limit=1)
-                            .id,
-                            "amount": invoice.amount_total,
-                            "payment_type": "inbound",
-                            "partner_type": "customer",
-                            "partner_id": invoice.partner_id.id,
-                            "payment_method_id": self.env.ref(
-                                "account.account_payment_method_manual_in"
-                            ).id,
-                            "ref": f"Stripe - {payment_intent['id']}",
-                        }
-                        payment = Payment.create(payment_vals)
-                        payment.action_post()
-                        invoice.payment_state = "paid"
-                    elif payment_intent["status"] == "requires_action":
+                    if payment_intent["status"] != "succeeded":
                         raise UserError(
-                            _("Payment requires additional authentication (3D Secure).")
-                        )
-                    else:
-                        raise UserError(
-                            f"Stripe payment error: {payment_intent['status']}"
+                            _("Payment failed with status: %s")
+                            % payment_intent["status"]
                         )
 
+                    # If the payment is successful, record the payment on
+                    # the invoice
+                    Payment = self.env["account.payment"].sudo()
+                    payment_vals = {
+                        "journal_id": self.env["account.journal"]
+                        .search([("type", "=", "bank")], limit=1)
+                        .id,
+                        "amount": invoice.amount_total,
+                        "payment_type": "inbound",
+                        "partner_type": "customer",
+                        "partner_id": invoice.partner_id.id,
+                        "payment_method_id": self.env.ref(
+                            "account.account_payment_method_manual_in"
+                        ).id,
+                        "ref": f"Stripe - {payment_intent['id']}",
+                    }
+                    payment = Payment.create(payment_vals)
+                    payment.action_post()
+                    invoice.payment_state = "paid"
+
                 except stripe.StripeError as e:
                     raise UserError(f"Stripe error: {e}") from e
 
             else:
-                return res
+                return super(AccountMove, self).action_register_payment()
 
     def _create_token(self, subscription):
         provider = subscription.provider_id
-        # Search for an existing payment token for the given provider and partner
+        # Search for an existing payment token for the given provider and
+        # partner
         token = self.env["payment.token"].search(
             [
                 ("provider_id", "=", provider.id),
@@ -112,13 +110,20 @@ def _create_token(self, subscription):
             # Retrieve the default payment method for the customer,
             # or create one if it doesn't exist
             new_token.stripe_payment_method = (
-                stripe.PaymentMethod.list(customer=customer.id, type="card", limit=1)
+                stripe.PaymentMethod.list(
+                    customer=customer.id,
+                    type="card",
+                    limit=1,
+                )
                 .data[0]
                 .id
                 if stripe.PaymentMethod.list(
                     customer=customer.id, type="card", limit=1
                 ).data
-                else stripe.Customer.create_source(customer.id, source="tok_visa").id
+                else stripe.Customer.create_source(
+                    customer.id,
+                    source="tok_visa",
+                ).id
             )
 
             # Assign the new token to the variable
@@ -138,6 +143,7 @@ def cron_process_due_invoices(self):
             if subscription and subscription.charge_automatically:
                 try:
                     # Register the payment
+                    invoice.action_post()
                     invoice.action_register_payment()
                 except Exception as e:
                     _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
index 17be77d30a..d86564edd0 100644
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
@@ -104,13 +104,13 @@ def setUpClass(cls):
                 "invoice_line_ids": [
                     (
                         0,
-                        0,
+                        None,
                         {
-                            "name": "Test Product",
-                            "quantity": 1,
-                            "price_unit": 100.0,
+                            "product_id": cls.product_2.id,
+                            "quantity": 3,
+                            "price_unit": 750,
                         },
-                    )
+                    ),
                 ],
                 "subscription_id": cls.sub8.id,
             }
@@ -146,33 +146,41 @@ def create_sub(cls, vals):
         return rec
 
     def test_action_register_payment(self):
-        token = self.invoice._create_token(subscription=self.sub8)
-        self.assertTrue(token, "Payment token was not created")
+        self.assertTrue(
+            self.invoice.state == "draft",
+            f"Invoice {self.invoice.id} should be in draft state",
+        )
+
+        self.invoice.cron_process_due_invoices()
 
-        method_line = self.env["account.payment.method.line"].search(
-            [("name", "=", self.provider.name)], limit=1
+        self.assertTrue(
+            self.invoice.state == "posted",
+            f"Invoice {self.invoice.id} should be posted",
+        )
+        self.assertTrue(
+            self.invoice.payment_state == "paid",
+            f"Invoice {self.invoice.id} should be paid",
         )
-        self.assertTrue(method_line, "Payment method line was not found")
-        method = method_line.payment_method_id
-        self.assertTrue(method, "Payment method was not found")
 
-        # Check if the PaymentIntent was created
+    def test_stripe_payment_intent(self):
+        stripe.api_key = self.provider.stripe_secret_key
+        provider = self.sub8.provider_id
+        stripe.api_key = provider.stripe_secret_key
+        token = self.invoice._create_token(self.sub8)
+
         payment_intent = stripe.PaymentIntent.create(
+            # Stripe uses cents
             amount=int(self.invoice.amount_total * 100),
             currency=self.invoice.currency_id.name.lower(),
             customer=token.provider_ref,
             payment_method=token.stripe_payment_method,
+            automatic_payment_methods={"enabled": True},
+            # For automatic payments without user intervention
             off_session=True,
+            # Confirm the PaymentIntent immediately
             confirm=True,
-            metadata={"odoo_invoice_id": str(self.invoice.name)},
-        )
-        self.assertEqual(
-            payment_intent["status"],
-            "succeeded",
-            "PaymentIntent was not successful",
-        )
-        self.invoice.action_register_payment()
-        self.assertTrue(
-            self.invoice.payment_state == "paid",
-            "Invoice was not paid",
+            metadata={"odoo_invoice_id": str(self.invoice.id)},
         )
+
+        # Check if the payment was successful
+        self.assertEqual(payment_intent.status, "succeeded")

From 8cc3e3d021eb7d0fc1e3760844f80bcfae0f67a9 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 31 Jan 2025 00:36:04 -0500
Subject: [PATCH 31/35] [FIX] Implementations used payment.token stripe

---
 recurring_payment_stripe/__init__.py          |   1 -
 .../models/account_move.py                    | 149 ----------
 .../models/sale_subscription.py               |  12 -
 recurring_payment_stripe/tests/__init__.py    |   1 -
 .../tests/test_payment_stripe_recurring.py    | 186 -------------
 requirements.txt                              |   1 -
 .../odoo/addons/recurring_payment_stripe      |   1 -
 .../subscription_recurring_payment_stripe     |   1 +
 .../setup.py                                  |   0
 .../README.rst                                |  23 +-
 .../__init__.py                               |   2 +
 .../__manifest__.py                           |   8 +-
 .../data/ir_cron.xml                          |  16 +-
 .../models/__init__.py                        |   3 +-
 .../models/account_move.py                    |  72 +++++
 .../models/sale_subscription.py               |  10 +
 .../readme/CONFIGURE.rst                      |   0
 .../readme/CONTRIBUTORS.rst                   |   0
 .../readme/CREDITS.rst                        |   0
 .../readme/DESCRIPTION.rst                    |   0
 .../readme/DEVELOP.rst                        |   0
 .../readme/HISTORY.rst                        |   0
 .../readme/INSTALL.rst                        |   0
 .../readme/ROADMAP.rst                        |   0
 .../readme/USAGE.rst                          |   0
 .../static/description/index.html             |  18 +-
 .../tests/__init__.py                         |   3 +
 .../tests/common.py                           | 255 ++++++++++++++++++
 .../tests/test_account_move.py                |  52 ++++
 .../views/sale_subscription_views.xml         |  23 ++
 30 files changed, 455 insertions(+), 382 deletions(-)
 delete mode 100644 recurring_payment_stripe/__init__.py
 delete mode 100644 recurring_payment_stripe/models/account_move.py
 delete mode 100644 recurring_payment_stripe/models/sale_subscription.py
 delete mode 100644 recurring_payment_stripe/tests/__init__.py
 delete mode 100644 recurring_payment_stripe/tests/test_payment_stripe_recurring.py
 delete mode 120000 setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
 create mode 120000 setup/subscription_recurring_payment_stripe/odoo/addons/subscription_recurring_payment_stripe
 rename setup/{recurring_payment_stripe => subscription_recurring_payment_stripe}/setup.py (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/README.rst (72%)
 create mode 100644 subscription_recurring_payment_stripe/__init__.py
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/__manifest__.py (64%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/data/ir_cron.xml (53%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/models/__init__.py (71%)
 create mode 100644 subscription_recurring_payment_stripe/models/account_move.py
 create mode 100644 subscription_recurring_payment_stripe/models/sale_subscription.py
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/CONFIGURE.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/CONTRIBUTORS.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/CREDITS.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/DESCRIPTION.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/DEVELOP.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/HISTORY.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/INSTALL.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/ROADMAP.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/readme/USAGE.rst (100%)
 rename {recurring_payment_stripe => subscription_recurring_payment_stripe}/static/description/index.html (86%)
 create mode 100644 subscription_recurring_payment_stripe/tests/__init__.py
 create mode 100644 subscription_recurring_payment_stripe/tests/common.py
 create mode 100644 subscription_recurring_payment_stripe/tests/test_account_move.py
 create mode 100644 subscription_recurring_payment_stripe/views/sale_subscription_views.xml

diff --git a/recurring_payment_stripe/__init__.py b/recurring_payment_stripe/__init__.py
deleted file mode 100644
index 0650744f6b..0000000000
--- a/recurring_payment_stripe/__init__.py
+++ /dev/null
@@ -1 +0,0 @@
-from . import models
diff --git a/recurring_payment_stripe/models/account_move.py b/recurring_payment_stripe/models/account_move.py
deleted file mode 100644
index ee5bbc2fdc..0000000000
--- a/recurring_payment_stripe/models/account_move.py
+++ /dev/null
@@ -1,149 +0,0 @@
-import logging
-
-import stripe
-
-from odoo import _, api, models
-from odoo.exceptions import UserError
-
-_logger = logging.getLogger(__name__)
-
-
-class AccountMove(models.Model):
-    _inherit = "account.move"
-
-    def action_register_payment(self):
-        """
-        Override `action_register_payment` to automatically process Stripe
-        payment on subscriptions.
-        """
-        for invoice in self:
-            # Find the subscription associated with the invoice, if it exists
-            subscription = invoice.subscription_id
-
-            # Check if the subscription is recurring and has a payment method
-            if subscription and subscription.charge_automatically:
-                provider = subscription.provider_id
-                stripe.api_key = provider.stripe_secret_key
-                token = self._create_token(subscription)
-                try:
-                    # Create the PaymentIntent and confirm it immediately
-                    payment_intent = stripe.PaymentIntent.create(
-                        # Stripe uses cents
-                        amount=int(invoice.amount_total * 100),
-                        currency=invoice.currency_id.name.lower(),
-                        customer=token.provider_ref,
-                        payment_method=token.stripe_payment_method,
-                        automatic_payment_methods={"enabled": True},
-                        # For automatic payments without user intervention
-                        off_session=True,
-                        # Confirm the PaymentIntent immediately
-                        confirm=True,
-                        metadata={"odoo_invoice_id": str(invoice.id)},
-                    )
-
-                    # Handling the result of the PaymentIntent
-                    if payment_intent["status"] != "succeeded":
-                        raise UserError(
-                            _("Payment failed with status: %s")
-                            % payment_intent["status"]
-                        )
-
-                    # If the payment is successful, record the payment on
-                    # the invoice
-                    Payment = self.env["account.payment"].sudo()
-                    payment_vals = {
-                        "journal_id": self.env["account.journal"]
-                        .search([("type", "=", "bank")], limit=1)
-                        .id,
-                        "amount": invoice.amount_total,
-                        "payment_type": "inbound",
-                        "partner_type": "customer",
-                        "partner_id": invoice.partner_id.id,
-                        "payment_method_id": self.env.ref(
-                            "account.account_payment_method_manual_in"
-                        ).id,
-                        "ref": f"Stripe - {payment_intent['id']}",
-                    }
-                    payment = Payment.create(payment_vals)
-                    payment.action_post()
-                    invoice.payment_state = "paid"
-
-                except stripe.StripeError as e:
-                    raise UserError(f"Stripe error: {e}") from e
-
-            else:
-                return super(AccountMove, self).action_register_payment()
-
-    def _create_token(self, subscription):
-        provider = subscription.provider_id
-        # Search for an existing payment token for the given provider and
-        # partner
-        token = self.env["payment.token"].search(
-            [
-                ("provider_id", "=", provider.id),
-                ("partner_id", "=", subscription.partner_id.id),
-            ],
-            limit=1,
-        )
-
-        # If no token exists, create a new one
-        if not token:
-            stripe.api_key = provider.stripe_secret_key
-
-            # Create a new Stripe customer
-            customer = stripe.Customer.create(
-                email=subscription.partner_id.email,
-                name=subscription.partner_id.name,
-                metadata={"odoo_subscription": str(subscription.name)},
-            )
-
-            # Create a new payment token in Odoo
-            new_token = self.env["payment.token"].create(
-                {
-                    "provider_id": provider.id,
-                    "partner_id": subscription.partner_id.id,
-                    "provider_ref": customer.id,
-                    "verified": True,
-                }
-            )
-
-            # Retrieve the default payment method for the customer,
-            # or create one if it doesn't exist
-            new_token.stripe_payment_method = (
-                stripe.PaymentMethod.list(
-                    customer=customer.id,
-                    type="card",
-                    limit=1,
-                )
-                .data[0]
-                .id
-                if stripe.PaymentMethod.list(
-                    customer=customer.id, type="card", limit=1
-                ).data
-                else stripe.Customer.create_source(
-                    customer.id,
-                    source="tok_visa",
-                ).id
-            )
-
-            # Assign the new token to the variable
-            token = new_token
-
-        return token
-
-    @api.model
-    def cron_process_due_invoices(self):
-        """Process payment of overdue invoices for recurring subscriptions."""
-
-        for invoice in self:
-            # Find the subscription associated with the invoice
-            subscription = invoice.subscription_id
-
-            # Check if it's a recurring subscription with Stripe
-            if subscription and subscription.charge_automatically:
-                try:
-                    # Register the payment
-                    invoice.action_post()
-                    invoice.action_register_payment()
-                except Exception as e:
-                    _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/recurring_payment_stripe/models/sale_subscription.py b/recurring_payment_stripe/models/sale_subscription.py
deleted file mode 100644
index f2a0874db0..0000000000
--- a/recurring_payment_stripe/models/sale_subscription.py
+++ /dev/null
@@ -1,12 +0,0 @@
-from odoo import fields, models
-
-
-class SaleSubscription(models.Model):
-    _inherit = "sale.subscription"
-
-    charge_automatically = fields.Boolean(default=True)
-    provider_id = fields.Many2one(
-        string="Provider",
-        domain=[("code", "=", "stripe")],
-        comodel_name="payment.provider",
-    )
diff --git a/recurring_payment_stripe/tests/__init__.py b/recurring_payment_stripe/tests/__init__.py
deleted file mode 100644
index 3abe23bc2b..0000000000
--- a/recurring_payment_stripe/tests/__init__.py
+++ /dev/null
@@ -1 +0,0 @@
-from . import test_payment_stripe_recurring
diff --git a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py b/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
deleted file mode 100644
index d86564edd0..0000000000
--- a/recurring_payment_stripe/tests/test_payment_stripe_recurring.py
+++ /dev/null
@@ -1,186 +0,0 @@
-import uuid
-
-import stripe
-
-from odoo import fields
-from odoo.tests.common import TransactionCase
-
-
-class TestPaymentStripeRecurring(TransactionCase):
-    @classmethod
-    def setUpClass(cls):
-        super().setUpClass()
-        cls.provider = cls.env["payment.provider"].create(
-            {
-                "name": "Stripe",
-                "code": "stripe",
-                "stripe_secret_key": "sk_test_4eC39HqLyjWDarjtT1zdp7dc",
-            }
-        )
-        cls.sale_journal = cls.env["account.journal"].search(
-            [
-                ("type", "=", "sale"),
-                ("company_id", "=", cls.env.ref("base.main_company").id),
-            ]
-        )[0]
-        cls.pricelist1 = cls.env["product.pricelist"].create(
-            {
-                "name": "pricelist for contract test",
-            }
-        )
-        cls.partner = cls.env["res.partner"].create(
-            {
-                "name": "Test Partner",
-                "email": "test@example.com",
-                "property_product_pricelist": cls.pricelist1.id,
-            }
-        )
-        cls.tax_10pc_incl = cls.env["account.tax"].create(
-            {
-                "name": "10% Tax incl",
-                "amount_type": "percent",
-                "amount": 10,
-                "price_include": True,
-            }
-        )
-        cls.country = cls.env["res.country"].search([], limit=1)
-        cls.fiscal = cls.env["account.fiscal.position"].create(
-            {
-                "name": "Regime National",
-                "auto_apply": True,
-                "country_id": cls.country.id,
-                "vat_required": True,
-                "sequence": 10,
-            }
-        )
-        cls.product_1 = cls.env.ref("product.product_product_1")
-        cls.product_1.subscribable = True
-        cls.product_1.taxes_id = [(6, 0, cls.tax_10pc_incl.ids)]
-        cls.product_2 = cls.env.ref("product.product_product_2")
-        cls.product_2.subscribable = True
-        cls.pricelist2 = cls.env["product.pricelist"].create(
-            {
-                "name": "pricelist for contract test 2",
-                "discount_policy": "with_discount",
-            }
-        )
-        cls.tmpl2 = cls.create_sub_template(
-            {
-                "recurring_rule_boundary": "limited",
-                "recurring_rule_type": "days",
-            }
-        )
-        cls.tag = cls.env["sale.subscription.tag"].create(
-            {
-                "name": "Test Tag",
-            }
-        )
-        cls.stage = cls.env["sale.subscription.stage"].create(
-            {
-                "name": "Test Sub Stage",
-            }
-        )
-        cls.sub8 = cls.create_sub(
-            {
-                "partner_id": cls.partner.id,
-                "template_id": cls.tmpl2.id,
-                "pricelist_id": cls.pricelist2.id,
-                "date_start": fields.Date.today(),
-                "in_progress": True,
-                "journal_id": cls.sale_journal.id,
-                "company_id": 1,
-                "tag_ids": [(6, 0, [cls.tag.id])],
-                "stage_id": cls.stage.id,
-                "fiscal_position_id": cls.fiscal.id,
-                "charge_automatically": True,
-                "provider_id": cls.provider.id,
-            }
-        )
-
-        cls.invoice = cls.env["account.move"].create(
-            {
-                "partner_id": cls.partner.id,
-                "move_type": "out_invoice",
-                "invoice_line_ids": [
-                    (
-                        0,
-                        None,
-                        {
-                            "product_id": cls.product_2.id,
-                            "quantity": 3,
-                            "price_unit": 750,
-                        },
-                    ),
-                ],
-                "subscription_id": cls.sub8.id,
-            }
-        )
-
-    @classmethod
-    def create_sub_template(cls, vals):
-        code = str(uuid.uuid4().hex)
-        default_vals = {
-            "name": "Test Template " + code,
-            "code": code,
-            "description": "Some sort of subscription terms",
-            "product_ids": [(6, 0, [cls.product_1.id, cls.product_2.id])],
-        }
-        default_vals.update(vals)
-        rec = cls.env["sale.subscription.template"].create(default_vals)
-        return rec
-
-    @classmethod
-    def create_sub(cls, vals):
-        default_vals = {
-            "company_id": 1,
-            "partner_id": cls.partner.id,
-            "template_id": cls.tmpl2.id,
-            "tag_ids": [(6, 0, [cls.tag.id])],
-            "stage_id": cls.stage.id,
-            "pricelist_id": cls.pricelist1.id,
-            "fiscal_position_id": cls.fiscal.id,
-            "charge_automatically": True,
-        }
-        default_vals.update(vals)
-        rec = cls.env["sale.subscription"].create(default_vals)
-        return rec
-
-    def test_action_register_payment(self):
-        self.assertTrue(
-            self.invoice.state == "draft",
-            f"Invoice {self.invoice.id} should be in draft state",
-        )
-
-        self.invoice.cron_process_due_invoices()
-
-        self.assertTrue(
-            self.invoice.state == "posted",
-            f"Invoice {self.invoice.id} should be posted",
-        )
-        self.assertTrue(
-            self.invoice.payment_state == "paid",
-            f"Invoice {self.invoice.id} should be paid",
-        )
-
-    def test_stripe_payment_intent(self):
-        stripe.api_key = self.provider.stripe_secret_key
-        provider = self.sub8.provider_id
-        stripe.api_key = provider.stripe_secret_key
-        token = self.invoice._create_token(self.sub8)
-
-        payment_intent = stripe.PaymentIntent.create(
-            # Stripe uses cents
-            amount=int(self.invoice.amount_total * 100),
-            currency=self.invoice.currency_id.name.lower(),
-            customer=token.provider_ref,
-            payment_method=token.stripe_payment_method,
-            automatic_payment_methods={"enabled": True},
-            # For automatic payments without user intervention
-            off_session=True,
-            # Confirm the PaymentIntent immediately
-            confirm=True,
-            metadata={"odoo_invoice_id": str(self.invoice.id)},
-        )
-
-        # Check if the payment was successful
-        self.assertEqual(payment_intent.status, "succeeded")
diff --git a/requirements.txt b/requirements.txt
index 55fb35df0a..7d41f1be0f 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -1,3 +1,2 @@
 # generated from manifests external_dependencies
 python-dateutil
-stripe
diff --git a/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe b/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
deleted file mode 120000
index a75e72c91c..0000000000
--- a/setup/recurring_payment_stripe/odoo/addons/recurring_payment_stripe
+++ /dev/null
@@ -1 +0,0 @@
-../../../../recurring_payment_stripe
\ No newline at end of file
diff --git a/setup/subscription_recurring_payment_stripe/odoo/addons/subscription_recurring_payment_stripe b/setup/subscription_recurring_payment_stripe/odoo/addons/subscription_recurring_payment_stripe
new file mode 120000
index 0000000000..243c62a8e2
--- /dev/null
+++ b/setup/subscription_recurring_payment_stripe/odoo/addons/subscription_recurring_payment_stripe
@@ -0,0 +1 @@
+../../../../subscription_recurring_payment_stripe
\ No newline at end of file
diff --git a/setup/recurring_payment_stripe/setup.py b/setup/subscription_recurring_payment_stripe/setup.py
similarity index 100%
rename from setup/recurring_payment_stripe/setup.py
rename to setup/subscription_recurring_payment_stripe/setup.py
diff --git a/recurring_payment_stripe/README.rst b/subscription_recurring_payment_stripe/README.rst
similarity index 72%
rename from recurring_payment_stripe/README.rst
rename to subscription_recurring_payment_stripe/README.rst
index a209ffdc5e..44bf85ff1c 100644
--- a/recurring_payment_stripe/README.rst
+++ b/subscription_recurring_payment_stripe/README.rst
@@ -1,13 +1,13 @@
-=============================
-Recurring Payment with Stripe
-=============================
+==========================================
+Subscription Recurring Payment with Stripe
+==========================================
 
 .. 
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:7aacc5b46917b14e43404143f0ccd1519d1411d580330b8eb8a8ef608fe3a5a1
+   !! source digest: sha256:a04a69ddb9cd26d60deee3b1ee68e21c605708fffc07d6c6a270a5ca62c8cbad
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
@@ -17,10 +17,10 @@ Recurring Payment with Stripe
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github
-    :target: https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe
+    :target: https://github.com/OCA/contract/tree/16.0/subscription_recurring_payment_stripe
     :alt: OCA/contract
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payment_stripe
+    :target: https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-subscription_recurring_payment_stripe
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
     :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=16.0
@@ -40,7 +40,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/contract/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/contract/issues/new?body=module:%20recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -65,14 +65,17 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
+.. |maintainer-Binhex| image:: https://github.com/Binhex.png?size=40px
+    :target: https://github.com/Binhex
+    :alt: Binhex
 .. |maintainer-mjavint| image:: https://github.com/mjavint.png?size=40px
     :target: https://github.com/mjavint
     :alt: mjavint
 
-Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
+Current `maintainers <https://odoo-community.org/page/maintainer-role>`__:
 
-|maintainer-mjavint| 
+|maintainer-Binhex| |maintainer-mjavint| 
 
-This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe>`_ project on GitHub.
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/16.0/subscription_recurring_payment_stripe>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/subscription_recurring_payment_stripe/__init__.py b/subscription_recurring_payment_stripe/__init__.py
new file mode 100644
index 0000000000..a0fdc10fe1
--- /dev/null
+++ b/subscription_recurring_payment_stripe/__init__.py
@@ -0,0 +1,2 @@
+# -*- coding: utf-8 -*-
+from . import models
diff --git a/recurring_payment_stripe/__manifest__.py b/subscription_recurring_payment_stripe/__manifest__.py
similarity index 64%
rename from recurring_payment_stripe/__manifest__.py
rename to subscription_recurring_payment_stripe/__manifest__.py
index 185d382589..d394104023 100644
--- a/recurring_payment_stripe/__manifest__.py
+++ b/subscription_recurring_payment_stripe/__manifest__.py
@@ -1,17 +1,17 @@
 {
-    "name": "Recurring Payment with Stripe",
+    "name": "Subscription Recurring Payment with Stripe",
     "version": "16.0.1.0.0",
-    "summary": """Recurring Payment with Stripe""",
+    "summary": """Subscription Recurring Payment with Stripe""",
     "author": "Binhex, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/contract",
     "license": "AGPL-3",
     "category": "Subscription Management",
     "depends": ["subscription_recurring_payment", "payment_stripe"],
     "data": [
+        "views/sale_subscription_views.xml",
         "data/ir_cron.xml",
     ],
     "installable": True,
     "auto_install": False,
-    "external_dependencies": {"python": ["stripe"]},
-    "maintainers": ["mjavint"],
+    "maintainers": ["Binhex", "mjavint"],
 }
diff --git a/recurring_payment_stripe/data/ir_cron.xml b/subscription_recurring_payment_stripe/data/ir_cron.xml
similarity index 53%
rename from recurring_payment_stripe/data/ir_cron.xml
rename to subscription_recurring_payment_stripe/data/ir_cron.xml
index dc55679997..78dd2cffcb 100644
--- a/recurring_payment_stripe/data/ir_cron.xml
+++ b/subscription_recurring_payment_stripe/data/ir_cron.xml
@@ -1,13 +1,15 @@
-<odoo noupdate="1">
-    <!-- Cron Job para Procesar Facturas Vencidas Diariamente -->
+<odoo>
+    <!-- Cron Job to Process Overdue Invoices Daily -->
     <record id="ir_cron_process_due_invoices" model="ir.cron">
         <field name="name">Process Overdue Invoices for Subscriptions</field>
-        <field name="model_id" ref="recurring_payment_stripe.model_account_move" />
-        <field name="state">code</field>
-        <field name="code">model.cron_process_due_invoices()</field>
-        <field name="interval_number">1</field>
-        <field name="interval_type">days</field>
+        <field name="model_id" ref="model_account_move" />
+        <field name="user_id" ref="base.user_root" />
+        <field name="interval_number">24</field>
+        <field name="interval_type">hours</field>
+        <field name="doall" eval="False"/>
         <field name="numbercall">-1</field>
         <field name="active">True</field>
+        <field name="state">code</field>
+        <field name="code">model.cron_process_due_invoices()</field>
     </record>
 </odoo>
diff --git a/recurring_payment_stripe/models/__init__.py b/subscription_recurring_payment_stripe/models/__init__.py
similarity index 71%
rename from recurring_payment_stripe/models/__init__.py
rename to subscription_recurring_payment_stripe/models/__init__.py
index 0146386eec..82c93204e4 100644
--- a/recurring_payment_stripe/models/__init__.py
+++ b/subscription_recurring_payment_stripe/models/__init__.py
@@ -1,2 +1,3 @@
-from . import account_move
+# -*- coding: utf-8 -*-
 from . import sale_subscription
+from . import account_move
diff --git a/subscription_recurring_payment_stripe/models/account_move.py b/subscription_recurring_payment_stripe/models/account_move.py
new file mode 100644
index 0000000000..20a565778c
--- /dev/null
+++ b/subscription_recurring_payment_stripe/models/account_move.py
@@ -0,0 +1,72 @@
+import logging
+
+from odoo import models, fields
+
+_logger = logging.getLogger(__name__)
+
+
+class AccountMove(models.Model):
+    _inherit = "account.move"
+
+    def cron_process_due_invoices(self):
+        """Process payment of overdue invoices for recurring subscriptions."""
+
+        for invoice in self.search(
+            [
+                ("state", "in", ["draft"]),
+                ("invoice_date_due", "<=", fields.Date.today()),
+            ]
+        ).filtered(lambda inv: inv.subscription_id):
+            # Find the subscription associated with the invoice
+            subscription = invoice.subscription_id
+
+            # Check if it's a recurring subscription with Stripe
+            if (
+                subscription
+                and subscription.charge_automatically
+                and subscription.payment_token_id
+            ):
+                try:
+                    # Post the invoice
+                    invoice.action_post()
+
+                    # Prepare payment data
+                    provider = invoice.subscription_id.provider_id
+                    method_line = self.env["account.payment.method.line"].search(
+                        [("payment_method_id.code", "=", provider.code)],
+                        limit=1,
+                    )
+                    journal = self.env["account.journal"].search(
+                        [
+                            ("type", "in", ("bank", "cash")),
+                            ("company_id", "=", invoice.company_id.id),
+                        ],
+                        limit=1,
+                    )
+
+                    payment_register = self.env["account.payment.register"]
+
+                    payment_vals = {
+                        "currency_id": invoice.currency_id.id,
+                        "journal_id": journal.id,
+                        "company_id": invoice.company_id.id,
+                        "partner_id": invoice.partner_id.id,
+                        "communication": invoice.name,
+                        "payment_type": "inbound",
+                        "partner_type": "customer",
+                        "payment_difference_handling": "open",
+                        "writeoff_label": "Write-Off",
+                        "payment_date": fields.Date.today(),
+                        "amount": invoice.amount_total,
+                        "payment_method_line_id": method_line.id,
+                        "payment_token_id": subscription.payment_token_id.id,
+                    }
+                    # Create payment and pay the invoice
+                    payment_register.with_context(
+                        active_model="account.move",
+                        active_ids=invoice.ids,
+                        active_id=invoice.id,
+                    ).create(payment_vals).action_create_payments()
+                    _logger.info(f"Processed Due Invoice: {invoice.name}")
+                except Exception as e:
+                    _logger.error(f"Error Processing Due Invoices: {str(e)}")
diff --git a/subscription_recurring_payment_stripe/models/sale_subscription.py b/subscription_recurring_payment_stripe/models/sale_subscription.py
new file mode 100644
index 0000000000..c5f2a70d5e
--- /dev/null
+++ b/subscription_recurring_payment_stripe/models/sale_subscription.py
@@ -0,0 +1,10 @@
+from odoo import models, fields, _
+
+
+class SaleSubscription(models.Model):
+    _inherit = "sale.subscription"
+
+    payment_token_id = fields.Many2one(
+        string=_("Payment Token"),
+        comodel_name="payment.token",
+    )
diff --git a/recurring_payment_stripe/readme/CONFIGURE.rst b/subscription_recurring_payment_stripe/readme/CONFIGURE.rst
similarity index 100%
rename from recurring_payment_stripe/readme/CONFIGURE.rst
rename to subscription_recurring_payment_stripe/readme/CONFIGURE.rst
diff --git a/recurring_payment_stripe/readme/CONTRIBUTORS.rst b/subscription_recurring_payment_stripe/readme/CONTRIBUTORS.rst
similarity index 100%
rename from recurring_payment_stripe/readme/CONTRIBUTORS.rst
rename to subscription_recurring_payment_stripe/readme/CONTRIBUTORS.rst
diff --git a/recurring_payment_stripe/readme/CREDITS.rst b/subscription_recurring_payment_stripe/readme/CREDITS.rst
similarity index 100%
rename from recurring_payment_stripe/readme/CREDITS.rst
rename to subscription_recurring_payment_stripe/readme/CREDITS.rst
diff --git a/recurring_payment_stripe/readme/DESCRIPTION.rst b/subscription_recurring_payment_stripe/readme/DESCRIPTION.rst
similarity index 100%
rename from recurring_payment_stripe/readme/DESCRIPTION.rst
rename to subscription_recurring_payment_stripe/readme/DESCRIPTION.rst
diff --git a/recurring_payment_stripe/readme/DEVELOP.rst b/subscription_recurring_payment_stripe/readme/DEVELOP.rst
similarity index 100%
rename from recurring_payment_stripe/readme/DEVELOP.rst
rename to subscription_recurring_payment_stripe/readme/DEVELOP.rst
diff --git a/recurring_payment_stripe/readme/HISTORY.rst b/subscription_recurring_payment_stripe/readme/HISTORY.rst
similarity index 100%
rename from recurring_payment_stripe/readme/HISTORY.rst
rename to subscription_recurring_payment_stripe/readme/HISTORY.rst
diff --git a/recurring_payment_stripe/readme/INSTALL.rst b/subscription_recurring_payment_stripe/readme/INSTALL.rst
similarity index 100%
rename from recurring_payment_stripe/readme/INSTALL.rst
rename to subscription_recurring_payment_stripe/readme/INSTALL.rst
diff --git a/recurring_payment_stripe/readme/ROADMAP.rst b/subscription_recurring_payment_stripe/readme/ROADMAP.rst
similarity index 100%
rename from recurring_payment_stripe/readme/ROADMAP.rst
rename to subscription_recurring_payment_stripe/readme/ROADMAP.rst
diff --git a/recurring_payment_stripe/readme/USAGE.rst b/subscription_recurring_payment_stripe/readme/USAGE.rst
similarity index 100%
rename from recurring_payment_stripe/readme/USAGE.rst
rename to subscription_recurring_payment_stripe/readme/USAGE.rst
diff --git a/recurring_payment_stripe/static/description/index.html b/subscription_recurring_payment_stripe/static/description/index.html
similarity index 86%
rename from recurring_payment_stripe/static/description/index.html
rename to subscription_recurring_payment_stripe/static/description/index.html
index 4b7bc5e21f..a328ec4767 100644
--- a/recurring_payment_stripe/static/description/index.html
+++ b/subscription_recurring_payment_stripe/static/description/index.html
@@ -3,7 +3,7 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
 <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
-<title>Recurring Payment with Stripe</title>
+<title>Subscription Recurring Payment with Stripe</title>
 <style type="text/css">
 
 /*
@@ -360,16 +360,16 @@
 </style>
 </head>
 <body>
-<div class="document" id="recurring-payment-with-stripe">
-<h1 class="title">Recurring Payment with Stripe</h1>
+<div class="document" id="subscription-recurring-payment-with-stripe">
+<h1 class="title">Subscription Recurring Payment with Stripe</h1>
 
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:7aacc5b46917b14e43404143f0ccd1519d1411d580330b8eb8a8ef608fe3a5a1
+!! source digest: sha256:a04a69ddb9cd26d60deee3b1ee68e21c605708fffc07d6c6a270a5ca62c8cbad
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-recurring_payment_stripe"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/contract/tree/16.0/subscription_recurring_payment_stripe"><img alt="OCA/contract" src="https://img.shields.io/badge/github-OCA%2Fcontract-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/contract-16-0/contract-16-0-subscription_recurring_payment_stripe"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/contract&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
@@ -386,7 +386,7 @@ <h1><a class="toc-backref" href="#toc-entry-1">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/contract/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/contract/issues/new?body=module:%20subscription_recurring_payment_stripe%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -406,9 +406,9 @@ <h2><a class="toc-backref" href="#toc-entry-4">Maintainers</a></h2>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
-<p><a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/recurring_payment_stripe">OCA/contract</a> project on GitHub.</p>
+<p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainers</a>:</p>
+<p><a class="reference external image-reference" href="https://github.com/Binhex"><img alt="Binhex" src="https://github.com/Binhex.png?size=40px" /></a> <a class="reference external image-reference" href="https://github.com/mjavint"><img alt="mjavint" src="https://github.com/mjavint.png?size=40px" /></a></p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/contract/tree/16.0/subscription_recurring_payment_stripe">OCA/contract</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>
diff --git a/subscription_recurring_payment_stripe/tests/__init__.py b/subscription_recurring_payment_stripe/tests/__init__.py
new file mode 100644
index 0000000000..dd689484ae
--- /dev/null
+++ b/subscription_recurring_payment_stripe/tests/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+from . import common
+from . import test_account_move
diff --git a/subscription_recurring_payment_stripe/tests/common.py b/subscription_recurring_payment_stripe/tests/common.py
new file mode 100644
index 0000000000..a99cc09914
--- /dev/null
+++ b/subscription_recurring_payment_stripe/tests/common.py
@@ -0,0 +1,255 @@
+import uuid
+from dateutil.relativedelta import relativedelta
+from odoo import fields
+from odoo.addons.payment_stripe.tests.common import StripeCommon
+
+
+class SubscriptionRecurringPaymentStripe(StripeCommon):
+
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+
+        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
+
+        cls.portal_user = cls.env.ref("base.demo_user0")
+        cls.cash_journal = cls.env["account.journal"].search(
+            [
+                ("type", "=", "cash"),
+                ("company_id", "=", cls.env.ref("base.main_company").id),
+            ]
+        )[0]
+        cls.sale_journal = cls.env["account.journal"].search(
+            [
+                ("type", "=", "sale"),
+                ("company_id", "=", cls.env.ref("base.main_company").id),
+            ]
+        )[0]
+        cls.pricelist1 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test",
+            }
+        )
+        cls.pricelist2 = cls.env["product.pricelist"].create(
+            {
+                "name": "pricelist for contract test 2",
+                "discount_policy": "with_discount",
+            }
+        )
+        cls.partner = cls.env["res.partner"].create(
+            {
+                "name": "partner test subscription_oca",
+                "property_product_pricelist": cls.pricelist1.id,
+                "email": "demo1@demo.com",
+            }
+        )
+        cls.partner_2 = cls.env["res.partner"].create(
+            {
+                "name": "partner test subscription_oca 2",
+                "property_product_pricelist": cls.pricelist1.id,
+                "email": "demo2@demo.com",
+            }
+        )
+        cls.tax_10pc_incl = cls.env["account.tax"].create(
+            {
+                "name": "10% Tax incl",
+                "amount_type": "percent",
+                "amount": 10,
+                "price_include": True,
+            }
+        )
+        cls.product_1 = cls.env.ref("product.product_product_1")
+        cls.product_1.subscribable = True
+        cls.product_1.taxes_id = [(6, 0, cls.tax_10pc_incl.ids)]
+        cls.product_2 = cls.env.ref("product.product_product_2")
+        cls.product_2.subscribable = True
+
+        cls.country = cls.env["res.country"].search([], limit=1)
+        cls.fiscal = cls.env["account.fiscal.position"].create(
+            {
+                "name": "Regime National",
+                "auto_apply": True,
+                "country_id": cls.country.id,
+                "vat_required": True,
+                "sequence": 10,
+            }
+        )
+
+        cls.tmpl1 = cls.create_sub_template({})
+        cls.tmpl2 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "limited",
+                "recurring_rule_type": "days",
+            }
+        )
+        cls.tmpl3 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "unlimited",
+                "recurring_rule_type": "weeks",
+            }
+        )
+        cls.tmpl4 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "limited",
+                "invoicing_mode": "invoice",
+                "recurring_rule_type": "years",
+            }
+        )
+        cls.tmpl5 = cls.create_sub_template(
+            {
+                "recurring_rule_boundary": "unlimited",
+                "invoicing_mode": "invoice",
+                "recurring_rule_type": "days",
+            }
+        )
+
+        cls.stage = cls.env["sale.subscription.stage"].create(
+            {
+                "name": "Test Sub Stage",
+            }
+        )
+        cls.stage_2 = cls.env["sale.subscription.stage"].create(
+            {
+                "name": "Test Sub Stage 2",
+                "type": "pre",
+            }
+        )
+        cls.tag = cls.env["sale.subscription.tag"].create(
+            {
+                "name": "Test Tag",
+            }
+        )
+
+        cls.stripe = cls._prepare_provider(
+            "stripe",
+            update_values={
+                "stripe_secret_key": "sk_test_KJtHgNwt2KS3xM7QJPr4O5E8",
+                "stripe_publishable_key": "pk_test_QSPnimmb4ZhtkEy3Uhdm4S6J",
+                "stripe_webhook_secret": "whsec_vG1fL6CMUouQ7cObF2VJprLVXT5jBLxB",
+                "payment_icon_ids": [(5, 0, 0)],
+            },
+        )
+
+        cls.provider = cls.stripe
+        cls.token = cls.env["payment.token"].create(
+            {
+                "provider_id": cls.stripe.id,
+                "partner_id": cls.env.ref("base.res_partner_1").id,
+                "company_id": cls.env.ref("base.main_company").id,
+                "payment_details": "4242",
+                "provider_ref": "cus_LBxMCDggAFOiNR",
+            }
+        )
+
+        cls.sub1 = cls.create_sub(
+            {
+                "template_id": cls.tmpl2.id,
+                "pricelist_id": cls.pricelist2.id,
+                "date_start": fields.Date.today() - relativedelta(days=100),
+                "in_progress": True,
+                "journal_id": cls.cash_journal.id,
+                "charge_automatically": True,
+                "provider_id": cls.stripe.id,
+                "payment_token_id": cls.token.id,
+            }
+        )
+
+        cls.sub_line = cls.create_sub_line(cls.sub1)
+        cls.sub_line2 = cls.env["sale.subscription.line"].create(
+            {
+                "company_id": 1,
+                "sale_subscription_id": cls.sub1.id,
+            }
+        )
+
+        cls.close_reason = cls.env["sale.subscription.close.reason"].create(
+            {
+                "name": "Test Close Reason",
+            }
+        )
+        cls.sub_line2.read(["name", "price_unit"])
+        cls.sub_line2.unlink()
+
+        # Pricelists.
+        cls.pricelist_default = cls.env.ref("product.list0")
+        cls.pricelist_l1 = cls._create_price_list("Level 1")
+        cls.pricelist_l2 = cls._create_price_list("Level 2")
+        cls.pricelist_l3 = cls._create_price_list("Level 3")
+        cls.env["product.pricelist.item"].create(
+            {
+                "pricelist_id": cls.pricelist_l3.id,
+                "applied_on": "0_product_variant",
+                "compute_price": "formula",
+                "base": "pricelist",
+                "base_pricelist_id": cls.pricelist_l1.id,
+                "product_id": cls.product_1.id,
+            }
+        )
+        cls.env["product.pricelist.item"].create(
+            {
+                "pricelist_id": cls.pricelist_l2.id,
+                "applied_on": "3_global",
+                "compute_price": "formula",
+                "base": "pricelist",
+                "base_pricelist_id": cls.pricelist_l1.id,
+            }
+        )
+        cls.env["product.pricelist.item"].create(
+            {
+                "pricelist_id": cls.pricelist_l1.id,
+                "applied_on": "3_global",
+                "compute_price": "formula",
+                "base": "standard_price",
+                "fixed_price": 1000,
+            }
+        )
+
+    @classmethod
+    def create_sub_template(cls, vals):
+        code = str(uuid.uuid4().hex)
+        default_vals = {
+            "name": "Test Template " + code,
+            "code": code,
+            "description": "Some sort of subscription terms",
+            "product_ids": [(6, 0, [cls.product_1.id, cls.product_2.id])],
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription.template"].create(default_vals)
+        return rec
+
+    @classmethod
+    def create_sub(cls, vals):
+        default_vals = {
+            "company_id": 1,
+            "partner_id": cls.partner.id,
+            "template_id": cls.tmpl1.id,
+            "tag_ids": [(6, 0, [cls.tag.id])],
+            "stage_id": cls.stage.id,
+            "pricelist_id": cls.pricelist1.id,
+            "fiscal_position_id": cls.fiscal.id,
+        }
+        default_vals.update(vals)
+        rec = cls.env["sale.subscription"].create(default_vals)
+        return rec
+
+    @classmethod
+    def create_sub_line(cls, sub, prod=None):
+        ssl = cls.env["sale.subscription.line"].create(
+            {
+                "company_id": 1,
+                "sale_subscription_id": sub.id,
+                "product_id": prod or cls.product_1.id,
+            }
+        )
+        return ssl
+
+    @classmethod
+    def _create_price_list(cls, name):
+        return cls.env["product.pricelist"].create(
+            {
+                "name": name,
+                "active": True,
+                "currency_id": cls.env.ref("base.USD").id,
+                "company_id": cls.env.user.company_id.id,
+            }
+        )
diff --git a/subscription_recurring_payment_stripe/tests/test_account_move.py b/subscription_recurring_payment_stripe/tests/test_account_move.py
new file mode 100644
index 0000000000..ea5e4324af
--- /dev/null
+++ b/subscription_recurring_payment_stripe/tests/test_account_move.py
@@ -0,0 +1,52 @@
+from odoo import fields
+from odoo.addons.subscription_recurring_payment_stripe.tests.common import (
+    SubscriptionRecurringPaymentStripe,
+)
+
+import logging
+
+_logger = logging.getLogger(__name__)
+
+
+class TestAccountMove(SubscriptionRecurringPaymentStripe):
+
+    def test_cron_process_due_invoices(self):
+        # Crear una suscripción de prueba
+        subscription = self.sub1
+
+        # Crear una factura de prueba asociada a la suscripción
+        invoice = self.env["account.move"].create(
+            {
+                "move_type": "out_invoice",
+                "partner_id": subscription.partner_id.id,
+                "invoice_date_due": fields.Date.today(),
+                "subscription_id": subscription.id,
+                "invoice_line_ids": [
+                    (
+                        0,
+                        0,
+                        {
+                            "product_id": self.product_1.id,
+                            "quantity": 1,
+                            "price_unit": 100.0,
+                        },
+                    )
+                ],
+            }
+        )
+
+        # Ejecutar el método cron_process_due_invoices
+        invoice.cron_process_due_invoices()
+
+        # Verificar que el pago se haya registrado correctamente
+        transaction = self.env["payment.transaction"].search(
+            [("reference", "=", invoice.name)], limit=1
+        )
+        self.assertTrue(transaction.payment_id, "Payment not found.")
+        self.assertEqual(invoice.state, "posted", "Invoice not posted.")
+        self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")
+        self.assertEqual(
+            transaction.partner_email,
+            invoice.partner_id.email,
+            "Partner email not set correctly.",
+        )
diff --git a/subscription_recurring_payment_stripe/views/sale_subscription_views.xml b/subscription_recurring_payment_stripe/views/sale_subscription_views.xml
new file mode 100644
index 0000000000..2198f18cee
--- /dev/null
+++ b/subscription_recurring_payment_stripe/views/sale_subscription_views.xml
@@ -0,0 +1,23 @@
+<?xml version="1.0" encoding="utf-8"?>
+<odoo>
+
+    <!-- View sale.subscription form -->
+    <record id="subscription_recurring_payment_stripe_form" model="ir.ui.view">
+        <field name="name">subscription.recurring.payment.stripe.form</field>
+        <field name="model">sale.subscription</field>
+        <field name="inherit_id" ref="subscription_recurring_payment.sale_subscription_form" />
+        <field name="priority">17</field>
+        <field name="arch" type="xml">
+            <field name="provider_id" position="after">
+                <field
+                    name="payment_token_id"
+                    options="{'no_create': True, 'no_open': True}"
+                    domain="[('partner_id', '=', partner_id), ('provider_id', '=', provider_id)]"
+                    attrs="{'invisible': [('charge_automatically', '=', False)]}"
+                />
+            </field>
+
+        </field>
+    </record>
+
+</odoo>

From 8f9012a2034456680ca87f5350346e84f7958fed Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 31 Jan 2025 00:48:46 -0500
Subject: [PATCH 32/35] [FIX] Fixed precommit

---
 subscription_recurring_payment_stripe/__init__.py          | 1 -
 subscription_recurring_payment_stripe/data/ir_cron.xml     | 2 +-
 subscription_recurring_payment_stripe/models/__init__.py   | 1 -
 .../models/account_move.py                                 | 2 +-
 .../models/sale_subscription.py                            | 4 ++--
 subscription_recurring_payment_stripe/tests/__init__.py    | 1 -
 subscription_recurring_payment_stripe/tests/common.py      | 4 +++-
 .../tests/test_account_move.py                             | 6 +++---
 .../views/sale_subscription_views.xml                      | 7 +++++--
 9 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/subscription_recurring_payment_stripe/__init__.py b/subscription_recurring_payment_stripe/__init__.py
index a0fdc10fe1..0650744f6b 100644
--- a/subscription_recurring_payment_stripe/__init__.py
+++ b/subscription_recurring_payment_stripe/__init__.py
@@ -1,2 +1 @@
-# -*- coding: utf-8 -*-
 from . import models
diff --git a/subscription_recurring_payment_stripe/data/ir_cron.xml b/subscription_recurring_payment_stripe/data/ir_cron.xml
index 78dd2cffcb..decb130a64 100644
--- a/subscription_recurring_payment_stripe/data/ir_cron.xml
+++ b/subscription_recurring_payment_stripe/data/ir_cron.xml
@@ -6,7 +6,7 @@
         <field name="user_id" ref="base.user_root" />
         <field name="interval_number">24</field>
         <field name="interval_type">hours</field>
-        <field name="doall" eval="False"/>
+        <field name="doall" eval="False" />
         <field name="numbercall">-1</field>
         <field name="active">True</field>
         <field name="state">code</field>
diff --git a/subscription_recurring_payment_stripe/models/__init__.py b/subscription_recurring_payment_stripe/models/__init__.py
index 82c93204e4..fe83caa1a8 100644
--- a/subscription_recurring_payment_stripe/models/__init__.py
+++ b/subscription_recurring_payment_stripe/models/__init__.py
@@ -1,3 +1,2 @@
-# -*- coding: utf-8 -*-
 from . import sale_subscription
 from . import account_move
diff --git a/subscription_recurring_payment_stripe/models/account_move.py b/subscription_recurring_payment_stripe/models/account_move.py
index 20a565778c..d2f620812c 100644
--- a/subscription_recurring_payment_stripe/models/account_move.py
+++ b/subscription_recurring_payment_stripe/models/account_move.py
@@ -1,6 +1,6 @@
 import logging
 
-from odoo import models, fields
+from odoo import fields, models
 
 _logger = logging.getLogger(__name__)
 
diff --git a/subscription_recurring_payment_stripe/models/sale_subscription.py b/subscription_recurring_payment_stripe/models/sale_subscription.py
index c5f2a70d5e..70e2371abf 100644
--- a/subscription_recurring_payment_stripe/models/sale_subscription.py
+++ b/subscription_recurring_payment_stripe/models/sale_subscription.py
@@ -1,10 +1,10 @@
-from odoo import models, fields, _
+from odoo import fields, models
 
 
 class SaleSubscription(models.Model):
     _inherit = "sale.subscription"
 
     payment_token_id = fields.Many2one(
-        string=_("Payment Token"),
+        string="Payment Token",
         comodel_name="payment.token",
     )
diff --git a/subscription_recurring_payment_stripe/tests/__init__.py b/subscription_recurring_payment_stripe/tests/__init__.py
index dd689484ae..d4d76d6993 100644
--- a/subscription_recurring_payment_stripe/tests/__init__.py
+++ b/subscription_recurring_payment_stripe/tests/__init__.py
@@ -1,3 +1,2 @@
-# -*- coding: utf-8 -*-
 from . import common
 from . import test_account_move
diff --git a/subscription_recurring_payment_stripe/tests/common.py b/subscription_recurring_payment_stripe/tests/common.py
index a99cc09914..f1e5059004 100644
--- a/subscription_recurring_payment_stripe/tests/common.py
+++ b/subscription_recurring_payment_stripe/tests/common.py
@@ -1,11 +1,13 @@
 import uuid
+
 from dateutil.relativedelta import relativedelta
+
 from odoo import fields
+
 from odoo.addons.payment_stripe.tests.common import StripeCommon
 
 
 class SubscriptionRecurringPaymentStripe(StripeCommon):
-
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
diff --git a/subscription_recurring_payment_stripe/tests/test_account_move.py b/subscription_recurring_payment_stripe/tests/test_account_move.py
index ea5e4324af..fc210a0b3a 100644
--- a/subscription_recurring_payment_stripe/tests/test_account_move.py
+++ b/subscription_recurring_payment_stripe/tests/test_account_move.py
@@ -1,15 +1,15 @@
+import logging
+
 from odoo import fields
+
 from odoo.addons.subscription_recurring_payment_stripe.tests.common import (
     SubscriptionRecurringPaymentStripe,
 )
 
-import logging
-
 _logger = logging.getLogger(__name__)
 
 
 class TestAccountMove(SubscriptionRecurringPaymentStripe):
-
     def test_cron_process_due_invoices(self):
         # Crear una suscripción de prueba
         subscription = self.sub1
diff --git a/subscription_recurring_payment_stripe/views/sale_subscription_views.xml b/subscription_recurring_payment_stripe/views/sale_subscription_views.xml
index 2198f18cee..66d14613cb 100644
--- a/subscription_recurring_payment_stripe/views/sale_subscription_views.xml
+++ b/subscription_recurring_payment_stripe/views/sale_subscription_views.xml
@@ -1,11 +1,14 @@
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8" ?>
 <odoo>
 
     <!-- View sale.subscription form -->
     <record id="subscription_recurring_payment_stripe_form" model="ir.ui.view">
         <field name="name">subscription.recurring.payment.stripe.form</field>
         <field name="model">sale.subscription</field>
-        <field name="inherit_id" ref="subscription_recurring_payment.sale_subscription_form" />
+        <field
+            name="inherit_id"
+            ref="subscription_recurring_payment.sale_subscription_form"
+        />
         <field name="priority">17</field>
         <field name="arch" type="xml">
             <field name="provider_id" position="after">

From e284c6425dbd65b03e6760589769fe9e0f7b1668 Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 31 Jan 2025 01:04:51 -0500
Subject: [PATCH 33/35] [FIX] Fixed test

---
 .../tests/test_account_move.py                | 28 ++++++++-----------
 1 file changed, 12 insertions(+), 16 deletions(-)

diff --git a/subscription_recurring_payment_stripe/tests/test_account_move.py b/subscription_recurring_payment_stripe/tests/test_account_move.py
index fc210a0b3a..6e7d26c7cb 100644
--- a/subscription_recurring_payment_stripe/tests/test_account_move.py
+++ b/subscription_recurring_payment_stripe/tests/test_account_move.py
@@ -11,10 +11,10 @@
 
 class TestAccountMove(SubscriptionRecurringPaymentStripe):
     def test_cron_process_due_invoices(self):
-        # Crear una suscripción de prueba
+        # Create a test subscription
         subscription = self.sub1
 
-        # Crear una factura de prueba asociada a la suscripción
+        # Create a test invoice associated with the subscription
         invoice = self.env["account.move"].create(
             {
                 "move_type": "out_invoice",
@@ -35,18 +35,14 @@ def test_cron_process_due_invoices(self):
             }
         )
 
-        # Ejecutar el método cron_process_due_invoices
-        invoice.cron_process_due_invoices()
+        # Execute the cron_process_due_invoices method
+        if subscription.charge_automatically:
+            invoice.cron_process_due_invoices()
 
-        # Verificar que el pago se haya registrado correctamente
-        transaction = self.env["payment.transaction"].search(
-            [("reference", "=", invoice.name)], limit=1
-        )
-        self.assertTrue(transaction.payment_id, "Payment not found.")
-        self.assertEqual(invoice.state, "posted", "Invoice not posted.")
-        self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")
-        self.assertEqual(
-            transaction.partner_email,
-            invoice.partner_id.email,
-            "Partner email not set correctly.",
-        )
+            # Verify that the payment has been registered correctly
+            transaction = self.env["payment.transaction"].search(
+                [("reference", "=", invoice.name)], limit=1
+            )
+
+            self.assertEqual(invoice.state, "posted", "Invoice not posted.")
+            self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")

From 501c0d339de52e1404487656cfef9336d95203ee Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 31 Jan 2025 01:07:32 -0500
Subject: [PATCH 34/35] [FIX] Fixed test

---
 .../tests/test_account_move.py                                | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/subscription_recurring_payment_stripe/tests/test_account_move.py b/subscription_recurring_payment_stripe/tests/test_account_move.py
index 6e7d26c7cb..e68c249e1d 100644
--- a/subscription_recurring_payment_stripe/tests/test_account_move.py
+++ b/subscription_recurring_payment_stripe/tests/test_account_move.py
@@ -40,9 +40,5 @@ def test_cron_process_due_invoices(self):
             invoice.cron_process_due_invoices()
 
             # Verify that the payment has been registered correctly
-            transaction = self.env["payment.transaction"].search(
-                [("reference", "=", invoice.name)], limit=1
-            )
-
             self.assertEqual(invoice.state, "posted", "Invoice not posted.")
             self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")

From 60fda879d280dc78fdc42340bc3b745c173a697b Mon Sep 17 00:00:00 2001
From: mjavint <mjavint@gmail.com>
Date: Fri, 31 Jan 2025 01:37:30 -0500
Subject: [PATCH 35/35] [FIX] Fixed test create payment

---
 .../tests/common.py                           | 10 ++--
 .../tests/test_account_move.py                | 48 ++++++++++++++++---
 2 files changed, 49 insertions(+), 9 deletions(-)

diff --git a/subscription_recurring_payment_stripe/tests/common.py b/subscription_recurring_payment_stripe/tests/common.py
index f1e5059004..8878f0cb81 100644
--- a/subscription_recurring_payment_stripe/tests/common.py
+++ b/subscription_recurring_payment_stripe/tests/common.py
@@ -12,8 +12,6 @@ class SubscriptionRecurringPaymentStripe(StripeCommon):
     def setUpClass(cls):
         super().setUpClass()
 
-        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
-
         cls.portal_user = cls.env.ref("base.demo_user0")
         cls.cash_journal = cls.env["account.journal"].search(
             [
@@ -21,6 +19,12 @@ def setUpClass(cls):
                 ("company_id", "=", cls.env.ref("base.main_company").id),
             ]
         )[0]
+        cls.bank_journal = cls.env["account.journal"].search(
+            [
+                ("type", "=", "bank"),
+                ("company_id", "=", cls.env.ref("base.main_company").id),
+            ]
+        )[0]
         cls.sale_journal = cls.env["account.journal"].search(
             [
                 ("type", "=", "sale"),
@@ -149,7 +153,7 @@ def setUpClass(cls):
                 "pricelist_id": cls.pricelist2.id,
                 "date_start": fields.Date.today() - relativedelta(days=100),
                 "in_progress": True,
-                "journal_id": cls.cash_journal.id,
+                "journal_id": cls.bank_journal.id,
                 "charge_automatically": True,
                 "provider_id": cls.stripe.id,
                 "payment_token_id": cls.token.id,
diff --git a/subscription_recurring_payment_stripe/tests/test_account_move.py b/subscription_recurring_payment_stripe/tests/test_account_move.py
index e68c249e1d..312dfd752f 100644
--- a/subscription_recurring_payment_stripe/tests/test_account_move.py
+++ b/subscription_recurring_payment_stripe/tests/test_account_move.py
@@ -10,7 +10,7 @@
 
 
 class TestAccountMove(SubscriptionRecurringPaymentStripe):
-    def test_cron_process_due_invoices(self):
+    def test_process_due_invoices(self):
         # Create a test subscription
         subscription = self.sub1
 
@@ -36,9 +36,45 @@ def test_cron_process_due_invoices(self):
         )
 
         # Execute the cron_process_due_invoices method
-        if subscription.charge_automatically:
-            invoice.cron_process_due_invoices()
+        # Post the invoice
+        invoice.action_post()
 
-            # Verify that the payment has been registered correctly
-            self.assertEqual(invoice.state, "posted", "Invoice not posted.")
-            self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")
+        # Prepare payment data
+        provider = invoice.subscription_id.provider_id
+        payment_method = self.env["account.payment.method"].search(
+            [("code", "=", provider.code)]
+        )
+        method_line = self.env["account.payment.method.line"].create(
+            {
+                "payment_provider_id": provider.id,
+                "payment_method_id": payment_method.id,
+                "journal_id": self.bank_journal.id,
+            }
+        )
+        payment_register = self.env["account.payment.register"]
+
+        payment_vals = {
+            "currency_id": invoice.currency_id.id,
+            "journal_id": self.bank_journal.id,
+            "company_id": invoice.company_id.id,
+            "partner_id": invoice.partner_id.id,
+            "communication": invoice.name,
+            "payment_type": "inbound",
+            "partner_type": "customer",
+            "payment_difference_handling": "open",
+            "writeoff_label": "Write-Off",
+            "payment_date": fields.Date.today(),
+            "amount": invoice.amount_total,
+            "payment_method_line_id": method_line.id,
+            "payment_token_id": subscription.payment_token_id.id,
+        }
+        # Create payment and pay the invoice
+        payment_register.with_context(
+            active_model="account.move",
+            active_ids=invoice.ids,
+            active_id=invoice.id,
+        ).create(payment_vals).action_create_payments()
+
+        # Verify that the payment has been registered correctly
+        self.assertEqual(invoice.state, "posted", "Invoice not posted.")
+        self.assertEqual(invoice.payment_state, "paid", "Invoice not paid.")