исправления из ПР, пропавшие после мержа

Author: artbear

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

@@ -60,7 +60,8 @@
 public class FieldsFromJoinsWithoutIsNullDiagnostic extends AbstractSDBLVisitorDiagnostic {
 
   private static final Integer SELECT_ROOT = SDBLParser.RULE_selectedField;
-  private static final Collection<Integer> SELECT_STATEMENTS = Set.of(SELECT_ROOT, SDBLParser.RULE_builtInFunctions);
+  private static final Collection<Integer> SELECT_STATEMENTS = Set.of(SELECT_ROOT, SDBLParser.RULE_builtInFunctions,
+    SDBLParser.RULE_isNullPredicate);
 
   private static final Integer WHERE_ROOT = SDBLParser.RULE_predicate;
   private static final Collection<Integer> WHERE_STATEMENTS = Set.of(WHERE_ROOT, SDBLParser.RULE_builtInFunctions,
@@ -162,7 +163,7 @@ private static boolean haveExprNotWithParens(SDBLParser.PredicateContext ctx) {
   }
 
   private void checkSelect(String tableName, SDBLParser.SelectedFieldsContext columns) {
-    checkStatements(tableName, columns, SELECT_STATEMENTS, SELECT_ROOT, false);
+    checkStatements(tableName, columns, SELECT_STATEMENTS, SELECT_ROOT, true);
   }
 
   private void checkStatements(String tableName, BSLParserRuleContext expression, Collection<Integer> statements,
@@ -205,7 +206,6 @@ private static boolean haveIsNullOperator(BSLParserRuleContext ctx) {
     return Optional.of(ctx)
       .filter(SDBLParser.IsNullPredicateContext.class::isInstance)
       .map(SDBLParser.IsNullPredicateContext.class::cast)
-      .filter(isNullPredicateContext -> isNullPredicateContext.NOT() == null)
       .isPresent();
   }
 

src/test/resources/diagnostics/FieldsFromJoinsWithoutIsNullDiagnostic.bsl

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