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

Данные о защищенных, запароленных модулях #413

Closed
wants to merge 8 commits into from

Conversation

artbear
Copy link
Contributor

@artbear artbear commented Jun 21, 2023

Для реализации 1c-syntax/bsl-language-server#3076

Описание

  • поддержка защищенных, запароленных модулей
  • поддержка защищенных, запароленных модулей для перерасчетов\Recalculation
    неожиданно для них отдельная обработка
  • (пока пропускаю, т.к. нет поддержки) Работа на внешних обработках, отчетах
    выделил отдельную задачу [NEW] Данные о защищенных, запароленных модулях внутри внешних отчетов и обработок #415
  • рефакторинг метода computeModules
  • ветка обновлена из master, т.к. в develop очень много изменений, которые вряд ли быстро попадут в релиз
    • а хочется выпустить релиз с поддержкой защищенных правил, чтобы реализовать правило в бсл лс
    • после выпуска релиза mdclasses я обязуюсь перенести эти изменения в ветку девелоп

Связанные задачи

Closes #414

Чеклист

Общие

  • Ветка PR обновлена из develop
  • Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
  • Изменения покрыты тестами
  • Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Дополнительно

рефакторинг метода computeModules
#3076
@@ -335,5 +360,8 @@ protected void checkChild(MdoReference parentMdoReference,
protected static Path getMDOPathDesigner(String path) {
return Paths.get(SRC_DESIGNER, path);
}
protected static Path getMDOPathDesigner(String path, String rootPath) {
Copy link
Member

Choose a reason for hiding this comment

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

@theshadowco обрати внимание

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@theshadowco обрати внимание

тут вообще хочется отрефакторить, привести вызовы всех похожих методов модуля к вызову общих модулей

  • getMDOPathDesigner(String path, String rootPath)
  • getMDOPathEDT(String path, String rootPath)

@artbear
Copy link
Contributor Author

artbear commented Jun 21, 2023

Я проверил, у меня все тесты прошли, кроме, конечно, проверки лицензии (год не тот во всех модулях, не только в моем коде)

@artbear
Copy link
Contributor Author

artbear commented Jun 21, 2023

Уточню - у меня успешно прошли тесты всего проекта, а не только мои тесты из МР

@artbear
Copy link
Contributor Author

artbear commented Jul 20, 2023

@theshadowco посмотри, доработка готова - Данные о защищенных, запароленных модулях

@theshadowco
Copy link
Member

@artbear
Закрываю, т.к. для старой версии либы. смотри связанный реквест

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants