Bug #79903

Processing of SVG, AI etc is broken

Added by Frans Saris over 2 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2017-02-19
Due date:
% Done:

100%

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

Description

When you try to use SVG of AI as images in a CE the (preview)processing in FE and BE is broken as the core tries to keep there existing file extension but isn't able to generate the correct content for this.

this was introduced with #73106 were before all files except jp(e)g, png and gif were changed to png or gif depending on the configuration.

ImageMagick/GraphicsMagick probably can handle more output formats, but the rest of our processing chain can handle that properly at the moment, so for now we need to keep the output formats to only those formats we know we can handle and are supported by the browsers to use directly in the HTML output.


Related issues

Related to TYPO3 Core - Bug #79836: fluid f:image SVG processing Closed 2017-02-15
Related to TYPO3 Core - Task #73106: Deprecate "thumbnails_png" Closed 2016-02-03

Associated revisions

Revision f1fbc80f (diff)
Added by Frans Saris over 2 years ago

[BUGFIX] Fixed processing of non jpg, png and gif images

Resolves: #79903
Releases: master
Change-Id: I159286a17bbb7dbab9d5ef696608e0e4d8558e7d
Reviewed-on: https://review.typo3.org/51751
Tested-by: TYPO3com <>
Reviewed-by: Anders Kostending <>
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>
Reviewed-by: Josef Glatz <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

History

#1 Updated by Frans Saris over 2 years ago

  • Description updated (diff)

#2 Updated by Frans Saris over 2 years ago

  • Description updated (diff)

#3 Updated by Gerrit Code Review over 2 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/51751

#4 Updated by Gerrit Code Review over 2 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/51751

#5 Updated by Gerrit Code Review over 2 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/51751

#6 Updated by Frans Saris over 2 years ago

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

#7 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF