Question

Здравствуйте,
проблема такая: я создал обработчик который при добавлении элемента в любой ИБ создает элемент в специально созданном ИБ. Но почему-то при добавлении элемента - во втором ИБ начинает создаваться бесконечное множество элементов пока не нажать "отменить"..

Вот код в init.php:

Код

<?// регистрируем обработчик AddEventHandler("iblock", "OnAfterIBlockElementAdd", Array("MyClass", "OnAfterIBlockElementAddHandler")); class MyClass {     // создаем обработчик события "OnAfterIBlockElementAdd"     function OnAfterIBlockElementAddHandler(&$arFields)     {                        if (CModule::IncludeModule("iblock"))                          { $el = new CIBlockElement; $arLoadProductArray = Array(   "IBLOCK_SECTION_ID" => 0,          // элемент лежит в корне раздела     "IBLOCK_ID"      => 20,     "NAME"           => "Элемент",     "ACTIVE"         => "Y",            // активен     "PREVIEW_TEXT"   => "текст для списка элементов",     "DETAIL_TEXT"    => "текст для детального просмотра"   ); $PRODUCT_ID = $el->Add($arLoadProductArray, true);    }   } } ?>

Подскажите где я что не так сделал(

Answer 1

Традионно принято валить все такое на модуль документооборота. :D

Answer 2

Функция CIBlockElement:Add() так же вызывает событие OnAfterIBlockElementAdd - у вас получается бесконечная рекурсия. Достаточно модифицировать первый ИФ до вида

Код
if($arFields["ID"] > 0 && $arFields["IBLOCK_ID"] != 20)

ИД ваше "специально созданного инфоблока" равен 20, исходя из кода добавления элемента, значит логично предположить, что в него его-же елементы дублировать не нужно, поэтому добавляем условие $arFields["IBLOCK_ID"] != 20 и рекурсии больше не будет.
Использовать в условии if (CModule::IncludeModule("iblock")) глупо, потомучто раз вызывается событие, значит модуль уже включен. Нужно проверять на $arFields["ID"]>0, поскольку событие вызывается даже тогда, когда произошла ошибка и элемент не добавлен: если элемент добавлен в $arFields["ID"] будет его ИД, а если не добавился то будет 0.

Answer 3

Цитата

Иван Курза пишет: Функция CIBlockElement:Add() так же вызывает событие OnAfterIBlockElementAdd - у вас получается бесконечная рекурсия. Достаточно модифицировать первый ИФ до вида Код if($arFields["ID"] > 0 && $arFields["IBLOCK_ID"] != 20) ИД ваше "специально созданного инфоблока" равен 20, исходя из кода добавления элемента, значит логично предположить, что в него его-же елементы дублировать не нужно, поэтому добавляем условие $arFields["IBLOCK_ID"] != 20 и рекурсии больше не будет. Использовать в условии if (CModule::IncludeModule("iblock")) глупо, потомучто раз вызывается событие, значит модуль уже включен. Нужно проверять на $arFields["ID"]>0, поскольку событие вызывается даже тогда, когда произошла ошибка и элемент не добавлен: если элемент добавлен в $arFields["ID"] будет его ИД, а если не добавился то будет 0.

Спасибо, тоже дошел до такого же решения =)