patrickpr / trapdirector

library/Trapdirector/Tables/TrapDirectorTableFilter.php

Spacing +6 added lines, -6 removed lines

@@ -8,10 +8,10 @@ 
     
     /********* Filter ********/
     /** @var string $filterString : string filter for db columns */
-    protected $filterString = '';
+    protected $filterString='';
     
     /** @var array $filterColumn : columns to apply filter to */
-    protected $filterColumn = array();
+    protected $filterColumn=array();
     
     protected $filterQuery='';
     
@@ -29,7 +29,7 @@ 
         {
             if ($filter != "") $filter.=' OR ';
             //$filter .= "'" . $column . "' LIKE '%" . $this->filterString. "%'";
-            $filter .= $column  . " LIKE '%" . $this->filterString. "%'";
+            $filter.=$column." LIKE '%".$this->filterString."%'";
         }
         //echo $filter;
         
@@ -40,8 +40,8 @@ 
 
     public function setFilter(string $filter, array $filterCol)
     {
-        $this->filterString = $filter;
-        $this->filterColumn = $filterCol;
+        $this->filterString=$filter;
+        $this->filterColumn=$filterCol;
         return $this;
     }
    
@@ -63,7 +63,7 @@ 
     {
         if (isset($getVars['f']))
         {
-            $this->filterQuery = $getVars['f'];
+            $this->filterQuery=$getVars['f'];
             $this->setFilter(html_entity_decode($getVars['f']), $this->columnNames);
         }
     }

library/Trapdirector/Tables/TrapDirectorTable.php

Spacing +46 added lines, -46 removed lines

@@ -12,16 +12,16 @@ 
     use TrapDirectorTableGrouping;
     
     /** @var array $titles table titles (name, display value) */
-    protected $titles = null;
+    protected $titles=null;
 
     /** @var array $content table content  (name, sb column name). name index must be the same as $titles*/
-    protected $content = null;
+    protected $content=null;
     
     /** @var array $columnNames names of columns for filtering */
-    protected $columnNames = array();
+    protected $columnNames=array();
     
     /** @var mixed $dbConn connection to database  */
-    protected $dbConn = null;
+    protected $dbConn=null;
     
     /** Current view **/
     protected $view;
@@ -30,21 +30,21 @@ 
     
     // Database stuff
    /** @var array $table (db ref, name) */
-    protected $table = array();
+    protected $table=array();
     
     /** @var mixed  $query Query in database; */
-    protected $query = null;
+    protected $query=null;
     
-    function __construct(array $table,array $titles, array $columns, array $columnNames, $dbConn , $view, $urlPath)
+    function __construct(array $table, array $titles, array $columns, array $columnNames, $dbConn, $view, $urlPath)
     {
-        $this->table = $table;
-        $this->titles = $titles;
-        $this->content = $columns;
-        $this->columnNames = $columnNames;
-        $this->dbConn = $dbConn;
+        $this->table=$table;
+        $this->titles=$titles;
+        $this->content=$columns;
+        $this->columnNames=$columnNames;
+        $this->dbConn=$dbConn;
         
-        $this->view = $view;
-        $this->urlPath = $urlPath;
+        $this->view=$view;
+        $this->urlPath=$urlPath;
         
         return $this;
     }
@@ -65,19 +65,19 @@ 
     
     protected function getCurrentURLAndQS(string $caller)
     {
-        $actionURL = $this->getCurrentURL() . '?' ;
+        $actionURL=$this->getCurrentURL().'?';
         $QSList=array();
         if ($caller != 'filter' && $this->curFilterQuery() != '')
-            array_push($QSList , $this->curFilterQuery());
+            array_push($QSList, $this->curFilterQuery());
         
         if ($caller != 'paging' && $caller != 'filter' && $this->curPagingQuery() != '')
-            array_push($QSList , $this->curPagingQuery());
+            array_push($QSList, $this->curPagingQuery());
  
         if ($caller != 'order' && $this->curOrderQuery() != '')
-            array_push($QSList , $this->curOrderQuery());
+            array_push($QSList, $this->curOrderQuery());
         
         if (count($QSList) != 0)
-            $actionURL .=  implode('&', $QSList) . '&';
+            $actionURL.=implode('&', $QSList).'&';
         
         return $actionURL;
     }
