allow usage of validator and filter plugin managers in input filter factory if form manager injected #4851

Closed
wants to merge 4 commits into
from

3 participants

@richardjennings

fix for issue #4801

Allow usage of Validator and Filter plugin managers in Input Filter factory if Form Manager Aware

@samsonasik

please provide unit test.

@weierophinney
Zend Framework member

@richardjennings Does #4788 fix the issues you encountered as well?

@richardjennings

@weierophinney No. Inputs with an undefined type are created with a clone of the InputFilter Factory validator/filter chains which do not have access to plugin managers.

I have provided an alternative fix, setting the plugin managers on the validator/filter chains if an InputFilterManager is provided with access to a ServiceLocatorInterface instance.

@weierophinney
Zend Framework member

@richardjennings Okay -- can I get some unit tests from you, then, please?

@richardjennings richardjennings check for servicelocator instance when inputfilterpluginmanager set a…
…nd inject validator and filter plugin managers into factory default filter and validator chains. Test setInputFilterPluginManager with and without ServiceLocator
1b26a41
@weierophinney weierophinney added a commit that referenced this pull request Aug 19, 2013
@weierophinney weierophinney Merge branch 'hotfix/4851' into develop
Forward port #4851
7f7c726
@weierophinney weierophinney added a commit that closed this pull request Aug 19, 2013
@weierophinney weierophinney Merge branch 'hotfix/4851'
Close #4851
d502f57
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge pull request zendframework/zendframework#4851 from trueaxiom/is…
…sue4801

allow usage of validator and filter plugin managers in input filter factory if form manager injected

Conflicts:
	tests/ZendTest/InputFilter/FactoryTest.php
2b8fd51
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4851' e059c77
@weierophinney weierophinney added a commit to zendframework/zend-inputfilter that referenced this pull request May 15, 2015
@weierophinney weierophinney Merge branch 'hotfix/4851' into develop 5f9f6fb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment