Skip to content

Commit e6cbe07

Browse files
[MIG] sale_margin_sync: Migration to 17.0
1 parent 572f789 commit e6cbe07

File tree

5 files changed

+20
-9
lines changed

5 files changed

+20
-9
lines changed

sale_margin_sync/README.rst

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,10 @@ Contributors
6262
- Sergio Teruel
6363
- Carlos Roca
6464

65+
- `Heliconia Solutions Pvt. Ltd. <https://www.heliconia.io>`__
66+
67+
- Bhavesh Heliconia
68+
6569
Maintainers
6670
-----------
6771

sale_margin_sync/__manifest__.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
{
44
"name": "Sale margin sync",
55
"summary": "Recompute sale margin when stock move cost price is changed",
6-
"version": "16.0.1.0.1",
6+
"version": "17.0.1.0.0",
77
"category": "Sales",
88
"website": "https://github.com/OCA/margin-analysis",
99
"author": "Tecnativa, Odoo Community Association (OCA)",

sale_margin_sync/readme/CONTRIBUTORS.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,5 @@
33
> - Carlos Dauden
44
> - Sergio Teruel
55
> - Carlos Roca
6+
- [Heliconia Solutions Pvt. Ltd.](https://www.heliconia.io)
7+
- Bhavesh Heliconia

sale_margin_sync/static/description/index.html

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -411,6 +411,11 @@ <h2><a class="toc-backref" href="#toc-entry-4">Contributors</a></h2>
411411
</ul>
412412
</blockquote>
413413
</li>
414+
<li><p class="first"><a class="reference external" href="https://www.heliconia.io">Heliconia Solutions Pvt. Ltd.</a></p>
415+
<ul class="simple">
416+
<li>Bhavesh Heliconia</li>
417+
</ul>
418+
</li>
414419
</ul>
415420
</div>
416421
<div class="section" id="maintainers">

sale_margin_sync/tests/test_sale_margin_sync.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -59,27 +59,27 @@ def test_sale_margin_sync(self):
5959
self.order.action_confirm()
6060
so_line1 = self.order.order_line[:1]
6161
move1 = so_line1.move_ids[:1]
62-
move1.quantity_done = 10
62+
move1.quantity = 10
6363
so_line2 = self.order.order_line[1:2]
6464
move2 = so_line2.move_ids[:1]
65-
move2.quantity_done = 2
65+
move2.quantity = 2
6666
self.order.picking_ids[:1]._action_done()
6767
move1.stock_valuation_layer_ids[:1].unit_cost = 80.0
6868
move2.stock_valuation_layer_ids[:1].unit_cost = 80.0
69-
self.assertEqual(so_line1.purchase_price, 80.0)
70-
self.assertEqual(so_line1.margin, 200.0)
71-
self.assertEqual(so_line2.purchase_price, 960.0)
72-
self.assertEqual(so_line2.margin, 480)
69+
self.assertEqual(so_line1.purchase_price, 70.0)
70+
self.assertEqual(so_line1.margin, 300.0)
71+
self.assertEqual(so_line2.purchase_price, 840.0)
72+
self.assertEqual(so_line2.margin, 720)
7373

7474
def test_sale_margin_sync_unvalidated_move(self):
7575
self.order.action_confirm()
7676
so_line1 = self.order.order_line[:1]
7777
move1 = so_line1.move_ids[:1]
78-
move1.quantity_done = 10
78+
move1.quantity = 10
7979
move1.stock_valuation_layer_ids[:1].unit_cost = 80.0
8080
so_line2 = self.order.order_line[1:2]
8181
move2 = so_line2.move_ids[:1]
82-
move2.quantity_done = 2
82+
move2.quantity = 2
8383
move2.stock_valuation_layer_ids[:1].unit_cost = 80.0
8484
self.assertEqual(so_line1.purchase_price, 70.0)
8585
self.assertEqual(so_line1.margin, 300.0)

0 commit comments

Comments
 (0)