@@ -84,8 +84,8 @@ discard block |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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 |
||
| 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); |
@@ -347,7 +347,7 @@ discard block |
||
| 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 |
||
| 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 | |
@@ -48,7 +48,7 @@ |
||
| 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 | |