Fix PasswordUpdater if password is null

Sync User::password property with PasswordAuthenticatedUserInterface
schvoy · Sep 9, 2024 · 3f7f31c · 3f7f31c
1 parent 3b793f9
commit 3f7f31c
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## 1.0.3
+
+* Fix PasswordUpdater if password is null 
+* Sync User::password property with PasswordAuthenticatedUserInterface
+
 ## 1.0.2
 
 * Improve the README.md

diff --git a/src/Entity/User.php b/src/Entity/User.php
@@ -20,7 +20,7 @@ class User extends IdBasedEntity implements UserInterface, PasswordAuthenticated
     protected array $roles = [];
 
     #[ORM\Column(type: Types::STRING)]
-    protected string $password;
+    protected ?string $password = null;
 
     public function __toString()
     {
@@ -68,7 +68,7 @@ public function setRoles(array $roles): void
     /**
      * @see UserInterface
      */
-    public function getPassword(): string
+    public function getPassword(): ?string
     {
         return $this->password;
     }

diff --git a/src/Security/PasswordUpdater.php b/src/Security/PasswordUpdater.php
@@ -28,7 +28,10 @@ public function hashPassword(UserInterface $user, string $plainPassword): void
             throw new UnsupportedUserException(sprintf('Instances of "%s" are not supported.', get_class($user)));
         }
 
-        if (false === $this->userPasswordHasher->needsRehash($user)) {
+        if (
+            null !== $user->getPassword()
+            && false === $this->userPasswordHasher->needsRehash($user)
+        ) {
             return;
         }