Bug #56274

Non-image files should have no thumbnails

Added by Alexander Stehlik over 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend User Interface
Target version:
Start date:
2014-02-25
Due date:
% Done:

100%

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

Description

When you upload a non-image file in the Backend the system still tries to generate a thumbnail which of course will fail.

The alt-Text will be displayed instead which is also invalid because it currently contains the description list that is used for displaying the file information (like filename and title).

See attached screenshot.

screenshot_invalid_thumbnail.png View (14.9 KB) Alexander Stehlik, 2014-02-25 15:44


Related issues

Duplicates TYPO3 Core - Bug #56886: File Upload (videos) in Page Resources, not recognizing a video file Closed 2014-03-13

Associated revisions

Revision 6db9872d (diff)
Added by Alexander Stehlik about 6 years ago

[BUGFIX] Fix thumbnail display for IRRE records

The system will only generate a thumbnail for IRRE records if the
related file object is an image file.

Additionally the UID of the file reference will be used as alternative
text and as title for the thumbnail so that it behaves like the icon
that is used for non-image files.

Resolves: #56274
Releases: 6.2
Change-Id: Ibe6530368434c384d5b7cca93e81b656c8c56383
Reviewed-on: https://review.typo3.org/27844
Reviewed-by: Frans Saris
Tested-by: Frans Saris
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

History

#1 Updated by Gerrit Code Review over 6 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27844

#2 Updated by Gerrit Code Review about 6 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27844

#3 Updated by Gerrit Code Review about 6 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27844

#4 Updated by Gerrit Code Review about 6 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27844

#5 Updated by Frans Saris about 6 years ago

  • Status changed from Under Review to Closed

Closed as this is a duplicate

#6 Updated by Alexander Stehlik about 6 years ago

Frans, you closed both tickets.

Should not at least one of the duplicates stay open. Ideally this one ;)

#7 Updated by Frans Saris about 6 years ago

  • Status changed from Closed to Under Review

oeps :)

#8 Updated by Alexander Stehlik about 6 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#9 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF