Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MOD] TransferringParametersBetweenClientAndServer - поддержка методов общих модулей с признаком "Вызов сервера" #3204

Open
l0patinmix opened this issue Dec 13, 2023 · 2 comments
Labels
component/diagnostics Доработка / создание диагностики

Comments

@l0patinmix
Copy link

Диагностика

Передача параметров между клиентом и сервером
https://1c-syntax.github.io/bsl-language-server/diagnostics/TransferringParametersBetweenClientAndServer/

Описание предлагаемых изменений

Сейчас проверяется только контекст формы. Но из клиентского метода формы может быть вызван серверный метод общего модуля, для которого установлен флаг "Вызов сервера". Кажется, можно сделать проверку чтобы у всех методов серверного общего модуля, для которого установлен флаг "Вызов сервера", для параметров было указано ключевое слово "Знач", независимо от того есть ли вызовы этого метода с клиента - потенциально это источник проблем. Такая проверка реализована в 1С:АПК.

@l0patinmix l0patinmix added component/diagnostics Доработка / создание диагностики type/discuss labels Dec 13, 2023
@artbear
Copy link
Contributor

artbear commented Dec 13, 2023

@l0patinmix хорошая идея, спасибо! руки дойдут, сделаю

поправьте название тикета, добавив в него название правила и суть изменения

предлагаю так - [MOD] TransferringParametersBetweenClientAndServer - поддержка методов общих модулей с признаком "Вызов сервера"

@l0patinmix l0patinmix changed the title [MOD] [MOD] TransferringParametersBetweenClientAndServer - поддержка методов общих модулей с признаком "Вызов сервера" Dec 13, 2023
@l0patinmix
Copy link
Author

@artbear отлично! поправил.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/diagnostics Доработка / создание диагностики
Projects
None yet
Development

No branches or pull requests

3 participants