NotificationChannels\Pushover\PushoverChannel

Complexity

Total Complexity

4

Size/Duplication

Total Lines	54
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	5
Bugs	1	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	send()	0	29	3

<?php

namespace NotificationChannels\Pushover;

use Illuminate\Notifications\Notification;
use Illuminate\Notifications\Events\NotificationFailed;
use Illuminate\Contracts\Events\Dispatcher;
use NotificationChannels\Pushover\Exceptions\ServiceCommunicationError;

class PushoverChannel
{
    /** @var Pushover */
    protected $pushover;

    /**
     * Create a new Pushover channel instance.
     *
     * @param  Pushover $pushover
     */
    public function __construct(Pushover $pushover, Dispatcher $events)
    {
        $this->pushover = $pushover;
        $this->events = $events;

The property events does not exist. Did you maybe forget to declare it?

    }

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

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

The method toPushover() does not seem to exist on object<Illuminate\Notifications\Notification>.

        try {
            $this->pushover->send([
                'user' => $pushoverKey,
                'message' => $message->content,
                'title' => $message->title,
                'timestamp' => $message->timestamp,
                'priority' => $message->priority,
                'url' => $message->url,
                'url_title' => $message->urlTitle,
                'sound' => $message->sound,
                'retry' => $message->retry,
                'expire' => $message->expire,
            ]);
        } catch (ServiceCommunicationError $serviceCommunicationError) {
            $this->events->fire(
                new NotificationFailed($notifiable, $notification, 'pushover', [
                    $serviceCommunicationError->getMessage()
                ])
            );
        }
    }
}