[json] "Format document" on a large JSON file adds extra bracket Fixes microsoft#91125

aeschli · aeschli · commit cfc07ec7aa80 · 2020-02-21T09:36:54.000+01:00
diff --git a/extensions/json-language-features/server/src/jsonServerMain.ts b/extensions/json-language-features/server/src/jsonServerMain.ts
@@ -450,7 +450,7 @@ connection.onDocumentRangeFormatting((formatParams, token) => {
 			const edits = languageService.format(document, formatParams.range, formatParams.options);
 			if (edits.length > formatterMaxNumberOfEdits) {
 				const newText = TextDocument.applyEdits(document, edits);
-				return [TextEdit.replace(Range.create(Position.create(0, 0), document.positionAt(document.getText().length - 1)), newText)];
+				return [TextEdit.replace(Range.create(Position.create(0, 0), document.positionAt(document.getText().length)), newText)];
 			}
 			return edits;
 		}

Original file line number	Diff line number	Diff line change
`@@ -450,7 +450,7 @@ connection.onDocumentRangeFormatting((formatParams, token) => {`
`450`	`450`	`const edits = languageService.format(document, formatParams.range, formatParams.options);`
`451`	`451`	`if (edits.length > formatterMaxNumberOfEdits) {`
`452`	`452`	`const newText = TextDocument.applyEdits(document, edits);`
`453`		`- return [TextEdit.replace(Range.create(Position.create(0, 0), document.positionAt(document.getText().length - 1)), newText)];`
	`453`	`+ return [TextEdit.replace(Range.create(Position.create(0, 0), document.positionAt(document.getText().length)), newText)];`
`454`	`454`	`}`
`455`	`455`	`return edits;`
`456`	`456`	`}`