App\Notifications\DeviceRegister

Complexity

Total Complexity

4

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	toNexmo()	0	5	1
A	toArray()	0	3	1
A	via()	0	3	1
A	__construct()	0	2	1

<?php

namespace App\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Notification;
use Illuminate\Notifications\Messages\NexmoMessage;

class DeviceRegister extends Notification
{
    use Queueable;

    /**
     * Create a new notification instance.
     *
     * @return void
     */
    public function __construct()
    {
        //
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed  $notifiable
     * @return string[]
     */
    public function via($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return [ 'nexmo' ];
    }

    /**
     * Get the SMS representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\NexmoMessage
     */
    public function toNexmo($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return (new NexmoMessage)
                    ->content('SmartSettia SMS: New device registered '.$this->device->uuid)

The property device does not exist on App\Notifications\DeviceRegister. Did you maybe forget to declare it?

                    ->from('12088851101');
    }

    /**
     * Get the array representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return array
     */
    public function toArray($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return [
            //
        ];
    }
}