[BUGFIX] Fix install tool image test path 64/49864/7
authorJohannes Goslar <jogo@kronberger-spiele.de>
Mon, 5 Sep 2016 11:54:25 +0000 (13:54 +0200)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Sat, 3 Dec 2016 10:18:44 +0000 (11:18 +0100)
The temppath for gdlib and combineImages is different
then all other image tests
The assets/images path is set in imageMagickConvert
for other image tests

Releases: master
Resolves: #77851
Change-Id: I4fec0422595045cfb5ca67a48f8e6dae79fc08d7
Reviewed-on: https://review.typo3.org/49864
Reviewed-by: Gleb Levitin <gleb.levitin@dkd.de>
Reviewed-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/install/Classes/Controller/Action/Tool/TestSetup.php

index 2b81ccc..e7d961f 100644 (file)
@@ -400,7 +400,7 @@ class TestSetup extends Action\AbstractAction
         $inputFile = $this->imageBasePath . 'TestInput/BackgroundOrange.gif';
         $overlayFile = $this->imageBasePath . 'TestInput/Test.jpg';
         $maskFile = $this->imageBasePath . 'TestInput/MaskBlackWhite.gif';
-        $resultFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix
+        $resultFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix
             . StringUtility::getUniqueId($imageProcessor->alternativeOutputKey . 'combine1') . '.jpg';
         $imageProcessor->combineExec($inputFile, $overlayFile, $maskFile, $resultFile);
         $result = $imageProcessor->getImageDimensions($resultFile);
@@ -417,7 +417,7 @@ class TestSetup extends Action\AbstractAction
         $inputFile = $this->imageBasePath . 'TestInput/BackgroundCombine.jpg';
         $overlayFile = $this->imageBasePath . 'TestInput/Test.jpg';
         $maskFile = $this->imageBasePath . 'TestInput/MaskCombine.jpg';
-        $resultFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix
+        $resultFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix
             . StringUtility::getUniqueId($imageProcessor->alternativeOutputKey . 'combine2') . '.jpg';
         $imageProcessor->combineExec($inputFile, $overlayFile, $maskFile, $resultFile);
         $result = $imageProcessor->getImageDimensions($resultFile);
@@ -457,7 +457,7 @@ class TestSetup extends Action\AbstractAction
             'color' => 'olive',
         ];
         $imageProcessor->makeBox($image, $conf, $workArea);
-        $outputFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdSimple') . '.' . $gifOrPng;
+        $outputFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdSimple') . '.' . $gifOrPng;
         $imageProcessor->ImageWrite($image, $outputFile);
         $result = $imageProcessor->getImageDimensions($outputFile);
         $testResults['simple'] = [];
@@ -477,7 +477,7 @@ class TestSetup extends Action\AbstractAction
             'color' => 'olive',
         ];
         $imageProcessor->makeBox($image, $conf, $workArea);
-        $outputFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdBox') . '.' . $gifOrPng;
+        $outputFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdBox') . '.' . $gifOrPng;
         $imageProcessor->ImageWrite($image, $outputFile);
         $result = $imageProcessor->getImageDimensions($outputFile);
         $testResults['box'] = [];
@@ -504,7 +504,7 @@ class TestSetup extends Action\AbstractAction
         ];
         $conf['BBOX'] = $imageProcessor->calcBBox($conf);
         $imageProcessor->makeText($image, $conf, $workArea);
-        $outputFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdText') . '.' . $gifOrPng;
+        $outputFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdText') . '.' . $gifOrPng;
         $imageProcessor->ImageWrite($image, $outputFile);
         $result = $imageProcessor->getImageDimensions($outputFile);
         $testResults['text'] = [];
@@ -520,7 +520,7 @@ class TestSetup extends Action\AbstractAction
             $conf['offset'] = '30,120';
             $conf['niceText'] = 1;
             $imageProcessor->makeText($image, $conf, $workArea);
-            $outputFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdNiceText') . '.' . $gifOrPng;
+            $outputFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('gdNiceText') . '.' . $gifOrPng;
             $imageProcessor->ImageWrite($image, $outputFile);
             $result = $imageProcessor->getImageDimensions($outputFile);
             $testResults['niceText']['title'] = 'Render text with TrueType font using \'niceText\' option';
@@ -556,7 +556,7 @@ class TestSetup extends Action\AbstractAction
             // Warning: Re-uses $image from above!
             $imageProcessor->makeShadow($image, $conf['shadow.'], $workArea, $conf);
             $imageProcessor->makeText($image, $conf, $workArea);
-            $outputFile = $imageProcessor->tempPath . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('GDwithText-niceText-shadow') . '.' . $gifOrPng;
+            $outputFile = $this->getImagesPath($imageProcessor) . $imageProcessor->filenamePrefix . StringUtility::getUniqueId('GDwithText-niceText-shadow') . '.' . $gifOrPng;
             $imageProcessor->ImageWrite($image, $outputFile);
             $result = $imageProcessor->getImageDimensions($outputFile);
             $testResults['shadow']['title'] = 'Render \'niceText\' with a shadow under';
@@ -691,4 +691,20 @@ class TestSetup extends Action\AbstractAction
         list($version) = explode(' ', trim($version));
         return trim($version);
     }
+
+    /**
+     * Return the temp image dir.
+     * If not exist it will be created
+     *
+     * @param GraphicalFunctions $imageProcessor
+     * @return string
+     */
+    protected function getImagesPath(GraphicalFunctions $imageProcessor)
+    {
+        $imagePath = $imageProcessor->tempPath . 'assets/images/';
+        if (!is_dir($imagePath)) {
+            GeneralUtility::mkdir_deep($imagePath);
+        }
+        return $imagePath;
+    }
 }