Rade\DI\Loader\DirectoryLoader::load()

Complexity

Conditions	5
Paths	8

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	5

Importance

Changes

0

<?php
    public function load($file, string $type = null): void
    {
        $file = \rtrim($file, '/');
        $path = $this->locator->locate($file);

        if ($this->container instanceof ContainerBuilder) {
            $this->container->addResource(new FileExistenceResource($file));

The method addResource() does not exist on Rade\DI\AbstractContainer. Since you implemented __call, consider adding a @method annotation.

            $this->container->addResource(new GlobResource($path, '/*', false));

It seems like $path can also be of type array; however, parameter $prefix of Symfony\Component\Config\Resource\GlobResource::__construct() does only seem to accept string, maybe add an additional type check?

        }

        foreach (\scandir($path) as $dir) {

It seems like $path can also be of type array; however, parameter $directory of scandir() does only seem to accept string, maybe add an additional type check?

            if ('.' !== $dir[0]) {
                if (\is_dir($path . '/' . $dir)) {

Are you sure $path of type array|string can be used in concatenation?

                    $dir .= '/'; // append / to allow recursion
                }

                $this->setCurrentDir($path);

It seems like $path can also be of type array; however, parameter $dir of Symfony\Component\Config\Loader\FileLoader::setCurrentDir() does only seem to accept string, maybe add an additional type check?

                $this->import($dir, null, false, $path);

It seems like $path can also be of type array; however, parameter $sourceResource of Symfony\Component\Config\Loader\FileLoader::import() does only seem to accept null|string, maybe add an additional type check?

            }