From e10ce07e3a19c0182bf878ae18821e09c3fe35ad Mon Sep 17 00:00:00 2001
From: gushuro <gushurovich@gmail.com>
Date: Thu, 8 Feb 2018 11:45:52 -0800
Subject: [PATCH] Emmet - Fixing bug in toggle comment with multi cursor
 (#42857)

* Fixing issue with toggleComment on multiple nested cursors

* Fix error when an edit is empty

* reordering

* adding a test for toggleComment on nested selections

* Adding test for stylesheet files

* Removing change that was making integration tests fail
---
 .../emmet/src/test/toggleComment.test.ts      | 108 ++++++++++++++++++
 extensions/emmet/src/toggleComment.ts         |  23 +++-
 2 files changed, 128 insertions(+), 3 deletions(-)

diff --git a/extensions/emmet/src/test/toggleComment.test.ts b/extensions/emmet/src/test/toggleComment.test.ts
index 46adb42a8d5..20d8a716d89 100644
--- a/extensions/emmet/src/test/toggleComment.test.ts
+++ b/extensions/emmet/src/test/toggleComment.test.ts
@@ -201,6 +201,52 @@ suite('Tests for Toggle Comment action from Emmet (HTML)', () => {
 			});
 		});
 	});
+
+	test('toggle comment with multiple cursors selecting parent and child nodes', () => {
+		const expectedContents = `
+	<div class="hello">
+		<ul>
+			<li><!--<span>Hello</span>--></li>
+			<!--<li><span>There</span></li>-->
+			<div><li><span>Bye</span></li></div>
+		</ul>
+		<!--<ul>
+			<li>Previously Commented Node</li>
+			<li>Another Node</li>
+		</ul>-->
+		<span/>
+		<!--<style>
+			.boo {
+				margin: 10px;
+				padding: 20px;
+			}
+			.hoo {
+				margin: 10px;
+				padding: 20px;
+			}
+		</style>-->
+	</div>
+	`;
+		return withRandomFileEditor(contents, 'html', (editor, doc) => {
+			editor.selections = [
+				new Selection(3, 17, 3, 17), // cursor inside the inner span element
+				new Selection(4, 5, 4, 5), // two cursors: one inside opening tag
+				new Selection(4, 17, 4, 17), // 		and the second inside the inner span element
+				new Selection(7, 3, 7, 3), // two cursors: one inside open tag of <ul> one of of whose children is already commented
+				new Selection(9, 10, 9, 10), // 	and the second inside inner li element, whose parent is selected
+				new Selection(12, 3, 12, 3), // four nested cursors: one inside the style open tag
+				new Selection(14, 8, 14, 8), // 	the second inside the css property inside the style tag
+				new Selection(18, 3, 18, 3), // 	the third inside the css rule inside the style tag
+				new Selection(19, 8, 19, 8) // 		and the fourth inside the css property inside the style tag
+			];
+
+			return toggleComment().then(() => {
+				assert.equal(doc.getText(), expectedContents);
+
+				return Promise.resolve();
+			});
+		});
+	});
 });
 
 suite('Tests for Toggle Comment action from Emmet (CSS)', () => {
@@ -470,6 +516,68 @@ suite('Tests for Toggle Comment action from Emmet in nested css (SCSS)', () => {
 		}
 	}`;
 
+	test('toggle comment with multiple cursors selecting nested nodes (SCSS)', () => {
+		const expectedContents = `
+	.one {
+		/*height: 42px;*/
+
+		/*.two {
+			width: 42px;
+		}*/
+
+		.three {
+			/*padding: 10px;*/
+		}
+	}`;
+		return withRandomFileEditor(contents, 'css', (editor, doc) => {
+			editor.selections = [
+				new Selection(2, 5, 2, 5), // cursor inside a property
+				new Selection(4, 4, 4, 4), // two cursors: one inside a nested rule
+				new Selection(5, 5, 5, 5), // 		and the second one inside a nested property
+				new Selection(9, 5, 9, 5) // cursor inside a property inside a nested rule
+			];
+
+			return toggleComment().then(() => {
+				assert.equal(doc.getText(), expectedContents);
+				return toggleComment().then(() => {
+					assert.equal(doc.getText(), contents);
+				return Promise.resolve();
+				});
+			});
+		});
+	});
+	test('toggle comment with multiple cursors selecting several nested nodes (SCSS)', () => {
+		const expectedContents = `
+	/*.one {
+		height: 42px;
+
+		.two {
+			width: 42px;
+		}
+
+		.three {
+			padding: 10px;
+		}
+	}*/`;
+		return withRandomFileEditor(contents, 'css', (editor, doc) => {
+			editor.selections = [
+				new Selection(1, 3, 1, 3), // cursor in the outside rule. And several cursors inside:
+				new Selection(2, 5, 2, 5), // cursor inside a property
+				new Selection(4, 4, 4, 4), // two cursors: one inside a nested rule
+				new Selection(5, 5, 5, 5), // 		and the second one inside a nested property
+				new Selection(9, 5, 9, 5) // cursor inside a property inside a nested rule
+			];
+
+			return toggleComment().then(() => {
+				assert.equal(doc.getText(), expectedContents);
+				return toggleComment().then(() => {
+					assert.equal(doc.getText(), contents);
+					return Promise.resolve();
+				});
+			});
+		});
+	});
+
 	test('toggle comment with multiple cursors, but no selection (SCSS)', () => {
 		const expectedContents = `
 	.one {
diff --git a/extensions/emmet/src/toggleComment.ts b/extensions/emmet/src/toggleComment.ts
index b18106d98a2..dc756f5b990 100644
--- a/extensions/emmet/src/toggleComment.ts
+++ b/extensions/emmet/src/toggleComment.ts
@@ -33,12 +33,29 @@ export function toggleComment(): Thenable<boolean> | undefined {
 	}
 
 	return editor.edit(editBuilder => {
+		let allEdits: vscode.TextEdit[][] = [];
 		editor.selections.reverse().forEach(selection => {
 			let edits = toggleCommentInternal(editor.document, selection, rootNode!);
-			edits.forEach(x => {
-				editBuilder.replace(x.range, x.newText);
-			});
+			if (edits.length > 0) {
+				allEdits.push(edits);
+			}
 		});
+
+		// Apply edits in order so we can skip nested ones.
+		allEdits.sort((arr1, arr2) => {
+			let result = arr1[0].range.start.line - arr2[0].range.start.line;
+			return result === 0 ? arr1[0].range.start.character - arr2[0].range.start.character : result;
+		});
+		let lastEditPosition = new vscode.Position(0, 0);
+		for (let i = 0; i < allEdits.length; i++) {
+			const edits = allEdits[i];
+			if (edits[0].range.end.isAfterOrEqual(lastEditPosition)) {
+				edits.forEach(x => {
+					editBuilder.replace(x.range, x.newText);
+					lastEditPosition = x.range.end;
+				});
+			}
+		}
 	});
 }
 
-- 
GitLab