Generator::__construct() - Code Metrics - Inspection of "add quality tools" - mshamaseen/laravel-repository-pattern - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 0a4c8d...e29dcc )

by Hamzah

created 2019-01-14 14:56 UTC

Generator::__construct() A

↳ Parent: Generator

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Shamaseen\Repository\Generator\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\File;

/**
 * Class RepositoryGenerator
 * @package Shamaseen\Repository\Generator\Commands
 */
class Generator extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'make:repository
    {name : Class (singular) for example User}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Create repository generator';

    /**
     * The repository name.
     *
     * @var string
     */
    protected $repoName;
    /**
     * The repository name space.
     *
     * @var string
     */
    protected $repoNamespace;

    /**
     * Create a new command instance.
     *
     * @return void

     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return void
     */
    public function handle()
    {

        $file = explode("/", (string)$this->argument('name'));

        $this->repoName=$file[count($file) - 1];
        unset($file[count($file) - 1]);
        $path = implode("/", $file);

        if(count($file) == 0){

            $this->repoNamespace=$this->repoName;
        }else{

            $name = $file[count($file) - 1];
            $this->repoNamespace=implode("\\", $file);
        }


        $this->generate($path, \Config::get('repository.controllers_path'), 'Controller');
        $this->generate($path, \Config::get('repository.models_path'), 'Entity');
        $this->generate($path, \Config::get('repository.requests_path'), 'Request');
        $this->generate($path, \Config::get('repository.interfaces_path'), 'Interface');
        $this->generate($path, \Config::get('repository.repositories_path'), 'Repository');

        File::append(\Config::get('repository.route_path').'web.php', "\n" . 'Route::resource(\'' . str_plural($name) . "', '{$name}Controller');");
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

    }


    /**
     * Get stub content to generate needed files
     *
     * @param string $type  determine which stub should choose to get content
     * @return false|string
     */
    protected function getStub($type)
    {
        return file_get_contents(\Config::get('repository.resources_path')."/stubs/$type.stub");
    }

    /**
     * @param string $name Class name
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     * @param string $path Class path
     * @param string $folder default path to generate in
     * @param string $type define which kind of files should generate
     */
    protected function generate($path, $folder, $type)
    {

        $template = str_replace(
            [
                '{{modelName}}',
                "{{modelNamePlural}}"
            ],
            [
                $this->repoName,
                $this->repoNamespace
            ],
            $this->getStub($type)
        );
        $path = $this->checkFolder(\Config::get('repository.app_path').'/'.$folder.$path."/");

        file_put_contents($path . "{$this->repoName}{$type}.php", $template);

    }


    /**
     * Check if folder exist
     * @param string $path class path
     * @return string
     */
    public function checkFolder($path)
    {

        if (!file_exists($path)) {
            mkdir($path, 0777, true);
        }
        return $path;
    }
}


1			<?php
2
3			namespace Shamaseen\Repository\Generator\Commands;
4
5			use Illuminate\Console\Command;
6			use Illuminate\Support\Facades\File;
7
8			/**
9			* Class RepositoryGenerator
10			* @package Shamaseen\Repository\Generator\Commands
11			*/
12			class Generator extends Command
13			{
14			/**
15			* The name and signature of the console command.
16			*
17			* @var string
18			*/
19			protected $signature = 'make:repository
20			{name : Class (singular) for example User}';
21
22			/**
23			* The console command description.
24			*
25			* @var string
26			*/
27			protected $description = 'Create repository generator';
28
29			/**
30			* The repository name.
31			*
32			* @var string
33			*/
34			protected $repoName;
35			/**
36			* The repository name space.
37			*
38			* @var string
39			*/
40			protected $repoNamespace;
41
42			/**
43			* Create a new command instance.
44			*
45			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
46			*/
47			public function __construct()
48			{
49			parent::__construct();
50			}
51
52			/**
53			* Execute the console command.
54			*
55			* @return void
56			*/
57			public function handle()
58			{
59
60			$file = explode("/", (string)$this->argument('name'));
61
62			$this->repoName=$file[count($file) - 1];
63			unset($file[count($file) - 1]);
64			$path = implode("/", $file);
65
66			if(count($file) == 0){
			0 ignored issues – show Coding Style introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Expected 1 space(s) after IF keyword; 0 found Loading history...
67			$this->repoNamespace=$this->repoName;
68			}else{
			0 ignored issues – show Coding Style introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Expected 1 space(s) after ELSE keyword; 0 found Loading history...
69			$name = $file[count($file) - 1];
70			$this->repoNamespace=implode("\\", $file);
71			}
72
73
74			$this->generate($path, \Config::get('repository.controllers_path'), 'Controller');
75			$this->generate($path, \Config::get('repository.models_path'), 'Entity');
76			$this->generate($path, \Config::get('repository.requests_path'), 'Request');
77			$this->generate($path, \Config::get('repository.interfaces_path'), 'Interface');
78			$this->generate($path, \Config::get('repository.repositories_path'), 'Repository');
79
80			File::append(\Config::get('repository.route_path').'web.php', "\n" . 'Route::resource(\'' . str_plural($name) . "', '{$name}Controller');");
			0 ignored issues – show Bug introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report The variable `$name` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
81
82			}
			0 ignored issues – show Coding Style introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Function closing brace must go on the next line following the body; found 1 blank lines before brace Loading history...
83
84			/**
85			* Get stub content to generate needed files
86			*
87			* @param string $type determine which stub should choose to get content
88			* @return false\|string
89			*/
90			protected function getStub($type)
91			{
92			return file_get_contents(\Config::get('repository.resources_path')."/stubs/$type.stub");
93			}
94
95			/**
96			* @param string $name Class name
			0 ignored issues – show Bug introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report There is no parameter named `$name`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
97			* @param string $path Class path
98			* @param string $folder default path to generate in
99			* @param string $type define which kind of files should generate
100			*/
101			protected function generate($path, $folder, $type)
102			{
103
104			$template = str_replace(
105			[
106			'{{modelName}}',
107			"{{modelNamePlural}}"
108			],
109			[
110			$this->repoName,
111			$this->repoNamespace
112			],
113			$this->getStub($type)
114			);
115			$path = $this->checkFolder(\Config::get('repository.app_path').'/'.$folder.$path."/");
			0 ignored issues – show Coding Style introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Consider using a different name than the parameter `$path`. This often makes code more readable. Loading history...
116			file_put_contents($path . "{$this->repoName}{$type}.php", $template);
117
118			}
			0 ignored issues – show Coding Style introduced 2019-01-14 15:00 UTC by Report Bug Copy Issue Report Function closing brace must go on the next line following the body; found 1 blank lines before brace Loading history...
119
120			/**
121			* Check if folder exist
122			* @param string $path class path
123			* @return string
124			*/
125			public function checkFolder($path)
126			{
127
128			if (!file_exists($path)) {
129			mkdir($path, 0777, true);
130			}
131			return $path;
132			}
133			}
134

mshamaseen / laravel-repository-pattern

Push — master ( 0a4c8d...e29dcc )

Generator::__construct() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like