Fix for missing required option for CollectionInputFilter #5926

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

Contributor

Since the CollectionInputFilter class has a $isRequired option defaulted to false, we need to have a required index in the factory array, so we can force validation on that element. Otherwise, it will always ignore the collection and return valid.

Contributor
danizord commented Mar 6, 2014

👍

@weierophinney weierophinney added this to the 2.3.0 milestone Mar 7, 2014
@weierophinney weierophinney self-assigned this Mar 7, 2014
@weierophinney weierophinney added a commit that referenced this pull request Mar 7, 2014
@weierophinney weierophinney [#5926] CS fixes
- Align operators
- Comma after last array element
fcdc32a
@weierophinney weierophinney added a commit that referenced this pull request Mar 7, 2014
@weierophinney weierophinney Merge branch 'hotfix/5926' into develop
Close #5926
aa809a3
Owner

Merged to develop for release with 2.3.0.

@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#5926 from fabiocarneir…
…o/hotfix-factory-collection-input-filter-required

Fix for missing required option for CollectionInputFilter
a255d48
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney [zendframework/zendframework#5926] CS fixes
- Align operators
- Comma after last array element
0d7ca6a
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/5926' into develop 1628530
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment