Merge pull request #6080 from magento-performance/MC-37069

[PERFORMANCE] Configurable Products with large quantities of children cannot be opened

Author: duhon

app/code/Magento/ConfigurableProduct/view/adminhtml/web/js/components/dynamic-rows-configurable.js

@@ -219,19 +219,18 @@ define([
             _.each(tmpData, function (row, index) {
                 path = this.dataScope + '.' + this.index + '.' + (this.startIndex + index);
                 row.attributes = $('<i></i>').text(row.attributes).html();
-                row.sku = row.sku;
                 this.source.set(path, row);
             }, this);
 
             this.source.set(this.dataScope + '.' + this.index, data);
             this.parsePagesData(data);
 
             // Render
-            dataCount = data.length;
+            dataCount = tmpData.length;
             elemsCount = this.elems().length;
 
             if (dataCount > elemsCount) {
-                this.getChildItems().each(function (elemData, index) {
+                tmpData.each(function (elemData, index) {
                     this.addChild(elemData, this.startIndex + index);
                 }, this);
             } else {
@@ -243,6 +242,15 @@ define([
             this.generateAssociatedProducts();
         },
 
+        /**
+         * Set initial property to records data
+         *
+         * @returns {Object} Chainable.
+         */
+        setInitialProperty: function () {
+            return this;
+        },
+
         /**
          * Parsed data
          *

app/code/Magento/Ui/view/base/web/js/dynamic-rows/dynamic-rows.js

@@ -1126,13 +1126,17 @@ define([
          * Update whether value differs from default value
          */
         setDifferedFromDefault: function () {
-            var recordData = utils.copy(this.recordData());
+            var recordData;
 
-            Array.isArray(recordData) && recordData.forEach(function (item) {
-                delete item['record_id'];
-            });
+            if (this.default) {
+                recordData = utils.copy(this.recordData());
+
+                Array.isArray(recordData) && recordData.forEach(function (item) {
+                    delete item['record_id'];
+                });
 
-            this.isDifferedFromDefault(!_.isEqual(recordData, this.default));
+                this.isDifferedFromDefault(!_.isEqual(recordData, this.default));
+            }
         },
 
         /**

dev/tests/js/jasmine/tests/app/code/Magento/ConfigurableProduct/view/adminhtml/web/js/components/dynamic-rows-configurable.test.js

@@ -41,11 +41,9 @@ define([
                     })
                 };
 
-            model.getChildItems = jasmine.createSpy().and.returnValue($(''));
             model.source = sourceMock;
             model.processingUnionInsertData(mockData);
             expect(model.source.get).toHaveBeenCalled();
-            expect(model.getChildItems).toHaveBeenCalled();
             expect(expectedData[1].sku).toBe('Conf&-sdfs');
         });