Add void return types

Author: wouterj

Adapter/AbstractConnection.php

@@ -30,6 +30,9 @@ public function __construct(array $config = [])
         $this->config = $resolver->resolve($config);
     }
 
+    /**
+     * @return void
+     */
     protected function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([

Adapter/ExtLdap/Connection.php

@@ -63,6 +63,8 @@ public function isBound(): bool
 
     /**
      * @param string $password WARNING: When the LDAP server allows unauthenticated binds, a blank $password will always be valid
+     *
+     * @return void
      */
     public function bind(string $dn = null, #[\SensitiveParameter] string $password = null)
     {
@@ -96,13 +98,19 @@ public function getResource()
         return $this->connection;
     }
 
+    /**
+     * @return void
+     */
     public function setOption(string $name, array|string|int|bool $value)
     {
         if (!@ldap_set_option($this->connection, ConnectionOptions::getOption($name), $value)) {
             throw new LdapException(sprintf('Could not set value "%s" for option "%s".', $value, $name));
         }
     }
 
+    /**
+     * @return array|string|int|null
+     */
     public function getOption(string $name)
     {
         if (!@ldap_get_option($this->connection, ConnectionOptions::getOption($name), $ret)) {
@@ -112,6 +120,9 @@ public function getOption(string $name)
         return $ret;
     }
 
+    /**
+     * @return void
+     */
     protected function configureOptions(OptionsResolver $resolver)
     {
         parent::configureOptions($resolver);
@@ -138,7 +149,7 @@ protected function configureOptions(OptionsResolver $resolver)
         });
     }
 
-    private function connect()
+    private function connect(): void
     {
         if ($this->connection) {
             return;
@@ -167,7 +178,7 @@ private function connect()
         }
     }
 
-    private function disconnect()
+    private function disconnect(): void
     {
         if ($this->connection) {
             ldap_unbind($this->connection);

Adapter/ExtLdap/EntryManager.php

@@ -111,6 +111,9 @@ public function removeAttributeValues(Entry $entry, string $attribute, array $va
         return $this;
     }
 
+    /**
+     * @return $this
+     */
     public function rename(Entry $entry, string $newRdn, bool $removeOldRdn = true)
     {
         $con = $this->getConnectionResource();

Adapter/ExtLdap/Query.php

@@ -160,7 +160,7 @@ public function getResources(): array
     /**
      * Resets pagination on the current connection.
      */
-    private function resetPagination()
+    private function resetPagination(): void
     {
         $con = $this->connection->getResource();
         $this->controlPagedResult(0, false, '');

Entry.php

@@ -96,6 +96,8 @@ public function getAttributes(): array
 
     /**
      * Sets a value for the given attribute.
+     *
+     * @return void
      */
     public function setAttribute(string $name, array $value)
     {
@@ -105,6 +107,8 @@ public function setAttribute(string $name, array $value)
 
     /**
      * Removes a given attribute.
+     *
+     * @return void
      */
     public function removeAttribute(string $name)
     {

Ldap.php

@@ -29,7 +29,7 @@ public function __construct(AdapterInterface $adapter)
         $this->adapter = $adapter;
     }
 
-    public function bind(string $dn = null, #[\SensitiveParameter] string $password = null)
+    public function bind(string $dn = null, #[\SensitiveParameter] string $password = null): void
     {
         $this->adapter->getConnection()->bind($dn, $password);
     }

Security/CheckLdapCredentialsListener.php

@@ -36,6 +36,9 @@ public function __construct(ContainerInterface $ldapLocator)
         $this->ldapLocator = $ldapLocator;
     }
 
+    /**
+     * @return void
+     */
     public function onCheckPassport(CheckPassportEvent $event)
     {
         $passport = $event->getPassport();

Security/LdapUser.php

@@ -75,7 +75,7 @@ public function getUserIdentifier(): string
         return $this->identifier;
     }
 
-    public function eraseCredentials()
+    public function eraseCredentials(): void
     {
         $this->password = null;
     }
@@ -85,7 +85,7 @@ public function getExtraFields(): array
         return $this->extraFields;
     }
 
-    public function setPassword(#[\SensitiveParameter] string $password)
+    public function setPassword(#[\SensitiveParameter] string $password): void
     {
         $this->password = $password;
     }