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

'type' in Input config may be plugin name #2

Merged
merged 1 commit into from
Jun 23, 2015
Merged

'type' in Input config may be plugin name #2

merged 1 commit into from
Jun 23, 2015

Conversation

halaxa
Copy link
Contributor

@halaxa halaxa commented May 29, 2015

Support for input plugin name in 'type' input configuration key for Zend\InputFilter\Factory::createInput().

@halaxa
Copy link
Contributor Author

halaxa commented Jun 15, 2015

Ping @weierophinney. Is there anything else I should add to this PR? Thank you.

@weierophinney weierophinney added this to the 2.5.2 milestone Jun 23, 2015
@weierophinney weierophinney self-assigned this Jun 23, 2015
weierophinney added a commit that referenced this pull request Jun 23, 2015
'type' in Input config may be plugin name
@weierophinney weierophinney merged commit de361b0 into zendframework:master Jun 23, 2015
weierophinney added a commit that referenced this pull request Jun 23, 2015
weierophinney added a commit that referenced this pull request Jun 23, 2015
weierophinney added a commit that referenced this pull request Jun 23, 2015
weierophinney added a commit to weierophinney/zend-inputfilter that referenced this pull request Nov 7, 2017
… manager

As reported in zendframework#8, a change in zendframework#2 now causes inputs pulled from the
plugin manager within `Factory::createInput()` to be injected with the
default filter and validator chains composed in the `Factory`.
This can cause filter and validator chains created during construction
or within the input's factory to be overwritten entirely.

This patch adds conditions to skip injection of the default filter and
validator chains on inputs pulled from the input filter manager, while
retaining the logic introduced in zendframework#2 to allow values in the
`$inputSpecification` to further configure the instance.
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.

2 participants