Add autowire for resource Controllers

[AdamKasp]

Q	A
Bug fix?	no?
New feature?	yes
BC breaks?	no
Deprecations?	no
License	MIT

[lchrusciel]

lint:container command should be added to pipeline

[loic425]

@lchrusciel @AdamKasp FYI, there is a hook here
https://github.com/Sylius/Sylius-Standard/blob/a9ec0c5ee46d3f61ad661414fb580bf16493358b/config/services.yaml#L20

[lchrusciel]

Hey Loic,
thanks for the advice. I've totally forgotten about your fix and it was pretty good. However, it works only when the people will customize it in their end application, and from what I've checked lint:container command will still complain about resource controllers:

In addition, with this fix, you will be able to use the FQCN of controllers in routing:

app_book_sortable_index:
    path: /sortable-books/
    methods: [GET]
    defaults:
        _controller: \App\Controller\BookController:indexAction
        _sylius:
            sortable: true

@AdamKasp can you add config to your PR? It should be changed in src/Bundle/test/config/routes.yaml, line 15th to be exact.

[loic425]

Hey Loic,
thanks for the advice. I've totally forgotten about your fix and it was pretty good. However, it works only when the people will customize it in their end application, [...]

@lchrusciel Yes, I just wanted you have this little hook on your mind :)

[pamil]

Why can't we use aliases in Symfony 4.4?

[AdamKasp]

Because we need here public aliases which will be provided in Symfony 5.2

[pamil]

It's available in Symfony 5.1 as well.

[pamil]

Could we change the comment? Or maybe put some if to find out if Symfony 5.1 is installed and then using an alias instead of copied definition?

Changes have been applied

[GSadee]

Thanks, Adam! 🎉