TomPHP\ConfigServiceProvider\ConfigureContainer

Complexity

Total Complexity

6

Size/Duplication

Total Lines	75
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	4
Bugs	0	Features	2

4 Methods

Rating	Name	Duplication	Size	Complexity
A	fromFiles()	0	6	1
A	fromArray()	0	6	1
A	prepareSettings()	0	13	1
A	configureContainer()	0	21	3

<?php

namespace TomPHP\ConfigServiceProvider;

final class ConfigureContainer
{
    /**
     * @api
     *
     * @param object $container
     * @param array  $patterns
     * @param array  $settings
     *
     * @return void
     */
    public static function fromFiles($container, array $patterns, $settings = [])
    {
        $settings = self::prepareSettings($settings);

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

        $appConfig = ApplicationConfig::fromFiles($patterns, $settings['config_separator']);
        self::configureContainer($container, $appConfig, $settings);
    }

    /**
     * @api
     *
     * @param object $container
     * @param array  $config
     * @param array  $settings
     *
     * @return void
     */
    public static function fromArray($container, array $config, $settings = [])
    {
        $settings = self::prepareSettings($settings);

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

        $appConfig = new ApplicationConfig($config, $settings['config_separator']);
        self::configureContainer($container, $appConfig, $settings);
    }

    /**
     * @param array $settings
     *
     * @return array
     */
    private static function prepareSettings(array $settings)
    {
        return array_merge(
            [
                'config_prefix'      => 'config',
                'config_separator'   => '.',
                'services_key'       => 'di.services',
                'inflectors_key'     => 'di.inflectors',
                'singleton_services' => false,
            ],
            $settings
        );
    }

    private static function configureContainer($container, ApplicationConfig $appConfig, array $settings)

Please use __construct() instead of a PHP4-style constructor that is named after the class.

    {
        $factory = new ConfiguratorFactory([
            'League\Container\Container' => 'TomPHP\ConfigServiceProvider\League\Configurator',
            'Pimple\Container'           => 'TomPHP\ConfigServiceProvider\Pimple\Configurator',
        ]);

        $configurator = $factory->create($container);

        $configurator->addApplicationConfig($appConfig, $settings['config_prefix']);

        if (isset($appConfig[$settings['services_key']])) {
            $configurator->addServiceConfig(
                new ServiceConfig($appConfig[$settings['services_key']], $settings['singleton_services'])
            );
        }

        if (isset($appConfig[$settings['inflectors_key']])) {
            $configurator->addInflectorConfig(new InflectorConfig($appConfig[$settings['inflectors_key']]));
        }
    }
}