Project

General

Profile

Actions

Bug #14636

closed

Listview ignores label_alt / label_alt_force configuration (with patch)

Added by Tim Franz over 19 years ago. Updated almost 18 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2005-04-05
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
3.8.0beta1
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The change from Rupert Germann in db_list_extra.inc to resolve mm-tables ignores the usage of label_alt and label_alt_force. The title is only constructed from the primaray title field.

reason: the corresponding method t3lib_BEfunc::getRecordTitle is never called. instead the title is generated in db_list_extra.inc by only calling t3lib_BEfunc::getProcessedValueExtra.

Side effect: By replacing t3lib_BEfunc::getProcessedValueExtra by t3lib_BEfunc::getRecordTitle the advanced title field generation is back, but the resolution of mm-fields gone. Solution for that: my enhancement #0000816 improves method getRecordTitle.

(issue imported from #M935)


Files

befunc_dblist.diff (1.97 KB) befunc_dblist.diff Administrator Admin, 2005-04-06 21:40
class.db_list_extra.inc.diff (1.32 KB) class.db_list_extra.inc.diff Administrator Admin, 2005-04-07 15:23

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Feature #14559: Prettier record titles (with diff)ClosedRupert Germann2005-02-19

Actions
Related to TYPO3 Core - Bug #14629: Title lenght in BE is not working if title has no valueClosedRupert Germann2005-03-30

Actions
Actions #1

Updated by Michael Stucki over 19 years ago

Hi Rupert, can you please have a look at this one? If you find some time to compile a patch for me, that would be great! Thanks. Michael

Actions #2

Updated by Rupert Germann over 19 years ago

that works. I made a patch that includes the patch from bug 816.

Actions #3

Updated by Michael Stucki over 19 years ago

Thanks, Rupi! Will add this and confirm when done...

Actions #4

Updated by Michael Stucki over 19 years ago

This is what Kasper just replied to me:

--- cut ---
No, you cannot commit this patch. The reason is:

getRecordTitle() is basing its decision to return a title only on the
"ctrl" part of TCA which is always loaded in memory.
t3lib_BEfunc::getProcessedValueExtra always loads the whole definition
of a table because it needs to investigate configuration in "columns"
key.
Since getRecordTitle() is supposes to be light weight and is implemented
so many places I cannot accept this possible overhead of processing for
such a small feature.

Therefore the implementation of the 'label_alt' and 'label_alt_force'
must be done in the
/cvsroot/typo3/TYPO3core/typo3/class.db_list_extra.inc file locally
somehow.
--- cut ---

Actions #5

Updated by Rupert Germann over 19 years ago

How about this:
the latest patch changes only the file class.db_list_extra.inc (in function renderListRow()) and fixes also the unprocessed record titles which are mentioned in bug 816

I want also mention bug 920 here, which is BE title related too.

Actions #6

Updated by Rupert Germann about 19 years ago

fixed in cvs

Actions

Also available in: Atom PDF