[MIG] product_variant_sale_price: Migration to 16.0

OCA · Mar 17, 2023 · a43bec5 · a43bec5
1 parent 1523ac8
commit a43bec5
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 13 deletions.
diff --git a/product_variant_sale_price/__manifest__.py b/product_variant_sale_price/__manifest__.py
@@ -5,7 +5,7 @@
 {
     "name": "Product Variant Sale Price",
     "summary": "Allows to write fixed prices in product variants",
-    "version": "15.0.1.0.0",
+    "version": "16.0.1.0.0",
     "category": "Product Management",
     "website": "https://github.com/OCA/product-variant",
     "author": "Tecnativa, " "Odoo Community Association (OCA)",

diff --git a/product_variant_sale_price/models/product_product.py b/product_variant_sale_price/models/product_product.py
@@ -34,27 +34,25 @@ def _compute_lst_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             price = product.fix_price or product.list_price
-            if "uom" in self.env.context:
-                price = product.uom_id._compute_price(
-                    price, uom_model.browse(self.env.context["uom"])
-                )
+            if self.env.context.get("uom"):
+                context_uom = uom_model.browse(self.env.context["uom"])
+                price = product.uom_id._compute_price(price, context_uom)
             product.lst_price = price
 
     def _compute_list_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             price = product.fix_price or product.product_tmpl_id.list_price
-            if "uom" in self.env.context:
-                price = product.uom_id._compute_price(
-                    price, uom_model.browse(self.env.context["uom"])
-                )
+            if self.env.context.get("uom"):
+                context_uom = uom_model.browse(self.env.context["uom"])
+                price = product.uom_id._compute_price(price, context_uom)
             product.list_price = price
 
     def _inverse_product_lst_price(self):
         uom_model = self.env["uom.uom"]
         for product in self:
             vals = {}
-            if "uom" in self.env.context:
+            if self.env.context.get("uom"):
                 vals["fix_price"] = product.uom_id._compute_price(
                     product.lst_price, uom_model.browse(self.env.context["uom"])
                 )

diff --git a/product_variant_sale_price/tests/test_product_product.py b/product_variant_sale_price/tests/test_product_product.py
@@ -67,11 +67,11 @@ def test_post_init_hook(self):
         )
 
         # Flush the records to DB before direct SQL
-        self.product_blue.flush()
-        self.product_red.flush()
+        self.product_blue.env.flush_all()
+        self.product_red.env.flush_all()
 
         set_sale_price_on_variant(self.cr, None, self.product_template.id)
-        self.product_template.product_variant_ids.invalidate_cache()
+        self.product_template.product_variant_ids.invalidate_recordset()
         self.assertEqual(
             self.product_template.list_price + 100.00, self.product_blue.lst_price
         )

diff --git a/setup/product_variant_sale_price/odoo/addons/product_variant_sale_price b/setup/product_variant_sale_price/odoo/addons/product_variant_sale_price
@@ -0,0 +1 @@
+../../../../product_variant_sale_price
diff --git a/setup/product_variant_sale_price/setup.py b/setup/product_variant_sale_price/setup.py
@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)