Project

General

Profile

Actions

Bug #94648

closed

GifBuilder does not recognize all changes in config when creating Hash

Added by manu zoli over 3 years ago. Updated 5 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Image Generation / GIFBUILDER
Target version:
-
Start date:
2021-07-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

In the Feature #64158 the way hashes are created was changed. Only certain parameters are taken into consideration when trying to figure out if a picture has to be reprocessed or if the cached picture can be loaded.

On line 740 the keys of the setup array are used but not the set values.

array_keys($this->setup)

Some additional parameters like width and offset are considered as well, but when you start to configure encapsulated images, those changes are not considered. This causes the GifBuilder to give back old cached images even when a new processing would be needed.

Example - the offset parameter of the 10 = Image is not gonna be considered:

$imgConf['file.']['format'] = $fileF[1];
$imgConf['file.']['backColor'] = 'transparent';
$imgConf['file.']['10'] = 'IMAGE';
$imgConf['file.']['10.']['file'] = $filePath;
$imgConf['file.']['10.']['file.']['width'] = $settings['width'];
$imgConf['file.']['10.']['offset'] = $settings['left'] . "," . $settings['top'];

I think this should be fixed by changing the line 740 to something like:

json_encode($this->setup),

This will be somewhat slower, but will at least yield the desired result.


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #93688: Gifbuilder - Caching Problem - no update of old combined imagesRejected2021-03-09

Actions
Related to TYPO3 Core - Feature #64158: A faster way to generate GIFBUILDER short MD5 hash inside a filenameClosed2015-01-07

Actions
Related to TYPO3 Core - Bug #97212: GifBuilder renders wrong image from example typoscript in documentationClosed2022-03-19

Actions
Actions

Also available in: Atom PDF