[MIG] product_uoms: Migration to 19.0

Part-of: #804
Signed-off-by: Juan Carreras <jc@adhoc.com.ar>

Author: migration-bot-adhoc

product_uoms/__manifest__.py

@@ -19,7 +19,7 @@
 ##############################################################################
 {
     "name": "Product UOMS",
-    "version": "18.0.1.0.0",
+    "version": "19.0.1.0.0",
     "category": "base.module_category_knowledge_management",
     "author": "ADHOC SA",
     "website": "www.adhoc.com.ar",
@@ -36,7 +36,7 @@
         "views/product_template_views.xml",
         "security/ir.model.access.csv",
     ],
-    "installable": False,
+    "installable": True,
     "auto_install": False,
     "application": False,
 }

product_uoms/demo/product_product_demo.xml

@@ -2,9 +2,7 @@
 <odoo noupdate="1">
     <record id="product_on_cm_saleable_on_m_and_km" model="product.product">
         <field name="name">Product On Centimeter Saleable on Meter and Km</field>
-        <field name="categ_id" ref="product.product_category_4"/>
         <field name="uom_id" ref="uom.product_uom_cm"/>
-        <field name="uom_po_id" ref="uom.product_uom_cm"/>
         <field name="standard_price">1</field>
         <field name="list_price">2</field>
         <field name="type">consu</field>

product_uoms/models/product_product.py

@@ -15,6 +15,13 @@ def get_product_uoms(self, product_uom, use=False):
         We send product uom so it can be send from sale or purchase
         """
         self.ensure_one()
-        return product_uom + self.env["uom.uom"].search(
-            [("category_id", "=", product_uom.category_id.id), ("id", "!=", product_uom.id)]
-        )
+        # Get the root reference UoM by traversing up the parent chain
+        root_uom = product_uom
+        while root_uom.relative_uom_id:
+            root_uom = root_uom.relative_uom_id
+
+        # Find all UoMs that have the same root reference
+        all_uoms = self.env["uom.uom"].search([])
+        compatible_uoms = all_uoms.filtered(lambda u: u._has_common_reference(product_uom) and u.id != product_uom.id)
+
+        return product_uom + compatible_uoms

product_uoms/models/product_template.py

@@ -9,9 +9,6 @@
 class ProductTemplate(models.Model):
     _inherit = "product.template"
 
-    uom_category_id = fields.Many2one(
-        related="uom_id.category_id",
-    )
     uom_ids = fields.One2many(
         "product.uoms",
         "product_tmpl_id",
@@ -25,6 +22,13 @@ class ProductTemplate(models.Model):
     @api.constrains("uom_ids", "uom_id")
     def _check_uoms(self):
         for rec in self:
-            uom_categories = rec.uom_ids.mapped("uom_id.category_id")
-            if len(uom_categories) > 1 or (uom_categories and uom_categories != rec.uom_id.category_id):
-                raise ValidationError(_("UOMs Category must be of the same " "UOM Category as Product Unit of Measure"))
+            # Use _has_common_reference() to check if UoMs are compatible
+            product_uom = rec.uom_id
+            for uom_line in rec.uom_ids:
+                if not product_uom._has_common_reference(uom_line.uom_id):
+                    raise ValidationError(
+                        _(
+                            "UOMs must have the same reference unit as the Product Unit of Measure (%s)",
+                            product_uom.name,
+                        )
+                    )

product_uoms/models/product_uoms.py

@@ -20,6 +20,7 @@ class ProductUoms(models.Model):
         required=True,
     )
 
-    _sql_constraints = [
-        ("uom_uniq", "unique(product_tmpl_id, uom_id)", ("UOM must be unique per Product Template!")),
-    ]
+    _uom_uniq = models.Constraint(
+        "unique(product_tmpl_id, uom_id)",
+        "UOM must be unique per Product Template!",
+    )

product_uoms/views/product_template_views.xml

@@ -7,11 +7,10 @@
         <field name="inherit_id" ref="product.product_template_form_view"/>
         <field name="arch" type="xml">
             <field name="product_tooltip" position="after">
-                <field name="uom_category_id" invisible="1"/>
                 <field name="uom_ids">
                     <list editable="bottom">
                         <field name="sequence" widget="handle"/>
-                        <field name="uom_id" domain="[('category_id', '=', parent.uom_category_id)]" options="{'no_create': True, 'no_open': True}"/>
+                        <field name="uom_id" options="{'no_create': True, 'no_open': True}"/>
                     </list>
                 </field>
             </field>