Bitrix подмена статуса заказа при изменении заказа?

Question

[Yuriy]

как подменить приходящий статус в событие и сохранить другой?

\Bitrix\Main\EventManager::getInstance()->addEventHandler('sale', 'OnSaleStatusOrderChange', ['Handler', 'OnSaleStatusOrderChange']);

class Handler {
  function OnSaleStatusOrderChange(Bitrix\Main\Event $event)
  {

    $order = $event->getParameter("ENTITY");
    $value = $event->getParameter("VALUE");
    $oldValue = $event->getParameter("OLD_VALUE");

    $paymentCollection = $order->getPaymentCollection();
    $isPaid = $paymentCollection->isPaid();


    if ($value === 'PA' && $isPaid === true) {
     // ??? Меняем статус на нужный нам ???
     }

Answer 1

[Роман Грицук]

Регистрируем обработчик

$bxEventManager = \Bitrix\Main\EventManager::getInstance();
$bxEventManager->addEventHandler(
    "sale",
    "OnSaleOrderBeforeSaved",
    array(
        Gricuk\Sale\Order::class,
        "changeStatusOnOrderCreate"
    )
);

Сам обработчик. У меня в примере логика на смена статуса заказа для определенных служб. Ее можно переписать на что угодно. Основное это $event->addResult

namespace Gricuk\Sale;

class Order
{
    public static function changeStatusOnOrderCreate(\Bitrix\Main\Event $event)
    {
        /** @var \Bitrix\Sale\Order $order */
        $order = $event->getParameter("ENTITY");

        $status = $order->getField("STATUS_ID");
        if ($status == "N") {
            /**  @var $paymentCollection \Bitrix\Sale\PaymentCollection */
            $paymentCollection = $order->getPaymentCollection();

            foreach ($paymentCollection as $payment) {
                /**  @var $payment \Bitrix\Sale\Payment */

                /**  @var $paySystem \Bitrix\Sale\PaySystem\Service */
                $paySystem = $payment->getPaySystem();

                if (!is_null($paySystem) && !$paySystem->isCash()) {
                    $order->setField("STATUS_ID", "NA");

                    $event->addResult(
                        new Main\EventResult(
                            Main\EventResult::SUCCESS, $order
                        )
                    );
                    break;
                }
            }
        }
    }
}

Пример 100% рабочий. Взят с прода.

Comments

[Айнур Валиев]

!$paySystem->isCash()
тут надо убрать отрицание

[Роман Грицук]

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

[Айнур Валиев]

Роман Грицук, с таким условием всем заказам проставляется статус "Оплачено"

[Роман Грицук]

Айнур Валиев, В Вашем проекте возможно так и есть. Повторюсь, это РАБОЧИЙ пример с сайта где работает именно так. Это часть бизнес логики и не имеет отношения к самому принципу изменения заказа на событии.

Answer 2

[PetrPo]

$newStatusId = 'F';
$order->setField('STATUS_ID', $newStatusId);
$order->save();

В доках пример есть Смена статуса

Comments

[Yuriy]

я так пробовал, зависает битрикс...
Мне нужна не смена статуса, а подмена входящих параметров.

[PetrPo]

Yuriy, я только что так пробовал и ничего не зависает, ищи ошибку в коде до этого. Все закомментируй и оставь только четыре строки и увидишь, что все работает

$order = $event->getParameter("ENTITY");
$newStatusId = 'F';
$order->setField('STATUS_ID', $newStatusId);
$order->save();

[Yuriy]

PetrPo, я же говорю, мне нужно не менять статус, а подменить входящие данные

[PetrPo]

Yuriy, в смысле?

[Yuriy]

PetrPo, так висит

\Bitrix\Main\EventManager::getInstance()->addEventHandler('sale', 'OnSaleStatusOrderChange', ['Handler', 'OnSaleStatusOrderChange']);

class Handler {
  function OnSaleStatusOrderChange(Bitrix\Main\Event $event)
  {

      $order->setField('STATUS_ID', 'PQ');
      $order->save();
  }

[PetrPo]

Yuriy, да ладно кэп. А где получение $order
$order = $event->getParameter("ENTITY");

[Yuriy]

PetrPo, вот полная версия

\Bitrix\Main\EventManager::getInstance()->addEventHandler('sale', 'OnSaleStatusOrderChange', ['Handler', 'OnSaleStatusOrderChange']);

class Handler {
  function OnSaleStatusOrderChange(Bitrix\Main\Event $event)
  {
    /** @var \Bitrix\Sale\Order $order */
    $order = $event->getParameter("ENTITY");
    $value = $event->getParameter("VALUE");
    $oldValue = $event->getParameter("OLD_VALUE");
    $paymentCollection = $order->getPaymentCollection();
    $isPaid = $paymentCollection->isPaid();


    if ($value === 'PA' && $isPaid === true) {
      $order->setField('STATUS_ID', 'PQ');
      $order->save();

    }

[PetrPo]

Yuriy, вопрос как звучит? как подменить приходящий статус в событие и сохранить другой?
Я тебе написал как это сделать, повторю еще раз. Оставь только эти строки в событии:

$order = $event->getParameter("ENTITY");
$order->setField('STATUS_ID', 'PQ');
$order->save();

Измени статус и увидишь что все работает. Если это так, то проблема в остальной твоей писанине, я в нее не вникал и дал ответ на поставленный вопрос...

[Yuriy]

PetrPo, висит при

$order->setField('STATUS_ID', 'PQ');
$order->save();

[PetrPo]

Yuriy, ну давай тогда в угадайку поиграем, почему у тебя не работает. Открой в консоли network и посмотри, что в ответе приходит и на что ругается

[Yuriy]

PetrPo, ты понимаешь что такое зависание?? какой может быть ответ если всё висит... срабатывает timeout 504 getway.. в логе тоже нет ошибок ни в битриксе ни в nginx

я уже сделал что мне нужно, немного по другому

[PetrPo]

Yuriy, молодец, напиши как сделал в ответ и отметь решением

[Yuriy]

PetrPo, всё как есть убрал save() и событие изменил на OnSaleOrderBeforeSaved

Answer 3

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

Не следует подменять статус на событии OnSaleStatusOrderChange, так как это событие вызывается при смене статуса. Лучше отследить нужное событие перед сохранением заказа и уже в нем изменить.

Comments

[Yuriy]

мне и нужно при смене статуса

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

Yuriy, вот тут и получается тонкость: при смене статуса вы хотите сменить статус тем самым породить событие смены статуса... В общем это не лучшая идея и Роман Грицук уже подсказал код для смены статуса "перед сохранением заказа", которое я и предложил.