diff options
| author | Pierre Masereel <pim@odoo.com> | 2019-09-10 12:57:37 +0000 |
|---|---|---|
| committer | Joseph Caburnay <jcb@odoo.com> | 2019-09-12 05:28:49 +0000 |
| commit | dc2e990aec67676a316566a5071ea28fb94b28f1 (patch) | |
| tree | 80af6e109da28150fe41c20139e9f71b39942c6f /addons/delivery/models/stock_picking.py | |
| parent | 26b692cbe299dc365b883721531fb6d13980ddd4 (diff) | |
[FIX] delivery: get correct carrier for invoicing policy
The carrier argument passed to the _create_delivery_line function
can be different to the carrier already existing in the sale order
so we should use the correct setting of carrier where we are calling
the function.
closes odoo/odoo#36424
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
Diffstat (limited to 'addons/delivery/models/stock_picking.py')
| -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 66bea256e92..2b2da5777b0 100644 --- a/addons/delivery/models/stock_picking.py +++ b/addons/delivery/models/stock_picking.py @@ -171,7 +171,7 @@ class StockPicking(models.Model): if sale_order and self.carrier_id.invoice_policy == 'real' and self.carrier_price: delivery_lines = sale_order.order_line.filtered(lambda l: l.is_delivery and l.currency_id.is_zero(l.price_unit) and l.product_id == self.carrier_id.product_id) if not delivery_lines: - sale_order._create_delivery_line(self.carrier_id, self.carrier_price, price_unit_in_description=False) + sale_order._create_delivery_line(self.carrier_id, self.carrier_price) else: delivery_line = delivery_lines[0] delivery_line[0].write({ |