Skip to content

Commit ade5a71

Browse files
committedApr 10, 2024·
[FIX] delivery_auto_refresh: dynamic carrier line
1 parent a1fdad5 commit ade5a71

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed
 

‎delivery_auto_refresh/models/sale_order.py

+4-3
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,10 @@ def _auto_refresh_delivery(self):
6161
if self._get_param_auto_add_delivery_line() and self.carrier_id:
6262
if self.state in {"draft", "sent"}:
6363
price_unit = self.carrier_id.rate_shipment(self)["price"]
64-
self.with_context(skip_validation_check=True)._create_delivery_line(
65-
self.carrier_id, price_unit
66-
)
64+
if not self.is_all_service:
65+
self.with_context(skip_validation_check=True)._create_delivery_line(
66+
self.carrier_id, price_unit
67+
)
6768
self.with_context(
6869
auto_refresh_delivery=True, skip_validation_check=True
6970
).write({"recompute_delivery_price": False})

‎delivery_auto_refresh/tests/test_delivery_auto_refresh.py

+1
Original file line numberDiff line numberDiff line change
@@ -273,6 +273,7 @@ def test_auto_refresh_so_and_unlink_line(self):
273273
self.assertFalse(delivery_line.exists())
274274

275275
def test_auto_add_delivery_line_add_service(self):
276+
self.env["ir.config_parameter"].sudo().set_param(self.param_name1, 1)
276277
service = self.env["product.product"].create(
277278
{"name": "Service Test", "type": "service"}
278279
)

0 commit comments

Comments
 (0)
Please sign in to comment.