Как спроектировать возврат ДС по заказу на laravel?

Question

[lexstile]

Делаю фичу, где при отмене заказа автоматически возвращаются деньги заказчику.
Причин не делать это автоматически - не нашел.

Сейчас логика работы выглядит так:
1. Отменяем заказ, статус переводится в CANCEL.
2. Далее, так как я подписан на изменение статуса, в слушателе я вызываю метод платежной системы на возврат.

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

Если я это делаю в контроллере, получается грязновато:

public function update(OrderUpdateRequest $request, Project $project, Order $order)
    {
        if (in_array($order->status, [Order::PAID, Order::CLOSE])) {
            if ($request->get('status') === Order::CANCEL && $order->isPaid() && !$order->isReverted() && $order->isNonCashType()) {
                $this->orderService->reverse($order);
            }
        }
        
        $order->fill($request->validated());
        $order->saveOrFail();

        return $this->baseJson(message: ResponseHelper::success(__('orders.messages.update.success')));
    }

Интересует 2 вопроса:
1. Отказаться от автоматического возвращения ДС и делать это по отдельному действию?
2. Если это делать автоматически, каким образом лучше - перед сменой статуса у заказа или после? (в первом случае, если не удастся вернуть деньги, то просто сработает exception и статус уже не поменяется)

Comments

[Дмитрий]

что гуглить

1 ссылка и 3 ссылка

Answer 1

[pLavrenov]

Почему бы не Использовать эвенты? при изменении модели в событии updating проверяем "грязное" ли поле status (это покажет был ли это запрос на изменение статуса) если грязное то обрабатываем в соответствии с требованиями, если получаем неправильный ответ от обработки - устанавливаем нужный статус или тот который был в чистой модели.

Итог - или отработает так как надо или т̶а̶к̶ ̶к̶а̶к̶ ̶н̶а̶д̶о̶ возврат к нужному статусу.

Comments

[lexstile]

Я не понял, то ли вы имели ввиду, я пробовал так:

public function handle(OrderUpdatedEvent $event)
    {
        $changes = collect($event->order->getChanges());

        if ($changes->has('status')) {
            $status = $changes->get('status');

            if ($status === Order::PAID) {
                $event->order->paid_at = now();
                $event->order->saveQuietly();
            }

            if ($status === Order::CANCEL && $event->order->isPaid() && !$event->order->isReverted() && $event->order->isNonCashType()) {
                # При отмене заказа:
                # если заказ был оплачен, не было полного возврата и тип оплаты безналичный
                # делаем возврат денежных средств
                resolve(OrderService::class)->reverse($event->order);
            }
        }
    }

[lexstile]

Кажется дошло, можно навесить событие еще до обновления?
У меня сейчас:

protected $dispatchesEvents = [
        'updated' => OrderUpdatedEvent::class,
    ];

А будет так:

protected $dispatchesEvents = [
        'updating' => OrderUpdatedEvent::class,
    ];

UPD: если это так, похоже, это решение моей проблемы

[pLavrenov]

lexstile, да, updating это до обновления, там можно отследить что конкретно меняется в модели.

[lexstile]

pLavrenov, спасибо большое!

[lexstile]

pLavrenov, только не очень понял, как поймать изменяемые поля, после изменения я делал:
$event->order->getChanges()
Но сейчас их ловить как-то иначе нужно.

[pLavrenov]

lexstile, тут почитать можно

[lexstile]

pLavrenov, благодарю, помогло:
getDirty()