Project

General

Profile

Actions

Feature #57482

closed

FlexForms: DisplayCondition for Sheet in ExtBase Plugin

Added by Ricky B. about 10 years ago. Updated over 5 years ago.

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

0%

Estimated time:
PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

The Display Condition Flag for Sheets doesn't work with Extbase-Settings Feature.
If Condition is written like

<displayCond>FIELD:General.settings.limit:=:2</displayCond>

the FormEngine.php (line 2824) splits in "General" and "settings" and "limit" but recognize only "settings" as fieldname.
The limit get lost.

My temp solution ist:

$tempCond = GeneralUtility::trimExplode('.', $splittedCondition[1]);
if (count($tempCond) > 1) {
    $sheetName = $tempCond[0];
    $fieldName = implode(".", array_slice($tempCond, 1));
} else {
    list($sheetName, $fieldName) = $tempCond;
}
$fieldValue = $editData['data'][$sheetName][$lang][$fieldName];
$splittedCondition[1] = $fieldName;
$dataStruct['ROOT']['TCEforms']['displayCond'] = join(':', $splittedCondition);
$fakeRow = array($fieldName => $fieldValue);


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #42333: Flexform displayConditionRejected2012-10-24

Actions
Related to TYPO3 Core - Bug #60339: displayCond for sheets in flexform with extbase settings namespaceClosed2014-07-15

Actions
Actions #1

Updated by Marcin Sągol about 10 years ago

I can confirm this behaviour and also vote for this extended feature, where we can use not only first level field/configuration parameter name but whole path to it like: foo.bar.baz in condiftions.

Actions #2

Updated by Mathias Schreiber over 8 years ago

  • Target version deleted (next-patchlevel)
Actions #3

Updated by Morton Jonuschat over 8 years ago

  • Category changed from Backend API to FormEngine aka TCEforms
  • Status changed from New to Resolved

Has been resolved with the FormEngine Rewrite. Appropriate tests have been added with commit 655b5353

Actions #4

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF