yoomoney / cms-drupal8

YooKassa payment module for Drupal 8.x
https://yookassa.ru/developers
3 stars 6 forks source link

Ну нужно дублировать в onReturn() действия из onNotify() #32

Closed superromeo closed 2 years ago

superromeo commented 2 years ago

Ну нужно дублировать в public function onReturn(OrderInterface $order, Request $request) действия из onNotify(). Так как повторно вызываются события, например, OrderEvents::ORDER_PAID.

ostulov commented 2 years ago

Добрый день!

Вызов события был добавлен по запросу https://github.com/yoomoney/cms-drupal8/issues/19 Можете, пожалуйста, более подробно описать, что сейчас не устраивает в работе модуля?

superromeo commented 2 years ago

Это событие вызывается дважды:

А должно только после оплаты.

ostulov commented 2 years ago

Для определения того, что оплата за заказ прошла успешно, используется два варианта:

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

superromeo commented 2 years ago

Это хорошо. Но Вы можете при возвращении в магазин проверять, были ли уже выполнены действия? И выполнять их только если они не были уже выполнены.

ostulov commented 2 years ago

Да, добавим такую проверку.

superromeo commented 2 years ago

Спасибо!

ostulov commented 2 years ago

Добавлено в последнем обновлении v2.2.8