Only show relevant columns in debug:router

mamazu · mamazu · commit 6eb665ab3259 · 2025-08-18T12:21:39.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php b/src/Symfony/Bundle/FrameworkBundle/Console/Descriptor/TextDescriptor.php
@@ -41,6 +41,7 @@ class TextDescriptor extends Descriptor
     private const VERB_COLORS = [
         'ANY' => 'default',
         'GET' => 'blue',
+        'QUERY' => 'blue',
         'HEAD' => 'magenta',
         'OPTIONS' => 'blue',
         'POST' => 'green',
@@ -84,7 +85,7 @@ protected function describeRouteCollection(RouteCollection $routes, array $optio
             }
 
             if ($showAliases) {
-                $row['Aliases'] = implode('|', ($this->getReverseAliases($routes))[$name] ?? []);
+                $row['Aliases'] = implode('|', $this->getReverseAliases($routes)[$name] ?? []);
             }
 
             $tableRows[] = $row;
@@ -616,12 +617,12 @@ private function formatRouterConfig(array $config): string
      */
     private function formatMethods(array $methods): string
     {
-        if (count($methods) === 0) {
+        if ([] === $methods) {
             $methods = ['ANY'];
         }
 
         return implode('|', array_map(
-            fn (string $method): string => sprintf('<fg=%s>%s</>', self::VERB_COLORS[$method] ?? 'default', $method),
+            fn (string $method): string => \sprintf('<fg=%s>%s</>', self::VERB_COLORS[$method] ?? 'default', $method),
             $methods
         ));
     }

Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,7 @@ class TextDescriptor extends Descriptor`
`41`	`41`	`private const VERB_COLORS = [`
`42`	`42`	`'ANY' => 'default',`
`43`	`43`	`'GET' => 'blue',`
	`44`	`+ 'QUERY' => 'blue',`
`44`	`45`	`'HEAD' => 'magenta',`
`45`	`46`	`'OPTIONS' => 'blue',`
`46`	`47`	`'POST' => 'green',`
`@@ -84,7 +85,7 @@ protected function describeRouteCollection(RouteCollection $routes, array $optio`
`84`	`85`	`}`
`85`	`86`
`86`	`87`	`if ($showAliases) {`
`87`		`- $row['Aliases'] = implode('\|', ($this->getReverseAliases($routes))[$name] ?? []);`
	`88`	`+ $row['Aliases'] = implode('\|', $this->getReverseAliases($routes)[$name] ?? []);`
`88`	`89`	`}`
`89`	`90`
`90`	`91`	`$tableRows[] = $row;`
`@@ -616,12 +617,12 @@ private function formatRouterConfig(array $config): string`
`616`	`617`	`*/`
`617`	`618`	`private function formatMethods(array $methods): string`
`618`	`619`	`{`
`619`		`- if (count($methods) === 0) {`
	`620`	`+ if ([] === $methods) {`
`620`	`621`	`$methods = ['ANY'];`
`621`	`622`	`}`
`622`	`623`
`623`	`624`	`return implode('\|', array_map(`
`624`		`- fn (string $method): string => sprintf('<fg=%s>%s</>', self::VERB_COLORS[$method] ?? 'default', $method),`
	`625`	`+ fn (string $method): string => \sprintf('<fg=%s>%s</>', self::VERB_COLORS[$method] ?? 'default', $method),`
`625`	`626`	`$methods`
`626`	`627`	`));`
`627`	`628`	`}`