From c0cf4049b2a4a7c51169ea88569fd7f89e503a59 Mon Sep 17 00:00:00 2001 From: Christophe Simonis Date: Fri, 29 Mar 2019 19:02:51 +0100 Subject: [PATCH] Revert "[FIX] delivery: picking weight not updated sometime" This commit is not needed (and not wanted) in saas~12.1 due to 07bef887af5d982b6b007e790c625d5289926953 that convert field `move_ids_without_package` to a computed field with an invert method. The invert method writes on `move_lines`, which will recompute the value of `weight` field correctly. This reverts commit b2b395467fec9daa6b8f1e97cacd3a739e48defe. --- addons/delivery/models/stock_picking.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/addons/delivery/models/stock_picking.py b/addons/delivery/models/stock_picking.py index 5d4da724ac6..accc102c2b7 100644 --- a/addons/delivery/models/stock_picking.py +++ b/addons/delivery/models/stock_picking.py @@ -111,7 +111,7 @@ class StockPicking(models.Model): for picking in self: picking.carrier_tracking_url = picking.carrier_id.get_tracking_link(picking) if picking.carrier_id and picking.carrier_tracking_ref else False - @api.depends('move_lines', 'move_ids_without_package') + @api.depends('move_lines') def _cal_weight(self): for picking in self: picking.weight = sum(move.weight for move in picking.move_lines if move.state != 'cancel')