BrainExe\Core\Console\ListServicesCommand

Complexity

Total Complexity

10

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	7

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	6	1
A	__construct()	0	6	1
A	execute()	0	21	3
B	addDefinition()	0	23	5

<?php

namespace BrainExe\Core\Console;

use BrainExe\Core\Annotations\Command as CommandAnnotation;
use BrainExe\Core\DependencyInjection\Rebuild;
use BrainExe\Core\Traits\EventDispatcherTrait;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Helper\Table;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;

/**
 * @CommandAnnotation
 */
class ListServicesCommand extends Command
{

    const TYPE_PRIVATE   = 'private';
    const TYPE_PROTECTED = 'protected';
    const TYPE_PUBLIC    = 'public';

    use EventDispatcherTrait;

    /**
     * @var Rebuild
     */
    private $rebuild;

    /**
     * @var ContainerBuilder
     */
    private $container;

    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this->setName('debug:list:services')
            ->setDescription('List all services')
            ->addArgument('visibility', InputArgument::OPTIONAL, 'public, protected or private');
    }

    /**
     * @param Rebuild $rebuild
     */
    public function __construct(Rebuild $rebuild)
    {
        $this->rebuild = $rebuild;

        parent::__construct();
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $this->container = $this->rebuild->buildContainer();

        $table = new Table($output);
        $table->setHeaders(['service-id', 'tags', 'visibility']);

        $ids = $this->container->getServiceIds();

        sort($ids);

        $visibility = $input->getArgument('visibility');

        foreach ($ids as $id) {
            if ($this->container->hasDefinition($id)) {

It seems like you code against a specific sub-type and not the parent class Symfony\Component\DependencyInjection\Container as the method hasDefinition() does only exist in the following sub-classes of Symfony\Component\DependencyInjection\Container: Container14\ProjectServiceContainer, Symfony\Component\DependencyInjection\ContainerBuilder. Maybe you want to instanceof check for one of these explicitly?

                $this->addDefinition($id, $table, $visibility);
            }
        }

        $table->render();
    }

    /**
     * @param string $id
     * @param Table $table
     * @param bool $restrictedVisibility
     */
    private function addDefinition($id, Table $table, $restrictedVisibility)
    {
        $definition = $this->container->getDefinition($id);

        if (!$definition->isPublic()) {
            $currentVisibility = self::TYPE_PRIVATE;
            $color = 'error';
        } elseif (strpos($id, '__') === 0) {
            $currentVisibility = self::TYPE_PROTECTED;
            $color = 'comment';
        } else {
            $currentVisibility = self::TYPE_PUBLIC;
            $color = 'info';
        }

        if (!$restrictedVisibility || $restrictedVisibility === $currentVisibility) {
            $table->addRow([
                $id,
                implode(', ', array_keys($definition->getTags())),
                "<$color>$currentVisibility</$color>"

As per coding-style, please use concatenation or sprintf for the variable $color instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $currentVisibility instead of interpolation.

            ]);
        }
    }
}