Project

General

Profile

Actions

Bug #69952

closed

Story #69617: FormEngine bugs

processForeignTableClause fails when used with select fields

Added by Mathias Schreiber about 9 years ago. Updated about 7 years ago.

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

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.5
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
Stabilization Sprint

Description

When processing the dataBaseRow select values become an array with only the selected option (thus one array entry).
This fails when given to the processForeignTableClause method because fullQuoteStr isn't too keen on arrays being given.

Quick fix is attached, although it seems a bit too easy ;-)


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #70523: processForeignTableClause in case of array should be a comma separated list (e.g. MM relations)RejectedMathias Schreiber2015-10-09

Actions
Actions #1

Updated by Gerrit Code Review about 9 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 http://review.typo3.org/43425

Actions #2

Updated by Mathias Schreiber about 9 years ago

  • Parent task set to #69617
Actions #3

Updated by Gerrit Code Review about 9 years ago

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

Actions #4

Updated by Gerrit Code Review about 9 years ago

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

Actions #5

Updated by Mathias Schreiber about 9 years ago

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

Updated by Riccardo De Contardi about 7 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF