Good afternoon.
 At the working site, and connected commerce commerceml, put to the test assembly
rucommerce to understand. Connect both the website to different accounts on "My store" In both cases, every time you sync maps goods (importer Product Nodes) are
added all the fields in the tab "Mapping". After some time on the tab "Mapping" is
simply impossible to go is not enough memory (php fatal error). What could be the
reason? matter commerceml, xpathparser or "My store"?

Добрый день.
На рабочий сайт подключил commerce и commerceml, на тестовый поставил сборку rucommerce чтобы разобраться. Подключил оба сайта к разным аккаунтам на "мой склад". В обоих случаях каждый раз после синхронизации отображений товаров(импортер Product Nodes ) добавляются все поля во вкладке "Mapping". через некоторое время на вкладку "Mapping" уже просто невозможно зайти, не хватает памяти (php fatal error). В чем может быть причина? дело в commerceml, xpathparser или "моем складе"?

CommentFileSizeAuthor
#12 feeds_config_change_rework-2601128-12.patch2.84 KBsin
#7 every_time_you_sync-2601128-7.patch724 bytesnikolay shapovalov
#6 2015-12-13 15_10_11.png15.3 KBnikolay shapovalov
#6 every_time_you_sync-2601128-6.patch2.04 KBnikolay shapovalov
2015-10-26-104431_1440x900_scrot.png83.19 KBkorzh-nick

Comments

kolebas created an issue. See original summary.

sin’s picture

sin commented

Hello, there is a bug in commerceml module field creation code. I think it was introduced by recent Feeds API change.

You may disable fields creation unchecking "Create product node properties" module option. You may revert Feeds feature in RU Commerce, delete and recreate broken Feed on your site.

I plan to rewrite fields synchronization and fix this problem for our next CommerceML exchange deployment with fields sync requirements.

korzh-nick’s picture

korzh-nick commented

thank

korzh-nick’s picture

korzh-nick commented

It did not help while using a terrible crutch. Run by the cron:
(не силен в английском)
в смысле ваш совет не помог, использую жуткий костыль - запускаю по крону:

drush features-revert product_nodes_feed_feature.feeds_importer
nikolay shapovalov’s picture

nikolay shapovalov commented

Та же беда, после импорта в 1С, получаю дубли, "Create product node properties" отключен.

nikolay shapovalov’s picture

nikolay shapovalov commented
Status: Active » Needs review
StatusFileSize
new every_time_you_sync-2601128-6.patch2.04 KB
new 2015-12-13 15_10_11.png15.3 KB

Добавил возможность отключать обновление настроек feeds importer.
На странице /admin/commerce/config/commerceml
Новый интерфейс настроек
Снимаем галочку и всё должно работать.

Только важный момент, если у вас включен Create product node properties, то словари созданы будут, а вот поля нет. Их надо делать руками.
Или как вариант, первый импорт из 1С делаем с включенными Create product node properties и Update importer settings, создатутся поля и словари. А потом эти две опции отключаем. Тестируейте патч.

nikolay shapovalov’s picture

nikolay shapovalov commented
StatusFileSize
new every_time_you_sync-2601128-7.patch724 bytes

Я подумал, наверное нет смысла в еще одной настройке, так что переделал патч.
Если отключен Create product node properties, то тогда обновляться feeds не будет.

nikolay shapovalov’s picture

nikolay shapovalov commented

Проверил патч №7 при отключенном Create product node properties новые mapping не создались.

sin’s picture

sin commented

Честно говоря, так и задумано было конечно, при отключенном Create product node properties не должны импортеры обновляться. Спасибо за патч, гляну в ближайшее время.

sin’s picture

sin commented
Status: Needs review » Needs work

_commerceml_update_importer_settings() is called to disable nodes unpublishing if we have only difference in commerceml and not full catalog. It must be done independent of fields creation. We need a fix it saving feed settings properly with current feeds release.

nikolay shapovalov’s picture

nikolay shapovalov commented

Я думаю, что модуль кроме русскоязычных пользователей, вряд ли пригодиться кому-то, давайте дальше на русском вести обсуждение.

Сергей, спасибо за модуль и за сборку. Удалось за 3 дня настроить интеграцию с 1С, благо заказчик подвернулся с такой задачей.
Планирую в конце недели заняться этим issue.
Как только начну над ним работать, поставлю себя ответственным.

sin’s picture

sin commented
Status: Needs work » Needs review
StatusFileSize
new feeds_config_change_rework-2601128-12.patch2.84 KB

Поправил перекрытие конфигурации Feeds для нодов на лету при импорте только изменений чтобы всегда пропускались не существующие в выгрузке (не измененнные) ноды. При полном импорте теперь точно должно работать нормально, а при импорте только изменений надо проверить. Нужно потестировать.

  • sin committed b0e5313 on 7.x-1.x 
    Issue #2601128 by kolebas, zniki.ru, sin: Fix feeds configuration...
sin’s picture

sin commented
Status: Needs review » Patch (to be ported)

Needs a port to 6.x