Shamaseen\Repository\Generator\Commands\Generator

Complexity

Total Complexity

7

Size/Duplication

Total Lines	122
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	26	2
A	getStub()	0	4	1
A	generate()	0	18	1
A	checkFolder()	0	8	2

<?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

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    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){

Expected 1 space(s) after IF keyword; 0 found

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

Expected 1 space(s) after ELSE keyword; 0 found

            $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');");

The variable $name does not seem to be defined for all execution paths leading up to this point.

    }

Function closing brace must go on the next line following the body; found 1 blank lines before brace

    /**
     * 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

There is no parameter named $name. Was it maybe removed?

     * @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."/");

Consider using a different name than the parameter $path. This often makes code more readable.

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

    }

Function closing brace must go on the next line following the body; found 1 blank lines before brace

    /**
     * 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;
    }
}