[TASK] Remove GFX/im_combine_filename setting 36/23836/3
authorChristian Kuhn <lolli@schwarzbu.ch>
Sat, 14 Sep 2013 23:31:07 +0000 (01:31 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Mon, 16 Sep 2013 05:06:50 +0000 (07:06 +0200)
Image Magick support for versions 4 and 5 was dropped. The setting
to differentiate combine from composite is obsolete now and composite
is used always.

Change-Id: I0fbc847167d340fd83315285745c63d53225891a
Resolves: #52011
Releases: 6.2
Reviewed-on: https://review.typo3.org/23836
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
typo3/sysext/core/Classes/Utility/CommandUtility.php
typo3/sysext/core/Configuration/DefaultConfiguration.php
typo3/sysext/install/Classes/Configuration/Image/CustomPreset.php
typo3/sysext/install/Classes/Configuration/Image/ImageMagick6Preset.php
typo3/sysext/install/Classes/Controller/StepController.php
typo3/sysext/install/Resources/Public/Images/TestReference/About.txt

index 4bd9913..4ffe2a9 100644 (file)
@@ -41,11 +41,6 @@ use TYPO3\CMS\Core\Utility\MathUtility;
 class GraphicalFunctions {
 
        // Internal configuration, set in init()
-       // The ImageMagick filename used for combining two images. This name changed during the versions.
-       /**
-        * @todo Define visibility
-        */
-       public $combineScript = 'combine';
 
        // If set, there is no frame pointer prepended to the filenames.
        /**
@@ -342,9 +337,6 @@ class GraphicalFunctions {
                // Setting default JPG parameters:
                $this->jpegQuality = MathUtility::forceIntegerInRange($gfxConf['jpg_quality'], 10, 100, 75);
                $this->cmds['jpg'] = ($this->cmds['jpeg'] = '-colorspace ' . $this->colorspace . ' -sharpen 50 -quality ' . $this->jpegQuality);
-               if ($gfxConf['im_combine_filename']) {
-                       $this->combineScript = $gfxConf['im_combine_filename'];
-               }
                if ($gfxConf['im_noFramePrepended']) {
                        $this->noFramePrepended = 1;
                }
index 2ea68a8..ab6480d 100644 (file)
@@ -116,7 +116,6 @@ class CommandUtility {
                }
                $path = \TYPO3\CMS\Core\Utility\GeneralUtility::fixWindowsFilePath($path);
                $im_version = strtolower($gfxConf['im_version_5']);
-               $combineScript = $gfxConf['im_combine_filename'] ? trim($gfxConf['im_combine_filename']) : 'combine';
                // This is only used internally, has no effect outside
                if ($command === 'combine') {
                        $command = 'composite';
@@ -129,7 +128,7 @@ class CommandUtility {
                        if ($im_version === 'im6') {
                                $switchCompositeParameters = TRUE;
                        }
-                       $path = escapeshellarg($path . ($command == 'composite' ? $combineScript : $command) . $isExt);
+                       $path = escapeshellarg($path . ($command == 'composite' ? 'composite' : $command) . $isExt);
                }
                // strip profile information for thumbnails and reduce their size
                if ($parameters && $command != 'identify' && $gfxConf['im_useStripProfileByDefault'] && $gfxConf['im_stripProfileCommand'] != '') {
index 60248ce..3838385 100644 (file)
@@ -48,7 +48,6 @@ return array(
                'im_mask_temp_ext_gif' => FALSE,                                // Boolean: This should be set if ImageMagick is version 5+. This is used in class.tslib_content.php for masking images and the extension png is normally used because it's faster than gif for temporary files. But png seems not to work with some IM 5+ versions, so...
                'im_mask_temp_ext_noloss' => 'miff',                    // String: Loss-less ImageMagick extension to use for mask processing in temporary filenames. Normally 'miff' (ImageMagick's OWN format) will do fine. However at least IM 5.4.9 has proved not to be able to write to its own file format which forced me to add this option. You can try and set this to tif/png/jpg if some masking doesn't work the way it should.
                'im_noScaleUp' => FALSE,                                                // Boolean: If set, images are not being scaled up if told so (in t3lib/stdgraphics.php)
-               'im_combine_filename' => 'combine',                             // String: Latest ImageMagick versions has changed the name of combine to composite. Configure here if needed.
                'im_noFramePrepended' => FALSE,                                 // Boolean: If set, the [x] frame indicator is NOT prepended to filenames in stdgraphic. Some IM5+ version didn't work at all with the typical [0]-prefix, which allow multipage pdf's and animated gif's to be scaled only for the first frame/page and that seriously cuts down rendering time. Set this flag only if your ImageMagick version cannot find the files. Notice that changing this flag causes temporary filenames to change, thus the server will begin scaling images again which were previously cached.
                'im_stripProfileCommand' => '+profile \'*\'',   // String: Specify the command to strip the profile information, which can reduce thumbnail size up to 60KB. Command can differ in IM/GM, IM also know the -strip command. See <a href="http://www.imagemagick.org/Usage/thumbnails/#profiles" target="_blank">imagemagick.org</a> for details
                'im_useStripProfileByDefault' => TRUE,                  // Boolean: If set, the im_stripProfileCommand is used with all IM Image operations by default. See tsRef for setting this parameter explocit for IMAGE generation.
index a47cbc3..d4e64bf 100644 (file)
@@ -44,7 +44,6 @@ class CustomPreset extends Configuration\AbstractCustomPreset implements Configu
                'GFX/im_imvMaskState' => 0,
                'GFX/im_v5effects' => 0,
                'GFX/im_mask_temp_ext_gif' => 0,
-               'GFX/im_combine_filename' => 'combine',
                'GFX/colorspace' => '',
        );
 }
index 6d2f187..09eeffe 100644 (file)
@@ -55,7 +55,6 @@ class ImageMagick6Preset extends AbstractImagePreset implements Configuration\Pr
                'GFX/im_imvMaskState' => 1,
                'GFX/im_v5effects' => 1,
                'GFX/im_mask_temp_ext_gif' => 1,
-               'GFX/im_combine_filename' => 'composite',
                'GFX/colorspace' => 'sRGB',
        );
 
index ba15de5..accea3b 100644 (file)
@@ -70,6 +70,8 @@ class StepController extends AbstractController {
                'FE/defaultTypoScript_editorcfg.',
                // #25099
                'FE/simulateStaticDocuments',
+               // 52011
+               'GFX/im_combine_filename',
                // #22687
                'GFX/gdlib_2',
                // #52010
index 347bec3..ac9d073 100644 (file)
@@ -8,7 +8,6 @@ These comparison images was generated by Kasper Skårhøj, using:
 $TYPO3_CONF_VARS['GFX']['im_path_lzw'] = '';
 $TYPO3_CONF_VARS['GFX']['TTFdpi'] = '96';
 $TYPO3_CONF_VARS['GFX']['gdlib_png'] = '1';
-$TYPO3_CONF_VARS['GFX']['im_combine_filename'] = 'composite';
 $TYPO3_CONF_VARS['GFX']['im_path'] = '/usr/bin/';
 $TYPO3_CONF_VARS['GFX']['im_version_5'] = 'im5';
 $TYPO3_CONF_VARS['GFX']['im_imvMaskState'] = '0';
@@ -30,4 +29,4 @@ WBMP Support          enabled
 
 - I was not able to create GIF files for reference so I just converted them directly with ImageMagick
 
-- Also LZW compression has NOT been used anywhere except the GD test images since ImageMagick doesn't support it.
+- Also LZW compression has NOT been used anywhere except the GD test images since ImageMagick doesn't support it.
\ No newline at end of file