Passed
Push — master ( ec3d7f...644e6d )
by Roeland
15:01 queued 10s
created
lib/private/User/Manager.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -100,7 +100,7 @@  discard block
 block discarded – undo
100 100
 		$this->dispatcher = $oldDispatcher;
101 101
 		$this->cache = $cacheFactory->createDistributed('user_backend_map');
102 102
 		$cachedUsers = &$this->cachedUsers;
103
-		$this->listen('\OC\User', 'postDelete', function ($user) use (&$cachedUsers) {
103
+		$this->listen('\OC\User', 'postDelete', function($user) use (&$cachedUsers) {
104 104
 			/** @var \OC\User\User $user */
105 105
 			unset($cachedUsers[$user->getUID()]);
106 106
 		});
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
227 227
 		$result = $this->checkPasswordNoLogging($loginName, $password);
228 228
 
229 229
 		if ($result === false) {
230
-			\OC::$server->getLogger()->warning('Login failed: \''. $loginName .'\' (Remote IP: \''. \OC::$server->getRequest()->getRemoteAddress(). '\')', ['app' => 'core']);
230
+			\OC::$server->getLogger()->warning('Login failed: \''.$loginName.'\' (Remote IP: \''.\OC::$server->getRequest()->getRemoteAddress().'\')', ['app' => 'core']);
231 231
 		}
232 232
 
233 233
 		return $result;
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
 			}
291 291
 		}
292 292
 
293
-		uasort($users, function ($a, $b) {
293
+		uasort($users, function($a, $b) {
294 294
 			/**
295 295
 			 * @var \OC\User\User $a
296 296
 			 * @var \OC\User\User $b
@@ -319,7 +319,7 @@  discard block
 block discarded – undo
319 319
 			}
320 320
 		}
321 321
 
322
-		usort($users, function ($a, $b) {
322
+		usort($users, function($a, $b) {
323 323
 			/**
324 324
 			 * @var \OC\User\User $a
325 325
 			 * @var \OC\User\User $b
@@ -469,7 +469,7 @@  discard block
 block discarded – undo
469 469
 	public function countUsersOfGroups(array $groups) {
470 470
 		$users = [];
471 471
 		foreach ($groups as $group) {
472
-			$usersIds = array_map(function ($user) {
472
+			$usersIds = array_map(function($user) {
473 473
 				return $user->getUID();
474 474
 			}, $group->getUsers());
475 475
 			$users = array_merge($users, $usersIds);
@@ -532,7 +532,7 @@  discard block
 block discarded – undo
532 532
 		$result->closeCursor();
533 533
 
534 534
 		if ($count !== false) {
535
-			$count = (int)$count;
535
+			$count = (int) $count;
536 536
 		} else {
537 537
 			$count = 0;
538 538
 		}
@@ -549,7 +549,7 @@  discard block
 block discarded – undo
549 549
 	 */
550 550
 	public function countDisabledUsersOfGroups(array $groups): int {
551 551
 		$queryBuilder = \OC::$server->getDatabaseConnection()->getQueryBuilder();
552
-		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT ' . $queryBuilder->getColumnName('uid') . ')'))
552
+		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT '.$queryBuilder->getColumnName('uid').')'))
553 553
 			->from('preferences', 'p')
554 554
 			->innerJoin('p', 'group_user', 'g', $queryBuilder->expr()->eq('p.userid', 'g.uid'))
555 555
 			->where($queryBuilder->expr()->eq('appid', $queryBuilder->createNamedParameter('core')))
@@ -562,7 +562,7 @@  discard block
 block discarded – undo
562 562
 		$result->closeCursor();
563 563
 
564 564
 		if ($count !== false) {
565
-			$count = (int)$count;
565
+			$count = (int) $count;
566 566
 		} else {
567 567
 			$count = 0;
568 568
 		}
@@ -586,7 +586,7 @@  discard block
 block discarded – undo
586 586
 
587 587
 		$query = $queryBuilder->execute();
588 588
 
589
-		$result = (int)$query->fetchOne();
589
+		$result = (int) $query->fetchOne();
590 590
 		$query->closeCursor();
591 591
 
592 592
 		return $result;
@@ -666,17 +666,17 @@  discard block
 block discarded – undo
666 666
 	public function getByEmail($email) {
667 667
 		$userIds = $this->config->getUsersForUserValueCaseInsensitive('settings', 'email', $email);
668 668
 
669
-		$users = array_map(function ($uid) {
669
+		$users = array_map(function($uid) {
670 670
 			return $this->get($uid);
671 671
 		}, $userIds);
672 672
 
673
-		return array_values(array_filter($users, function ($u) {
673
+		return array_values(array_filter($users, function($u) {
674 674
 			return ($u instanceof IUser);
675 675
 		}));
676 676
 	}
677 677
 
678 678
 	private function verifyUid(string $uid): bool {
679
-		$appdata = 'appdata_' . $this->config->getSystemValueString('instanceid');
679
+		$appdata = 'appdata_'.$this->config->getSystemValueString('instanceid');
680 680
 
681 681
 		if (\in_array($uid, [
682 682
 			'.htaccess',
@@ -688,8 +688,8 @@  discard block
 block discarded – undo
688 688
 			return false;
689 689
 		}
690 690
 
691
-		$dataDirectory = $this->config->getSystemValueString('datadirectory', \OC::$SERVERROOT . '/data');
691
+		$dataDirectory = $this->config->getSystemValueString('datadirectory', \OC::$SERVERROOT.'/data');
692 692
 
693
-		return !file_exists(rtrim($dataDirectory, '/') . '/' . $uid);
693
+		return !file_exists(rtrim($dataDirectory, '/').'/'.$uid);
694 694
 	}
695 695
 }
Please login to merge, or discard this patch.