Spatie\ServerMonitor\Notifications\Notifications\CheckFailed

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	100 %

Coupling/Cohesion

Components	2
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	toMail()	7	7	1
A	toSlack()	12	12	1
A	setEvent()	6	6	1
A	shouldSend()	8	8	2

<?php

namespace Spatie\ServerMonitor\Notifications\Notifications;

use Carbon\Carbon;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Messages\SlackAttachment;
use Illuminate\Notifications\Messages\SlackMessage;
use Spatie\ServerMonitor\Events\CheckFailed as CheckFailedEvent;
use Spatie\ServerMonitor\Models\Enums\CheckStatus;
use Spatie\ServerMonitor\Notifications\BaseNotification;

class CheckFailed extends BaseNotification

This class seems to be duplicated in your project.

{
    /** @var \Spatie\ServerMonitor\Events\CheckWarning */
    public $event;

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)

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

    {
        return (new MailMessage)
            ->error()
            ->subject($this->getSubject())
            ->line($this->getMessageText());
    }

    public function toSlack($notifiable)

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

    {
        return (new SlackMessage)
            ->error()
            ->attachment(function (SlackAttachment $attachment) {
                $attachment
                    ->title($this->getSubject())
                    ->content($this->getMessageText())
                    ->fallback($this->getMessageText())
                    ->timestamp(Carbon::now());
            });
    }

    public function setEvent(CheckFailedEvent $event)
    {
        $this->event = $event;

It seems like $event of type object<Spatie\ServerMonitor\Events\CheckFailed> is incompatible with the declared type object<Spatie\ServerMonitor\Events\CheckWarning> of property $event.

        return $this;
    }

    public function shouldSend(): bool
    {
        if (! $this->getCheck()->hasStatus(CheckStatus::FAILED)) {
            return false;
        }

        return ! $this->getCheck()->isThrottlingFailedNotifications();
    }
}