fomvasss / laravel-notification-channel-sms-ru

src/SmsRuChannel.php

<?php

namespace NotificationChannels\SmsRu;

use Illuminate\Events\Dispatcher;

The type Illuminate\Events\Dispatcher was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Notifications\Events\NotificationFailed;
use NotificationChannels\SmsRu\Exceptions\CouldNotSendNotification;
use Illuminate\Notifications\Notification;

class SmsRuChannel
{
    /**
     * @var SmsRuApi
     */
    protected $smsApi;

    /**
     * @var Dispatcher
     */
    protected $events;

    /**
     * SmsRuChannel constructor.
     * @param SmsRuApi $smsApi
     * @param Dispatcher $events
     */
    public function __construct(SmsRuApi $smsApi, Dispatcher $events)
    {
        $this->smsApi = $smsApi;
        $this->events = $events;
    }
    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  Notification  $notification
     *
     * @throws CouldNotSendNotification
     *
     * @return array|null
     */
    public function send($notifiable, Notification $notification)
    {
        try {
            $recipient = $this->getRecipient($notifiable);
            $message = $notification->toSmsru($notifiable);

The method toSmsru() does not exist on Illuminate\Notifications\Notification.

            if (is_string($message)) {
                $message = new SmsRuMessage($message);
            }
            if (! $message instanceof SmsRuMessage) {
                throw CouldNotSendNotification::invalidMessageObject($message);
            }

            return $this->smsApi->sendMessage($recipient, $message);

The expression return $this->smsApi->sendMessage($recipient, $message) returns the type Zelenin\SmsRu\Response\SmsResponse which is incompatible with the documented return type array|null.

        } catch (Exception $exception) {
            $event = new NotificationFailed($notifiable, $notification, 'smsru', ['message' => $exception->getMessage(), 'exception' => $exception]);

            if (function_exists('event')) { // Use event helper when possible to add Lumen support
                event($event);
            } else {
                $this->events->fire($event);
            }
        }
    }

    protected function getRecipient($notifiable)
    {
        if ($notifiable->routeNotificationFor('smsru')) {
            return $notifiable->routeNotificationFor('smsru');
        }

        if (isset($notifiable->phone)) {
            return $notifiable->phone;
        }

        throw CouldNotSendNotification::invalidReceiver();
    }
}