Spatie\UptimeMonitor\Notifications\Notifications\SslCheckSucceeded

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	23.68 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	toMail()	0	9	1
A	toSlack()	9	9	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\SslCheckSucceeded as ValidSslCertificateFoundEvent;
use Spatie\UptimeMonitor\Notifications\BaseNotification;

class SslCheckSucceeded extends BaseNotification
{
    /** @var \Spatie\UptimeMonitor\Events\SslCheckSucceeded */
    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("Found a valid certificate for {$this->event->monitor->url}.")
            ->line('Found a valid certificate');

        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("Found a valid ssl certificate for {$this->event->monitor->url}")
            ->attachment(function (SlackAttachment $attachment) {
                $attachment->fields($this->getMonitorProperties());
            });
    }

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

        return $this;
    }
}