MagentoDataCollector::getName()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 4

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
dl 0
loc 4
rs 10
c 0
b 0
f 0
cc 1
nc 1
nop 0
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 by
$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