From da2abe6a2c00e6313aa55c9be8a86a555be91433 Mon Sep 17 00:00:00 2001 From: Pierre Masereel Date: Wed, 24 Oct 2018 08:01:29 +0000 Subject: [PATCH 1/2] [FIX] procurement_jit: do not unreserve picking When a quantity is increased in SO, the related picking is unreserved then re-assigned, we can just call assign, as we don't want to lose what has already been done in the picking. As the decreased quantity on SO is not propagated on move, not unreserving won't cause trouble. --- addons/procurement_jit/sale.py | 1 - 1 file changed, 1 deletion(-) diff --git a/addons/procurement_jit/sale.py b/addons/procurement_jit/sale.py index b35cd0e606f..9ad3cf50b1d 100644 --- a/addons/procurement_jit/sale.py +++ b/addons/procurement_jit/sale.py @@ -14,6 +14,5 @@ class SaleOrderLine(models.Model): for order in orders: reassign = order.picking_ids.filtered(lambda x: x.state=='confirmed' or (x.state in ['waiting', 'assigned'] and not x.printed)) if reassign: - reassign.do_unreserve() reassign.action_assign() return res From 5954cd563ed5960aa0ecf8fa8e36530554a0091d Mon Sep 17 00:00:00 2001 From: Pierre Masereel Date: Fri, 26 Oct 2018 09:21:51 +0000 Subject: [PATCH 2/2] [FIX] stock: remove level package when unreserve if necessary --- addons/stock/models/stock_picking.py | 1 + 1 file changed, 1 insertion(+) diff --git a/addons/stock/models/stock_picking.py b/addons/stock/models/stock_picking.py index 310f7b6bef5..fec28f96de1 100644 --- a/addons/stock/models/stock_picking.py +++ b/addons/stock/models/stock_picking.py @@ -682,6 +682,7 @@ class Picking(models.Model): def do_unreserve(self): for picking in self: picking.move_lines._do_unreserve() + picking.package_level_ids.filtered(lambda p: not p.move_ids).unlink() @api.multi def button_validate(self):