Merge PR #762 into 16.0

Signed-off-by pedrobaeza

product_multi_company/README.rst

@@ -84,6 +84,10 @@ Contributors
   * Prabakaran <[email protected]>
   * Karthik <[email protected]>
 
+* `Moduon <https://www.moduon.team/>`__:
+
+  * Jairo Llopis
+
 Maintainers
 ~~~~~~~~~~~
 

product_multi_company/__init__.py

@@ -1,3 +1,2 @@
 from . import models
 from .hooks import post_init_hook
-from .hooks import uninstall_hook

product_multi_company/__manifest__.py

@@ -12,5 +12,4 @@
     "depends": ["base_multi_company", "product"],
     "data": ["views/product_template_view.xml"],
     "post_init_hook": "post_init_hook",
-    "uninstall_hook": "uninstall_hook",
 }

product_multi_company/hooks.py

@@ -1,34 +1,10 @@
 # Copyright 2015-2016 Pedro M. Baeza <[email protected]>
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
-
-import logging
-
 from odoo import SUPERUSER_ID, api
 
-_logger = logging.getLogger(__name__)
-
-try:
-    from odoo.addons.base_multi_company import hooks
-except ImportError:
-    _logger.info("Cannot find `base_multi_company` module in addons path.")
+from odoo.addons.base_multi_company import hooks
 
 
 def post_init_hook(cr, registry):
-    env = api.Environment(cr, SUPERUSER_ID, {})
-    # Change access rule
-    rule = env.ref("product.product_comp_rule")
-    rule.write(
-        {
-            "domain_force": (
-                "['|', ('company_ids', 'in', company_ids),"
-                "('company_ids', '=', False)]"
-            ),
-        }
-    )
-
-
-def uninstall_hook(cr, registry):
-    hooks.uninstall_hook(
-        cr,
-        "product.product_comp_rule",
-    )
+    api.Environment(cr, SUPERUSER_ID, {})
+    hooks.fill_company_ids(cr, "product.template")

product_multi_company/migrations/16.0.1.0.3/pre-migrate.py

@@ -0,0 +1,34 @@
+# Copyright 2025 Moduon Team S.L.
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl-3.0)
+from logging import getLogger
+
+from openupgradelib import openupgrade
+
+from odoo.tools import safe_eval
+
+_logger = getLogger(__name__)
+
+PREVIOUS_DOMAIN = [
+    "|",
+    ("company_ids", "in", "COMPANY_IDS"),
+    ("company_ids", "=", False),
+]
+UPSTREAM_DOMAIN = " [('company_id', 'in', company_ids + [False])]"
+
+
+@openupgrade.migrate()
+def migrate(env, version):
+    """Restore upstream partner rule."""
+    rule = env.ref("product.product_comp_rule", False)
+    if not rule:
+        return
+    try:
+        domain = safe_eval(
+            rule.domain_force, locals_dict={"company_ids": "COMPANY_IDS"}
+        )
+    except Exception:
+        _logger.warning("Unable to evaluate domain_force")
+        return
+    if domain == PREVIOUS_DOMAIN:
+        rule.domain_force = UPSTREAM_DOMAIN
+        _logger.info("Restored upstream partner rule")

product_multi_company/models/__init__.py

@@ -1,5 +1,4 @@
 # Copyright 2015 Serv. Tecnol. Avanzados - Pedro M. Baeza
 # License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 
-from . import product_product
 from . import product_template

product_multi_company/readme/CONTRIBUTORS.rst

@@ -10,3 +10,7 @@
 
   * Prabakaran <[email protected]>
   * Karthik <[email protected]>
+
+* `Moduon <https://www.moduon.team/>`__:
+
+  * Jairo Llopis

product_multi_company/static/description/index.html

@@ -430,6 +430,10 @@ <h2><a class="toc-backref" href="#toc-entry-6">Contributors</a></h2>
 <li>Karthik &lt;<a class="reference external" href="mailto:karthik&#64;sodexis.com">karthik&#64;sodexis.com</a>&gt;</li>
 </ul>
 </li>
+<li><a class="reference external" href="https://www.moduon.team/">Moduon</a>:<ul>
+<li>Jairo Llopis</li>
+</ul>
+</li>
 </ul>
 </div>
 <div class="section" id="maintainers">

product_multi_company/tests/test_product_multi_company.py

@@ -136,11 +136,3 @@ def test_search_product(self):
             ]
         )
         self.assertEqual(searched_products, expected_products)
-
-    def test_uninstall(self):
-        from ..hooks import uninstall_hook
-
-        uninstall_hook(self.env.cr, None)
-        rule = self.env.ref("product.product_comp_rule")
-        domain = " [('company_id', 'in', [False, user.company_id.id])]"
-        self.assertEqual(rule.domain_force, domain)