Leverage array_is_list(), get_debug_type(), is_countable(), is_iterable(), str_contains() and str_starts_with()

fancyweb · fancyweb · commit b6f485318e65 · 2022-04-08T18:22:35.000+02:00
diff --git a/Authentication/AuthenticatorManager.php b/Authentication/AuthenticatorManager.php
@@ -91,7 +91,7 @@ public function supports(Request $request): ?bool
         if (null !== $this->logger) {
             $context = ['firewall_name' => $this->firewallName];
 
-            if ($this->authenticators instanceof \Countable || \is_array($this->authenticators)) {
+            if (is_countable($this->authenticators)) {
                 $context['authenticators'] = \count($this->authenticators);
             }
 

Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ public function supports(Request $request): ?bool`
`91`	`91`	`if (null !== $this->logger) {`
`92`	`92`	`$context = ['firewall_name' => $this->firewallName];`
`93`	`93`
`94`		`- if ($this->authenticators instanceof \Countable \|\| \is_array($this->authenticators)) {`
	`94`	`+ if (is_countable($this->authenticators)) {`
`95`	`95`	`$context['authenticators'] = \count($this->authenticators);`
`96`	`96`	`}`
`97`	`97`