[MIG] sale_margin_delivered: Migration to 17.0

Author: BhaveshHeliconia

sale_margin_delivered/README.rst

@@ -105,6 +105,9 @@ Contributors
 
 - Eduardo de Miguel (`Moduon <https://www.moduon.team/>`__)
 - Rafael Blasco (`Moduon <https://www.moduon.team/>`__)
+- `Heliconia Solutions Pvt. Ltd. <https://www.heliconia.io>`__
+
+  - Bhavesh Heliconia
 
 Maintainers
 -----------

sale_margin_delivered/__manifest__.py

@@ -2,7 +2,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 {
     "name": "Sale Margin Delivered",
-    "version": "16.0.2.0.0",
+    "version": "17.0.1.0.0",
     "author": "Tecnativa, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/margin-analysis",
     "category": "Sales",

sale_margin_delivered/migrations/16.0.1.0.0/post-migration.py

@@ -81,7 +81,10 @@ def _compute_margin_delivered(self):
             )
 
             if line.product_id.type != "product":
-                currency = line.order_id.pricelist_id.currency_id
+                currency = (
+                    line.order_id.pricelist_id.currency_id
+                    or line.company_id.currency_id
+                )
                 price = line.purchase_price
                 line.margin_delivered = currency.round(
                     line.price_subtotal - (price * line.qty_delivered)

sale_margin_delivered/migrations/16.0.2.0.0/post-migration.py

@@ -5,3 +5,5 @@
   - Pilar Vargas
 - Eduardo de Miguel ([Moduon](https://www.moduon.team/))
 - Rafael Blasco ([Moduon](https://www.moduon.team/))
+- [Heliconia Solutions Pvt. Ltd.](https://www.heliconia.io)
+  - Bhavesh Heliconia

sale_margin_delivered/models/sale_margin.py

@@ -448,6 +448,10 @@ <h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
 </li>
 <li>Eduardo de Miguel (<a class="reference external" href="https://www.moduon.team/">Moduon</a>)</li>
 <li>Rafael Blasco (<a class="reference external" href="https://www.moduon.team/">Moduon</a>)</li>
+<li><a class="reference external" href="https://www.heliconia.io">Heliconia Solutions Pvt. Ltd.</a><ul>
+<li>Bhavesh Heliconia</li>
+</ul>
+</li>
 </ul>
 </div>
 <div class="section" id="maintainers">

sale_margin_delivered/readme/CONTRIBUTORS.md

@@ -1,12 +1,12 @@
 #  Copyright 2019 Tecnativa - Sergio Teruel
 #  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl
+from odoo import Command
+from odoo.tests import Form
 
-from odoo.tests import Form, TransactionCase
+from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT, BaseCommon
 
-from odoo.addons.base.tests.common import DISABLED_MAIL_CONTEXT
 
-
-class TestSaleMarginDelivered(TransactionCase):
+class TestSaleMarginDelivered(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
@@ -28,9 +28,7 @@ def setUpClass(cls):
             {
                 "name": "Test pricelist",
                 "item_ids": [
-                    (
-                        0,
-                        0,
+                    Command.create(
                         {
                             "applied_on": "3_global",
                             "compute_price": "formula",
@@ -87,8 +85,8 @@ def test_sale_margin_delivered(self):
         sale_order.action_confirm()
         picking = sale_order.picking_ids
         picking.action_assign()
-        picking.move_line_ids.qty_done = 3.0
-        picking._action_done()
+        picking.move_ids.quantity = 3.0
+        picking.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 30.0)
         self.assertEqual(order_line.margin_delivered_percent, 0.5)
@@ -100,8 +98,8 @@ def test_sale_margin_delivered_excess(self):
         sale_order.action_confirm()
         picking = sale_order.picking_ids
         picking.action_assign()
-        picking.move_line_ids.qty_done = 12.0
-        picking._action_done()
+        picking.move_line_ids.quantity = 12.0
+        picking.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 120.0)
         self.assertEqual(order_line.margin_delivered_percent, 0.5)
@@ -121,15 +119,15 @@ def _create_return(self, picking, qty_refund=3.0, to_refund=False):
         return_wiz.product_return_moves.write(
             {"quantity": qty_refund, "to_refund": to_refund}
         )
-        new_picking_id, pick_type_id = return_wiz._create_returns()
-        return self.env["stock.picking"].browse(new_picking_id)
+        res = return_wiz.create_returns()
+        return self.env["stock.picking"].browse(res["res_id"])
 
     def _validate_so_picking(self, sale_order, qty_done=6.0):
         """Validate picking"""
         picking = sale_order.picking_ids
         picking.action_assign()
-        picking.move_line_ids.qty_done = qty_done
-        picking._action_done()
+        picking.move_line_ids.quantity = qty_done
+        picking.with_context(skip_backorder=True).button_validate()
         return picking
 
     def test_sale_margin_delivered_return_to_refund(self):
@@ -138,8 +136,8 @@ def test_sale_margin_delivered_return_to_refund(self):
         sale_order.action_confirm()
         picking = self._validate_so_picking(sale_order, qty_done=6.0)
         picking_return = self._create_return(picking, qty_refund=3.0, to_refund=True)
-        picking_return.move_line_ids.qty_done = 3.0
-        picking_return._action_done()
+        picking_return.move_line_ids.quantity = 3.0
+        picking_return.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 30.0)
         self.assertEqual(order_line.margin_delivered_percent, 0.5)
@@ -151,8 +149,8 @@ def test_sale_margin_delivered_return_to_refund_excess(self):
         sale_order.action_confirm()
         picking = self._validate_so_picking(sale_order, qty_done=12.0)
         picking_return = self._create_return(picking, qty_refund=3.0, to_refund=True)
-        picking_return.move_line_ids.qty_done = 3.0
-        picking_return._action_done()
+        picking_return.move_line_ids.quantity = 3.0
+        picking_return.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 90.0)
         self.assertEqual(order_line.margin_delivered_percent, 0.5)
@@ -164,7 +162,7 @@ def test_sale_margin_delivered_return_no_refund(self):
         sale_order.action_confirm()
         picking = self._validate_so_picking(sale_order, qty_done=6.0)
         picking_return = self._create_return(picking, qty_refund=3.0, to_refund=False)
-        picking_return.move_line_ids.qty_done = 3.0
+        picking_return.move_line_ids.quantity = 3.0
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 60.0)
@@ -177,7 +175,7 @@ def test_sale_margin_delivered_return_no_refund_excess(self):
         sale_order.action_confirm()
         picking = self._validate_so_picking(sale_order, qty_done=12.0)
         picking_return = self._create_return(picking, qty_refund=3.0, to_refund=False)
-        picking_return.move_line_ids.qty_done = 3.0
+        picking_return.move_line_ids.quantity = 3.0
         picking_return._action_done()
         order_line = sale_order.order_line[:1]
         self.assertEqual(order_line.margin_delivered, 120.0)
@@ -192,8 +190,8 @@ def test_sale_margin_delivered_precision(self):
         sale_order.action_confirm()
         picking = sale_order.picking_ids
         picking.action_assign()
-        picking.move_line_ids.qty_done = 6.0
-        picking._action_done()
+        picking.move_line_ids.quantity = 6.0
+        picking.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         # price_subtotal is rounded
         self.assertEqual(order_line.price_subtotal, 100.45)
@@ -212,8 +210,8 @@ def test_sale_margin_no_cost(self):
         sale_order.action_confirm()
         picking = sale_order.picking_ids
         picking.action_assign()
-        picking.move_line_ids.qty_done = 6.0
-        picking._action_done()
+        picking.move_line_ids.quantity = 6.0
+        picking.with_context(skip_backorder=True).button_validate()
         order_line = sale_order.order_line[:1]
         # price_subtotal is rounded
         self.assertEqual(order_line.margin_delivered, 120)