[BUGFIX] Correctly apply Extbase validator options and add tests
* Check supplied options * Apply default-values of validator if needed * Check for required options * Provide options via constructor instead of via setOptions() (which is deprecated) * Add tests for AbstractValidator and AbstractCompositeValidator Resolves: #52208 Resolves: #57164 Releases: 6.2, 6.1 Change-Id: I589a30bea74144eed51948ecad3143ab145c958f Reviewed-on: https://review.typo3.org/28938 Reviewed-by: Marc Bastian Heinrichs Tested-by: Marc Bastian Heinrichs Reviewed-by: Pascal Dürsteler Tested-by: Pascal Dürsteler Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
Showing
- typo3/sysext/extbase/Classes/Validation/Validator/AbstractValidator.php 33 additions, 5 deletions...xtbase/Classes/Validation/Validator/AbstractValidator.php
- typo3/sysext/extbase/Classes/Validation/Validator/BooleanValidator.php 15 additions, 1 deletion...extbase/Classes/Validation/Validator/BooleanValidator.php
- typo3/sysext/extbase/Classes/Validation/ValidatorResolver.php 1 addition, 5 deletions...3/sysext/extbase/Classes/Validation/ValidatorResolver.php
- typo3/sysext/extbase/Tests/Unit/Validation/Validator/AbstractCompositeValidatorTest.php 78 additions, 0 deletions...t/Validation/Validator/AbstractCompositeValidatorTest.php
- typo3/sysext/extbase/Tests/Unit/Validation/Validator/AbstractValidatorTest.php 78 additions, 0 deletions...Tests/Unit/Validation/Validator/AbstractValidatorTest.php
- typo3/sysext/extbase/Tests/Unit/Validation/Validator/Fixture/AbstractCompositeValidatorClass.php 52 additions, 0 deletions...ion/Validator/Fixture/AbstractCompositeValidatorClass.php
- typo3/sysext/extbase/Tests/Unit/Validation/Validator/Fixture/AbstractValidatorClass.php 52 additions, 0 deletions...t/Validation/Validator/Fixture/AbstractValidatorClass.php
- typo3/sysext/extbase/Tests/Unit/Validation/Validator/StringLengthValidatorTest.php 0 additions, 2 deletions...s/Unit/Validation/Validator/StringLengthValidatorTest.php
- typo3/sysext/extbase/Tests/Unit/Validation/ValidatorResolverTest.php 5 additions, 4 deletions...t/extbase/Tests/Unit/Validation/ValidatorResolverTest.php
Please register or sign in to comment