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

Доработаны диагностические сообщения в проверке md-object-attribute-comment-incorrect-type #1342

Conversation

VAGoncharov
Copy link
Contributor

@VAGoncharov VAGoncharov commented Jun 19, 2023

Что сделано

  • Улучшены диагностические сообщения проверки

Чек-лист

Общее:

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

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

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

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

Closes #1340
Closes #1345

Copy link
Collaborator

@iArtemv iArtemv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А тут точно не две разные проверки объединены в одну?

resultAceptor.addIssue(DEFAULT_CHECK_MESSAGE, BASIC_FEATURE__TYPE);
String msg = MessageFormat.format(Messages.MdObjectAttributeCommentCheck_message,
Messages.MdObjectAttributeCommentCheck_Is_compound_type);
resultAceptor.addIssue(msg, BASIC_FEATURE__TYPE);
return;
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для получения eContainer() определенного типа ближайший вверх, можно использовать утильный метод EcoreUtil2.getContainerOfType

@VAGoncharov
Copy link
Contributor Author

А тут точно не две разные проверки объединены в одну?

Т.к. эти проверки проверяют реквизит "Комментарий" у объектов метаданных, решили в свое время объединить в одну. Никто не был против.

@marmyshev marmyshev added this to the 0.7 для EDT 2023.3 milestone Dec 23, 2023
@marmyshev marmyshev merged commit 1d3e0b5 into 1C-Company:master Dec 23, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants