Passed
Push — master ( ff53ce...185c6a )
by Patrick
01:57
created
library/Trapdirector/TrapsProcess/Mib.php 1 patch
Braces   +23 added lines, -9 removed lines patch added patch discarded remove patch
@@ -86,8 +86,7 @@  discard block
 block discarded – undo
86 86
                 }
87 87
                 $this->logging->log('Trap updated : '.$this->oidDesc['name'] . ' / OID : '.$this->oidDesc['oid'],DEBUG );
88 88
                 return 1;
89
-            }
90
-            else
89
+            } else
91 90
             {
92 91
                 $this->logging->log('Trap unchanged : '.$this->oidDesc['name'] . ' / OID : '.$this->oidDesc['oid'],DEBUG );
93 92
                 return 0;
@@ -103,7 +102,9 @@  discard block
 block discarded – undo
103 102
             'values (:oid, :name , :type ,:mib ,:tc , :display_hint'.
104 103
             ', :syntax, :type_enum, :description )';
105 104
         
106
-        if ($this->trapsDB->trapDBType == 'pgsql') $sql .= 'RETURNING id';
105
+        if ($this->trapsDB->trapDBType == 'pgsql') {
106
+        	$sql .= 'RETURNING id';
107
+        }
107 108
         
108 109
         $sqlQuery=$db_conn->prepare($sql);
109 110
         
@@ -143,7 +144,9 @@  discard block
 block discarded – undo
143 144
                 }
144 145
                 
145 146
                 $inserted_id=$ret_code->fetch(PDO::FETCH_ASSOC)['LAST_INSERT_ID()'];
146
-                if ($inserted_id==false) throw new Exception("Weird SQL error : last_insert_id returned false : open issue");
147
+                if ($inserted_id==false) {
148
+                	throw new Exception("Weird SQL error : last_insert_id returned false : open issue");
149
+                }
147 150
                 $this->dbOidIndex[$this->oidDesc['oid']]['id']=$inserted_id;
148 151
                 break;
149 152
             default:
@@ -409,12 +412,14 @@  discard block
 block discarded – undo
409 412
         $this->oidDesc['name']=$match[1]; // Name
410 413
         $this->oidDesc['type']=$match[2]; // type (21=trap, 0: may be trap, else : not trap
411 414
         
412
-        if ($this->oidDesc['type']==0) // object type=0 : check if v1 trap
415
+        if ($this->oidDesc['type']==0) {
416
+        	// object type=0 : check if v1 trap
413 417
         {
414 418
             // Check if next is suboid -> in that case is cannot be a trap
415 419
             if (preg_match("/^".$this->oidDesc['oid']."/",$this->objectsAll[$curElement+1]))
416 420
             {
417 421
                 $this->timing['type0_check_time'] += microtime(true) - $this->timing['base_time'];
422
+        }
418 423
                 $this->timing['type0_check_num']++;
419 424
                 return true;
420 425
             }
@@ -432,9 +437,11 @@  discard block
 block discarded – undo
432 437
             // Force as trap.
433 438
             $this->oidDesc['type']=21;
434 439
         }
435
-        if ($onlyTraps===true && $this->oidDesc['type']!=21) // if only traps and not a trap, continue
440
+        if ($onlyTraps===true && $this->oidDesc['type']!=21) {
441
+        	// if only traps and not a trap, continue
436 442
         {
437 443
             $this->timing['nottrap_time'] += microtime(true) - $this->timing['base_time'];
444
+        }
438 445
             $this->timing['nottrap_num']++;
439 446
             return true;
440 447
         }
@@ -494,7 +501,10 @@  discard block
 block discarded – undo
494 501
             $this->timing['num_traps']++;
495 502
             
496 503
             $this->logging->log('Found trap : '.$this->oidDesc['name'] . ' / OID : '.$this->oidDesc['oid'],INFO );
497
-            if ($display_progress) echo '#'; // echo a # when trap found
504
+            if ($display_progress) {
505
+            	echo '#';
506
+            }
507
+            // echo a # when trap found
498 508
             
499 509
             //################################
500 510
             // get trap objects & source MIB
@@ -514,7 +524,9 @@  discard block
 block discarded – undo
514 524
             $this->oidDesc['mib']=$match[1];
515 525
             
516 526
             $numLine=1;
517
-            while (isset($snmptrans[$numLine]) && !preg_match('/^[\t ]+DESCRIPTION[\t ]+"(.*)/',$snmptrans[$numLine],$match)) $numLine++;
527
+            while (isset($snmptrans[$numLine]) && !preg_match('/^[\t ]+DESCRIPTION[\t ]+"(.*)/',$snmptrans[$numLine],$match)) {
528
+            	$numLine++;
529
+            }
518 530
             if (isset($snmptrans[$numLine]))
519 531
             {
520 532
                 $snmptrans[$numLine] = preg_replace('/^[\t ]+DESCRIPTION[\t ]+"/','',$snmptrans[$numLine]);
@@ -541,7 +553,9 @@  discard block
 block discarded – undo
541 553
             if (($update==0) && ($check_change===false))
542 554
             { // Trapd didn't change & force check disabled
543 555
                 $this->timing['objects_time'] += microtime(true) - $this->timing['base_time'];
544
-                if ($display_progress) echo "C";
556
+                if ($display_progress) {
557
+                	echo "C";
558
+                }
545 559
                 continue;
546 560
             }
547 561
             
Please login to merge, or discard this patch.