Project

General

Profile

Actions

Bug #83591

closed

GraphicsMagick + [GFX][processor_effects] = '0' is set to '-1' just by entering the Install Tool

Added by S P almost 7 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Install Tool
Target version:
-
Start date:
2018-01-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
On Location Sprint

Description

Whenever entering the Install Tool, the setting [GFX][processor_effects] is set to '-1'.

I started with the GraphicsMagick preset and altered the LocalConfiguration.php by hand, setting [GFX][processor_effects] to '0'. This works until I click again on the Install Tool module. Navigating within the Install Tool directly after changing the value, you have to re-enter the Install Tool completly.

I work-around this by setting the value in AdditionalConfiguration.php which takes precedence over LocalConfiguration.php, but ideally this should be fixed in the core.

Bug #81700 is probably related. I'm not sure if this happens with ImageMagick as well, because I only have access to GraphicsMagick in the setup where I encountered this.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #81700: Images get sharpend even if [GFX][processor_effects] is set to -1Closed2017-06-26

Actions
Related to TYPO3 Core - Task #82768: Graphics APIClosedBenni Mack2017-10-16

Actions
Actions #1

Updated by S P almost 7 years ago

Whoops, the unfinished sentence should read:
Navigating within the Install Tool directly after changing the value does not trigger the effect, you have to re-enter the Install Tool completly (which immediatly triggers it).

Actions #2

Updated by Susanne Moog almost 7 years ago

  • Category set to Install Tool
Actions #3

Updated by Stephan Großberndt over 6 years ago

  • Related to Bug #81700: Images get sharpend even if [GFX][processor_effects] is set to -1 added
Actions #4

Updated by Guido S. over 6 years ago

It's not a bug, it's a feature :-)

typo3\sysext\install\Classes\Service\SilentConfigurationUpgradeService.php checks if GraphicsMagick is active, and return -1 for processor_effects for GM:

        if ((string)$currentProcessorValue !== '') {
            if ($currentProcessorMaskValue != 0) {
                $changedValues['GFX/processor_allowTemporaryMasksAsPng'] = 0;
            }
            if ($currentProcessorValue === 'GraphicsMagick') {
                if ($currentProcessorEffectsValue != -1) {
                    $changedValues['GFX/processor_effects'] = -1;
                }
            }
        }

This could be a historic decision because sharpen might not always be available to GM. The code is also active in the current master.

My two cents: We can remove this check, see current gm doc: http://www.graphicsmagick.org/GraphicsMagick.html#details-sharpen If a gm version does not support sharpen, you can disable this manually at processor_effects.

Actions #5

Updated by Riccardo De Contardi over 6 years ago

  • Sprint Focus set to On Location Sprint
Actions #6

Updated by Gerrit Code Review over 6 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/57319

Actions #7

Updated by Susanne Moog over 6 years ago

  • Assignee set to Marc Willmann
Actions #8

Updated by Gerrit Code Review over 6 years ago

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/57324

Actions #9

Updated by Gerrit Code Review over 6 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/57319

Actions #10

Updated by Gerrit Code Review over 6 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/57319

Actions #11

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57339

Actions #12

Updated by Marc Willmann over 6 years ago

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

Updated by Gerrit Code Review over 6 years ago

  • Status changed from Resolved 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/57391

Actions #14

Updated by Gerrit Code Review over 6 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/57391

Actions #15

Updated by Gerrit Code Review over 6 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/57391

Actions #16

Updated by Mathias Brodala over 6 years ago

Actions #17

Updated by Gerrit Code Review over 6 years ago

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

Actions #18

Updated by Gerrit Code Review over 6 years ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/57395

Actions #19

Updated by Mathias Brodala over 6 years ago

  • Status changed from Under Review to Resolved
Actions #20

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF