CheckSucceeded::toSlack() - Code Metrics - Inspection of "commit" - spatie/laravel-server-monitor - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 434343...7ded52 )

by Freek

created 2017-02-24 21:01 UTC

CheckSucceeded::toSlack() A

↳ Parent: CheckSucceeded

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	8
nc	1
nop	1

<?php

namespace Spatie\ServerMonitor\Notifications\Notifications;

use Carbon\Carbon;
use Illuminate\Notifications\Messages\MailMessage;
use Spatie\ServerMonitor\Models\Enums\CheckStatus;
use Illuminate\Notifications\Messages\SlackMessage;
use Illuminate\Notifications\Messages\SlackAttachment;
use Spatie\ServerMonitor\Notifications\BaseNotification;
use Spatie\ServerMonitor\Events\CheckSucceeded as CheckSucceededEvent;

class CheckSucceeded extends BaseNotification
{
    /** @var \Spatie\ServerMonitor\Events\CheckWarning */
    public $event;

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

    {
        return (new MailMessage)
            ->subject($this->getSubject())
            ->line($this->getMessageText());
    }

    public function toSlack($notifiable)

    {
        return (new SlackMessage)
            ->attachment(function (SlackAttachment $attachment) {
                $attachment
                    ->title($this->getSubject())
                    ->content($this->getMessageText())
                    ->fallback($this->getMessageText())
                    ->timestamp(Carbon::now());
            });
    }

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


        return $this;
    }

    public function shouldSend(): bool
    {
        return $this->getCheck()->hasStatus(CheckStatus::SUCCESS);
    }
}


1			<?php
2
3			namespace Spatie\ServerMonitor\Notifications\Notifications;
4
5			use Carbon\Carbon;
6			use Illuminate\Notifications\Messages\MailMessage;
7			use Spatie\ServerMonitor\Models\Enums\CheckStatus;
8			use Illuminate\Notifications\Messages\SlackMessage;
9			use Illuminate\Notifications\Messages\SlackAttachment;
10			use Spatie\ServerMonitor\Notifications\BaseNotification;
11			use Spatie\ServerMonitor\Events\CheckSucceeded as CheckSucceededEvent;
12
13			class CheckSucceeded extends BaseNotification
14			{
15			/** @var \Spatie\ServerMonitor\Events\CheckWarning */
16			public $event;
17
18			/**
19			* Get the mail representation of the notification.
20			*
21			* @param mixed $notifiable
22			* @return \Illuminate\Notifications\Messages\MailMessage
23			*/
24			public function toMail($notifiable)
			0 ignored issues – show Unused Code introduced 2017-02-18 18:45 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...
25			{
26			return (new MailMessage)
27			->subject($this->getSubject())
28			->line($this->getMessageText());
29			}
30
31			public function toSlack($notifiable)
			0 ignored issues – show Unused Code introduced 2017-02-18 18:45 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...
32			{
33			return (new SlackMessage)
34			->attachment(function (SlackAttachment $attachment) {
35			$attachment
36			->title($this->getSubject())
37			->content($this->getMessageText())
38			->fallback($this->getMessageText())
39			->timestamp(Carbon::now());
40			});
41			}
42
43			public function setEvent(CheckSucceededEvent $event)
44			{
45			$this->event = $event;
			0 ignored issues – show Documentation Bug introduced 2017-02-18 18:45 UTC by Report Bug Copy Issue Report It seems like `$event` of type `object<Spatie\ServerMoni...\Events\CheckSucceeded>` is incompatible with the declared type `object<Spatie\ServerMonitor\Events\CheckWarning>` 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...
46
47			return $this;
48			}
49
50			public function shouldSend(): bool
51			{
52			return $this->getCheck()->hasStatus(CheckStatus::SUCCESS);
53			}
54			}
55

spatie / laravel-server-monitor

Push — master ( 434343...7ded52 )

CheckSucceeded::toSlack() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like