Passed
Push — master ( d011df...0d46fa )
by Joas
15:48 queued 11s
created
lib/private/User/Database.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
 	 * @return string display name
248 248
 	 */
249 249
 	public function getDisplayName($uid): string {
250
-		$uid = (string)$uid;
250
+		$uid = (string) $uid;
251 251
 		$this->loadUser($uid);
252 252
 		return empty($this->cache[$uid]['displayname']) ? $uid : $this->cache[$uid]['displayname'];
253 253
 	}
@@ -275,9 +275,9 @@  discard block
 block discarded – undo
275 275
 				$query->expr()->eq('configkey', $query->expr()->literal('email')))
276 276
 			)
277 277
 			// sqlite doesn't like re-using a single named parameter here
278
-			->where($query->expr()->iLike('uid', $query->createPositionalParameter('%' . $this->dbConn->escapeLikeParameter($search) . '%')))
279
-			->orWhere($query->expr()->iLike('displayname', $query->createPositionalParameter('%' . $this->dbConn->escapeLikeParameter($search) . '%')))
280
-			->orWhere($query->expr()->iLike('configvalue', $query->createPositionalParameter('%' . $this->dbConn->escapeLikeParameter($search) . '%')))
278
+			->where($query->expr()->iLike('uid', $query->createPositionalParameter('%'.$this->dbConn->escapeLikeParameter($search).'%')))
279
+			->orWhere($query->expr()->iLike('displayname', $query->createPositionalParameter('%'.$this->dbConn->escapeLikeParameter($search).'%')))
280
+			->orWhere($query->expr()->iLike('configvalue', $query->createPositionalParameter('%'.$this->dbConn->escapeLikeParameter($search).'%')))
281 281
 			->orderBy($query->func()->lower('displayname'), 'ASC')
282 282
 			->addOrderBy('uid_lower', 'ASC')
283 283
 			->setMaxResults($limit)
@@ -286,7 +286,7 @@  discard block
 block discarded – undo
286 286
 		$result = $query->execute();
287 287
 		$displayNames = [];
288 288
 		while ($row = $result->fetch()) {
289
-			$displayNames[(string)$row['uid']] = (string)$row['displayname'];
289
+			$displayNames[(string) $row['uid']] = (string) $row['displayname'];
290 290
 		}
291 291
 
292 292
 		return $displayNames;
@@ -315,8 +315,8 @@  discard block
 block discarded – undo
315 315
 			))
316 316
 			->where($query->expr()->eq('k.known_to', $query->createNamedParameter($searcher)))
317 317
 			->andWhere($query->expr()->orX(
318
-				$query->expr()->iLike('u.uid', $query->createNamedParameter('%' . $this->dbConn->escapeLikeParameter($pattern) . '%')),
319
-				$query->expr()->iLike('u.displayname', $query->createNamedParameter('%' . $this->dbConn->escapeLikeParameter($pattern) . '%'))
318
+				$query->expr()->iLike('u.uid', $query->createNamedParameter('%'.$this->dbConn->escapeLikeParameter($pattern).'%')),
319
+				$query->expr()->iLike('u.displayname', $query->createNamedParameter('%'.$this->dbConn->escapeLikeParameter($pattern).'%'))
320 320
 			))
321 321
 			->orderBy('u.displayname', 'ASC')
322 322
 			->addOrderBy('u.uid_lower', 'ASC')
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
 		$result = $query->execute();
327 327
 		$displayNames = [];
328 328
 		while ($row = $result->fetch()) {
329
-			$displayNames[(string)$row['uid']] = (string)$row['displayname'];
329
+			$displayNames[(string) $row['uid']] = (string) $row['displayname'];
330 330
 		}
331 331
 
332 332
 		return $displayNames;
@@ -364,7 +364,7 @@  discard block
 block discarded – undo
364 364
 				if (!empty($newHash)) {
365 365
 					$this->updatePassword($loginName, $newHash);
366 366
 				}
367
-				return (string)$row['uid'];
367
+				return (string) $row['uid'];
368 368
 			}
369 369
 		}
370 370
 
@@ -380,7 +380,7 @@  discard block
 block discarded – undo
