fix config.form.php

Author: stonebuzz

front/config.form.php

@@ -37,6 +37,7 @@
 Session::checkRight("config", UPDATE);
 
 $PluginSccmConfig = new PluginSccmConfig();
+$sccmDB = new PluginSccmSccmdb();
 global $DB;
 
 if (isset($_POST["update"])) {
@@ -46,39 +47,20 @@
    }
 
    $PluginSccmConfig->update($_POST);
-
-   Toolbox::logInFile('sccm', "Updating configuration ".$_POST['sccm_config_name']." ".$_POST['id']." ...\n", true);
-   
-   $sccmDB = new PluginSccmSccmdb();
    $sccmDB->testConfiguration($_POST['id']);
-
    Html::redirect(PluginSccmConfig::searchUrl());
 } else if (isset($_POST["add"])) {
-   Toolbox::logInFile('sccm', "Inserting configuration ".$_POST['sccm_config_name']." ...\n", true);
-   $insertedId = $PluginSccmConfig->add($_POST);
-   
-   if ($insertedId) {
-      $sccmDB = new PluginSccmSccmdb();
-      $sccmDB->testConfiguration($insertedId);
-   } else {
-      Toolbox::logInFile('sccm', "Error inserting configuration ".$_POST['sccm_config_name']." ".$DB->error()." ...\n", true);
-      Session::addMessageAfterRedirect("Error inserting configuration.", false, ERROR, false);   
+   if ($PluginSccmConfig->add($_POST)) {
+       if ($_SESSION['glpibackcreated']) {
+           Html::redirect($track->getLinkURL());
+       }
    }
-
-   Html::redirect(PluginSccmConfig::searchUrl());
-} else if (isset($_POST["purge"])) {   
+   Html::back();
+} else if (isset($_POST["purge"])) {
    $PluginSccmConfig->delete($_POST, 1);
-   Html::redirect(PluginSccmConfig::searchUrl());
+   $PluginSccmConfig->redirectToList();
 }
 
-Html::header(
-   PluginSccmConfig::getTypeName(),
-   $_SERVER["PHP_SELF"],
-   "config",
-   PluginSccmMenu::class,
-   "configuration"
-);             
-
+Html::header(PluginSccmConfig::getTypeName(), $_SERVER["PHP_SELF"], "config", PluginSccmMenu::class, "configuration");
 $PluginSccmConfig->display($_GET);
-
-Html::footer();
+Html::footer();