Spatie\UptimeMonitor\Notifications\Notifications\MonitorSucceeded

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	20.93 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	toMail()	0	9	1
A	toSlack()	9	9	1
A	isStillRelevant()	0	4	1
A	setEvent()	0	6	1

<?php

namespace Spatie\UptimeMonitor\Notifications\Notifications;

use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Messages\SlackAttachment;
use Illuminate\Notifications\Messages\SlackMessage;
use Spatie\UptimeMonitor\Events\MonitorSucceeded as MonitorSucceededEvent;
use Spatie\UptimeMonitor\Models\Enums\UptimeStatus;
use Spatie\UptimeMonitor\Notifications\BaseNotification;

class MonitorSucceeded extends BaseNotification
{
    /** @var \Spatie\UptimeMonitor\Events\MonitorSucceeded */
    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.

    {
        $mailMessage = (new MailMessage)
            ->success()
            ->subject("Site {$this->event->monitor->url} is up.")
            ->line('Site is up');

        return $mailMessage;
    }

    public function toSlack($notifiable)

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

This method seems to be duplicated in your project.

    {
        return (new SlackMessage)
            ->success()
            ->content("Site {$this->event->monitor->url} is up")
            ->attachment(function (SlackAttachment $attachment) {
                $attachment->fields($this->getMonitorProperties());
            });
    }

    public function isStillRelevant(): bool
    {
        return $this->event->monitor->uptime_status != UptimeStatus::DOWN;
    }

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

        return $this;
    }
}