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

Question

[Анатолий Теорский]

Добрый день.

Не вызывается обработчик события, зарегистрированный в своем модуле в файле ../install/index.php

RegisterModuleDependences('sale', 'OnSaleOrderSaved', $this->MODULE_ID, '\\Yolo\\Bonus\\EventHandler', 'myFunction');

тут все ок, он зарегистрировался, я так уже с другими делал)

Теперь в файле ../lib/Eventhandler.php

<?php
namespace Yolo\Bonus;
require_once($_SERVER['DOCUMENT_ROOT'] . '/bitrix/modules/main/include/prolog_before.php');

$module_id = "yolo.bonus";
\CModule::IncludeModule('main');
\CModule::IncludeModule('sale');
\CModule::IncludeModule($module_id);

use Bitrix\Main;
use Bitrix\Main\Loader;
use Bitrix\Main\Localization\Loc;
use Bitrix\Main\Application;
use Bitrix\Main\Type\DateTime;
use Yolo\Bonus;
use Yolo\Bonus\Core;
use Bitrix\Sale;
use Bitrix\Sale\Order;
use Bitrix\Sale\Basket;
use Bitrix\Sale\Delivery;
use Bitrix\Sale\PaySystem;
class EventHandler{
  public function OnSaleStatusOrder($order_id, $status_val){
	// ...	
  }
	
  public function myFunction(\Bitrix\Main\Event $event)
{
    /** @var Order $order */
    $order = $event->getParameter("ENTITY");
    $oldValues = $event->getParameter("VALUES");
    $isNew = $event->getParameter("IS_NEW");

    if ($isNew)
    {
        $sum = $order->getPrice();
        // . . . 
    }
}
}
?>

В итоге выходит ошибка :

Argument 1 passed to Yolo\Bonus\EventHandler::myFunction() must be an instance of Bitrix\Main\Event, instance of Bitrix\Sale\Order given (0)

Я так понимаю, что в функцию передается объект Bitrix\Sale\Order а не Bitrix\Main\Event

Я уже подключил все что можно, и все равно не понимаю в чем дело.
На самом деле я только только начал писать модуль и возможно много чего не понимаю еще, но вроде все было логично до этого момента.
В документации показано что должен передаваться Bitrix\Main\Event, а по факту передается Bitrix\Sale\Order...

Подскажите пожалуйста.

Answer 1

[PetrPo]

Пришлось потупить немного...

Если зарегистрируешь так событие, твой вариант будет ок работать

$eventManager = \Bitrix\Main\EventManager::getInstance();
$eventManager->registerEventHandler('sale', 'OnSaleOrderSaved', $this->MODULE_ID, '\\Yolo\\Bonus\\EventHandler', 'myFunction');

P.S. не забудь удалить свое событие из базы и почистить кеш

P.P.S. в доках на счет объекта \Bitrix\Main\Event $event все верно написано, просто замес старого и нового апи получился)) Таже херня получится между AddEventHandler и $eventManager->addEventHandler

Comments

[PetrPo]

Если интересно, я дальше покопался в чем же разница, расскажу на примере AddEventHandler и $eventManager->addEventHandler (условимся что $eventManager = \Bitrix\Main\EventManager::getInstance())

1. в AddEventHandler вызывается $eventManager->addEventHandlerCompatible


2. как в $eventManager->addEventHandlerCompatible так и в $eventManager->addEventHandler вызывается $this->addEventHandlerInternal где разница только в последнем параметре, который называется $version и равен 1 и 2 соответственно

3. дальше метод send --> sendToEventHandler и видим такую картину


Вот таким нехитрым способом битриксоиды закостылили переход от старой версии к новой))

Answer 2

[Евгений]

В документации сказано "объект заказа", в исходниках если посмотреть, то тоже увидим что, передается класс Order. Соответственно поменяйте тип параметра. Ошибка в документации есть, а конкретно в приведенном примере.


Вот вызов события

$event = new Main\Event('sale', EventActions::EVENT_ON_ORDER_SAVED, array(
                'ENTITY' => $this,
                'IS_NEW' => $this->isNew,
                'IS_CHANGED' => $this->isChanged(),
                'VALUES' => $this->fields->getOriginalValues(),
            ));

И как выше чуть написали - параметры объявляйте все, сколько передается, static писать тоже, это не фатальные ошибки, но так не правильно.

Comments

[Анатолий Теорский]

Да, я поменял формат объекта, а вы тогда не подскажите как мне можно проверить на то что это только что оформленный заказ?
По сути мне нужно событие которое внесет изменения сразу же после оформления заказа.

[Евгений]

Анатолий Теорский, так там вот как раз второй параметр, который показывает новый заказ или нет

Answer 3

[Андрей Николаев]

Во первых, такие методы стоит помечать как static, чтобы не вызывать ошибки самого php.
Во вторых, есть описание событий - https://dev.1c-bitrix.ru/api_d7/bitrix/sale/events... в которых сказано что обработчик события принимает 3 параметра.

Comments

[Анатолий Теорский]

На счет static - учту
На счет 3 параметра, разница не большая, что я приму один, что 3 параметра.
Смысл в том что в теории он должен принять объект Bitrix\Main\Event а принимает Bitrix\Sale\Order.
Я конечно могу работать и с этим объектом, но мне нужно отлавливать именно новый заказ - а значит мне нужна переменная $isNew или любая другая проверка что это новый заказ

[Андрей Николаев]

Анатолий Теорский, вовсе нет. Это особый тип события, который совершенно не обязательно должен следовать общему поведению. В документации явно указано какой тип параметров он принимает.
Более того, это будет не одна входная переменная, а целых 3.