Project

General

Profile

Actions

Bug #103108

closed

Undefined index: processor_stripColorProfileCommand

Added by Mike Street 9 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2024-02-13
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
11
PHP Version:
7.4
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

Before 11.5.35, processor_stripColorProfileCommand existed as a predefined string. This has now been removed in favour of a safer processor_stripColorProfileParameters.

processor_stripColorProfileCommand can still be defined and used in the code (by the looks of things), however, we were previously appending to it and now get an Undefined index: when running our sites.

We were appending with:

$GLOBALS['TYPO3_CONF_VARS']['GFX']['processor_stripColorProfileCommand'] .= ' -quality ' . $GLOBALS['TYPO3_CONF_VARS']['GFX']['jpg_quality'] . '% -sampling-factor 2x2,1x1,1x1';

It's also worth noting our command is also now missing +profile \'*\' at the start.

I appreciate this default needed to be removed so the array params took precedence. I'm happy to update the code - more raising it for visibility.

Related changelog: https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/11.5.x/Important-102799-TYPO3_CONF_VARSGFXprocessor_stripColorProfileParametersOptionAdded.html

Actions #1

Updated by Chris Müller 9 months ago

  • Status changed from New to Needs Feedback

There is no "processor_stripColorProfileCommand" default configuration available anymore:
https://github.com/TYPO3/typo3/commit/07f7ceca7537423a0ce196c20c23d92d1c2a7681#diff-d7a98dbd38a8309e3c7f644a8c3708393e486f0c7a2fadc6151d9d5b6681548b

So, you have to switch to the new processor_stripColorProfileParameters setting (or add the previous default to your setting instead of concatenating).

Actions #2

Updated by Christian Kuhn 9 months ago

  • Status changed from Needs Feedback to Closed

Hey. It looks as if there is nothing we can actually do here. It sounds as if the reporter opened the issue to raise additional awareness. I hope its ok to close here.

Actions

Also available in: Atom PDF