Fix array props support (#191)

Starcounter-Jack · Aug 29, 2017 · 5312d1d · 5312d1d
1 parent b1d192a
commit 5312d1d
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 5 deletions.
diff --git a/dist/fast-json-patch.js b/dist/fast-json-patch.js
@@ -296,7 +296,12 @@ var objOps = {
 /* The operations applicable to an array. Many are the same as for the object */
 var arrOps = {
     add: function (arr, i, document) {
-        arr.splice(i, 0, this.value);
+        if (helpers_1.isInteger(i)) {
+            arr.splice(i, 0, this.value);
+        }
+        else {
+            arr[i] = this.value;
+        }
         // this may be needed when using '-' in an array
         return { newDocument: document, index: i };
     },
@@ -441,8 +446,10 @@ function applyOperation(document, operation, validateOperation, mutateDocument)
                 else {
                     if (validateOperation && !helpers_1.isInteger(key)) {
                         throw new exports.JsonPatchError("Expected an unsigned base-10 integer value, making the new referenced value the array element with the zero-based index", "OPERATION_PATH_ILLEGAL_ARRAY_INDEX", 0, operation.path, operation);
+                    } // only parse key when it's an integer for `arr.prop` to work
+                    else if (helpers_1.isInteger(key)) {
+                        key = ~~key;
                     }
-                    key = ~~key;
                 }
                 if (t >= len) {
                     if (validateOperation && operation.op === "add" && key > obj.length) {