From: Raphaël Gertz <git@rapsys.eu>
Date: Wed, 29 Nov 2023 15:21:48 +0000 (+0100)
Subject: Adapt entity and user checker to match with symfony classes and interfaces
X-Git-Tag: 0.3.1~23
X-Git-Url: https://git.rapsys.eu/userbundle/commitdiff_plain/49d823e99f9815fca231b76aceef894531e7b42c

Adapt entity and user checker to match with symfony classes and interfaces
---

diff --git a/Checker/UserChecker.php b/Checker/UserChecker.php
index e34284e..744a35e 100644
--- a/Checker/UserChecker.php
+++ b/Checker/UserChecker.php
@@ -32,17 +32,13 @@ class UserChecker extends InMemoryUserChecker {
 		}
 
 		//With not activated user
-        if (!$user->isActivated()) {
-            $ex = new UnactivatedException('Account is not activated');
-            $ex->setUser($user);
-            throw $ex;
-        }
+		if (!$user->isActivated()) {
+			$ex = new UnactivatedException('User Account is not activated');
+			$ex->setUser($user);
+			throw $ex;
+		}
 
-		//With disabled user
-        if ($user->isDisabled()) {
-            $ex = new DisabledException('Account is disabled');
-            $ex->setUser($user);
-            throw $ex;
-        }
+		//Call parent checkPreAuth
+		parent::checkPostAuth($user);
 	}
 }
diff --git a/Entity/User.php b/Entity/User.php
index 4eee930..809d363 100644
--- a/Entity/User.php
+++ b/Entity/User.php
@@ -454,23 +454,19 @@ class User implements UserInterface, PasswordAuthenticatedUserInterface {
 	/**
 	 * Check if account is activated
 	 *
-	 * It was from deprecated AdvancedUserInterface, see if it's used anymore
-	 *
-	 * @see vendor/symfony/security-core/User/AdvancedUserInterface.php
+	 * @see vendor/rapsys/userbundle/Checker/UserChecker.php
 	 */
 	public function isActivated(): bool {
 		return $this->active;
 	}
 
 	/**
-	 * Check if account is disabled
-	 *
-	 * It was from deprecated AdvancedUserInterface, see if it's used anymore
+	 * Check if account is enabled
 	 *
-	 * @see vendor/symfony/security-core/User/AdvancedUserInterface.php
+	 * @see vendor/symfony/security-core/User/InMemoryUserChecker.php
 	 */
-	public function isDisabled(): bool {
-		return $this->disabled;
+	public function isEnabled(): bool {
+		return !$this->disabled;
 	}
 
 	/**