Fouladgar\MobileVerification\Notifications\Channels\VerificationChannel

Complexity

Total Complexity

3

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	send()	0	10	2

<?php

namespace Fouladgar\MobileVerification\Notifications\Channels;

use Fouladgar\MobileVerification\Contracts\SMSClient;
use Fouladgar\MobileVerification\Notifications\Messages\MobileVerificationMessage;
use Illuminate\Notifications\Notification;

class VerificationChannel
{
    /**
     * SMSClient (i.e. Nexmo).
     *
     * @var SMSClient
     */
    protected $SMSClient;

    /**
     * VerificationChannel constructor.
     *
     * @param SMSClient $SMSClient
     */
    public function __construct(SMSClient $SMSClient)
    {
        $this->SMSClient = $SMSClient;
    }

    /**
     * Send the given notification.
     *
     * @param $notifiable
     * @param Notification $notification
     *
     * @return mixed|void
     */
    public function send($notifiable, Notification $notification)
    {
        if (! $notifiable->routeNotificationFor('verification_mobile', $notification)) {
            return;
        }

        /** @var MobileVerificationMessage $message */
        $message = $notification->toMobile($notifiable);

The method toMobile() does not exist on Illuminate\Notifications\Notification. It seems like you code against a sub-type of Illuminate\Notifications\Notification such as Fouladgar\MobileVerification\Notifications\VerifyMobile.

        return $this->SMSClient->sendMessage($message->getPayload());
    }
}