Kunstmaan\TranslatorBundle\Toolbar\TranslatorDataCollector

Complexity

Total Complexity

13

Size/Duplication

Total Lines	117
Duplicated Lines	7.69 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	2
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getAccessRoles()	0	4	1
B	collectData()	0	35	4
A	collect()	9	9	3
A	getTemplateData()	0	4	1
A	getName()	0	4	1
A	isEnabled()	0	4	1
A	reset()	0	4	1

<?php

namespace Kunstmaan\TranslatorBundle\Toolbar;

use Kunstmaan\AdminBundle\Helper\Toolbar\AbstractDataCollector;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;

class TranslatorDataCollector extends AbstractDataCollector
{
    /**
     * @var DataCollectorTranslator
     */
    private $translator;

    /**
     * @var UrlGeneratorInterface
     */
    private $urlGenerator;

    /**
     * TranslatorDataCollector constructor.
     *
     * @param DataCollectorTranslator $translator
     * @param UrlGeneratorInterface   $urlGenerator
     */
    public function __construct(DataCollectorTranslator $translator, UrlGeneratorInterface $urlGenerator)
    {
        $this->translator = $translator;
        $this->urlGenerator = $urlGenerator;
    }

    /**
     * @return array

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

     */
    public function getAccessRoles()
    {
        return ['ROLE_ADMIN'];
    }

    /**
     * @return array

Consider making the return type a bit more specific; maybe use array<string,array<string,string|array>>.

     */
    public function collectData()
    {
        $route = $this->urlGenerator->generate('KunstmaanTranslatorBundle_settings_translations');

        $options = [
            'filter_columnname' => [
                'keyword'
            ],
            'filter_uniquefilterid' => [
                1
            ],
            'filter_comparator_1' => 'equals',
            'filter' => 'filter',
        ];

        $translations = [];

        foreach ($this->translator->getCollectedMessages() as $message) {
            if ($message['state'] !== DataCollectorTranslator::MESSAGE_EQUALS_FALLBACK && !empty($message['id'])) {
                $options['filter_value_1'] = $message['id'];
                $translations[$message['id']] = [
                    'id' => $message['id'],
                    'message' => $message['translation'],
                    'route' => $this->urlGenerator->generate('KunstmaanTranslatorBundle_settings_translations', $options)
                ];
            }
        }

        $data = [
            'route' => $route,
            'translations' => $translations
        ];

        return ['data' => $data];
    }

    /**
     * @param Request         $request
     * @param Response        $response
     * @param \Exception|null $exception
     */
    public function collect(Request $request, Response $response, \Exception $exception = null)

This method seems to be duplicated in your project.

    {
        if (!$this->showDataCollection($request, $response) || !$this->isEnabled()) {
            $this->data = false;

It seems like false of type false is incompatible with the declared type array of property $data.

        }
        else {
            $this->data = $this->collectData();
        }
    }

    /**
     * Gets the data for template
     *
     * @return array The request events
     */
    public function getTemplateData()
    {
        return $this->data;
    }

    /**
     * @return string
     */
    public function getName()
    {
        return 'kuma_translation';
    }

    /**
     * @return bool
     */
    public function isEnabled()
    {
        return true;
    }

    public function reset()
    {
        $this->data = [];
    }
}