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

InputFilter\Factory can't handle config with null input #5986

Conversation

bacinsky
Copy link
Contributor

Same issue as with #4971

InputFilter\Factory should handle null value, when we want to cancel the input filter input statically (by config).

I'm ready to send a PR.

@bacinsky bacinsky changed the title InputFilter\Factory can't handle config with null elements InputFilter\Factory can't handle config with null input Mar 17, 2014
@Ocramius
Copy link
Member

Ocramius commented Apr 3, 2014

@bacinsky do you have a PR for this?

@bacinsky
Copy link
Contributor Author

bacinsky commented Apr 9, 2014

I'll write it next weekend, too busy right now.

…iling test testCanCreateInputFilterWithNullInputs()
…e testCanCreateInputFilterWithNullInputs()
@weierophinney weierophinney added this to the 2.3.1 milestone Apr 14, 2014
@weierophinney weierophinney self-assigned this Apr 14, 2014
weierophinney pushed a commit that referenced this pull request Apr 14, 2014
weierophinney pushed a commit that referenced this pull request Apr 14, 2014
weierophinney added a commit that referenced this pull request Apr 14, 2014
@weierophinney
Copy link
Member

Cherry-picked to master for release with 2.3.1.

weierophinney pushed a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
…Test, added failing test testCanCreateInputFilterWithNullInputs()
weierophinney pushed a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
…ded fix for the testCanCreateInputFilterWithNullInputs()
weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants