minor symfony#61530 [DependencyInjection] add routing.controller to the list of known DI tags (xabbuh)

nicolas-grekas · nicolas-grekas · commit b0f0bdf0b994 · 2025-08-26T14:43:04.000+02:00
This PR was merged into the 7.4 branch. Discussion ---------- [DependencyInjection] add routing.controller to the list of known DI tags | Q | A | ------------- | --- | Branch? | 7.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | | License | MIT re-reading symfony#61492 I wonder if we don't have to add the new tag to this list Commits ------- 593ee8f add routing.controller to the list of known DI tags
diff --git a/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/UnusedTagsPass.php b/src/Symfony/Bundle/FrameworkBundle/DependencyInjection/Compiler/UnusedTagsPass.php
@@ -78,6 +78,7 @@ class UnusedTagsPass implements CompilerPassInterface
         'proxy',
         'remote_event.consumer',
         'routing.condition_service',
+        'routing.controller',
         'routing.expression_language_function',
         'routing.expression_language_provider',
         'routing.loader',
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/UnusedTagsPassUtils.php b/src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/UnusedTagsPassUtils.php
@@ -19,6 +19,7 @@ public static function getDefinedTags(): array
     {
         $tags = [
             'proxy' => true,
+            'routing.controller' => true,
         ];
 
         // get all tags used in XML configs

Original file line number	Diff line number	Diff line change
`@@ -19,6 +19,7 @@ public static function getDefinedTags(): array`
`19`	`19`	`{`
`20`	`20`	`$tags = [`
`21`	`21`	`'proxy' => true,`
	`22`	`+ 'routing.controller' => true,`
`22`	`23`	`];`
`23`	`24`
`24`	`25`	`// get all tags used in XML configs`