SiteUp::isStillRelevant() - Code Metrics - Inspection of "wip" - spatie/laravel-uptime-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2d48be...f6c8f6 )

by Freek

created 2016-11-15 23:10 UTC

SiteUp::isStillRelevant() A

↳ Parent: SiteUp

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	2
c	0
b	0
f	0
nc	1
nop	0
dl	0
loc	4
rs	10

<?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\SiteUp as SiteUpEvent;
use Spatie\UptimeMonitor\Models\Enums\UptimeStatus;
use Spatie\UptimeMonitor\Notifications\BaseNotification;

class SiteUp extends BaseNotification
{
    /** @var \Spatie\UptimeMonitor\Events\SiteDown */
    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("Site {$this->event->site->url} is up.")
            ->line('Site is up');

        return $mailMessage;
    }

    public function toSlack($notifiable)

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

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

    public function setEvent(SiteUpEvent $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\SiteUp as SiteUpEvent;
9			use Spatie\UptimeMonitor\Models\Enums\UptimeStatus;
10			use Spatie\UptimeMonitor\Notifications\BaseNotification;
11
12			class SiteUp extends BaseNotification
13			{
14			/** @var \Spatie\UptimeMonitor\Events\SiteDown */
15			public $event;
16
17			/**
18			* Get the mail representation of the notification.
19			*
20			* @param mixed $notifiable
21			* @return \Illuminate\Notifications\Messages\MailMessage
22			*/
23			public function toMail($notifiable)
			0 ignored issues – show Unused Code introduced 2016-09-20 05:47 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...
24			{
25			$mailMessage = (new MailMessage)
26			->success()
27			->subject("Site {$this->event->site->url} is up.")
28			->line('Site is up');
29
30			return $mailMessage;
31			}
32
33			public function toSlack($notifiable)
			0 ignored issues – show Unused Code introduced 2016-09-20 05:47 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...
34			{
35			return (new SlackMessage)
36			->success()
37			->content("Site {$this->event->site->url} is up")
38			->attachment(function (SlackAttachment $attachment) {
39			$attachment->fields($this->getSiteProperties());
40			});
41			}
42
43			public function isStillRelevant(): bool
44			{
45			return $this->event->site->uptime_status != UptimeStatus::DOWN;
46			}
47
48			public function setEvent(SiteUpEvent $event)
49			{
50			$this->event = $event;
			0 ignored issues – show Documentation Bug introduced 2016-09-24 22:03 UTC by Report Bug Copy Issue Report It seems like `$event` of type `object<Spatie\UptimeMonitor\Events\SiteUp>` is incompatible with the declared type `object<Spatie\UptimeMonitor\Events\SiteDown>` of property `$event`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
51
52			return $this;
53			}
54			}
55

spatie / laravel-uptime-monitor

GitHub Access Token became invalid

Push — master ( 2d48be...f6c8f6 )

SiteUp::isStillRelevant() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like