[IMP] product_barcode_constraint_per_company

Simplify logic

product_barcode_constraint_per_company/__manifest__.py

@@ -1,18 +1,19 @@
+# Copyright 2023 Cetmix OÜ
 # Copyright (C) 2020-Today: GRAP (http://www.grap.coop)
 # @author: Sylvain LE GAL (https://twitter.com/legalsylvain)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 {
     "name": "Product Barcode Constraint per Company",
-    "version": "14.0.1.0.1",
+    "version": "14.0.2.0.0",
     "category": "Product",
     "summary": "Change the product barcode constraint"
     ", allowing the same barcode for differents companies",
-    "author": "GRAP, Odoo Community Association (OCA)",
+    "author": "GRAP, Cetmix, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/stock-logistics-barcode",
     "license": "AGPL-3",
     "depends": [
-        "product",
+        "product_variant_company",
     ],
     "installable": True,
 }

product_barcode_constraint_per_company/models/product_product.py

@@ -1,25 +1,19 @@
+# Copyright 2023 Cetmix OÜ
 # Copyright (C) 2020-Today: GRAP (http://www.grap.coop)
 # @author: Sylvain LE GAL (https://twitter.com/legalsylvain)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from odoo import _, fields, models
+from odoo import _, models
 
 
 class ProductProduct(models.Model):
     _inherit = "product.product"
 
-    tmpl_company_id = fields.Many2one(
-        related="product_tmpl_id.company_id",
-        string="Company of the related Template",
-        store=True,
-    )
-
-    def _auto_init(self):
-        for i, constraint in enumerate(self._sql_constraints):
-            if constraint[0] == "barcode_uniq":
-                self._sql_constraints[i] = (
-                    "barcode_uniq",
-                    "unique(barcode, tmpl_company_id)",
-                    _("A barcode can only be assigned to one product per company !"),
-                )
-        return super(ProductProduct, self)._auto_init()
+    _sql_constraints = [
+        # Replace constraint with same name
+        (
+            "barcode_uniq",
+            "unique(barcode, company_id)",
+            _("A barcode can only be assigned to one product per company !"),
+        )
+    ]

product_barcode_constraint_per_company/readme/CONTRIBUTORS.rst

@@ -1,2 +1,4 @@
 * Sylvain LE GAL <https://twitter.com/legalsylvain>
 * Sulivan Lominchar <slominchar@archeti.com>
+* Cetmix <@cetmix.com>
+* Dessan Hemrayev <dessanhemrayev@gmail.com>

product_barcode_constraint_per_company/tests/test_module.py

@@ -24,10 +24,6 @@ def test_create_same_company(self):
             product2 = self._create_product("Product 2", self.company_1)
             product2.flush()
 
-    def test_create_different_company(self):
-        self._create_product("Product 1", self.company_1)
-        self._create_product("Product 2", self.company_2)
-
     def _create_product(self, name, company):
         vals = {
             "name": name,