Merge pull request #134 from RipClaw2971/#1281

fixes #1281
This commit is contained in:
Michael Kaufmann
2013-10-21 09:11:31 -07:00

View File

@@ -269,6 +269,8 @@ class ApsInstaller extends ApsParser
$this->db->query('DELETE FROM `' . TABLE_APS_INSTANCES . '` WHERE `ID` = ' . $this->db->escape($Row['InstanceID']));
$this->db->query('DELETE FROM `' . TABLE_APS_SETTINGS . '` WHERE `InstanceID` = ' . $this->db->escape($Row['InstanceID']));
if($this->RealPath != '' && checkDisallowedPaths($this->RealPath))
{
//remove data, #273
if($this->DomainPath != '' && $this->DomainPath != '/') {
self::UnlinkRecursive($this->RealPath . $this->DomainPath . '/');
@@ -284,6 +286,7 @@ class ApsInstaller extends ApsParser
}
}
}
}
/**
* setup all environment variables from the wizard, they're all needed by the installation script
@@ -336,6 +339,12 @@ class ApsInstaller extends ApsParser
$mapping_url = $Xml->mapping['url'];
}
if ($this->RealPath == '' || !checkDisallowedPaths($this->RealPath))
{
$this->db->query('UPDATE `' . TABLE_APS_INSTANCES . '` SET `Status` = ' . INSTANCE_ERROR . ' WHERE `ID` = ' . $this->db->escape($Row['InstanceID']));
return false;
}
if($Task == TASK_INSTALL)
{
//FIXME truncate customer directory