minor #35213 [Security\Guard] Fix missing typehints (chalasr)

fabpot · fabpot · commit de34f229a9f5 · 2020-01-07T20:10:35.000+01:00
This PR was merged into the 4.3 branch. Discussion ---------- [Security\Guard] Fix missing typehints | Q | A | ------------- | --- | Branch? | 4.3 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Commits ------- 2ef8771 [Security\Guard] Fix missing typehints
diff --git a/src/Symfony/Component/Security/Guard/Firewall/GuardAuthenticationListener.php b/src/Symfony/Component/Security/Guard/Firewall/GuardAuthenticationListener.php
@@ -51,7 +51,7 @@ class GuardAuthenticationListener implements ListenerInterface
      * @param iterable|AuthenticatorInterface[] $guardAuthenticators   The authenticators, with keys that match what's passed to GuardAuthenticationProvider
      * @param LoggerInterface                   $logger                A LoggerInterface instance
      */
-    public function __construct(GuardAuthenticatorHandler $guardHandler, AuthenticationManagerInterface $authenticationManager, string $providerKey, $guardAuthenticators, LoggerInterface $logger = null)
+    public function __construct(GuardAuthenticatorHandler $guardHandler, AuthenticationManagerInterface $authenticationManager, string $providerKey, iterable $guardAuthenticators, LoggerInterface $logger = null)
     {
         if (empty($providerKey)) {
             throw new \InvalidArgumentException('$providerKey must not be empty.');
diff --git a/src/Symfony/Component/Security/Guard/Provider/GuardAuthenticationProvider.php b/src/Symfony/Component/Security/Guard/Provider/GuardAuthenticationProvider.php
@@ -45,7 +45,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface
      * @param UserProviderInterface             $userProvider        The user provider
      * @param string                            $providerKey         The provider (i.e. firewall) key
      */
-    public function __construct($guardAuthenticators, UserProviderInterface $userProvider, string $providerKey, UserCheckerInterface $userChecker)
+    public function __construct(iterable $guardAuthenticators, UserProviderInterface $userProvider, string $providerKey, UserCheckerInterface $userChecker)
     {
         $this->guardAuthenticators = $guardAuthenticators;
         $this->userProvider = $userProvider;

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ class GuardAuthenticationListener implements ListenerInterface`
`51`	`51`	`* @param iterable\|AuthenticatorInterface[] $guardAuthenticators The authenticators, with keys that match what's passed to GuardAuthenticationProvider`
`52`	`52`	`* @param LoggerInterface $logger A LoggerInterface instance`
`53`	`53`	`*/`
`54`		`- public function __construct(GuardAuthenticatorHandler $guardHandler, AuthenticationManagerInterface $authenticationManager, string $providerKey, $guardAuthenticators, LoggerInterface $logger = null)`
	`54`	`+ public function __construct(GuardAuthenticatorHandler $guardHandler, AuthenticationManagerInterface $authenticationManager, string $providerKey, iterable $guardAuthenticators, LoggerInterface $logger = null)`
`55`	`55`	`{`
`56`	`56`	`if (empty($providerKey)) {`
`57`	`57`	`throw new \InvalidArgumentException('$providerKey must not be empty.');`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ class GuardAuthenticationProvider implements AuthenticationProviderInterface`
`45`	`45`	`* @param UserProviderInterface $userProvider The user provider`
`46`	`46`	`* @param string $providerKey The provider (i.e. firewall) key`
`47`	`47`	`*/`
`48`		`- public function __construct($guardAuthenticators, UserProviderInterface $userProvider, string $providerKey, UserCheckerInterface $userChecker)`
	`48`	`+ public function __construct(iterable $guardAuthenticators, UserProviderInterface $userProvider, string $providerKey, UserCheckerInterface $userChecker)`
`49`	`49`	`{`
`50`	`50`	`$this->guardAuthenticators = $guardAuthenticators;`
`51`	`51`	`$this->userProvider = $userProvider;`