symfony · nicolas-grekas · Dec 13, 2019 · Dec 7, 2019
diff --git a/UPGRADE-5.1.md b/UPGRADE-5.1.md
@@ -4,8 +4,7 @@ UPGRADE FROM 5.0 to 5.1
 FrameworkBundle
 ---------------

- * Marked `MicroKernelTrait::configureRoutes()` as `@internal` and `@final`.
- * Deprecated not overriding `MicroKernelTrait::configureRouting()`.
+ * Deprecated passing a `RouteCollectionBuiler` to `MicroKernelTrait::configureRoutes()`, type-hint `RoutingConfigurator` instead

 HttpFoundation
 --------------

diff --git a/UPGRADE-6.0.md b/UPGRADE-6.0.md
@@ -4,8 +4,7 @@ UPGRADE FROM 5.x to 6.0
 FrameworkBundle
 ---------------

- * Removed `MicroKernelTrait::configureRoutes()`.
- * Made `MicroKernelTrait::configureRouting()` abstract.
+ * `MicroKernelTrait::configureRoutes()` is now always called with a `RoutingConfigurator`

 HttpFoundation
 --------------

diff --git a/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md b/src/Symfony/Bundle/FrameworkBundle/CHANGELOG.md
@@ -4,10 +4,10 @@ CHANGELOG
 5.1.0
 -----

- * Marked `MicroKernelTrait::configureRoutes()` as `@internal` and `@final`.
- * Deprecated not overriding `MicroKernelTrait::configureRouting()`.
+ * Made `MicroKernelTrait::configureContainer()` compatible with `ContainerConfigurator`
 * Added a new `mailer.message_bus` option to configure or disable the message bus to use to send mails.
 * Added flex-compatible default implementations for `MicroKernelTrait::registerBundles()` and `getProjectDir()`
+ * Deprecated passing a `RouteCollectionBuiler` to `MicroKernelTrait::configureRoutes()`, type-hint `RoutingConfigurator` instead

 5.0.0
 -----

diff --git a/src/Symfony/Bundle/FrameworkBundle/Kernel/MicroKernelTrait.php b/src/Symfony/Bundle/FrameworkBundle/Kernel/MicroKernelTrait.php
@@ -13,8 +13,10 @@

 use Symfony\Component\Config\Loader\LoaderInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
