Skip to content

Conversation

@Morkhe
Copy link

@Morkhe Morkhe commented Apr 23, 2022

Собсно, #26
Изменил формат разбора правил регистрации, т.к. некорректно выгружались отборы

Copy link
Owner

Choose a reason for hiding this comment

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

@Morkhe предлагаю разбить на два выражения (СтрШаблон в отдельную переменную)

Copy link
Owner

Choose a reason for hiding this comment

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

Аналогично в выражении ниже

Copy link
Owner

Choose a reason for hiding this comment

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

@Morkhe без "_"

Copy link
Owner

Choose a reason for hiding this comment

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

Аналогично у других методов

Copy link
Owner

Choose a reason for hiding this comment

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

@Morkhe что такое "zzzzz"?

Copy link
Author

Choose a reason for hiding this comment

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

Это решает проблему с сохранением исходной сортировки внутри группы правил регистрации. В конвертации данных отсутствует поле "Порядок", поэтому сортировка идет по коду элемента.
Если не добавить какую-нибудь ерунду, то свойства группы встают после дочерних правил (индекс "0000000001" < "1")
image

Copy link
Owner

Choose a reason for hiding this comment

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

@Morkhe предлагаю вынести условие в функцию

@otymko
Copy link
Owner

otymko commented Apr 24, 2022

@Morkhe просьба, раз дорабатываем тесты, переименовать файл "РазобратьсяПравилаОбмена_test.os" под общий стиль других тестов (Тест_РазобратьПравилаОбмена.os)

@otymko
Copy link
Owner

otymko commented Apr 24, 2022

@Morkhe в остальном все просто прекрасно, спасибо!

@Morkhe Morkhe requested a review from otymko May 6, 2022 09:18
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.

2 participants