Bug #84635

Folow-Up to https://review.typo3.org/#/c/55789/ - Unused code / ignored 8'typeicon_classes']['default'] and some other things

Added by Andreas Allacher 12 months ago. Updated 10 days ago.

Status:
New
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2018-04-06
Due date:
% Done:

0%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

With the bugfix of #82850 and the https://review.typo3.org/#/c/55789/
if there is an issue with the record it now won't result in an exception.
However, the introduced code https://github.com/TYPO3/TYPO3.CMS/commit/aa4ce89d5263d5258ae8e35dc5daeb9eb9fae561#diff-8a7e6fca77c3b402775cb5b9ca9f5e99 sets

 $icon = $GLOBALS['TCA'][$table]['ctrl']['typeicon_classes']['default'];

which is completely ignored when retrieving the icon.

History

#1 Updated by Andreas Allacher 12 months ago

Also I think the table name should be part of the explanation - if there is a table. Currently only the ID and the icon would be shown.
Of course, if the configuration is missing entirely and not just the record then this won't work but then the icon will not be available either, so in that case it might just be best to output the link identifier and the uid.

#2 Updated by Andreas Allacher 12 months ago

  • Subject changed from Folow-Up to https://review.typo3.org/#/c/55789/ - Unused code / ignored 8'typeicon_classes']['default'] to Folow-Up to https://review.typo3.org/#/c/55789/ - Unused code / ignored 8'typeicon_classes']['default'] and some other things

#3 Updated by Riccardo De Contardi 10 days ago

Does this issue still exists on 9.5.5 or later? The code reported on the description in the file typo3/sysext/backend/Classes/Form/Element/InputLinkElement.php has been changed AFAICS

Also available in: Atom PDF