Spatie\ServerMonitor\Notifications\Notifications\CheckSucceeded

Complexity

Total Complexity

4

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	toMail()	0	6	1
A	toSlack()	0	11	1
A	setEvent()	0	6	1
A	shouldSend()	0	4	1

<?php

namespace Spatie\ServerMonitor\Notifications\Notifications;

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

class CheckSucceeded extends BaseNotification
{
    /** @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)
            ->subject($this->getSubject())
            ->line($this->getMessageText());
    }

    public function toSlack($notifiable)

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

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

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

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

        return $this;
    }

    public function shouldSend(): bool
    {
        return $this->getCheck()->hasStatus(CheckStatus::SUCCESS);
    }
}