Merge pull request #286 from kenjis/fix-exception-structure

fix: Exception structure

Author: kenjis

src/Authentication/AuthenticationException.php

@@ -3,9 +3,9 @@
 namespace CodeIgniter\Shield\Authentication;
 
 use CodeIgniter\HTTP\Exceptions\HTTPException;
-use Exception;
+use CodeIgniter\Shield\Exceptions\RuntimeException;
 
-class AuthenticationException extends Exception
+class AuthenticationException extends RuntimeException
 {
     protected $code = 403;
 

src/Authentication/Authenticators/AccessTokens.php

@@ -7,11 +7,11 @@
 use CodeIgniter\Shield\Authentication\AuthenticationException;
 use CodeIgniter\Shield\Authentication\AuthenticatorInterface;
 use CodeIgniter\Shield\Entities\User;
+use CodeIgniter\Shield\Exceptions\InvalidArgumentException;
 use CodeIgniter\Shield\Models\TokenLoginModel;
 use CodeIgniter\Shield\Models\UserIdentityModel;
 use CodeIgniter\Shield\Models\UserModel;
 use CodeIgniter\Shield\Result;
-use InvalidArgumentException;
 
 class AccessTokens implements AuthenticatorInterface
 {

src/Authentication/Authenticators/Session.php

@@ -13,14 +13,13 @@
 use CodeIgniter\Shield\Authentication\Passwords;
 use CodeIgniter\Shield\Entities\User;
 use CodeIgniter\Shield\Entities\UserIdentity;
+use CodeIgniter\Shield\Exceptions\InvalidArgumentException;
 use CodeIgniter\Shield\Exceptions\LogicException;
 use CodeIgniter\Shield\Models\LoginModel;
 use CodeIgniter\Shield\Models\RememberModel;
 use CodeIgniter\Shield\Models\UserIdentityModel;
 use CodeIgniter\Shield\Models\UserModel;
 use CodeIgniter\Shield\Result;
-use Exception;
-use InvalidArgumentException;
 use stdClass;
 
 class Session implements AuthenticatorInterface
@@ -812,8 +811,6 @@ public function recordActiveDate(): void
      * and stores the necessary info in the db and a cookie.
      *
      * @see https://paragonie.com/blog/2015/04/secure-authentication-php-with-long-term-persistence
-     *
-     * @throws Exception
      */
     protected function rememberUser(User $user): void
     {

src/Authorization/AuthorizationException.php

@@ -2,9 +2,9 @@
 
 namespace CodeIgniter\Shield\Authorization;
 
-use Exception;
+use CodeIgniter\Shield\Exceptions\RuntimeException;
 
-class AuthorizationException extends Exception
+class AuthorizationException extends RuntimeException
 {
     protected $code = 401;
 

src/Authorization/Groups.php

@@ -3,7 +3,7 @@
 namespace CodeIgniter\Shield\Authorization;
 
 use CodeIgniter\Shield\Entities\Group;
-use RuntimeException;
+use CodeIgniter\Shield\Exceptions\RuntimeException;
 
 /**
  * Provides utility feature for working with

src/Exceptions/BaseException.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace CodeIgniter\Shield\Exceptions;
+
+use Throwable;
+
+/**
+ * Base Exception Interface for Shield
+ */
+interface BaseException extends Throwable
+{
+}

src/Exceptions/InvalidArgumentException.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace CodeIgniter\Shield\Exceptions;
+
+class InvalidArgumentException extends LogicException implements BaseException
+{
+}

src/Exceptions/LogicException.php

@@ -2,6 +2,6 @@
 
 namespace CodeIgniter\Shield\Exceptions;
 
-class LogicException extends \LogicException
+class LogicException extends \LogicException implements BaseException
 {
 }

src/Exceptions/RuntimeException.php

@@ -2,6 +2,6 @@
 
 namespace CodeIgniter\Shield\Exceptions;
 
-class RuntimeException extends \RuntimeException
+class RuntimeException extends \RuntimeException implements BaseException
 {
 }

src/Models/LoginModel.php

@@ -7,7 +7,6 @@
 use CodeIgniter\Shield\Authentication\Authenticators\Session;
 use CodeIgniter\Shield\Entities\Login;
 use CodeIgniter\Shield\Entities\User;
-use Exception;
 use Faker\Generator;
 
 class LoginModel extends Model
@@ -81,8 +80,6 @@ public function lastLogin(User $user): ?Login
 
     /**
      * Generate a fake login for testing
-     *
-     * @throws Exception
      */
     public function fake(Generator &$faker): Login
     {