Bug #50301
closedGenerated thumbnail's URL with BackendUtility::getThumbNail don't work
0%
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.
Updated by Gerrit Code Review over 11 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
Updated by Markus Klein over 11 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; }
Updated by Gerrit Code Review over 11 years ago
Patch set 2 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499
Updated by Gerrit Code Review over 11 years ago
Patch set 3 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499
Updated by Gerrit Code Review over 11 years ago
Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499
Updated by Gerrit Code Review over 11 years ago
Patch set 5 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22499
Updated by Markus Klein about 11 years ago
- Status changed from Under Review to Accepted
- Is Regression set to No
Current patch has been abandoned.
Updated by Benni Mack about 10 years ago
please also see https://review.typo3.org/#/c/35694/
Updated by Mathias Schreiber almost 9 years ago
- Status changed from Accepted to Resolved