From 3e4311eb5e03be368f1cbc5b4fff067d814af115 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Tue, 3 Jan 2023 11:23:03 +0100
Subject: [PATCH] Do not enforce empty shipment cost method for draft sale

As draft sale can be created by other modules, we must put such constraint
when quoting the sale.

Closes #11900
---
 modules/sale_shipment_cost/sale.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/sale_shipment_cost/sale.py b/modules/sale_shipment_cost/sale.py
index 78392ca72c..5230f493db 100644
--- a/modules/sale_shipment_cost/sale.py
+++ b/modules/sale_shipment_cost/sale.py
@@ -98,7 +98,7 @@ class Sale(metaclass=PoolMeta):
             ('shipment', "On Shipment"),
             ], "Shipment Cost Method",
         domain=[
-            If(~Eval('carrier'),
+            If(~Eval('carrier') & (Eval('state') != 'draft'),
                 ('shipment_cost_method', '=', None),
                 ()),
             ],