Fix Add class qualifier to static member access outside declaring class not working correctly for switch statements

Fixes #263

Author: AB-xdev

CHANGELOG.md

@@ -1,3 +1,6 @@
+## 1.4.1
+* Fix ``Add class qualifier to static member access outside declaring class`` not working correctly for ``switch`` statements #263
+
 ## 1.4.0
 * Dropped support for IntelliJ versions < 2024.3
   * This is required to fix a few deprecations and remove some workarounds #171

src/main/java/software/xdev/saveactions/processors/java/inspection/style/CustomUnqualifiedStaticUsageInspection.java

@@ -4,6 +4,7 @@
 import org.jetbrains.annotations.NotNull;
 
 import com.intellij.psi.JavaResolveResult;
+import com.intellij.psi.PsiCaseLabelElementList;
 import com.intellij.psi.PsiClass;
 import com.intellij.psi.PsiElement;
 import com.intellij.psi.PsiEnumConstant;
@@ -15,7 +16,7 @@
 import com.intellij.psi.PsiMethodCallExpression;
 import com.intellij.psi.PsiModifier;
 import com.intellij.psi.PsiReferenceExpression;
-import com.intellij.psi.PsiSwitchLabelStatement;
+import com.intellij.psi.PsiSwitchLabelStatementBase;
 import com.intellij.psi.util.PsiTreeUtil;
 import com.intellij.psi.util.PsiUtil;
 import com.siyeh.ig.BaseInspectionVisitor;
@@ -113,7 +114,7 @@ private boolean isUnqualifiedStaticAccess(final PsiReferenceExpression expressio
 				return false;
 			}
 			final PsiMember member = (PsiMember)element;
-			if(member instanceof PsiEnumConstant && expression.getParent() instanceof PsiSwitchLabelStatement)
+			if(this.isEnumInSwitch(member, expression))
 			{
 				return false;
 			}
@@ -125,5 +126,19 @@ private boolean isUnqualifiedStaticAccess(final PsiReferenceExpression expressio
 			}
 			return member.hasModifierProperty(PsiModifier.STATIC);
 		}
+		
+		private boolean isEnumInSwitch(final PsiMember member, final PsiReferenceExpression expression)
+		{
+			if(!(member instanceof PsiEnumConstant))
+			{
+				return false;
+			}
+			
+			final PsiElement parent = expression.getParent();
+			return parent instanceof PsiCaseLabelElementList
+				|| parent != null && parent.getParent() instanceof PsiSwitchLabelStatementBase
+				// This was the original code and might be needed for older java versions
+				|| parent instanceof PsiSwitchLabelStatementBase;
+		}
 	}
 }