fix: handle null values on the password validator (#100)

ArkEcosystemArchive · Mar 24, 2021 · db2fe55 · db2fe55
1 parent 70ce1ab
commit db2fe55
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/src/Rules/Password.php b/src/Rules/Password.php
@@ -26,6 +26,9 @@ class Password extends Fortify
      */
     public function passes($attribute, $value)
     {
+        // Handle potential NULL values
+        $value = $value ?: '';
+
         if ($this->needsLowercase($value)) {
             return false;
         }

diff --git a/tests/Rules/PasswordTest.php b/tests/Rules/PasswordTest.php
@@ -4,6 +4,12 @@
 
 use ARKEcosystem\Fortify\Rules\Password;
 
+it('handle null values', function () {
+    $rule = (new Password())->requireUppercase()->requireNumeric()->requireSpecialCharacter();
+
+    expect($rule->passes('password', null))->toBeFalse();
+});
+
 it('can check for lowercase requirements', function () {
     $rule = (new Password())->requireLowercase();