Merge pull request #2873 from qtLex/fix/IllegalStateExceptionInCreateVariablesByName

Fix/illegal state exception in create variables by name

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/context/computer/VariableSymbolComputer.java

@@ -37,6 +37,7 @@
 import org.antlr.v4.runtime.tree.ParseTree;
 import org.antlr.v4.runtime.tree.TerminalNode;
 import org.eclipse.lsp4j.Range;
+import org.eclipse.lsp4j.SymbolKind;
 
 import java.util.ArrayList;
 import java.util.Collections;
@@ -129,6 +130,11 @@ public ParseTree visitParam(BSLParser.ParamContext ctx) {
       return ctx;
     }
 
+    // При ошибках в разборе может получиться, что два параметра с одним именем в модуле
+    if (currentMethod.getSymbolKind() == SymbolKind.Module && moduleVariables.containsKey(ctx.IDENTIFIER().getText())) {
+      return ctx;
+    }
+
     var variable = VariableSymbol.builder()
       .name(ctx.IDENTIFIER().getText().intern())
       .scope(currentMethod)
@@ -141,7 +147,11 @@ public ParseTree visitParam(BSLParser.ParamContext ctx) {
       .build();
     variables.add(variable);
 
-    currentMethodVariables.put(ctx.IDENTIFIER().getText(), ctx.IDENTIFIER().getText());
+    if (currentMethod.getSymbolKind() == SymbolKind.Module) {
+      moduleVariables.put(ctx.IDENTIFIER().getText(), ctx.IDENTIFIER().getText());
+    } else {
+      currentMethodVariables.put(ctx.IDENTIFIER().getText(), ctx.IDENTIFIER().getText());
+    }
     return ctx;
   }
 

src/test/resources/references/ReferenceIndexFillerVariableTest.bsl

@@ -48,4 +48,34 @@
 
 КонецФункции
 
+#Если Клиент Тогда
+
+Процедура Выполнить(Значение0)
+
+    Значение0 = 1;
+
+    Если Истина Тогда
+
+        Возврат 10;
+
+    КонецЕсли;
+
+КонецПроцедуры
+
+#Иначе
+
+Функция Выполнить(Значение0)
+
+    Значение0 = 1;
+
+    Если Истина Тогда
+
+        Возврат 10;
+
+    КонецЕсли;
+
+КонецФункции
+
+#КонецЕсли
+
 Метод(Модуль, Модуль2);