Make sure we set the correct replacement range for dot member completions
Fixes #91105
This commit is contained in:
parent
31f757f254
commit
a10e4cb911
|
@ -83,7 +83,15 @@ class MyCompletionItem extends vscode.CompletionItem {
|
||||||
if (completionContext.isMemberCompletion && completionContext.dotAccessorContext) {
|
if (completionContext.isMemberCompletion && completionContext.dotAccessorContext) {
|
||||||
this.filterText = completionContext.dotAccessorContext.text + (this.insertText || this.label);
|
this.filterText = completionContext.dotAccessorContext.text + (this.insertText || this.label);
|
||||||
if (!this.range) {
|
if (!this.range) {
|
||||||
this.range = completionContext.dotAccessorContext.range;
|
const replacementRange = this.getReplaceRange(line);
|
||||||
|
if (replacementRange) {
|
||||||
|
this.range = {
|
||||||
|
inserting: completionContext.dotAccessorContext.range,
|
||||||
|
replacing: completionContext.dotAccessorContext.range.union(replacementRange),
|
||||||
|
};
|
||||||
|
} else {
|
||||||
|
this.range = completionContext.dotAccessorContext.range;
|
||||||
|
}
|
||||||
this.insertText = this.filterText;
|
this.insertText = this.filterText;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -135,7 +143,6 @@ class MyCompletionItem extends vscode.CompletionItem {
|
||||||
} else {
|
} else {
|
||||||
return wordStart === '#' ? undefined : this.tsEntry.name.replace(/^#/, '');
|
return wordStart === '#' ? undefined : this.tsEntry.name.replace(/^#/, '');
|
||||||
}
|
}
|
||||||
return undefined;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
// For `this.` completions, generally don't set the filter text since we don't want them to be overly prioritized. #74164
|
// For `this.` completions, generally don't set the filter text since we don't want them to be overly prioritized. #74164
|
||||||
|
@ -162,6 +169,16 @@ class MyCompletionItem extends vscode.CompletionItem {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
const replaceRange = this.getReplaceRange(line);
|
||||||
|
if (replaceRange) {
|
||||||
|
this.range = {
|
||||||
|
inserting: new vscode.Range(replaceRange.start, this.position),
|
||||||
|
replacing: replaceRange
|
||||||
|
};
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
private getReplaceRange(line: string) {
|
||||||
const wordRange = this.document.getWordRangeAtPosition(this.position);
|
const wordRange = this.document.getWordRangeAtPosition(this.position);
|
||||||
let replaceRange = wordRange;
|
let replaceRange = wordRange;
|
||||||
|
|
||||||
|
@ -177,12 +194,7 @@ class MyCompletionItem extends vscode.CompletionItem {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (replaceRange) {
|
return replaceRange;
|
||||||
this.range = {
|
|
||||||
inserting: new vscode.Range(replaceRange.start, this.position),
|
|
||||||
replacing: replaceRange
|
|
||||||
};
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private static convertKind(kind: string): vscode.CompletionItemKind {
|
private static convertKind(kind: string): vscode.CompletionItemKind {
|
||||||
|
|
|
@ -622,4 +622,29 @@ suite('TypeScript Completions', () => {
|
||||||
`Config: ${config}`);
|
`Config: ${config}`);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
|
test('Replace should work after this. (#91105)', async () => {
|
||||||
|
await updateConfig(testDocumentUri, { [Config.insertMode]: 'replace' });
|
||||||
|
|
||||||
|
const editor = await createTestEditor(testDocumentUri,
|
||||||
|
`class A {`,
|
||||||
|
` abc = 1`,
|
||||||
|
` foo() {`,
|
||||||
|
` this.$0abc`,
|
||||||
|
` }`,
|
||||||
|
`}`,
|
||||||
|
);
|
||||||
|
|
||||||
|
await acceptFirstSuggestion(testDocumentUri, _disposables);
|
||||||
|
|
||||||
|
assertEditorContents(editor,
|
||||||
|
joinLines(
|
||||||
|
`class A {`,
|
||||||
|
` abc = 1`,
|
||||||
|
` foo() {`,
|
||||||
|
` this.abc`,
|
||||||
|
` }`,
|
||||||
|
`}`,
|
||||||
|
));
|
||||||
|
});
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue