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

Ложное срабатывание проверки: common-module-type (повторное использование на время вызова) #1455

Open
iliashevelev opened this issue Aug 11, 2024 · 1 comment
Assignees
Labels
bug Something isn't working standards Поддержка стандартов

Comments

@iliashevelev
Copy link

Название/код проверки

Код проверки: common-module-type

Версия плагина: 0.7.0

Пример кода/метаданных содержащих ошибку

Общий модуль с постфиксом ПовтИсп (Cached) и вариантом повторного использования возвращаемых значений "На время вызова".

При использовании варианта кеширования "На время сеанса" ложного срабатывания нет.

Почему это НЕ ошибка

3.2.3. Модули, предназначенные для реализации на сервере или на клиенте функций с повторным использованием возвращаемых значений (на время вызова или на время сеанса), именуются с постфиксом "ПовтИсп" (англ. "Cached") и "КлиентПовтИсп" (англ. "ClientCached") соответственно.

В целом, в стандартах разработки нет ограничений на использование модулей с вариантом повторного использования возвращаемых значений "На время вызова".

Дополнительные материалы

common-module-type-cached-issue

Проверка реализована сравнением фактических свойств модуля с эталонным набором свойств.
То есть, в ней не предусмотрено, что равно допустимыми могут быть несколько значений свойства (в примере - на время вызова или на время сеанса).

"Эталонный" набор свойств модуля этого "типа" содержит вариант "На время сеанса" (DURING_SESSION) - он описан здесь:

COMMON_MODULE__RETURN_VALUES_REUSE, ReturnValuesReuse.DURING_SESSION)),

Сама проверка соответствия набора свойств эталону выполняется просто как values.equals() здесь:
https://github.com/1C-Company/v8-code-style/blob/2afdf4580d3b4847b665b94cbb1a72f3089be795/bundles/com.e1c.v8codestyle.md/src/com/e1c/v8codestyle/md/commonmodule/check/CommonModuleType.java#L114C17-L114C30

@iliashevelev iliashevelev added bug Something isn't working standards Поддержка стандартов labels Aug 11, 2024
@Roffild
Copy link

Roffild commented Aug 26, 2024

Общий модуль для типа "Серверный модуль с повторным использованием возвращаемых значений" имеет некорректные настройки: Клиент (управляемое приложение)

bug

У меня Комбо :D

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working standards Поддержка стандартов
Projects
None yet
Development

No branches or pull requests

3 participants