Changeset 5890

Timestamp:

07/24/08 13:46:44 (2 months ago)

Author:

jmathis

Message:

Remove PHP Warning and notive in host configuration when template field is empty

Files:

• trunk/centreon/www/include/configuration/configObject/host/DB-Func.php (modified) (3 diffs)

trunk/centreon/www/include/configuration/configObject/host/DB-Func.php

@@ -842 +842 @@
                         while ($hst =& $DBRESULT->fetchRow())
                                 $oldTp[$hst["host_tpl_id"]] = $hst["host_tpl_id"];                      
-                        for ($i=0;$i <= $_POST['nbOfSelect']; $i++)
-                        {
+                        for ($i=0;$i <= $_POST['nbOfSelect']; $i++){
                                 $tpSelect = "tpSelect_" . $i;
-                                $newTp[$_POST[$tpSelect]] = $_POST[$tpSelect];
+                                if (isset($_POST[$tpSelect]))
+                                        $newTp[$_POST[$tpSelect]] = $_POST[$tpSelect];
                         }
-                        foreach ($oldTp as $val)
-                        {
+                        
+                        foreach ($oldTp as $val){
                                 /*
                                  * if not set, then that means a template was removed
@@ -861 +861 @@
                         if (PEAR::isError($DBRESULT))
                                 print "DB Error : ".$DBRESULT->getDebugInfo()."<br />";
-                        for ($i=0, $j = 1;$i <= $_POST['nbOfSelect']; $i++)
-                        {                       
+                        for ($i=0, $j = 1;$i <= $_POST['nbOfSelect']; $i++){
                                 $tpSelect = "tpSelect_" . $i;
-                                if (!isset($already_stored[$_POST[$tpSelect]]) && $_POST[$tpSelect]) {
+                                if (isset($_POST[$tpSelect]) && !isset($already_stored[$_POST[$tpSelect]]) && $_POST[$tpSelect]) {
                                         $rq = "INSERT INTO host_template_relation (`host_host_id`, `host_tpl_id`, `order`) VALUES (". $host_id .", ". $_POST[$tpSelect] .", ". $j .")";
                                         $DBRESULT =& $pearDB->query($rq);
@@ -881 +880 @@
                         $DBRESULT =& $pearDB->query("DELETE FROM `on_demand_macro_host` WHERE `host_host_id`='".$host_id."'");
                         
-                        for ($i=0; $i <= $_POST['nbOfMacro']; $i++)
-                        {                       
+                        for ($i=0; $i <= $_POST['nbOfMacro']; $i++){                    
                                 $macInput = "macroInput_" . $i;
                                 $macValue = "macroValue_" . $i;