patrickpr / trapdirector

library/Trapdirector/Tables/TrapDirectorTablePaging.php

Indentation +84 added lines, -84 removed lines

@@ -6,72 +6,72 @@ 
 trait TrapDirectorTablePaging
 {
    
-    /*************** Paging *************/
-    protected $maxPerPage = 25;
+	/*************** Paging *************/
+	protected $maxPerPage = 25;
     
-    protected $currentPage = 0;
+	protected $currentPage = 0;
         
-    /**** var & func of TrapDirectorTable used ***/
-    protected $query;
-    abstract protected function getCurrentURLAndQS(string $caller);
-    abstract public function applyFilter();
+	/**** var & func of TrapDirectorTable used ***/
+	protected $query;
+	abstract protected function getCurrentURLAndQS(string $caller);
+	abstract public function applyFilter();
     
-    /*****************  Paging and counting *********/
+	/*****************  Paging and counting *********/
     
-    public function countQuery()
-    {
-        $this->query = $this->dbConn->select();
-        $this->query = $this->query
-            ->from(
-                $this->table,
-                array('COUNT(*)')
-                );
-        $this->applyFilter();                   
-    }
+	public function countQuery()
+	{
+		$this->query = $this->dbConn->select();
+		$this->query = $this->query
+			->from(
+				$this->table,
+				array('COUNT(*)')
+				);
+		$this->applyFilter();                   
+	}
     
-    public function count()
-    {
-        $this->countQuery();
-        return $this->dbConn->fetchOne($this->query);
-    }
+	public function count()
+	{
+		$this->countQuery();
+		return $this->dbConn->fetchOne($this->query);
+	}
     
-    public function setMaxPerPage(int $max)
-    {
-        $this->maxPerPage = $max;
-    }
+	public function setMaxPerPage(int $max)
+	{
+		$this->maxPerPage = $max;
+	}
     
-    protected function getPagingQuery(array $getVars)
-    {
-        if (isset($getVars['page']))
-        {
-            $this->currentPage = $getVars['page'];
-        }
-    }
+	protected function getPagingQuery(array $getVars)
+	{
+		if (isset($getVars['page']))
+		{
+			$this->currentPage = $getVars['page'];
+		}
+	}
 
-    protected function curPagingQuery()
-    {
-        if ($this->currentPage == '') return '';
-        return 'page='.$this->currentPage;
-    }
+	protected function curPagingQuery()
+	{
+		if ($this->currentPage == '') return '';
+		return 'page='.$this->currentPage;
+	}
     
-    public function renderPagingHeader()
-    {
-        $count = $this->count();
-        if ($count <= $this->maxPerPage )
-        {
-            return  'count : ' . $this->count() . '<br>';
-        }
+	public function renderPagingHeader()
+	{
+		$count = $this->count();
+		if ($count <= $this->maxPerPage )
+		{
+			return  'count : ' . $this->count() . '<br>';
+		}
         
-        if ($this->currentPage == 0) $this->currentPage = 1;
+		if ($this->currentPage == 0) $this->currentPage = 1;
         
-        $numPages = intdiv($count , $this->maxPerPage);
-        if ($count % $this->maxPerPage != 0 ) $numPages++;
+		$numPages = intdiv($count , $this->maxPerPage);
+		if ($count % $this->maxPerPage != 0 ) $numPages++;
         
-        $html = '<div class="pagination-control" role="navigation">';
-        $html .= '<ul class="nav tab-nav">';
-        if ($this->currentPage <=1)
-        {
-            $html .= '
+		$html = '<div class="pagination-control" role="navigation">';
+		$html .= '<ul class="nav tab-nav">';
+		if ($this->currentPage <=1)
+		{
+			$html .= '
                 <li class="nav-item disabled" aria-hidden="true">
                     <span class="previous-page">
                             <span class="sr-only">Previous page</span>
@@ -79,35 +79,35 @@ 
                      </span>
                 </li>
                ';
-        }
-        else 
-        {
-            $html .= '
+		}
+		else 
+		{
+			$html .= '
                 <li class="nav-item">
                     <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. ($this->currentPage - 1 ).'" class="previous-page" >
                             <i aria-hidden="true" class="icon-angle-double-left"></i>            
                      </a>
                 </li>
             ';
-        }
+		}
         
-        for ($i=1; $i <= $numPages ; $i++)
-        {
-            $active = ($this->currentPage == $i) ? 'active' : '';
-            $first = ($i-1)*$this->maxPerPage+1;
-            $last = $i * $this->maxPerPage;
-            if ($last > $count) $last = $count;
-            $display = 'Show rows '. $first . ' to '. $last .' of '. $count;
-            $html .= '<li class="' . $active . ' nav-item">
+		for ($i=1; $i <= $numPages ; $i++)
+		{
+			$active = ($this->currentPage == $i) ? 'active' : '';
+			$first = ($i-1)*$this->maxPerPage+1;
+			$last = $i * $this->maxPerPage;
+			if ($last > $count) $last = $count;
+			$display = 'Show rows '. $first . ' to '. $last .' of '. $count;
+			$html .= '<li class="' . $active . ' nav-item">
                     <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. $i .'" title="' . $display . '" aria-label="' . $display . '">
                     '.$i.'                
                     </a>
                 </li>';
-        }
+		}
         
-        if ($this->currentPage == $numPages)
-        {
-            $html .= '
+		if ($this->currentPage == $numPages)
+		{
+			$html .= '
                 <li class="nav-item disabled" aria-hidden="true">
                     <span class="previous-page">
                             <span class="sr-only">Previous page</span>
@@ -115,28 +115,28 @@ 
                      </span>
                 </li>
                ';
-        }
-        else
-        {
-            $html .= '
+		}
+		else
+		{
+			$html .= '
                 <li class="nav-item">
                     <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. ($this->currentPage + 1 ).'" class="next-page">
                             <i aria-hidden="true" class="icon-angle-double-right"></i>
                      </a>
                 </li>
             ';
-        }
+		}
         
-        $html .= '</ul> </div>';
+		$html .= '</ul> </div>';
         
-        return $html;
-    }
+		return $html;
+	}
     
-    public function applyPaging()
-    {
-        $this->query->limitPage($this->currentPage,$this->maxPerPage);
-        return $this;
-    }
+	public function applyPaging()
+	{
+		$this->query->limitPage($this->currentPage,$this->maxPerPage);
+		return $this;
+	}
     
     
 }
