Kunstmaan\DashboardBundle\DependencyInjection\KunstmaanDashboardExtension

Complexity

Total Complexity

24

Size/Duplication

Total Lines	86
Duplicated Lines	73.26 %

Coupling/Cohesion

Components	1
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	load()	11	11	1
A	prepend()	0	4	1
A	loadGoogleAnalyticsParameters()	0	7	1
A	addClientIdParameter()	13	13	5
A	addClientSecretParameter()	13	13	5
A	addDeveloperKeyParameter()	13	13	5
A	addApplicationNameParameter()	13	13	6

<?php

namespace Kunstmaan\DashboardBundle\DependencyInjection;

use Symfony\Component\Config\FileLocator;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
use Symfony\Component\DependencyInjection\Loader;
use Symfony\Component\HttpKernel\DependencyInjection\Extension;

/**
 * This is the class that loads and manages your bundle configuration
 *
 * To learn more see {@link http://symfony.com/doc/current/cookbook/bundles/extension.html}
 */
class KunstmaanDashboardExtension extends Extension implements PrependExtensionInterface
{
    /**
     * {@inheritdoc}
     */
    public function load(array $configs, ContainerBuilder $container)

This method seems to be duplicated in your project.

    {
        $configuration = new Configuration();
        $config = $this->processConfiguration($configuration, $configs);

        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
        $loader->load('services.yml');
        $loader->load('commands.yml');

        $this->loadGoogleAnalyticsParameters($container, $config);
    }

    public function prepend(ContainerBuilder $container)
    {
        $container->prependExtensionConfig('framework', ['esi' => ['enabled' => true]]);
    }

    private function loadGoogleAnalyticsParameters(ContainerBuilder $container, array $config)
    {
        $this->addApplicationNameParameter($container, $config);
        $this->addClientIdParameter($container, $config);
        $this->addClientSecretParameter($container, $config);
        $this->addDeveloperKeyParameter($container, $config);
    }

    private function addClientIdParameter(ContainerBuilder $container, array $config)

This method seems to be duplicated in your project.

    {
        $clientId = $container->hasParameter('google.api.client_id') ? $container->getParameter('google.api.client_id') : '';
        if (null === $config['google_analytics']['api']['client_id'] && $clientId !== '') {
            @trigger_error('Not providing a value for the "kunstmaan_dashboard.google_analytics.api.client_id" config while setting the "google.api.client_id" parameter is deprecated since KunstmaanDashboardBundle 5.2, this config value will replace the "google.api.client_id" parameter in KunstmaanDashboardBundle 6.0.', E_USER_DEPRECATED);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        }

        if (null !== $config['google_analytics']['api']['client_id']) {
            $clientId = $config['google_analytics']['api']['client_id'];
        }

        $container->setParameter('kunstmaan_dashboard.google_analytics.api.client_id', $clientId);
    }

    private function addClientSecretParameter(ContainerBuilder $container, array $config)

This method seems to be duplicated in your project.

    {
        $clientSecret = $container->hasParameter('google.api.client_secret') ? $container->getParameter('google.api.client_secret') : '';
        if (null === $config['google_analytics']['api']['client_secret'] && $clientSecret !== '') {
            @trigger_error('Not providing a value for the "kunstmaan_dashboard.google_analytics.api.client_secret" config while setting the "google.api.client_secret" parameter is deprecated since KunstmaanDashboardBundle 5.2, this config value will replace the "google.api.client_secret" parameter in KunstmaanDashboardBundle 6.0.', E_USER_DEPRECATED);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        }

        if (null !== $config['google_analytics']['api']['client_secret']) {
            $clientSecret = $config['google_analytics']['api']['client_secret'];
        }

        $container->setParameter('kunstmaan_dashboard.google_analytics.api.client_secret', $clientSecret);
    }

    private function addDeveloperKeyParameter(ContainerBuilder $container, array $config)

This method seems to be duplicated in your project.

    {
        $devKey = $container->hasParameter('google.api.dev_key') ? $container->getParameter('google.api.dev_key') : '';
        if (null === $config['google_analytics']['api']['dev_key'] && $devKey !== '') {
            @trigger_error('Not providing a value for the "kunstmaan_dashboard.google_analytics.api.dev_key" config while setting the "google.api.dev_key" parameter is deprecated since KunstmaanDashboardBundle 5.2, this config value will replace the "google.api.dev_key" parameter in KunstmaanDashboardBundle 6.0.', E_USER_DEPRECATED);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        }

        if (null !== $config['google_analytics']['api']['dev_key']) {
            $devKey = $config['google_analytics']['api']['dev_key'];
        }

        $container->setParameter('kunstmaan_dashboard.google_analytics.api.dev_key', $devKey);
    }

    private function addApplicationNameParameter(ContainerBuilder $container, array $config)

This method seems to be duplicated in your project.

    {
        $appName = $container->hasParameter('google.api.app_name') ? $container->getParameter('google.api.app_name') : 'Kuma Analytics Dashboard';
        if (null === $config['google_analytics']['api']['app_name'] && $appName !== '' && $appName !== 'Kuma Analytics Dashboard') {
            @trigger_error('Not providing a value for the "kunstmaan_dashboard.google_analytics.api.app_name" config while setting the "google.api.app_name" parameter is deprecated since KunstmaanDashboardBundle 5.2, this config value will replace the "google.api.app_name" parameter in KunstmaanDashboardBundle 6.0.', E_USER_DEPRECATED);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        }

        if (null !== $config['google_analytics']['api']['app_name']) {
            $appName = $config['google_analytics']['api']['app_name'];
        }

        $container->setParameter('kunstmaan_dashboard.google_analytics.api.app_name', $appName);
    }
}