Bug #87430
closedFields which are not required render with required attribute
0%
Description
A text field will render with the required attribute even though the "required" setting is not present in the yaml config.
How to reproduce:
Create a form with two text fields, the first set to "required" (see yaml config below).
The required field must be positioned before the non-required field, if it is not the bug does not occur.
Expected outcome:
Only the first field should be required.
Actual outcome
Both fields are set to required, where the second field also has the "required" attribute, but not the label indicator (see attached screenshot).
Suspected cause
The formvh:translateElementProperty
viewhelper
renderingOptions:
submitButtonLabel: Submit
type: Form
identifier: test
label: Test
prototypeName: standard
renderables:
-
renderingOptions:
previousButtonLabel: 'Previous step'
nextButtonLabel: 'Neue Seite'
type: Page
identifier: page-1
label: Step
renderables:
-
defaultValue: ''
type: Text
identifier: text-1
label: 'Required field'
validators:
-
identifier: NotEmpty
properties:
fluidAdditionalAttributes:
required: required
-
defaultValue: ''
type: Text
identifier: text-2
label: 'Not required field'
Files
Updated by Jarvis H almost 6 years ago
Just realized this was already reported here: https://forge.typo3.org/issues/86890
This issue can be closed
Updated by Josef Glatz almost 6 years ago
- Is duplicate of Bug #86890: AbstractTagBasedViewHelper recycles the TagBuilder added
Updated by Josef Glatz almost 6 years ago
- Status changed from New to Closed
Thanks for the update Jarvis!
Updated by Gerrit Code Review almost 6 years ago
- Status changed from Closed to Under Review
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59554
Updated by Gerrit Code Review almost 6 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59554
Updated by Gerrit Code Review almost 6 years ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59554
Updated by Gerrit Code Review almost 6 years ago
Patch set 1 for branch 9.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/59609
Updated by Gerrit Mohrmann almost 6 years ago
- Status changed from Under Review to Resolved
Applied in changeset 3469afec25bf599e7c5782a5ac6f4d71ffcf946d.