NotificationChannels\Textlocal\TextlocalChannel

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

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

<?php

namespace NotificationChannels\Textlocal;

use Illuminate\Notifications\Notification;

class TextlocalChannel
{
    public function __construct(TextlocalClient $client)
    {
        $this->textlocal = $client;

The property textlocal does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

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

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

The method toTextlocal() does not exist on Illuminate\Notifications\Notification. It seems like you code against a sub-type of said class. However, the method does not exist in Illuminate\Auth\Notifications\ResetPassword or Illuminate\Auth\Notifications\VerifyEmail. Are you sure you never get one of those?

        if (is_string($message)) {
            $message = new TextlocalMessage($message);
        }

        return $this->textlocal->message($to, $message);
    }
}