Skip to content

Commit caa034f

Browse files
committed
wip
1 parent 7a1bd88 commit caa034f

File tree

4 files changed

+10
-5
lines changed

4 files changed

+10
-5
lines changed

src/Tempest/Validation/src/Rules/IsFloat.php

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,10 @@ public function isValid(mixed $value): bool
2020
return true;
2121
}
2222

23+
if ($value === null || $value === false || $value === '') {
24+
return false;
25+
}
26+
2327
if (floatval($value) == $value) {
2428
return true;
2529
}

src/Tempest/Validation/src/Rules/IsInteger.php

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,7 @@ public function isValid(mixed $value): bool
2020
return true;
2121
}
2222

23-
if ($value === null) {
24-
return false;
25-
}
26-
27-
if ($value === '') {
23+
if ($value === null || $value === false || $value === '') {
2824
return false;
2925
}
3026

src/Tempest/Validation/tests/Rules/IsFloatTest.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,8 @@ public function test_float(): void
2020
$this->assertTrue($rule->isValid(0.1));
2121
$this->assertTrue($rule->isValid('0.1'));
2222
$this->assertFalse($rule->isValid('a'));
23+
$this->assertFalse($rule->isValid(''));
24+
$this->assertFalse($rule->isValid(null));
25+
$this->assertFalse($rule->isValid(false));
2326
}
2427
}

src/Tempest/Validation/tests/Rules/IsIntegerTest.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,5 +20,7 @@ public function test_integer(): void
2020
$this->assertTrue($rule->isValid('1'));
2121
$this->assertFalse($rule->isValid('a'));
2222
$this->assertFalse($rule->isValid(''));
23+
$this->assertFalse($rule->isValid(null));
24+
$this->assertFalse($rule->isValid(false));
2325
}
2426
}

0 commit comments

Comments
 (0)