Skip to content

Issues 55:Изменен файл настроек, добавлено логирование в ЖР - #56

Draft
Arcius7012 wants to merge 3 commits into
otymko:developfrom
Arcius7012:develop
Draft

Issues 55:Изменен файл настроек, добавлено логирование в ЖР#56
Arcius7012 wants to merge 3 commits into
otymko:developfrom
Arcius7012:develop

Conversation

@Arcius7012

Copy link
Copy Markdown

Добрый день!
Делал для себя, буду рад если пригодится в основном проекте

@Stepa86

Stepa86 commented May 24, 2020

Copy link
Copy Markdown
Collaborator

Изменение файла параметров - ок, а вот запись в журнал сомнительна.

При полностью правильном процессе - АПК создается/разворачивается, затем в нее подгружается инфа о проверяемой конфе, выполняется проверка, выгружается результат и база дропается.

Получить информацию о процессе можно из спец. файла лога - если это написать или из лога раннера, который в stdout пишет, если писать Сообщить( в коде.

Я бы наверно заменил вывод в журнал на аналог логера:

  1. Новый параметр УровеньЛогирования
  2. Вывод через Сообщить в зависимости от уровня

@otymko

otymko commented Jun 25, 2020

Copy link
Copy Markdown
Owner

@Arcius7012 Поддерживаю комментарии @Stepa86 выше. По хорошему должен быть общий метод логирования с уровнями. Очень частный вариант использования 1С: АПК + acc_export -> запустили проверку, после анализа и экспорта база удаляется.

@otymko
otymko marked this pull request as draft June 26, 2020 13:32
@Arcius7012

Copy link
Copy Markdown
Author

Ок, сегодня-завтра сделаю

@Arcius7012

Copy link
Copy Markdown
Author

Уровень логирования добавил в обработку и настройки, я только не понял как правильно разложить обработку acc-export.epf в каталог "src\acc-export"

@Arcius7012 Arcius7012 closed this Jul 6, 2020
@Arcius7012 Arcius7012 reopened this Jul 6, 2020
@Arcius7012

Copy link
Copy Markdown
Author

Разобрался, настроил прекоммит

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