MAGETWO-52974: CLONE - Configurable product options not saved when editing

Author: Olga Nakonechna

app/code/Magento/ConfigurableProduct/view/frontend/web/js/configurable-customer-data.js

@@ -11,7 +11,7 @@ require([
     },
     configurableWidget,
     productOptions,
-    tempProductOptions,
+    changedProductOptions,
     cartData = customerData.get('cart'),
     productId = $(selectors.productIdSelector).val(),
     updateConfigurableOptions,
@@ -40,21 +40,21 @@ require([
         if (!(data && data.items && data.items.length && productId)) {
             return false;
         }
-        tempProductOptions = data.items.find(function (item) {
+        changedProductOptions = data.items.find(function (item) {
             return item['product_id'] === productId;
         });
-        tempProductOptions = tempProductOptions && tempProductOptions.options &&
-            tempProductOptions.options.reduce(function (obj, val) {
+        changedProductOptions = changedProductOptions && changedProductOptions.options &&
+            changedProductOptions.options.reduce(function (obj, val) {
                 obj[val['option_id']] = val['option_value'];
 
                 return obj;
             }, {});
 
-        if (JSON.stringify(productOptions || {}) === JSON.stringify(tempProductOptions || {})) {
+        if (JSON.stringify(productOptions || {}) === JSON.stringify(changedProductOptions || {})) {
             return false;
         }
 
-        productOptions = tempProductOptions;
+        productOptions = changedProductOptions;
 
         return true;
     };

app/code/Magento/Swatches/view/frontend/web/js/configurable-customer-data.js

@@ -12,7 +12,7 @@ require([
     },
     swatchWidget,
     productOptions,
-    tempProductOptions,
+    changedProductOptions,
     cartData = customerData.get('cart'),
     productId = $(selectors.productIdSelector).val(),
     updateSwatchOptions,
@@ -40,21 +40,21 @@ require([
         if (!(data && data.items && data.items.length && productId)) {
             return false;
         }
-        tempProductOptions = data.items.find(function (item) {
+        changedProductOptions = data.items.find(function (item) {
             return item['product_id'] === productId;
         });
-        tempProductOptions = tempProductOptions && tempProductOptions.options &&
-            tempProductOptions.options.reduce(function (obj, val) {
+        changedProductOptions = changedProductOptions && changedProductOptions.options &&
+            changedProductOptions.options.reduce(function (obj, val) {
                 obj[val['option_id']] = val['option_value'];
 
                 return obj;
             }, {});
 
-        if (JSON.stringify(productOptions || {}) === JSON.stringify(tempProductOptions || {})) {
+        if (JSON.stringify(productOptions || {}) === JSON.stringify(changedProductOptions || {})) {
             return false;
         }
 
-        productOptions = tempProductOptions;
+        productOptions = changedProductOptions;
 
         return true;
     };

app/code/Magento/Swatches/view/frontend/web/js/swatch-renderer.js

@@ -962,17 +962,17 @@ define([
                 var elem = this.element.find('.' + this.options.classes.attributeClass +
                     '[attribute-id="' + attributeId + '"] [option-id="' + optionId + '"]'),
                     parentInput = elem.parent();
-            
-                    if (elem.hasClass('selected')) {
-                        return;
-                    }
 
-                    if (parentInput.hasClass(this.options.classes.selectClass)) {
-                        parentInput.val(optionId);
-                        parentInput.trigger('change');
-                    } else {
-                        elem.trigger('click');
-                    }
+                if (elem.hasClass('selected')) {
+                    return;
+                }
+
+                if (parentInput.hasClass(this.options.classes.selectClass)) {
+                    parentInput.val(optionId);
+                    parentInput.trigger('change');
+                } else {
+                    elem.trigger('click');
+                }
             }, this));
         },