Skip to content

Commit 25f4a07

Browse files
committed
minor #710 Remove unnecessary configuration for controllers (yceruto)
This PR was merged into the master branch. Discussion ---------- Remove unnecessary configuration for controllers This configuration is only necessary for service controllers that don't extend from `Controller` or `AbstractController`, which is not the case here. It is not clear in the recipe (in this sense) but the documentation is clear enough: > http://symfony.com/doc/current/controller.html#fetching-services-as-controller-arguments > If you're not using the default configuration, you can tag your service manually with `controller.service_arguments`. Commits ------- 1682f19 Remove unnecessary configuration for controllers
2 parents ccb2cc7 + 1682f19 commit 25f4a07

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

config/services.yaml

-6
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,6 @@ services:
3030
# but if a service is unused, it's removed anyway
3131
exclude: '../src/{DataFixtures,Entity,Migrations,Tests}'
3232

33-
# controllers are imported separately to make sure they're public
34-
# and have a tag that allows actions to type-hint services
35-
App\Controller\:
36-
resource: '../src/Controller'
37-
tags: ['controller.service_arguments']
38-
3933
# when the service definition only contains arguments, you can omit the
4034
# 'arguments' key and define the arguments just below the service class
4135
App\EventSubscriber\CommentNotificationSubscriber:

0 commit comments

Comments
 (0)