Bug #50301

Generated thumbnail's URL with BackendUtility::getThumbNail don't work

Added by Matthias Toscanelli over 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2013-07-22
Due date:
% Done:

0%

TYPO3 Version:
6.0
PHP Version:
5.4
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When a thumbnail's URL is generated with BackendUtility::getThumbNail, the "thumbs.php" script can't display it.
It trows an exception:

TYPO3 Fatal Error: The requested image does not exist and/or MD5 checksum did not match. If the target file exists and its file name contains special characters, the setting of $TYPO3_CONF_VARS[SYS][systemLocale] might be wrong.

The bug comes from MD5 checksum:

  • In "BackendUtility::getThumbNail", the checksum is calculated with "md5"
  • In "ThumbnailView::init", the checksum is calculated with "GeneralUtility::shortMD5"

I don't know which one should be changed, i let you choose.


Related issues

Related to TYPO3 Core - Bug #59215: t3lib_BEfunc::getThumbnail doesn't work anymore Closed 2014-05-29
Blocked by TYPO3 Core - Feature #33751: Make use of File Abstraction Layer in Backend Closed 2012-02-07

History

#1 Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499

#2 Updated by Markus Klein over 6 years ago

For version 6.0 the FAL Team removed the wrong line of code. (http://review.typo3.org/10655)

$md5pre47 = t3lib_div::shortMD5($check);
$md5from47 = md5($check);
if (!strcmp($md5pre47, $md5sum) || !strcmp($md5from47, $md5sum)) {
    $OK = TRUE;
}

was reduced to:

$md5_real = t3lib_div::shortMD5($check);
if (!strcmp($md5_real, $md5sum)) {
    $OK = TRUE;
}

#3 Updated by Gerrit Code Review over 6 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499

#4 Updated by Gerrit Code Review over 6 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499

#5 Updated by Gerrit Code Review over 6 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499

#6 Updated by Gerrit Code Review over 6 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499

#7 Updated by Markus Klein about 6 years ago

  • Status changed from Under Review to Accepted
  • Is Regression set to No

Current patch has been abandoned.

#9 Updated by Mathias Schreiber almost 4 years ago

  • Status changed from Accepted to Resolved

#10 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF