From f4e9ef595a127ef7ee40865e5f59ec7c5ebc5f44 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Krier?= Date: Sun, 20 Oct 2024 22:12:32 +0200 Subject: [PATCH] Select by default the first active carrier The available carries can contain inactive carries because it is just a list of carriers matching the criteria. Closes #12888 --- modules/sale_shipment_cost/sale.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/modules/sale_shipment_cost/sale.py b/modules/sale_shipment_cost/sale.py index 7fa8e68346..3ff1d5c923 100644 --- a/modules/sale_shipment_cost/sale.py +++ b/modules/sale_shipment_cost/sale.py @@ -177,7 +177,10 @@ class Sale(metaclass=PoolMeta): elif self.shipment_cost_method: if (not self.carrier or self.carrier not in self.available_carriers): - self.carrier = self.available_carriers[0] + for carrier in self.available_carriers: + if carrier.active: + self.carrier = carrier + break @fields.depends( 'carrier', 'shipment_cost_method',