LaravelModulize\Console\Commands\BaseGeneratorCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	12	2
A	__construct()	0	5	1

<?php

namespace LaravelModulize\Console\Commands;

use Illuminate\Filesystem\Filesystem;
use Illuminate\Console\GeneratorCommand;
use LaravelModulize\Contracts\ModulizerRepositoryInterface;

abstract class BaseGeneratorCommand extends GeneratorCommand
{
    /**
     * Instance of the repository
     *
     * @var \LaravelModulize\Contracts\ModulizerRepositoryInterface
     */
    protected $repository;

    protected $module;
    protected $modulePath;

    /**
     * Create a new command instance.
     *
     * @param \LaravelModulize\Contracts\ModulizerRepositoryInterface $repository
     * @return void
     */
    public function __construct(Filesystem $files, ModulizerRepositoryInterface $repository)
    {
        parent::__construct($files);

        $this->repository = $repository;
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $this->module = $this->argument('module');
        $this->modulePath = $this->repository->getModulePath($this->module);

It seems like $this->module can also be of type null and string[]; however, parameter $module of LaravelModulize\Contracts\ModulizerRepositoryInterface::getModulePath() does only seem to accept string, maybe add an additional type check?

        if (!$this->repository->filesExist($this->modulePath)) {
            $this->call('modulize:make:module', [
                'module' => $this->module,
            ]);
        }

        parent::handle();
    }
}