[FIX] website_sale_payment_term_acquirer: Fixed behavior for public u…

…ser.
OCA · Jul 16, 2023 · 2a0f995 · 2a0f995
1 parent 0b23513
commit 2a0f995
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 3 deletions.
diff --git a/website_sale_payment_term_acquirer/models/sale_order.py b/website_sale_payment_term_acquirer/models/sale_order.py
@@ -1,9 +1,14 @@
-from odoo import models
+from odoo import api, models
 
 
 class SaleOrder(models.Model):
     _inherit = "sale.order"
 
+    @api.model
+    def get_payment_term_by_order_id(self, order_id):
+        """Get property term by order id"""
+        return self.sudo().browse(order_id).partner_id.property_payment_term_id
+
     def _create_payment_transaction(self, vals):
         transactions = super()._create_payment_transaction(vals)
         for transaction in transactions:

diff --git a/website_sale_payment_term_acquirer/views/payment_templates.xml b/website_sale_payment_term_acquirer/views/payment_templates.xml
@@ -7,10 +7,22 @@
         name="Payment Trace"
         priority="100"
     >
+        <form position="before">
+            <t
+                t-set="payment_term_id"
+                t-value="env['sale.order'].get_payment_term_by_order_id(order_id)"
+            />
+            <t t-if="not payment_term_id">
+                <t
+                    t-set="acquirers"
+                    t-value="list(filter(lambda acq: not acq.display_main_payment_term, acquirers))"
+                />
+            </t>
+        </form>
         <span class="payment_option_name" position="replace">
             <t
-                t-if="acq.display_main_payment_term"
-                t-esc="user_id.partner_id.property_payment_term_id.display_name"
+                t-if="acq.display_main_payment_term and payment_term_id"
+                t-esc="payment_term_id.display_name"
             />
             <t t-else="" t-esc="acq.display_as or acq.name" />
             <div