Use Hash::needsRehash in HasAttributes::castAttributeAsHashedString i…

…nstead of Hash::isHashed
bastien-phi · Oct 17, 2023 · ce8e63e · ce8e63e
1 parent 2f88a30
commit ce8e63e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php b/src/Illuminate/Database/Eloquent/Concerns/HasAttributes.php
@@ -1316,7 +1316,7 @@ public static function encryptUsing($encrypter)
      */
     protected function castAttributeAsHashedString($key, $value)
     {
-        return $value !== null && ! Hash::isHashed($value) ? Hash::make($value) : $value;
+        return $value !== null && Hash::needsRehash($value) ? Hash::make($value) : $value;
     }
 
     /**

diff --git a/tests/Integration/Database/EloquentModelHashedCastingTest.php b/tests/Integration/Database/EloquentModelHashedCastingTest.php
@@ -30,9 +30,9 @@ protected function defineDatabaseMigrationsAfterDatabaseRefreshed()
 
     public function testHashed()
     {
-        $this->hasher->expects('isHashed')
+        $this->hasher->expects('needsRehash')
             ->with('this is a password')
-            ->andReturnFalse();
+            ->andReturnTrue();
 
         $this->hasher->expects('make')
             ->with('this is a password')
@@ -51,9 +51,9 @@ public function testHashed()
 
     public function testNotHashedIfAlreadyHashed()
     {
-        $this->hasher->expects('isHashed')
+        $this->hasher->expects('needsRehash')
             ->with('already-hashed-password')
-            ->andReturnTrue();
+            ->andReturnFalse();
 
         $subject = HashedCast::create([
             'password' => 'already-hashed-password',