Passed
Push — newinternal ( a30d14...b14046 )
by Simon
24:52 queued 14:51
created
includes/Helpers/SearchHelpers/SearchHelperBase.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -84,8 +84,8 @@  discard block
 block discarded – undo
84 84
     {
85 85
         $this->whereClause .= $whereClauseSection;
86 86
 
87
-        $countQuery = 'SELECT /* SearchHelper */ COUNT(*) FROM ' . $this->table . ' origin ';
88
-        $countQuery .= $this->joinClause . $this->whereClause;
87
+        $countQuery = 'SELECT /* SearchHelper */ COUNT(*) FROM '.$this->table.' origin ';
88
+        $countQuery .= $this->joinClause.$this->whereClause;
89 89
 
90 90
         $query = $this->buildQuery(array('*'));
91 91
         $query .= $this->applyOrder();
@@ -147,7 +147,7 @@  discard block
 block discarded – undo
147 147
                 throw new ApplicationLogicException('Cannot apply distinct to column fetch already using group by');
148 148
             }
149 149
 
150
-            $this->groupByClause = ' GROUP BY origin.' . $column;
150
+            $this->groupByClause = ' GROUP BY origin.'.$column;
151 151
         }
152 152
 
153 153
         $statement = $this->getData(array($column));
@@ -176,8 +176,8 @@  discard block
 block discarded – undo
176 176
      */
177 177
     public function getRecordCount(&$count)
178 178
     {
179
-        $query = 'SELECT /* SearchHelper */ COUNT(*) FROM ' . $this->table . ' origin ';
180
-        $query .= $this->joinClause . $this->whereClause;
179
+        $query = 'SELECT /* SearchHelper */ COUNT(*) FROM '.$this->table.' origin ';
180
+        $query .= $this->joinClause.$this->whereClause;
181 181
 
182 182
         $statement = $this->database->prepare($query);
183 183
         $statement->execute($this->parameterList);
@@ -224,7 +224,7 @@  discard block
 block discarded – undo
224 224
     private function applyOrder()
225 225
     {
226 226
         if ($this->orderBy !== null) {
227
-            return ' ORDER BY ' . $this->orderBy;
227
+            return ' ORDER BY '.$this->orderBy;
228 228
         }
229 229
 
230 230
         return '';
@@ -257,11 +257,11 @@  discard block
 block discarded – undo
257 257
     {
258 258
         $colData = array();
259 259
         foreach ($columns as $c) {
260
-            $colData[] = 'origin.' . $c;
260
+            $colData[] = 'origin.'.$c;
261 261
         }
262 262
 
263
-        $query = "SELECT {$this->modifiersClause} /* SearchHelper */ " . implode(', ', $colData) . ' FROM ' . $this->table . ' origin ';
264
-        $query .= $this->joinClause . $this->whereClause . $this->groupByClause;
263
+        $query = "SELECT {$this->modifiersClause} /* SearchHelper */ ".implode(', ', $colData).' FROM '.$this->table.' origin ';
264
+        $query .= $this->joinClause.$this->whereClause.$this->groupByClause;
265 265
 
266 266
         return $query;
267 267
     }
@@ -282,7 +282,7 @@  discard block
 block discarded – undo
282 282
         // You can't use IN() with parameters directly, so let's munge something together.
283 283
         // Let's create a string of question marks, which will do as positional parameters.
284 284
         $valueCount = count($values);
285
-        $inSection = str_repeat('?,', $valueCount - 1) . '?';
285
+        $inSection = str_repeat('?,', $valueCount - 1).'?';
286 286
 
287 287
         $this->whereClause .= " AND {$column} IN ({$inSection})";
288 288
         $this->parameterList = array_merge($this->parameterList, $values);
Please login to merge, or discard this patch.
includes/Helpers/LogHelper.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -347,7 +347,7 @@  discard block
 block discarded – undo
347 347
     {
348 348
         $userIds = array();
349 349
 
350
-	    foreach ($logs as $logEntry) {
350
+        foreach ($logs as $logEntry) {
351 351
             if (!$logEntry instanceof Log) {
352 352
                 // if this happens, we've done something wrong with passing back the log data.
353 353
                 throw new Exception('Log entry is not an instance of a Log, this should never happen.');
@@ -368,7 +368,7 @@  discard block
 block discarded – undo
368 368
 
369 369
         $logData = array();
370 370
 
371
-	    foreach ($logs as $logEntry) {
371
+        foreach ($logs as $logEntry) {
372 372
             $objectDescription = self::getObjectDescription($logEntry->getObjectId(), $logEntry->getObjectType(),
373 373
                 $database, $configuration);
374 374
 
Please login to merge, or discard this patch.
includes/Exceptions/AccessDeniedException.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
         $currentUser = User::getCurrent($database);
49 49
         $this->assign('currentUser', $currentUser);
50 50
 
51
-        if($this->securityManager !== null) {
51
+        if ($this->securityManager !== null) {
52 52
             $this->setupNavMenuAccess($currentUser);
53 53
         }
54 54
 
Please login to merge, or discard this patch.