Merge pull request #2755 from 1c-syntax/fix/SkipBrokenQuery

Пропуск диагностик по битым запросам

Author: nixel2007

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

@@ -23,6 +23,8 @@
 
 import com.github._1c_syntax.bsl.languageserver.context.DocumentContext;
 import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticInfo;
+import com.github._1c_syntax.bsl.languageserver.utils.Trees;
+import com.github._1c_syntax.bsl.parser.SDBLParser;
 import com.github._1c_syntax.bsl.parser.SDBLParserBaseVisitor;
 import lombok.Getter;
 import lombok.Setter;
@@ -49,4 +51,13 @@ public List<Diagnostic> getDiagnostics(DocumentContext documentContext) {
 
     return diagnosticStorage.getDiagnostics();
   }
+
+  @Override
+  public ParseTree visitQueryPackage(SDBLParser.QueryPackageContext ctx) {
+    if (Trees.treeContainsErrors(ctx)) {
+      return ctx;
+    }
+    return super.visitQueryPackage(ctx);
+  }
+
 }

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

@@ -214,7 +214,6 @@ private static boolean haveIsNullFunction(BSLParserRuleContext ctx) {
       .filter(SDBLParser.BuiltInFunctionsContext.class::isInstance)
       .map(SDBLParser.BuiltInFunctionsContext.class::cast)
       .map(SDBLParser.BuiltInFunctionsContext::ISNULL)
-      .filter(Objects::nonNull)
       .isPresent();
   }
 

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

@@ -43,6 +43,11 @@
 )
 public class QueryParseErrorDiagnostic extends AbstractSDBLVisitorDiagnostic {
 
+  @Override
+  public ParseTree visitQueryPackage(SDBLParser.QueryPackageContext ctx) {
+    return visitChildren(ctx);
+  }
+
   @Override
   // анализируем каждый запрос пакета
   public ParseTree visitQueries(SDBLParser.QueriesContext ctx) {

src/test/resources/diagnostics/AssignAliasFieldsInQueryDiagnostic.bsl

@@ -55,3 +55,17 @@
         |		Таблица.МестоХранения КАК МестоХранения
         |	ИЗ
         |		РегистрНакопления.ПартииНоменклатуры.ОстаткиИОбороты(&НачалоПериода, &КонецПериода, День, ДвиженияИГраницыПериода, ";
+
+// Битый запрос с npe на полях
+ТекстЗапроса = "
+	|ВЫБРАТЬ
+	|	ВТ_ТЧ.НомерСтроки
+	|ИЗ
+	|	&ВТ_Цены КАК ВТ_Цены
+	|;
+	|
+	|////////////////////////////////////////////////////////////////////////////////
+	|ВЫБРАТЬ
+	|	" + ПоляТЧДокумента + "
+	|ИЗ
+	|	&ВТ_ТЧ КАК Товары";

src/test/resources/diagnostics/FieldsFromJoinsWithoutIsNullDiagnostic.bsl

@@ -205,4 +205,22 @@
     |ПО Склады15.Кладовщик = Сотрудники15.Ссылка
     |ГДЕ Сотрудники15.Реквизит ЕСТЬ NULL // не ошибка
     |";
-КонецПроцедуры
+КонецПроцедуры
+
+ // Разрыв в соединении роняет с npe
+ Запрос = "ВЫБРАТЬ
+           |	ДополнительныеСведения.КодИФНСФЛ КАК КодИФНСФЛ
+           |ИЗ
+           |	РегистрСведений.АдресныеОбъекты КАК АдресныеОбъектыУровень0
+           |
+           |	ОБЪЕДИНИТЬ ВСЕ
+           |
+           |	ВЫБРАТЬ
+           |	ДополнительныеСведения.КодИФНСФЛ
+           |ИЗ
+           |	РегистрСведений.АдресныеОбъекты КАК АдресныеОбъектыУровень0
+           |		ЛЕВОЕ СОЕДИНЕНИЕ РегистрСведений.АдресныеОбъекты КАК АдресныеОбъектыУровень1
+           |		ПО АдресныеОбъектыУровень0.РодительскийИдентификатор = АдресныеОбъектыУровень1.Идентификатор
+           |		ЛЕВОЕ СОЕДИНЕНИЕ РегистрСведений."+АдресныеОбъекты+" КАК АдресныеОбъектыУровень2
+           |		ПО (АдресныеОбъектыУровень1.РодительскийИдентификатор = АдресныеОбъектыУровень2.Идентификатор)
+           |";