Merge pull request #1483 from qtLex/fix/RedundantAccessToObject

Исправление FP на доступе через индекс

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/RedundantAccessToObjectDiagnostic.java

@@ -134,7 +134,11 @@ public ParseTree visitComplexIdentifier(BSLParser.ComplexIdentifierContext ctx)
       return ctx;
     }
 
-    if (PATTERN.matcher(identifier.getText()).matches() && modifiers.get(0) != null) {
+    if (
+      PATTERN.matcher(identifier.getText()).matches()
+      && modifiers.get(0) != null
+      && modifiers.get(0).accessIndex() == null
+    ) {
       diagnosticStorage.addDiagnostic(ctx.getStart());
     }
 

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/RedundantAccessToObjectDiagnosticTest.java

@@ -48,12 +48,11 @@ void test() {
 
     List<Diagnostic> diagnostics = getDiagnostics();
 
-    assertThat(diagnostics).hasSize(10);
+    assertThat(diagnostics).hasSize(9);
     assertThat(diagnostics, true)
       .hasRange(2, 4, 2, 14)
       .hasRange(3, 4, 3, 14)
       .hasRange(6, 18, 6, 28)
-      .hasRange(7, 18, 7, 28)
       .hasRange(8, 18, 8, 28)
       .hasRange(10, 4, 10, 14)
       .hasRange(16, 4, 16, 14)

src/test/resources/diagnostics/RedundantAccessToObjectDiagnostic.bsl

@@ -4,9 +4,9 @@
     ЭтотОбъект.Договор                     = Данные.Договор;    // Ошибка
     ЭтотОбъект["ПолеКонтактнойИнформации"] = Данные.Телефон;    // Тут ошибки быть не должно
 
-    Переменная  = ЭтотОбъект.Значенние;    // Ошибка
-    Переменная2 = ЭтотОбъект["Значенние"]; // Ошибка
-    Переменная3 = ЭтотОбъект.Значенние();  // Ошибка
+    Переменная  = ЭтотОбъект.Значение;    // Ошибка
+    Переменная2 = ЭтотОбъект["Значение"]; // Тут ошибки быть не должно
+    Переменная3 = ЭтотОбъект.Значение();  // Ошибка
 
     ЭтотОбъект.ВыполнитьЗаполнениеПоСтруктуре(Данные);          // Ошибка