minor #50889 [Security] Add native return type to Firewall::getSubscribedEvents() (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit e869eb6d98c2 · 2023-07-05T15:05:44.000+02:00
This PR was merged into the 6.4 branch. Discussion ---------- [Security] Add native return type to `Firewall::getSubscribedEvents()` | Q | A | ------------- | --- | Branch? | 6.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Required to make #50852 green without breaking compat between v6 and v7. Commits ------- 8069f49 [Security] Add native return type to `Firewall::getSubscribedEvents()`
diff --git a/src/Symfony/Component/Security/Http/CHANGELOG.md b/src/Symfony/Component/Security/Http/CHANGELOG.md
@@ -5,6 +5,7 @@ CHANGELOG
 ---
 
  * `UserValueResolver` no longer implements `ArgumentValueResolverInterface`
+ * Add native return type to `Firewall::getSubscribedEvents()` 
 
 6.3
 ---
diff --git a/src/Symfony/Component/Security/Http/Firewall.php b/src/Symfony/Component/Security/Http/Firewall.php
@@ -108,10 +108,7 @@ public function onKernelFinishRequest(FinishRequestEvent $event)
         }
     }
 
-    /**
-     * @return array
-     */
-    public static function getSubscribedEvents()
+    public static function getSubscribedEvents(): array
     {
         return [
             KernelEvents::REQUEST => ['onKernelRequest', 8],

Original file line number	Diff line number	Diff line change
`@@ -108,10 +108,7 @@ public function onKernelFinishRequest(FinishRequestEvent $event)`
`108`	`108`	`}`
`109`	`109`	`}`
`110`	`110`
`111`		`- /**`
`112`		`- * @return array`
`113`		`- */`
`114`		`- public static function getSubscribedEvents()`
	`111`	`+ public static function getSubscribedEvents(): array`
`115`	`112`	`{`
`116`	`113`	`return [`
`117`	`114`	`KernelEvents::REQUEST => ['onKernelRequest', 8],`