ReliqArts\Docweaver\Console\Command\Publish::handle()

Complexity

Conditions	3
Paths	2

Size

Total Lines	22
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function handle(): void
    {
        $productName = $this->argument('product');
        $productSource = $this->argument('source');
        $skipConfirmation = $this->option('y');
        $confirmationMessage = sprintf(
            "This command will attempt to pull documentation for product (%s) from %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?

            $productSource
        );

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

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

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

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

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

            $this->displayResult($result);
        }