patrickpr / trapdirector

library/Trapdirector/TrapsProcess/Mib.php

Braces +23 added lines, -9 removed lines

@@ -97,8 +97,7 @@ 
                 }
                 $this->logging->log('Trap updated : '.$name . ' / OID : '.$oid,DEBUG );
                 return 1;
-            }
-            else
+            } else
             {
                 $this->logging->log('Trap unchanged : '.$name . ' / OID : '.$oid,DEBUG );
                 return 0;
@@ -114,7 +113,9 @@ 
             'values (:oid, :name , :type ,:mib ,:tc , :display_hint'.
             ', :syntax, :type_enum, :description )';
         
-        if ($this->trapsDB->trapDBType == 'pgsql') $sql .= 'RETURNING id';
+        if ($this->trapsDB->trapDBType == 'pgsql') {
+        	$sql .= 'RETURNING id';
+        }
         
         $sqlQuery=$db_conn->prepare($sql);
         
@@ -154,7 +155,9 @@ 
                 }
                 
                 $inserted_id=$ret_code->fetch(PDO::FETCH_ASSOC)['LAST_INSERT_ID()'];
-                if ($inserted_id==false) throw new Exception("Weird SQL error : last_insert_id returned false : open issue");
+                if ($inserted_id==false) {
+                	throw new Exception("Weird SQL error : last_insert_id returned false : open issue");
+                }
                 $this->dbOidIndex[$oid]['id']=$inserted_id;
                 break;
             default:
@@ -396,12 +399,14 @@ 
         $name=$match[1]; // Name
         $type=$match[2]; // type (21=trap, 0: may be trap, else : not trap
         
-        if ($type==0) // object type=0 : check if v1 trap
+        if ($type==0) {
+        	// object type=0 : check if v1 trap
         {
             // Check if next is suboid -> in that case is cannot be a trap
             if (preg_match("/^$oid/",$this->objectsAll[$curElement+1]))
             {
                 $this->timing['type0_check_time'] += microtime(true) - $this->timing['base_time'];
+        }
                 $this->timing['type0_check_num']++;
                 return true;
             }
@@ -419,9 +424,11 @@ 
             // Force as trap.
             $type=21;
         }
-        if ($onlyTraps===true && $type!=21) // if only traps and not a trap, continue
+        if ($onlyTraps===true && $type!=21) {
+        	// if only traps and not a trap, continue
         {
             $this->timing['nottrap_time'] += microtime(true) - $this->timing['base_time'];
+        }
             $this->timing['nottrap_num']++;
             return true;
         }
@@ -480,7 +487,10 @@ 
             $this->timing['num_traps']++;
             
             $this->logging->log('Found trap : '.$name . ' / OID : '.$oid,INFO );
-            if ($display_progress) echo '#'; // echo a # when trap found
+            if ($display_progress) {
+            	echo '#';
+            }
+            // echo a # when trap found
             
             // get trap objects & source MIB
             $retVal=0;
@@ -499,7 +509,9 @@ 
             $trapMib=$match[1];
             
             $numLine=1;$trapDesc='';
-            while (isset($snmptrans[$numLine]) && !preg_match('/^[\t ]+DESCRIPTION[\t ]+"(.*)/',$snmptrans[$numLine],$match)) $numLine++;
+            while (isset($snmptrans[$numLine]) && !preg_match('/^[\t ]+DESCRIPTION[\t ]+"(.*)/',$snmptrans[$numLine],$match)) {
+            	$numLine++;
+            }
             if (isset($snmptrans[$numLine]))
             {
                 $snmptrans[$numLine] = preg_replace('/^[\t ]+DESCRIPTION[\t ]+"/','',$snmptrans[$numLine]);
@@ -524,7 +536,9 @@ 
             if (($update==0) && ($check_change===false))
             { // Trapd didn't change & force check disabled
                 $this->timing['objects_time'] += microtime(true) - $this->timing['base_time'];
-                if ($display_progress) echo "C";
+                if ($display_progress) {
+                	echo "C";
+                }
                 continue;
             }