Bug #71824

Story #69617: FormEngine bugs

displayCond FIELD can't operate on array values

Added by Sebastian Michaelsen almost 4 years ago. Updated almost 2 years ago.

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

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

Records passed to the EvaluateDisplayConditions provider can contain array values, which can't be properly processed with the capabilities of FIELD.

I have two suggestions:
  • The array values should be converted back to scalar values (like comma separated lists) for this evaluation => backwards compatible.
  • Operators are added that can handle array values (like COUNT or CONTAINS) => breaking change

I'll add an example extension in a moment.

bug_71824.zip (2.21 KB) Sebastian Michaelsen, 2015-11-25 12:39


Related issues

Related to TYPO3 Core - Bug #70265: DisplayCond based on array input fails Closed 2015-10-01

Associated revisions

Revision 5ef971d1 (diff)
Added by Morton Jonuschat almost 4 years ago

[BUGFIX] FormEngine: Handle empty arrays in display conditions

Records passed to the EvaluateDisplayConditions provider can contain
array values which can be empty in the case of new records. Handle this
condition in the evaluator.

Resolves: #71824
Releases: master
Change-Id: Ie5aa3f74913059f2a1e5933a8314dee6cb0fa1fa
Reviewed-on: https://review.typo3.org/45214
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Markus Klein <>
Tested-by: Markus Klein <>

History

#2 Updated by Christian Kuhn almost 4 years ago

  • Parent task set to #69617

#3 Updated by Gerrit Code Review almost 4 years ago

  • Status changed from New 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/45214

#4 Updated by Morton Jonuschat almost 4 years ago

  • Assignee set to Morton Jonuschat
  • Target version set to 7.6.1

#5 Updated by Morton Jonuschat almost 4 years ago

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

#6 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF