Битрикс — почему не применяется купон к заказу?

Question

[Николай]

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

В карточке товара есть покупка в 1 клик, открывается popup, заполняется телефон, жмется кнопка и уходят ajax запрос на oneClickBuy.php
Задача, пробить номер телефона по базе скидок, и если он найден, вернуть % скидки.
Далее по этому % применить купон правил корзины.
Встрял на применении купона.
Как только я не танцевал с кодом, ничего не получалось.
В итоге решил вынести для более удобной отладки в отдельный файл со статичным id заказа и всё сразу заработало...
Вот код файла, в котором применяется купон с кодом pl_10 применяется к заказу 13233

require_once($_SERVER['DOCUMENT_ROOT'] . '/bitrix/modules/main/include/prolog_before.php');
use Bitrix\Main\Loader;
use Bitrix\Sale\DiscountCouponsManager;
use Bitrix\Sale;

$orderId = 13233;
$coupon = "pl_10";

if(isset($coupon)){
    $couponinfo = DiscountCouponsManager::getData($coupon, true);
    if ($couponinfo['ACTIVE'] == 'Y') {
        $discountOrder = Sale\Order::load($orderId);
        Sale\DiscountCouponsManager::add($coupon);
        $discounts = $discountOrder->getDiscount();
        $discounts->calculate();
        $discountOrder->refreshData();
        $discountOrder->save();
    }
}

Вставляю этот код в oneclickBuy.php почти в самый конец, перед

$arResult["ORDER_ID"] = $orderId;
die(\Bitrix\Main\Web\Json::encode($arResult));

И купон не применяется к заказу.
Никаких ошибок нет, просто стоимость в заказе остается прежней.

if(isset($coupon)){
    $file = "/@del/coupon.log";
    Diag\Debug::writeToFile("coupon ".$coupon." found", "", $file);
    $couponinfo = DiscountCouponsManager::getData($coupon, true);
    if ($couponinfo['ACTIVE'] == 'Y') {
        Diag\Debug::writeToFile("coupon ".$coupon." active", "", $file);
        $discountOrder = Sale\Order::load($orderId);

        Sale\DiscountCouponsManager::add($coupon);
        $discounts = $discountOrder->getDiscount();
        $discounts->calculate();
        $discountOrder->refreshData();
        $discountOrder->save();
        Diag\Debug::writeToFile("coupon ".$coupon." for order id ".$orderId." saved", "", $file);
        Diag\Debug::writeToFile("errors: ".print_r(Sale\DiscountCouponsManager::getErrors(),true), "", $file);
    }
    Diag\Debug::writeToFile("done", "", $file);
}

coupon pl_10 found
coupon pl_10 active
coupon pl_10 for order id 13233 saved
errors: Array
(
)
done

Т.е. все переменные подставляются, ошибок нет...
Как так?
Куда копать?

Comments

[Александр Маджугин]

Так купон не применяется или стоимость остается прежней?
Если получить список купонов после применения:

$lstCoupons = Sale\DiscountCouponsManager::get(false, [], true, false);

Купон есть в списке?

[Николай]

Александр Маджугин,

Так купон не применяется или стоимость остается прежней?

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

Купон есть в списке?

Да есть,

coupons list: Array
(
[0] => pl_10
)

При чем он есть в списке и до применения купона например тут

if(isset($coupon)){

    $file = "/@del/coupon.log";
    $lstCoupons = Sale\DiscountCouponsManager::get(false, [], true, false);
    Diag\Debug::writeToFile("coupons list: ".print_r($lstCoupons, true), "", $file);
    $couponinfo = DiscountCouponsManager::getData($coupon, true);
    if ($couponinfo['ACTIVE'] == 'Y') {

        Diag\Debug::writeToFile("coupon ".$coupon." active", "", $file);
        $discountOrder = Sale\Order::load($orderId);

        Sale\DiscountCouponsManager::add($coupon);
        $discounts = $discountOrder->getDiscount();
        $discounts->calculate();
        $discountOrder->refreshData();
        $discountOrder->save();
    }
}

[Александр Маджугин]

Значит он применен. А то что цена не меняется может быть следствием что не проходит по условию скидка. Сделайте скидку с предельно простым условием - 10% скажем на все для веех и проверьте.

Я сейчас посмотрел на одно работающем проекте контроллер применяющий купон выглядит так:

/*
     * проверяет есть ли товар с $productid в корзине и возвращает его количество
    */
    public function applyCouponAction(string $coupon): array
    {

        $lstCoupons = Sale\DiscountCouponsManager::get(false, [], true, false);
        $arDebug = ['$lstCoupons' => $lstCoupons];
        // получим купоны и проверим нет ли этого купона среди примененных
        foreach ($lstCoupons as $ecoupon) {
            if ($ecoupon == $coupon) {
                $coupon = false;
                break;
            }
        }
        
        if ($coupon) {
            $couponChanged = Sale\DiscountCouponsManager::add($coupon);
            if (!$couponChanged) Sale\DiscountCouponsManager::delete($coupon);
            $arDebug['$couponChanged'] = $couponChanged;
            
        }
        
        
        $arResponce = $this->getAction();
        if (APPLICATION_ENV == 'dev') $arResponce['debug'] = $arDebug;
        return $arResponce;
    }
    #

Вот прям работающий код.

[Николай]

Александр Маджугин,
При чем тут условия?
Я же пишу что в отдельном файле этот код работает, при переносе в oneClickBuy.php перестает
По нему применяется купон и уменьшается цена, в админке у заказа видно примененный купон.
Это та же база, тот же купон, тот же товар, тот же заказ.
Этот код перестаёт работать когда попадает в файл обработки заказа по 1 клику.
Т.е. тут работает https://onlinephp.io/c/9a790
тут не работает https://onlinephp.io/c/a382b

Что-то в этом файле мешает применению купона, вопрос что и почему нет никаких ошибок...

[Александр Маджугин]

Николай, тогда ищете разницу в среде где код выполняется. Вижу что логируете... логируйте подробнее.

Вот правда - если бы передо мной стояла задача с этим разобраться, я бы начал с того, что уменьшил число строк в скрипте хотя бы до 100. А лучше до 50-60. Потому что если вам кажется что вы понимаете кусок кода больше 100 строк, то вам кажется и вы даже не понимаете что не понимаете этот кусок.

[tgarl]

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

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

[Николай]

tgarl, упрощать правила некуда

Ссылка удалена модератором.

Нарушение п.3.9 Регламента.

Кроме того я уже неоднократно писал что купон работает если применять его из отдельного файла.
Мне очистка буфера не нужна, этот код писался до меня, у меня задача добавить скидку по номеру телефона.
Тем не менее $APPLICATION->RestartBuffer() ни при чем, если его убрать ничего не меняется.

[tgarl]

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

[Николай]

tgarl, ограничений нет, выбраны все группы пользователей

Ссылка удалена модератором.

Нарушение п.3.9 Регламента.

Дело точно не в настройках, всё активно, без ограничений, без сроков, это касается и правил и их купонов.

[tgarl]

попробуйте код немного изменить

$discountOrder = Sale\Order::load($orderId);

        Sale\DiscountCouponsManager::add($coupon);
        $discounts = $discountOrder->getDiscount();
		$discounts->setOrderRefresh(true);
        $discounts->calculate();
        //$discountOrder->refreshData();
		$basketOrder = $discountOrder->getBasket();
		$basketOrder->refreshData(["PRICE", "COUPONS"]);
		$discountOrder->doFinalAction(true);
        $discountOrder->save();

[Николай]

tgarl, я пробовал и этот вариант, пробовал делать DiscountCouponsManager::init() внутри onecklickbuy отказывается работать хоть тресни.
Ваш код я на всякий случай конечно скопипастил и проверил, без изменений.

В переписке с Александром я писал что

$lstCoupons = Sale\DiscountCouponsManager::get(false, [], true, false);

возвращает купон еще до применения.
Поковырявшись понял что купон также применяется при проверке номера телефона через ajax на форме 1 клик.
Думал может внутри битрикса какая-то транзакция открывается убрал применения, принудительно сделал
unset($_SESSION["CATALOG_USER_COUPONS"]);
И в качестве "на всякий"

DiscountCouponsManager::delete($coupon);
DiscountCouponsManager::clear(true);
DiscountCouponsManager::clearApply(true);

К сожалению не помогло.

$lstCoupons = Sale\DiscountCouponsManager::get(false, [], true, false);

теперь пустой, но купон так и не применяется...

[Модератор]

Николай, Когда вы регистрировались, вы обещали выполнять Регламент. Ваши сообщения нарушили п.3.9 Регламента.
Если вы будете продолжать нарушать Регламент, ваши вопросы могут быть удалены, а аккаунт заблокирован модераторами временно или навсегда.
Будьте благоразумны.