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

BaseInputFilter handles missing data properly #5638

Merged
merged 3 commits into from
Jan 16, 2014
Merged

BaseInputFilter handles missing data properly #5638

merged 3 commits into from
Jan 16, 2014

Conversation

macnibblet
Copy link
Contributor

Previously BaseInputFilter::populate would pass a null value to setValue
this would cause the ArrayInput to throw an exception even if the
ArrayInput is not part of the validation group.

Something else that should be looked at is introducing a validator
instead of using a exception Zend\InputFilter\ArrayInput filter because
the exception will break the application if not managed correctly.

Antoine Hedgecock added 2 commits December 22, 2013 15:42
Previously BaseInputFilter::populate would pass a null value to setValue
this would cause the ArrayInput to throw an exception even if the
ArrayInput is not part of the validation group.

Something else that should be looked at is introducing a validator
instead of using a exception Zend\InputFilter\ArrayInput filter because
the exception will break the application if not managed correctly.
@macnibblet macnibblet closed this Dec 22, 2013
@macnibblet macnibblet reopened this Dec 22, 2013
weierophinney added a commit that referenced this pull request Jan 16, 2014
…ation

BaseInputFilter handles missing data properly
weierophinney added a commit that referenced this pull request Jan 16, 2014
@weierophinney weierophinney merged commit 7e0db40 into zendframework:develop Jan 16, 2014
@ghost ghost assigned weierophinney Jan 16, 2014
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
…otfix/baseinputfilter-population

BaseInputFilter handles missing data properly
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants