bsl-language-server icon indicating copy to clipboard operation
bsl-language-server copied to clipboard

WiP. исправил FP в RefOveruse при использовании в запросе таблиц, которые являются ТЧ

Open ovcharenko-di opened this issue 2 years ago • 5 comments

Описание

добавил проверку на то, что источник данных является табличной частью

Связанные задачи

Closes #1800

Чеклист

Общие

  • [x] Ветка PR обновлена из develop
  • [x] Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
  • [x] Изменения покрыты тестами
  • [x] Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

  • [ ] Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

реализация сделана "в лоб", поэтому надо бы придумать еще тест-кейсов

ovcharenko-di avatar Oct 08 '21 11:10 ovcharenko-di

нужно проверить на сложных запросах, где псевдонимы таблиц могут повторяться (соединение с подзапросом, ВТ)

ovcharenko-di avatar Oct 08 '21 11:10 ovcharenko-di

@ovcharenko-di а вообще без обращения к метаданным сложно отличить табличную часть от виртуальной таблицы

например. таблица РегистрыСведений.Р1.СрезПоследних() КАК Р1 поле Р1.Ссылка.Реквизит

можно заложить на скобки вирт.таблицы но тогда вариант без скобок будет ФП выдавать - РегистрыСведений.Р1.СрезПоследних

artbear avatar Oct 08 '21 12:10 artbear

Хотя новый парсер умеет определять имена вирт.таблиц

artbear avatar Oct 08 '21 12:10 artbear

В девелоп залит новый парсер запросов. Прошу ребэйзнуть и проверить работоспособность

nixel2007 avatar Oct 08 '21 17:10 nixel2007

/rebase

nixel2007 avatar Jul 19 '22 04:07 nixel2007