App\Console\Commands\MakeCrud

Complexity

Total Complexity

2

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	18	1

<?php

namespace App\Console\Commands;

use Artisan;
use Illuminate\Console\Command;

class MakeCrud extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'make:crud {name}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Creates a new backpack crud';

    /**
     * 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.
     */
    public function handle()
    {
        $argument  = str_singular($this->argument('name'));

It seems like $this->argument('name') targeting Illuminate\Console\Command::argument() can also be of type array or null; however, str_singular() does only seem to accept string, maybe add an additional type check?

        $plural    = title_case(str_plural($argument));
        $snakeName = snake_case($argument);
        $camelName = camel_case($argument);

        $routeArgument   = sprintf("CRUD::resource('%s', '%sCrudController');", $snakeName, $camelName);
        $sidebarArgument = sprintf(
            "<li><a href='{{ backpack_url('%s') }}'><i class='fa fa-question-circle'></i> <span>%s</span></a></li>",
            $argument,
            $plural
        );

        Artisan::call('backpack:crud', ['name' => $snakeName]);
        Artisan::call('backpack:base:add-custom-route', ['code' => $routeArgument]);
        Artisan::call('backpack:base:add-sidebar-content', ['code' => $sidebarArgument]);
    }
}