Yiisoft\Yii\Console\CommandLoader

Complexity

Total Complexity

8

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Test Coverage

Coverage

88%

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getCommandInstance()	0	8	2
A	get()	0	17	2
A	has()	0	3	2
A	getNames()	0	3	1

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Console;

use Psr\Container\ContainerInterface;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Command\LazyCommand;
use Symfony\Component\Console\CommandLoader\CommandLoaderInterface;
use Symfony\Component\Console\Exception\CommandNotFoundException;

final class CommandLoader implements CommandLoaderInterface
{
    private ContainerInterface $container;

    /**
     * @var array<string, string>
     */
    private array $commandMap;

    /**
     * @param array<string, string> $commandMap An array with command names as keys and service ids as values
     */
    public function __construct(ContainerInterface $container, array $commandMap)
    {
        $this->container = $container;
        $this->commandMap = $commandMap;
    }

    /**
     * {@inheritdoc}
     */
    public function get(string $name)
    {
        if (!$this->has($name)) {
            throw new CommandNotFoundException(sprintf('Command "%s" does not exist.', $name));
        }

        /** @var Command $commandClass */
        $commandClass = $this->commandMap[$name];
        $description = $commandClass::getDefaultDescription() ?? $this->getCommandInstance($name)->getName();

        return new LazyCommand(
            $name,
            [],
            $description,

It seems like $description can also be of type null; however, parameter $description of Symfony\Component\Console\Command\LazyCommand::__construct() does only seem to accept string, maybe add an additional type check?

            false,
            function () use ($name) {
                return $this->getCommandInstance($name);
            }
        );
    }

    private function getCommandInstance(string $name): Command
    {
        $command = $this->container->get($this->commandMap[$name]);
        if ($command->getName() !== $name) {
            $command->setName($name);
        }

        return $command;
    }

    /**
     * {@inheritdoc}
     */
    public function has(string $name)
    {
        return isset($this->commandMap[$name]) && $this->container->has($this->commandMap[$name]);
    }

    /**
     * {@inheritdoc}
     */
    public function getNames()
    {
        return array_keys($this->commandMap);
    }
}