Symplify\PHP7_Sculpin\Core\Configuration\YamlFileConfigurationBuilder::internalBuild()

Complexity

Conditions	10
Paths	28

Size

Total Lines	49
Code Lines	26

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	17.2354

Importance

Changes

0

<?php
    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;
    }
}