Project

General

Profile

Actions

Bug #71323

closed

Story #69617: FormEngine bugs

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

Added by Christian Weiske over 8 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
FormEngine aka TCEforms
Target version:
Start date:
2015-11-04
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
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


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Task #71607: FormEngine: Flex section deletionClosed2015-11-16

Actions
Actions #1

Updated by Christian Kuhn over 8 years ago

  • Target version set to 7 LTS
Actions #2

Updated by Morton Jonuschat over 8 years ago

  • Status changed from New to Accepted
  • Assignee set to Morton Jonuschat
Actions #3

Updated by Morton Jonuschat over 8 years ago

  • Status changed from Accepted to In Progress
Actions #4

Updated by Morton Jonuschat over 8 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 #5

Updated by Frank Nägler over 8 years ago

  • Assignee changed from Morton Jonuschat to Frank Nägler
Actions #6

Updated by Gerrit Code Review over 8 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 #7

Updated by Christian Kuhn over 8 years ago

  • Parent task set to #69617
Actions #8

Updated by Frank Nägler over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #9

Updated by Riccardo De Contardi almost 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF