[MIG] contract: Migration to 19.0

Author: BhaveshHeliconia

contract/README.rst

@@ -21,13 +21,13 @@ Recurring - Contracts Management
     :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/18.0/contract
+    :target: https://github.com/OCA/contract/tree/19.0/contract
     :alt: OCA/contract
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/contract-18-0/contract-18-0-contract
+    :target: https://translation.odoo-community.org/projects/contract-19-0/contract-19-0-contract
     :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=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/contract&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -98,9 +98,9 @@ Usage
 
 |image2|
 
-.. |image| image:: https://raw.githubusercontent.com/OCA/contract/18.0/contract/static/src/screenshots/portal-my.png
-.. |image1| image:: https://raw.githubusercontent.com/OCA/contract/18.0/contract/static/src/screenshots/portal-list.png
-.. |image2| image:: https://raw.githubusercontent.com/OCA/contract/18.0/contract/static/src/screenshots/portal-detail.png
+.. |image| image:: https://raw.githubusercontent.com/OCA/contract/19.0/contract/static/src/screenshots/portal-my.png
+.. |image1| image:: https://raw.githubusercontent.com/OCA/contract/19.0/contract/static/src/screenshots/portal-list.png
+.. |image2| image:: https://raw.githubusercontent.com/OCA/contract/19.0/contract/static/src/screenshots/portal-detail.png
 
 Known issues / Roadmap
 ======================
@@ -114,7 +114,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:%20contract%0Aversion:%2018.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:%20contract%0Aversion:%2019.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.
 
@@ -158,6 +158,10 @@ Contributors
 
      - Antoni Marroig <amarroig@apsl.net>
 
+- `Heliconia Solutions Pvt. Ltd. <https://www.heliconia.io>`__
+
+     - Bhavesh Heliconia
+
 Maintainers
 -----------
 
@@ -171,6 +175,6 @@ 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/18.0/contract>`_ project on GitHub.
+This module is part of the `OCA/contract <https://github.com/OCA/contract/tree/19.0/contract>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

contract/__manifest__.py

@@ -9,9 +9,10 @@
 # Copyright 2020 Tecnativa - Víctor Martínez
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
+
 {
     "name": "Recurring - Contracts Management",
-    "version": "18.0.2.0.6",
+    "version": "19.0.1.0.0",
     "category": "Contract Management",
     "license": "AGPL-3",
     "author": "Tecnativa, ACSONE SA/NV, Odoo Community Association (OCA)",
@@ -29,7 +30,6 @@
         "data/template_mail_notification.xml",
         "data/mail_message_subtype.xml",
         "data/ir_ui_menu.xml",
-        "wizards/contract_manually_create_invoice.xml",
         "views/contract_tag.xml",
         "views/contract_template.xml",
         "views/contract_template_line.xml",
@@ -38,6 +38,7 @@
         "views/res_partner_view.xml",
         "views/res_config_settings.xml",
         "views/contract_portal_templates.xml",
+        "wizards/contract_manually_create_invoice.xml",
     ],
     "assets": {
         "web.assets_frontend": ["contract/static/src/scss/frontend.scss"],

contract/controllers/main.py

@@ -1,7 +1,7 @@
 # Copyright 2020-2022 Tecnativa - Víctor Martínez
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import _, http
+from odoo import http
 from odoo.exceptions import AccessError, MissingError
 from odoo.http import request
 
@@ -50,9 +50,12 @@ def portal_my_contracts(
             return request.redirect("/my")
         domain = self._get_filter_domain(kw)
         searchbar_sortings = {
-            "date": {"label": _("Date"), "order": "recurring_next_date desc"},
-            "name": {"label": _("Name"), "order": "name desc"},
-            "code": {"label": _("Reference"), "order": "code desc"},
+            "date": {
+                "label": request.env._("Date"),
+                "order": "recurring_next_date desc",
+            },
+            "name": {"label": request.env._("Name"), "order": "name desc"},
+            "code": {"label": request.env._("Reference"), "order": "code desc"},
         }
         # default sort by order
         if not sortby:

contract/migrations/18.0.2.0.0/end-migrate.py

@@ -13,7 +13,6 @@
 
 from odoo import Command, api, fields, models
 from odoo.exceptions import ValidationError
-from odoo.osv import expression
 
 _logger = logging.getLogger(__name__)
 
@@ -266,7 +265,6 @@ def _onchange_contract_template_id(self):
                 (
                     field.compute,
                     field.related,
-                    field.automatic,
                     field.readonly,
                     field.company_dependent,
                     field.name in self.NO_SYNC,
@@ -452,7 +450,9 @@ def _get_related_invoices(self):
         )
         # we are forced to always search for this for not losing possible <=v11
         # generated invoices
-        invoices |= self.env["account.move"].search([("old_contract_id", "=", self.id)])
+        invoices |= self.env["account.move"].search(
+            [("old_contract_id", "=", self.ids)]
+        )
         return invoices
 
     def _get_computed_currency(self):
@@ -549,12 +549,11 @@ def _get_contracts_to_invoice_domain(self, date_ref=None):
         This method builds the domain to use to find all
         contracts (contract.contract) to invoice.
         :param date_ref: optional reference date to use instead of today
-        :return: list (domain) usable on contract.contract
+        :return: Domain object usable on contract.contract
         """
-        domain = []
         if not date_ref:
             date_ref = fields.Date.context_today(self)
-        domain.extend([("recurring_next_date", "<=", date_ref)])
+        domain = fields.Domain([("recurring_next_date", "<=", date_ref)])
         return domain
 
     def _get_lines_to_invoice(self, date_ref):
@@ -682,7 +681,7 @@ def _cron_recurring_create(self, date_ref=False, create_type="invoice"):
         if not date_ref:
             date_ref = fields.Date.context_today(self)
         domain = self._get_contracts_to_invoice_domain(date_ref)
-        domain = expression.AND(
+        domain = fields.Domain.AND(
             [
                 domain,
                 [("generation_type", "=", create_type)],

contract/migrations/18.0.2.0.0/pre-migrate.py

@@ -27,7 +27,7 @@ class ContractLine(models.Model):
         string="Contract",
         required=True,
         index=True,
-        auto_join=True,
+        bypass_search_access=True,
         ondelete="cascade",
     )
     # replace from abstract to add the store=True

contract/models/contract.py

@@ -6,6 +6,8 @@
 # Copyright 2018 ACSONE SA/NV
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
+import json
+
 from odoo import api, fields, models
 from odoo.exceptions import ValidationError
 
@@ -42,18 +44,16 @@ class ContractTemplateLine(models.Model):
         readonly=False,
     )
     quantity = fields.Float(default=1.0, required=True)
-    product_uom_category_id = fields.Many2one(
-        comodel_name="uom.category",
-        related="product_id.uom_id.category_id",
-        readonly=True,
+    uom_id_domain = fields.Binary(
+        compute="_compute_uom_id_domain",
     )
     uom_id = fields.Many2one(
         comodel_name="uom.uom",
         compute="_compute_uom_id",
         store=True,
         readonly=False,
         string="Unit of Measure",
-        domain="[('category_id', '=', product_uom_category_id)]",
+        domain="uom_id_domain",
     )
 
     # === Pricing ===
@@ -153,12 +153,40 @@ def _compute_name(self):
                 )
                 line.name = product.get_product_multiline_description_sale()
 
-    @api.depends("product_id")
+    @api.depends("product_id", "product_id.uom_id", "product_id.uom_ids")
+    def _compute_uom_id_domain(self):
+        for record in self:
+            if record.product_id:
+                allowed_uoms = record.product_id.uom_id | record.product_id.uom_ids
+                allowed_ids = allowed_uoms.ids
+                if allowed_ids:
+                    record.uom_id_domain = json.dumps([("id", "in", allowed_ids)])
+                else:
+                    record.uom_id_domain = json.dumps([("id", "=", False)])
+            else:
+                record.uom_id_domain = json.dumps([("id", "=", False)])
+
+    @api.depends("product_id", "product_id.uom_id")
     def _compute_uom_id(self):
         for line in self:
-            if not line.uom_id or (
-                line.product_id.uom_id.category_id.id != line.uom_id.category_id.id
-            ):
+            if not line.product_id:
+                line.uom_id = False
+                continue
+            if line.uom_id_domain:
+                domain = json.loads(line.uom_id_domain)
+                allowed_ids = []
+                for clause in domain:
+                    if isinstance(clause, (list, tuple)) and len(clause) == 3:
+                        if clause[0] == "id" and clause[1] == "in":
+                            allowed_ids = clause[2]
+                            break
+                if allowed_ids and line.product_id.uom_id.id in allowed_ids:
+                    line.uom_id = line.product_id.uom_id
+                elif allowed_ids:
+                    line.uom_id = allowed_ids[0]
+                else:
+                    line.uom_id = False
+            else:
                 line.uom_id = line.product_id.uom_id
 
     @api.depends("contract_id.contract_type")