Почему не работает событие в битрикс?

Question

[Anumik]

Добрый день
Стоит задача изменять цену при выгрузке, в init.php добавил такое событие

AddEventHandler("catalog", "OnBeforePriceUpdate", array("MyClass", "OnBeforePriceUpdateHandler"));
class MyClass {
   function OnBeforePriceUpdateHandler($PRICE_ID, $arFields)
      {
         $NewPrice = 5000;
         $arPrice = array(
            "PRODUCT_ID" => $arFields['PRODUCT_ID'],
            "CATALOG_GROUP_ID" => 1,
            "PRICE" => $NewPrice,
            "CURRENCY" => "RUB"
         );
         CPrice::Update($PRICE_ID, $arPrice);
      }
}

но оно почему то вызывает 500 ошибку
Так же пробовал такое событие:

AddEventHandler("catalog", "OnBeforePriceUpdate", array("MyClass", "OnBeforePriceUpdateHandler"));
class MyClass
{
   function OnBeforePriceUpdateHandler($PRICE_ID, $arFields)
   {
      $db_price = CPrice::GetList(
         array(),
         array(
            "ID" => $PRICE_ID
         )
      );
      if($ar_price = $db_price->Fetch())
      {
         if($ar_price['PRICE'] != $arFields['PRICE'])
         {
            $arFields['PRICE']=$arFields['PRICE']/100*110;
         }
      }
   }
}

но оно вообще не сработало

Answer 1

[Олег Максименко]

В первом случае вы получаете зацикливание вызовов обработчиков.

Comments

[Anumik]

А как его убрать?

[Anumik]

AddEventHandler("catalog", "OnBeforePriceUpdate", array("MyClass", "OnBeforePriceUpdateHandler"));
class MyClass {
   protected static $handlerDisallow = false;
   function OnBeforePriceUpdateHandler($PRICE_ID, $arFields)
      {
         if (self::$handlerDisallow)
           return; 
         self::$handlerDisallow = true;
         $NewPrice = 5000;
         $arPrice = array(
            "PRODUCT_ID" => $arFields['PRODUCT_ID'],
            "CATALOG_GROUP_ID" => 1,
            "PRICE" => $NewPrice,
            "CURRENCY" => "RUB"
         );
         CPrice::Update($PRICE_ID, $arPrice);
         self::$handlerDisallow = false;
      }
}

Попытался исправить вот так, но выдало ошибку
`Call to undefined method MyClass::isEnabledHandler()`

[Anumik]

AddEventHandler("catalog", "OnBeforePriceUpdate", array("MyClass", "OnBeforePriceUpdateHandler"));
class MyClass {
   protected static $handlerDisallow = false;
   function OnBeforePriceUpdateHandler($PRICE_ID, $arFields)
      {
         if (self::$handlerDisallow)
         return;
         self::$handlerDisallow = true;
         $NewPrice = 5000;
         $arPrice = array(
            "PRODUCT_ID" => $arFields['PRODUCT_ID'],
            "CATALOG_GROUP_ID" => 1,
            "PRICE" => $NewPrice,
            "CURRENCY" => "RUB"
         );
         CPrice::Update($PRICE_ID, $arPrice);
         self::$handlerDisallow = false;
      }
}

Вот сейчас код срабатывает без ошибки, но цена не меняется

[Олег Максименко]

Anumik: цена-то меняется, но после перезаписывается штатными средствами. Используйте хотя бы событие OnPriceUpdate.

[Anumik]

Олег Максименко: заменил, все равно не меняется цена

[Олег Максименко]

Anumik: дебаж. пиши в лог все обновления цены вместе с трейсом

Answer 2

[Станислав Харахнин]

Просто измените значение поля и верните это значение.
OnBeforePriceUpdateHandler($PRICE_ID, &$arFields) - писать так, чтобы получить массив по ссылке и изменять его значение, а не работать в другим экземпляром.

Исправьте это и используйте второй вариант