From 647678e505019b4c1c10acf285b55ee24bde5801 Mon Sep 17 00:00:00 2001
From: Benni Mack <benni@typo3.org>
Date: Sun, 8 Oct 2017 14:29:23 +0200
Subject: [PATCH] [!!!][TASK] Clean up graphical functions library
Properties which are only accessed inside of
GraphicalFunctions and should remain inside are marked
as protected, are migrated to arrays (to avoid inList calls
instead of in_array()), removed or renamed to remove
classic kasper's-style double negation namings.
Resolves: #82768
Releases: master
Change-Id: If45a9198c3a2979a6fe0a1619a9841bd37a2708e
Reviewed-on: https://review.typo3.org/54318
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
---
.../Classes/Imaging/GraphicalFunctions.php | 156 +++++++-----------
.../Processing/LocalCropScaleMaskHelper.php | 1 -
.../Processing/LocalPreviewHelper.php | 1 -
.../Configuration/DefaultConfiguration.php | 2 +-
.../DefaultConfigurationDescription.yaml | 8 +-
...ationOptionsForImageManipulationPHPAPI.rst | 63 +++++++
.../frontend/Classes/Imaging/GifBuilder.php | 2 +-
.../Controller/EnvironmentController.php | 9 +-
8 files changed, 135 insertions(+), 107 deletions(-)
create mode 100644 typo3/sysext/core/Documentation/Changelog/master/Breaking-82768-ConfigurationOptionsForImageManipulationPHPAPI.rst
diff --git a/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php b/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
index ae0391d72ac5..8afcbefbc849 100644
--- a/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
+++ b/typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
@@ -47,9 +47,9 @@ class GraphicalFunctions
/**
* File formats supported by gdlib. This variable get's filled in "init" method
*
- * @var string
+ * @var array
*/
- public $gdlibExtensions = '';
+ protected $gdlibExtensions = [];
/**
* defines the RGB colorspace to use
@@ -100,24 +100,26 @@ class GraphicalFunctions
public $truecolorColors = 16777215;
/**
- * Commalist of file extensions perceived as images by TYPO3.
- * List should be set to 'gif,png,jpeg,jpg' if IM is not available. Lowercase and no spaces between!
+ * Allowed file extensions perceived as images by TYPO3.
+ * List should be set to 'gif,png,jpeg,jpg' if IM is not available.
*
- * @var string
+ * @var array
*/
- public $imageFileExt = 'gif,jpg,jpeg,png,tif,bmp,tga,pcx,ai,pdf';
+ protected $imageFileExt = ['gif', 'jpg', 'jpeg', 'png', 'tif', 'bmp', 'tga', 'pcx', 'ai', 'pdf'];
/**
- * Commalist of web image extensions (can be shown by a webbrowser)
+ * Web image extensions (can be shown by a webbrowser)
*
- * @var string
+ * @var array
*/
- public $webImageExt = 'gif,jpg,jpeg,png';
+ protected $webImageExt = ['gif', 'jpg', 'jpeg', 'png'];
/**
- * @var string
+ * Enable ImageMagick effects, disabled by default as IM5+ effects slow down the image generation
+ *
+ * @var bool
*/
- public $NO_IM_EFFECTS = '';
+ protected $processorEffectsEnabled = false;
/**
* @var array
@@ -130,14 +132,14 @@ class GraphicalFunctions
];
/**
- * @var string
+ * @var bool
*/
- public $NO_IMAGE_MAGICK = '';
+ protected $NO_IMAGE_MAGICK = false;
/**
* @var bool
*/
- public $mayScaleUp = 1;
+ protected $mayScaleUp = true;
/**
* Filename prefix for images scaled in imageMagickConvert()
@@ -158,21 +160,14 @@ class GraphicalFunctions
*
* @var bool
*/
- public $dontCheckForExistingTempFile = 0;
+ public $dontCheckForExistingTempFile = false;
/**
* Prevents imageMagickConvert() from compressing the gif-files with self::gifCompress()
*
* @var bool
*/
- public $dontCompress = 0;
-
- /**
- * For debugging ONLY!
- *
- * @var bool
- */
- public $dontUnlinkTempFiles = 0;
+ public $dontCompress = false;
/**
* For debugging only.
@@ -202,13 +197,6 @@ class GraphicalFunctions
*/
protected $saveAlphaLayer = false;
- /**
- * Prefix for relative paths. Used in "show_item.php" script. Is prefixed the output file name IN imageMagickConvert()
- *
- * @var string
- */
- public $absPrefix = '';
-
/**
* ImageMagick scaling command; "-geometry" or "-sample". Used in makeText() and imageMagickConvert()
*
@@ -221,28 +209,28 @@ class GraphicalFunctions
*
* @var string
*/
- public $im5fx_blurSteps = '1x2,2x2,3x2,4x3,5x3,5x4,6x4,7x5,8x5,9x5';
+ protected $im5fx_blurSteps = '1x2,2x2,3x2,4x3,5x3,5x4,6x4,7x5,8x5,9x5';
/**
* Used by v5_sharpen() to simulate 10 continuous steps of sharpening.
*
* @var string
*/
- public $im5fx_sharpenSteps = '1x2,2x2,3x2,2x3,3x3,4x3,3x4,4x4,4x5,5x5';
+ protected $im5fx_sharpenSteps = '1x2,2x2,3x2,2x3,3x3,4x3,3x4,4x4,4x5,5x5';
/**
* This is the limit for the number of pixels in an image before it will be rendered as JPG instead of GIF/PNG
*
* @var int
*/
- public $pixelLimitGif = 10000;
+ protected $pixelLimitGif = 10000;
/**
* Array mapping HTML color names to RGB values.
*
* @var array
*/
- public $colMap = [
+ protected $colMap = [
'aqua' => [0, 255, 255],
'black' => [0, 0, 0],
'blue' => [0, 0, 255],
@@ -266,12 +254,12 @@ class GraphicalFunctions
*
* @var CharsetConverter
*/
- public $csConvObj;
+ protected $csConvObj;
/**
* @var int
*/
- public $jpegQuality = 85;
+ protected $jpegQuality = 85;
/**
* @var string
@@ -299,7 +287,7 @@ class GraphicalFunctions
/**
* @var array
*/
- public $OFFSET;
+ protected $OFFSET;
/**
* @var resource
@@ -314,13 +302,14 @@ class GraphicalFunctions
{
$gfxConf = $GLOBALS['TYPO3_CONF_VARS']['GFX'];
if (function_exists('imagecreatefromjpeg') && function_exists('imagejpeg')) {
- $this->gdlibExtensions .= ',jpg,jpeg';
+ $this->gdlibExtensions[] = 'jpg';
+ $this->gdlibExtensions[] = 'jpeg';
}
if (function_exists('imagecreatefrompng') && function_exists('imagepng')) {
- $this->gdlibExtensions .= ',png';
+ $this->gdlibExtensions[] = 'png';
}
if (function_exists('imagecreatefromgif') && function_exists('imagegif')) {
- $this->gdlibExtensions .= ',gif';
+ $this->gdlibExtensions[] = 'gif';
}
if ($gfxConf['processor_colorspace'] && in_array($gfxConf['processor_colorspace'], $this->allowedColorSpaceNames, true)) {
@@ -328,7 +317,7 @@ class GraphicalFunctions
}
if (!$gfxConf['processor_enabled']) {
- $this->NO_IMAGE_MAGICK = 1;
+ $this->NO_IMAGE_MAGICK = true;
}
// Setting default JPG parameters:
$this->jpegQuality = MathUtility::forceIntegerInRange($gfxConf['jpg_quality'], 10, 100, 85);
@@ -336,24 +325,21 @@ class GraphicalFunctions
if ($gfxConf['gdlib_png']) {
$this->gifExtension = 'png';
}
- $this->imageFileExt = $gfxConf['imagefile_ext'];
+ $this->imageFileExt = GeneralUtility::trimExplode(',', $gfxConf['imagefile_ext']);
// Boolean. This is necessary if using ImageMagick 5+.
// Effects in Imagemagick 5+ tends to render very slowly!!
// - therefore must be disabled in order not to perform sharpen, blurring and such.
- $this->NO_IM_EFFECTS = 1;
$this->cmds['jpg'] = $this->cmds['jpeg'] = '-colorspace ' . $this->colorspace . ' -quality ' . $this->jpegQuality;
// ... but if 'processor_effects' is set, enable effects
if ($gfxConf['processor_effects']) {
- $this->NO_IM_EFFECTS = 0;
+ $this->processorEffectsEnabled = true;
$this->cmds['jpg'] .= $this->v5_sharpen(10);
$this->cmds['jpeg'] .= $this->v5_sharpen(10);
}
// Secures that images are not scaled up.
- if (!$gfxConf['processor_allowUpscaling']) {
- $this->mayScaleUp = 0;
- }
+ $this->mayScaleUp = (bool)$gfxConf['processor_allowUpscaling'];
$this->csConvObj = GeneralUtility::makeInstance(CharsetConverter::class);
}
@@ -377,14 +363,14 @@ class GraphicalFunctions
if ($conf['file'] && $conf['mask']) {
$imgInf = pathinfo($conf['file']);
$imgExt = strtolower($imgInf['extension']);
- if (!GeneralUtility::inList($this->gdlibExtensions, $imgExt)) {
+ if (!in_array($imgExt, $this->gdlibExtensions, true)) {
$BBimage = $this->imageMagickConvert($conf['file'], $this->gifExtension);
} else {
$BBimage = $this->getImageDimensions($conf['file']);
}
$maskInf = pathinfo($conf['mask']);
$maskExt = strtolower($maskInf['extension']);
- if (!GeneralUtility::inList($this->gdlibExtensions, $maskExt)) {
+ if (!in_array($maskExt, $this->gdlibExtensions, true)) {
$BBmask = $this->imageMagickConvert($conf['mask'], $this->gifExtension);
} else {
$BBmask = $this->getImageDimensions($conf['mask']);
@@ -441,11 +427,9 @@ class GraphicalFunctions
$im = $backIm;
}
// Unlink files from process
- if (!$this->dontUnlinkTempFiles) {
- unlink($theDest);
- unlink($theImage);
- unlink($theMask);
- }
+ unlink($theDest);
+ unlink($theImage);
+ unlink($theMask);
}
}
}
@@ -461,7 +445,7 @@ class GraphicalFunctions
public function copyImageOntoImage(&$im, $conf, $workArea)
{
if ($conf['file']) {
- if (!GeneralUtility::inList($this->gdlibExtensions, $conf['BBOX'][2])) {
+ if (!in_array($conf['BBOX'][2], $this->gdlibExtensions, true)) {
$conf['BBOX'] = $this->imageMagickConvert($conf['BBOX'][3], $this->gifExtension);
$conf['file'] = $conf['BBOX'][3];
}
@@ -641,7 +625,7 @@ class GraphicalFunctions
$this->ImageWrite($maskImg, $fileMask);
imagedestroy($maskImg);
// Downscales the mask
- if ($this->NO_IM_EFFECTS) {
+ if (!$this->processorEffectsEnabled) {
$command = trim($this->scalecmd . ' ' . $w . 'x' . $h . '! -negate');
} else {
$command = trim($conf['niceText.']['before'] . ' ' . $this->scalecmd . ' ' . $w . 'x' . $h . '! ' . $conf['niceText.']['after'] . ' -negate');
@@ -670,11 +654,9 @@ class GraphicalFunctions
$im = $backIm;
}
// Deleting temporary files;
- if (!$this->dontUnlinkTempFiles) {
- unlink($fileMenu);
- unlink($fileColor);
- unlink($fileMask);
- }
+ unlink($fileMenu);
+ unlink($fileColor);
+ unlink($fileMask);
}
}
}
@@ -1411,7 +1393,7 @@ class GraphicalFunctions
$workArea = $this->applyOffset($workArea, GeneralUtility::intExplode(',', $conf['offset']));
$blurRate = MathUtility::forceIntegerInRange((int)$conf['blur'], 0, 99);
// No effects if ImageMagick ver. 5+
- if (!$blurRate || $this->NO_IM_EFFECTS) {
+ if (!$blurRate || !$this->processorEffectsEnabled) {
$txtConf['fontColor'] = $conf['color'];
$this->makeText($im, $txtConf, $workArea);
} else {
@@ -1485,11 +1467,9 @@ class GraphicalFunctions
}
}
// Deleting temporary files;
- if (!$this->dontUnlinkTempFiles) {
- unlink($fileMenu);
- unlink($fileColor);
- unlink($fileMask);
- }
+ unlink($fileMenu);
+ unlink($fileColor);
+ unlink($fileMask);
}
}
@@ -1597,12 +1577,12 @@ class GraphicalFunctions
$commands .= ' -gamma ' . (float)$value;
break;
case 'blur':
- if (!$this->NO_IM_EFFECTS) {
+ if ($this->processorEffectsEnabled) {
$commands .= $this->v5_blur($value);
}
break;
case 'sharpen':
- if (!$this->NO_IM_EFFECTS) {
+ if ($this->processorEffectsEnabled) {
$commands .= $this->v5_sharpen($value);
}
break;
@@ -1701,7 +1681,7 @@ class GraphicalFunctions
$conf['offset'] = $cords[0] . ',' . $cords[1];
$cords = $this->objPosition($conf, $this->workArea, [$cords[2], $cords[3]]);
$newIm = imagecreatetruecolor($cords[2], $cords[3]);
- $cols = $this->convertColor($conf['backColor'] ? $conf['backColor'] : $this->setup['backColor']);
+ $cols = $this->convertColor($conf['backColor'] ?: $this->setup['backColor']);
$Bcolor = imagecolorallocate($newIm, $cols[0], $cols[1], $cols[2]);
imagefilledrectangle($newIm, 0, 0, $cords[2], $cords[3], $Bcolor);
$newConf = [];
@@ -1747,11 +1727,9 @@ class GraphicalFunctions
// Clears workArea to total image
$this->setWorkArea('');
}
- if (!$this->dontUnlinkTempFiles) {
- unlink($theFile);
- if ($theNewFile[3] && $theNewFile[3] != $theFile) {
- unlink($theNewFile[3]);
- }
+ unlink($theFile);
+ if ($theNewFile[3] && $theNewFile[3] != $theFile) {
+ unlink($theNewFile[3]);
}
}
}
@@ -2092,7 +2070,7 @@ class GraphicalFunctions
$newExt = $info[2];
}
if ($newExt === 'web') {
- if (GeneralUtility::inList($this->webImageExt, $info[2])) {
+ if (in_array($info[2], $this->webImageExt, true)) {
$newExt = $info[2];
} else {
$newExt = $this->gif_or_jpg($info[2], $info[0], $info[1]);
@@ -2101,7 +2079,7 @@ class GraphicalFunctions
}
}
}
- if (!GeneralUtility::inList($this->imageFileExt, $newExt)) {
+ if (!in_array($newExt, $this->imageFileExt, true)) {
return null;
}
@@ -2155,7 +2133,7 @@ class GraphicalFunctions
}
// Making the temporary filename:
GeneralUtility::mkdir_deep(PATH_site . 'typo3temp/assets/images/');
- $output = $this->absPrefix . 'typo3temp/assets/images/' . $this->filenamePrefix . $theOutputName . '.' . $newExt;
+ $output = PATH_site . 'typo3temp/assets/images/' . $this->filenamePrefix . $theOutputName . '.' . $newExt;
if ($this->dontCheckForExistingTempFile || !file_exists($output)) {
$this->imageMagickExec($imagefile, $output, $command, $frame);
}
@@ -2185,7 +2163,7 @@ class GraphicalFunctions
public function getImageDimensions($imageFile)
{
preg_match('/([^\\.]*)$/', $imageFile, $reg);
- if (file_exists($imageFile) && GeneralUtility::inList($this->imageFileExt, strtolower($reg[0]))) {
+ if (file_exists($imageFile) && in_array(strtolower($reg[0]), $this->imageFileExt, true)) {
if ($returnArr = $this->getCachedImageDimensions($imageFile)) {
return $returnArr;
}
@@ -2632,9 +2610,7 @@ class GraphicalFunctions
$this->w = imagesx($im);
$this->h = imagesy($im);
}
- if (!$this->dontUnlinkTempFiles) {
- unlink($theFile);
- }
+ unlink($theFile);
}
/**
@@ -2737,10 +2713,9 @@ class GraphicalFunctions
$result = false;
switch ($ext) {
case 'jpg':
-
case 'jpeg':
if (function_exists('imagejpeg')) {
- if ($quality == 0) {
+ if ($quality === 0) {
$quality = $this->jpegQuality;
}
$result = imagejpeg($destImg, $theImage, $quality);
@@ -2791,7 +2766,6 @@ class GraphicalFunctions
}
break;
case 'jpg':
-
case 'jpeg':
if (function_exists('imagecreatefromjpeg')) {
return imagecreatefromjpeg($sourceImg);
@@ -2872,13 +2846,11 @@ class GraphicalFunctions
}
}
// Unlink files from process
- if (!$this->dontUnlinkTempFiles) {
- if ($origName) {
- @unlink($origName);
- }
- if ($postName) {
- @unlink($postName);
- }
+ if ($origName) {
+ @unlink($origName);
+ }
+ if ($postName) {
+ @unlink($postName);
}
}
return $retCol;
diff --git a/typo3/sysext/core/Classes/Resource/Processing/LocalCropScaleMaskHelper.php b/typo3/sysext/core/Classes/Resource/Processing/LocalCropScaleMaskHelper.php
index 101258cf2d58..cdff4495038d 100644
--- a/typo3/sysext/core/Classes/Resource/Processing/LocalCropScaleMaskHelper.php
+++ b/typo3/sysext/core/Classes/Resource/Processing/LocalCropScaleMaskHelper.php
@@ -65,7 +65,6 @@ class LocalCropScaleMaskHelper
/** @var $gifBuilder GifBuilder */
$gifBuilder = GeneralUtility::makeInstance(GifBuilder::class);
$gifBuilder->init();
- $gifBuilder->absPrefix = PATH_site;
$configuration = $targetFile->getProcessingConfiguration();
$configuration['additionalParameters'] = $this->modifyImageMagickStripProfileParameters($configuration['additionalParameters'], $configuration);
diff --git a/typo3/sysext/core/Classes/Resource/Processing/LocalPreviewHelper.php b/typo3/sysext/core/Classes/Resource/Processing/LocalPreviewHelper.php
index 999a92bf32f6..a90d7535f88a 100644
--- a/typo3/sysext/core/Classes/Resource/Processing/LocalPreviewHelper.php
+++ b/typo3/sysext/core/Classes/Resource/Processing/LocalPreviewHelper.php
@@ -119,7 +119,6 @@ class LocalPreviewHelper
/** @var $gifBuilder \TYPO3\CMS\Frontend\Imaging\GifBuilder */
$gifBuilder = GeneralUtility::makeInstance(\TYPO3\CMS\Frontend\Imaging\GifBuilder::class);
$gifBuilder->init();
- $gifBuilder->absPrefix = PATH_site;
$info = $gifBuilder->getImageDimensions($originalFileName);
$newInfo = $gifBuilder->getImageScale($info, $configuration['width'], $configuration['height'], []);
$result = [
diff --git a/typo3/sysext/core/Configuration/DefaultConfiguration.php b/typo3/sysext/core/Configuration/DefaultConfiguration.php
index e889afebc7bc..7bb1f4c0346c 100644
--- a/typo3/sysext/core/Configuration/DefaultConfiguration.php
+++ b/typo3/sysext/core/Configuration/DefaultConfiguration.php
@@ -30,7 +30,7 @@ return [
'processor_path' => '/usr/bin/',
'processor_path_lzw' => '/usr/bin/',
'processor' => 'ImageMagick',
- 'processor_effects' => 0,
+ 'processor_effects' => false,
'processor_allowUpscaling' => true,
'processor_allowFrameSelection' => true,
'processor_allowTemporaryMasksAsPng' => false,
diff --git a/typo3/sysext/core/Configuration/DefaultConfigurationDescription.yaml b/typo3/sysext/core/Configuration/DefaultConfigurationDescription.yaml
index d1b449b09a99..be6523c67fe6 100644
--- a/typo3/sysext/core/Configuration/DefaultConfigurationDescription.yaml
+++ b/typo3/sysext/core/Configuration/DefaultConfigurationDescription.yaml
@@ -35,12 +35,8 @@ GFX:
'GraphicsMagick': 'Choose GraphicsMagick for processing images'
description: 'Select which external software on the server should process images - see also the Preset functionality to see what is available.'
processor_effects:
- type: int
- allowedValues:
- '-1': 'Do not sharpen images by default'
- '0': 'Disable all effects from ImageMagick/GraphicsMagick'
- '1': 'All; blur and sharpening is allowed in ImageMagick'
- description: 'Apply special ImageMagick functionality when processing images'
+ type: bool
+ description: 'If enabled, apply blur and sharpening in ImageMagick/GraphicMagick functions'
processor_allowUpscaling:
type: bool
description: 'If set, images can be scaled up if told so (in <code>\TYPO3\CMS\Core\Imaging\GraphicalFunctions</code>)'
diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-82768-ConfigurationOptionsForImageManipulationPHPAPI.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-82768-ConfigurationOptionsForImageManipulationPHPAPI.rst
new file mode 100644
index 000000000000..5595eac3a375
--- /dev/null
+++ b/typo3/sysext/core/Documentation/Changelog/master/Breaking-82768-ConfigurationOptionsForImageManipulationPHPAPI.rst
@@ -0,0 +1,63 @@
+.. include:: ../../Includes.txt
+
+=======================================================================
+Breaking: #82768 - Configuration Options for Image Manipulation PHP API
+=======================================================================
+
+See :issue:`82768`
+
+Description
+===========
+
+The main PHP class `GraphicalFunctions` for rendering images based on ImageMagick/GraphicsMagick
+and/or GDlib has been cleaned up in order to optimize various places within the code itself,
+making more use of the proper "init()" function setting all relevant options.
+
+The following previously public properties are therefore either set to "protected"
+or removed/renamed as part of the streaming process, removing the possibility to
+override any of the settings other than via the `init()` method within
+GraphicalFunctions:
+
+ * GraphicalFunctions->gdlibExtensions
+ * GraphicalFunctions->imageFileExt
+ * GraphicalFunctions->webImageExt
+ * GraphicalFunctions->NO_IM_EFFECTS
+ * GraphicalFunctions->NO_IMAGE_MAGICK
+ * GraphicalFunctions->mayScaleUp
+ * GraphicalFunctions->dontCompress
+ * GraphicalFunctions->dontUnlinkTempFiles
+ * GraphicalFunctions->absPrefix
+ * GraphicalFunctions->im5fx_blurSteps
+ * GraphicalFunctions->im5fx_sharpenSteps
+ * GraphicalFunctions->pixelLimitGif
+ * GraphicalFunctions->colMap
+ * GraphicalFunctions->csConvObj
+ * GraphicalFunctions->jpegQuality
+ * GraphicalFunctions->OFFSET
+
+Additionally, the option to disable the deletion of tempFiles have been removed.
+
+The global configuration option :php:`$TYPO3_CONF_VARS[GFX][processor_effects]`
+is a boolean option now.
+
+
+Impact
+======
+
+Setting any of the PHP properties above will have no effect anymore.
+
+
+Affected Installations
+======================
+
+Any TYPO3 installation with a extension accessing directly GraphicalFunctions or GifBuilder API
+via PHP and using any of the properties above.
+
+
+Migration
+=========
+
+Ensure all options are properly set when calling :php:`GraphicalFunctions->init()` and remove
+all calls to get or set values from the previously public properties.
+
+.. index:: LocalConfiguration, PHP-API, NotScanned
\ No newline at end of file
diff --git a/typo3/sysext/frontend/Classes/Imaging/GifBuilder.php b/typo3/sysext/frontend/Classes/Imaging/GifBuilder.php
index abd3cb3de21d..9991ab3850c5 100644
--- a/typo3/sysext/frontend/Classes/Imaging/GifBuilder.php
+++ b/typo3/sysext/frontend/Classes/Imaging/GifBuilder.php
@@ -663,7 +663,7 @@ class GifBuilder extends GraphicalFunctions
*/
public function getResource($file, $fileArray)
{
- if (!GeneralUtility::inList($this->imageFileExt, $fileArray['ext'])) {
+ if (!in_array($fileArray['ext'], $this->imageFileExt, true)) {
$fileArray['ext'] = $this->gifExtension;
}
/** @var ContentObjectRenderer $cObj */
diff --git a/typo3/sysext/install/Classes/Controller/EnvironmentController.php b/typo3/sysext/install/Classes/Controller/EnvironmentController.php
index 9582c61cceb6..e34b3c65bcff 100644
--- a/typo3/sysext/install/Classes/Controller/EnvironmentController.php
+++ b/typo3/sysext/install/Classes/Controller/EnvironmentController.php
@@ -717,7 +717,7 @@ class EnvironmentController extends AbstractController
$conf['niceText'] = 1;
$conf['shadow.'] = [
'offset' => '2,2',
- 'blur' => $imageProcessor->NO_IM_EFFECTS ? '90' : '20',
+ 'blur' => '20',
'opacity' => '50',
'color' => 'black'
];
@@ -745,10 +745,9 @@ class EnvironmentController extends AbstractController
{
$imageProcessor = GeneralUtility::makeInstance(GraphicalFunctions::class);
$imageProcessor->init();
- $imageProcessor->absPrefix = PATH_site;
- $imageProcessor->dontCheckForExistingTempFile = 1;
+ $imageProcessor->dontCheckForExistingTempFile = true;
$imageProcessor->filenamePrefix = 'installTool-';
- $imageProcessor->dontCompress = 1;
+ $imageProcessor->dontCompress = true;
$imageProcessor->alternativeOutputKey = 'typo3InstallTest';
return $imageProcessor;
}
@@ -971,7 +970,7 @@ class EnvironmentController extends AbstractController
*/
protected function getImagesPath(GraphicalFunctions $imageProcessor): string
{
- $imagePath = $imageProcessor->absPrefix . 'typo3temp/assets/images/';
+ $imagePath = PATH_site . 'typo3temp/assets/images/';
if (!is_dir($imagePath)) {
GeneralUtility::mkdir_deep($imagePath);
}
--
GitLab