+use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator;
+use Symfony\Component\Routing\RouteCollection;
 use Symfony\Component\Routing\RouteCollectionBuilder;

 /**
@@ -25,20 +27,6 @@
 */
 trait MicroKernelTrait
 {
-    /**
-     * Add or import routes into your application.
-     *
-     *     $routes->import('config/routing.yml');
-     *     $routes->add('/admin', 'App\Controller\AdminController::dashboard', 'admin_dashboard');
-     *
-     * @final since Symfony 5.1, override configureRouting() instead
-     *
-     * @internal since Symfony 5.1, use configureRouting() instead
-     */
-    protected function configureRoutes(RouteCollectionBuilder $routes)
-    {
-    }
-
    /**
     * Adds or imports routes into your application.
     *
@@ -48,29 +36,26 @@ protected function configureRoutes(RouteCollectionBuilder $routes)
     *         ->controller('App\Controller\AdminController::dashboard')
     *     ;
     */
-    protected function configureRouting(RoutingConfigurator $routes): void
-    {
-        @trigger_error(sprintf('Not overriding the "%s()" method is deprecated since Symfony 5.1 and will trigger a fatal error in 6.0.', __METHOD__), E_USER_DEPRECATED);
-    }
+    abstract protected function configureRoutes(RoutingConfigurator $routes);

    /**
     * Configures the container.
     *
     * You can register extensions:
     *
-     *     $c->loadFromExtension('framework', [
+     *     $c->extension('framework', [
     *         'secret' => '%secret%'
     *     ]);
     *
     * Or services:
     *
-     *     $c->register('halloween', 'FooBundle\HalloweenProvider');
+     *     $c->services()->set('halloween', 'FooBundle\HalloweenProvider');
     *
     * Or parameters:
     *
-     *     $c->setParameter('halloween', 'lot of fun');
+     *     $c->parameters()->set('halloween', 'lot of fun');
     */
-    abstract protected function configureContainer(ContainerBuilder $c, LoaderInterface $loader);
+    abstract protected function configureContainer(ContainerConfigurator $c);

    /**
     * {@inheritdoc}
@@ -120,9 +105,31 @@ public function registerContainerConfiguration(LoaderInterface $loader)
                $kernelDefinition->addTag('kernel.event_subscriber');
            }

-            $this->configureContainer($container, $loader);
            $container->addObjectResource($this);
            $container->fileExists($this->getProjectDir().'/config/bundles.php');
+
+            try {
+                $this->configureContainer($container, $loader);
+
+                return;
+            } catch (\TypeError $e) {
+                $file = $e->getFile();
+
+                if (0 !== strpos($e->getMessage(), sprintf('Argument 1 passed to %s::configureContainer() must be an instance of %s,', static::class, ContainerConfigurator::class))) {
+                    throw $e;
+                }
+            }
+
+            $kernelLoader = $loader->getResolver()->resolve($file);
+            $kernelLoader->setCurrentDir(\dirname($file));
+            $instanceof = &\Closure::bind(function &() { return $this->instanceof; }, $kernelLoader, $kernelLoader)();
+
+            try {
+                $this->configureContainer(new ContainerConfigurator($container, $kernelLoader, $instanceof, $file, $file), $loader);
+            } finally {
+                $instanceof = [];
+                $kernelLoader->registerAliasesForSinglyImplementedInterfaces();
+            }
        });
    }

@@ -131,17 +138,26 @@ public function registerContainerConfiguration(LoaderInterface $loader)
     */
    public function loadRoutes(LoaderInterface $loader)
    {
-        $routes = new RouteCollectionBuilder($loader);
-        $this->configureRoutes($routes);
-        $collection = $routes->build();
+        $file = (new \ReflectionObject($this))->getFileName();
+        $kernelLoader = $loader->getResolver()->resolve($file);
+        $kernelLoader->setCurrentDir(\dirname($file));
+        $collection = new RouteCollection();

-        if (0 !== \count($collection)) {
-            @trigger_error(sprintf('Adding routes via the "%s:configureRoutes()" method is deprecated since Symfony 5.1 and will have no effect in 6.0; use "configureRouting()" instead.', self::class), E_USER_DEPRECATED);
+        try {
+            $this->configureRoutes(new RoutingConfigurator($collection, $kernelLoader, $file, $file));
+
+            return $collection;
+        } catch (\TypeError $e) {
+            if (0 !== strpos($e->getMessage(), sprintf('Argument 1 passed to %s::configureRoutes() must be an instance of %s,', static::class, RouteCollectionBuilder::class))) {
+                throw $e;
+            }
        }

-        $file = (new \ReflectionObject($this))->getFileName();
-        $this->configureRouting(new RoutingConfigurator($collection, $loader, null, $file));
+        @trigger_error(sprintf('Using type "%s" for argument 1 of method "%s:configureRoutes()" is deprecated since Symfony 5.1, use "%s" instead.', RouteCollectionBuilder::class, self::class, RoutingConfigurator::class), E_USER_DEPRECATED);
+
+        $routes = new RouteCollectionBuilder($loader);
+        $this->configureRoutes($routes);

-        return $collection;
+        return $routes->build();
    }
 }
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/ConcreteMicroKernel.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/ConcreteMicroKernel.php
@@ -80,7 +80,7 @@ public function __destruct()
        $fs->remove($this->cacheDir);
    }

