T1771 Migrate recurring_contract to v17

CompassionCH · Aug 29, 2024 · 3dd6e04 · 3dd6e04
1 parent 55fb581
commit 3dd6e04
Show file tree

Hide file tree

Showing 15 changed files with 123 additions and 631 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -104,16 +104,6 @@ repos:
     hooks:
       - id: pyupgrade
         args: ["--keep-percent-format"]
-  - repo: https://github.com/acsone/setuptools-odoo
-    rev: 3.1.8
-    hooks:
-      - id: setuptools-odoo-make-default
-      - id: setuptools-odoo-get-requirements
-        args:
-          - --output
-          - requirements.txt
-          - --header
-          - "# generated from manifests external_dependencies"
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.1.3
     hooks:

diff --git a/recurring_contract/__manifest__.py b/recurring_contract/__manifest__.py
@@ -29,7 +29,7 @@
 {
     "name": "Recurring contract",
     "summary": "Contract for recurring invoicing",
-    "version": "16.0.1.0.0",
+    "version": "17.0.1.0.0",
     "license": "AGPL-3",
     "author": "Compassion CH",
     "development_status": "Production/Stable",

diff --git a/recurring_contract/data/balance_product_for_migr.xml b/recurring_contract/data/balance_product_for_migr.xml
@@ -4,14 +4,12 @@
     @author Simon Gonzalez <[email protected]>
     The licence is in the file __manifest__.py
 -->
-<odoo>
-    <data noupdate="1">
-        <!-- PRODUCT FOR THE MIGRATION 14.0.1.0.5 -->
-        <record id="product_balance_migr" model="product.product">
-            <field name="name">Balance</field>
-            <field name="type">consu</field>
-            <field name="default_code">TODO MODIFY</field>
-            <field name="categ_id">1</field>
-        </record>
-    </data>
+<odoo noupdate="1">
+    <!-- PRODUCT FOR THE MIGRATION 14.0.1.0.5 -->
+    <record id="product_balance_migr" model="product.product">
+        <field name="name">Balance</field>
+        <field name="type">consu</field>
+        <field name="default_code">TODO MODIFY</field>
+        <field name="categ_id">1</field>
+    </record>
 </odoo>
diff --git a/recurring_contract/data/pricelist_item_base_automation.xml b/recurring_contract/data/pricelist_item_base_automation.xml
@@ -5,6 +5,12 @@
     The licence is in the file __manifest__.py
 -->
 <odoo>
+    <record model="ir.actions.server" id="action_pricelist_item_update">
+        <field name="name">Update contract line amount</field>
+        <field name="model_id" ref="product.model_product_pricelist_item" />
+        <field name="state">code</field>
+        <field name="code">records.update_cl_amount()</field>
+    </record>
     <record id="automated_action" model="base.automation">
         <field
             name="name"
@@ -18,7 +24,9 @@
         />
         <field name="trg_date_range">0</field>
         <field name="trg_date_range_type">minutes</field>
-        <field name="state">code</field>
-        <field name="code">records.update_cl_amount()</field>
+        <field
+            name="action_server_ids"
+            eval="[(4, ref('action_pricelist_item_update'))]"
+        />
     </record>
 </odoo>
diff --git a/recurring_contract/models/contract_group.py b/recurring_contract/models/contract_group.py
@@ -58,7 +58,6 @@ class ContractGroup(models.Model):
         string="Invoice Suspended Until",
         help="Date at which the sponsor should receive invoices again.",
         tracking=True,
-        states={"draft": [("readonly", False)]},
     )
     partner_id = fields.Many2one(
         "res.partner",
@@ -547,15 +546,11 @@ def build_inv_line_data(
             contract = contract_line.contract_id
             line_name = product.name
             if contract_line.pricelist_item_count:
-                price = contract.pricelist_id.get_product_price(
-                    product, qty, contract.partner_id, invoicing_date
+                price = contract.pricelist_id._get_product_price(
+                    product, qty, date=invoicing_date
                 )
             else:
                 price = contract_line.amount
-            if product.pricelist_item_count > 0:
-                price = contract.pricelist_id.get_product_price(
-                    product, qty, self.partner_id, invoicing_date
-                )
         elif gift_wizard:
             product = gift_wizard.product_id
             qty = gift_wizard.quantity

diff --git a/recurring_contract/models/move.py b/recurring_contract/models/move.py
@@ -43,13 +43,9 @@ def _compute_last_payment(self):
             else:
                 invoice.last_payment = False
 
-    def register_payment(
-        self, payment_line, writeoff_acc_id=False, writeoff_journal_id=False
-    ):
+    def action_register_payment(self):
         """After registering a payment post a message of the bank statement linked"""
-        out = super().register_payment(
-            payment_line, writeoff_acc_id, writeoff_journal_id
-        )
+        out = super().action_register_payment()
         self.message_post_bank_statement_notes()
         return out
 
@@ -74,22 +70,14 @@ def _get_bank_statement_notes(self):
         )
         return statement_line_ids.filtered("narration").mapped("narration")
 
-    def action_invoice_paid(self):
+    def _invoice_paid_hook(self):
         """Call invoice_paid method on related contracts."""
-        res = super().action_invoice_paid()
+        res = super()._invoice_paid_hook()
         for invoice in self:
             contracts = invoice.mapped("invoice_line_ids.contract_id")
             contracts.invoice_paid(invoice)
         return res
 
-    def action_invoice_re_open(self):
-        """Call invoice_unpaid method on related contract."""
-        res = super().action_invoice_re_open()
-        for invoice in self:
-            contracts = invoice.mapped("invoice_line_ids.contract_id")
-            contracts.invoice_unpaid(invoice)
-        return res
-
     def reconcile_after_clean(self):
         """
         Called after clean invoices. If invoices can be reconciled

diff --git a/recurring_contract/models/move_line.py b/recurring_contract/models/move_line.py
@@ -8,7 +8,7 @@
 #
 ##############################################################################
 
-from odoo import _, api, fields, models
+from odoo import _, fields, models
 from odoo.exceptions import UserError
 
 
@@ -34,17 +34,6 @@ class MoveLine(models.Model):
         related="move_id.payment_state", store=True, readonly=True, index=True
     )
 
-    @api.onchange("product_id")
-    def _onchange_product_id(self):
-        # workaround an odoo bug :
-        # could be fixed by applying this change here
-        # - self.analytic_tag_ids = rec.analytic_tag_ids.ids
-        # + self.analytic_tag_ids = rec.analytic_tag_ids
-        # https://github.com/odoo/odoo/blame/12.0/addons/account_analytic_default/models/account_analytic_default.py#L100
-        self.analytic_tag_ids = self.env["account.analytic.tag"]
-        res = super()._onchange_product_id()
-        return res
-
     def group_reconcile(self, matched_lines, credit_or_debit="debit"):
         """
         Will reconcile the current recordset with any required lines taken from the

diff --git a/recurring_contract/models/product_pricelist_item.py b/recurring_contract/models/product_pricelist_item.py
@@ -14,8 +14,8 @@ def update_cl_amount(self):
                 ]
             )
             for cl in contract_lines:
-                price = cl.contract_id.pricelist_id.get_product_price(
-                    cl.product_id, cl.quantity, cl.contract_id.partner_id
+                price = cl.contract_id.pricelist_id._get_product_price(
+                    cl.product_id, cl.quantity
                 )
                 if cl.amount != price:
                     cl.amount = price