diff options
| author | Christophe Simonis <chs@odoo.com> | 2019-03-29 19:02:51 +0100 |
|---|---|---|
| committer | Christophe Simonis <chs@odoo.com> | 2019-03-29 19:02:51 +0100 |
| commit | c0cf4049b2a4a7c51169ea88569fd7f89e503a59 (patch) | |
| tree | 26dc98c4341bfe30c203ed5ba380efc984d67dcf | |
| parent | 2a06f4dcf3ab03d3d0b6ed14d6ccd12140063b27 (diff) | |
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.
| -rw-r--r-- | addons/delivery/models/stock_picking.py | 2 |
1 files changed, 1 insertions, 1 deletions
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') |