ENGCOM-5502: Fixed an issue where an attribute named length would break the use default value checkbox from saving #23725

Author: VladimirZaets

lib/web/mage/utils/misc.js

@@ -231,7 +231,7 @@ define([
             separator = separator || '-';
 
             _.each(data, function (value, key) {
-                if (_.isObject(value) && !value.length) {
+                if (_.isObject(value) && !Array.isArray(value)) {
                     this.filterFormData(value, suffix, separator);
                 } else if (_.isString(key) && ~key.indexOf(suffix)) {
                     data[key.split(separator)[0]] = value;

lib/web/mage/utils/objects.js

@@ -148,7 +148,18 @@ define([
             separator = separator || '.';
             result = result || {};
 
-            _.each(data, function (node, name) {
+            if (!data) {
+                return result;
+            }
+
+            // UnderscoreJS each breaks when an object has a length property so we use Object.keys
+            _.each(Object.keys(data), function (name) {
+                var node = data[name];
+
+                if ({}.toString.call(node) === '[object Function]') {
+                    return;
+                }
+
                 if (parent) {
                     name = parent + separator + name;
                 }
@@ -436,3 +447,4 @@ define([
         }
     };
 });
+