Project

General

Profile

Actions

Bug #81700

closed

Images get sharpend even if [GFX][processor_effects] is set to -1

Added by Klaus Moser almost 7 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Image Generation / GIFBUILDER
Target version:
-
Start date:
2017-06-26
Due date:
% Done:

100%

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

Description

In the current 8.7.2 version images are sharpened even if [GFX][processor_effects] is set to -1 (Do not sharpen images by default).

The reason for that is a wrong if condition in typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php on line 371.

is:

if ($gfxConf['processor_effects']) {
    $this->NO_IM_EFFECTS = 0;
    $this->cmds['jpg'] .= $this->v5_sharpen(10);
    $this->cmds['jpeg'] .= $this->v5_sharpen(10);
}

should be:

if ($gfxConf['processor_effects'] > 0) {
    $this->NO_IM_EFFECTS = 0;
    $this->cmds['jpg'] .= $this->v5_sharpen(10);
    $this->cmds['jpeg'] .= $this->v5_sharpen(10);
}

Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #83591: GraphicsMagick + [GFX][processor_effects] = '0' is set to '-1' just by entering the Install ToolClosedMarc Willmann2018-01-17

Actions
Related to TYPO3 Core - Bug #85059: Use int cast to prevent comparison failuresClosedFrank Nägler2018-05-22

Actions
Actions #1

Updated by Stefan P over 6 years ago

I can confirm this issue. After upgrading to TYPO3 8 the images suddenly get sharpened and this is most likely the cause.

Actions #2

Updated by Riccardo De Contardi about 6 years ago

  • Category set to Image Generation / GIFBUILDER
Actions #3

Updated by Gerrit Code Review about 6 years ago

  • Status changed from New to Under Review

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

Actions #4

Updated by Gerrit Code Review about 6 years ago

Patch set 2 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/56494

Actions #5

Updated by Stephan Großberndt about 6 years ago

  • Related to Bug #83591: GraphicsMagick + [GFX][processor_effects] = '0' is set to '-1' just by entering the Install Tool added
Actions #6

Updated by Gerrit Code Review about 6 years ago

Patch set 3 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/56494

Actions #7

Updated by Guido S. almost 6 years ago

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

Updated by Frank Nägler almost 6 years ago

  • Related to Bug #85059: Use int cast to prevent comparison failures added
Actions #9

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF