diff options
Diffstat (limited to 'modules-available')
| -rw-r--r-- | modules-available/sysconfig/addconfig.inc.php | 4 | ||||
| -rw-r--r-- | modules-available/sysconfig/inc/configtgz.inc.php | 10 |
2 files changed, 7 insertions, 7 deletions
diff --git a/modules-available/sysconfig/addconfig.inc.php b/modules-available/sysconfig/addconfig.inc.php index 27af31e8..456ee989 100644 --- a/modules-available/sysconfig/addconfig.inc.php +++ b/modules-available/sysconfig/addconfig.inc.php @@ -29,7 +29,7 @@ abstract class AddConfig_Base if (($editId = Request::any('edit', 0, 'int')) !== 0) { self::$instance->edit = ConfigTgz::get($editId); if (self::$instance->edit === null) - ErrorHandler::traceError('Invalid config id for editing'); + ErrorHandler::traceError('Invalid config id for editing: ' . $editId); Util::addRedirectParam('edit', self::$instance->edit->id()); } } @@ -182,7 +182,7 @@ class AddConfig_Finish extends AddConfig_Base $this->edit->update($title, $modules); $this->config = $this->edit; } - if ($this->config->generate(true, 150) === false) { + if ($this->config->generate($this->edit === null, 150) === false) { Message::addError('unsuccessful-action'); Util::redirect('?do=SysConfig&action=addconfig'); } diff --git a/modules-available/sysconfig/inc/configtgz.inc.php b/modules-available/sysconfig/inc/configtgz.inc.php index 8ac87908..a54aaef9 100644 --- a/modules-available/sysconfig/inc/configtgz.inc.php +++ b/modules-available/sysconfig/inc/configtgz.inc.php @@ -123,10 +123,11 @@ class ConfigTgz if (!is_array($task) || !isset($task['id']) || Taskmanager::isFailed($task)) { // Failed... Taskmanager::addErrorMessage($task); - if (!$deleteOnError) + if (!$deleteOnError) { $this->markFailed(); - else + } else { $this->delete(); + } return false; } // Still running, add callback @@ -324,9 +325,8 @@ class ConfigTgz public static function get(int $configId): ?ConfigTgz { - $ret = Database::queryFirst("SELECT configid, title, filepath FROM configtgz WHERE configid = :configid", array( - 'configid' => $configId - )); + $ret = Database::queryFirst("SELECT configid, title, filepath FROM configtgz WHERE configid = :configid", + ['configid' => $configId]); if ($ret === false) return null; return self::instanceFromRow($ret); |