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

[java-inspections] ConvertToRecordFix: don't inherit from InspectionGadgetsFix

Browse Source 
Merge-request: IJ-MR-161186
Merged-by: Bartek Pacia <bartek.pacia@jetbrains.com>

GitOrigin-RevId: af4993f42e033b4e02ebdd134f68da17c797593f
This commit is contained in:
Bartek Pacia
2025-04-24 19:54:19 +00:00
committed by intellij-monorepo-bot
parent 2b83c72394
commit 25d6134265
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
java/java-impl-inspections/src/com/intellij/codeInspection/classCanBeRecord/ConvertToRecordFix.java
View File

@@ -5,6 +5,7 @@ import com.intellij.codeInsight.AnnotationTargetUtil;
import com.intellij.codeInsight.AnnotationUtil;
import com.intellij.codeInsight.ExceptionUtil;
import com.intellij.codeInsight.intention.preview.IntentionPreviewInfo;
import com.intellij.codeInspection.LocalQuickFix;
import com.intellij.codeInspection.ProblemDescriptor;
import com.intellij.codeInspection.util.IntentionFamilyName;
import com.intellij.java.JavaBundle;
@@ -22,7 +23,6 @@ import com.intellij.util.ObjectUtils;
import com.intellij.util.SmartList;
import com.intellij.util.containers.ContainerUtil;
import com.intellij.util.containers.MultiMap;
import com.siyeh.ig.InspectionGadgetsFix;
import com.siyeh.ig.callMatcher.CallMatcher;
import com.siyeh.ig.memory.InnerClassReferenceVisitor;
import com.siyeh.ig.psiutils.MethodUtils;
@@ -36,11 +36,11 @@ import java.util.stream.Collectors;
import static com.intellij.psi.CommonClassNames.JAVA_LANG_OBJECT;
import static com.intellij.psi.PsiModifier.*;
public class ConvertToRecordFix extends InspectionGadgetsFix {
public final class ConvertToRecordFix implements LocalQuickFix {
private final boolean mySuggestAccessorsRenaming;
private final @NotNull List<String> myIgnoredAnnotations;
ConvertToRecordFix(boolean suggestAccessorsRenaming, @NotNull List<String> ignoredAnnotations) {
public ConvertToRecordFix(boolean suggestAccessorsRenaming, @NotNull List<String> ignoredAnnotations) {
mySuggestAccessorsRenaming = suggestAccessorsRenaming;
myIgnoredAnnotations = ignoredAnnotations;
}
@@ -56,7 +56,7 @@ public class ConvertToRecordFix extends InspectionGadgetsFix {
}
@Override
protected void doFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
public void applyFix(@NotNull Project project, @NotNull ProblemDescriptor descriptor) {
final ConvertToRecordProcessor processor = getRecordProcessor(descriptor);
if (processor == null) return;
// Without the next line, the conflicts view is not shown
@@ -81,7 +81,7 @@ public class ConvertToRecordFix extends InspectionGadgetsFix {
private @Nullable ConvertToRecordProcessor getRecordProcessor(ProblemDescriptor descriptor) {
PsiElement psiElement = descriptor.getPsiElement();
if (psiElement == null) return null;
if (psiElement == null || !psiElement.isValid()) return null;
PsiClass psiClass = ObjectUtils.tryCast(psiElement.getParent(), PsiClass.class);
if (psiClass == null) return null;
@@ -470,10 +470,7 @@ public class ConvertToRecordFix extends InspectionGadgetsFix {
@Override
public String toString() {
return "FieldAccessorCandidate{" +
"myFieldAccessor=" + myFieldAccessor +
", myBackingField=" + myBackingField +
'}';
return "FieldAccessorCandidate{" + "myFieldAccessor=" + myFieldAccessor + ", myBackingField=" + myBackingField + '}';
}
}