Project

General

Profile

Actions

Bug #17057

closed

Record alt_label doesn't get processed

Added by Ralf Hettinger over 17 years ago. Updated over 16 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
-
Start date:
2007-02-28
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.1
PHP Version:
4.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

This one is somehow related to http://bugs.typo3.org/view.php?id=4943 according to the concerned function. But this bug here has another topic: alt_labels aren't processed anymore (e.g. datetime evaluation) - a regression since version 4.0.

The reason is that the processing functionality was removed from typo3/class.db_list_extra.inc and it seems plausible to do this in t3lib_BEfunc::getRecordTitle because alt_labels are handled anyway.

Attached patch is not against SVN but against 4.1.0RC2. Functionality of David Bruehlmeier's patch in bug 4943 is applied (code of the attached patch might be slightly faster if no label_alt_force is set by avoiding one if clause).

(issue imported from #M5094)


Files

0005094.patch (1015 Bytes) 0005094.patch Administrator Admin, 2007-02-28 21:54
0005094_1.patch (1015 Bytes) 0005094_1.patch Administrator Admin, 2007-02-28 22:02
5094_vg.patch (1.39 KB) 5094_vg.patch Administrator Admin, 2007-03-14 11:57
5094_v2.patch (1.15 KB) 5094_v2.patch Administrator Admin, 2007-04-19 15:00
5094_vg2.patch (997 Bytes) 5094_vg2.patch Administrator Admin, 2007-04-27 21:53
Actions #1

Updated by Ralf Hettinger over 17 years ago

So sorry. Again a first version with a typo. Please use patch _1

Actions #2

Updated by Volker Graubaum over 17 years ago

works fine for me ;-)

Actions #3

Updated by Volker Graubaum over 17 years ago

sorry doesn't work fine, a few other things get mixed up
see attached patch

Actions #4

Updated by Volker Graubaum over 17 years ago

Additional information to this bug:

1) no alt_label is used anymore, if label_alt_force is NOT set, also
the label ist empty

2) the alt_label isn't parsed, but this seemed to be also in the title
will have a look at it later this afternoon -)

Actions #5

Updated by Ralf Hettinger over 17 years ago

Volker, I must admit that I don't understand why you suggest a second loop within the first one. I don't think that this will help but potentially will slow the list module down.

Could you let me know which other things got messed up by the first patch, please?

Actions #7

Updated by Tobias Klepp over 17 years ago

Hello,

I have a problem with the list view in the backend. After the update to the version 4.1.1 the date in the list view is not formated as date, only the timestamp is indicated. This concerns only the title column, not additional columns, which are set manually.

Geetings, Tobias Klepp

Actions #8

Updated by Ralf Hettinger over 17 years ago

Attached patch 5094_v2.patch against 4.1.1 is known to solve Tobias Klepp's issue and issues that might come up for tt_news label fields.

I still have no idea which "other things might get mixed up" but have none of them experienced so far.

@Volker Graubaum: Your feedback would be very welcome.

Actions #9

Updated by Volker Graubaum over 17 years ago

Sorry, no time the last weeks,
solution will be found at the devDays :-)

Actions #10

Updated by Volker Graubaum over 17 years ago

Still not sure, what I meant, except I now detected das IRRE don't use label_alt.
Change you're last patch, so

$t = t3lib_BEfunc::getProcessedValue($table,$fN,$t);
comes before the label_alt_force check, since it should also parse "normal" alt labels....
Actions #12

Updated by Ingo Schmitt over 17 years ago

Patch works for me fine. Would be great to have it in the core.

Actions #13

Updated by Michael Stucki over 17 years ago

Fixed in TYPO3_4-1 and Trunk.

Actions

Also available in: Atom PDF