Listener::onKernelResponse() - Code Metrics - Inspection of "rerun scrutinizer" - zicht/url-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — up-php7 ( a2fe11...4e1c1e )

by Erik

created 2018-01-22 10:57 UTC

Listener::onKernelResponse() A

↳ Parent: Listener

Complexity

Conditions	4
Paths	3

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	4

Importance

Changes

Metric	Value
cc	4
eloc	5
nc	3
nop	1
dl	0
loc	8
ccs	6
cts	6
cp	1
crap	4
rs	9.2
c	0
b	0
f	0

<?php
/**
 * @author Gerard van Helden <[email protected]>
 * @copyright Zicht Online <http://zicht.nl>
 */
namespace Zicht\Bundle\UrlBundle\Logging;

use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
use Symfony\Component\HttpKernel\HttpKernelInterface;

/**
 * Listener used for logging
 */
class Listener
{
    /**
     * @var \Zicht\Bundle\UrlBundle\Entity\ErrorLog
     */
    protected $log = null;

    /**
     * Construct with the passed logger service
     *
     * @param Logging $logging
     */
    public function __construct(Logging $logging)
    {
        $this->logging = $logging;

    }


    /**
     * Create log entry if a kernelexception occurs.
     *
     * @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event
     * @return void
     */
    public function onKernelException(GetResponseForExceptionEvent $event)
    {
        $this->log = $this->logging->createLog($event->getRequest(), $event->getException()->getMessage());
    }

    /**
     * Save the log entry (if any) if the error response about to be sent is not handled otherwise.
     *
     * @param \Symfony\Component\HttpKernel\Event\FilterResponseEvent $e
     * @return void
     */
    public function onKernelResponse(FilterResponseEvent $e)
    {
        if (isset($this->log)) {
            if ($e->getRequestType() === HttpKernelInterface::MASTER_REQUEST
                && ($status = $e->getResponse()->getStatusCode()) >= 400
            ) {
                $this->log->setStatus($status);
                $this->logging->flush($this->log);
            }
        }
    }
}


1		<?php
2		/**
3		* @author Gerard van Helden <[email protected]>
4		* @copyright Zicht Online <http://zicht.nl>
5		*/
6		namespace Zicht\Bundle\UrlBundle\Logging;
7
8		use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
9		use Symfony\Component\HttpKernel\Event\FilterResponseEvent;
10		use Symfony\Component\HttpKernel\HttpKernelInterface;
11
12		/**
13		* Listener used for logging
14		*/
15		class Listener
16		{
17		/**
18		* @var \Zicht\Bundle\UrlBundle\Entity\ErrorLog
19		*/
20		protected $log = null;
21
22		/**
23		* Construct with the passed logger service
24		*
25		* @param Logging $logging
26		*/
27	1	public function __construct(Logging $logging)
28		{
29	1	$this->logging = $logging;
		0 ignored issues – show Bug Best Practice introduced 2018-01-19 11:22 UTC by Report Bug Copy Issue Report The property `logging` does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly. Loading history...
30	1	}
31
32
33		/**
34		* Create log entry if a kernelexception occurs.
35		*
36		* @param \Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent $event
37		* @return void
38		*/
39	1	public function onKernelException(GetResponseForExceptionEvent $event)
40		{
41	1	$this->log = $this->logging->createLog($event->getRequest(), $event->getException()->getMessage());
42	1	}
43
44		/**
45		* Save the log entry (if any) if the error response about to be sent is not handled otherwise.
46		*
47		* @param \Symfony\Component\HttpKernel\Event\FilterResponseEvent $e
48		* @return void
49		*/
50	1	public function onKernelResponse(FilterResponseEvent $e)
51		{
52	1	if (isset($this->log)) {
53	1	if ($e->getRequestType() === HttpKernelInterface::MASTER_REQUEST
54	1	&& ($status = $e->getResponse()->getStatusCode()) >= 400
55		) {
56	1	$this->log->setStatus($status);
57	1	$this->logging->flush($this->log);
58		}
59		}
60	1	}
61		}
62

zicht / url-bundle

Push — up-php7 ( a2fe11...4e1c1e )

Listener::onKernelResponse() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like