Bug #45833: BE List Module - issues with call to itemsProcFunc and non-selected fields - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #45833

closed

BE List Module - issues with call to itemsProcFunc and non-selected fields

Added by RVVN no-lastname-given over 11 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Should have

Assignee:

Category:

Target version:

4.6.17

Start date:

2013-02-25

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.6

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

TYPO3 4.6.16 apparently added a completely new fragment of code to "Have labels respect possible itemsProcFunc results" in BE List module, using t3lib_TCEForms->procItems.

file:
typo3_src/typo3/class.db_list_extra.inc

function:
renderListRow()

line:
634

The 'row' Array from the data object passed to itemsProcFunc only contains selected fields.
Problem is, if a itemsProcFunc needs the value of a non-selected field, an exception is raised because the 'row' Array doesn't contain this field.

In my personal case, I use an extension (Realty Manager - realty) with an itemsProcFunc for the "district" field that needs the value of the "city" field in order to retrieve districts for this city.
It's OK if I select "city" field, but crashes if not.

I'm not sure that's a core related ('row' Array may now contain all fields ?) or if that itemsProcFunc needs to be re-designed (maybe using pure TCA configuration with 'foreign_table'…).

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Michael Stucki over 11 years ago

Status changed from New to Under Review

I sent a mail to the author of the change that caused this problem, asking him to take a look at your bug.

Actions

Copy link

Updated by Christian Kuhn over 11 years ago

How to reproduce realty isuse, verified in core version 4.6:

install ext:realty and dependencies (eg. in 4.6)
Add some sysfolder to page
Go to list module, create a record 'Realty object', safe + close -> db error in list module.

The patch that introduced the issue has multiple other flaws and is reverted in all versions for now.

Actions

Copy link