diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/AttachableEventHandlerNameCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/AttachableEventHandlerNameCheck.java index 4a42b7e5..417ebd68 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/AttachableEventHandlerNameCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/AttachableEventHandlerNameCheck.java @@ -116,6 +116,11 @@ public class AttachableEventHandlerNameCheck Invocation inv = (Invocation)object; FeatureAccess method = inv.getMethodAccess(); String methodName = method.getName(); + if (methodName == null) + { + return; + } + String actionPattern = parameters.getString(PARAM_ACTION_PATTERN); if (!(method instanceof DynamicFeatureAccess) || StringUtils.isEmpty(actionPattern) || inv.getParams().size() != 2 || !(inv.getParams().get(1) instanceof StringLiteral) diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/EventHandlerBooleanParamCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/EventHandlerBooleanParamCheck.java index dbeb7eb6..56201b74 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/EventHandlerBooleanParamCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/EventHandlerBooleanParamCheck.java @@ -226,7 +226,12 @@ public class EventHandlerBooleanParamCheck int index = params.indexOf(param); Map eventHandlers = getAllModuleEvents(module); - CaseInsensitiveString methodName = new CaseInsensitiveString(method.getName()); + String name = method.getName(); + if (name == null) + { + return null; + } + CaseInsensitiveString methodName = new CaseInsensitiveString(name); Event event = eventHandlers.get(methodName); if (event == null && isCorrectModuleForCustomHandlers(module)) { diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportMethodInCommandFormModuleCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportMethodInCommandFormModuleCheck.java index 01faf487..597eca1a 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportMethodInCommandFormModuleCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportMethodInCommandFormModuleCheck.java @@ -131,13 +131,19 @@ public final class ExportMethodInCommandFormModuleCheck if (method.isExport()) { + String name = method.getName(); + if (name == null) + { + return; + } + if (commandModule) { - exportMethods.put(method.getName(), method); + exportMethods.put(name, method); } else if (serverMethodCheck && clientMethodCheck) { - exportMethods.put(method.getName(), method); + exportMethods.put(name, method); } else if (serverMethodCheck ^ clientMethodCheck) { @@ -147,7 +153,7 @@ public final class ExportMethodInCommandFormModuleCheck .intersect(serverMethodCheck ? Environments.ALL_SERVERS : Environments.ALL_CLIENTS); if (enivronmetsObject.containsAny(checkingEnvs)) { - exportMethods.put(method.getName(), method); + exportMethods.put(name, method); } } } diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportProcedureMissingCommentCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportProcedureMissingCommentCheck.java index d48972ac..f7cfb7a2 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportProcedureMissingCommentCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExportProcedureMissingCommentCheck.java @@ -43,7 +43,7 @@ public class ExportProcedureMissingCommentCheck extends AbstractModuleStructureCheck { private static final String CHECK_ID = "export-procedure-missing-comment"; //$NON-NLS-1$ - + @Override public String getCheckId() { @@ -72,9 +72,14 @@ public class ExportProcedureMissingCommentCheck if (method.isExport() && verifyTopRegion(getTopParentRegion(method)) && isMethodHasNoComment(NodeModelUtils.findActualNodeFor(method))) { - resultAceptor.addIssue( - MessageFormat.format(Messages.ExportProcedureMissingCommentCheck_Export_procedure_missing_comment, - method.getName()), + String name = method.getName(); + if (name == null) + { + return; + } + + resultAceptor.addIssue(MessageFormat + .format(Messages.ExportProcedureMissingCommentCheck_Export_procedure_missing_comment, name), McorePackage.Literals.NAMED_ELEMENT__NAME); } } diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExtensionMethodPrefixCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExtensionMethodPrefixCheck.java index a9b11c91..1163883c 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExtensionMethodPrefixCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ExtensionMethodPrefixCheck.java @@ -88,7 +88,7 @@ public class ExtensionMethodPrefixCheck { String prefix = getNamePrefix((IExtensionProject)extension); String methodName = method.getName(); - if (!StringUtils.isEmpty(prefix) && !methodName.startsWith(prefix)) + if (methodName != null && !StringUtils.isEmpty(prefix) && !methodName.startsWith(prefix)) { resultAceptor.addIssue( MessageFormat.format(Messages.ExtensionMethodPrefixCheck_Ext_method__0__should_have__1__prefix, diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleEmptyMethodCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleEmptyMethodCheck.java index 522f36f3..b8cca732 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleEmptyMethodCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleEmptyMethodCheck.java @@ -78,6 +78,10 @@ public final class ModuleEmptyMethodCheck { Method method = (Method)object; String methodName = method.getName(); + if (methodName == null) + { + return; + } String excludeNamePattern = parameters.getString(EXCLUDE_METHOD_NAME_PATTERN_PARAMETER_NAME); boolean allowMethodComments = parameters.getBoolean(ALLOW_METHOD_COMMENTS_PARAMETER_NAME); diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureEventRegionsCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureEventRegionsCheck.java index d60ef68e..f7bddd3f 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureEventRegionsCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureEventRegionsCheck.java @@ -106,9 +106,15 @@ public class ModuleStructureEventRegionsCheck else if (!ModuleType.COMMON_MODULE.equals(moduleType) && !eventHandlersName.equalsIgnoreCase(name) && method.isEvent()) { + String methodName = method.getName(); + if (methodName == null) + { + return; + } resultAceptor.addIssue( MessageFormat.format(Messages.ModuleStructureEventRegionsCheck_Event_handler__0__not_region__1, - method.getName(), eventHandlersName), McorePackage.Literals.NAMED_ELEMENT__NAME); + methodName, eventHandlersName), + McorePackage.Literals.NAMED_ELEMENT__NAME); } } diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureMethodInRegionCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureMethodInRegionCheck.java index b704d9c8..7322ba18 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureMethodInRegionCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/ModuleStructureMethodInRegionCheck.java @@ -78,8 +78,7 @@ public class ModuleStructureMethodInRegionCheck protected void configureCheck(CheckConfigurer builder) { builder.title(Messages.ModuleStructureMethodInRegionCheck_Title) - .description( - Messages.ModuleStructureMethodInRegionCheck_Description) + .description(Messages.ModuleStructureMethodInRegionCheck_Description) .complexity(CheckComplexity.NORMAL) .severity(IssueSeverity.MINOR) .issueType(IssueType.CODE_STYLE) @@ -141,8 +140,14 @@ public class ModuleStructureMethodInRegionCheck private void addIssue(ResultAcceptor resultAceptor, Method method, String regions) { + String name = method.getName(); + if (name == null) + { + return; + } + resultAceptor.addIssue(MessageFormat.format( - Messages.ModuleStructureMethodInRegionCheck_Method_should_be_placed_in_one_of_the_standard_regions, - method.getName(), regions), McorePackage.Literals.NAMED_ELEMENT__NAME); + Messages.ModuleStructureMethodInRegionCheck_Method_should_be_placed_in_one_of_the_standard_regions, name, + regions), McorePackage.Literals.NAMED_ELEMENT__NAME); } } diff --git a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/RedundantExportMethodCheck.java b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/RedundantExportMethodCheck.java index 32439a25..373f7fd1 100644 --- a/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/RedundantExportMethodCheck.java +++ b/bundles/com.e1c.v8codestyle.bsl/src/com/e1c/v8codestyle/bsl/check/RedundantExportMethodCheck.java @@ -146,6 +146,10 @@ public final class RedundantExportMethodCheck Module module = EcoreUtil2.getContainerOfType(method, Module.class); String name = method.getName(); + if (name == null) + { + return; + } if (isNotExclusion(parameters, method, monitor) && !isScheduledJobOrEventSubscription(module, name, monitor) && !existLocalNotifyDescription(module, name, monitor) && !haveCallerInOtherModule(method, monitor)) { diff --git a/docs/checks/bsl.md b/docs/checks/bsl.md index 841c613a..7974efc0 100644 --- a/docs/checks/bsl.md +++ b/docs/checks/bsl.md @@ -1,14 +1,16 @@ # Проверки модулей 1С -Общее количество проверок: 81 +Общее количество проверок: 87 | Код проверки | Наименование | |--------------|--------------| | [begin-transaction](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/begin-transaction.md) | После начала транзакции отсуствует блок Попытка-Исключение | | [bsl-canonical-pragma](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-canonical-pragma.md) | Аннотация написана канонически | | [bsl-nstr-string-literal-format](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-nstr-string-literal-format.md) | НСтр формат строкового литерала | +| [bsl-variable-name-invalid](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-variable-name-invalid.md) | Правила образования имен переменных | | [change-and-validate-instead-of-around](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/change-and-validate-instead-of-around.md) | Используется аннотация &ИзменениеИКонтроль вместо &Вместо | +| [code-after-async-call](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/code-after-async-call.md) | Код расположен после асинхронного вызова | | [commit-transaction](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/commit-transaction.md) | Проверка нарушения схемы работы с транзакциями | | [common-module-missing-api](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/common-module-missing-api.md) | Общий модуль должен иметь хотя бы один экспортный метод | | [common-module-named-self-reference](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/common-module-named-self-reference.md) | Избыточное обращение по собственному имени внутри общего модуля | @@ -47,6 +49,8 @@ | [invocation-parameter-type-intersect](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/invocation-parameter-type-intersect.md) | Вызываемый тип пересекается с типом параметра | | [lock-out-of-try](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/lock-out-of-try.md) | Вызов "Заблокировать()" находится вне попытки | | [manager-module-named-self-reference](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/manager-module-named-self-reference.md) | Избыточное обращение по собственному имени внутри модуля менеджера | +| [method-isinrole-role-exist](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-isinrole-role-exist.md) | Обращение к несуществующей роли | +| [method-optional-parameter-before-required](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-optional-parameter-before-required.md) | Необязательные параметры процедуры/функции расположены перед обязательными | | [method-param-value-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-param-value-type.md) | Параметр метода имеет тип | | [method-too-many-params](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-too-many-params.md) | Метод содержит слишком много параметров | | [missing-temporary-file-deletion](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/missing-temporary-file-deletion.md) | Отсутствует удаление временного файла после использования. | @@ -71,6 +75,7 @@ | [new-font](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/new-font.md) | Использование конструкции "Новый Шрифт" | | [notify-description-to-server-procedure](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/notify-description-to-server-procedure.md) | Описание оповещения на серверную процедуру | | [object-module-export-variable](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/object-module-export-variable.md) | Использование переменных в программных модулях | +| [optional-form-parameter-access](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/optional-form-parameter-access.md) | Обращение к опциональному параметру формы | | [property-return-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/property-return-type.md) | Свойство объекта имеет тип возвращаемого значения | | [public-method-caching](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/public-method-caching.md) | Проверка кэширования программного интерфейса | | [query-in-loop](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/query-in-loop.md) | Запрос в цикле | @@ -82,6 +87,7 @@ | [structure-consructor-too-many-keys](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/structure-consructor-too-many-keys.md) | Конструктор структуры содержит слишком много ключей | | [structure-consructor-value-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/structure-consructor-value-type.md) | Типизация значений в конструкторе структуры | | [typed-value-adding-to-untyped-collection](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/typed-value-adding-to-untyped-collection.md) | Добавление типизированного значения в не типизированную коллекцию | +| [unknown-form-parameter-access](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/unknown-form-parameter-access.md) | Обращение к несуществующему параметру формы | | [use-non-recommended-method](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/use-non-recommended-method.md) | Использование не рекомендуемых методов | | [using-form-data-to-value](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/using-form-data-to-value.md) | Использование РеквизитФормыВЗначение и ДанныеФормыВЗначение | | [using-isinrole](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/using-isinrole.md) | Использован метод "РольДоступна" | diff --git a/docs/checks/check_index.md b/docs/checks/check_index.md index e8e98ce0..ae0b84d2 100644 --- a/docs/checks/check_index.md +++ b/docs/checks/check_index.md @@ -1,11 +1,11 @@ # Индекс проверок -Общее количество проверок 1С:Стандарты разработки V8: 146 +Общее количество проверок 1С:Стандарты разработки V8: 155 -- form: 7 -- md: 26 -- bsl: 81 +- form: 9 +- md: 27 +- bsl: 87 - ql: 8 - right: 24 @@ -16,7 +16,9 @@ | [begin-transaction](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/begin-transaction.md) | После начала транзакции отсуствует блок Попытка-Исключение | | [bsl-canonical-pragma](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-canonical-pragma.md) | Аннотация написана канонически | | [bsl-nstr-string-literal-format](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-nstr-string-literal-format.md) | НСтр формат строкового литерала | +| [bsl-variable-name-invalid](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/bsl-variable-name-invalid.md) | Правила образования имен переменных | | [change-and-validate-instead-of-around](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/change-and-validate-instead-of-around.md) | Используется аннотация &ИзменениеИКонтроль вместо &Вместо | +| [code-after-async-call](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/code-after-async-call.md) | Код расположен после асинхронного вызова | | [commit-transaction](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/commit-transaction.md) | Проверка нарушения схемы работы с транзакциями | | [common-module-missing-api](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/common-module-missing-api.md) | Общий модуль должен иметь хотя бы один экспортный метод | | [common-module-name-cached](../../bundles/com.e1c.v8codestyle.md/markdown/ru/common-module-name-cached.md) | Общий модуль с повторно используемыми значениями | @@ -32,6 +34,7 @@ | [common-module-type](../../bundles/com.e1c.v8codestyle.md/markdown/ru/common-module-type.md) | Общий модуль имеет некорректный тип | | [configuration-data-lock-mode](../../bundles/com.e1c.v8codestyle.md/markdown/ru/configuration-data-lock-mode.md) | Режим блокировки данных конфигурации | | [constructor-function-return-section](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/constructor-function-return-section.md) | Секция возвращаемого значения функции-конструктора данных | +| [data-composition-conditional-appearance-use](../../bundles/com.e1c.v8codestyle.form/markdown/ru/data-composition-conditional-appearance-use.md) | Условное оформление в формах | | [data-exchange-load](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/data-exchange-load.md) | Проверка ОбменДанными.Загрузка в обработчике события | | [db-object-anyref-type](../../bundles/com.e1c.v8codestyle.md/markdown/ru/db-object-anyref-type.md) | Реквизиты составного типа, такие как ЛюбаяСсылка и аналогичные | | [db-object-ref-non-ref-type](../../bundles/com.e1c.v8codestyle.md/markdown/ru/db-object-ref-non-ref-type.md) | Реквизиты составного типа | @@ -64,6 +67,7 @@ | [extension-variable-prefix](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/extension-variable-prefix.md) | У имени переменной отсутствует префикс расширения | | [form-commands-single-action-handler](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-commands-single-action-handler.md) | У каждого действия команды должна быть назначена своя процедура-обработчик | | [form-dynamic-list-item-title](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-dynamic-list-item-title.md) | Пустой заголовок для колонок динамического списка | +| [form-item-visible-settings-by-roles](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-item-visible-settings-by-roles.md) | Использована ролевая настройка видимости (редактирования, использования) для элемента формы | | [form-items-single-event-handler](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-items-single-event-handler.md) | У каждого события должна быть назначена своя процедура-обработчик | | [form-list-field-ref-not-added](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-list-field-ref-not-added.md) | Реквизит "Ссылка" динамического списка не выведен в таблицу на форме | | [form-list-ref-use-always-flag-disabled](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-list-ref-use-always-flag-disabled.md) | У реквизита "Ссылка" динамического списка выключен признак "Использовать всегда" | @@ -72,6 +76,7 @@ | [form-module-pragma](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/form-module-pragma.md) | Использование директив компиляции модуля формы | | [form-self-reference](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/form-self-reference.md) | Использование устаревшего псевдонима | | [function-return-value-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/function-return-value-type.md) | Функция возвращает типизированное значение | +| [functional-option-privileged-get-mode](../../bundles/com.e1c.v8codestyle.md/markdown/ru/functional-option-privileged-get-mode.md) | В функциональной опции не установлен флаг "Привилегированный режим при получении" | | [input-field-list-choice-mode](../../bundles/com.e1c.v8codestyle.form/markdown/ru/input-field-list-choice-mode.md) | В полях форм со списками выбора следует всегда устанавливать свойство **РежимВыбораИзСписка** в значение Истина | | [invocation-form-event-handler](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/invocation-form-event-handler.md) | Программный вызов обработчика события формы | | [invocation-parameter-type-intersect](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/invocation-parameter-type-intersect.md) | Вызываемый тип пересекается с типом параметра | @@ -84,6 +89,8 @@ | [mdo-name-length](../../bundles/com.e1c.v8codestyle.md/markdown/ru/mdo-name-length.md) | Длина имени объекта метаданных | | [mdo-ru-name-unallowed-letter](../../bundles/com.e1c.v8codestyle.md/markdown/ru/mdo-ru-name-unallowed-letter.md) | Проверка наличия буквы "ё" в имени, синониме или комментарии объекта метаданных | | [mdo-scheduled-job-description](../../bundles/com.e1c.v8codestyle.md/markdown/ru/mdo-scheduled-job-description.md) | Задано наименование предопределенного регламентного задания | +| [method-isinrole-role-exist](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-isinrole-role-exist.md) | Обращение к несуществующей роли | +| [method-optional-parameter-before-required](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-optional-parameter-before-required.md) | Необязательные параметры процедуры/функции расположены перед обязательными | | [method-param-value-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-param-value-type.md) | Параметр метода имеет тип | | [method-too-many-params](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/method-too-many-params.md) | Метод содержит слишком много параметров | | [missing-temporary-file-deletion](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/missing-temporary-file-deletion.md) | Отсутствует удаление временного файла после использования. | @@ -108,6 +115,7 @@ | [new-font](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/new-font.md) | Использование конструкции "Новый Шрифт" | | [notify-description-to-server-procedure](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/notify-description-to-server-procedure.md) | Описание оповещения на серверную процедуру | | [object-module-export-variable](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/object-module-export-variable.md) | Использование переменных в программных модулях | +| [optional-form-parameter-access](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/optional-form-parameter-access.md) | Обращение к опциональному параметру формы | | [property-return-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/property-return-type.md) | Свойство объекта имеет тип возвращаемого значения | | [public-method-caching](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/public-method-caching.md) | Проверка кэширования программного интерфейса | | [ql-camel-case-string-literal](../../bundles/com.e1c.v8codestyle.ql/markdown/ru/ql-camel-case-string-literal.md) | Строковый литерал в запросе содержит не КемелКейс контент | @@ -154,6 +162,7 @@ | [structure-consructor-value-type](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/structure-consructor-value-type.md) | Типизация значений в конструкторе структуры | | [subsystem-synonym-too-long](../../bundles/com.e1c.v8codestyle.md/markdown/ru/subsystem-synonym-too-long.md) | Длина названия раздела превышает 35 символов | | [typed-value-adding-to-untyped-collection](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/typed-value-adding-to-untyped-collection.md) | Добавление типизированного значения в не типизированную коллекцию | +| [unknown-form-parameter-access](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/unknown-form-parameter-access.md) | Обращение к несуществующему параметру формы | | [unsafe-password-ib-storage](../../bundles/com.e1c.v8codestyle.md/markdown/ru/unsafe-password-ib-storage.md) | Небезопасное хранение паролей в информационной базе | | [use-non-recommended-method](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/use-non-recommended-method.md) | Использование не рекомендуемых методов | | [using-form-data-to-value](../../bundles/com.e1c.v8codestyle.bsl/markdown/ru/using-form-data-to-value.md) | Использование РеквизитФормыВЗначение и ДанныеФормыВЗначение | diff --git a/docs/checks/checks_index.txt b/docs/checks/checks_index.txt index 6559d17c..c690d709 100644 --- a/docs/checks/checks_index.txt +++ b/docs/checks/checks_index.txt @@ -1,7 +1,9 @@ begin-transaction bsl-canonical-pragma bsl-nstr-string-literal-format +bsl-variable-name-invalid change-and-validate-instead-of-around +code-after-async-call commit-transaction common-module-missing-api common-module-name-cached @@ -17,6 +19,7 @@ common-module-named-self-reference common-module-type configuration-data-lock-mode constructor-function-return-section +data-composition-conditional-appearance-use data-exchange-load db-object-anyref-type db-object-ref-non-ref-type @@ -49,6 +52,7 @@ extension-method-prefix extension-variable-prefix form-commands-single-action-handler form-dynamic-list-item-title +form-item-visible-settings-by-roles form-items-single-event-handler form-list-field-ref-not-added form-list-ref-use-always-flag-disabled @@ -57,6 +61,7 @@ form-module-missing-pragma form-module-pragma form-self-reference function-return-value-type +functional-option-privileged-get-mode input-field-list-choice-mode invocation-form-event-handler invocation-parameter-type-intersect @@ -69,6 +74,8 @@ md-standard-attribute-synonym-empty mdo-name-length mdo-ru-name-unallowed-letter mdo-scheduled-job-description +method-isinrole-role-exist +method-optional-parameter-before-required method-param-value-type method-too-many-params missing-temporary-file-deletion @@ -93,6 +100,7 @@ new-color new-font notify-description-to-server-procedure object-module-export-variable +optional-form-parameter-access property-return-type public-method-caching ql-camel-case-string-literal @@ -139,6 +147,7 @@ structure-consructor-too-many-keys structure-consructor-value-type subsystem-synonym-too-long typed-value-adding-to-untyped-collection +unknown-form-parameter-access unsafe-password-ib-storage use-non-recommended-method using-form-data-to-value diff --git a/docs/checks/form.md b/docs/checks/form.md index 28c4aaa1..7ff307ec 100644 --- a/docs/checks/form.md +++ b/docs/checks/form.md @@ -1,12 +1,14 @@ # Проверки Форм 1С -Общее количество проверок: 7 +Общее количество проверок: 9 | Код проверки | Наименование | |--------------|--------------| +| [data-composition-conditional-appearance-use](../../bundles/com.e1c.v8codestyle.form/markdown/ru/data-composition-conditional-appearance-use.md) | Условное оформление в формах | | [form-commands-single-action-handler](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-commands-single-action-handler.md) | У каждого действия команды должна быть назначена своя процедура-обработчик | | [form-dynamic-list-item-title](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-dynamic-list-item-title.md) | Пустой заголовок для колонок динамического списка | +| [form-item-visible-settings-by-roles](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-item-visible-settings-by-roles.md) | Использована ролевая настройка видимости (редактирования, использования) для элемента формы | | [form-items-single-event-handler](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-items-single-event-handler.md) | У каждого события должна быть назначена своя процедура-обработчик | | [form-list-field-ref-not-added](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-list-field-ref-not-added.md) | Реквизит "Ссылка" динамического списка не выведен в таблицу на форме | | [form-list-ref-use-always-flag-disabled](../../bundles/com.e1c.v8codestyle.form/markdown/ru/form-list-ref-use-always-flag-disabled.md) | У реквизита "Ссылка" динамического списка выключен признак "Использовать всегда" | diff --git a/docs/checks/md.md b/docs/checks/md.md index 10f472cc..6067cacb 100644 --- a/docs/checks/md.md +++ b/docs/checks/md.md @@ -1,7 +1,7 @@ # Проверки метаданных 1С -Общее количество проверок: 26 +Общее количество проверок: 27 | Код проверки | Наименование | |--------------|--------------| @@ -20,6 +20,7 @@ | [db-object-ref-non-ref-type](../../bundles/com.e1c.v8codestyle.md/markdown/ru/db-object-ref-non-ref-type.md) | Реквизиты составного типа | | [document-post-in-privileged-mode](../../bundles/com.e1c.v8codestyle.md/markdown/ru/document-post-in-privileged-mode.md) | В документе, предполагающем проведение, не установлен флаг "Привилегированный режим при проведении / отмене проведения" | | [extension-md-object-prefix](../../bundles/com.e1c.v8codestyle.md/markdown/ru/extension-md-object-prefix.md) | У имени объекта отсутствует префикс расширения | +| [functional-option-privileged-get-mode](../../bundles/com.e1c.v8codestyle.md/markdown/ru/functional-option-privileged-get-mode.md) | В функциональной опции не установлен флаг "Привилегированный режим при получении" | | [md-list-object-presentation](../../bundles/com.e1c.v8codestyle.md/markdown/ru/md-list-object-presentation.md) | Не заполнено ни представление объекта, ни представление списка | | [md-object-attribute-comment-incorrect-type](../../bundles/com.e1c.v8codestyle.md/markdown/ru/md-object-attribute-comment-incorrect-type.md) | Реквизит «Комментарий» у документов | | [md-object-attribute-comment-not-exist](../../bundles/com.e1c.v8codestyle.md/markdown/ru/md-object-attribute-comment-not-exist.md) | Документ не имеет реквизита "Комментарий" |