diff --git a/plugins/lombok/src/main/java/de/plushnikov/intellij/plugin/inspection/LombokGetterMayBeUsedInspection.java b/plugins/lombok/src/main/java/de/plushnikov/intellij/plugin/inspection/LombokGetterMayBeUsedInspection.java index 92359d5938a0..323c60c58a8a 100644 --- a/plugins/lombok/src/main/java/de/plushnikov/intellij/plugin/inspection/LombokGetterMayBeUsedInspection.java +++ b/plugins/lombok/src/main/java/de/plushnikov/intellij/plugin/inspection/LombokGetterMayBeUsedInspection.java @@ -1,10 +1,7 @@ // Copyright 2000-2023 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license. package de.plushnikov.intellij.plugin.inspection; -import com.intellij.codeInspection.CleanupLocalInspectionTool; -import com.intellij.codeInspection.LocalQuickFix; -import com.intellij.codeInspection.ProblemDescriptor; -import com.intellij.codeInspection.ProblemsHolder; +import com.intellij.codeInspection.*; import com.intellij.openapi.project.Project; import com.intellij.openapi.util.Pair; import com.intellij.psi.*; @@ -192,9 +189,13 @@ public class LombokGetterMayBeUsedInspection extends LombokJavaInspectionBase im ) { if (myHolder != null) { final LocalQuickFix fix = new LombokGetterMayBeUsedFix(Objects.requireNonNull(psiClass.getName())); + final PsiIdentifier psiClassNameIdentifier = psiClass.getNameIdentifier(); myHolder.registerProblem(psiClass, LombokBundle.message("inspection.lombok.getter.may.be.used.display.class.message", - psiClass.getName()), fix); + psiClass.getName()), + ProblemHighlightType.GENERIC_ERROR_OR_WARNING, + psiClassNameIdentifier != null ? psiClassNameIdentifier.getTextRangeInParent() : psiClass.getTextRange(), + fix); } else if (lombokGetterMayBeUsedFix != null) { lombokGetterMayBeUsedFix.effectivelyDoFix(psiClass, fieldsAndMethods, annotatedFields); } diff --git a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/FieldsWithGetter.java b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/FieldsWithGetter.java index 712afb7baf72..d2f5837baff5 100644 --- a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/FieldsWithGetter.java +++ b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/FieldsWithGetter.java @@ -1,6 +1,6 @@ // Copyright 2000-2023 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license. -public class FieldsWithGetter { +public class FieldsWithGetter { private int bar; public int getBar() { @@ -24,7 +24,7 @@ return InstanceField.this.fooBar; } } - public class AllInstanceFields { + public class AllInstanceFields { private int bar; private boolean Baz; private int fooBar; @@ -41,7 +41,7 @@ public int getFooBar() { return AllInstanceFields.this.fooBar; } - } + } public class StaticField { private static int bar; private int fieldWithoutGetter; @@ -50,4 +50,4 @@ return bar; } } -} \ No newline at end of file +} \ No newline at end of file diff --git a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/InstanceAndStaticFields.java b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/InstanceAndStaticFields.java index fd6ab2d31bea..b1dde280e01f 100644 --- a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/InstanceAndStaticFields.java +++ b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/InstanceAndStaticFields.java @@ -1,6 +1,6 @@ // Copyright 2000-2023 JetBrains s.r.o. and contributors. Use of this source code is governed by the Apache 2.0 license. -public class InstanceAndStaticFields { +public class InstanceAndStaticFields { private static int staticField; private int instanceField; @@ -11,4 +11,4 @@ public int getInstanceField() { return instanceField; } -} \ No newline at end of file +} \ No newline at end of file diff --git a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeAnnotatedField.java b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeAnnotatedField.java index ee78c7aeaebd..165074529a83 100644 --- a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeAnnotatedField.java +++ b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeAnnotatedField.java @@ -2,12 +2,12 @@ import lombok.Getter; -public class ClassWithAnnotatedField { +public class ClassWithAnnotatedField { private int canditateField; @Getter private int annotatedField; public int getCanditateField() { - return canditateField; + return canditateField; } } \ No newline at end of file diff --git a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeFieldInInnerClass.java b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeFieldInInnerClass.java index eba077e415ec..cbb0467242ce 100644 --- a/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeFieldInInnerClass.java +++ b/plugins/lombok/testData/inspection/lombokGetterMayBeUsed/beforeFieldInInnerClass.java @@ -1,11 +1,11 @@ // "Use lombok @Getter for 'InnerClass'" "true" public class Foo { - public class InnerClass { + public class InnerClass { private int bar; public int getBar() { - return bar; // Keep this comment + return bar; // Keep this comment } } } \ No newline at end of file