ReliqArts\Docweaver\Console\Command\Update

Complexity

Total Complexity

3

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	20	3

<?php

declare(strict_types=1);

namespace ReliqArts\Docweaver\Console\Command;

use Carbon\Carbon;
use Illuminate\Console\Command;

class Update extends SingleProductCommand
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'docweaver:update
                            {product? : Product name}
                            {--y|y : Whether to skip confirmation}
                            ';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Update documentation for product';

    /**
     * Execute the console command.
     */
    public function handle(): void
    {
        $productName = $this->argument('product');
        $skipConfirmation = $this->option('y');
        $confirmationMessage = sprintf(
            "This command will attempt to update documentation for product (%s). \n"
            . 'Please ensure your internet connection is stable. Ready?',
            $productName

It seems like $productName can also be of type string[]; however, parameter $args of sprintf() does only seem to accept string, maybe add an additional type check?

        );

        $this->comment(PHP_EOL
            . "<info>♣♣♣</info> Docweaver DocumentationPublisher \n"
            . 'Help is here, try: php artisan docweaver:update --help');

        if ($skipConfirmation || $this->confirm($confirmationMessage)) {
            $this->info("Updating {$productName}.\nT: " . Carbon::now()->toCookieString() . "\n----------");

            $result = $this->publisher->update($productName, $this);

It seems like $productName can also be of type null and string[]; however, parameter $productName of ReliqArts\Docweaver\Contract\Documentation\Publisher::update() does only seem to accept string, maybe add an additional type check?

            $this->displayResult($result);
        }
    }
}