SquareetLabs\LaravelSmsUp\SmsUpChannel::send()

Complexity

Conditions	3
Paths	3

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    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));
    }