[NEW] [Запросы] Не существует реквизита метаданного, к таблице которого обращаемся внутри запроса

artbear commented 3 years ago

Описание проблемы, ошибки, которую надо диагностировать

При доработке кода 1С и изменения модели метаданных нередко остаются неверные запросы, в которых идет обращение к неверным реквизитам/полям таблицы метаданного, потому что реквизит переименовали или удалили. Также ошибочные имена таблиц могут возникать при ручном изменения запросов, без использования конструктора запросов. Конфигуратор и EDT выполняют проверки кода на правильность при изменении метаданных, но эти проверки могут быть пропущены.

Поэтому нужно контролировать правильность обращения к метаданным внутри текстов запросов.

Пример запроса с неверным именем поля

ВЫБРАТЬ
    Таблица.НесуществующийРеквизит КАК Поле
ИЗ
    РегистрСведений.Регистр1 КАК Таблица

При выполнении подобных запросов будет возникать ошибка исполнения.

Ссылка на источник, подтверждающее нарушение либо обоснование наличия проблемы

Параметры диагностики

Тип Статья на русском

[x] :ant: Ошибка
[ ] :cop: Уязвимость
[ ] :guardsman: Потенциальная уязвимость
[ ] :poop: Качество кода
[ ] :trollface: Другое

Важность Статья на русском

[x] :broken_heart: Блокирующая / Blocker
[ ] :heart: Критическая / Critical
[ ] :yellow_heart: Важная / Major
[ ] :blue_heart: Незначительная / Minor
[ ] :green_heart: Информационная / Info
[ ] :revolving_hearts: Другое

Тэги Статья на русском

[ ] STANDARD - "Нарушение стандартов 1С"
[ ] LOCKINOS - "Не будет работать в другой ОС"
[x] SQL - "Проблема с запросом"
[ ] PERFORMANCE - "Проблема производительности"
[ ] BRAINOVERLOAD - "Непонятный код"
[ ] BADPRACTICE - "Плохая практика программирования"
[ ] CLUMSY - "Излишние действия"
[ ] DESIGN - "Ошибка в проектировании"
[x] SUSPICIOUS - "Подозрительный код"
[ ] UNPREDICTABLE - "Непредсказуемо работающий код"
[ ] DEPRECATED - "Устаревшая функциональность"
[ ] ERROR - "Ошибочная конструкция"
[ ] LOCALIZE - "Проблемы локализации"

Время на исправление (минут)

5 минут

Дополнительная информация

Связанные ишузы

artbear commented 3 years ago

@nixel2007 @theshadowco Назначьте на меня, я сделаю реализацию

nixel2007 commented 3 years ago

а зачем отдельное правило? почему не сделать в рамках #1822?

artbear commented 3 years ago

@nixel2007 Правила немного о разном

не существует метаданного или реквизита метаданного
и реализация существенно различается
- для метаданного все очень просто - тупо подписаться на mdo
- для реквизитов сложная логика - нужно анализировать все места, в которых могут использоваться поля, а их немало
  - в ПР для NULL я накушался таких мест )

Поэтому я и предлагаю оставить правила разделенными

1c-syntax / bsl-language-server