-
-
Notifications
You must be signed in to change notification settings - Fork 30
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[bc\break] Remove deprecated and validators auto injections #7
Comments
Code coverage drop its for a difference between phpunit code coverage versions. Originally posted by @Maks3w at zendframework/zend-inputfilter#83 (comment) |
@Maks3w can you drop into the summary how users can emulate the behavior using validators? Most will likely see this issue, see that the functionality was removed, but not know how to accomplish the behavior on their own. Which brings me to the next point: can you start a migration/changes document detailing the change in behavior, please? Originally posted by @weierophinney at zendframework/zend-inputfilter#83 (comment) |
@Maks3w Are you still working on this? Originally posted by @DASPRiD at zendframework/zend-inputfilter#83 (comment) |
I'm not sure if continue with this or rewrite the component. There is not urgency for to merge this Originally posted by @Maks3w at zendframework/zend-inputfilter#83 (comment) |
zendframework/zend-inputfilter#9 this issue just took up a whole afternoon. I was neck deep in filter chain debugging when a colleague found the above issue and we were able to turn off the auto injection. |
Zend\Validator\NotEmpty should be added to the ValidatorChain when expected.
Zend\Validator\File\UploadFile should be added to the ValidatorChain when expected.
ToDo:
Originally posted by @Maks3w at zendframework/zend-inputfilter#83
The text was updated successfully, but these errors were encountered: