Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

activate commented Exception in Validator/Between::__construct #6581

Closed

Conversation

samsonasik
Copy link
Contributor

  • if parameter is an array, it should have 'min' and 'max'.
  • if the array parameter count !==2 and doesn't have 'min' or 'max' anymore, throws an Exception, because if array parameter count == 2, then 1st is a 'min', and 2nd is a 'max' like at
// at ZendTest\Validator\ValidatorChainTest::testIsValidWithParameters()
$this->assertTrue(StaticValidator::execute(5, 'Between', array(1, 10)));

@samsonasik samsonasik force-pushed the val-miss-min-or-max branch 2 times, most recently from 22a5f96 to 0240b73 Compare August 20, 2014 17:29
@mwillbanks mwillbanks added this to the 2.4.0 milestone Aug 27, 2014
@Ocramius Ocramius self-assigned this Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
@Ocramius
Copy link
Member

@samsonasik merged into develop at 0cf733b, thanks!

@Ocramius Ocramius closed this Nov 22, 2014
gianarb pushed a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-validator that referenced this pull request May 15, 2015
gianarb pushed a commit to zendframework/zend-file that referenced this pull request May 15, 2015
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants