From 84b0ec4da361adab9a4fa1943491fc4c745c762a Mon Sep 17 00:00:00 2001 From: "Pedro M. Baeza" Date: Thu, 8 Feb 2018 09:17:54 +0100 Subject: [PATCH] [FIX] delivery: Get correct carrier price on picking created from SO Steps to reproduce: * Create a sales order (SO). * Assign a carrier. * Don't press on "Set price" button next to carrier. * Validate SO. * Validate picking. * Delivery line is added to the SO with price 0. This is because the delivery SO line is added with the value of picking carrier_price field, but that field hasn't been set. If you modify on view carrier field, there's an onchange that fills correct price, so I use that onchange method for getting same results. --- addons/delivery/models/stock_move.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/addons/delivery/models/stock_move.py b/addons/delivery/models/stock_move.py index 92fb62d7dc3..1f0b48663e6 100644 --- a/addons/delivery/models/stock_move.py +++ b/addons/delivery/models/stock_move.py @@ -38,4 +38,7 @@ class StockMove(models.Model): pickings.write({ 'carrier_id': proc.sale_line_id.order_id.carrier_id.id, }) + # Get correct carrier price + for picking in pickings: + picking.onchange_carrier() return res