-
-
Notifications
You must be signed in to change notification settings - Fork 440
Replace Zend_Validation #4612
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
Replace Zend_Validation #4612
Conversation
|
sad Also, you might need to look at this:
|
I tried, but writing rules is more intuitive with that ... imho.
I am aware of it, but latest version requies php 8.1. |
for the current changes in this MR, Symfony Validator might be easy too. the bigger use would be that we could validate entire objects like Customer Address at once Also the Intl/Translation support is nice, so we could have the errors already translated by the System |
I would be in favor of switching this to Symfony validation. Looking at this from a bigger picture, it would be best if we picked a library of components (e.g. Symfony/Laminas/etc.) and stuck to those throughout the project when replacing outdated libraries. IMO not doing this will increase the maintenance burden long term with differing packages/conventions to keep track of. |
# Conflicts: # composer.json # composer.lock
|
Did it. Looks good, but found some issues ... wait for #4758. |
Description (*)
WIP (90%), but ready to review.
This attempts to replace another Zend_ library.
I did not choose laminas (as earlier suggested in #3199), nor symfony. No framework, straight-forward validation.
See https://github.com/Respect/Validation
Related Pull Requests