Feature #62886

Epic #62885: Strong defaults

Set config.meaningfulTempFilePrefix by default

Added by Georg Ringer almost 5 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
-
Target version:
Start date:
2014-11-12
Due date:
% Done:

100%

PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

Having config.meaningfulTempFilePrefix as default makes the output of TYPO3 better

Associated revisions

Revision 94e915fe (diff)
Added by Georg Ringer over 4 years ago

[!!!][TASK] Gifbuilder: Always use full filename of original file as prefix

As of now, images created with GIFBUILDER were created by a
hash in typo3temp/GB/. With the TypoScript option
config.meaningfulTempFilePrefix it was possible to prepend the
first X characters of the original filename in the filename.
This was due to some FS limitations on Windows.

With FAL the option was not used anymore, but Gifbuilder
still used this option.

In order to streamline the generation of temporary
files, images created by the Gifbuilder now contain
the full filename of the original file as well.

The TypoScript option "config.meaningfulTempFilePrefix"
has no effect anymore now.

Resolves: #62886
Releases: master
Change-Id: I746aa5d8c2e483212c28419dadb68a5cd5d1282d
Reviewed-on: http://review.typo3.org/34015
Reviewed-by: Eric Chavaillaz <>
Tested-by: Eric Chavaillaz <>
Reviewed-by: Benjamin Mack <>
Tested-by: Benjamin Mack <>

History

#1 Updated by Gerrit Code Review almost 5 years ago

  • Status changed from Accepted 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 http://review.typo3.org/34015

#2 Updated by Gerrit Code Review almost 5 years ago

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

#3 Updated by Gerrit Code Review almost 5 years ago

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

#4 Updated by Gerrit Code Review almost 5 years ago

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

#5 Updated by Gerrit Code Review almost 5 years ago

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

#6 Updated by Gerrit Code Review almost 5 years ago

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

#7 Updated by Mathias Schreiber almost 5 years ago

  • Target version changed from 7.0 to 7.1 (Cleanup)

#8 Updated by Georg Ringer over 4 years ago

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

#9 Updated by Philipp Gampe over 4 years ago

The discussion why the setting is not needed any more is missing here and in the issue.

#10 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF