MDS/openide
1
0
Fork 0
mirror of https://gitflic.ru/project/openide/openide.git synced 2026-10-02 10:14:57 +07:00
Code Issues Packages Projects Releases Wiki Activity

IJ-CR-150661 [java-inspection] IDEA-311157 deleting component doesn't check usages

Browse Source 
- fix test
- fast exit for unnamed variables

GitOrigin-RevId: cc6d6f646613e231aa4ab8a72693b95fbc3f1248
This commit is contained in:
Mikhail Pyltsin
2024-12-03 21:33:19 +00:00
committed by intellij-monorepo-bot
parent 94efe9cb4d
commit 9348d2cfb5
2 changed files with 114 additions and 106 deletions
Download Patch File Download Diff File Expand all files Collapse all files
java/java-impl-refactorings/src/com/intellij/refactoring/changeSignature/JavaChangeSignatureUsageProcessor.java
+1 -2
View File
@@ -1430,7 +1430,6 @@ public final class JavaChangeSignatureUsageProcessor implements ChangeSignatureU
//getter
PsiMethod explicitGetter = JavaPsiRecordUtil.getAccessorForRecordComponent(component);
if (explicitGetter != null) {
PsiManager manager = component.getManager();
for (PsiReference psiReference : ReferencesSearch.search(explicitGetter, explicitGetter.getUseScope(), false)) {
PsiElement paramRef = psiReference.getElement();
conflictDescriptions.putValue(paramRef, JavaRefactoringBundle.message("record.component.used.in.method.body.warning", component.getName()));
@@ -1475,7 +1474,7 @@ public final class JavaChangeSignatureUsageProcessor implements ChangeSignatureU
}
if (pattern instanceof PsiTypeTestPattern typeTestPattern) {
PsiPatternVariable variable = typeTestPattern.getPatternVariable();
if (variable == null) return;
if (variable == null || variable.isUnnamed()) return;
if(VariableAccessUtils.variableIsUsed(variable, context.getContainingFile())) {
conflictDescriptions.putValue(typeTestPattern, JavaRefactoringBundle.message("record.component.used.in.method.body.warning", recordComponent.getName()));
return;