Как пересчитать заказ с учетом всех скидок с использованием события OnSaleOrderSaved?

Question

[tgarl]

Есть оформление заказа еще на старом API. В битриксе есть такой фукционал как допуслуги.
Мне нужно внести выбор допуслуг в оформление.

Галочки я добавил. На основании этих галочек заполняется свойство заказа. Далее я использую событие OnSaleOrderSaved где проверяю заполненность свойства и если не пустое проставляю стоимость доставки. Но вся беда, что скидки на доставку при этом не учитываются. Если нажать пересчитать заказ, то все встанет на места как нужно, но это не вариант, так как клиенты оплачивают заказ сразу, и менеджеры не вкурсе что заказ не верно посчитан.
вот мой результирующий код на текущий момент и как обойти эту ситуацию не могу понять.

spoiler

//событие OnSaleOrderSaved
public static function OnSaleOrderSavedEnded(Event $event)
    {
        $order = $event->getParameter('ENTITY');
        $isNew = $event->getParameter("IS_NEW");
        if ($isNew) {
         $orderProperties = $order->getPropertyCollection();
         $propertyValue = $orderProperties->getItemByOrderPropertyId(61)->getValue();
         if(intval($propertyValue)>0){
            $shipmentCollection = $order->getShipmentCollection();
            foreach ($shipmentCollection as $shipment)
            {
               if ($shipment->isSystem()) continue;
               $shipment->setExtraServices([$propertyValue=>'Y']);

               $resCalc = \Bitrix\Sale\Delivery\Services\Manager::calculateDeliveryPrice($shipment,$shipment->getField("DELIVERY_ID"));
               if($resCalc->isSuccess()){
                  $arShipments1 = $resCalc->getExtraServicesPrice();
                  $service = \Bitrix\Sale\Delivery\Services\Manager::getById($shipment->getField("DELIVERY_ID"));
                  $deliveryData = [
                     'DELIVERY_ID' => $service['ID'],
                     'DELIVERY_NAME' => $service['NAME'],
                     'ALLOW_DELIVERY' => 'N',
                     'PRICE_DELIVERY' => $arShipments1,
                     'BASE_PRICE_DELIVERY' => $arShipments1,
                     'CUSTOM_PRICE_DELIVERY' => 'N',
                     'CURRENCY' => $order->getCurrency(),
                  ];
                  $shipment->setFields($deliveryData);
               }                                
            }
            $now_order_id=$order->getId();
            $order->save();

            $order_resave = \Bitrix\Sale\Order::load($now_order_id);
            $discount = $order_resave->getDiscount();
            \Bitrix\Sale\DiscountCouponsManager::clearApply(true);
            \Bitrix\Sale\DiscountCouponsManager::useSavedCouponsForApply(true);
            $discount->setOrderRefresh(true);
            $discount->setApplyResult(array());

            if (!($basket = $order_resave->getBasket())) {
               throw new \Bitrix\Main\ObjectNotFoundException('Entity "Basket" not found');
            }

            $basket->refreshData(array('PRICE', 'COUPONS'));
            $discount->calculate();

            $order_resave->setField("PRICE", $order_resave->getPrice());
            $order_resave->save();
         }
      }
}

Comments

[Евгений]

Давно дело было плохо помню что делал, но у меня вот так был вызов купонов.

DiscountCouponsManager::finalApply();
DiscountCouponsManager::saveApplied();

А вы получается очищаете купоны.

[tgarl]

и вызываю пересчет

$basket->refreshData(array('PRICE', 'COUPONS'));
$discount->calculate();

если на отдельной странице вызвать

spoiler

$order_resave = \Bitrix\Sale\Order::load($now_order_id);
            $discount = $order_resave->getDiscount();
            \Bitrix\Sale\DiscountCouponsManager::clearApply(true);
            \Bitrix\Sale\DiscountCouponsManager::useSavedCouponsForApply(true);
            $discount->setOrderRefresh(true);
            $discount->setApplyResult(array());

            if (!($basket = $order_resave->getBasket())) {
               throw new \Bitrix\Main\ObjectNotFoundException('Entity "Basket" not found');
            }

            $basket->refreshData(array('PRICE', 'COUPONS'));
            $discount->calculate();

            $order_resave->setField("PRICE", $order_resave->getPrice());
            $order_resave->save();

то пересчет срабатывает, в событии нет

[Евгений]

tgarl, надо посмотреть в исходниках когда вызывается событие и какой код выполняется после него, чтобы понять что происходит, а то может вам проще кастомизировать оформление заказа, чем работать через событие.

[tgarl]

Оформление заказа старое, в нем нет D7, соответсвенно нет обработок ExtraServices. Как добавить выбор допуслуг на старом API я не нашел. Поэтому остается вариант событий. У события "перед сохранением" нет еще отгрузок, а у после сохранения (OnSaleOrderSaved) не могу пересчитать заказ со всеми скидками.

Answer 1

[Aisamiery]

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

Во-вторых, не совсем понятно что вы тут делаете, так как вы сбрасываете все скидки на заказе, а потом пытаетесь их пересчитать

Comments

[tgarl]

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

[Aisamiery]

Тут надо смотреть выше по коду, видимо ваш заказ сохраняется несколько раз и первый раз когда еще отгрузки нет. Попробуйте пропускать события где нет нужных вам данных и не проверять на IS_NEW а ставить своё служебное поле, что заказ обработан/пересчита или что там ещё и ориентируйтесь на это свойство.

Answer 2

[tgarl]

Я был не прав что отказался от before. Нужно все же на событии до сохранения. Но беда остается, что нет пересчета окончательной цены с учетом скидок на доставку. По логике пересчет должен производится сам, но этого не происходит. Заказ по своей цене, доставка по своей.