Symplify\PHP7_Sculpin\Configuration\YamlFileConfigurationBuilder

Complexity

Total Complexity

11

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

62.96%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
C	internalBuild()	0	49	10

<?php

/*
 * This file is a part of Sculpin.
 *
 * (c) Dragonfly Development Inc.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Symplify\PHP7_Sculpin\Configuration;

use Dflydev\DotAccessConfiguration\AbstractConfigurationBuilder;
use Dflydev\DotAccessConfiguration\ConfigurationInterface;
use Dflydev\DotAccessData\Util as DotAccessDataUtil;
use Symfony\Component\Yaml\Yaml;

final class YamlFileConfigurationBuilder extends AbstractConfigurationBuilder
{
    /**
     * @var string[]
     */
    private $yamlConfigurationFilenames;

    public function __construct(array $yamlConfigurationFilenames)
    {
        $this->yamlConfigurationFilenames = $yamlConfigurationFilenames;
    }

    /**
     * {@inheritdoc}.
     */
    public function internalBuild(ConfigurationInterface $configuration)
    {
        $config = [];
        $imports = [];
        foreach ($this->yamlConfigurationFilenames as $yamlConfigurationFilename) {
            if (file_exists($yamlConfigurationFilename)) {
                $config = DotAccessDataUtil::mergeAssocArray(
                    $config, Yaml::parse(file_get_contents($yamlConfigurationFilename))
                );

                if (isset($config['imports'])) {
                    foreach ((array) $config['imports'] as $file) {
                        if (0 === strpos($file, '/')) {
                            // Absolute path
                            $imports[] = $file;
                        } else {
                            if ($realpath = realpath(dirname($yamlConfigurationFilename).'/'.$file)) {
                                $imports[] = $realpath;
                            }
                        }
                    }
                }
            }
        }

        if ($imports) {

The expression $imports of type array 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.

            $importsBuilder = new static($imports);

            // We want to reconfigure the imports builder to have the
            // same basic configuration as this instance.
            $this->reconfigure($importsBuilder);

            $configuration->import($importsBuilder->build());

            $internalImports = $configuration->get('imports');
        } else {
            $internalImports = null;
        }

        $configuration->importRaw($config);

        if ($internalImports) {
            foreach ((array) $internalImports as $import) {
                $configuration->append('imports', $import);
            }
        }

        return $configuration;
    }
}