Command::notify() - Code Metrics - Inspection of "[3.6] [WIP] Adds scheduler" - laravel-zero/framework - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — stable (#25)

by Nuno

created 2017-09-20 22:01 UTC

Command::notify() A

↳ Parent: Command

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	9
nc	1
nop	3
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0

<?php

namespace LaravelZero\Framework\Commands;

use Illuminate\Console\Scheduling\Schedule;
use Illuminate\Contracts\Container\Container;
use Illuminate\Console\Command as BaseCommand;
use NunoMaduro\LaravelDesktopNotifier\Contracts\Notifier;
use NunoMaduro\LaravelDesktopNotifier\Contracts\Notification;

/**
 * This is the Zero Framework abstract command class.
 *
 * @author Nuno Maduro <[email protected]>
 */
abstract class Command extends BaseCommand
{
    /**
     * Execute the console command. Here goes the command
     * code.
     *
     * @return void
     */
    abstract public function handle(): void;

    /**
     * Returns the application container.
     *
     * @return \Illuminate\Contracts\Container\Container
     */
    public function getContainer(): Container
    {
        return $this->getLaravel();
    }

    /**
     * Define the command's schedule.
     *
     * @param  \Illuminate\Console\Scheduling\Schedule  $schedule
     *
     * @return void
     */
    public function schedule(Schedule $schedule): void

    {
    }

    /**
     * Gets the concrete implementation of the notifier. Then
     * creates a new notification and send it through the notifier.
     *
     * @param string $text
     * @param string $body
     * @param string|null $icon
     */
    public function notify(string $text, string $body, $icon = null): void
    {
        $notifier = $this->getContainer()
            ->make(Notifier::class);

        $notification = $this->getContainer()
            ->make(Notification::class)
            ->setTitle($text)
            ->setBody($body)
            ->setIcon($icon);

        $notifier->send($notification);
    }
}


1			<?php
2
3			namespace LaravelZero\Framework\Commands;
4
5			use Illuminate\Console\Scheduling\Schedule;
6			use Illuminate\Contracts\Container\Container;
7			use Illuminate\Console\Command as BaseCommand;
8			use NunoMaduro\LaravelDesktopNotifier\Contracts\Notifier;
9			use NunoMaduro\LaravelDesktopNotifier\Contracts\Notification;
10
11			/**
12			* This is the Zero Framework abstract command class.
13			*
14			* @author Nuno Maduro <[email protected]>
15			*/
16			abstract class Command extends BaseCommand
17			{
18			/**
19			* Execute the console command. Here goes the command
20			* code.
21			*
22			* @return void
23			*/
24			abstract public function handle(): void;
25
26			/**
27			* Returns the application container.
28			*
29			* @return \Illuminate\Contracts\Container\Container
30			*/
31			public function getContainer(): Container
32			{
33			return $this->getLaravel();
34			}
35
36			/**
37			* Define the command's schedule.
38			*
39			* @param \Illuminate\Console\Scheduling\Schedule $schedule
40			*
41			* @return void
42			*/
43			public function schedule(Schedule $schedule): void
			0 ignored issues – show Unused Code introduced 2017-09-20 20:41 UTC by Report Bug Copy Issue Report The parameter `$schedule` 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...
44			{
45			}
46
47			/**
48			* Gets the concrete implementation of the notifier. Then
49			* creates a new notification and send it through the notifier.
50			*
51			* @param string $text
52			* @param string $body
53			* @param string\|null $icon
54			*/
55			public function notify(string $text, string $body, $icon = null): void
56			{
57			$notifier = $this->getContainer()
58			->make(Notifier::class);
59
60			$notification = $this->getContainer()
61			->make(Notification::class)
62			->setTitle($text)
63			->setBody($body)
64			->setIcon($icon);
65
66			$notifier->send($notification);
67			}
68			}
69

laravel-zero / framework

Pull Request — stable (#25)

Command::notify() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like