App\Notification\Pushover\PushoverNotificationHandler

Complexity

Total Complexity

11

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	initialize()	0	6	2
A	canHandle()	0	4	3
A	getName()	0	2	1
A	__construct()	0	1	1
A	handle()	0	20	4

<?php

namespace App\Notification\Pushover;

use App\Notification\Notification;
use App\Notification\NotificationHandlerInterface;
use App\Settings\NotificationSettings;
use App\Utils\ArrayUtils;
use Psr\Log\LoggerInterface;
use Serhiy\Pushover\Api\Message\Message;
use Serhiy\Pushover\Api\Message\Notification as PushoverNotification;
use Serhiy\Pushover\Api\Message\Priority;
use Serhiy\Pushover\Api\Message\Sound;
use Serhiy\Pushover\Application;
use Serhiy\Pushover\Recipient;

class PushoverNotificationHandler implements NotificationHandlerInterface {

    private ?Application $application = null;

    public function __construct(private readonly NotificationSettings $notificationSettings, private readonly LoggerInterface $logger) { }

    public function canHandle(Notification $notification): bool {
        return !empty($this->notificationSettings->getPushoverApiToken())
            && !empty($notification->getRecipient()->getPushoverToken())
            && ArrayUtils::inArray($notification->getRecipient()->getUserType(), $this->notificationSettings->getPushoverEnabledUserTypes()) !== false;
    }

    private function initialize(): void {
        if($this->application !== null) {
            return;
        }

        $this->application = new Application($this->notificationSettings->getPushoverApiToken());

It seems like $this->notificationSettings->getPushoverApiToken() can also be of type null; however, parameter $token of Serhiy\Pushover\Application::__construct() does only seem to accept string, maybe add an additional type check?

    }

    public function handle(Notification $notification): void {
        $this->initialize();;

        $recipient = new Recipient($notification->getRecipient()->getPushoverToken());

It seems like $notification->getRecipient()->getPushoverToken() can also be of type null; however, parameter $userKey of Serhiy\Pushover\Recipient::__construct() does only seem to accept string, maybe add an additional type check?

        $message = new Message($notification->getContent(), $notification->getSubject());
        if(!empty($notification->getLink()) && !empty($notification->getLinkText())) {
            $message->setUrl($notification->getLink());
            $message->setUrlTitle($notification->getLinkText());
        }
        $message->setPriority(new Priority(Priority::NORMAL));

        $pushoverNotification = new PushoverNotification($this->application, $recipient, $message);

It seems like $this->application can also be of type null; however, parameter $application of Serhiy\Pushover\Api\Message\Notification::__construct() does only seem to accept Serhiy\Pushover\Application, maybe add an additional type check?

        $pushoverNotification->setSound(new Sound(Sound::PUSHOVER));

        $response = $pushoverNotification->push();

        if(!$response->isSuccessful()) {
            $this->logger->alert('Pushover-Benachrichtung wurde nicht erfolgreich abgeschickt.', [
                'errors' => $response->getErrors(),
                'receipt' => $response->getReceipt()
            ]);
        }
    }

    public function getName(): string {
        return 'pushover';
    }
}