-
Notifications
You must be signed in to change notification settings - Fork 104
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
Диагностика: Запрещенное имя методанных #1850
Conversation
...va/com/github/_1c_syntax/bsl/languageserver/diagnostics/ForbiddenMetadataNameDiagnostic.java
Outdated
Show resolved
Hide resolved
private void checkName(String name, MDOReference mdoReference) { | ||
if (FORBIDDEN_NAMES_PATTERN.matcher(name).matches()) { | ||
String mdoRef; | ||
if (serverConfiguration.getLanguage() == Language.RU) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
увы, но так... надо что-то думать с mdoref для печати
|
||
@Override | ||
protected void check() { | ||
Ranges.getFirstSignificantTokenRange(documentContext.getTokens()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
а такое срабатывание погасится через diagnosticIgnorance?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
нет, так же как и остальные по подобному принципу, например общие модули
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
надо бы потом отдельно это зачинить как-нибудь
Kudos, SonarCloud Quality Gate passed! |
Описание
Реализована диагностика, обнаруживающая метаданные с именами, использование которых в запросе не рекомендуется стандартом 1С.
Замечания вешаются на первый рендж модуля по приоритету
Связанные задачи
Closes: #1130
Чеклист
Общие
gradlew precommit
)Для диагностик
Дополнительно