Skip to content

Класс NotificationCanceled ничего не обрабатывает #56

Open
linuxoid opened this issue Feb 9, 2022 · 5 comments

Comments

@linuxoid
Copy link

linuxoid commented Feb 9, 2022

$payment = $app->make(YooKassaNotificationCanceled::class, ['source' => $request]);
$payment->getType() - ничего не возвращает
$payment->getObject() - ничего не возвращает

т.к. в классе нет конструктора. Заменив public function fromArray($source) на public function __construct(array $source) решило проблему.

@ostulov
Copy link

ostulov commented Feb 9, 2022

Добрый день!

В нашем SDK нет класса YooKassaNotificationCanceled. Возможно Вы используете другое SDK или бандл для Laravel.

@linuxoid
Copy link
Author

linuxoid commented Feb 9, 2022

Прошу прощения, скопировал и вставил не глядя. Я переименовал этот класс:

use YooKassa\Model\Notification\NotificationCanceled as YooKassaNotificationCanceled;

Естественно я имел в виду класс NotificationCanceled и использую ваш SDK:
https://github.com/yoomoney/yookassa-sdk-php/blob/master/lib/Model/Notification/NotificationCanceled.php

@ostulov
Copy link

ostulov commented Feb 10, 2022

Чтобы использовать все возможности, которые дает создание объектов в Laravel, необходимо предварительно зарегистрировать в контейнере объект (в данном случае NotificationCanceled) с нужными аргументами через new.

@linuxoid
Copy link
Author

linuxoid commented Feb 10, 2022

А в чем причина, что в NotificationSucceeded и NotificationRefundSucceeded есть конструктор, а в NotificationCanceled нет? С точки зрения последовательности.

@ostulov
Copy link

ostulov commented Feb 10, 2022

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants