Merge pull request #12 from chrismwendt/do-not-mutate

Do not mutate the given path
microsoft · Aug 24, 2018 · 85c67a6 · 85c67a6
2 parents 5d3db88 + 80e0c7e
commit 85c67a6
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/impl/edit.ts b/src/impl/edit.ts
@@ -12,7 +12,8 @@ export function removeProperty(text: string, path: JSONPath, formattingOptions:
 	return setProperty(text, path, void 0, formattingOptions);
 }
 
-export function setProperty(text: string, path: JSONPath, value: any, formattingOptions: FormattingOptions, getInsertionIndex?: (properties: string[]) => number): Edit[] {
+export function setProperty(text: string, originalPath: JSONPath, value: any, formattingOptions: FormattingOptions, getInsertionIndex?: (properties: string[]) => number): Edit[] {
+	let path = originalPath.slice()
 	let errors: ParseError[] = [];
 	let root = parseTree(text, errors);
 	let parent: Node | undefined = void 0;
@@ -160,4 +161,4 @@ export function applyEdit(text: string, edit: Edit): string {
 
 export function isWS(text: string, offset: number) {
 	return '\r\n \t'.indexOf(text.charAt(offset)) !== -1;
-}
+}