380 380
 	private function loadUser($uid) {
381 381
 		$this->fixDI();
382 382
 
383
-		$uid = (string)$uid;
383
+		$uid = (string) $uid;
384 384
 		if (!isset($this->cache[$uid])) {
385 385
 			//guests $uid could be NULL or ''
386 386
 			if ($uid === '') {
@@ -404,8 +404,8 @@  discard block
 block discarded – undo
404 404
 
405 405
 			// "uid" is primary key, so there can only be a single result
406 406
 			if ($row !== false) {
407
-				$this->cache[$uid]['uid'] = (string)$row['uid'];
408
-				$this->cache[$uid]['displayname'] = (string)$row['displayname'];
407
+				$this->cache[$uid]['uid'] = (string) $row['uid'];
408
+				$this->cache[$uid]['displayname'] = (string) $row['displayname'];
409 409
 			} else {
410 410
 				return false;
411 411
 			}
@@ -426,8 +426,8 @@  discard block
 block discarded – undo
426 426
 		$limit = $this->fixLimit($limit);
427 427
 
428 428
 		$users = $this->getDisplayNames($search, $limit, $offset);
429
-		$userIds = array_map(function ($uid) {
430
-			return (string)$uid;
429
+		$userIds = array_map(function($uid) {
430
+			return (string) $uid;
431 431
 		}, array_keys($users));
432 432
 		sort($userIds, SORT_STRING | SORT_FLAG_CASE);
433 433
 		return $userIds;
@@ -452,7 +452,7 @@  discard block
 block discarded – undo
452 452
 	 */
453 453
 	public function getHome(string $uid) {
454 454
 		if ($this->userExists($uid)) {
455
-			return \OC::$server->getConfig()->getSystemValue('datadirectory', \OC::$SERVERROOT . '/data') . '/' . $uid;
455
+			return \OC::$server->getConfig()->getSystemValue('datadirectory', \OC::$SERVERROOT.'/data').'/'.$uid;
456 456
 		}
457 457
 
458 458
 		return false;
@@ -524,7 +524,7 @@  discard block
 block discarded – undo
524 524
 
525 525
 	public function getRealUID(string $uid): string {
526 526
 		if (!$this->userExists($uid)) {
527
-			throw new \RuntimeException($uid . ' does not exist');
527
+			throw new \RuntimeException($uid.' does not exist');
528 528
 		}
529 529
 
530 530
 		return $this->cache[$uid]['uid'];
Please login to merge, or discard this patch.
lib/private/User/Manager.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
 		$this->dispatcher = $oldDispatcher;
102 102
 		$this->cache = $cacheFactory->createDistributed('user_backend_map');
103 103
 		$cachedUsers = &$this->cachedUsers;
104
-		$this->listen('\OC\User', 'postDelete', function ($user) use (&$cachedUsers) {
104
+		$this->listen('\OC\User', 'postDelete', function($user) use (&$cachedUsers) {
105 105
 			/** @var \OC\User\User $user */
106 106
 			unset($cachedUsers[$user->getUID()]);
107 107
 		});
@@ -228,7 +228,7 @@  discard block
 block discarded – undo
228 228
 		$result = $this->checkPasswordNoLogging($loginName, $password);
229 229
 
230 230
 		if ($result === false) {
231
-			\OC::$server->getLogger()->warning('Login failed: \''. $loginName .'\' (Remote IP: \''. \OC::$server->getRequest()->getRemoteAddress(). '\')', ['app' => 'core']);
231
+			\OC::$server->getLogger()->warning('Login failed: \''.$loginName.'\' (Remote IP: \''.\OC::$server->getRequest()->getRemoteAddress().'\')', ['app' => 'core']);
232 232
 		}
233 233
 
234 234
 		return $result;
@@ -291,7 +291,7 @@  discard block
 block discarded – undo
291 291
 			}
292 292
 		}
293 293
 
294
-		uasort($users, function ($a, $b) {
294
+		uasort($users, function($a, $b) {
295 295
 			/**
296 296
 			 * @var \OC\User\User $a
297 297
 			 * @var \OC\User\User $b
@@ -320,7 +320,7 @@  discard block
 block discarded – undo
320 320
 			}
321 321
 		}
322 322
 
323
-		usort($users, function ($a, $b) {
323
+		usort($users, function($a, $b) {
324 324
 			/**
325 325
 			 * @var \OC\User\User $a
326 326
 			 * @var \OC\User\User $b
@@ -355,7 +355,7 @@  discard block
 block discarded – undo
355 355
 			}
356 356
 		}
357 357
 
358
-		usort($users, function ($a, $b) {
358
+		usort($users, function($a, $b) {
359 359
 			/**
360 360
 			 * @var IUser $a
361 361
 			 * @var IUser $b
@@ -505,7 +505,7 @@  discard block
 block discarded – undo
505 505
 	public function countUsersOfGroups(array $groups) {
506 506
 		$users = [];
507 507
 		foreach ($groups as $group) {
508
-			$usersIds = array_map(function ($user) {
508
+			$usersIds = array_map(function($user) {
509 509
 				return $user->getUID();
510 510
 			}, $group->getUsers());
511 511
 			$users = array_merge($users, $usersIds);
@@ -568,7 +568,7 @@  discard block
 block discarded – undo
568 568
 		$result->closeCursor();
569 569
 
570 570
 		if ($count !== false) {
571
-			$count = (int)$count;
571
+			$count = (int) $count;
572 572
 		} else {
573 573
 			$count = 0;
574 574
 		}
@@ -585,7 +585,7 @@  discard block
 block discarded – undo
585 585
 	 */
586 586
 	public function countDisabledUsersOfGroups(array $groups): int {
587 587
 		$queryBuilder = \OC::$server->getDatabaseConnection()->getQueryBuilder();
588
-		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT ' . $queryBuilder->getColumnName('uid') . ')'))
588
+		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT '.$queryBuilder->getColumnName('uid').')'))
589 589
 			->from('preferences', 'p')
590 590
 			->innerJoin('p', 'group_user', 'g', $queryBuilder->expr()->eq('p.userid', 'g.uid'))
591 591
 			->where($queryBuilder->expr()->eq('appid', $queryBuilder->createNamedParameter('core')))
@@ -598,7 +598,7 @@  discard block
 block discarded – undo
598 598
 		$result->closeCursor();
599 599
 
600 600
 		if ($count !== false) {
601
-			$count = (int)$count;
601
+			$count = (int) $count;
602 602
 		} else {
603 603
 			$count = 0;
604 604
 		}
@@ -622,7 +622,7 @@  discard block
 block discarded – undo
622 622
 
623 623
 		$query = $queryBuilder->execute();
624 624
 
625
-		$result = (int)$query->fetchOne();
625
+		$result = (int) $query->fetchOne();
626 626
 		$query->closeCursor();
627 627
 
628 628
 		return $result;
@@ -702,17 +702,17 @@  discard block
 block discarded – undo
702 702
 	public function getByEmail($email) {
703 703
 		$userIds = $this->config->getUsersForUserValueCaseInsensitive('settings', 'email', $email);
704 704
 
705
-		$users = array_map(function ($uid) {
705
+		$users = array_map(function($uid) {
706 706
 			return $this->get($uid);
707 707
 		}, $userIds);
708 708
 
709
-		return array_values(array_filter($users, function ($u) {
709
+		return array_values(array_filter($users, function($u) {
710 710
 			return ($u instanceof IUser);
711 711
 		}));
712 712
 	}
713 713
 
714 714
 	private function verifyUid(string $uid): bool {
715
-		$appdata = 'appdata_' . $this->config->getSystemValueString('instanceid');
715
+		$appdata = 'appdata_'.$this->config->getSystemValueString('instanceid');
716 716
 
717 717
 		if (\in_array($uid, [
718 718
 			'.htaccess',
@@ -725,8 +725,8 @@  discard block
 block discarded – undo
725 725
 			return false;
726 726
 		}
727 727
 
728
-		$dataDirectory = $this->config->getSystemValueString('datadirectory', \OC::$SERVERROOT . '/data');
728
+		$dataDirectory = $this->config->getSystemValueString('datadirectory', \OC::$SERVERROOT.'/data');
729 729
 
730
-		return !file_exists(rtrim($dataDirectory, '/') . '/' . $uid);
730
+		return !file_exists(rtrim($dataDirectory, '/').'/'.$uid);
731 731
 	}
732 732
 }
Please login to merge, or discard this patch.
lib/private/Collaboration/Collaborators/UserPlugin.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 						}
125 125
 					}
126 126
 
127
-					uasort($users, function ($a, $b) {
127
+					uasort($users, function($a, $b) {
128 128
 						/**
129 129
 						 * @var \OC\User\User $a
130 130
 						 * @var \OC\User\User $b
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
 					'message' => $userStatus->getMessage(),
170 170
 					'icon' => $userStatus->getIcon(),
171 171
 					'clearAt' => $userStatus->getClearAt()
172
-						? (int)$userStatus->getClearAt()->format('U')
172
+						? (int) $userStatus->getClearAt()->format('U')
173 173
 						: null,
174 174
 				];
175 175
 			}
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
 							'message' => $userStatus->getMessage(),
254 254
 							'icon' => $userStatus->getIcon(),
255 255
 							'clearAt' => $userStatus->getClearAt()
256
-								? (int)$userStatus->getClearAt()->format('U')
256
+								? (int) $userStatus->getClearAt()->format('U')
257 257
 								: null,
258 258
 						];
259 259
 					}
Please login to merge, or discard this patch.