diff options
| author | Pierre Masereel <pim@odoo.com> | 2019-08-20 12:38:20 +0000 |
|---|---|---|
| committer | fw-bot <otis@odoo.com> | 2019-10-14 16:31:36 +0000 |
| commit | 6800fcf3c13c893b47d81ad3624f309b9e0e5989 (patch) | |
| tree | d695da9acd67242c6e6fb96518768e8d868a8523 /addons/delivery/models | |
| parent | 5288816baed88232eb3b0093fa3cc0330f21706a (diff) | |
[FIX] delivery: set carrier on order when carrier line added
When you are adding a line on your sale order that is a delivery line,
the carrier is not always set on the sale order which means that when
the carrier is not explicitly set on the sale order, the carrier won't
be set on picking created from the sale order.
When you are using the wizard on sale order, it works, but when the
carrier is selected on the e-commerce, it is not set on the sale order.
To avoid such difference in behavior, we are writing the carrier_id of
sale order in the function 'set_delivery_line' insread of the wizard and
e-commerce.
closes odoo/odoo#35855
closes odoo/odoo#38740
Original-signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
X-original-commit: a15f9921fa1ce92c41c263a8a2c6a32342a8cb1b
Signed-off-by: pimodoo <pimodoo@users.noreply.github.com>
Diffstat (limited to 'addons/delivery/models')
| -rw-r--r-- | addons/delivery/models/sale_order.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/addons/delivery/models/sale_order.py b/addons/delivery/models/sale_order.py index 07ba7d80c91..464b36aa726 100644 --- a/addons/delivery/models/sale_order.py +++ b/addons/delivery/models/sale_order.py @@ -48,6 +48,7 @@ class SaleOrder(models.Model): self._remove_delivery_line() for order in self: + order.carrier_id = carrier.id order._create_delivery_line(carrier, amount) return True |