AuditLoggerChain::stopQuery() - Code Metrics - Inspection of "Nested LoggerChains causing big call stack" - DamienHarper/DoctrineAuditBundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#72)

unknown

created 2019-09-05 12:46 UTC

AuditLoggerChain::stopQuery() A

↳ Parent: AuditLoggerChain

Complexity

Conditions	2
Paths	2

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	2
c	1
b	0
f	0
nc	2
nop	0
dl	0
loc	4
rs	10

<?php
/**
 * Created by PhpStorm.
 * User: [email protected]
 * Date: 8/14/19
 * Time: 3:50 PM
 */

namespace DH\DoctrineAuditBundle\DBAL;


use Doctrine\DBAL\Logging\SQLLogger;

class AuditLoggerChain implements SQLLogger
{
    /** @var SQLLogger[] */
    private $loggers = [];

    /**
     * Adds a logger in the chain.
     *
     * @param SQLLogger $logger
     * @return void
     */
    public function addLogger(SQLLogger $logger)
    {
        $this->loggers[] = $logger;
    }

    /**
     * {@inheritdoc}
     */
    public function startQuery($sql, ?array $params = null, ?array $types = null)
    {
        foreach ($this->loggers as $logger) {
            $logger->startQuery($sql, $params, $types);
        }
    }

    /**
     * {@inheritdoc}
     */
    public function stopQuery()
    {
        foreach ($this->loggers as $logger) {
            $logger->stopQuery();
        }
    }

    /**
     * @return SQLLogger[]
     */
    public function getLoggers()
    {
        return $this->loggers;
    }
}

1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: [email protected]
5			* Date: 8/14/19
6			* Time: 3:50 PM
7			*/
8
9			namespace DH\DoctrineAuditBundle\DBAL;
10
11
12			use Doctrine\DBAL\Logging\SQLLogger;
13
14			class AuditLoggerChain implements SQLLogger
15			{
16			/** @var SQLLogger[] */
17			private $loggers = [];
18
19			/**
20			* Adds a logger in the chain.
21			*
22			* @param SQLLogger $logger
23			* @return void
24			*/
25			public function addLogger(SQLLogger $logger)
26			{
27			$this->loggers[] = $logger;
28			}
29
30			/**
31			* {@inheritdoc}
32			*/
33			public function startQuery($sql, ?array $params = null, ?array $types = null)
34			{
35			foreach ($this->loggers as $logger) {
36			$logger->startQuery($sql, $params, $types);
37			}
38			}
39
40			/**
41			* {@inheritdoc}
42			*/
43			public function stopQuery()
44			{
45			foreach ($this->loggers as $logger) {
46			$logger->stopQuery();
47			}
48			}
49
50			/**
51			* @return SQLLogger[]
52			*/
53			public function getLoggers()
54			{
55			return $this->loggers;
56			}
57			}

DamienHarper / DoctrineAuditBundle

Pull Request — master (#72)

AuditLoggerChain::stopQuery() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like