-    protected function configureRouting(RoutingConfigurator $routes): void
+    protected function configureRoutes(RoutingConfigurator $routes): void
    {
        $routes->add('halloween', '/')->controller('kernel::halloweenAction');
        $routes->add('danger', '/danger')->controller('kernel::dangerousAction');

diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/MicroKernelTraitTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/MicroKernelTraitTest.php
@@ -19,18 +19,6 @@

 class MicroKernelTraitTest extends TestCase
 {
-    /**
-     * @group legacy
-     * @expectedDeprecation Adding routes via the "Symfony\Bundle\FrameworkBundle\Tests\Kernel\MicroKernelWithConfigureRoutes:configureRoutes()" method is deprecated since Symfony 5.1 and will have no effect in 6.0; use "configureRouting()" instead.
-     * @expectedDeprecation Not overriding the "Symfony\Bundle\FrameworkBundle\Kernel\MicroKernelTrait::configureRouting()" method is deprecated since Symfony 5.1 and will trigger a fatal error in 6.0.
-     */
-    public function testConfigureRoutingDeprecated()
-    {
-        $kernel = new MicroKernelWithConfigureRoutes('test', false);
-        $kernel->boot();
-        $kernel->handle(Request::create('/'));
-    }
-
    public function test()
    {
        $kernel = new ConcreteMicroKernel('test', false);

diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/MicroKernelWithConfigureRoutes.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Kernel/MicroKernelWithConfigureRoutes.php
diff --git a/src/Symfony/Bundle/FrameworkBundle/composer.json b/src/Symfony/Bundle/FrameworkBundle/composer.json
@@ -27,7 +27,7 @@
        "symfony/polyfill-mbstring": "~1.0",
        "symfony/filesystem": "^4.4|^5.0",
        "symfony/finder": "^4.4|^5.0",
-        "symfony/routing": "^5.1"
+        "symfony/routing": "^5.0"
    },
    "require-dev": {
        "doctrine/annotations": "~1.7",

diff --git a/src/Symfony/Component/Routing/CHANGELOG.md b/src/Symfony/Component/Routing/CHANGELOG.md
@@ -5,7 +5,6 @@ CHANGELOG
 -----

 * Deprecated `RouteCollectionBuilder` in favor of `RoutingConfigurator`.
- * Added support for a generic loader to `RoutingConfigurator`.

 5.0.0
 -----

diff --git a/src/Symfony/Component/Routing/Loader/Configurator/RoutingConfigurator.php b/src/Symfony/Component/Routing/Loader/Configurator/RoutingConfigurator.php
@@ -11,9 +11,7 @@

 namespace Symfony\Component\Routing\Loader\Configurator;

-use Symfony\Component\Config\Exception\LoaderLoadException;
-use Symfony\Component\Config\Loader\FileLoader;
-use Symfony\Component\Config\Loader\LoaderInterface;
+use Symfony\Component\Routing\Loader\PhpFileLoader;
 use Symfony\Component\Routing\RouteCollection;

 /**
@@ -27,7 +25,7 @@ class RoutingConfigurator
    private $path;
    private $file;

-    public function __construct(RouteCollection $collection, LoaderInterface $loader, ?string $path, string $file)
+    public function __construct(RouteCollection $collection, PhpFileLoader $loader, string $path, string $file)
    {
        $this->collection = $collection;
        $this->loader = $loader;
@@ -40,7 +38,9 @@ public function __construct(RouteCollection $collection, LoaderInterface $loader
     */
    final public function import($resource, string $type = null, bool $ignoreErrors = false, $exclude = null): ImportConfigurator
    {
-        $imported = $this->load($resource, $type, $ignoreErrors, $exclude) ?: [];
+        $this->loader->setCurrentDir(\dirname($this->path));
+
+        $imported = $this->loader->import($resource, $type, $ignoreErrors, $this->file, $exclude) ?: [];
        if (!\is_array($imported)) {
            return new ImportConfigurator($this->collection, $imported);
        }
@@ -57,34 +57,4 @@ final public function collection(string $name = ''): CollectionConfigurator
    {
        return new CollectionConfigurator($this->collection, $name);
    }
-
-    /**
-     * @param string|string[]|null $exclude
-     *
-     * @return RouteCollection|RouteCollection[]|null
-     */
-    private function load($resource, ?string $type, bool $ignoreErrors, $exclude)
-    {
-        $loader = $this->loader;
-
-        if (!$loader->supports($resource, $type)) {
-            if (null === $resolver = $loader->getResolver()) {
-                throw new LoaderLoadException($resource, $this->file, null, null, $type);
-            }
-
-            if (false === $loader = $resolver->resolve($resource, $type)) {
-                throw new LoaderLoadException($resource, $this->file, null, null, $type);
-            }
-        }
-
-        if (!$loader instanceof FileLoader) {
-            return $loader->load($resource, $type);
-        }
-
-        if (null !== $this->path) {
-            $this->loader->setCurrentDir(\dirname($this->path));
-        }
-
-        return $this->loader->import($resource, $type, $ignoreErrors, $this->file, $exclude);
-    }
 }
diff --git a/src/Symfony/Component/Routing/RouteCollectionBuilder.php b/src/Symfony/Component/Routing/RouteCollectionBuilder.php
@@ -14,6 +14,9 @@
 use Symfony\Component\Config\Exception\LoaderLoadException;
 use Symfony\Component\Config\Loader\LoaderInterface;
 use Symfony\Component\Config\Resource\ResourceInterface;
+use Symfony\Component\Routing\Loader\Configurator\RoutingConfigurator;
+
+@trigger_error(sprintf('The "%s" class is deprecated since Symfony 5.1, use "%s" instead.', RouteCollectionBuilder::class, RoutingConfigurator::class), E_USER_DEPRECATED);

 /**
 * Helps add and import routes into a RouteCollection.