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

Стандарт 467: Избыточное обращение внутри модуля через его имя или псевдоним ЭтотОбъект (к методу, свойству или реквизиту) #1020

Merged

Conversation

mgalios
Copy link
Contributor

@mgalios mgalios commented Apr 20, 2022

Что сделано

Добавлены 4 проверки:

  1. Проверка всех модулей на избыточно обращение к самому себе через псевдоним ЭтотОбъект/ThisObject.
  2. Проверка модулей форм на обращение к самому себе через устаревший псевдоним ЭтаФорма/ThisForm.
  3. Проверка общих модулей на обращение к самому себе по собственному имени (кроме модулей с повторным использованием возвращаемых значений).
  4. Проверка модулей менеджеров на обращение к самому себе по собственному имени.

Для каждой проверки добавлен квикфикс.

Чек-лист

Общее:

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

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

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

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

Closes #310

@mgalios mgalios requested a review from vadimeg April 26, 2022 04:29
@vadimeg
Copy link
Collaborator

vadimeg commented Apr 26, 2022

Вопросов по коду больше нет. Вливать или еще планируется что-то менять/добавлять?

@mgalios
Copy link
Contributor Author

mgalios commented Apr 26, 2022

Вопросов по коду больше нет. Вливать или еще планируется что-то менять/добавлять?

Больше ничего не планируется.

@vadimeg vadimeg merged commit 6b08761 into 1C-Company:master Apr 26, 2022
@vadimeg vadimeg added this to the 0.3 для EDT 2022.1 milestone Apr 26, 2022
@vadimeg vadimeg added the bsl 1C Built-in Script Language label Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bsl 1C Built-in Script Language
Projects
None yet
3 participants