#457 Проверка ОбменДанными.Загрузка проверяет расширения

[marmyshev]

Что сделано

• Исправлена проверка - события проверяются в расширении по аннотации в которой указано процедура обработчик события ПриЗаписи, ПередЗаписью и ПередУдалением

Чек-лист

Общее:

• ветка PR обновлена из master и нет конфликтов
• Тесты-кейсы, JUnit тесты правильного и неправильного состояния
• Измененные Вами исходники отформатированы в соответствии с конвенцией
• Авто-аудит (SonarQube и CheckStyle) пройден, покрытие кода хорошее, ошибок нет, плохой код устранен
• Добавлена запись в ИСТОРИЮ ИЗМЕНЕНИЯ, включаемая в пользовательскую документацию плагина

Если применимо:

• Пользовательская документация на доп.инструменты написана (на русском)
• Описание проверок - на двух языках

Закрываемые задачи

Closes #457

[marmyshev]

@DoublesunRUS сделаешь аудит?

[DoublesunRUS]

if (monitor.isCanceled() || !(object instanceof Procedure)

У нас же проверка только процедуры проверяет. Зачем тут повторная проверка типов?
И монитор проверять первой строкой бессмысленно.

[marmyshev]

У нас же проверка только процедуры проверяет. Зачем тут повторная проверка типов?
И монитор проверять первой строкой бессмысленно.

@DoublesunRUS Укажи, где в реквесте ты увидел такой код (добавленный или измененный)?

[DoublesunRUS]

У нас же проверка только процедуры проверяет. Зачем тут повторная проверка типов?
И монитор проверять первой строкой бессмысленно.

@DoublesunRUS Укажи, где в реквесте ты увидел такой код (добавленный или измененный)?

А изменение типа у метода какое отношение имеет к указанной проблеме?

[marmyshev]

А изменение типа у метода какое отношение имеет к указанной проблеме?

Это логическая ошибка - обработчик события не может быть "функцией" - только процедурой. Я это увидел - решил заодно исправить.

Я остальной код не менял, несмотря на то что он может быть признан плохим кодом - в реквесте следует комментировать только то что менялось в реквесте или имеет явное отношение к решаемой проблеме.

[DoublesunRUS]

А изменение типа у метода какое отношение имеет к указанной проблеме?

Это логическая ошибка - обработчик события не может быть "функцией" - только процедурой. Я это увидел - решил заодно исправить.

Я остальной код не менял, несмотря на то что он может быть признан плохим кодом - в реквесте следует комментировать только то что менялось в реквесте или имеет явное отношение к решаемой проблеме.

Я увидел что изменился тип, пошел проверить код который его вызывает. Увидел что в вызове кода также есть недочеты.

[sonarcloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

68.0% Coverage
0.0% Duplication