Merge pull request #1449 from 1c-syntax/feature/fix1446

Исправление fp MissingReturnedValueDescription на описании метода
1c-syntax · Nov 16, 2020 · dfa204c · dfa204c
2 parents 74d6291 + b4eb19f
commit dfa204c
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 5 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -59,7 +59,7 @@ dependencies {
     api("org.eclipse.lsp4j", "org.eclipse.lsp4j", "0.9.0")
 
     // 1c-syntax
-    api("com.github.1c-syntax", "bsl-parser", "7dbb670a9e197bb74112094415092a778a349f1e") {
+    api("com.github.1c-syntax", "bsl-parser", "4b60c5dd4e22b5f540c5669190906632b6cf30cc") {
         exclude("com.tunnelvisionlabs", "antlr4-annotations")
         exclude("com.ibm.icu", "*")
         exclude("org.antlr", "ST4")

diff --git a/.../_1c_syntax/bsl/languageserver/diagnostics/MissingReturnedValueDescriptionDiagnostic.java b/.../_1c_syntax/bsl/languageserver/diagnostics/MissingReturnedValueDescriptionDiagnostic.java
@@ -41,7 +41,6 @@
     DiagnosticTag.STANDARD,
     DiagnosticTag.BADPRACTICE
   }
-
 )
 public class MissingReturnedValueDescriptionDiagnostic extends AbstractSymbolTreeDiagnostic {
 
@@ -81,15 +80,15 @@ public void visitMethod(MethodSymbol methodSymbol) {
       return;
     }
 
-    // тип возвращаемого значения должен иметь описание
+    // тип возвращаемого значения должен иметь описание или быть сложным
     var typesWithoutDescription = returnedValueDescription.stream()
-      .filter((TypeDescription typeDescription) -> typeDescription.getDescription().isEmpty())
+      .filter((TypeDescription typeDescription) ->
+        typeDescription.getDescription().isEmpty() && typeDescription.getParameters().isEmpty())
       .map(TypeDescription::getName)
       .collect(Collectors.joining(", "));
     if (!typesWithoutDescription.isEmpty()) {
       diagnosticStorage.addDiagnostic(methodSymbol.getSubNameRange(),
         info.getResourceString("typesWithoutDescription", typesWithoutDescription));
     }
   }
-
 }
diff --git a/src/test/resources/diagnostics/MissingReturnedValueDescriptionDiagnostic.bsl b/src/test/resources/diagnostics/MissingReturnedValueDescriptionDiagnostic.bsl
@@ -39,3 +39,27 @@
 // - Неопределено - если неизвестно
 Функция Пример7()
 КонецФункции
+
+// Параметры подключения к администрируемой информационной базе кластера.
+//
+// Возвращаемое значение:
+//  Структура:
+//    * ИмяВКластере - Строка - имя администрируемой информационной базы в кластере серверов,
+//    * ИмяАдминистратораИнформационнойБазы - Строка - имя пользователя информационной базы с правами
+//                  администратора (если для информационной базы не задан список пользователей ИБ - используется
+//                  пустая строка),
+//    * ПарольАдминистратораИнформационнойБазы - Строка - пароль пользователя информационной базы
+//                  с правами администратора (если для информационной базы не задан список пользователей ИБ или
+//                  для пользователя ИБ не установлен пароль - используется пустая строка).
+//
+Функция ПараметрыАдминистрированияИнформационнойБазыКластера() Экспорт
+
+	Результат = Новый Структура();
+
+	Результат.Вставить("ИмяВКластере", "");
+	Результат.Вставить("ИмяАдминистратораИнформационнойБазы", "");
+	Результат.Вставить("ПарольАдминистратораИнформационнойБазы", "");
+
+	Возврат Результат;
+
+КонецФункции