Merge pull request #143 from craue/fix-empty-env-argument

fixed that passing an empty array as env argument would clean default environment variables within the forked process
KnpLabs · May 7, 2015 · f29edc4 · f29edc4
2 parents 904df09 + 145c8b5
commit f29edc4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/Knp/Snappy/AbstractGenerator.php b/src/Knp/Snappy/AbstractGenerator.php
@@ -43,7 +43,7 @@ public function __construct($binary, array $options = array(), array $env = null
 
         $this->setBinary($binary);
         $this->setOptions($options);
-        $this->env = $env;
+        $this->env = empty($env) ? null : $env;
     }
 
     public function __destruct()
@@ -361,7 +361,7 @@ protected function createTemporaryFile($content = null, $extension = null)
         if (null !== $content) {
             file_put_contents($filename, $content);
         }
-        
+
         $this->temporaryFiles[] = $filename;
 
         return $filename;