Eccube\Log\Monolog\Listener\EccubeListener

Complexity

Total Complexity

15

Size/Duplication

Total Lines	126
Duplicated Lines	21.43 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	onKernelRequestEarly()	0	8	2
A	onKernelRequest()	9	9	2
A	onKernelController()	9	9	2
A	onKernelResponse()	9	9	2
A	onKernelTerminate()	0	5	1
A	onKernelException()	0	17	3
A	getSubscribedEvents()	0	20	1
A	getRoute()	0	4	1

<?php

namespace Eccube\Log\Monolog\Listener;

use Eccube\Log\Log;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Event\PostResponseEvent;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Symfony\Component\HttpKernel\KernelEvents;

/**
 * EccubeListener
 */
class EccubeListener implements EventSubscriberInterface
{

    protected $log;

    public function __construct(Log $log)

Missing function doc comment

    {
        $this->log = $log;
    }

    /**
     * @param GetResponseEvent $event
     */
    public function onKernelRequestEarly(GetResponseEvent $event)
    {
        if (!$event->isMasterRequest()) {
            return;
        }

        $this->log->info('INIT');
    }

    /**
     * @param GetResponseEvent $event
     */
    public function onKernelRequest(GetResponseEvent $event)

This method seems to be duplicated in your project.

    {
        if (!$event->isMasterRequest()) {
            return;
        }

        $route = $this->getRoute($event->getRequest());
        $this->log->info('PROCESS START', array($route));
    }

    /**
     * @param FilterControllerEvent $event
     */
    public function onKernelController(FilterControllerEvent $event)

This method seems to be duplicated in your project.

    {
        if (!$event->isMasterRequest()) {
            return;
        }

        $route = $this->getRoute($event->getRequest());
        $this->log->info('LOGIC START', array($route));
    }

    /**
     * @param FilterResponseEvent $event
     */
    public function onKernelResponse(FilterResponseEvent $event)

This method seems to be duplicated in your project.

    {
        if (!$event->isMasterRequest()) {
            return;
        }

        $route = $this->getRoute($event->getRequest());
        $this->log->info('LOGIC END', array($route));
    }

    /**
     * @param PostResponseEvent $event
     */
    public function onKernelTerminate(PostResponseEvent $event)
    {
        $route = $this->getRoute($event->getRequest());
        $this->log->info('PROCESS END', array($route));
    }

    /**
     * @param GetResponseForExceptionEvent $event
     */
    public function onKernelException(GetResponseForExceptionEvent $event)
    {
        $e = $event->getException();
        if ($e instanceof HttpExceptionInterface && $e->getStatusCode() < 500) {
            $this->log->info($e->getMessage(), array($e->getStatusCode()));

Blank line found at end of control structure

        } else {
            $message = sprintf(
                '%s: %s (uncaught exception) at %s line %s',
                get_class($e),
                $e->getMessage(),
                $e->getFile(),
                $e->getLine()
            );
            $this->log->error($message, array('exception' => $e));
        }
    }

    /**
     * {@inheritdoc}
     */
    public static function getSubscribedEvents()
    {
        return array(

            KernelEvents::REQUEST => array(
                // Application::initRenderingで、フロント/管理画面の判定が行われた後に実行
                array('onKernelRequestEarly', 500),
                // SecurityServiceProviderで、認証処理が完了した後に実行.
                array('onKernelRequest', 6),
            ),
            KernelEvents::RESPONSE => array('onKernelResponse', 0),
            KernelEvents::CONTROLLER => array('onKernelController', 0),
            KernelEvents::TERMINATE => array('onKernelTerminate', 0),
            /*
             * Priority -4 is used to come after those from SecurityServiceProvider (0)
             * but before the error handlers added with Silex\Application::error (defaults to -8)
             */
            KernelEvents::EXCEPTION => array('onKernelException', -4),
        );
    }

    /**
     * ルーティング名を取得する.
     *
     * @param $request
     * @return string
     */
    private function getRoute($request)
    {
        return $request->attributes->get('_route');
    }
}