App\Console\Commands\DisableModuleCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	handle()	0	16	2

<?php

namespace App\Console\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Artisan;
use Nwidart\Modules\Facades\Module;

class DisableModuleCommand extends Command
{
    protected $signature  = 'tb_module:disable {module}';
    protected $description = 'Disable, but do not remove a Tech Bench Add On Module';

    /**
     * Create a new command instance
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command
     */
    public function handle()
    {
        $this->module = Module::find($this->argument('module'));

The property module does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        //  Verify Module exists
        if(!$this->module)
        {
            $this->error('Unable to find module');
            return 1;
        }

        Artisan::call('module:disable '.$this->module->getStudlyName());

        $this->info('Module has been disabled');

        return Command::SUCCESS;
    }
}