ExceptionListener::onKernelException() - Code Metrics - Inspection of "Merge remote-tracking branch 'upstream/4.0' into d..." - kiy0taka/ec-cube - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — dev/recommend-plugins ( b4eeb7...acd8c2 )

by Kiyotaka

created 2018-09-06 11:16 UTC

ExceptionListener::onKernelException() C

↳ Parent: ExceptionListener

Complexity

Conditions	11
Paths	78

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	11
nc	78
nop	1
dl	0
loc	51
rs	6.9224
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

/*
 * This file is part of EC-CUBE
 *
 * Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
 *
 * http://www.lockon.co.jp/
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Eccube\EventListener;

use Eccube\Request\Context;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Symfony\Component\HttpKernel\KernelEvents;

class ExceptionListener implements EventSubscriberInterface
{
    /**
     * @var \Twig_Environment
     */
    private $twig;

    /**
     * @var Context
     */
    protected $requestContext;

    /**
     * ExceptionListener constructor.
     */
    public function __construct(\Twig_Environment $twig, Context $requestContext)
    {
        $this->twig = $twig;
        $this->requestContext = $requestContext;
    }

    public function onKernelException(GetResponseForExceptionEvent $event)
    {
        $title = 'システムエラーが発生しました。';
        $message = '大変お手数ですが、サイト管理者までご連絡ください。';
        $statusCode = Response::HTTP_INTERNAL_SERVER_ERROR;

        $exception = $event->getException();

        if ($exception instanceof HttpExceptionInterface) {
            $statusCode = $exception->getStatusCode();
            switch ($statusCode) {
                case 400:
                case 401:
                case 403:
                case 405:
                case 406:
                    $title = 'アクセスできません。';
                    if ($exception->getMessage()) {
                        $message = $exception->getMessage();
                    } else {
                        $message = 'お探しのページはアクセスができない状況にあるか、移動もしくは削除された可能性があります。';
                    }
                    break;
                case 404:
                    $title = 'ページがみつかりません。';
                    $message = 'URLに間違いがないかご確認ください。';
                    break;
                default:
                    break;
            }
        }

        log_error('システムエラーが発生しました。', [
            $exception->getMessage(),
            $exception->getFile(),
            $exception->getLine(),
            $exception->getTraceAsString()
        ]);

        try {
            $file = $this->requestContext->isAdmin() ? '@admin/error.twig' : 'error.twig';
            $content = $this->twig->render($file, [
                'error_title' => $title,
                'error_message' => $message,
            ]);
        } catch (\Exception $ignore) {
            $content = $title;
        }

        $event->setResponse(Response::create($content, $statusCode));
    }

    /**
     * Returns an array of event names this subscriber wants to listen to.
     *
     * The array keys are event names and the value can be:
     *
     *  * The method name to call (priority defaults to 0)
     *  * An array composed of the method name to call and the priority
     *  * An array of arrays composed of the method names to call and respective
     *    priorities, or 0 if unset
     *
     * For instance:
     *
     *  * array('eventName' => 'methodName')
     *  * array('eventName' => array('methodName', $priority))
     *  * array('eventName' => array(array('methodName1', $priority), array('methodName2')))
     *
     * @return array The event names to listen to
     */
    public static function getSubscribedEvents()
    {
        return [
            KernelEvents::EXCEPTION => ['onKernelException'],
        ];
    }
}


1			<?php
2
3			/*
4			* This file is part of EC-CUBE
5			*
6			* Copyright(c) LOCKON CO.,LTD. All Rights Reserved.
7			*
8			* http://www.lockon.co.jp/
9			*
10			* For the full copyright and license information, please view the LICENSE
11			* file that was distributed with this source code.
12			*/
13
14			namespace Eccube\EventListener;
15
16			use Eccube\Request\Context;
17			use Symfony\Component\EventDispatcher\EventSubscriberInterface;
18			use Symfony\Component\HttpFoundation\Response;
19			use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
20			use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
21			use Symfony\Component\HttpKernel\KernelEvents;
22
23			class ExceptionListener implements EventSubscriberInterface
24			{
25			/**
26			* @var \Twig_Environment
27			*/
28			private $twig;
29
30			/**
31			* @var Context
32			*/
33			protected $requestContext;
34
35			/**
36			* ExceptionListener constructor.
37			*/
38			public function __construct(\Twig_Environment $twig, Context $requestContext)
39			{
40			$this->twig = $twig;
41			$this->requestContext = $requestContext;
42			}
43
44			public function onKernelException(GetResponseForExceptionEvent $event)
45			{
46			$title = 'システムエラーが発生しました。';
47			$message = '大変お手数ですが、サイト管理者までご連絡ください。';
48			$statusCode = Response::HTTP_INTERNAL_SERVER_ERROR;
49
50			$exception = $event->getException();
51
52			if ($exception instanceof HttpExceptionInterface) {
53			$statusCode = $exception->getStatusCode();
54			switch ($statusCode) {
55			case 400:
56			case 401:
57			case 403:
58			case 405:
59			case 406:
60			$title = 'アクセスできません。';
61			if ($exception->getMessage()) {
62			$message = $exception->getMessage();
63			} else {
64			$message = 'お探しのページはアクセスができない状況にあるか、移動もしくは削除された可能性があります。';
65			}
66			break;
67			case 404:
68			$title = 'ページがみつかりません。';
69			$message = 'URLに間違いがないかご確認ください。';
70			break;
71			default:
72			break;
73			}
74			}
75
76			log_error('システムエラーが発生しました。', [
77			$exception->getMessage(),
78			$exception->getFile(),
79			$exception->getLine(),
80			$exception->getTraceAsString()
81			]);
82
83			try {
84			$file = $this->requestContext->isAdmin() ? '@admin/error.twig' : 'error.twig';
85			$content = $this->twig->render($file, [
86			'error_title' => $title,
87			'error_message' => $message,
88			]);
89			} catch (\Exception $ignore) {
90			$content = $title;
91			}
92
93			$event->setResponse(Response::create($content, $statusCode));
94			}
95
96			/**
97			* Returns an array of event names this subscriber wants to listen to.
98			*
99			* The array keys are event names and the value can be:
100			*
101			* * The method name to call (priority defaults to 0)
102			* * An array composed of the method name to call and the priority
103			* * An array of arrays composed of the method names to call and respective
104			* priorities, or 0 if unset
105			*
106			* For instance:
107			*
108			* * array('eventName' => 'methodName')
109			* * array('eventName' => array('methodName', $priority))
110			* * array('eventName' => array(array('methodName1', $priority), array('methodName2')))
111			*
112			* @return array The event names to listen to
113			*/
114			public static function getSubscribedEvents()
115			{
116			return [
117			KernelEvents::EXCEPTION => ['onKernelException'],
118			];
119			}
120			}
121

kiy0taka / ec-cube

Push — dev/recommend-plugins ( b4eeb7...acd8c2 )

ExceptionListener::onKernelException() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like