From 6dc02294a04f0c6d7d45bf7f72ae1c08684b2c87 Mon Sep 17 00:00:00 2001 From: "Tiffany Chang (tic)" Date: Wed, 20 Sep 2023 14:13:34 +0200 Subject: [PATCH] [FIX] mrp: don't add missing consump warning line to backorder Previous fix odoo/odoo#131279 missed marking component lines added in from the consumption warning wizard as "additional". This made it so the added line was also added into the backorder when it should NOT have been (since the backorder should only be based on the original MO's values). Followup of task: 3456604 closes odoo/odoo#136728 X-original-commit: 7a45b691781e2862e87da7914a8b3b9a6487b7e9 Signed-off-by: Steve Van Essche Signed-off-by: Tiffany Chang (tic) --- addons/mrp/tests/test_order.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/addons/mrp/tests/test_order.py b/addons/mrp/tests/test_order.py index 513020ba129..b5acc16a4c6 100644 --- a/addons/mrp/tests/test_order.py +++ b/addons/mrp/tests/test_order.py @@ -3465,6 +3465,10 @@ class TestMrpOrder(TestMrpCommon): self.assertEqual(move.product_uom_qty, 48, "expected qty should be unchanged") self.assertEqual(move.quantity_done, 4, "expected qty was not applied as qty to be done (UoM was possibly not correctly converted)") self.assertEqual(mo2.state, 'done') + # double check that backorder qtys are also correct + mo2_backorder = mo2.procurement_group_id.mrp_production_ids[-1] + self.assertEqual(len(mo2_backorder.move_raw_ids), 1, "missing line should NOT have been added in") + self.assertEqual(mo2_backorder.move_raw_ids[0].product_uom_qty, 12, "backorder values are based on original MO, not current bom") #### scenario 3 - repeated comp move #### # bom.bom_line_ids[0]/product_id = p2