Migrate UnusedLocalVariableCheck to kotlin-analysis-api

SonarSource · Feb 6, 2025 · f1b2295 · f1b2295
1 parent 4242471
commit f1b2295
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/sonar-kotlin-checks/src/main/java/org/sonarsource/kotlin/checks/UnusedLocalVariableCheck.kt b/sonar-kotlin-checks/src/main/java/org/sonarsource/kotlin/checks/UnusedLocalVariableCheck.kt
@@ -25,17 +25,18 @@ import org.sonarsource.kotlin.api.checks.AbstractCheck
 import org.sonarsource.kotlin.api.checks.getVariableType
 import org.sonarsource.kotlin.api.frontend.KotlinFileContext
 
-@org.sonarsource.kotlin.api.frontend.K1only
 @Rule(key = "S1481")
 class UnusedLocalVariableCheck : AbstractCheck() {
 
     override fun visitKtFile(file: KtFile, context: KotlinFileContext) {
-        context.diagnostics
+        context.kaDiagnostics
             .filter {
-                it.factory == Errors.UNUSED_VARIABLE &&
-                    it.psiElement.getVariableType(context.bindingContext)?.getKotlinTypeFqName(false) != "kotlin.Nothing"
+                it.factoryName == Errors.UNUSED_VARIABLE.name &&
+                        // TODO remove after switch to K2:
+                        (it.psi as KtNamedDeclaration).getVariableType(context.bindingContext)
+                            ?.getKotlinTypeFqName(false) != "kotlin.Nothing"
             }
-            .map { it.psiElement as KtNamedDeclaration }
+            .map { it.psi as KtNamedDeclaration }
             .forEach {
                 context.reportIssue(it.nameIdentifier!!, """Remove this unused "${it.name}" local variable.""")
             }