RequireDevTools::handle() - Code Metrics - Inspection of "remove DevTools does not support L10 notice in ins..." - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — remove-devtools-l10-notice ( c012c0 )

by Cristian

created 2023-02-27 17:17 UTC

RequireDevTools::handle() A

↳ Parent: RequireDevTools

Complexity

Conditions	5
Paths	5

Size

Total Lines	54
Code Lines	29

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

Metric	Value
cc	5
eloc	29
c	3
b	0
f	0
nc	5
nop	0
dl	0
loc	54
rs	9.1448

How to fix Long Method

<?php

namespace Backpack\CRUD\app\Console\Commands\Addons;

use Illuminate\Console\Command;

class RequireDevTools extends Command
{
    use \Backpack\CRUD\app\Console\Commands\Traits\PrettyCommandOutput;

    use \Backpack\CRUD\app\Console\Commands\Traits\AddonsHelper;


    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'backpack:require:devtools
                                {--debug} : Show process output or not. Useful for debugging.';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Require Backpack DevTools on dev';

    /**
     * Backpack addons install attribute.
     *
     * @var array
     */
    public static $addon = [
        'name'        => 'DevTools',
        'description' => [
            'Helps generate models, migrations, operations and CRUDs',
        ],
        'path'    => 'vendor/backpack/devtools',
        'command' => 'backpack:require:devtools',
    ];

    /**
     * Execute the console command.
     *
     * @return mixed Command-line output
     */
    public function handle()
    {
        // Check if it is installed
        if ($this->isInstalled()) {
            $this->newLine();
            $this->line(sprintf('  %s was already installed', self::$addon['name']), 'fg=red');
            $this->newLine();

            return;
        }

        $this->newLine();
        $this->infoBlock('Connecting to the Backpack add-on repository');

        // Check if repositories exists
        $this->composerRepositories();

        // Check for authentication
        $this->checkForAuthentication();

        $this->newLine();
        $this->progressBlock($this->description);

        // Require package
        try {
            $this->composerRequire('backpack/devtools', ['--dev', '--with-all-dependencies']);
        } catch (\Throwable $e) {
            $this->errorProgressBlock();
            $this->line('  '.$e->getMessage(), 'fg=red');
            $this->newLine();

            return;
        }

        // Display general error in case it failed
        if (! $this->isInstalled()) {
            $this->errorProgressBlock();
            $this->note('For further information please check the log file.');
            $this->note('You can also follow the manual installation process documented in https://backpackforlaravel.com/addons/');
            $this->newLine();

            return;
        }

        // Finish
        $this->closeProgressBlock();
        $this->newLine();

        // manually include the command in the run-time
        if (! class_exists(\Backpack\DevTools\Console\Commands\InstallDevTools::class)) {
filter:
    dependency_paths: ["lib/*"]
            include base_path('vendor/backpack/devtools/src/Console/Commands/InstallDevTools.php');
        }

        $this->call(\Backpack\DevTools\Console\Commands\InstallDevTools::class);
    }

    public function isInstalled()
    {
        return file_exists(self::$addon['path'].'/composer.json');
    }
}


1			<?php
2
3			namespace Backpack\CRUD\app\Console\Commands\Addons;
4
5			use Illuminate\Console\Command;
6
7			class RequireDevTools extends Command
8			{
9			use \Backpack\CRUD\app\Console\Commands\Traits\PrettyCommandOutput;
			0 ignored issues – show introduced 2022-11-08 09:40 UTC by Report Bug Copy Issue Report The trait `Backpack\CRUD\app\Consol...its\PrettyCommandOutput` requires some properties which are not provided by `Backpack\CRUD\app\Consol...\Addons\RequireDevTools`: `$progressBar`, `$statusColor`, `$status` Loading history...
10			use \Backpack\CRUD\app\Console\Commands\Traits\AddonsHelper;
			0 ignored issues – show Bug introduced 2022-11-08 09:40 UTC by Report Bug Copy Issue Report The trait `Backpack\CRUD\app\Consol...nds\Traits\AddonsHelper` requires the property `$repositories` which is not provided by `Backpack\CRUD\app\Consol...\Addons\RequireDevTools`. Loading history...
11
12			/**
13			* The name and signature of the console command.
14			*
15			* @var string
16			*/
17			protected $signature = 'backpack:require:devtools
18			{--debug} : Show process output or not. Useful for debugging.';
19
20			/**
21			* The console command description.
22			*
23			* @var string
24			*/
25			protected $description = 'Require Backpack DevTools on dev';
26
27			/**
28			* Backpack addons install attribute.
29			*
30			* @var array
31			*/
32			public static $addon = [
33			'name' => 'DevTools',
34			'description' => [
35			'Helps generate models, migrations, operations and CRUDs',
36			],
37			'path' => 'vendor/backpack/devtools',
38			'command' => 'backpack:require:devtools',
39			];
40
41			/**
42			* Execute the console command.
43			*
44			* @return mixed Command-line output
45			*/
46			public function handle()
47			{
48			// Check if it is installed
49			if ($this->isInstalled()) {
50			$this->newLine();
51			$this->line(sprintf(' %s was already installed', self::$addon['name']), 'fg=red');
52			$this->newLine();
53
54			return;
55			}
56
57			$this->newLine();
58			$this->infoBlock('Connecting to the Backpack add-on repository');
59
60			// Check if repositories exists
61			$this->composerRepositories();
62
63			// Check for authentication
64			$this->checkForAuthentication();
65
66			$this->newLine();
67			$this->progressBlock($this->description);
68
69			// Require package
70			try {
71			$this->composerRequire('backpack/devtools', ['--dev', '--with-all-dependencies']);
72			} catch (\Throwable $e) {
73			$this->errorProgressBlock();
74			$this->line(' '.$e->getMessage(), 'fg=red');
75			$this->newLine();
76
77			return;
78			}
79
80			// Display general error in case it failed
81			if (! $this->isInstalled()) {
82			$this->errorProgressBlock();
83			$this->note('For further information please check the log file.');
84			$this->note('You can also follow the manual installation process documented in https://backpackforlaravel.com/addons/');
85			$this->newLine();
86
87			return;
88			}
89
90			// Finish
91			$this->closeProgressBlock();
92			$this->newLine();
93
94			// manually include the command in the run-time
95			if (! class_exists(\Backpack\DevTools\Console\Commands\InstallDevTools::class)) {
			0 ignored issues – show Bug introduced 2021-09-18 13:38 UTC by Report Bug Copy Issue Report The type `Backpack\DevTools\Console\Commands\InstallDevTools` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
96			include base_path('vendor/backpack/devtools/src/Console/Commands/InstallDevTools.php');
97			}
98
99			$this->call(\Backpack\DevTools\Console\Commands\InstallDevTools::class);
100			}
101
102			public function isInstalled()
103			{
104			return file_exists(self::$addon['path'].'/composer.json');
105			}
106			}
107

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Push — remove-devtools-l10-notice ( c012c0 )

RequireDevTools::handle() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like