From 3aa31f8bcbf87c83458ac65189edbe6bd63c4290 Mon Sep 17 00:00:00 2001 From: Jared Kipe Date: Tue, 24 Mar 2020 09:16:30 -0700 Subject: [PATCH] FIX `connector_opencart` refactor the coupon naming that overwrote the local variable named values --- connector_opencart/models/sale_order/importer.py | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/connector_opencart/models/sale_order/importer.py b/connector_opencart/models/sale_order/importer.py index dd8f1308..ddd20665 100644 --- a/connector_opencart/models/sale_order/importer.py +++ b/connector_opencart/models/sale_order/importer.py @@ -81,12 +81,11 @@ class SaleOrderImportMapper(Component): line_builder.price_unit = -float(coupon.get('amount', 0.0)) line_builder.product = coupon_product # `order.line.builder` does not allow naming. - values = line_builder.get_line() + line_values = line_builder.get_line() code = coupon.get('code') if code: - values['name'] = '%s Code: %s' % (coupon_product.name, code) - line = (0, 0, values) - values['order_line'].append(line) + line_values['name'] = '%s Code: %s' % (coupon_product.name, code) + values['order_line'].append((0, 0, line_values)) return values def _add_shipping_line(self, map_record, values):