Let me ping @nicolas-grekas and kindly ask him for a review, because we keep getting more and more pull requests to fix/change the microkernel example and I no longer know if it's right or wrong. Thanks!

This could target 3.4 btw.

Thanks @nicolas-grekas! About the issue in the screenshot, how can I fix it please? I don't know where is the problem in the documentation.

I can't tell precisely without knowing the code. Why don't you extend AbstractController? Or did you forget to enable autoconfiguration? Does adding the container.service_subscriber tag explicitly fix the issue? etc

It's not my code, it's come from the documentation: https://symfony.com/doc/current/configuration/micro_kernel_trait.html#advanced-example-twig-annotations-and-the-web-debug-toolbar.

The tutorial doesn't work and I want to fix it :)

I created an other PR for this fix.

Fyi, I switched the base branch of this PR to 5.1 again. If you switch the base, you also need to rebase your original branch to prevent adding many unrelated commits to this PR. However, we can also switch to 3.4 for you during the merge, so it's fine to keep it like it is now :)

Without testing the code, I think we forget to register the controller as a service in this example. You can try adding something like:

    protected function configureContainer(ContainerBuilder $c, LoaderInterface $loader)
    {
        // ...
        $loader->load(__DIR__.'/../config/services.yaml');

        // ...
    }

And:

# config/services.yaml
services:
    _defaults:
        autowire: true
        autoconfigure: true

    App\:
        resource: '../src/*'

Hello @wouterj,
Thanks for the tips, I didn't know.

Your code works on my side! I just added this missing part in the PR :)

Wouter, if you have some time, could you please merge this PR when it's ready? I don't feel confident enough making changes in this article. Thanks!

Fine by me. I'll do tomorrow!

Tomorrow took a bit longer, but I've now fully tested the example code in this article and updated some more bits. Thanks for working on this @Surfoo!

In e1c35a5, I've done a couple more updates:

• Use the new configurator signature for configureContainer() as well. This removes the need for the Yaml file - it can be done in PHP directly.
• Added some PHP typing
• Fixed some paths in the imports

I've also merged this in 5.1 instead of 3.4. The method signatures in this example require symfony/symfony#34873 which was merged in 5.1. Before 5.1, the RouteCollection and ContainerBuilder classes where used in the microkernel. Those examples are still working fine (problems started when we only updated the type hints, but not the method calls).