Shopware\Psh\Config\ConfigFactory

Complexity

Total Complexity

13

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	gatherConfigs()	0	32	5
A	loadImports()	0	27	5
A	mergeConfigs()	0	10	2

<?php declare(strict_types=1);

namespace Shopware\Psh\Config;

use function array_merge;
use function array_pop;
use function array_shift;
use function count;
use function glob;
use function is_dir;

class ConfigFactory
{
    /**
     * @var ConfigMerger
     */
    private $configMerger;

    /**
     * @var ConfigFileFinder
     */
    private $configFileFinder;

    /**
     * @var ConfigLoader[]
     */
    private $configLoaders;

    public function __construct(
        ConfigMerger $configMerger,
        ConfigFileFinder $configFileFinder,
        array $configLoaders
    ) {
        $this->configMerger = $configMerger;
        $this->configFileFinder = $configFileFinder;
        $this->configLoaders = $configLoaders;
    }

    /**
     * @return Config[]
     */
    public function gatherConfigs(
        string $fromPath,
        array $configFiles,
        array $overwrittenConsts
    ): array {
        $configs = [];
        $additionalConfigs = [[]];

        foreach ($configFiles as $configFile) {
            foreach ($this->configLoaders as $configLoader) {
                if (!$configLoader->isSupported($configFile)) {
                    continue;
                }

                $config = $configLoader
                    ->load($configFile, $overwrittenConsts);

                $additionalConfigs[] = $this
                    ->loadImports($config, $fromPath, $overwrittenConsts);

                $configs[] = $config;
            }
        }

        if (!$configs) {

The expression $configs 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.

            return [];
        }

        $config = $this->configMerger->mergeOverride(...$configs);

$configs is of type array, but the function expects a object<Shopware\Psh\Config\Config>.

        return array_merge([$config], ...array_merge(...$additionalConfigs));
    }

    private function loadImports(
        Config $config,
        string $fromPath,
        array $overwrittenConsts
    ): array {
        $additionalConfigs = [];

        foreach ($config->getImports() as $importPath) {
            foreach (glob($fromPath . '/' . $importPath) as $foundOccurrence) {
                if (!is_dir($foundOccurrence)) {
                    continue;
                }

                $foundConfigFiles = $this->configFileFinder
                    ->discoverConfigInDirectory($foundOccurrence);

                if (count($foundConfigFiles) === 0) {
                    continue;
                }

                $additionalConfigs[] = $this
                    ->gatherConfigs($foundOccurrence, $foundConfigFiles, $overwrittenConsts);
            }
        }

        return $additionalConfigs;
    }

    /**
     * @param Config[] $configs
     */
    public function mergeConfigs(array $configs): Config
    {
        $mainConfig = array_shift($configs);

        while (0 !== count($configs)) {
            $mainConfig = $this->configMerger->mergeImport($mainConfig, array_pop($configs));

It seems like $mainConfig can be null; however, mergeImport() does not accept null, maybe add an additional type check?

        }

        return $mainConfig;
    }
}