Project

General

Profile

Actions

Bug #18429

closed

required fields in Flexform are not correct marked with Exclamation Sign

Added by Steffen Kamper about 16 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
-
Start date:
2008-03-12
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

In TCEForms required fields are marked with the yellow Exclamation Sign, also the Tabs containing those fields.

If a TCEForm contains a flexform with tabs and required fields, they are not marked.
If there is one required field inside: only complete flexform is marked with Exclamation Sign
If there are more than one required fields inside, no field is marked.
The fields insifde Flexforms are never marked.

If fields in Flexform are IRRE records the required check will fail also because the naming of the field is different then in the form.

to reproduce it look at pbsurvey which has required fields in flexform as IRRE fields
(issue imported from #M7832)


Files

0007832.patch (4.88 KB) 0007832.patch Administrator Admin, 2008-03-17 18:01

Related issues 6 (0 open6 closed)

Related to TYPO3 Core - Bug #18427: Required fields within flexform section elements cause Javascript errorsClosedOliver Hader2008-03-11

Actions
Related to TYPO3 Core - Bug #18751: JavaScript error in TBE_EDITOR causes login expired dialogClosedOliver Hader2008-05-06

Actions
Related to TYPO3 Core - Bug #47372: Problem with enforcing an image in a news record.Closed2013-04-19

Actions
Has duplicate TYPO3 Core - Bug #21894: Flexforms: no exclamation mark for required fieldsClosedOliver Hader2009-12-29

Actions
Has duplicate TYPO3 Core - Feature #17900: Marking required fields in flexformsRejectedSteffen Gebert2007-12-09

Actions
Blocked by TYPO3 Core - Bug #17122: flexform <eval>required</eval> on type "input" applies only on last input-fieldClosed2007-03-15

Actions
Actions #1

Updated by Steffen Kamper about 16 years ago

Now i know that this isn't IRRE. It's Kaspers new flexform arrays, unfortunally this is not documented anywhere.

Actions #2

Updated by Oliver Hader about 16 years ago

The attached patch is just a beginning, it adds the flexform sheets/tabs to the stack of nested elements.

Next steps:
  • add required-icon to each field of a flexform block
  • extend the required nested checks to use flexform sheets (=tabs) and fields
  • really remove flexform sections (currently they are only hidden and not removed)
  • create a better handling for adding/removing dynamic flexform secions
    -> add: register some stuff in JavaScript
    -> remove: ensure that no other part relies on a removide flexform section
Actions #3

Updated by Lina Wolf over 13 years ago

Duplicate of 0013117 witch has some sponsoring

Actions #4

Updated by Steffen Kamper over 13 years ago

i don't see any sponsoring there ;)

Actions #5

Updated by Steffen Kamper over 13 years ago

olly did some investigations and i looked to this yesterday too. From my debugs, the flexform field is in the nested stackes. I used this debug in line 202:
if (autoNotify) {
console.log(elementData, elementName);

which showed that the required field in in elementName. Only the elementData is the complete flexform.
Imho this is because of the recursive render of flexforms in t3lib_tceforms.

2 things have to be done:
1) fields in flexform also need the rec_ and cm_ image rendered in front
2) the field has to be added to the stack (see olly's patch)

then the javascript will work as expected and the correct image will be set.

Actions #6

Updated by Jerome BLUM over 12 years ago

This bug is still present in TYPO3 4.4. The patch is no longer compatible with this version.

Actions #7

Updated by Christian Weiske about 11 years ago

When #17122 is fixed, this issue is fixable.

Actions #8

Updated by Lars no-lastname-given almost 11 years ago

Nothing happend here until today...

Actions #9

Updated by most wanted about 10 years ago

We just tested it with the current master and can confirm that this applies also for TYPO3 6.2.

Fields configured with 'eval' is set to 'required' are not marked with exclamation mark and the record can be saved even if the field is not filled with data.

Exception: last input field (see #17122)

Actions #10

Updated by Alexander Stehlik about 10 years ago

It also seems to affect group fields with a configured minitems value.

The FlexForm can not be saved (JavaScript alert about missing fields pops up) but no exclamation mark is visible.

Actions #11

Updated by Rares Luca almost 10 years ago

Alexander Stehlik wrote:

It also seems to affect group fields with a configured minitems value.

The FlexForm can not be saved (JavaScript alert about missing fields pops up) but no exclamation mark is visible.

I can confirm this too on TYPO3 6.2.1, but I can see the exclamation marks!
(If I remove the minitems from the flexform, the issue is gone)

Actions #12

Updated by Michael Voehringer about 9 years ago

Alexander Stehlik wrote:

It also seems to affect group fields with a configured minitems value.

The FlexForm can not be saved (JavaScript alert about missing fields pops up) but no exclamation mark is visible.

I can also confirm this behave of the validation issue with filefields with an minitems of 1. I found another bug #47372 which fixes the same error with TCA-objects, but the fix did not work for flexforms.

Actions #13

Updated by Alexander Opitz almost 9 years ago

  • Description updated (diff)
  • Category set to Backend User Interface
  • Status changed from Accepted to Needs Feedback
  • Target version deleted (0)
  • Is Regression set to No

What the state of this issue?
Does it still happen with TYPO3 CMS 6.2.14?

Actions #14

Updated by Lorenz Ulrich almost 9 years ago

Alexander Opitz wrote:

What the state of this issue?
Does it still happen with TYPO3 CMS 6.2.14?

Yes. There was no improvement in TYPO3 6.2 regarding this bug.

Actions #15

Updated by Christian Kuhn almost 9 years ago

this will probably not fixed in 6.2 anymore. there was a lot of effort in 7 to improve the required handling and it is not possible to backport this stuff to 6.2.

Actions #16

Updated by Christian Kuhn almost 9 years ago

verified: this is fixed in 7, but will not be changed in 6.2. i'll close the issue as 'won't fix' for 6.2 now.

Actions #17

Updated by Christian Kuhn almost 9 years ago

  • Status changed from Needs Feedback to New
Actions #18

Updated by Christian Kuhn almost 9 years ago

  • TYPO3 Version changed from 4.2 to 6.2
  • PHP Version changed from 5.2 to 5.3
Actions #19

Updated by Christian Kuhn almost 9 years ago

  • Status changed from New to Closed
Actions

Also available in: Atom PDF