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

Поддержка конфигураций на английском коде #2199

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

ava1c
Copy link
Contributor

@ava1c ava1c commented Sep 12, 2024

реализация #2198

сделанные изменения

добавлена проверка значений возвращаемых платформой на английском языке
с этими изменениями у меня работает тестирование УНФ переведённой на английский код

@Pr-Mex
Copy link
Owner

Pr-Mex commented Sep 12, 2024

run tests

Copy link
Contributor

Choose a reason for hiding this comment

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

@Pr-Mex явных проблем не вижу. Но если количество таких проверок будет возрастать - я бы добавил асбтракцию над именами... Т.е. не проверять каждый раз на перечень свойств, а определить отдельно коллекцию свойств для проверки. В которой ключ "Ссылка" будет иметь значения "Ссылка; Ref" - и проверять вхождение в список. Возможно, стоит так сделать сразу - оставляю решение за тобой.

@Pr-Mex
Copy link
Owner

Pr-Mex commented Sep 14, 2024

run tests

1 similar comment
@Pr-Mex
Copy link
Owner

Pr-Mex commented Sep 16, 2024

run tests

@@ -2130,6 +2150,11 @@
МетаОбъект = Метаданные.НайтиПоТипу(Тип);
ПолноеИмяОбъекта = МетаОбъект.ПолноеИмя();
ПолноеИмяМенеджера = СтрЗаменить(ПолноеИмяОбъекта, ".", "Менеджер.");

Если СтрНайти(ПолноеИмяОбъекта, "Catalog") > 0 Тогда
Copy link
Owner

Choose a reason for hiding this comment

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

Нужно заменить
СтрНайти
на
Найти
Иначе ломается совместимость с 8.2

@Pr-Mex
Copy link
Owner

Pr-Mex commented Sep 18, 2024

@ava1c
Написал комментарий.
Надо убрать СтрНайти

@ava1c
Copy link
Contributor Author

ava1c commented Sep 25, 2024

@Pr-Mex поменял на Найти()

@Pr-Mex
Copy link
Owner

Pr-Mex commented Sep 29, 2024

run tests

@Pr-Mex Pr-Mex merged commit 5b16836 into Pr-Mex:develop Oct 2, 2024
1 check passed
@Pr-Mex
Copy link
Owner

Pr-Mex commented Oct 2, 2024

@ava1c
Спасибо!

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