Skip to content

#627. Добавлено соответствие для английского языка свойств объектов. #628

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jun 28, 2016
Merged

#627. Добавлено соответствие для английского языка свойств объектов. #628

merged 4 commits into from
Jun 28, 2016

Conversation

@poabor
Copy link
Contributor

@poabor poabor commented Jun 27, 2016

реализация #[Номер issue]

сделанные доработки

Есть ограничение - автоматически не распознается вариант встроенного языка. Для этого создан реквизит, который нужно указать вручную.

@poabor
#627. Добавлено соответствие для английского языка свойств объектов.
f2ac7e0 
Есть ограничение - автоматически не распознается вариант встроенного языка. Для этого создан реквизит, который нужно указать вручную.
artbear
artbear reviewed Jun 28, 2016
View reviewed changes
src/Plugins/СериализаторMXL/ObjectModule.bsl Outdated
менеджерОбъекта = Документы[МетаданныеТипаРеквизита.Имя];
ИначеЕсли Найти(ПолноеИмяТипаРеквизита, "Перечисление") = 1 Тогда
ИначеЕсли Найти(ПолноеИмяТипаРеквизита, ВернутьЗначениеСоответствия("Перечисление")) Тогда
Copy link
Member

@artbear artbear Jun 28, 2016

Choose a reason for hiding this comment

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

Неверный копи-паст :(
В исходной строке есть сравнение с 1
Найти(ПолноеИмяТипаРеквизита, "Перечисление") = 1
а в текущей строке нет.

@artbear
Copy link
Member

artbear commented Jun 28, 2016

@poabor
Правильно я понимаю, что

  1. в xddTestRunner ты изменил только номер версии?
  2. Изменял свой код в 8.3?

@poabor
Copy link
Contributor Author

poabor commented Jun 28, 2016

Ок. поправлю.

  1. в xddTestRunner ты изменил только номер версии?
    да
  2. Изменял свой код в 8.3?
    да

@artbear
Copy link
Member

artbear commented Jun 28, 2016

  1. Предлагаю изменения в xddTestRunner (и его исходниках) откатить как несущественные.
    У меня есть соседний PR, в котором серьезно меняется код браузера и некоторых плагинов. Не хотелось бы серьезно затруднять работу по этому PR
  2. Сохрани свои правки сериализатора в версии 8.2.19
    У нас пока поддерживается 8.2.17, а сохранение в 8.3 ломает совместимость с 8.2.17

@poabor
Copy link
Contributor Author

poabor commented Jun 28, 2016

ок.

@artbear artbear mentioned this pull request Jun 28, 2016
Closed
poabor added 3 commits June 28, 2016 16:37
@poabor
Revert "#627. Добавлено соответствие для английского языка свойств об…
838cf0f 
…ъектов."

This reverts commit f2ac7e0.
@poabor
#627. Добавление соответствия для английского языка для свойств объек…
5030f0f 
…тов.

Убраны несущественные комментарии.
Сериализатор сохранен в версии 8.2.19.130
@poabor
#629.
9b1f743 
Установлена версия сериализатора в "3.0.0.5"
Длина реквизита "Имя" увеличина до 100 символов.
Длина реквизита "ПолноеИмя" увеличино до 255 символов.
@poabor
Copy link
Contributor Author

poabor commented Jun 28, 2016

поправил.
Надо ещё делать PR?

@artbear
Copy link
Member

artbear commented Jun 28, 2016

Нет, все твои коммиты помещены в твою же ветку develop, из которой создан PR и автоматически подхватываются в этот PR.

Изменения посмотрю

@artbear artbear merged commit e226221 into xDrivenDevelopment:develop Jun 28, 2016
@artbear artbear self-assigned this Jun 28, 2016
@artbear artbear added this to the 4.0.1.0 milestone Jun 28, 2016
@artbear
Copy link
Member

artbear commented Jun 28, 2016

@poabor Большое спасибо за работу.

@artbear artbear mentioned this pull request Jul 6, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@artbear artbear

Labels

enhancement Генерация макета данных

Projects

None yet

Milestone

4.0.0.7

Development

Successfully merging this pull request may close these issues.

2 participants

@poabor @artbear