Fixing command move regression (microsoft#214577)

* adding console logs

* polishing the code

Author: aiday-mar

src/vs/editor/contrib/linesOperations/browser/moveLinesCommand.ts

@@ -42,6 +42,13 @@ export class MoveLinesCommand implements ICommand {
 
 	public getEditOperations(model: ITextModel, builder: IEditOperationBuilder): void {
 
+		const getLanguageId = () => {
+			return model.getLanguageId();
+		};
+		const getLanguageIdAtPosition = (lineNumber: number, column: number) => {
+			return model.getLanguageIdAtPosition(lineNumber, column);
+		};
+
 		const modelLineCount = model.getLineCount();
 
 		if (this._isMovingDown && this._selection.endLineNumber === modelLineCount) {
@@ -63,20 +70,6 @@ export class MoveLinesCommand implements ICommand {
 
 		const { tabSize, indentSize, insertSpaces } = model.getOptions();
 		const indentConverter = this.buildIndentConverter(tabSize, indentSize, insertSpaces);
-		const virtualModel: IVirtualModel = {
-			tokenization: {
-				getLineTokens: (lineNumber: number) => {
-					return model.tokenization.getLineTokens(lineNumber);
-				},
-				getLanguageId: () => {
-					return model.getLanguageId();
-				},
-				getLanguageIdAtPosition: (lineNumber: number, column: number) => {
-					return model.getLanguageIdAtPosition(lineNumber, column);
-				},
-			},
-			getLineContent: null as unknown as (lineNumber: number) => string,
-		};
 
 		if (s.startLineNumber === s.endLineNumber && model.getLineMaxColumn(s.startLineNumber) === 1) {
 			// Current line is empty
@@ -120,12 +113,25 @@ export class MoveLinesCommand implements ICommand {
 						insertingText = newIndentation + this.trimStart(movingLineText);
 					} else {
 						// no enter rule matches, let's check indentatin rules then.
-						virtualModel.getLineContent = (lineNumber: number) => {
-							if (lineNumber === s.startLineNumber) {
-								return model.getLineContent(movingLineNumber);
-							} else {
-								return model.getLineContent(lineNumber);
-							}
+						const virtualModel: IVirtualModel = {
+							tokenization: {
+								getLineTokens: (lineNumber: number) => {
+									if (lineNumber === s.startLineNumber) {
+										return model.tokenization.getLineTokens(movingLineNumber);
+									} else {
+										return model.tokenization.getLineTokens(lineNumber);
+									}
+								},
+								getLanguageId,
+								getLanguageIdAtPosition,
+							},
+							getLineContent: (lineNumber: number) => {
+								if (lineNumber === s.startLineNumber) {
+									return model.getLineContent(movingLineNumber);
+								} else {
+									return model.getLineContent(lineNumber);
+								}
+							},
 						};
 						const indentOfMovingLine = getGoodIndentForLine(
 							this._autoIndent,
@@ -159,14 +165,29 @@ export class MoveLinesCommand implements ICommand {
 						}
 					} else {
 						// it doesn't match onEnter rules, let's check indentation rules then.
-						virtualModel.getLineContent = (lineNumber: number) => {
-							if (lineNumber === s.startLineNumber) {
-								return insertingText;
-							} else if (lineNumber >= s.startLineNumber + 1 && lineNumber <= s.endLineNumber + 1) {
-								return model.getLineContent(lineNumber - 1);
-							} else {
-								return model.getLineContent(lineNumber);
-							}
+						const virtualModel: IVirtualModel = {
+							tokenization: {
+								getLineTokens: (lineNumber: number) => {
+									if (lineNumber === s.startLineNumber) {
+										return model.tokenization.getLineTokens(movingLineNumber);
+									} else if (lineNumber >= s.startLineNumber + 1 && lineNumber <= s.endLineNumber + 1) {
+										return model.tokenization.getLineTokens(lineNumber - 1);
+									} else {
+										return model.tokenization.getLineTokens(lineNumber);
+									}
+								},
+								getLanguageId,
+								getLanguageIdAtPosition,
+							},
+							getLineContent: (lineNumber: number) => {
+								if (lineNumber === s.startLineNumber) {
+									return insertingText;
+								} else if (lineNumber >= s.startLineNumber + 1 && lineNumber <= s.endLineNumber + 1) {
+									return model.getLineContent(lineNumber - 1);
+								} else {
+									return model.getLineContent(lineNumber);
+								}
+							},
 						};
 
 						const newIndentatOfMovingBlock = getGoodIndentForLine(
@@ -204,12 +225,25 @@ export class MoveLinesCommand implements ICommand {
 				builder.addEditOperation(new Range(s.endLineNumber, model.getLineMaxColumn(s.endLineNumber), s.endLineNumber, model.getLineMaxColumn(s.endLineNumber)), '\n' + movingLineText);
 
 				if (this.shouldAutoIndent(model, s)) {
-					virtualModel.getLineContent = (lineNumber: number) => {
-						if (lineNumber === movingLineNumber) {
-							return model.getLineContent(s.startLineNumber);
-						} else {
-							return model.getLineContent(lineNumber);
-						}
+					const virtualModel: IVirtualModel = {
+						tokenization: {
+							getLineTokens: (lineNumber: number) => {
+								if (lineNumber === movingLineNumber) {
+									return model.tokenization.getLineTokens(s.startLineNumber);
+								} else {
+									return model.tokenization.getLineTokens(lineNumber);
+								}
+							},
+							getLanguageId,
+							getLanguageIdAtPosition,
+						},
+						getLineContent: (lineNumber: number) => {
+							if (lineNumber === movingLineNumber) {
+								return model.getLineContent(s.startLineNumber);
+							} else {
+								return model.getLineContent(lineNumber);
+							}
+						},
 					};
 
 					const ret = this.matchEnterRule(model, indentConverter, tabSize, s.startLineNumber, s.startLineNumber - 2);