Fix: Patch to nested object is changing object content order #300

Author: mesqueeb

dist/index.cjs.js

@@ -331,6 +331,15 @@ function isIncrementHelper(payload) {
         payload.isIncrementHelper === true);
 }
 
+function convertHelpers(originVal, newVal) {
+    if (isWhat.isArray(originVal) && isArrayHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    if (isWhat.isNumber(originVal) && isIncrementHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    return newVal; // always return newVal as fallback!!
+}
 /**
  * Creates the params needed to $set a target based on a nested.path
  *
@@ -434,17 +443,6 @@ function pluginMutations (userState) {
             }
             if (!ref)
                 return error('patch-no-ref');
-            function convertHelpers(originVal, newVal) {
-                if (isWhat.isArray(originVal) && isArrayHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                if (isWhat.isNumber(originVal) && isIncrementHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                return newVal; // always return newVal as fallback!!
-            }
-            // const refPropsPicked = filter(ref, Object.keys(patches))
-            // const patchesSanitised = merge({ extensions: [convertHelpers] }, refPropsPicked, patches)
             var patchesFlat = flatten.flattenObject(patches);
             for (var _i = 0, _b = Object.entries(patchesFlat); _i < _b.length; _i++) {
                 var _c = _b[_i], path = _c[0], value = _c[1];

dist/index.esm.js

@@ -324,6 +324,15 @@ function isIncrementHelper(payload) {
         payload.isIncrementHelper === true);
 }
 
+function convertHelpers(originVal, newVal) {
+    if (isArray(originVal) && isArrayHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    if (isNumber(originVal) && isIncrementHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    return newVal; // always return newVal as fallback!!
+}
 /**
  * Creates the params needed to $set a target based on a nested.path
  *
@@ -427,17 +436,6 @@ function pluginMutations (userState) {
             }
             if (!ref)
                 return error('patch-no-ref');
-            function convertHelpers(originVal, newVal) {
-                if (isArray(originVal) && isArrayHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                if (isNumber(originVal) && isIncrementHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                return newVal; // always return newVal as fallback!!
-            }
-            // const refPropsPicked = filter(ref, Object.keys(patches))
-            // const patchesSanitised = merge({ extensions: [convertHelpers] }, refPropsPicked, patches)
             var patchesFlat = flattenObject(patches);
             for (var _i = 0, _b = Object.entries(patchesFlat); _i < _b.length; _i++) {
                 var _c = _b[_i], path = _c[0], value = _c[1];

package.json

@@ -1,6 +1,6 @@
 {
   "name": "vuex-easy-firestore",
-  "version": "1.35.8",
+  "version": "1.35.9",
   "description": "Easy coupling of firestore and a vuex module. 2-way sync with 0 boilerplate!",
   "main": "dist/index.cjs.js",
   "module": "dist/index.esm.js",
@@ -72,14 +72,8 @@
   },
   "ava": {
     "compileEnhancements": false,
-    "extensions": [
-      "ts"
-    ],
-    "require": [
-      "ts-node/register"
-    ],
-    "helpers": [
-      "**/helpers/**/*"
-    ]
+    "extensions": ["ts"],
+    "require": ["ts-node/register"],
+    "helpers": ["**/helpers/**/*"]
   }
 }

test/helpers/index.cjs.js

@@ -850,6 +850,15 @@ function isIncrementHelper(payload) {
         payload.isIncrementHelper === true);
 }
 
+function convertHelpers(originVal, newVal) {
+    if (isWhat.isArray(originVal) && isArrayHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    if (isWhat.isNumber(originVal) && isIncrementHelper(newVal)) {
+        newVal = newVal.executeOn(originVal);
+    }
+    return newVal; // always return newVal as fallback!!
+}
 /**
  * Creates the params needed to $set a target based on a nested.path
  *
@@ -953,17 +962,6 @@ function pluginMutations (userState) {
             }
             if (!ref)
                 return error('patch-no-ref');
-            function convertHelpers(originVal, newVal) {
-                if (isWhat.isArray(originVal) && isArrayHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                if (isWhat.isNumber(originVal) && isIncrementHelper(newVal)) {
-                    newVal = newVal.executeOn(originVal);
-                }
-                return newVal; // always return newVal as fallback!!
-            }
-            // const refPropsPicked = filter(ref, Object.keys(patches))
-            // const patchesSanitised = merge({ extensions: [convertHelpers] }, refPropsPicked, patches)
             var patchesFlat = flatten.flattenObject(patches);
             for (var _i = 0, _b = Object.entries(patchesFlat); _i < _b.length; _i++) {
                 var _c = _b[_i], path = _c[0], value = _c[1];