Bug #71323: Flexform: Removing section element with empty required field prevents form saving - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #71323

closed

Story #69617: FormEngine bugs

Flexform: Removing section element with empty required field prevents form saving

Added by Christian Weiske about 9 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Frank Nägler

Category:

FormEngine aka TCEforms

Target version:

7 LTS

Start date:

2015-11-04

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

I've got a flexform section which I can add elements to. One of the fields inside the section has a required attribute set.

When adding the element to the section and focusing the input field, it gets marked with a red exclamation mark. Also the "General" form tab gets marked with an exclamation mark.
I cannot save the form until I fill out the field.

Now when I remove the section element, I still cannot save the form - even though the reason for the validation failure has gone.

Happens with TYPO3 7.5, flux 7.2.3 and fluidcontent 4.3.3

Minimal fluidcontent example:

<?xml version="1.0" encoding="utf-8"?>
<html xmlns="http://www.w3.org/1999/xhtml" lang="en" 
     xmlns:f="http://typo3.org/ns/TYPO3/Fluid/ViewHelpers" 
     xmlns:flux="http://typo3.org/ns/FluidTYPO3/Flux/ViewHelpers">
  <f:layout name="Content" />

  <f:section name="Configuration">
    <flux:form id="test" options="{group: 'FCE'}">
      <flux:form.section name="settings.links" label="Links">
        <flux:form.object name="link" label="Link">
          <flux:field.input name="url" label="URL" required="1"/>
        </flux:form.object>
      </flux:form.section>
    </flux:form>
  </f:section>
</html>

Files

2015-11-04 typo3 flexform nothing savefail.png (68.9 KB) 2015-11-04 typo3 flexform nothing savefail.png

Christian Weiske, 2015-11-04 15:50

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Christian Kuhn about 9 years ago

Target version set to 7 LTS

Actions

Copy link

Updated by Morton Jonuschat about 9 years ago

Status changed from New to Accepted
Assignee set to Morton Jonuschat

Actions

Copy link

Updated by Morton Jonuschat about 9 years ago

Status changed from Accepted to In Progress

Actions

Copy link

Updated by Morton Jonuschat about 9 years ago

Status changed from In Progress to Accepted

This is trickier than initially thought. Steps to reproduce without flux:

In EXT:styleguide/Flex_3.xml change element_2 to have <required>1</required>

Adding and then removing a section triggers the error.

Actions

Copy link

Updated by Frank Nägler about 9 years ago

Assignee changed from Morton Jonuschat to Frank Nägler

Actions

Copy link

Updated by Gerrit Code Review about 9 years ago

Status changed from Accepted to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/44732

Actions

Copy link