squareetlabs / LaravelSmsUp

src/SmsUpChannel.php

<?php

namespace SquareetLabs\LaravelSmsUp;

use Illuminate\Support\Facades\Event;
use SquareetLabs\LaravelSmsUp\Events\SmsUpMessageWasSent;
use SquareetLabs\LaravelSmsUp\Exceptions\CouldNotSendNotification;
use Illuminate\Notifications\Notification;
use SquareetLabs\LaravelSmsUp\Facades;

/**
 * Class SmsUpChannel
 * @package SquareetLabs\LaravelSmsUp
 */
class SmsUpChannel
{
    /**
     * SmsUpChannel constructor.
     */
    public function __construct()
    {
    }

    /**
     * @param $notifiable
     * @param Notification $notification
     * @throws CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        /** @var SmsUpMessage $message */
        $message = $notification->toSmsUp($notifiable);

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

        if (empty($message->getTo())) {
            if (!$to = $notifiable->routeNotificationFor('smsUp')) {
                throw CouldNotSendNotification::missingRecipient();
            }
            $message->to($to);
        }

        $messages = [
            $message->formatData()
        ];
        $response = Facades\SmsUp::sendMessages($messages);

The method sendMessages() does not exist on SquareetLabs\LaravelSmsUp\Facades\SmsUp. Since you implemented __callStatic, consider adding a @method annotation.

        $responseArray = [];
        array_push($responseArray, json_decode($response->getBody(), true));
        $reponseMessage = new SmsUpResponse($responseArray[0]);

        Event::dispatch(new SmsUpMessageWasSent($message, $reponseMessage));
    }
}