Hyde\Console\Commands\PublishConfigsCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	19	1
A	parseTagFromSelection()	0	5	1

<?php

declare(strict_types=1);

namespace Hyde\Console\Commands;

use Hyde\Hyde;
use Hyde\Console\Concerns\Command;
use Illuminate\Support\Facades\Artisan;

use function array_search;
use function sprintf;

/**
 * Publish the Hyde config files.
 */
class PublishConfigsCommand extends Command
{
    /** @var string */
    protected $signature = 'publish:configs';

    /** @var string */
    protected $description = 'Publish the default configuration files';

    public function handle(): int
    {
        $options = [
            'All configs',
            '<comment>hyde-configs</comment>: Main configuration files',
            '<comment>support-configs</comment>: Laravel and package configuration files',
        ];
        $selection = $this->choice('Which configuration files do you want to publish?', $options, 'All configs');

        $tag = $this->parseTagFromSelection($selection, $options);

It seems like $selection can also be of type array; however, parameter $selection of Hyde\Console\Commands\PublishConfigsCommand::parseTagFromSelection() does only seem to accept string, maybe add an additional type check?

        Artisan::call('vendor:publish', [
            '--tag' => $tag,
            '--force' => true,
        ], $this->output);

        $this->infoComment(sprintf('Published config files to [%s]', Hyde::path('config')));

The method path() does not exist on Hyde\Hyde. Since you implemented __callStatic, consider adding a @method annotation.

        return Command::SUCCESS;
    }

    protected function parseTagFromSelection(string $selection, array $options): string
    {
        $tags = ['configs', 'hyde-configs', 'support-configs'];

        return $tags[array_search($selection, $options)];
    }
}