Thanks to visit codestin.com
Credit goes to github.com

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

BaseInputFilter handles missing data properly#5638

Merged
weierophinney merged 3 commits into
zendframework:developfrom
awartoft:hotfix/baseinputfilter-population
Jan 16, 2014
Merged

BaseInputFilter handles missing data properly#5638
weierophinney merged 3 commits into
zendframework:developfrom
awartoft:hotfix/baseinputfilter-population

Conversation

@awartoft
Copy link
Copy Markdown
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.
@awartoft awartoft closed this Dec 22, 2013
@awartoft awartoft 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