NotificationChannels\Cmsms\CmsmsChannel

Complexity

Total Complexity

4

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

30%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?php

declare(strict_types=1);

namespace NotificationChannels\Cmsms;

use Illuminate\Notifications\Notification;

class CmsmsChannel
{
    /** @var CmsmsClient */
    protected $client;

    /**
     * @param CmsmsClient $client
     */
    public function __construct(CmsmsClient $client)
    {
        $this->client = $client;
    }

    /**
     * Send the given notification.
     *
     * @param mixed                                  $notifiable
     * @param \Illuminate\Notifications\Notification $notification
     *
     * @throws \NotificationChannels\Cmsms\Exceptions\CouldNotSendNotification
     */
    public function send($notifiable, Notification $notification)
    {
        if (!$recipient = $notifiable->routeNotificationFor('Cmsms')) {
            return;
        }

        $message = $notification->toCmsms($notifiable);

The method toCmsms() does not exist on Illuminate\Notifications\Notification. It seems like you code against a sub-type of Illuminate\Notifications\Notification such as NotificationChannels\Cmsms\Test\TestNotification.

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

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