Skip to content

Commit e309222

Browse files
authored
[1.x] Fix password rule (#211)
* Fix password rule * Adjust logic
1 parent a246ccf commit e309222

File tree

2 files changed

+4
-0
lines changed

2 files changed

+4
-0
lines changed

src/Rules/Password.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,8 @@ class Password implements Rule
5151
*/
5252
public function passes($attribute, $value)
5353
{
54+
$value = is_scalar($value) ? (string) $value : '';
55+
5456
if ($this->requireUppercase && Str::lower($value) === $value) {
5557
return false;
5658
}

tests/PasswordRuleTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@ public function test_password_rule()
1212
$rule = new Password;
1313

1414
$this->assertTrue($rule->passes('password', 'password'));
15+
$this->assertTrue($rule->passes('password', 234234234));
16+
$this->assertFalse($rule->passes('password', ['foo' => 'bar']));
1517
$this->assertFalse($rule->passes('password', 'secret'));
1618

1719
$this->assertTrue(Str::contains($rule->message(), 'must be at least 8 characters'));

0 commit comments

Comments
 (0)