explain the explicit call to trapForSet

Palindrom · Aug 15, 2019 · e81f6a4 · e81f6a4
1 parent 21af2ec
commit e81f6a4
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/src/jsonpatcherproxy.js b/src/jsonpatcherproxy.js
@@ -118,6 +118,8 @@ const JSONPatcherProxy = (function() {
     if (isTreeAnArray && !isNonSerializableArrayProperty) {
       const index = parseInt(key, 10);
       if (index > tree.length) {
+        // force call trapForSet for implicit undefined elements of the array added by the JS engine
+        // because JSON-Patch spec prohibits adding an index that is higher than array.length
         trapForSet(instance, tree, (index - 1) + '', undefined);
       }
     }