Skip to content

Commit 4856b09

Browse files
committed
[IMP] delivery_purchase: Set carrier_price when creating a picking
If we have a delivery price of 10 on the order, it is expected to show 10 on the picking (without needing to be set to that amount when set the picking as done). TT45656
1 parent 894bd10 commit 4856b09

File tree

2 files changed

+3
-0
lines changed

2 files changed

+3
-0
lines changed

delivery_purchase/models/purchase_order.py

+1
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ def _prepare_picking(self):
3939
res = super()._prepare_picking()
4040
if self.carrier_id:
4141
res["carrier_id"] = self.carrier_id.id
42+
res["carrier_price"] = self.delivery_price
4243
return res
4344

4445
def _create_delivery_line(self, carrier, price_unit):

delivery_purchase/tests/test_delivery_purchase.py

+2
Original file line numberDiff line numberDiff line change
@@ -105,9 +105,11 @@ def test_delivery_purchase(self):
105105
self.assertEqual(self.purchase.delivery_price, 30)
106106

107107
def test_picking_carrier_01(self):
108+
self.assertEqual(self.purchase.delivery_price, 20)
108109
self.purchase.button_confirm()
109110
picking = self.purchase.picking_ids
110111
self.assertEqual(picking.carrier_id, self.carrier_fixed)
112+
self.assertEqual(picking.carrier_price, 20)
111113
picking.carrier_id = self.carrier_rules.id
112114
self._action_picking_validate(picking)
113115
self.assertEqual(picking.carrier_price, 10)

0 commit comments

Comments
 (0)