Add missing PHPdoc return types

Author: wouterj

Debug/TraceableFirewallListener.php

@@ -28,6 +28,9 @@ final class TraceableFirewallListener extends FirewallListener
     private array $wrappedListeners = [];
     private array $authenticatorsInfo = [];
 
+    /**
+     * @return array
+     */
     public function getWrappedListeners()
     {
         return $this->wrappedListeners;

DependencyInjection/Security/Factory/AbstractFactory.php

@@ -14,7 +14,6 @@
 use Symfony\Component\Config\Definition\Builder\NodeDefinition;
 use Symfony\Component\DependencyInjection\ChildDefinition;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
-use Symfony\Component\DependencyInjection\Reference;
 
 /**
  * AbstractFactory is the base class for all classes inheriting from
@@ -73,6 +72,9 @@ public function addConfiguration(NodeDefinition $node)
         }
     }
 
+    /**
+     * @return string
+     */
     protected function createAuthenticationSuccessHandler(ContainerBuilder $container, string $id, array $config)
     {
         $successHandlerId = $this->getSuccessHandlerId($id);
@@ -92,6 +94,9 @@ protected function createAuthenticationSuccessHandler(ContainerBuilder $containe
         return $successHandlerId;
     }
 
+    /**
+     * @return string
+     */
     protected function createAuthenticationFailureHandler(ContainerBuilder $container, string $id, array $config)
     {
         $id = $this->getFailureHandlerId($id);
@@ -109,11 +114,17 @@ protected function createAuthenticationFailureHandler(ContainerBuilder $containe
         return $id;
     }
 
+    /**
+     * @return string
+     */
     protected function getSuccessHandlerId(string $id)
     {
         return 'security.authentication.success_handler.'.$id.'.'.str_replace('-', '_', $this->getKey());
     }
 
+    /**
+     * @return string
+     */
     protected function getFailureHandlerId(string $id)
     {
         return 'security.authentication.failure_handler.'.$id.'.'.str_replace('-', '_', $this->getKey());

DependencyInjection/Security/UserProvider/InMemoryFactory.php

@@ -37,6 +37,9 @@ public function create(ContainerBuilder $container, string $id, array $config)
         $definition->addArgument($users);
     }
 
+    /**
+     * @return string
+     */
     public function getKey()
     {
         return 'memory';

DependencyInjection/Security/UserProvider/LdapFactory.php

@@ -40,6 +40,9 @@ public function create(ContainerBuilder $container, string $id, array $config)
         ;
     }
 
+    /**
+     * @return string
+     */
     public function getKey()
     {
         return 'ldap';

Security/FirewallContext.php

@@ -38,6 +38,9 @@ public function __construct(iterable $listeners, ExceptionListener $exceptionLis
         $this->config = $config;
     }
 
+    /**
+     * @return FirewallConfig|null
+     */
     public function getConfig()
     {
         return $this->config;
@@ -51,11 +54,17 @@ public function getListeners(): iterable
         return $this->listeners;
     }
 
+    /**
+     * @return ExceptionListener|null
+     */
     public function getExceptionListener()
     {
         return $this->exceptionListener;
     }
 
+    /**
+     * @return LogoutListener|null
+     */
     public function getLogoutListener()
     {
         return $this->logoutListener;