Разработчикам

Ошибочный порядок записи XML

RSS

Ошибочный порядок записи XML, Ошибка в 1С, Доколе?

Пользователь 25773

Эксперт

Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008

24.01.2014 17:12:23

При выгрузке товаров получаем вот такое окно:

Код

{Обработка.ОбменССайтом.МодульОбъекта(3481)}: Ошибка при вызове метода контекста (ЗаписатьКонецЭлемента)
   ЗаписьXML.ЗаписатьКонецЭлемента();
по причине:
Ошибочный порядок записи XML

Ошибка возникает в последнем дополнении Битрикса для "1С Управление торговлей" 10.3.26.2.

Ошибка возникает при выгрузке файла offers.xml, если ни у одного товара нет цен.

//////////////////////////

Зачем написал? Чтобы все знали.

Когда уже будет публичный баг-трекер, где можно увидеть все косяки той или иной версии? У нас 2 программиста 3 часа искали в 1С, почему такая ошибка.

Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Пользователь 25773

Эксперт

Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008

27.01.2014 16:48:46

Найден баг. Строка 2404. Внутренняя обработка ОбменССайтом

Кто-то из старой версии не убрал важный фрагмент и сделал одно лишнее закрытие тега XML.

Код
Если РезультатЗапроса.Пустой() Тогда ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента(); КонецЕсли;

Этот фрагмент следует убрать. Например так:

Исходный текст:

