CertificateExpiresSoon::toMail() - Code Metrics - Inspection of "Better names" - spatie/laravel-uptime-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#33)

by Freek

created 2016-11-17 22:47 UTC

CertificateExpiresSoon::toMail() A

↳ Parent: CertificateExpiresSoon

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	7
nc	2
nop	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\CertificateExpiresSoon as SoonExpiringSslCertificateFoundEvent;
use Spatie\UptimeMonitor\Notifications\BaseNotification;

class CertificateExpiresSoon extends BaseNotification
{
    /** @var \Spatie\UptimeMonitor\Events\CertificateExpiresSoon */
    public $event;

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

    {
        $mailMessage = (new MailMessage)
            ->subject($this->getMessageText())
            ->line($this->getMessageText());

        foreach ($this->getMonitorProperties() as $name => $value) {
            $mailMessage->line($name.': '.$value);
        }

        return $mailMessage;
    }

    public function toSlack($notifiable)

    {
        return (new SlackMessage)
            ->content($this->getMessageText())
            ->attachment(function (SlackAttachment $attachment) {
                $attachment->fields($this->getMonitorProperties());
            });
    }

    protected function getMessageText(): string
    {
        return "The certificate for {$this->event->monitor->url} will expire in {$this->event->monitor->certificate_expiration_date->diffInDays()} days.";
    }

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

        return $this;
    }
}


1			<?php
2
3			namespace Spatie\UptimeMonitor\Notifications\Notifications;
4
5			use Illuminate\Notifications\Messages\MailMessage;
6			use Illuminate\Notifications\Messages\SlackAttachment;
7			use Illuminate\Notifications\Messages\SlackMessage;
8			use Spatie\UptimeMonitor\Events\CertificateExpiresSoon as SoonExpiringSslCertificateFoundEvent;
9			use Spatie\UptimeMonitor\Notifications\BaseNotification;
10
11			class CertificateExpiresSoon extends BaseNotification
12			{
13			/** @var \Spatie\UptimeMonitor\Events\CertificateExpiresSoon */
14			public $event;
15
16			/**
17			* Get the mail representation of the notification.
18			*
19			* @param mixed $notifiable
20			* @return \Illuminate\Notifications\Messages\MailMessage
21			*/
22			public function toMail($notifiable)
			0 ignored issues – show Unused Code introduced 2016-11-12 22:10 UTC by Report Bug Copy Issue Report The parameter `$notifiable` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
23			{
24			$mailMessage = (new MailMessage)
25			->subject($this->getMessageText())
26			->line($this->getMessageText());
27
28			foreach ($this->getMonitorProperties() as $name => $value) {
29			$mailMessage->line($name.': '.$value);
30			}
31
32			return $mailMessage;
33			}
34
35			public function toSlack($notifiable)
			0 ignored issues – show Unused Code introduced 2016-11-12 22:10 UTC by Report Bug Copy Issue Report The parameter `$notifiable` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
36			{
37			return (new SlackMessage)
38			->content($this->getMessageText())
39			->attachment(function (SlackAttachment $attachment) {
40			$attachment->fields($this->getMonitorProperties());
41			});
42			}
43
44			protected function getMessageText(): string
45			{
46			return "The certificate for {$this->event->monitor->url} will expire in {$this->event->monitor->certificate_expiration_date->diffInDays()} days.";
47			}
48
49			public function setEvent(SoonExpiringSslCertificateFoundEvent $event)
50			{
51			$this->event = $event;
52
53			return $this;
54			}
55			}
56

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Pull Request — master (#33)

CertificateExpiresSoon::toMail() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like