diff --git a/addons/delivery/models/stock_picking.py b/addons/delivery/models/stock_picking.py index b100f2ae294..e85ef74ce39 100644 --- a/addons/delivery/models/stock_picking.py +++ b/addons/delivery/models/stock_picking.py @@ -141,6 +141,17 @@ class StockPicking(models.Model): """ self.ensure_one() view_id = self.env.ref('delivery.choose_delivery_package_view_form').id + context = dict( + self.env.context, + current_package_carrier_type=self.carrier_id.delivery_type, + default_picking_id=self.id + ) + # As we pass the `delivery_type` ('fixed' by default) in a key who correspond + # to the `package_carrier_type` ('none' to default), we make a conversion. + # No need conversion for other carriers as the `delivery_type` and + #`package_carrier_type` will be the same in these cases. + if context['current_package_carrier_type'] == 'fixed': + context['current_package_carrier_type'] = 'none' return { 'name': _('Package Details'), 'type': 'ir.actions.act_window', @@ -149,11 +160,7 @@ class StockPicking(models.Model): 'view_id': view_id, 'views': [(view_id, 'form')], 'target': 'new', - 'context': dict( - self.env.context, - current_package_carrier_type=self.carrier_id.delivery_type, - default_picking_id=self.id - ), + 'context': context, } def send_to_shipper(self): diff --git a/addons/delivery/wizard/choose_delivery_package_views.xml b/addons/delivery/wizard/choose_delivery_package_views.xml index 899452c94ec..2eb671e0e8f 100644 --- a/addons/delivery/wizard/choose_delivery_package_views.xml +++ b/addons/delivery/wizard/choose_delivery_package_views.xml @@ -6,7 +6,7 @@
-