@@ -89,8 +89,8 @@ 
      */
     public function getBaseQuery()
     {
-        $this->query = $this->dbConn->select();
-        $this->query = $this->query->from(
+        $this->query=$this->dbConn->select();
+        $this->query=$this->query->from(
             $this->table,
             $this->content
             );
@@ -118,10 +118,10 @@ 
     
     public function renderTitles()
     {
-        $html = "<thead>\n<tr>\n";
+        $html="<thead>\n<tr>\n";
         foreach ($this->titles as $name => $values)
         {
-            $titleOrder = $this->titleOrder($name);  // TODO : put this as function of order trait
+            $titleOrder=$this->titleOrder($name); // TODO : put this as function of order trait
             if ($titleOrder != NULL)
             {
                 if (isset($this->order[$titleOrder]))
@@ -139,59 +139,59 @@ 
                 {
                     $titleOrder.='ASC';
                 }
-                $actionURL = $this->getCurrentURLAndQS('order').'o='.$titleOrder;
-                $html .= '<th><a href="'.$actionURL.'">' . $values . '</a></th>';
+                $actionURL=$this->getCurrentURLAndQS('order').'o='.$titleOrder;
+                $html.='<th><a href="'.$actionURL.'">'.$values.'</a></th>';
             }
             else 
             {
-                $html .= '<th>' . $values . '</th>';
+                $html.='<th>'.$values.'</th>';
             }          
         }
-        $html .= "</tr>\n</thead>\n";
+        $html.="</tr>\n</thead>\n";
         return $html;
     }
     
-    public function renderLine( $value)
+    public function renderLine($value)
     {
-        $html = '';
-        $titleNames = array_keys($this->titles);
-        foreach ($titleNames as $name )
+        $html='';
+        $titleNames=array_keys($this->titles);
+        foreach ($titleNames as $name)
         {
-            $html .= '<td>';
-            $html .= $value->$name;
-            $html .= "</td>\n";
+            $html.='<td>';
+            $html.=$value->$name;
+            $html.="</td>\n";
         }
         return $html;
     }
     
     public function renderTable(array $values)
     {
-       $html = '<tbody id="obj_table_body">';
-       foreach($values as $value)
+       $html='<tbody id="obj_table_body">';
+       foreach ($values as $value)
        {
-           $html .= $this->groupingNextLine($value);
+           $html.=$this->groupingNextLine($value);
            
-           $html .= "<tr>\n";
-           $html .= $this->renderLine($value);
-           $html .= "</tr>\n";
+           $html.="<tr>\n";
+           $html.=$this->renderLine($value);
+           $html.="</tr>\n";
        }
-       $html .= '</tbody>';
+       $html.='</tbody>';
        return $html;
     }
     
     public function render()
     {
-        $html = '';
+        $html='';
         
         
-        $values = $this->fullQuery();
+        $values=$this->fullQuery();
         $this->initGrouping();
         
         $html.="<table class='simple common-table table-row-selectable'>\n";
         
-        $html .= $this->renderTitles();
-        $html .= $this->renderTable($values);
-        $html .= '</table>'; 
+        $html.=$this->renderTitles();
+        $html.=$this->renderTable($values);
+        $html.='</table>'; 
         
 
         return $html;

library/Trapdirector/Tables/TrapDirectorTablePaging.php

Spacing +31 added lines, -31 removed lines

@@ -7,17 +7,17 @@ 
 {
    
     /*************** Paging *************/
-    protected $maxPerPage = 25;
+    protected $maxPerPage=25;
     
-    protected $currentPage = 0;
+    protected $currentPage=0;
         
  
     /*****************  Paging and counting *********/
     
     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(*)')
@@ -33,14 +33,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'];
         }
     }
 
@@ -52,22 +52,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>
@@ -78,24 +78,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>';
@@ -103,7 +103,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>
@@ -114,23 +114,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/TrapDirectorTableOrder.php

Spacing +8 added lines, -8 removed lines

@@ -6,8 +6,8 @@ 
 trait TrapDirectorTableOrder
 { 
     /** @var array $order : (db column, 'ASC' | 'DESC') */
-    protected $order = array();
-    protected $orderQuery = '';   
+    protected $order=array();
+    protected $orderQuery='';   
     
    /***************** Ordering ********************/
     
@@ -22,16 +22,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;
     }
 
@@ -45,9 +45,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]";

library/Trapdirector/Tables/TrapDirectorTableGrouping.php

Spacing +11 added lines, -11 removed lines

@@ -20,31 +20,31 @@
     
     public function setGrouping($columnDBName)
     {
-        $this->groupingColumn = $columnDBName;
-        $this->grouppingActive = TRUE;
+        $this->groupingColumn=$columnDBName;
+        $this->grouppingActive=TRUE;
     }
     
     public function initGrouping()
     {
-        $this->groupingVal = '';
-        $this->groupingColSpan = count($this->titles);
+        $this->groupingVal='';
+        $this->groupingColSpan=count($this->titles);
     }
     
-    public function groupingPrintData( $value)
+    public function groupingPrintData($value)
     {
-        $html = "$value";
+        $html="$value";
         return $html;
     }
     
-    public function groupingNextLine( $values)
+    public function groupingNextLine($values)
     {
         if ($this->grouppingActive === FALSE) return '';
         
-        $dbcol = $this->groupingColumn;
-        if ($this->groupingVal == '' || $this->groupingVal != $values->$dbcol )
+        $dbcol=$this->groupingColumn;
+        if ($this->groupingVal == '' || $this->groupingVal != $values->$dbcol)
         {
-            $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 '';