Bug #103111
open
processor_stripColorProfileParameters splits up single value into multiple values
0%
Description
When migrating from the previous "GFX / processor_stripColorProfileCommand" to the new "GFX / processor_stripColorProfileParameters" configuration, it no longer seems possible to use a value like "+profile '!iptc,*'" via the TYPO3 Install Tool.
When the actual value contains a comma, the Install Tool splits the otherwise single parameter "!iptc,*" into two parameters "!iptc" and "*".
How is the intended usage to configure such parameters in the new "processor_stripColorProfileParameters" style?
Updated by Chris Müller 3 months ago
- Subject changed from processor_stripColorProfileParameters spits up single value into multiple values to processor_stripColorProfileParameters splits up single value into multiple values
Updated by Markus Klein 3 months ago
First entry should be +profile second array entry should be !iptc,* in the new config.
The parsing bug should be fixed.
Updated by Markus Klein 3 months ago
- Target version deleted (
next-patchlevel)
@Stefan Wagler Can you please post here, how you configured 'processor_stripColorProfileParameters' exactly?
Updated by Stefan Wagler 3 months ago
· Edited
We use the following configuration entry in the "GFX" section of the "settings.php" file:
'processor_stripColorProfileCommand' => '+profile \'!iptc,*\'',
When using the Install Tool and set the following option for "[GFX][processor_stripColorProfileParameters]":
+profile '!iptc,*'
the "settings.php" file contains the following entry:
'processor_stripColorProfileParameters' => [
'+profile \'!iptc',
'*\'',
],
Note the second parameter '!iptc,*' exists partly in the first array entry as well as in the second entry.
Updated by Markus Klein 3 months ago
· Edited
- Status changed from New to Accepted
Thanks for the clarification.
When using the Install Tool and set the following option for "[GFX][processor_stripColorProfileParameters]": +profile '!iptc,*'
This is wrong. As documented for this setting (Comma separated list of parameters), you should enter +profile,"!iptc,*".
Workaround: one should be able to fix this by manually changing the settings.php file.
Regard the parsing bug: The Install Tool needs to cope with quoted parts in such comma-separated lists.
Updated by Gerrit Code Review 3 months ago
- Status changed from Accepted to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83003
Updated by Gerrit Code Review 3 months ago
Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/83003