CleaniqueCoders\Console\Hook\MakeHookCommand

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	7	1
A	execute()	0	11	2

<?php

namespace CleaniqueCoders\Console\Hook;

use CleaniqueCoders\Console\Commander;
use Illuminate\Support\Str;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;

class MakeHookCommand extends Commander
{
    /**
     * Configure the command options.
     */
    protected function configure()
    {
        $this
            ->setName('hook')
            ->setDescription('Hook package to a Laravel project locally')
            ->addArgument('name', InputArgument::REQUIRED)
            ->addArgument('to', InputArgument::REQUIRED);
    }

    /**
     * Execute the command.
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $name      = $input->getArgument('name');
        $pathOrUrl = $input->getArgument('to');

        $status = $this->composerLink(Str::snake($name), $pathOrUrl);

It seems like $name can also be of type string[]; however, parameter $value of Illuminate\Support\Str::snake() does only seem to accept string, maybe add an additional type check?

It seems like $pathOrUrl can also be of type string[]; however, parameter $pathOrUrl of CleaniqueCoders\Console\Commander::composerLink() does only seem to accept string, maybe add an additional type check?

        if ($status) {

The condition $status is always true.

            $output->writeln('<info>Packaged linked.</info>');
        } else {
            $output->writeln('<comment>Unable to link the package.</comment>');
        }
    }
}