MagentoDataCollector::getName() - Code Metrics - Inspection of "Change Factory class to use ContainterAwareTrait,..." - smalot/magento-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#4)

unknown

created 2018-04-11 14:16 UTC

MagentoDataCollector::getName() A

↳ Parent: MagentoDataCollector

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/**
 * @project Magento Bridge for Symfony 2.
 *
 * @author  Sébastien MALOT <[email protected]>
 * @license MIT
 * @url     <https://github.com/smalot/magento-bundle>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Smalot\MagentoBundle\DataCollector;

use Smalot\MagentoBundle\Logger\LoggerInterface;
use Smalot\MagentoBundle\Logger\MagentoLogger;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\DataCollector\DataCollector;

/**
 * Class MagentoDataCollector
 *
 * @package Smalot\MagentoBundle\Collector
 */
class MagentoDataCollector extends DataCollector
{
    /**
     * @var MagentoLogger
     */
    protected $logger;

    /**
     * @param LoggerInterface $logger
     */
    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
    }

    /**
     * @param Request    $request
     * @param Response   $response
     * @param \Exception $exception
     */
    public function collect(Request $request, Response $response, \Exception $exception = null)
    {
        if ($this->logger) {
            $this->data = array('magento' => $this->logger->getCalls());
        } else {
            $this->data = array('magento' => array());
        }
    }

    /**
     * Return total duration in seconds.
     *
     * @return float
     */
    public function getTime()
    {
        $time = 0.0;

        foreach ($this->data['magento'] as $log) {
            $time += $log['time'];
        }

        return $time;
    }

    /**
     * @return array
     */
    public function getCalls()
    {
        $calls = array();

        foreach ($this->data['magento'] as $call) {
            $calls[$call['connection']][] = $call;
        }

        return $calls;
    }

    /**
     * @return int
     */
    public function getCallCount()
    {
        return count($this->data['magento']);
    }

    /**
     * @return int
     */
    public function getInvalidEntityCount()
    {
        return 0;
    }

    /**
     * @return string
     */
    public function getName()
    {
        return 'magento';
    }
    
    /**
     * 
     */
    public function reset() {
        $this->data = array();
    }
}


1			<?php
2
3			/**
4			* @project Magento Bridge for Symfony 2.
5			*
6			* @author Sébastien MALOT <[email protected]>
7			* @license MIT
8			* @url <https://github.com/smalot/magento-bundle>
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 Smalot\MagentoBundle\DataCollector;
15
16			use Smalot\MagentoBundle\Logger\LoggerInterface;
17			use Smalot\MagentoBundle\Logger\MagentoLogger;
18			use Symfony\Component\HttpFoundation\Request;
19			use Symfony\Component\HttpFoundation\Response;
20			use Symfony\Component\HttpKernel\DataCollector\DataCollector;
21
22			/**
23			* Class MagentoDataCollector
24			*
25			* @package Smalot\MagentoBundle\Collector
26			*/
27			class MagentoDataCollector extends DataCollector
28			{
29			/**
30			* @var MagentoLogger
31			*/
32			protected $logger;
33
34			/**
35			* @param LoggerInterface $logger
36			*/
37			public function __construct(LoggerInterface $logger)
38			{
39			$this->logger = $logger;
			0 ignored issues – show Documentation Bug introduced 2016-08-14 14:12 UTC by Report Bug Copy Issue Report `$logger` is of type `object<Smalot\MagentoBun...Logger\LoggerInterface>`, but the property `$logger` was declared to be of type `object<Smalot\MagentoBundle\Logger\MagentoLogger>`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
40			}
41
42			/**
43			* @param Request $request
44			* @param Response $response
45			* @param \Exception $exception
46			*/
47			public function collect(Request $request, Response $response, \Exception $exception = null)
48			{
49			if ($this->logger) {
50			$this->data = array('magento' => $this->logger->getCalls());
51			} else {
52			$this->data = array('magento' => array());
53			}
54			}
55
56			/**
57			* Return total duration in seconds.
58			*
59			* @return float
60			*/
61			public function getTime()
62			{
63			$time = 0.0;
64
65			foreach ($this->data['magento'] as $log) {
66			$time += $log['time'];
67			}
68
69			return $time;
70			}
71
72			/**
73			* @return array
74			*/
75			public function getCalls()
76			{
77			$calls = array();
78
79			foreach ($this->data['magento'] as $call) {
80			$calls[$call['connection']][] = $call;
81			}
82
83			return $calls;
84			}
85
86			/**
87			* @return int
88			*/
89			public function getCallCount()
90			{
91			return count($this->data['magento']);
92			}
93
94			/**
95			* @return int
96			*/
97			public function getInvalidEntityCount()
98			{
99			return 0;
100			}
101
102			/**
103			* @return string
104			*/
105			public function getName()
106			{
107			return 'magento';
108			}
109
110			/**
111			*
112			*/
113			public function reset() {
114			$this->data = array();
115			}
116			}
117

smalot / magento-bundle

Pull Request — master (#4)

MagentoDataCollector::getName() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like