Add conflicts detecting for local variables and parameters

Author: waleedyaseen

src/main/kotlin/io/runescript/plugin/ide/refactoring/RsRenameProcessor.kt

@@ -0,0 +1,41 @@
+package io.runescript.plugin.ide.refactoring
+
+import com.intellij.openapi.editor.Editor
+import com.intellij.openapi.project.Project
+import com.intellij.openapi.util.NlsContexts
+import com.intellij.psi.PsiElement
+import com.intellij.psi.ResolveState
+import com.intellij.refactoring.rename.RenamePsiElementProcessor
+import com.intellij.refactoring.rename.RenamePsiFileProcessor
+import com.intellij.refactoring.rename.RenameRefactoringDialog
+import com.intellij.util.containers.MultiMap
+import io.runescript.plugin.ide.RsBundle
+import io.runescript.plugin.lang.psi.RsLocalVariableExpression
+import io.runescript.plugin.lang.psi.scope.RsLocalVariableResolver
+import io.runescript.plugin.lang.psi.scope.RsResolveMode
+import io.runescript.plugin.lang.psi.scope.RsScopesUtil
+
+class RsRenameProcessor : RenamePsiElementProcessor() {
+
+    override fun createDialog(
+        project: Project, element: PsiElement, nameSuggestionContext: PsiElement?, editor: Editor?
+    ): RenameRefactoringDialog? {
+        return RenamePsiFileProcessor.PsiFileRenameDialog(project, element, nameSuggestionContext, editor)
+    }
+
+    override fun findExistingNameConflicts(
+        element: PsiElement, newName: String, conflicts: MultiMap<PsiElement?, @NlsContexts.DialogMessage String?>
+    ) {
+        val resolver = RsLocalVariableResolver(newName, RsResolveMode.Both)
+        RsScopesUtil.walkUpScopes(resolver, ResolveState.initial(), element)
+        if (resolver.declaration != null) {
+            conflicts.putValue(
+                resolver.declaration, RsBundle.message("refactoring.error.duplicate.local.variable", newName)
+            )
+        }
+    }
+
+    override fun canProcessElement(element: PsiElement): Boolean {
+        return element is RsLocalVariableExpression
+    }
+}

src/main/resources/META-INF/plugin.xml

@@ -37,6 +37,7 @@
 
         <!-- Rename Handler -->
         <renameHandler implementation="io.runescript.plugin.ide.refactoring.RsMemberInplaceRenameHandler"/>
+        <renamePsiElementProcessor implementation="io.runescript.plugin.ide.refactoring.RsRenameProcessor"/>
         <lang.namesValidator language="RuneScript" implementationClass="io.runescript.plugin.ide.refactoring.RsNamesValidator"/>
         <lang.namesValidator language="RuneScriptSymbol" implementationClass="io.runescript.plugin.ide.refactoring.RsNamesValidator"/>
 

src/main/resources/messages/RsBundle.properties

@@ -46,6 +46,9 @@ module.builder.backend.neptune=Neptune
 module.builder.backend.rscp=RSCP
 module.builder.backend.neptune.compiler.path=Compiler path:
 
+# Refactoring
+refactoring.error.duplicate.local.variable=An existing local variable {0} has the same name
+
 # Inspections
 inspection.error.unresolved.procedure=Unresolved procedure `{0}`
 inspection.error.unresolved.clientscript=Unresolved clientscript `{0}`