Merge PR #909 into 18.0

Signed-off-by JordiBForgeFlow

Author: OCA-git-bot

product_multi_company_stock/models/product_template.py

@@ -13,9 +13,10 @@ class ProductTemplate(models.Model):
     def _check_company_ids(self):
         for record in self:
             if record.company_ids:
+                variants = record.with_context(active_test=False).product_variant_ids
                 quants = self.env["stock.quant"].search(
                     [
-                        ("product_id", "in", record.product_variant_ids.ids),
+                        ("product_id", "in", variants.ids),
                         ("company_id", "not in", record.company_ids.ids),
                         ("quantity", "!=", 0),
                         ("location_id.usage", "=", "internal"),
@@ -33,7 +34,7 @@ def _check_company_ids(self):
                     )
                 moves = self.env["stock.move"].search(
                     [
-                        ("product_id", "in", record.product_variant_ids.ids),
+                        ("product_id", "in", variants.ids),
                         ("company_id", "not in", record.company_ids.ids),
                     ]
                 )

product_multi_company_stock/tests/test_product_multi_company_stock.py

@@ -62,3 +62,57 @@ def test_remove_company_with_quants_or_moves(self):
         with self.assertRaises(UserError) as error_move:
             product.write({"company_ids": [(6, 0, [self.company_2.id])]})
         self.assertIn("stock moves", str(error_move.exception))
+
+    def test_remove_company_with_quants_or_moves_archived_product(self):
+        product = self.env["product.product"].create(
+            {
+                "name": "Test Product",
+                "is_storable": True,
+                "company_ids": [(6, 0, [self.company_1.id, self.company_2.id])],
+                "active": False,
+            }
+        )
+        internal_loc = self.env["stock.location"].create(
+            {
+                "name": "Test Internal Location",
+                "usage": "internal",
+                "company_id": self.company_1.id,
+            }
+        )
+        dest_loc = self.env["stock.location"].create(
+            {
+                "name": "Test Customer Location",
+                "usage": "customer",
+                "company_id": self.company_1.id,
+            }
+        )
+        quant = self.env["stock.quant"].create(
+            {
+                "product_id": product.id,
+                "location_id": internal_loc.id,
+                "company_id": self.company_1.id,
+                "quantity": 10,
+            }
+        )
+        with self.assertRaises(UserError) as error_quant:
+            product.write({"company_ids": [(6, 0, [self.company_2.id])]})
+        self.assertIn("stock quantities", str(error_quant.exception))
+        quant.unlink()
+        move = self.env["stock.move"].create(
+            {
+                "name": "Test Stock Move",
+                "product_id": product.id,
+                "product_uom_qty": 10,
+                "product_uom": product.uom_id.id,
+                "location_id": internal_loc.id,
+                "location_dest_id": dest_loc.id,
+                "company_id": self.company_1.id,
+            }
+        )
+
+        move._action_confirm()
+        move._action_assign()
+        move._action_done()
+        with self.assertRaises(UserError) as error_move:
+            product.write({"company_ids": [(6, 0, [self.company_2.id])]})
+        self.assertIn("stock moves", str(error_move.exception))