Merge branch 'master' of github.com:heiglandreas/Org_Heigl_Ghostscript

Author: heiglandreas

.travis.yml

@@ -1,4 +1,5 @@
 sudo: false
+dist: trusty
 
 language: php
 
@@ -21,6 +22,11 @@ env:
     - GH_REF: github.com:heiglandreas/Org_Heigl_Ghostscript.git
     - ENCRYPTION_LABEL: "f38516f65eab"
 
+addons:
+  apt:
+    packages:
+      - ghostscript
+
 matrix:
   fast_finish: true
   include:
@@ -55,6 +61,9 @@ matrix:
     - php: 7.1
       env:
        - DEPS=latest
+    - php: 7.2
+      env:
+      - DEPS=latest
     - php: nightly
       env:
         - DEPS=lowest

src/Ghostscript.php

@@ -282,7 +282,7 @@ public static function setGsPath($path = null)
             throw new \InvalidArgumentException('The given file is not executable');
         }
 
-        @exec($path . ' -v', $result);
+        @exec('"' . $path . '" -v', $result);
         $content = implode("\n", $result);
         if (false === stripos($content, 'ghostscript')) {
             throw new \InvalidArgumentException('No valid Ghostscript found');
@@ -363,9 +363,10 @@ public function getInputFile()
      */
     public function setOutputFile($name = 'output')
     {
-        if (0 !== strpos($name, DIRECTORY_SEPARATOR)) {
+        if ($this->isRelative($name)) {
             $name = $this->getBasePath() . DIRECTORY_SEPARATOR . $name;
         }
+
         $this->outfile = $name;
 
         return $this;
@@ -382,7 +383,7 @@ public function setOutputFile($name = 'output')
      */
     public function getOutputFile()
     {
-        if (0 !== strpos($this->outfile, DIRECTORY_SEPARATOR)) {
+        if ($this->isRelative($this->outfile)) {
             return $this->getBasePath() . DIRECTORY_SEPARATOR . $this->outfile;
         }
 
@@ -440,7 +441,7 @@ public function getRenderString()
         if (null === $this->getInputFile()) {
             return '';
         }
-        $string  = self::getGsPath();
+        $string  = '"' . self::getGsPath() . '"';
         $string .= ' -dSAFER -dQUIET -dNOPLATFONTS -dNOPAUSE -dBATCH';
         $string .= ' -sOutputFile="' . $this->getOutputFileName() . '"';
         $string .= $this->getDevice()->getParameterString();
@@ -652,11 +653,11 @@ public function setUseCie($useCie = true)
         return $this;
     }
 
-     /**
-      * Shall we use the CIE map for color-conversions?
-      *
-      * @return boolean
-      */
+    /**
+     * Shall we use the CIE map for color-conversions?
+     *
+     * @return boolean
+     */
     public function useCie()
     {
         return (bool) $this->useCie;
@@ -889,6 +890,19 @@ public function getOutputFileName()
 
         return $basename;
     }
+
+    private function isRelative($path)
+    {
+        if (0 === strpos($path, DIRECTORY_SEPARATOR)) {
+            return false;
+        }
+
+        if (1 === strpos($path, ':\\') && preg_match('/^[A-Za-z]/', $path)) {
+            return false;
+        }
+
+        return true;
+    }
 }
 
 try {

tests/GhostscriptTest.php

@@ -185,7 +185,7 @@ public function testRenderingStrings()
         $f = new Ghostscript();
         $dir = __DIR__ . DIRECTORY_SEPARATOR . 'support' . DIRECTORY_SEPARATOR;
         $filename = $dir . 'test.pdf';
-        $path = Ghostscript::getGsPath();
+        $path = '"' . Ghostscript::getGsPath() . '"';
         $this->assertEquals('', $f->getRenderString());
         $f->setInputFile($filename);
         $expect = $path . ' -dSAFER -dQUIET -dNOPLATFONTS -dNOPAUSE -dBATCH -sOutputFile="' . $dir  . 'output.png" -sDEVICE=pngalpha -r72 "' .$filename . '"';
@@ -288,4 +288,25 @@ public function testThatSettingPathToSomethingGsLikeWorks()
             Ghostscript::class
         );
     }
+
+    /** @dataProvider provideRelativePaths */
+    public function testThatRelativePathsAreCorrectlyDetected($path, $result)
+    {
+        $reflection = new \ReflectionClass(Ghostscript::class);
+        $method = $reflection->getMethod('isRelative');
+        $method->setAccessible(true);
+
+        $this->assertSame($result, $method->invokeArgs(new Ghostscript(), [$path]));
+    }
+
+    public function provideRelativePaths()
+    {
+        return [
+            ['/test/foo', false],
+            ['a:\foo', false],
+            ['bar', true],
+            ['\foob', true],
+            ['a:/test', true],
+        ];
+    }
 }