zendframework / zend-inputfilter

InputFilter component from Zend Framework
BSD 3-Clause "New" or "Revised" License
64 stars 50 forks source link

pass configured input filter plugin manager to generated input filter factories #77

Closed abacaphiliac closed 8 years ago

abacaphiliac commented 8 years ago

pass configured input filter plugin manager to generated input filter factories to allow for complex input filter nesting. this enables reuse of named input filter services within nested input filter contexts.

i added more logic in the factory than i had originally intended, in order to preserve backwards compatibility.

Maks3w commented 8 years ago
  1. You've added 3 new conditions but there's only 1 test. Should be at least 5.
  2. We are adopting SM v3 across all repositories and I'll wait to this task to be over before merge this
abacaphiliac commented 8 years ago

@Maks3w thanks for pointing out the lack of path coverage. i was able to reduce a bit of the complexity and i think my update now includes the line and path coverage you expect.

since it sounds like this must wait, would you like me to open a new pull request to develop instead of master?

do you know where i can find information on when SM v3 will be completed? i'm interested in reviewing the changes and contributing, if possible. do you anticipate that the new major version of this dependency will require a major version change for this package as well?

Maks3w commented 8 years ago

Target branch not affect to this PR to be on hold after the SM upgrade is done.

If you want you can forward that task proposing the SM adoption

Examples

https://github.com/zendframework/zend-serializer/pull/2 https://github.com/zendframework/zend-cache/pull/22 https://github.com/zendframework/zend-mvc/pull/36

weierophinney commented 8 years ago

3, merged today, accomplishes this, and will release with a forthcoming 2.7.0.

abacaphiliac commented 8 years ago

excellent! thanks @weierophinney : )