Spatie\ServerMonitor\Notifications\BaseNotification

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
	shouldSend()	0	1	?
A	via()	0	4	1
A	getCheck()	0	4	1
A	getMessageText()	0	4	1
A	getSubject()	0	4	1

<?php

namespace Spatie\ServerMonitor\Notifications;

use Illuminate\Notifications\Notification;
use Spatie\ServerMonitor\Models\Check;

abstract class BaseNotification extends Notification
{
    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed $notifiable
     * @return array
     */
    public function via($notifiable)

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

    {
        return config('server-monitor.notifications.notifications.'.static::class);
    }

    public function getCheck(): Check
    {
        return $this->event->check;

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

    }

    protected function getMessageText(): ?string
    {
        return ucfirst($this->getCheck()->last_run_message);
    }

    protected function getSubject(): string
    {
        return "{$this->getCheck()->type} on {$this->getCheck()->host->name}";
    }

    abstract public function shouldSend(): bool;
}