opt-in to include value on array item move

benjamine · Feb 7, 2013 · 015a022 · 015a022
1 parent 9f8b7b5
commit 015a022
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/README.md b/README.md
@@ -144,11 +144,8 @@ Array diffing:
                     "name": "La Plata"
                 },0,0],
             "_4": [
-                // moved from index 4 to index 2
-                {
-                    "name": "Rosario",
-                    "population": 1137520
-                },2,3]
+                // move from index 4 to index 2
+                '',2,3]
         }
     }
     */

diff --git a/src/jsondiffpatch.js b/src/jsondiffpatch.js
@@ -13,7 +13,8 @@
     }
     jdp.version = '0.0.6';
     jdp.config = {
-        textDiffMinLength: 60
+        textDiffMinLength: 60,
+        includeValueOnArrayMove: false
     };
 
     var sequenceDiffer = {
@@ -134,6 +135,10 @@
                             if (areTheSameByIndex(removedItems[index1], index)) {
                                 // store position move as: [originalValue, newPosition, 3]
                                 diff['_' + removedItems[index1]].splice(1, 2, index, 3);
+                                if (!jdp.config.includeValueOnArrayMove) {
+                                    // don't include moved value on diff, to save bytes
+                                    diff['_' + removedItems[index1]][0] = '';
+                                }
                                 tryObjectInnerDiff(removedItems[index1], index);
                                 removedItems.splice(index1, 1);
                                 isMove = true;