Skip to content

Commit f963941

Browse files
CsCherrYYCsCherrYY
committed
support rename references
Signed-off-by: Shi Chen <chenshi@microsoft.com>
1 parent 06e6564 commit f963941

12 files changed

Lines changed: 296 additions & 8 deletions

File tree

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/ResourceUtils.java‎

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -383,4 +383,17 @@ public static IMarker createWarningMarker(String type, IResource resource, Strin
383383
marker.setAttribute(IMarker.SEVERITY, IMarker.SEVERITY_WARNING);
384384
return marker;
385385
}
386+
387+
/**
388+
* Creates a simple info marker with the given type, id and status message to the given resource.
389+
*/
390+
public static IMarker createInfoMarker(String type, IResource resource, String message, int id, int start, int end) throws CoreException {
391+
IMarker marker = resource.createMarker(type);
392+
marker.setAttribute(IJavaModelMarker.ID, id);
393+
marker.setAttribute(IMarker.MESSAGE, message);
394+
marker.setAttribute(IMarker.SEVERITY, IMarker.SEVERITY_INFO);
395+
marker.setAttribute(IMarker.CHAR_START, start);
396+
marker.setAttribute(IMarker.CHAR_END, end);
397+
return marker;
398+
}
386399
}

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/CorrectionMessages.java‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -127,6 +127,7 @@ private CorrectionMessages() {
127127
public static String LocalCorrectionsSubProcessor_hiding_field_label;
128128
public static String LocalCorrectionsSubProcessor_rename_var_label;
129129
public static String LocalCorrectionsSubProcessor_hiding_argument_label;
130+
public static String LocalCorrectionsSubProcessor_manual_rename_label;
130131
public static String LocalCorrectionsSubProcessor_setparenteses_description;
131132
public static String LocalCorrectionsSubProcessor_setparenteses_instanceof_description;
132133
public static String LocalCorrectionsSubProcessor_InferGenericTypeArguments;

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/CorrectionMessages.properties‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@ LocalCorrectionsSubProcessor_hiding_local_label=Rename local variable ''{0}''
115115
LocalCorrectionsSubProcessor_hiding_field_label=Rename field ''{0}''
116116
LocalCorrectionsSubProcessor_rename_var_label=Rename ''{0}''
117117
LocalCorrectionsSubProcessor_hiding_argument_label=Rename argument ''{0}''
118+
LocalCorrectionsSubProcessor_manual_rename_label=Rename usages of ''{0}'' to ''{1}''
118119
LocalCorrectionsSubProcessor_setparenteses_description=Put ''{0}'' expression in parentheses
119120
LocalCorrectionsSubProcessor_InferGenericTypeArguments=Infer Generic Type Arguments...
120121
LocalCorrectionsSubProcessor_InferGenericTypeArguments_description=Start the 'Infer Generic Type Arguments' refactoring

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/QuickFixProcessor.java‎

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@
5151
import org.eclipse.jdt.ls.core.internal.corrections.proposals.ReplaceCorrectionProposal;
5252
import org.eclipse.jdt.ls.core.internal.corrections.proposals.TypeMismatchSubProcessor;
5353
import org.eclipse.jdt.ls.core.internal.corrections.proposals.UnresolvedElementsSubProcessor;
54+
import org.eclipse.jdt.ls.core.internal.handlers.BaseDocumentLifeCycleHandler;
55+
import org.eclipse.jdt.ls.core.internal.handlers.DocumentLifeCycleHandler;
5456
import org.eclipse.jdt.ls.core.internal.handlers.OrganizeImportsHandler;
5557
import org.eclipse.jdt.ls.core.internal.text.correction.ModifierCorrectionSubProcessor;
5658
import org.eclipse.lsp4j.CodeActionKind;
@@ -75,7 +77,7 @@ private static int moveBack(int offset, int start, String ignoreCharacters, ICom
7577
return start;
7678
}
7779

78-
public List<ChangeCorrectionProposal> getCorrections(CodeActionParams params, IInvocationContext context, IProblemLocationCore[] locations) throws CoreException {
80+
public List<ChangeCorrectionProposal> getCorrections(CodeActionParams params, IInvocationContext context, IProblemLocationCore[] locations, DocumentLifeCycleHandler documentLifeCycleHandler) throws CoreException {
7981
if (locations == null || locations.length == 0) {
8082
return Collections.emptyList();
8183
}
@@ -84,7 +86,7 @@ public List<ChangeCorrectionProposal> getCorrections(CodeActionParams params, II
8486
for (int i = 0; i < locations.length; i++) {
8587
IProblemLocationCore curr = locations[i];
8688
if (handledProblems(curr, locations, handledProblems)) {
87-
process(params, context, curr, resultingCollections);
89+
process(params, context, curr, resultingCollections, documentLifeCycleHandler);
8890
}
8991
}
9092
return resultingCollections;
@@ -107,7 +109,7 @@ private static boolean handledProblems(IProblemLocationCore location, IProblemLo
107109
return handledProblems.add(problemId);
108110
}
109111

110-
private void process(CodeActionParams params, IInvocationContext context, IProblemLocationCore problem, Collection<ChangeCorrectionProposal> proposals) throws CoreException {
112+
private void process(CodeActionParams params, IInvocationContext context, IProblemLocationCore problem, Collection<ChangeCorrectionProposal> proposals, DocumentLifeCycleHandler documentLifeCycleHandler) throws CoreException {
111113
int id = problem.getProblemId();
112114
if (id == 0) { // no proposals for none-problem locations
113115
return;
@@ -668,7 +670,9 @@ private void process(CodeActionParams params, IInvocationContext context, IProbl
668670
// TypeAnnotationSubProcessor.addMoveTypeAnnotationToTypeProposal(context,
669671
// problem, proposals);
670672
// break;
671-
673+
case BaseDocumentLifeCycleHandler.RENAME_REFERENCE_PROBLEM_ID:
674+
LocalCorrectionsSubProcessor.getManualRenameCorrectionProposals(context, problem, proposals, documentLifeCycleHandler);
675+
break;
672676
default:
673677
String str = problem.toString();
674678
System.out.println(str);

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/corrections/proposals/LocalCorrectionsSubProcessor.java‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,16 @@
2121
package org.eclipse.jdt.ls.core.internal.corrections.proposals;
2222

2323
import java.util.ArrayList;
24+
import java.util.Arrays;
2425
import java.util.Collection;
2526
import java.util.HashMap;
2627
import java.util.Hashtable;
2728
import java.util.List;
2829
import java.util.Map;
30+
import java.util.Objects;
2931

32+
import org.eclipse.core.resources.IMarker;
33+
import org.eclipse.core.resources.IResource;
3034
import org.eclipse.core.runtime.CoreException;
3135
import org.eclipse.jdt.core.ICompilationUnit;
3236
import org.eclipse.jdt.core.IType;
@@ -101,7 +105,9 @@
101105
import org.eclipse.jdt.internal.ui.fix.UnnecessaryCodeCleanUpCore;
102106
import org.eclipse.jdt.internal.ui.text.correction.IProblemLocationCore;
103107
import org.eclipse.jdt.internal.ui.util.ASTHelper;
108+
import org.eclipse.jdt.ls.core.internal.JDTUtils;
104109
import org.eclipse.jdt.ls.core.internal.JavaLanguageServerPlugin;
110+
import org.eclipse.jdt.ls.core.internal.Messages;
105111
import org.eclipse.jdt.ls.core.internal.corext.refactoring.surround.ExceptionAnalyzer;
106112
import org.eclipse.jdt.ls.core.internal.corext.refactoring.surround.SurroundWithTryCatchRefactoring;
107113
import org.eclipse.jdt.ls.core.internal.corrections.CorrectionMessages;
@@ -111,13 +117,18 @@
111117
import org.eclipse.jdt.ls.core.internal.corrections.proposals.ChangeMethodSignatureProposal.ChangeDescription;
112118
import org.eclipse.jdt.ls.core.internal.corrections.proposals.ChangeMethodSignatureProposal.InsertDescription;
113119
import org.eclipse.jdt.ls.core.internal.corrections.proposals.ChangeMethodSignatureProposal.RemoveDescription;
120+
import org.eclipse.jdt.ls.core.internal.handlers.BaseDocumentLifeCycleHandler;
121+
import org.eclipse.jdt.ls.core.internal.handlers.DocumentLifeCycleHandler;
122+
import org.eclipse.jdt.ls.core.internal.text.correction.CUCorrectionCommandProposal;
114123
import org.eclipse.jdt.ls.core.internal.text.correction.ModifierCorrectionSubProcessor;
115124
import org.eclipse.jdt.ls.core.internal.text.correction.QuickAssistProcessor;
116125
import org.eclipse.lsp4j.CodeActionKind;
126+
import org.eclipse.lsp4j.Range;
117127

118128
public class LocalCorrectionsSubProcessor {
119129

120130
private static final String ADD_STATIC_ACCESS_ID = "org.eclipse.jdt.ui.correction.changeToStatic"; //$NON-NLS-1$
131+
private static final String RENAME_REFERENCES_COMMAND = "java.action.renameReferences";
121132

122133
public static void addUncaughtExceptionProposals(IInvocationContext context, IProblemLocationCore problem, Collection<ChangeCorrectionProposal> proposals) throws CoreException {
123134
ICompilationUnit cu = context.getCompilationUnit();
@@ -1109,4 +1120,33 @@ public static void addValueForAnnotationProposals(IInvocationContext context, IP
11091120
}
11101121
}
11111122

1123+
public static void getManualRenameCorrectionProposals(IInvocationContext context, IProblemLocationCore problem, Collection<ChangeCorrectionProposal> proposals, DocumentLifeCycleHandler documentLifeCycleHandler) throws CoreException {
1124+
if (!JavaLanguageServerPlugin.getPreferencesManager().getPreferences().getRenameReferencesEnabled()) {
1125+
return;
1126+
}
1127+
ICompilationUnit cu = context.getCompilationUnit();
1128+
if (cu == null) {
1129+
return;
1130+
}
1131+
IResource resource = cu.getResource();
1132+
if (resource == null) {
1133+
return;
1134+
}
1135+
IMarker[] markers = resource.findMarkers(BaseDocumentLifeCycleHandler.RENAME_REFERENCE_MARKER_ID, false, IResource.DEPTH_ONE);
1136+
Arrays.stream(markers).forEach(marker -> {
1137+
try {
1138+
if (Objects.equals(problem.getOffset(), marker.getAttribute(IMarker.CHAR_START)) && Objects.equals(problem.getOffset() + problem.getLength(), marker.getAttribute(IMarker.CHAR_END))) {
1139+
String originalName = (String) marker.getAttribute("originalName");
1140+
String newName = (String) marker.getAttribute("newName");
1141+
Range range = JDTUtils.toRange(cu, problem.getOffset(), problem.getLength());
1142+
if (originalName != null && newName != null) {
1143+
String label = Messages.format(CorrectionMessages.LocalCorrectionsSubProcessor_manual_rename_label, new String[]{ originalName, newName });
1144+
proposals.add(new CUCorrectionCommandProposal(label, CodeActionKind.QuickFix, cu, IProposalRelevance.RENAME_REFACTORING_QUICK_FIX, RENAME_REFERENCES_COMMAND, Arrays.asList(cu.getResource().getLocationURI().toString(), originalName, newName, range)));
1145+
}
1146+
}
1147+
} catch (CoreException e) {
1148+
// Do nothing
1149+
}
1150+
});
1151+
}
11121152
}

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BaseDocumentLifeCycleHandler.java‎

Lines changed: 123 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,22 +46,30 @@
4646
import org.eclipse.jdt.core.IJavaElement;
4747
import org.eclipse.jdt.core.IJavaModelMarker;
4848
import org.eclipse.jdt.core.IJavaProject;
49+
import org.eclipse.jdt.core.IMember;
4950
import org.eclipse.jdt.core.IPackageFragment;
5051
import org.eclipse.jdt.core.IProblemRequestor;
52+
import org.eclipse.jdt.core.ISourceRange;
53+
import org.eclipse.jdt.core.IType;
5154
import org.eclipse.jdt.core.JavaCore;
5255
import org.eclipse.jdt.core.JavaModelException;
5356
import org.eclipse.jdt.core.WorkingCopyOwner;
5457
import org.eclipse.jdt.core.compiler.IProblem;
5558
import org.eclipse.jdt.core.dom.CompilationUnit;
59+
import org.eclipse.jdt.core.dom.Modifier;
5660
import org.eclipse.jdt.core.manipulation.CoreASTProvider;
5761
import org.eclipse.jdt.internal.core.OpenableElementInfo;
5862
import org.eclipse.jdt.internal.core.PackageFragment;
5963
import org.eclipse.jdt.ls.core.internal.DocumentAdapter;
64+
import org.eclipse.jdt.ls.core.internal.IConstants;
6065
import org.eclipse.jdt.ls.core.internal.JDTUtils;
6166
import org.eclipse.jdt.ls.core.internal.JavaLanguageServerPlugin;
6267
import org.eclipse.jdt.ls.core.internal.JobHelpers;
68+
import org.eclipse.jdt.ls.core.internal.Messages;
6369
import org.eclipse.jdt.ls.core.internal.MovingAverage;
6470
import org.eclipse.jdt.ls.core.internal.ProjectUtils;
71+
import org.eclipse.jdt.ls.core.internal.ResourceUtils;
72+
import org.eclipse.jdt.ls.core.internal.corrections.CorrectionMessages;
6573
import org.eclipse.jdt.ls.core.internal.corrections.DiagnosticsHelper;
6674
import org.eclipse.jdt.ls.core.internal.managers.InvisibleProjectImporter;
6775
import org.eclipse.jdt.ls.core.internal.managers.ProjectsManager;
@@ -103,6 +111,9 @@ public abstract class BaseDocumentLifeCycleHandler {
103111
*/
104112
private static final long PUBLISH_DIAGNOSTICS_MAX_DEBOUNCE = 2000; /*ms*/
105113

114+
public static final String RENAME_REFERENCE_MARKER_ID = IConstants.PLUGIN_ID + ".renameReferenceMarker";
115+
public static final int RENAME_REFERENCE_PROBLEM_ID = IProblem.Syntax + 2000;
116+
106117
private CoreASTProvider sharedASTProvider;
107118
private WorkspaceJob validationTimer;
108119
private WorkspaceJob publishDiagnosticsJob;
@@ -111,6 +122,9 @@ public abstract class BaseDocumentLifeCycleHandler {
111122
private MovingAverage movingAverageForValidation = new MovingAverage(DOCUMENT_LIFECYCLE_MAX_DEBOUNCE);
112123
private MovingAverage movingAverageForDiagnostics = new MovingAverage(PUBLISH_DIAGNOSTICS_MIN_DEBOUNCE);
113124

125+
private Map<String, List<MemberInfo>> documentMemberInfos = new HashMap<>();
126+
private Map<String, List<IMarker>> documentMarkers = new HashMap<>();
127+
114128
public BaseDocumentLifeCycleHandler(boolean delayValidation) {
115129
this.sharedASTProvider = CoreASTProvider.getInstance();
116130
if (delayValidation) {
@@ -303,11 +317,18 @@ public IProblemRequestor getProblemRequestor(ICompilationUnit workingCopy) {
303317

304318
};
305319
int flags = ICompilationUnit.FORCE_PROBLEM_DETECTION | ICompilationUnit.ENABLE_BINDINGS_RECOVERY | ICompilationUnit.ENABLE_STATEMENTS_RECOVERY;
320+
if (JavaLanguageServerPlugin.getPreferencesManager().getPreferences().getRenameReferencesEnabled()) {
321+
provideRenameReferencesMarkers(unit);
322+
}
306323
unit.reconcile(ICompilationUnit.NO_AST, flags, wcOwner, monitor);
307324
}
308325

309326
public void didClose(DidCloseTextDocumentParams params) {
310327
documentVersions.remove(params.getTextDocument().getUri());
328+
ICompilationUnit cu = JDTUtils.resolveCompilationUnit(params.getTextDocument().getUri());
329+
if (cu != null) {
330+
this.documentMemberInfos.remove(cu.getResource().getLocationURI().toString());
331+
}
311332
ISchedulingRule rule = JDTUtils.getRule(params.getTextDocument().getUri());
312333
try {
313334
ResourcesPlugin.getWorkspace().run(new IWorkspaceRunnable() {
@@ -410,6 +431,9 @@ public ICompilationUnit handleOpen(DidOpenTextDocumentParams params) {
410431
// see https://github.com/redhat-developer/vscode-java/issues/274
411432
checkPackageDeclaration(uri, unit);
412433
inferInvisibleProjectSourceRoot(unit);
434+
if (JavaLanguageServerPlugin.getPreferencesManager().getPreferences().getRenameReferencesEnabled()) {
435+
this.documentMemberInfos.put(unit.getResource().getLocationURI().toString(), this.getMemberInfos(unit));
436+
}
413437
} catch (JavaModelException e) {
414438
JavaLanguageServerPlugin.logException("Error while opening document. URI: " + uri, e);
415439
}
@@ -467,6 +491,87 @@ public ICompilationUnit handleChanged(DidChangeTextDocumentParams params) {
467491
return unit;
468492
}
469493

494+
private void provideRenameReferencesMarkers(ICompilationUnit unit) {
495+
String uri = unit.getResource().getLocationURI().toString();
496+
List<MemberInfo> memberInfos = this.getMemberInfos(unit);
497+
try {
498+
if (this.documentMemberInfos.containsKey(uri)) {
499+
// delete existing markers
500+
List<IMarker> currentMarkers = this.documentMarkers.get(uri);
501+
if (currentMarkers != null) {
502+
unit.getResource().deleteMarkers(RENAME_REFERENCE_MARKER_ID, false, IResource.DEPTH_ONE);
503+
currentMarkers.clear();
504+
} else {
505+
currentMarkers = new ArrayList<>();
506+
this.documentMarkers.put(uri, currentMarkers);
507+
}
508+
List<MemberInfo> cacheMemberInfos = this.documentMemberInfos.get(uri);
509+
if (cacheMemberInfos.size() == memberInfos.size()) {
510+
for (int i = 0; i < memberInfos.size(); i++) {
511+
MemberInfo memberInfo = memberInfos.get(i);
512+
String name = memberInfo.getName();
513+
MemberInfo cacheMemberInfo = cacheMemberInfos.get(i);
514+
if (name == null) {
515+
this.documentMemberInfos.put(uri, memberInfos);
516+
break;
517+
}
518+
if (cacheMemberInfo.getName().equals(name)) {
519+
continue;
520+
}
521+
String message = Messages.format(CorrectionMessages.LocalCorrectionsSubProcessor_manual_rename_label, new String[]{ cacheMemberInfo.getName(), name });
522+
IMarker marker = ResourceUtils.createInfoMarker(RENAME_REFERENCE_MARKER_ID, unit.getResource(), message, RENAME_REFERENCE_PROBLEM_ID, memberInfo.getNameRange().getOffset(), memberInfo.getNameRange().getOffset() + memberInfo.getNameRange().getLength());
523+
marker.setAttribute("originalName", cacheMemberInfo.getName());
524+
marker.setAttribute("newName", name);
525+
currentMarkers.add(marker);
526+
}
527+
} else {
528+
// update the storage names list
529+
this.documentMemberInfos.put(uri, memberInfos);
530+
}
531+
} else {
532+
// initialize the storage names list
533+
this.documentMemberInfos.put(uri, memberInfos);
534+
}
535+
} catch (CoreException e) {
536+
// Do nothing
537+
}
538+
}
539+
540+
private List<MemberInfo> getMemberInfos(ICompilationUnit unit) {
541+
List<MemberInfo> members = new ArrayList<>();
542+
try {
543+
IType[] types = unit.getAllTypes();
544+
for (IType type : types) {
545+
for (IJavaElement child : type.getChildren()) {
546+
if (child instanceof IMember member && Modifier.isPublic(member.getFlags())) {
547+
members.add(new MemberInfo(member.getElementName(), member.getNameRange()));
548+
}
549+
}
550+
}
551+
} catch (JavaModelException e) {
552+
// do nothing
553+
}
554+
return members;
555+
}
556+
557+
public void cleanRenameReferenceCache(ICompilationUnit unit) {
558+
if (unit == null) {
559+
return;
560+
}
561+
IResource resource = unit.getResource();
562+
if (resource == null) {
563+
return;
564+
}
565+
String uri = resource.getLocationURI().toString();
566+
this.documentMemberInfos.remove(uri);
567+
this.documentMarkers.remove(uri);
568+
try {
569+
unit.getResource().deleteMarkers(RENAME_REFERENCE_MARKER_ID, false, IResource.DEPTH_ONE);
570+
} catch (CoreException e) {
571+
// do nothing
572+
}
573+
}
574+
470575
public ICompilationUnit handleClosed(DidCloseTextDocumentParams params) {
471576
String uri = params.getTextDocument().getUri();
472577
ICompilationUnit unit = JDTUtils.resolveCompilationUnit(uri);
@@ -743,4 +848,22 @@ public void checkChanged() {
743848

744849
}
745850

851+
private static class MemberInfo {
852+
private String name;
853+
private ISourceRange nameRange;
854+
855+
public MemberInfo(String name, ISourceRange nameRange) {
856+
this.name = name;
857+
this.nameRange = nameRange;
858+
}
859+
860+
public String getName() {
861+
return name;
862+
}
863+
864+
public ISourceRange getNameRange() {
865+
return nameRange;
866+
}
867+
}
868+
746869
}

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/CodeActionHandler.java‎

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ public class CodeActionHandler {
8181
private NonProjectFixProcessor nonProjectFixProcessor;
8282

8383
private PreferenceManager preferenceManager;
84+
private DocumentLifeCycleHandler documentLifeCycleHandler;
8485

8586
public CodeActionHandler(PreferenceManager preferenceManager) {
8687
this.preferenceManager = preferenceManager;
@@ -91,6 +92,11 @@ public CodeActionHandler(PreferenceManager preferenceManager) {
9192
this.nonProjectFixProcessor = new NonProjectFixProcessor(preferenceManager);
9293
}
9394

95+
public CodeActionHandler(PreferenceManager preferenceManager, DocumentLifeCycleHandler documentLifeCycleHandler) {
96+
this(preferenceManager);
97+
this.documentLifeCycleHandler = documentLifeCycleHandler;
98+
}
99+
94100
public List<Either<Command, CodeAction>> getCodeActionCommands(CodeActionParams params, IProgressMonitor monitor) {
95101
if (monitor.isCanceled()) {
96102
return Collections.emptyList();
@@ -159,7 +165,7 @@ public List<Either<Command, CodeAction>> getCodeActionCommands(CodeActionParams
159165
if (containsKind(codeActionKinds, CodeActionKind.QuickFix)) {
160166
try {
161167
codeActions.addAll(nonProjectFixProcessor.getCorrections(params, context, locations));
162-
List<ChangeCorrectionProposal> quickfixProposals = this.quickFixProcessor.getCorrections(params, context, locations);
168+
List<ChangeCorrectionProposal> quickfixProposals = this.quickFixProcessor.getCorrections(params, context, locations, documentLifeCycleHandler);
163169
this.quickFixProcessor.addAddAllMissingImportsProposal(context, quickfixProposals);
164170
Set<ChangeCorrectionProposal> quickSet = new TreeSet<>(comparator);
165171
quickSet.addAll(quickfixProposals);

‎org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/JDTLanguageServer.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -704,7 +704,7 @@ public CompletableFuture<List<Either<SymbolInformation, DocumentSymbol>>> docume
704704
@Override
705705
public CompletableFuture<List<Either<Command, CodeAction>>> codeAction(CodeActionParams params) {
706706
logInfo(">> document/codeAction");
707-
CodeActionHandler handler = new CodeActionHandler(this.preferenceManager);
707+
CodeActionHandler handler = new CodeActionHandler(this.preferenceManager, this.documentLifeCycleHandler);
708708
return computeAsync((monitor) -> {
709709
waitForLifecycleJobs(monitor);
710710
return handler.getCodeActionCommands(params, monitor);
@@ -813,7 +813,7 @@ public CompletableFuture<Either3<Range, PrepareRenameResult, PrepareRenameDefaul
813813
@Override
814814
public CompletableFuture<WorkspaceEdit> rename(RenameParams params) {
815815
logInfo(">> document/rename");
816-
RenameHandler handler = new RenameHandler(preferenceManager);
816+
RenameHandler handler = new RenameHandler(preferenceManager, this.documentLifeCycleHandler);
817817
return computeAsync((monitor) -> {
818818
waitForLifecycleJobs(monitor);
819819
return handler.rename(params, monitor);

0 commit comments

Comments
 (0)