From 6f026e54df4b6823d88e5d518fff4e737a464b69 Mon Sep 17 00:00:00 2001
From: Alexander Zolotov <alexander.zolotov@jetbrains.com>
Date: Mon, 7 Mar 2016 17:09:26 +0300
Subject: [PATCH] Move inspection to DevKit group + remove redundant
 suppression

---
 plugins/devkit/resources/META-INF/plugin.xml              | 2 +-
 .../QuickFixGetFamilyNameViolationInspection.java         | 8 +++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/plugins/devkit/resources/META-INF/plugin.xml b/plugins/devkit/resources/META-INF/plugin.xml
index cce061ba9670..ad836dc17357 100644
--- a/plugins/devkit/resources/META-INF/plugin.xml
+++ b/plugins/devkit/resources/META-INF/plugin.xml
@@ -147,7 +147,7 @@
                      groupKey="inspections.group.name"
                      enabledByDefault="false" isInternal="true"
                      level="ERROR" implementationClass="org.jetbrains.idea.devkit.inspections.internal.HighlightVisitorInternalInspection"/>
-    <localInspection groupPath="Java" language="JAVA" shortName="QuickFixGetFamilyNameViolation"
+    <localInspection language="JAVA" shortName="QuickFixGetFamilyNameViolation"
                      displayName="QuickFix's getFamilyName() implementation must not depend on a specific context"
                      groupKey="inspections.group.name"
                      enabledByDefault="true"
diff --git a/plugins/devkit/src/inspections/QuickFixGetFamilyNameViolationInspection.java b/plugins/devkit/src/inspections/QuickFixGetFamilyNameViolationInspection.java
index 3a78f5950d10..acbf2b37c7b7 100644
--- a/plugins/devkit/src/inspections/QuickFixGetFamilyNameViolationInspection.java
+++ b/plugins/devkit/src/inspections/QuickFixGetFamilyNameViolationInspection.java
@@ -34,14 +34,16 @@ public class QuickFixGetFamilyNameViolationInspection extends DevKitInspectionBa
   @Nullable
   @Override
   public ProblemDescriptor[] checkMethod(@NotNull PsiMethod method, @NotNull InspectionManager manager, boolean isOnTheFly) {
-    if ("getFamilyName".equals(method.getName()) && method.getParameterList().getParametersCount() == 0 && !method.hasModifierProperty(PsiModifier.ABSTRACT)) {
+    if ("getFamilyName".equals(method.getName()) &&
+        method.getParameterList().getParametersCount() == 0 &&
+        !method.hasModifierProperty(PsiModifier.ABSTRACT)) {
       final PsiClass aClass = method.getContainingClass();
       if (InheritanceUtil.isInheritor(aClass, QuickFix.class.getName()) && doesMethodViolate(method)) {
         final PsiIdentifier identifier = method.getNameIdentifier();
         LOG.assertTrue(identifier != null);
-        //noinspection DialogTitleCapitalization
         return new ProblemDescriptor[]{
-          manager.createProblemDescriptor(identifier, "QuickFix's getFamilyName() implementation must not depend on a specific context", (LocalQuickFix) null, ProblemHighlightType.GENERIC_ERROR_OR_WARNING, true)};
+          manager.createProblemDescriptor(identifier, "QuickFix's getFamilyName() implementation must not depend on a specific context",
+                                          (LocalQuickFix)null, ProblemHighlightType.GENERIC_ERROR_OR_WARNING, true)};
       }
     }
     return null;