changeset 0b785a4d3328 in modules/sale_shipment_cost:default
details: 
https://hg.tryton.org/modules/sale_shipment_cost?cmd=changeset;node=0b785a4d3328
description:
        Always set a new shipment cost line on quotation

        The optimization to skip creation based on existing line with the same 
cost
        does not ensure that other properties like taxes or product are valid.

        issue9696
        review292441002
diffstat:

 sale.py |  6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diffs (16 lines):

diff -r 4bffa3c7f8ba -r 0b785a4d3328 sale.py
--- a/sale.py   Fri Oct 16 19:19:58 2020 +0200
+++ b/sale.py   Mon Oct 19 00:13:08 2020 +0200
@@ -214,11 +214,7 @@
         removed = []
         lines = list(self.lines or [])
         for line in self.lines:
-            if line.type != 'line' or not line.shipment_cost:
-                continue
-            if cost_line and line.shipment_cost == cost:
-                cost_line = None
-            else:
+            if line.type == 'line' and line.shipment_cost:
                 lines.remove(line)
                 removed.append(line)
         if cost_line:

Reply via email to