Kunstmaan\AdminBundle\Helper\Toolbar\AbstractDataCollector

Complexity

Total Complexity

12

Size/Duplication

Total Lines	70
Duplicated Lines	8.57 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getTemplate()	0	4	1
A	setTemplate()	0	6	1
A	setAdminRouteHelper()	0	4	1
B	showDataCollection()	6	18	9

<?php

namespace Kunstmaan\AdminBundle\Helper\Toolbar;

use Kunstmaan\AdminBundle\Helper\AdminRouteHelper;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\DataCollector\DataCollector as BaseDataCollector;

/**
 * AbstractDataCollector.
 */
abstract class AbstractDataCollector extends BaseDataCollector implements DataCollectionInterface
{
    /**
     * @var array
     */
    protected $data = [];

    /**
     * @var string
     */
    private $template;

    /**
     * @var AdminRouteHelper
     */
    protected $adminRouteHelper;

    /**
     * @return mixed

Consider making the return type a bit more specific; maybe use string.

     */
    public function getTemplate()
    {
        return $this->template;
    }

    /**
     * @param $template
     *
     * @return $this
     */
    public function setTemplate($template)
    {
        $this->template = $template;

        return $this;
    }

    /**
     * @param AdminRouteHelper $adminRouteHelper
     */
    public function setAdminRouteHelper(AdminRouteHelper $adminRouteHelper)
    {
        $this->adminRouteHelper = $adminRouteHelper;
    }

    /**
     * @param Request  $request
     * @param Response $response
     *
     * @return bool
     */
    public function showDataCollection(Request $request, Response $response)
    {
        $url = $request->getRequestUri();

        // do not capture redirects or modify XML HTTP Requests
        if ($request->isXmlHttpRequest() || $this->adminRouteHelper->isAdminRoute($url) || !$this->isEnabled()) {
            return false;
        }

        if ($response->isRedirection() || ($response->headers->has('Content-Type') && false === strpos($response->headers->get('Content-Type'), 'html'))

This code seems to be duplicated across your project.

            || 'html' !== $request->getRequestFormat()
            || false !== stripos($response->headers->get('Content-Disposition'), 'attachment;')
        ) {
            return false;
        }

        return true;
    }
}