Merge pull request #196 from kenjis/fix-logout-return-type

refactor: change logout() return type to void

Author: kenjis

src/Authentication/AuthenticatorInterface.php

@@ -50,7 +50,7 @@ public function loginById($userId): void;
      *
      * @see https://codeigniter4.github.io/CodeIgniter4/extending/authentication.html
      */
-    public function logout(): bool;
+    public function logout(): void;
 
     /**
      * Returns the currently logged in user.

src/Authentication/Authenticators/AccessTokens.php

@@ -189,11 +189,9 @@ public function loginById($userId): void
     /**
      * Logs the current user out.
      */
-    public function logout(): bool
+    public function logout(): void
     {
         $this->user = null;
-
-        return true;
     }
 
     /**

src/Authentication/Authenticators/Session.php

@@ -660,10 +660,10 @@ public function loginById($userId): void
     /**
      * Logs the current user out.
      */
-    public function logout(): bool
+    public function logout(): void
     {
         if ($this->user === null) {
-            return true;
+            return;
         }
 
         helper('cookie');
@@ -686,11 +686,9 @@ public function logout(): bool
         $this->rememberModel->purgeRememberTokens($this->user);
 
         // Trigger logout event
-        $result = Events::trigger('logout', $this->user);
+        Events::trigger('logout', $this->user);
 
         $this->user = null;
-
-        return $result;
     }
 
     /**