Browse Source

Use process component instead of passthru, fixes #5501

Jordi Boggiano 8 years ago
parent
commit
a242f7e9ba
1 changed files with 7 additions and 6 deletions
  1. 7 6
      src/Composer/Command/HomeCommand.php

+ 7 - 6
src/Composer/Command/HomeCommand.php

@@ -118,19 +118,20 @@ EOT
     {
         $url = ProcessExecutor::escape($url);
 
+        $process = new ProcessExecutor($this->getIO());
         if (Platform::isWindows()) {
-            return passthru('start "web" explorer "' . $url . '"');
+            return $process->execute('start "web" explorer "' . $url . '"');
         }
 
-        passthru('which xdg-open', $linux);
-        passthru('which open', $osx);
+        $linux = $process->execute('which xdg-open');
+        $osx = $process->execute('which open');
 
         if (0 === $linux) {
-            passthru('xdg-open ' . $url);
+            $process->execute('xdg-open ' . $url);
         } elseif (0 === $osx) {
-            passthru('open ' . $url);
+            $process->execute('open ' . $url);
         } else {
-            $this->getIO()->writeError('no suitable browser opening command found, open yourself: ' . $url);
+            $this->getIO()->writeError('No suitable browser opening command found, open yourself: ' . $url);
         }
     }