Issues (1)

src/Kernel.php (1 issue)

1
<?php
2
3
namespace App;
4
5
use Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait;
6
use Symfony\Component\Config\Loader\LoaderInterface;
7
use Symfony\Component\Config\Resource\FileResource;
8
use Symfony\Component\DependencyInjection\ContainerBuilder;
9
use Symfony\Component\HttpKernel\Kernel as BaseKernel;
10
use Symfony\Component\Routing\RouteCollectionBuilder;
11
12
class Kernel extends BaseKernel
13
{
14
    use MicroKernelTrait;
15
16
    private const CONFIG_EXTS = '.{php,xml,yaml,yml}';
17
18
    public function registerBundles(): iterable
19
    {
20
        $contents = require $this->getProjectDir().'/config/bundles.php';
21
        foreach ($contents as $class => $envs) {
22
            if ($envs[$this->environment] ?? $envs['all'] ?? false) {
23
                yield new $class();
0 ignored issues
show
Bug Best Practice introduced by
The expression yield new $class() returns the type Generator which is incompatible with the return type mandated by Symfony\Component\HttpKe...face::registerBundles() of Symfony\Component\HttpKe...dleInterface[]|iterable.

In the issue above, the returned value is violating the contract defined by the mentioned interface.

Let's take a look at an example:

interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
Loading history...
24
            }
25
        }
26
    }
27
28
    public function getProjectDir(): string
29
    {
30
        return \dirname(__DIR__);
31
    }
32
33
    protected function configureContainer(ContainerBuilder $container, LoaderInterface $loader): void
34
    {
35
        $container->addResource(new FileResource($this->getProjectDir().'/config/bundles.php'));
36
        $container->setParameter('container.dumper.inline_class_loader', \PHP_VERSION_ID < 70400 || !ini_get('opcache.preload'));
37
        $container->setParameter('container.dumper.inline_factories', true);
38
        $confDir = $this->getProjectDir().'/config';
39
40
        $loader->load($confDir.'/{packages}/*'.self::CONFIG_EXTS, 'glob');
41
        $loader->load($confDir.'/{packages}/'.$this->environment.'/*'.self::CONFIG_EXTS, 'glob');
42
        $loader->load($confDir.'/{services}'.self::CONFIG_EXTS, 'glob');
43
        $loader->load($confDir.'/{services}_'.$this->environment.self::CONFIG_EXTS, 'glob');
44
    }
45
46
    protected function configureRoutes(RouteCollectionBuilder $routes): void
47
    {
48
        $confDir = $this->getProjectDir().'/config';
49
50
        $routes->import($confDir.'/{routes}/'.$this->environment.'/*'.self::CONFIG_EXTS, '/', 'glob');
51
        $routes->import($confDir.'/{routes}/*'.self::CONFIG_EXTS, '/', 'glob');
52
        $routes->import($confDir.'/{routes}'.self::CONFIG_EXTS, '/', 'glob');
53
    }
54
}
55