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

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Contributor

bacinsky commented Apr 14, 2014

Same issue as with zendframework#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 changed the title from InputFilter\Factory can't handle config with null elements to InputFilter\Factory can't handle config with null input Mar 17, 2014

Member

Ocramius commented Apr 3, 2014

@bacinsky do you have a PR for this?

Contributor

bacinsky commented Apr 9, 2014

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

weierophinney added this to the 2.3.1 milestone Apr 14, 2014

weierophinney self-assigned this Apr 14, 2014

@weierophinney weierophinney added a commit that referenced this pull request Apr 14, 2014

@bacinsky @weierophinney bacinsky + weierophinney #5986 Updated ZendTest\InputFilter\FactoryTest, added failing test te…
…stCanCreateInputFilterWithNullInputs()
26da363

@weierophinney weierophinney added a commit that referenced this pull request Apr 14, 2014

@bacinsky @weierophinney bacinsky + weierophinney #5986 Updated Zend\InputFilter\Factory, added fix for the testCanCrea…
…teInputFilterWithNullInputs()
e0946e2

@weierophinney weierophinney added a commit that referenced this pull request Apr 14, 2014

@weierophinney weierophinney Merge branch 'hotfix/5986' into develop
Forward port #5986
2313132
Owner

weierophinney commented Apr 14, 2014

Cherry-picked to master for release with 2.3.1.

@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015

@bacinsky @weierophinney bacinsky + weierophinney zendframework/zendframework#5986 Updated ZendTest\InputFilter\Factory…
…Test, added failing test testCanCreateInputFilterWithNullInputs()
38eff5a

@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015

@bacinsky @weierophinney bacinsky + weierophinney zendframework/zendframework#5986 Updated Zend\InputFilter\Factory, ad…
…ded fix for the testCanCreateInputFilterWithNullInputs()
335d39a

@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/5986' b1321e5

@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015

@weierophinney weierophinney Merge branch 'hotfix/5986' into develop e5d2180
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment