Potievdev\SlimRbac\Component\Config\RbacConfigLoader::loadConfigs()

Complexity

Conditions	2
Paths	2

Size

Total Lines	17
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public static function loadConfigs(): array
    {
        $configDirectories = [
            __DIR__ . '/../../../config',
            __DIR__ . '/../../../../../',
            __DIR__ . '/../../../../../config',
        ];

        $fileLocator = new FileLocator($configDirectories);
        $fileName = $fileLocator->locate('sr_config.yaml');

        if ($fileName === null) {

The condition $fileName === null is always false.

            throw ConfigNotFoundException::configFileNotFound($configDirectories);
        }

        return (new Processor())
            ->processConfiguration(new RbacConfigStructure(), Yaml::parseFile($fileName));

It seems like $fileName can also be of type string[]; however, parameter $filename of Symfony\Component\Yaml\Yaml::parseFile() does only seem to accept string, maybe add an additional type check?

    }
}