Bug #86910: All text input fields get required="required" attribute - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #86910

closed

All text input fields get required="required" attribute

Added by Alexander Grein about 6 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

Form Framework

Target version:

Start date:

2018-11-12

Due date:

% Done:

Estimated time:

TYPO3 Version:

PHP Version:

7.2

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

To reproduce, 1. generate a test.form.yml using a simple text Node:

identifier: test
label: 'Test'
prototypeName: standard
type: Form
renderables:
  -
    identifier: page-1
    type: Page
    label: 'Page 1'
    renderables:
      -
        identifier: test
        type: Text
        label: Test
        properties:
          fluidAdditionalAttributes:
            placeholder: Test

2. Choose this form in a form content element
3. Open page with the generated form in the frontend

Expected: A form containing a input field without a required="required" attribute

Reality: A form with a required="required" attribute

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Alexander Grein about 6 years ago

Additional Info about this bug:

It only happens right after clearing caches and reload the form page the first time.

After reloading the same page again, the additional required attribute disappears.

Actions

Copy link

Updated by Wouter Wolters about 6 years ago

Is duplicate of Bug #86890: AbstractTagBasedViewHelper recycles the TagBuilder added

Actions

Copy link

Updated by Wouter Wolters about 6 years ago

Status changed from New to Closed

Duplicates #86890 please continue there

Actions

Copy link

Updated by Jonas Eberle about 6 years ago

Just to add this here:

I cannot reproduce with just 1 (unrequired) field. To reproduce, I have to add 1 required field before.
Do you only have 1 form on this page?

Actions

Copy link

Updated by Alexander Grein about 6 years ago

No, I have several required fields before.
I did not think that mattered.

Actions

Copy link

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

Actions

Copy link

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

Actions

Copy link

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

Actions

Copy link

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

Actions

Copy link

#10