Стандарт 773: Отсутствует обязательная проверка признака ОбменДанными.Загрузка в обработчике события

[marmyshev]

Название проверки
Отсутствует обязательная проверка признака ОбменДанными.Загрузка в обработчике события

Английское название проверки
Mandatory checking of DataExchange.Load is absent in event handler

Правило анализа кода/метаданных
Для обработчика события в модуле объекта находим все стейтменты в условии которого есть "ОбменДанными.Загрузка" или аналогичная процедура, в теле условия должен быть возврат.

Вторая проверка может быть опциональной (отключена по умолчанию), если проверка "ОбменДанными.Загрузка" находится не в начале обработчика события, если разработчик сознательно решил выполнять какие-либо действия до проверки.

Мета-информация (пожалуйста, заполните если знаете):

• Номер стандарта: 773, 752, 464, 465
• Код проверки: data-exchange-load
• Тип ошибки: PORTABILITY
• Критичность: MAJOR
• Код ошибки АПК: 75

Параметры проверки

• Список процедур, в которых проверяется обмен данными, перечисленные через запятую

Текст ошибки

• Отсутствует обязательная проверка признака ОбменДанными.Загрузка в обработчике события "ПриЗаписи"
• Нет возврата в проверке ОбменДанными.Загрузка
• Проверка ОбменДанными.Загрузка должна быть в начале обработчика события

Пример некорректного решения
Приведите пример(ы) некорректного кода (или метаданных)

Процедура ПередЗаписью(Отказ)
// код обработчика
// ...
КонецПроцедуры

Описание, почему так делать нельзя
Все действия в процедурах-обработчиков событий ПередЗаписью, ПриЗаписи, ПередУдалением должны выполняться после проверки на ОбменДанными.Загрузка.

Это необходимо для того, чтобы никакая бизнес-логика объекта не выполнялась при записи объекта через механизм обмена данными, поскольку она уже была выполнена для объекта в том узле, где он был создан. В этом случае все данные загружаются в ИБ «как есть», без искажений (изменений), проверок или каких-либо других дополнительных действий, препятствующих загрузке данных.

Пример корректного решения

Процедура ПередЗаписью(Отказ)
Если ОбменДанными.Загрузка Тогда
     Возврат;
КонецЕсли;

// код обработчика
// ...
КонецПроцедуры

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

• Ссылка на стандарт на сайте ИТС https://its.1c.ru/db/v8std#content:773:hdoc
• https://its.1c.ru/db/v8std#content:752:hdoc
• https://its.1c.ru/db/v8std#content:464:hdoc
• https://its.1c.ru/db/v8std#content:465:hdoc