LiittleCookie/valitron
1
0
Fork 0
mirror of https://github.com/vlucas/valitron.git synced 2025-12-30 23:01:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add type check for stringLength

Browse Source 
Return 0 if $value is not string
This commit is contained in:
USAMI Kenta 2015-11-20 16:18:42 +09:00
parent 29c7d35db6
commit 28254e76c2
2 changed files with 28 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/Valitron/Validator.php
View File

@ -244,7 +244,7 @@ class Validator
return $length >= $params[0] && $length <= $params[1];
}
// Length same
return $length == $params[0];
return ($length !== false) && $length == $params[0];
}
/**
@ -259,7 +259,7 @@ class Validator
{
$length = $this->stringLength($value);
return $length >= $params[0] && $length <= $params[1];
return ($length !== false) && $length >= $params[0] && $length <= $params[1];
}
/**
@ -273,7 +273,9 @@ class Validator
*/
protected function validateLengthMin($field, $value, $params)
{
return $this->stringLength($value) >= $params[0];
$length = $this->stringLength($value);
return ($length !== false) && $length >= $params[0];
}
/**
@ -287,18 +289,22 @@ class Validator
*/
protected function validateLengthMax($field, $value, $params)
{
return $this->stringLength($value) <= $params[0];
$length = $this->stringLength($value);
return ($length !== false) && $length <= $params[0];
}
/**
* Get the length of a string
*
* @param string $value
* @return int
* @return int|false
*/
protected function stringLength($value)
{
if (function_exists('mb_strlen')) {
if (!is_string($value)) {
return false;
} elseif (function_exists('mb_strlen')) {
return mb_strlen($value);
}

16
tests/Valitron/ValidateTest.php
View File

@ -131,6 +131,14 @@ class ValidateTest extends BaseTestCase
$v = new Validator(array('str' => 'sad'));
$v->rule('length', 'str', 6);
$this->assertFalse($v->validate());
$v = new Validator(array('test' => array()));
$v->rule('length', 'test', 1);
$this->assertFalse($v->validate());
$v = new Validator(array('test' => new stdClass));
$v->rule('length', 'test', 1);
$this->assertFalse($v->validate());
}
public function testLengthBetweenValid()
@ -145,6 +153,14 @@ class ValidateTest extends BaseTestCase
$v = new Validator(array('str' => 'sad'));
$v->rule('lengthBetween', 'str', 4, 10);
$this->assertFalse($v->validate());
$v = new Validator(array('test' => array()));
$v->rule('lengthBetween', 'test', 50, 60);
$this->assertFalse($v->validate());
$v = new Validator(array('test' => new stdClass));
$v->rule('lengthBetween', 'test', 99, 100);
$this->assertFalse($v->validate());
}
public function testLengthMinValid()