divineniiquaye / rade-di

src/Loader/DirectoryLoader.php

<?php

declare(strict_types=1);

/*
 * This file is part of DivineNii opensource projects.
 *
 * PHP version 7.4 and above required
 *
 * @author    Divine Niiquaye Ibok <divineibok@gmail.com>
 * @copyright 2021 DivineNii (https://divinenii.com/)
 * @license   https://opensource.org/licenses/BSD-3-Clause License
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Rade\DI\Loader;

use Rade\DI\ContainerBuilder;
use Symfony\Component\Config\Resource\FileExistenceResource;
use Symfony\Component\Config\Resource\GlobResource;

/**
 * DirectoryLoader is a recursive loader to go through directories.
 *
 * @author Divine Niiquaye Ibok <divineibok@gmail.com>
 */
class DirectoryLoader extends FileLoader
{
    /**
     * {@inheritdoc}
     */
    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?

            }
        }
    }

    /**
     * {@inheritdoc}
     */
    public function supports($resource, string $type = null)
    {
        if ('directory' === $type) {
            return true;
        }

        return null === $type && \is_string($resource) && '/' === \substr($resource, -1);
    }
}