NotificationChannels\NetGsm\NetGsmChannel

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
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	13	3

<?php

namespace NotificationChannels\NetGsm;

use Illuminate\Notifications\Notification;

class NetGsmChannel
{
    /** @var \NotificationChannels\NetGsm\NetGsmClient */
    protected $client;

    /**
     * NetGsmChannel constructor.
     * @param  NetGsmClient  $client
     */
    public function __construct(NetGsmClient $client)
    {
        $this->client = $client;
    }

    /**
     * Send the given notification.
     *
     * @param  mixed  $notifiable
     * @param  \Illuminate\Notifications\Notification  $notification
     *
     * @throws \NotificationChannels\NetGsm\Exceptions\CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        $message = $notification->toNetGsm($notifiable);

The method toNetGsm() does not exist on Illuminate\Notifications\Notification. Are you sure you never get this type here, but always one of the subclasses?

        if (is_string($message)) {
            $message = NetGsmMessage::create($message);
        }

        if ($to = $notifiable->routeNotificationFor('netgsm')) {
            $message->setRecipients($to);
        }

        $this->client->send($message);
    }
}