Kunstmaan\AdminBundle\EventListener\ConsoleExceptionSubscriber

Complexity

Total Complexity

5

Size/Duplication

Total Lines	55
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	4	4	1
A	getSubscribedEvents()	6	6	1
A	onConsoleError()	9	9	2
A	logCommandError()	12	12	1

<?php

namespace Kunstmaan\AdminBundle\EventListener;

use Psr\Log\LoggerInterface;
use Symfony\Component\Console\ConsoleEvents;
use Symfony\Component\Console\Event\ConsoleErrorEvent;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;

/**
 * Class ConsoleExceptionSubscriber.
 */
final class ConsoleExceptionSubscriber implements EventSubscriberInterface

This class seems to be duplicated in your project.

{
    /** @var LoggerInterface */
    private $logger;

    /**
     * ConsoleExceptionListener constructor.
     *
     * @param LoggerInterface $logger
     */
    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;
    }

    /**
     * @return array

Consider making the return type a bit more specific; maybe use array<*,string>.

     */
    public static function getSubscribedEvents()
    {
        return [
            ConsoleEvents::ERROR => 'onConsoleError',
        ];
    }

    /**
     * @param ConsoleErrorEvent $event
     */
    public function onConsoleError(ConsoleErrorEvent $event)
    {
        $command = $event->getCommand();
        $error = $event->getError();

        if (null !== $command) {
            $this->logCommandError($command, $error);
        }
    }

    /**
     * @param $command
     * @param $error
     */
    private function logCommandError($command, $error)
    {
        $message = sprintf(
            '%s: %s (uncaught error) at %s line %s while running console command `%s`',
            \get_class($error),
            $error->getMessage(),
            $error->getFile(),
            $error->getLine(),
            $command->getName()
        );
        $this->logger->critical($message, ['error' => $error]);
    }
}