Symplify\DefaultAutowire\Config\Definition\ConfigurationResolver

Complexity

Total Complexity

2

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

88.89%

Importance

Changes	3
Bugs	1	Features	2

1 Method

Rating	Name	Duplication	Size	Complexity
A	resolveFromContainerBuilder()	0	15	2

<?php

/*
 * This file is part of Symplify
 * Copyright (c) 2016 Tomas Votruba (http://tomasvotruba.cz).
 */

namespace Symplify\DefaultAutowire\Config\Definition;

use Symfony\Component\Config\Definition\Processor;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symplify\DefaultAutowire\SymplifyDefaultAutowireBundle;

final class ConfigurationResolver
{
    /**
     * @var string[]
     */
    private $resolvedConfiguration;

    /**
     * {@inheritdoc}
     */
    public function resolveFromContainerBuilder(ContainerBuilder $containerBuilder)
    {
        if ($this->resolvedConfiguration) {

The expression $this->resolvedConfiguration of type string[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            return $this->resolvedConfiguration;
        }

        $processor = new Processor();
        $configs = $containerBuilder->getExtensionConfig(SymplifyDefaultAutowireBundle::ALIAS);
        $configs = $processor->processConfiguration(new Configuration(), $configs);

        $this->resolvedConfiguration = $containerBuilder->getParameterBag()
            ->resolveValue($configs);

        return $this->resolvedConfiguration;
    }
}