Smalot\MagentoBundle\DataCollector\MagentoDataCollector

Complexity

Total Complexity

11

Size/Duplication

Total Lines	90
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	collect()	0	8	2
A	getTime()	0	10	2
A	getCalls()	0	10	2
A	getCallCount()	0	4	1
A	getInvalidEntityCount()	0	4	1
A	getName()	0	4	1
A	reset()	0	3	1

<?php

/**
 * @project Magento Bridge for Symfony 2.
 *
 * @author  Sébastien MALOT <sebastien@malot.fr>
 * @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;

$logger is of type object<Smalot\MagentoBundle\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?

    }

    /**
     * @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();
    }
}