CertificateCheckSucceeded::toSlack() - 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

CertificateCheckSucceeded::toSlack() A

↳ Parent: CertificateCheckSucceeded

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0
cc	1
eloc	6
nc	1
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\CertificateCheckSucceeded as ValidCertificateFoundEvent;
use Spatie\UptimeMonitor\Notifications\BaseNotification;

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

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

    {
        $mailMessage = (new MailMessage)
            ->success()
            ->subject("The certificate check for {$this->event->monitor->url} succeeded.")
            ->line("The certificate check for {$this->event->monitor->url} succeeded.");

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

        return $mailMessage;
    }

    public function toSlack($notifiable)

    {
        return (new SlackMessage)
            ->success()
            ->content("The certificate check for {$this->event->monitor->url} succeeded.")
            ->attachment(function (SlackAttachment $attachment) {
                $attachment->fields($this->getMonitorProperties());
            });
    }

    public function setEvent(ValidCertificateFoundEvent $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\CertificateCheckSucceeded as ValidCertificateFoundEvent;
9			use Spatie\UptimeMonitor\Notifications\BaseNotification;
10
11			class CertificateCheckSucceeded extends BaseNotification
12			{
13			/** @var \Spatie\UptimeMonitor\Events\CertificateCheckSucceeded */
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 21:55 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			->success()
26			->subject("The certificate check for {$this->event->monitor->url} succeeded.")
27			->line("The certificate check for {$this->event->monitor->url} succeeded.");
28
29			foreach ($this->getMonitorProperties() as $name => $value) {
30			$mailMessage->line($name.': '.$value);
31			}
32
33			return $mailMessage;
34			}
35
36			public function toSlack($notifiable)
			0 ignored issues – show Unused Code introduced 2016-11-12 21:55 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...
37			{
38			return (new SlackMessage)
39			->success()
40			->content("The certificate check for {$this->event->monitor->url} succeeded.")
41			->attachment(function (SlackAttachment $attachment) {
42			$attachment->fields($this->getMonitorProperties());
43			});
44			}
45
46			public function setEvent(ValidCertificateFoundEvent $event)
47			{
48			$this->event = $event;
49
50			return $this;
51			}
52			}
53

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Pull Request — master (#33)

CertificateCheckSucceeded::toSlack() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like