Mkk\GearmanBundle\DependencyInjection\GearmanExtension

Complexity

Total Complexity

1

Size/Duplication

Total Lines	56
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	load()	0	43	1

<?php

/**
 * Gearman Bundle for Symfony2 / Symfony3
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * Feel free to edit as you please, and have fun.
 *
 * @author Marc Morera <yuhu@mmoreram.com>
 */

namespace Mkk\GearmanBundle\DependencyInjection;

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

/**
 * This is the class that loads and manages your bundle configuration
 */
class GearmanExtension extends Extension
{
    /**
     * Loads a specific configuration.
     *
     * @param array            $config    An array of configuration values
     * @param ContainerBuilder $container A ContainerBuilder instance
     *
     * @throws \InvalidArgumentException When provided tag is not defined in this extension
     *
     * @api
     */
    public function load(array $config, ContainerBuilder $container)
    {
        $configuration = new Configuration();
        $config = $this->processConfiguration($configuration, $config);

Consider using a different name than the parameter $config. This often makes code more readable.

Equals sign not aligned with surrounding assignments; expected 8 spaces but found 1 space

        /**
         * Setting all config elements as DI parameters to inject them
         */
        $container->setParameter(
            'gearman.bundles',
            $config['bundles']
        );

        $container->setParameter(
            'gearman.servers',
            $config['servers']
        );

        $container->setParameter(
            'gearman.default.settings',
            $config['defaults']
        );

        $container->setParameter(
            'gearman.default.settings.generate_unique_key',
            $config['defaults']['generate_unique_key']
        );

        $loader = new YamlFileLoader(
            $container,
            new FileLocator(__DIR__ . '/../Resources/config')
        );

        /**
         * Loading DI definitions
         */
        $loader->load('classes.yml');
        $loader->load('services.yml');
        $loader->load('commands.yml');
        $loader->load('eventDispatchers.yml');
        $loader->load('generators.yml');
        $loader->load('externals.yml');
    }
}

As per coding style, files should not end with a newline character.