Merge pull request #1483 from qtLex/fix/RedundantAccessToObject

Исправление FP на доступе через индекс
1c-syntax · Dec 14, 2020 · 36b135b · 36b135b
2 parents f0b0bd0 + 8a96be5
commit 36b135b
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/...m/github/_1c_syntax/bsl/languageserver/diagnostics/RedundantAccessToObjectDiagnostic.java b/...m/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());
     }
 

diff --git a/...thub/_1c_syntax/bsl/languageserver/diagnostics/RedundantAccessToObjectDiagnosticTest.java b/...thub/_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)

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