Yiisoft\Yii\Debug\Collector\CommandCollector

Complexity

Total Complexity

9

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	reset()	0	3	1
A	getCollected()	0	3	1
A	getIndexData()	0	6	1
B	collect()	0	28	6

<?php

declare(strict_types=1);

namespace Yiisoft\Yii\Debug\Collector;

use Symfony\Component\Console\Event\ConsoleCommandEvent;
use Symfony\Component\Console\Event\ConsoleErrorEvent;
use Symfony\Component\Console\Event\ConsoleTerminateEvent;

final class CommandCollector implements CollectorInterface, IndexCollectorInterface
{
    use CollectorTrait;

    private array $commands = [];

    public function getCollected(): array
    {
        return $this->commands;
    }

    public function collect(object $event): void
    {
        if (!is_object($event) || !$this->isActive()) {
            return;
        }

        if ($event instanceof ConsoleCommandEvent) {
            $this->commands[get_class($event)] = [
                'command' => $event->getCommand(),
                'input' => $event->getInput()->__toString(),

The method __toString() does not exist on Symfony\Component\Console\Input\InputInterface. It seems like you code against a sub-type of said class. However, the method does not exist in Symfony\Component\Console\Input\StreamableInputInterface or Symfony\Component\Console\Input\Input or Symfony\Component\Console\Input\Input. Are you sure you never get one of those?

                'output' => $event->getOutput()->fetch(),

The method fetch() does not exist on Symfony\Component\Console\Output\OutputInterface. It seems like you code against a sub-type of Symfony\Component\Console\Output\OutputInterface such as Symfony\Component\Console\Output\BufferedOutput or Symfony\Component\Console\Output\TrimmedBufferOutput or Yiisoft\Yii\Console\Output\ConsoleBufferedOutput or Yiisoft\Yii\Console\Output\ConsoleBufferedOutput.

            ];
        }
        if ($event instanceof ConsoleErrorEvent) {
            $this->commands[get_class($event)] = [
                'command' => $event->getCommand(),
                'input' => $event->getInput()->__toString(),
                'output' => $event->getOutput()->fetch(),
                'error' => $event->getError()->getMessage(),
                'exitCode' => $event->getExitCode(),
            ];
        }
        if ($event instanceof ConsoleTerminateEvent) {
            $this->commands[get_class($event)] = [
                'command' => $event->getCommand(),
                'input' => $event->getInput()->__toString(),
                'output' => $event->getOutput()->fetch(),
                'exitCode' => $event->getExitCode(),
            ];
        }
    }

    public function getIndexData(): array
    {
        $command = $this->commands[ConsoleCommandEvent::class];
        return [
            'command' => $command['input'],
            'commandClass' => get_class($command['command']),
        ];
    }

    private function reset(): void

The method reset() is not used, and could be removed.

    {
        $this->commands = [];
    }
}