\ No newline at end of file

Spacing +31 added lines, -31 removed lines

@@ -7,9 +7,9 @@ 
 {
    
     /*************** Paging *************/
-    protected $maxPerPage = 25;
+    protected $maxPerPage=25;
     
-    protected $currentPage = 0;
+    protected $currentPage=0;
         
     /**** var & func of TrapDirectorTable used ***/
     protected $query;
@@ -20,8 +20,8 @@ 
     
     public function countQuery()
     {
-        $this->query = $this->dbConn->select();
-        $this->query = $this->query
+        $this->query=$this->dbConn->select();
+        $this->query=$this->query
             ->from(
                 $this->table,
                 array('COUNT(*)')
@@ -37,14 +37,14 @@ 
     
     public function setMaxPerPage(int $max)
     {
-        $this->maxPerPage = $max;
+        $this->maxPerPage=$max;
     }
     
     protected function getPagingQuery(array $getVars)
     {
         if (isset($getVars['page']))
         {
-            $this->currentPage = $getVars['page'];
+            $this->currentPage=$getVars['page'];
         }
     }
 
@@ -56,22 +56,22 @@ 
     
     public function renderPagingHeader()
     {
-        $count = $this->count();
-        if ($count <= $this->maxPerPage )
+        $count=$this->count();
+        if ($count <= $this->maxPerPage)
         {
-            return  'count : ' . $this->count() . '<br>';
+            return  'count : '.$this->count().'<br>';
         }
         
-        if ($this->currentPage == 0) $this->currentPage = 1;
+        if ($this->currentPage == 0) $this->currentPage=1;
         
-        $numPages = intdiv($count , $this->maxPerPage);
-        if ($count % $this->maxPerPage != 0 ) $numPages++;
+        $numPages=intdiv($count, $this->maxPerPage);
+        if ($count % $this->maxPerPage != 0) $numPages++;
         
-        $html = '<div class="pagination-control" role="navigation">';
-        $html .= '<ul class="nav tab-nav">';
-        if ($this->currentPage <=1)
+        $html='<div class="pagination-control" role="navigation">';
+        $html.='<ul class="nav tab-nav">';
+        if ($this->currentPage <= 1)
         {
-            $html .= '
+            $html.='
                 <li class="nav-item disabled" aria-hidden="true">
                     <span class="previous-page">
                             <span class="sr-only">Previous page</span>
@@ -82,24 +82,24 @@ 
         }
         else 
         {
-            $html .= '
+            $html.='
                 <li class="nav-item">
-                    <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. ($this->currentPage - 1 ).'" class="previous-page" >
+                    <a href="'. $this->getCurrentURLAndQS('paging').'&page='.($this->currentPage - 1).'" class="previous-page" >
                             <i aria-hidden="true" class="icon-angle-double-left"></i>            
                      </a>
                 </li>
             ';
         }
         
-        for ($i=1; $i <= $numPages ; $i++)
+        for ($i=1; $i <= $numPages; $i++)
         {
-            $active = ($this->currentPage == $i) ? 'active' : '';
-            $first = ($i-1)*$this->maxPerPage+1;
-            $last = $i * $this->maxPerPage;
-            if ($last > $count) $last = $count;
-            $display = 'Show rows '. $first . ' to '. $last .' of '. $count;
-            $html .= '<li class="' . $active . ' nav-item">
-                    <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. $i .'" title="' . $display . '" aria-label="' . $display . '">
+            $active=($this->currentPage == $i) ? 'active' : '';
+            $first=($i - 1) * $this->maxPerPage + 1;
+            $last=$i * $this->maxPerPage;
+            if ($last > $count) $last=$count;
+            $display='Show rows '.$first.' to '.$last.' of '.$count;
+            $html.='<li class="'.$active.' nav-item">
+                    <a href="'. $this->getCurrentURLAndQS('paging').'&page='.$i.'" title="'.$display.'" aria-label="'.$display.'">
                     '.$i.'                
                     </a>
                 </li>';
@@ -107,7 +107,7 @@ 
         
         if ($this->currentPage == $numPages)
         {
-            $html .= '
+            $html.='
                 <li class="nav-item disabled" aria-hidden="true">
                     <span class="previous-page">
                             <span class="sr-only">Previous page</span>
@@ -118,23 +118,23 @@ 
         }
         else
         {
-            $html .= '
+            $html.='
                 <li class="nav-item">
-                    <a href="'. $this->getCurrentURLAndQS('paging') .'&page='. ($this->currentPage + 1 ).'" class="next-page">
+                    <a href="'. $this->getCurrentURLAndQS('paging').'&page='.($this->currentPage + 1).'" class="next-page">
                             <i aria-hidden="true" class="icon-angle-double-right"></i>
                      </a>
                 </li>
             ';
         }
         
-        $html .= '</ul> </div>';
+        $html.='</ul> </div>';
         
         return $html;
     }
     
     public function applyPaging()
     {
-        $this->query->limitPage($this->currentPage,$this->maxPerPage);
+        $this->query->limitPage($this->currentPage, $this->maxPerPage);
         return $this;
     }
     

library/Trapdirector/Tables/TrapDirectorTableFilter.php

Indentation +53 added lines, -53 removed lines

@@ -6,75 +6,75 @@
 trait TrapDirectorTableFilter
 {
     
-    /********* Filter ********/
-    /** @var string $filterString : string filter for db columns */
-    protected $filterString = '';
+	/********* Filter ********/
+	/** @var string $filterString : string filter for db columns */
+	protected $filterString = '';
     
-    /** @var array $filterColumn : columns to apply filter to */
-    protected $filterColumn = array();
+	/** @var array $filterColumn : columns to apply filter to */
+	protected $filterColumn = array();
     
-    protected $filterQuery='';
+	protected $filterQuery='';
     
-    /**** var & func of TrapDirectorTable used ***/
-    protected $query;
-    abstract protected function getCurrentURLAndQS(string $caller);
+	/**** var & func of TrapDirectorTable used ***/
+	protected $query;
+	abstract protected function getCurrentURLAndQS(string $caller);
     
-    /**************** Filtering ******************/
+	/**************** Filtering ******************/
     
-    public function applyFilter()
-    {
-        if ($this->filterString == '' || count($this->filterColumn) == 0)
-        {
-            return $this;
-        }
-        $filter='';
-        foreach ($this->filterColumn as $column)
-        {
-            if ($filter != "") $filter.=' OR ';
-            //$filter .= "'" . $column . "' LIKE '%" . $this->filterString. "%'";
-            $filter .= $column  . " LIKE '%" . $this->filterString. "%'";
-        }
-        //echo $filter;
+	public function applyFilter()
+	{
+		if ($this->filterString == '' || count($this->filterColumn) == 0)
+		{
+			return $this;
+		}
+		$filter='';
+		foreach ($this->filterColumn as $column)
+		{
+			if ($filter != "") $filter.=' OR ';
+			//$filter .= "'" . $column . "' LIKE '%" . $this->filterString. "%'";
+			$filter .= $column  . " LIKE '%" . $this->filterString. "%'";
+		}
+		//echo $filter;
         
-        $this->query=$this->query->where($filter);
+		$this->query=$this->query->where($filter);
 
-        return $this;
-    }
+		return $this;
+	}
 
-    public function setFilter(string $filter, array $filterCol)
-    {
-        $this->filterString = $filter;
-        $this->filterColumn = $filterCol;
-        return $this;
-    }
+	public function setFilter(string $filter, array $filterCol)
+	{
+		$this->filterString = $filter;
+		$this->filterColumn = $filterCol;
+		return $this;
+	}
    
-    public function renderFilter()
-    {
+	public function renderFilter()
+	{
         
-        $html=' <form id="genfilter" name="mainFilterGen"
+		$html=' <form id="genfilter" name="mainFilterGen"
 			enctype="application/x-www-form-urlencoded"
 			action="'.$this->getCurrentURLAndQS('filter').'"
 			method="get">';
-        $html.='<input type="text" name="f" title="Search is simple! Try to combine multiple words"
+		$html.='<input type="text" name="f" title="Search is simple! Try to combine multiple words"
 	placeholder="Search..."  value="'.$this->filterQuery.'">';
 
-        $html.='</form>';
-        return $html;
-    }
+		$html.='</form>';
+		return $html;
+	}
  
-    public function getFilterQuery(array $getVars)
-    {
-        if (isset($getVars['f']))
-        {
-            $this->filterQuery = $getVars['f'];
-            $this->setFilter(html_entity_decode($getVars['f']), $this->columnNames);
-        }
-    }
+	public function getFilterQuery(array $getVars)
+	{
+		if (isset($getVars['f']))
+		{
+			$this->filterQuery = $getVars['f'];
+			$this->setFilter(html_entity_decode($getVars['f']), $this->columnNames);
+		}
+	}
     
-    protected function curFilterQuery()
-    {
-        if ($this->filterQuery == '') return '';
-        return 'f='.$this->filterQuery;
-    }
+	protected function curFilterQuery()
+	{
+		if ($this->filterQuery == '') return '';
+		return 'f='.$this->filterQuery;
+	}
     
 }
\ No newline at end of file

library/Trapdirector/Tables/HandlerTable.php

Indentation +120 added lines, -120 removed lines

@@ -7,137 +7,137 @@
 class HandlerTable extends TrapDirectorTable
 {
 
-    protected $status_display=array(
-        -2	=>'ignore',
-        -1 => '-',
-        0	=> 'OK',
-        1	=> 'warning',
-        2	=> 'critical',
-        3	=> 'unknown',);
+	protected $status_display=array(
+		-2	=>'ignore',
+		-1 => '-',
+		0	=> 'OK',
+		1	=> 'warning',
+		2	=> 'critical',
+		3	=> 'unknown',);
     
-    // translate
-    protected $doTranslate=false;
-    protected $MIB;
+	// translate
+	protected $doTranslate=false;
+	protected $MIB;
     
-    // categories
-    protected $categories = NULL;
+	// categories
+	protected $categories = NULL;
     
-    public function setCategoriesArray(array $categories)
-    {
-        $this->categories = $categories;
-    }
+	public function setCategoriesArray(array $categories)
+	{
+		$this->categories = $categories;
+	}
     
-    public function groupingPrintData( string $value)
-    {
-        if ($this->groupingColumn == 'rule_type')
-        {
-            if ($this->categories == NULL || (! isset($this->categories[$value])))
-                return 'Unknown category ('.$value.')';
-            return 'Category : '. $this->categories[$value];
-        }
-        $html = "$value";
-        return $html;
-    }
+	public function groupingPrintData( string $value)
+	{
+		if ($this->groupingColumn == 'rule_type')
+		{
+			if ($this->categories == NULL || (! isset($this->categories[$value])))
+				return 'Unknown category ('.$value.')';
+			return 'Category : '. $this->categories[$value];
+		}
+		$html = "$value";
+		return $html;
+	}
     
-    public function setMibloader($mibloader)
-    {
-        $this->MIB=$mibloader;
-        $this->doTranslate=true;
-    }
+	public function setMibloader($mibloader)
+	{
+		$this->MIB=$mibloader;
+		$this->doTranslate=true;
+	}
 
-    public function titleOrder($name)
-    {
-        switch ($name)
-        {
-            case 'host_name' : return $this->content[$name]; break;
-            case 'source_ip' : return 'ip4'; break;
-            default: return $this->content[$name];	
-        }
-        return NULL;
-    }
+	public function titleOrder($name)
+	{
+		switch ($name)
+		{
+			case 'host_name' : return $this->content[$name]; break;
+			case 'source_ip' : return 'ip4'; break;
+			default: return $this->content[$name];	
+		}
+		return NULL;
+	}
     
-    public function getCurrentURL()
-    {
-        return Url::fromPath($this->urlPath . '/handler');
-    }
+	public function getCurrentURL()
+	{
+		return Url::fromPath($this->urlPath . '/handler');
+	}
     
-    public function renderLine($row)
-      {
-          $html = '';
-          $firstCol = true;
+	public function renderLine($row)
+	  {
+		  $html = '';
+		  $firstCol = true;
                
-          $titleNames = array_keys($this->titles);
-          foreach ($titleNames as $rowkey )
-          {        
-              // Check missing value
-              if (property_exists($row, $rowkey))
-              {
-                  switch ($rowkey)
-                  {
-                      case 'action_match': // display text levels
-                      case 'action_nomatch':
-                          $val=$this->status_display[$row->$rowkey];
-                          break;
-                      case 'trap_oid': // try to traslate oids.
+		  $titleNames = array_keys($this->titles);
+		  foreach ($titleNames as $rowkey )
+		  {        
+			  // Check missing value
+			  if (property_exists($row, $rowkey))
+			  {
+				  switch ($rowkey)
+				  {
+					  case 'action_match': // display text levels
+					  case 'action_nomatch':
+						  $val=$this->status_display[$row->$rowkey];
+						  break;
+					  case 'trap_oid': // try to traslate oids.
                           
-                          if ($this->doTranslate === true)
-                          {
-                              $oidName = $this->MIB->translateOID($row->$rowkey);
-                              if (isset($oidName['name']))
-                              {
-                                  $val=$oidName['name'];
-                              }
-                              else
-                              {
-                                  $val = $row->$rowkey;
-                              }
-                          }
-                          else
-                          {
-                              $val = $row->$rowkey;
-                          }
-                          break;
-                      case 'host_name': // switch to hostgroup if name is null
-                          if ($row->$rowkey == null)
-                          {
-                              $val = $row->host_group_name;
-                          }
-                          else
-                          {
-                              $val = $row->$rowkey;
-                          }
-                          break;
-                      default:
-                          $val = $row->$rowkey;
-                  }
-                  if ($rowkey == 'trap_oid' && $this->doTranslate===true)
-                  {
+						  if ($this->doTranslate === true)
+						  {
+							  $oidName = $this->MIB->translateOID($row->$rowkey);
+							  if (isset($oidName['name']))
+							  {
+								  $val=$oidName['name'];
+							  }
+							  else
+							  {
+								  $val = $row->$rowkey;
+							  }
+						  }
+						  else
+						  {
+							  $val = $row->$rowkey;
+						  }
+						  break;
+					  case 'host_name': // switch to hostgroup if name is null
+						  if ($row->$rowkey == null)
+						  {
+							  $val = $row->host_group_name;
+						  }
+						  else
+						  {
+							  $val = $row->$rowkey;
+						  }
+						  break;
+					  default:
+						  $val = $row->$rowkey;
+				  }
+				  if ($rowkey == 'trap_oid' && $this->doTranslate===true)
+				  {
                       
-                  }
-              } else {
-                  $val = '-';
-              }
-              if ($firstCol === true) { // Put link in first column for trap detail.
-                  $html .= '<td class="traphover">'
-                      . $this->view->qlink(
-                          $this->view->escape($val),
-                          Url::fromPath(
-                              $this->urlPath . '/handler/add',
-                              array('ruleid' => $row->id)
-                              )
-                          );
-                  if ($row->comment != '')
-                  {
-                      $html.= '<span class="tohover">'. $row->comment .'</span></td>';
-                  }
-                  $html.= '</td>';
-              } else {
-                  $html .= '<td>' . $this->view->escape($val) . '</td>';
-              }
-              $firstCol=false;
+				  }
+			  } else {
+				  $val = '-';
+			  }
+			  if ($firstCol === true) { // Put link in first column for trap detail.
+				  $html .= '<td class="traphover">'
+					  . $this->view->qlink(
+						  $this->view->escape($val),
+						  Url::fromPath(
+							  $this->urlPath . '/handler/add',
+							  array('ruleid' => $row->id)
+							  )
+						  );
+				  if ($row->comment != '')
+				  {
+					  $html.= '<span class="tohover">'. $row->comment .'</span></td>';
+				  }
+				  $html.= '</td>';
+			  } else {
+				  $html .= '<td>' . $this->view->escape($val) . '</td>';
+			  }
+			  $firstCol=false;
               
-          }
-          return $html;
-      }
+		  }
+		  return $html;
+	  }
 
 }
\ No newline at end of file

Spacing +24 added lines, -24 removed lines

@@ -20,22 +20,22 @@ 
     protected $MIB;
     
     // categories
-    protected $categories = NULL;
+    protected $categories=NULL;
     
     public function setCategoriesArray(array $categories)
     {
-        $this->categories = $categories;
+        $this->categories=$categories;
     }
     
-    public function groupingPrintData( string $value)
+    public function groupingPrintData(string $value)
     {
         if ($this->groupingColumn == 'rule_type')
         {
-            if ($this->categories == NULL || (! isset($this->categories[$value])))
+            if ($this->categories == NULL || (!isset($this->categories[$value])))
                 return 'Unknown category ('.$value.')';
-            return 'Category : '. $this->categories[$value];
+            return 'Category : '.$this->categories[$value];
         }
-        $html = "$value";
+        $html="$value";
         return $html;
     }
     
@@ -58,16 +58,16 @@ 
     
     public function getCurrentURL()
     {
-        return Url::fromPath($this->urlPath . '/handler');
+        return Url::fromPath($this->urlPath.'/handler');
     }
     
     public function renderLine($row)
       {
-          $html = '';
-          $firstCol = true;
+          $html='';
+          $firstCol=true;
                
-          $titleNames = array_keys($this->titles);
-          foreach ($titleNames as $rowkey )
+          $titleNames=array_keys($this->titles);
+          foreach ($titleNames as $rowkey)
           {        
               // Check missing value
               if (property_exists($row, $rowkey))
@@ -82,57 +82,57 @@ 
                           
                           if ($this->doTranslate === true)
                           {
-                              $oidName = $this->MIB->translateOID($row->$rowkey);
+                              $oidName=$this->MIB->translateOID($row->$rowkey);
                               if (isset($oidName['name']))
                               {
                                   $val=$oidName['name'];
                               }
                               else
                               {
-                                  $val = $row->$rowkey;
+                                  $val=$row->$rowkey;
                               }
                           }
                           else
                           {
-                              $val = $row->$rowkey;
+                              $val=$row->$rowkey;
                           }
                           break;
                       case 'host_name': // switch to hostgroup if name is null
                           if ($row->$rowkey == null)
                           {
-                              $val = $row->host_group_name;
+                              $val=$row->host_group_name;
                           }
                           else
                           {
-                              $val = $row->$rowkey;
+                              $val=$row->$rowkey;
                           }
                           break;
                       default:
-                          $val = $row->$rowkey;
+                          $val=$row->$rowkey;
                   }
-                  if ($rowkey == 'trap_oid' && $this->doTranslate===true)
+                  if ($rowkey == 'trap_oid' && $this->doTranslate === true)
                   {
                       
                   }
               } else {
-                  $val = '-';
+                  $val='-';
               }
               if ($firstCol === true) { // Put link in first column for trap detail.
-                  $html .= '<td class="traphover">'
+                  $html.='<td class="traphover">'
                       . $this->view->qlink(
                           $this->view->escape($val),
                           Url::fromPath(
-                              $this->urlPath . '/handler/add',
+                              $this->urlPath.'/handler/add',
                               array('ruleid' => $row->id)
                               )
                           );
                   if ($row->comment != '')
                   {
-                      $html.= '<span class="tohover">'. $row->comment .'</span></td>';
+                      $html.='<span class="tohover">'.$row->comment.'</span></td>';
                   }
-                  $html.= '</td>';
+                  $html.='</td>';
               } else {
-                  $html .= '<td>' . $this->view->escape($val) . '</td>';
+                  $html.='<td>'.$this->view->escape($val).'</td>';
               }
               $firstCol=false;
               

library/Trapdirector/Tables/TrapDirectorTableGrouping.php

Indentation +67 added lines, -67 removed lines

@@ -7,85 +7,85 @@
 {
     
   
-    /*************** Grouping ************/
+	/*************** Grouping ************/
     
-    /** @var boolean $grouppingActive set to true if grouping is active by query or function call */
-    protected $grouppingActive=false;
+	/** @var boolean $grouppingActive set to true if grouping is active by query or function call */
+	protected $grouppingActive=false;
     
-    /** @var string $groupingColumn Name of column (can be hidden) for grouping */
-    protected $groupingColumn='';
+	/** @var string $groupingColumn Name of column (can be hidden) for grouping */
+	protected $groupingColumn='';
     
-    /**@var string $groupingVal Current value of grouping column in row (used while rendering) */
-    protected $groupingVal='';
+	/**@var string $groupingVal Current value of grouping column in row (used while rendering) */
+	protected $groupingVal='';
     
-    /** @var integer $groupingColSpan colspan of grouping line : set to table titles in init */
-    protected $groupingColSpan=1;
+	/** @var integer $groupingColSpan colspan of grouping line : set to table titles in init */
+	protected $groupingColSpan=1;
       
-    /*****************  Grouping ****************/
+	/*****************  Grouping ****************/
     
-    /**
-     * Set grouping. column must be DB name
-     * @param string $columnDBName
-     */
-    public function setGrouping(string $columnDBName)
-    {
-        $this->groupingColumn = $columnDBName;
-        $this->grouppingActive = TRUE;
-    }
+	/**
+	 * Set grouping. column must be DB name
+	 * @param string $columnDBName
+	 */
+	public function setGrouping(string $columnDBName)
+	{
+		$this->groupingColumn = $columnDBName;
+		$this->grouppingActive = TRUE;
+	}
     
-    /**
-     * Init of grouping before rendering
-     */
-    public function initGrouping()
-    {
-        $this->groupingVal = '';
-        $this->groupingColSpan = count($this->titles);
-    }
+	/**
+	 * Init of grouping before rendering
+	 */
+	public function initGrouping()
+	{
+		$this->groupingVal = '';
+		$this->groupingColSpan = count($this->titles);
+	}
     
-    /**
-     * Function to print grouping value (for ovveride in specific tables)
-     * @param string $value
-     * @return string
-     */
-    public function groupingPrintData( string $value )
-    {
-        $html = "$value";
-        return $html;
-    }
+	/**
+	 * Function to print grouping value (for ovveride in specific tables)
+	 * @param string $value
+	 * @return string
+	 */
+	public function groupingPrintData( string $value )
+	{
+		$html = "$value";
+		return $html;
+	}
     
-    /**
-     * When to display new grouping line  (for ovveride in specific tables)
-     * @param string $val1 Current value in grouping
-     * @param string $val2 Value of current line
-     * @return boolean TRUE if a new grouping line is needed.
-     */
-    public function groupingEvalNext(string $val1, string $val2)
-    {
-        if ($val1 != $val2)
-            return TRUE;
-        else
-            return FALSE;
-    }
+	/**
+	 * When to display new grouping line  (for ovveride in specific tables)
+	 * @param string $val1 Current value in grouping
+	 * @param string $val2 Value of current line
+	 * @return boolean TRUE if a new grouping line is needed.
+	 */
+	public function groupingEvalNext(string $val1, string $val2)
+	{
+		if ($val1 != $val2)
+			return TRUE;
+		else
+			return FALSE;
+	}
     
-    /**
-     * Called before each line to check if grouping line is needed.
-     * @param mixed $values
-     * @return string with line or empty.
-     */
-    public function groupingNextLine( $values)
-    {
-        if ($this->grouppingActive === FALSE) return '';
+	/**
+	 * Called before each line to check if grouping line is needed.
+	 * @param mixed $values
+	 * @return string with line or empty.
+	 */
+	public function groupingNextLine( $values)
+	{
+		if ($this->grouppingActive === FALSE) return '';
         
-        $dbcol = $this->groupingColumn;
-        if ($this->groupingVal == '' || $this->groupingEvalNext($this->groupingVal ,$values->$dbcol) === TRUE )
-        {
-            $this->groupingVal = $values->$dbcol;
-            $html = '<tr><th colspan="'. $this->groupingColSpan .'">'. $this->groupingPrintData($this->groupingVal) .'</th></tr>';
-            return $html;
-        }
-        return '';
+		$dbcol = $this->groupingColumn;
+		if ($this->groupingVal == '' || $this->groupingEvalNext($this->groupingVal ,$values->$dbcol) === TRUE )
+		{
+			$this->groupingVal = $values->$dbcol;
+			$html = '<tr><th colspan="'. $this->groupingColSpan .'">'. $this->groupingPrintData($this->groupingVal) .'</th></tr>';
+			return $html;
+		}
+		return '';
         
-    }
+	}
     
   
 }
\ No newline at end of file

Spacing +11 added lines, -11 removed lines

@@ -29,8 +29,8 @@ 
      */
     public function setGrouping(string $columnDBName)
     {
-        $this->groupingColumn = $columnDBName;
-        $this->grouppingActive = TRUE;
+        $this->groupingColumn=$columnDBName;
+        $this->grouppingActive=TRUE;
     }
     
     /**
@@ -38,8 +38,8 @@ 
      */
     public function initGrouping()
     {
-        $this->groupingVal = '';
-        $this->groupingColSpan = count($this->titles);
+        $this->groupingVal='';
+        $this->groupingColSpan=count($this->titles);
     }
     
     /**
@@ -47,9 +47,9 @@ 
      * @param string $value
      * @return string
      */
-    public function groupingPrintData( string $value )
+    public function groupingPrintData(string $value)
     {
-        $html = "$value";
+        $html="$value";
         return $html;
     }
     
@@ -72,15 +72,15 @@ 
      * @param mixed $values
      * @return string with line or empty.
      */
-    public function groupingNextLine( $values)
+    public function groupingNextLine($values)
     {
         if ($this->grouppingActive === FALSE) return '';
         
-        $dbcol = $this->groupingColumn;
-        if ($this->groupingVal == '' || $this->groupingEvalNext($this->groupingVal ,$values->$dbcol) === TRUE )
+        $dbcol=$this->groupingColumn;
+        if ($this->groupingVal == '' || $this->groupingEvalNext($this->groupingVal, $values->$dbcol) === TRUE)
         {
-            $this->groupingVal = $values->$dbcol;
-            $html = '<tr><th colspan="'. $this->groupingColSpan .'">'. $this->groupingPrintData($this->groupingVal) .'</th></tr>';
+            $this->groupingVal=$values->$dbcol;
+            $html='<tr><th colspan="'.$this->groupingColSpan.'">'.$this->groupingPrintData($this->groupingVal).'</th></tr>';
             return $html;
         }
         return '';

Braces +8 added lines, -5 removed lines

@@ -61,10 +61,11 @@ 
      */
     public function groupingEvalNext(string $val1, string $val2)
     {
-        if ($val1 != $val2)
-            return TRUE;
-        else
-            return FALSE;
+        if ($val1 != $val2) {
+                    return TRUE;
+        } else {
+                    return FALSE;
+        }
     }
     
     /**
@@ -74,7 +75,9 @@ 
      */
     public function groupingNextLine( $values)
     {
-        if ($this->grouppingActive === FALSE) return '';
+        if ($this->grouppingActive === FALSE) {
+        	return '';
+        }
         
         $dbcol = $this->groupingColumn;
         if ($this->groupingVal == '' || $this->groupingEvalNext($this->groupingVal ,$values->$dbcol) === TRUE )

library/Trapdirector/Tables/TrapDirectorTableOrder.php

Indentation +50 added lines, -50 removed lines

@@ -5,65 +5,65 @@
 
 trait TrapDirectorTableOrder
 { 
-    /** @var array $order : (db column, 'ASC' | 'DESC') */
-    protected $order = array();
-    /** @var string $orderQuery passed by GET */
-    protected $orderQuery = '';   
+	/** @var array $order : (db column, 'ASC' | 'DESC') */
+	protected $order = array();
+	/** @var string $orderQuery passed by GET */
+	protected $orderQuery = '';   
     
-    /** used var & functions of trapDirectorTable **/
-    protected $query;
+	/** used var & functions of trapDirectorTable **/
+	protected $query;
     
    /***************** Ordering ********************/
     
-    public function applyOrder()
-    {
-        if (count($this->order) == 0)
-        {
-            return $this;
-        }
-        $orderSQL='';
-        foreach ($this->order as $column => $direction)
-        {
-            if ($orderSQL != "") $orderSQL.=',';
+	public function applyOrder()
+	{
+		if (count($this->order) == 0)
+		{
+			return $this;
+		}
+		$orderSQL='';
+		foreach ($this->order as $column => $direction)
+		{
+			if ($orderSQL != "") $orderSQL.=',';
             
-            $orderSQL .= $column . ' ' . $direction;
-        }
-        $this->query = $this->query->order($orderSQL);
+			$orderSQL .= $column . ' ' . $direction;
+		}
+		$this->query = $this->query->order($orderSQL);
         
-        return $this;
-    }
+		return $this;
+	}
     
-    public function setOrder(array $order)
-    {
-        $this->order = $order;
-        return $this;
-    }
+	public function setOrder(array $order)
+	{
+		$this->order = $order;
+		return $this;
+	}
 
-    public function isOrderSet()
-    {
-        if (count($this->order) == 0) return FALSE;
-        return TRUE;
-    }
+	public function isOrderSet()
+	{
+		if (count($this->order) == 0) return FALSE;
+		return TRUE;
+	}
     
-    public function getOrderQuery(array $getVars)
-    {
-        if (isset($getVars['o']))
-        {
-            $this->orderQuery = $getVars['o'];
-            $match = array();
-            if (preg_match('/(.*)(ASC|DESC)$/', $this->orderQuery , $match))
-            {
-                $orderArray=array($match[1] => $match[2]);
-                echo "$match[1] => $match[2]";
-                $this->setOrder($orderArray);
-            }
-        }
-    }
+	public function getOrderQuery(array $getVars)
+	{
+		if (isset($getVars['o']))
+		{
+			$this->orderQuery = $getVars['o'];
+			$match = array();
+			if (preg_match('/(.*)(ASC|DESC)$/', $this->orderQuery , $match))
+			{
+				$orderArray=array($match[1] => $match[2]);
+				echo "$match[1] => $match[2]";
+				$this->setOrder($orderArray);
+			}
+		}
+	}
     
-    protected function curOrderQuery()
-    {
-        if ($this->orderQuery == '') return '';
-        return 'o='.$this->orderQuery;
-    }
+	protected function curOrderQuery()
+	{
+		if ($this->orderQuery == '') return '';
+		return 'o='.$this->orderQuery;
+	}
         
 }
\ No newline at end of file

Spacing +8 added lines, -8 removed lines

@@ -6,9 +6,9 @@ 
 trait TrapDirectorTableOrder
 { 
     /** @var array $order : (db column, 'ASC' | 'DESC') */
-    protected $order = array();
+    protected $order=array();
     /** @var string $orderQuery passed by GET */
-    protected $orderQuery = '';   
+    protected $orderQuery='';   
     
     /** used var & functions of trapDirectorTable **/
     protected $query;
@@ -26,16 +26,16 @@ 
         {
             if ($orderSQL != "") $orderSQL.=',';
             
-            $orderSQL .= $column . ' ' . $direction;
+            $orderSQL.=$column.' '.$direction;
         }
-        $this->query = $this->query->order($orderSQL);
+        $this->query=$this->query->order($orderSQL);
         
         return $this;
     }
     
     public function setOrder(array $order)
     {
-        $this->order = $order;
+        $this->order=$order;
         return $this;
     }
 
@@ -49,9 +49,9 @@ 
     {
         if (isset($getVars['o']))
         {
-            $this->orderQuery = $getVars['o'];
-            $match = array();
-            if (preg_match('/(.*)(ASC|DESC)$/', $this->orderQuery , $match))
+            $this->orderQuery=$getVars['o'];
+            $match=array();
+            if (preg_match('/(.*)(ASC|DESC)$/', $this->orderQuery, $match))
             {
                 $orderArray=array($match[1] => $match[2]);
                 echo "$match[1] => $match[2]";