Код

   Если РезультатЗапроса.Пустой() Тогда
      ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента();
   КонецЕсли;

   Выборка = РезультатЗапроса.Выбрать();
   Всего = Выборка.Количество();
   ОбъектCMLПакетПредложений.ЗаписатьНачалоЭлемента("Предложения" ) ;
   
   Пока Выборка.Следующий() Цикл
      
      СписокЗначенийCML = Новый СписокЗначений;
      СтруктураВозврата.ВыгруженоПредложений = СтруктураВозврата.ВыгруженоПредложений + 1;
      
      #Если Клиент Тогда
      Если СтруктураВозврата.ВыгруженоПредложений % 100 = 0 Тогда
         
         ОтобразитьСостояние("Каталог """ + ПараметрыКаталога.Каталог + """: Выгружено объектов пакета предложений: " + СтруктураВозврата.ВыгруженоПредложений + " из " + Всего + ". Выгружается: " + Выборка.НоменклатураСсылка + " " + Выборка.ХарактеристикаСсылка);
         
      КонецЕсли;
      #КонецЕсли
      
      РИ_ЗаполнитьСписокЗначенийПредложения(СписокЗначенийCML, Выборка, МассивСкладов, ВыгрузкаКраткихИзменений);
      
      ОбъектCMLПакетПредложений.ЗаписатьНачалоЭлемента("Предложение" ) ;
      ЗаписатьCMLПоСпискуЗначений(ОбъектCMLПакетПредложений, СписокЗначенийCML);
      ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента();
      
   КонецЦикла;
   ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента();

Заменить на

Код

   Если РезультатЗапроса.Пустой() Тогда
      //   ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента(); 
   Иначе

      Выборка = РезультатЗапроса.Выбрать();
      Всего = Выборка.Количество();
      ОбъектCMLПакетПредложений.ЗаписатьНачалоЭлемента("Предложения" ) ;
      
      Пока Выборка.Следующий() Цикл
         
         СписокЗначенийCML = Новый СписокЗначений;
         СтруктураВозврата.ВыгруженоПредложений = СтруктураВозврата.ВыгруженоПредложений + 1;
         
         #Если Клиент Тогда
         Если СтруктураВозврата.ВыгруженоПредложений % 100 = 0 Тогда
            
            ОтобразитьСостояние("Каталог """ + ПараметрыКаталога.Каталог + """: Выгружено объектов пакета предложений: " + СтруктураВозврата.ВыгруженоПредложений + " из " + Всего + ". Выгружается: " + Выборка.НоменклатураСсылка + " " + Выборка.ХарактеристикаСсылка);
            
         КонецЕсли;
         #КонецЕсли
         
         РИ_ЗаполнитьСписокЗначенийПредложения(СписокЗначенийCML, Выборка, МассивСкладов, ВыгрузкаКраткихИзменений);
         
         ОбъектCMLПакетПредложений.ЗаписатьНачалоЭлемента("Предложение") ;
         ЗаписатьCMLПоСпискуЗначений(ОбъектCMLПакетПредложений, СписокЗначенийCML);
         ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента();
         
      КонецЦикла;
      
      ОбъектCMLПакетПредложений.ЗаписатьКонецЭлемента();
   
   КонецЕсли;

Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#3 27.01.2014 18:32:39 В ближайшей версии модуля обмена эта бага будет поправлена.

Пользователь 25773

Эксперт

Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008

28.01.2014 01:22:02

А можно тупо выложить дополнение-фикс для той же версии 1С?

Вот идея - исправлять критическе ошибки в тот же день
http://idea.1c-bitrix.ru/significantly-accelerate-product-updates-fix-critical-bugs/

Сколько еще ждать ближайшую версию модуля? Пол-года?

Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Александр Денисюк

Администратор

Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013

28.01.2014 11:10:16

Я пересобрал версию модуля вчера. На днях должна обновиться на сайте. Я кста, тоже считаю, что критичные ошибки должны правиться очень оперативно. Другое дело с адаптациями... И да, много кто уже поставил последний модуль и только от вас пришло сообщение о ошибке, поэтому ошибка не столь критична. Она вылазит только при определенных условиях.

Пользователь 25773

Эксперт

Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008

28.01.2014 18:12:28

Спасибо. Я надеюсь ошибка с частями архива тоже будет исправлена:
http://dev.1c-bitrix.ru/community/forums/forum26/topic60094/

Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#7 28.01.2014 19:20:37 Она тоже. Но в тоже время для УПП, КА и продуктов стран СНГ пока обновлена.

Пользователь 238860

Заглянувший

Сообщений: 4 Регистрация: 29.01.2014

29.01.2014 11:12:24

Цитата
Александр Денисюк пишет: много кто уже поставил последний модуль

Поставили, ошибку получили.
Использовали фикс из темы, все заработало. Но потом нас настиг шок: Выгрузка вместо 40 мегабайтов стала весить 400.

КА 1.1.42.2

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#9 29.01.2014 12:20:25 В КА исправлено в модуле версии 1.1.42.4. Просто она еще не обновлена на сайте.

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#10 29.01.2014 12:25:44 По поводу размера - нужно смотреть сами XML файлы.

Пользователь 238860 Заглянувший Сообщений: 4 Регистрация: 29.01.2014	#11 30.01.2014 14:10:45 Обновленный модуль для КА России выложен неправильно. Внутри модуль от УТ Беларуси

Александр Денисюк

Администратор

Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013

#12

30.01.2014 16:03:35

Цитата
Илья пишет: Обновленный модуль для КА России выложен неправильно. Внутри модуль от УТ Беларуси

Сообщил людям. Сейчас должны поправить.

Пользователь 238860

Заглянувший

Сообщений: 4 Регистрация: 29.01.2014

#13

30.01.2014 16:28:22

Что то как то не сложилось с этим обновлением. Пытаюсь удалить реализацию. Вылетает ошибка

Код

Ошибка при выполнении обработчика - 'ОбработкаУдаленияПроведения'
по причине:
{ОбщийМодуль.ПроцедурыОбменаССайтом.Модуль(272)}: Метод объекта не обнаружен (ЕстьТабЧастьДокумента)

Если закоментировать

Код

Если ОбщегоНазначения.ЕстьТабЧастьДокумента("Товары", МетаданныеДокумента) Тогда
         Для каждого СтрокаТЧ Из Объект.Товары Цикл
            // РИ ++
            //ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов, СтрокаТЧ.Номенклатура);
            Если ЗначениеЗаполнено(СтрокаТЧ.Номенклатура) Тогда
               ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов, СтрокаТЧ.Номенклатура);
            КонецЕсли;
            // РИ --
         КонецЦикла;
      ИначеЕсли ОбщегоНазначения.ЕстьТабЧастьДокумента("Заказы", МетаданныеДокумента) Тогда
         Для каждого Заказ Из Объект.Заказы Цикл
            // РИ ++
            //Для каждого СтрокаТЧ Из Заказ.ЗаказПокупателя.Товары Цикл
            //   ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов, СтрокаТЧ.Номенклатура);
            //КонецЦикла;
            Если МетаданныеДокумента.Имя = "ЗакрытиеВнутреннихЗаказов" Тогда
               ИмяРеквизитаТЧ = "ВнутреннийЗаказ";
            Иначе
               ИмяРеквизитаТЧ = "ЗаказПокупателя";
            КонецЕсли;
            Для каждого СтрокаТЧ Из Заказ[ИмяРеквизитаТЧ].Товары Цикл
               Если ЗначениеЗаполнено(СтрокаТЧ.Номенклатура) Тогда
                  ПланыОбмена.ЗарегистрироватьИзменения(МассивУзлов, СтрокаТЧ.Номенклатура);
               КонецЕсли;
            КонецЦикла;
            // РИ --
         КонецЦикла;
      КонецЕсли;

То все удаляется, но опять с ошибками.

Александр Денисюк

Администратор

Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013

#14

30.01.2014 21:21:45

Это все из за конфигурации. Модуль для КА делался на основе УТ. А конфигурации немного отличаются. Всех отличий не учтешь. Посмотрю в чем дело. Оказалось, что в КА(да и скорее всего УПП), нет функции ОбщегоНазначения.ЕстьТабЧастьДокумента.

Пользователь 238860

Заглянувший

Сообщений: 4 Регистрация: 29.01.2014

#15

31.01.2014 11:02:18

Цитата
Александр Денисюк пишет: Модуль для КА делался на основе УТ

И тут понеслось. Обмен заказами не обновляет заказы.

Код

{ОбщийМодуль.РИ_ПроцедурыОбменаССайтомБитрикс.Модуль(1786)}: Ошибка при вызове метода контекста (ВыполнитьПакет)
   МассивРезультатов = Запрос.ВыполнитьПакет();
по причине:
Ошибка выполнения запроса
по причине:
В данной транзакции уже происходили ошибки!

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#16 31.01.2014 11:17:47 Вполне вероятно - связано с предыдущей ошибкой. Посмотрю. Сегодня думаю поправлю.

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#17 31.01.2014 11:50:23 Тут можно скачать новую версию КА. На Сайте она обновится чуть позже. Проверил подписки на события + перезапись заказов. http://yadi.sk/d/iZGB0A6fH47cC

Пользователь 25773

Эксперт

Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008

#18

31.01.2014 16:28:37

исправленная версия для 10.3.26.4 http://1c.1c-bitrix.ru/ecommerce/download.php

Версия с ошибками версия была для 10.3.26.2.

Мы не смогли у 1С найти 10.3.26.4. Её не бывает. Следующая после 10.3.26.2 идет 10.3.27.1

Будет ли работать дополнение и оказываться техническая поддержка если мы поставим дополнение для 10.3.26.4 на 10.3.26.2?

Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Александр Денисюк Администратор Сообщений: 6454 Баллов: 1107 Регистрация: 16.10.2013	#19 31.01.2014 19:29:36 Эм. Это версия модуля. При запуске установщика пишется для какой она версии. Если память не изменяет 10.3.26.2.

Пользователь 25773 Эксперт Сообщений: 866 Баллов: 153 Регистрация: 16.06.2008	#20 31.01.2014 20:03:09 Да верно, в коде у вас 2.9.26.4 для УТ 10.3.26.2. Исправьте на странице http://1c.1c-bitrix.ru/ecommerce/download.php название версии. Это ошибка. Мы заставили заказчика искать несуществующую версию 1С. Модуль «Продвинутый обмен с 1C» http://marketplace.1c-bitrix.ru/solutions/askaron.pro1c/, создание сайтов и интеграция с 1С http://askaron.ru,

Ошибочный порядок записи XML

Продукты

Управление сайтом

Битрикс24

Интернет-магазин + CRM

Решения

Для интернет-магазинов

Каталог готовых решений

Внедрение

Выбрать партнера

Проверить партнера

Стать партнером