Ошибки в правилах плагина BSL к Сонару

#1

То, что замечали у себя:

  1. Ложное срабатывание по правилу bsl:StandardRegionCheck в модуле, где есть единственная область по стандарту вида
    #Область ОбработчикиСобытийЭлементовТаблицыФормы<ИмяТаблицыФормы>
    //Код процедур и функций
    #КонецОбласти
  2. Формируется замечание по правилу bsl:LineLength для многострочных текстов запросов, если последняя строка в тексте запроса больше 120 символов.
  3. В правиле bsl:MethodSize учитывается каждая строка многострочного текста запроса.
1 Симпатия
#2

Спасибо, учтем!

#3

Еще наткнулись: ложное определение строки, как текста запроса из-за “Выбрать”:

Настройки.Вставить("ЗаголовокКомандыВыбораКаталога", "Выбрать каталог");

Плагин определяет строку “Выбрать каталог” как текст запроса и начинает к ней применять правила для запросов.
Но, думаю, вряд ли тут как-то можно избежать таких ложных срабатываний.

#4

вроде бы делались уточнения парсера, что после ВЫБРАТЬ может быть только одиночный литерал, а остальное не компилироваться. Там точно ошибка правил запросов или именно ошибка парсинга запроса?

#5

@artbear я добавил в тестовые случаи. @a.nenashev все там можно. @nixel2007 там уже не так, там уже чуть сложней определение

#6

Начинает к этой строке выдавать замечания, как к тексту запроса.
Что “Выбрать”, а не каноничное для запросов “ВЫБРАТЬ” и т.п.

#7

просто довольно сложно без каких-то допущений автоматически определить, что в
Настройки.Вставить("ЗаголовокКомандыВыбораКаталога", "Выбрать каталог");
значение “Выбрать каталог” - это не текст запроса,
а в
Настройки.Вставить("ВЭтомСвойствеТекстЗапросаСЗамечаниями", "Выбрать истина");
уже текст

1 Симпатия
#8

В следующем релизе это исправление будет.

#9

ура, жду больше года :slight_smile: