1
0
Fork 0

Refactored xdebughandler

pull/6010/merge
Alex Bowers 2016-12-29 12:08:18 +00:00 committed by Jordi Boggiano
parent 2c39c43a9e
commit e161218819
1 changed files with 5 additions and 5 deletions

View File

@ -63,7 +63,10 @@ class XdebugHandler
$args = explode('|', strval(getenv(self::ENV_ALLOW)), 2); $args = explode('|', strval(getenv(self::ENV_ALLOW)), 2);
if ($this->needsRestart($args[0])) { if ($this->needsRestart($args[0])) {
$this->prepareRestart($command) && $this->restart($command); if ($this->prepareRestart()) {
$command = $this->getCommand();
$this->restart($command);
}
return; return;
} }
@ -127,18 +130,15 @@ class XdebugHandler
* - tmp ini file creation * - tmp ini file creation
* - environment variable creation * - environment variable creation
* *
* @param null|string $command The command to run, set by method
*
* @return bool * @return bool
*/ */
private function prepareRestart(&$command) private function prepareRestart()
{ {
$this->tmpIni = ''; $this->tmpIni = '';
$iniPaths = IniHelper::getAll(); $iniPaths = IniHelper::getAll();
$files = $this->getWorkingSet($iniPaths, $replace); $files = $this->getWorkingSet($iniPaths, $replace);
if ($this->writeTmpIni($files, $replace)) { if ($this->writeTmpIni($files, $replace)) {
$command = $this->getCommand();
return $this->setEnvironment($iniPaths); return $this->setEnvironment($iniPaths);
} }