Bug #38154
Validator shorthand notation does not work as documented
| Status: | Resolved | Start date: | 2012-06-18 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Sebastian Kurfuerst | % Done: | 100% |
|
| Category: | Validation | |||
| Target version: | TYPO3 Flow Base Distribution - 1.1 beta 3 | |||
| PHP Version: | Complexity: | |||
| Has patch: | No | Affected Flow version: | Git 1.1 |
Description
According to http://flow3.typo3.org/documentation/guide/partiii/validation.html#creating-validator-instances-the-validatorresolver it is possible to use Your.Package:Foo as validator type. This does not work, only Your\Package:Foo works.
Related issues
Associated revisions
[BUGFIX] Correctly implement validator resolver short notation
Currently, the short notation for validators which has been introduced
with Id807dbfa78ae1388aa8cd53bfc578893eade427c does only support syntax
like TYPO3\FLOW3:EmailAddress; but we should rather support TYPO3.FLOW3:EmailAddress
as this is how we use package keys at all other places which are not PHP code.
Change-Id: Icbab36f39955254a59dced0abee0ba4ef11130b5
Fixes: #38154
Related: #33862
Releases: 1.1, 1.2
[BUGFIX] Correctly implement validator resolver short notation
Currently, the short notation for validators which has been introduced
with Id807dbfa78ae1388aa8cd53bfc578893eade427c does only support syntax
like TYPO3\FLOW3:EmailAddress; but we should rather support TYPO3.FLOW3:EmailAddress
as this is how we use package keys at all other places which are not PHP code.
Change-Id: Icbab36f39955254a59dced0abee0ba4ef11130b5
Fixes: #38154
Related: #33862
Releases: 1.1, 1.2
History
#1 Updated by Gerrit Code Review about 3 years ago
- Status changed from Accepted to Under Review
Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/11778
#2 Updated by Gerrit Code Review about 3 years ago
Patch set 1 for branch FLOW3-1.1 has been pushed to the review server.
It is available at http://review.typo3.org/12167
#3 Updated by Sebastian Kurfuerst about 3 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset fdfca9eb878fda281fc777afadb911b0bf26d6e9.
#4 Updated by Karsten Dambekalns about 3 years ago
- Target version changed from 1.1 RC1 to 1.1 beta 3