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


#1

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

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

#2

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


#3

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

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

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


#4

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


#5

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


#6

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


#7

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


#8

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


#9

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