2 Правила "Неверно задан обработчик метода http-сервиса" и "Неверно задан обработчик операции web-сервиса" #1792
Conversation
добавил английский файл + precommit
artbear changed the title 
| Реализованы правила "Неверно задан обработчик метода http-сервиса" ПР готов к ревью |
.../com/github/_1c_syntax/bsl/languageserver/diagnostics/WrongHttpServiceHandlerDiagnostic.java Outdated Show resolved Hide resolved
.../com/github/_1c_syntax/bsl/languageserver/diagnostics/WrongHttpServiceHandlerDiagnostic.java Outdated Show resolved Hide resolved
.../com/github/_1c_syntax/bsl/languageserver/diagnostics/WrongHttpServiceHandlerDiagnostic.java Outdated Show resolved Hide resolved
убрал тег Design вызываю метод SymbolTree#getMethodSymbol
пример правильного метода-обработчика
| @nixel2007 все замечания исправлены. посмотришь? |
| @nixel2007 все замечания исправлены. посмотришь? |
| @nixel2007 падение одной сборки не по вине кода из ПР, я проверил падающий тест testCompatibilityMode837(), он работает. |
| @artbear надо конфликты порешать |
| @nixel2007 |
Документация хромает. Ни из названия ни из сообщения ни из описания я так и не понял, что она диагностирует, что неверно, что надо исправить. Также прошу убрать мусорную вводную "Разработчики допускают ошибки" |
| исправлю "Разработчики иногда допускают ошибки" Если есть предложения, что стоит написать для исправления, предлагайте. мне казалось, что текст описания вполне себе понятный что тут непонятного-то? |
| Нужно стиль подачи информации изменить: |
| Упрощенно - выкидываешь первую свою строку, а остальное приводишь в формат пояснения ошибки и мер по лечению |
…bHttpServices # Conflicts: # docs/diagnostics/index.md # docs/en/diagnostics/index.md # src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/parameters-schema.json # src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/schema.json
| @asosnoviy @theshadowco уточнил описание правила и примеры посмотрите. |
| /rebase |
# Conflicts: # docs/diagnostics/index.md # docs/en/diagnostics/index.md # src/main/resources/com/github/_1c_syntax/bsl/languageserver/configuration/parameters-schema.json
5 participants
Описание
Реализованы правила
Добавлена проверка количества параметров у методов-обработчиков
Из-за отсутствия данных о параметрах операции веб-сервиса не удалось сделать проверку соответствия числа параметров из описания операции сервиса и числа параметров из метода-обработчика в модуле сервиса - сделал ишуз 1c-syntax/mdclasses#269
Связанные задачи
Closes #1222
Чеклист
Общие
gradlew precommit)Для диагностик
Дополнительно