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

СериализаторMXL - Доработана загрузка данных регистра бухгалтерии. #313

Merged
merged 4 commits into from Sep 13, 2018

Conversation

AnatoliyBritko
Copy link
Contributor

@AnatoliyBritko AnatoliyBritko commented Sep 6, 2018

При загрузке данных регистра бухгалтерии настраиваю поиск по ГУИД.
Получаю такую таблицу
image
1.
При поиске значения в цикле выполняется обход всех возможных типов значения субконто и для каждого типа выполняется попытка получения значения по ГУИД см. Значение = менеджерОбъекта.ПолучитьСсылку(Новый УникальныйИдентификатор(стрПоле.Значение));

Ошибка возникает если в ПВХ для субконто задан в качестве возможных - тип перечисление (в типовых это как правило ставки НДС). Т.к. для менеджера перечисления нет метода ПолучитьСсылку, то получаем ошибку.
Добавил проверку на тип перечисление, и для обработки исключительной ситуации возвращаю строку "Объект не найден" т.к. на верхнем уровне стоит проверка на вхождение строки.

  1. Доработал ДобавитьРеквизитыМетаданныхВНаборРеквизитов т.к. для РБ необходимо разделять СубконтоДТ и СубконтоКТ.
    Без этих изменений получаем падение при попытке получения метаданных реквизита.
    image
    Получение идет например для СубконтоДт1, но как видно на скрине в наборе такого значения нет.

… тестов (ТестДолжен_ОткрытьФормуСуществующегоЭлементаСправочника).
…ной ситуации при попытке получения значения перечисления по ссылке.
Удалил лишние комментарии.
@AnatoliyBritko
Copy link
Contributor Author

Еще 2 вопроса попутно:

  1. В PR залетело 2 коммита, хотя надо только 2 последних. Что я сделал не так?
  2. Смотрел тесты для плагина СериализаторMXL - там для регистров бухгалтерии стоит заглушка
    КонтекстЯдра.ПропуститьТест("Тест временно отключен"); с чем это связано?

@artbear artbear self-assigned this Sep 13, 2018
@artbear artbear added the Генерация данных Сериализатор данных label Sep 13, 2018
@artbear artbear added this to the 5.6.0 milestone Sep 13, 2018
@artbear artbear merged commit 7fbee5e into vanessa-opensource:develop Sep 13, 2018
@artbear
Copy link
Collaborator

artbear commented Sep 13, 2018

@AnatoliyBritko Огромное спасибо за очередную доработку.
Круто!

@artbear
Copy link
Collaborator

artbear commented Sep 13, 2018

Смотрел тесты для плагина СериализаторMXL - там для регистров бухгалтерии стоит заглушка
КонтекстЯдра.ПропуститьТест("Тест временно отключен"); с чем это связано?

@AnatoliyBritko точно уже не помню, были какие-то странные падения :(

Возможно, с твоими доработками все и взлетит.

Проверишь эти тесты?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Генерация данных Сериализатор данных
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants