Bug #50301
Generated thumbnail's URL with BackendUtility::getThumbNail don't work
| Status: | Accepted | Start date: | 2013-07-22 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | ||
| Assigned To: | - | % Done: | 0% |
|
| Category: | Backend API | Spent time: | - | |
| Target version: | - | |||
| TYPO3 Version: | 6.0 | Is Regression: | No | |
| PHP Version: | 5.4 | Sprint Focus: | ||
| Complexity: |
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
History
#1 Updated by Gerrit Code Review about 2 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 about 2 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 almost 2 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 almost 2 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 almost 2 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 almost 2 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 over 1 year ago
- Status changed from Under Review to Accepted
- Is Regression set to No
Current patch has been abandoned.
#8 Updated by Benjamin Mack 7 months ago
please also see https://review.typo3.org/#/c/35694/