Passed
Push — master ( 6331f1...762a8b )
by Joas
11:56 queued 10s
created
lib/private/User/Manager.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 		$this->config = $config;
79 79
 		$this->dispatcher = $dispatcher;
80 80
 		$cachedUsers = &$this->cachedUsers;
81
-		$this->listen('\OC\User', 'postDelete', function ($user) use (&$cachedUsers) {
81
+		$this->listen('\OC\User', 'postDelete', function($user) use (&$cachedUsers) {
82 82
 			/** @var \OC\User\User $user */
83 83
 			unset($cachedUsers[$user->getUID()]);
84 84
 		});
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 		$result = $this->checkPasswordNoLogging($loginName, $password);
185 185
 
186 186
 		if ($result === false) {
187
-			\OC::$server->getLogger()->warning('Login failed: \''. $loginName .'\' (Remote IP: \''. \OC::$server->getRequest()->getRemoteAddress(). '\')', ['app' => 'core']);
187
+			\OC::$server->getLogger()->warning('Login failed: \''.$loginName.'\' (Remote IP: \''.\OC::$server->getRequest()->getRemoteAddress().'\')', ['app' => 'core']);
188 188
 		}
189 189
 
190 190
 		return $result;
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 			}
234 234
 		}
235 235
 
236
-		uasort($users, function ($a, $b) {
236
+		uasort($users, function($a, $b) {
237 237
 			/**
238 238
 			 * @var \OC\User\User $a
239 239
 			 * @var \OC\User\User $b
@@ -262,7 +262,7 @@  discard block
 block discarded – undo
262 262
 			}
263 263
 		}
264 264
 
265
-		usort($users, function ($a, $b) {
265
+		usort($users, function($a, $b) {
266 266
 			/**
267 267
 			 * @var \OC\User\User $a
268 268
 			 * @var \OC\User\User $b
@@ -345,7 +345,7 @@  discard block
 block discarded – undo
345 345
 
346 346
 		$this->emit('\OC\User', 'preCreateUser', [$uid, $password]);
347 347
 		$state = $backend->createUser($uid, $password);
348
-		if($state === false) {
348
+		if ($state === false) {
349 349
 			throw new \InvalidArgumentException($l->t('Could not create user'));
350 350
 		}
351 351
 		$user = $this->getUserObject($uid, $backend);
@@ -371,13 +371,13 @@  discard block
 block discarded – undo
371 371
 		foreach ($this->backends as $backend) {
372 372
 			if ($backend->implementsActions(Backend::COUNT_USERS)) {
373 373
 				$backendUsers = $backend->countUsers();
374
-				if($backendUsers !== false) {
375
-					if($backend instanceof IUserBackend) {
374
+				if ($backendUsers !== false) {
375
+					if ($backend instanceof IUserBackend) {
376 376
 						$name = $backend->getBackendName();
377 377
 					} else {
378 378
 						$name = get_class($backend);
379 379
 					}
380
-					if(isset($userCountStatistics[$name])) {
380
+					if (isset($userCountStatistics[$name])) {
381 381
 						$userCountStatistics[$name] += $backendUsers;
382 382
 					} else {
383 383
 						$userCountStatistics[$name] = $backendUsers;
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
 	 */
398 398
 	public function countUsersOfGroups(array $groups) {
399 399
 		$users = [];
400
-		foreach($groups as $group) {
400
+		foreach ($groups as $group) {
401 401
 			$usersIds = array_map(function($user) {
402 402
 				return $user->getUID();
403 403
 			}, $group->getUsers());
@@ -461,7 +461,7 @@  discard block
 block discarded – undo
461 461
 		$result->closeCursor();
462 462
 		
463 463
 		if ($count !== false) {
464
-			$count = (int)$count;
464
+			$count = (int) $count;
465 465
 		} else {
466 466
 			$count = 0;
467 467
 		}
@@ -478,7 +478,7 @@  discard block
 block discarded – undo
478 478
 	 */
479 479
 	public function countDisabledUsersOfGroups(array $groups): int {
480 480
 		$queryBuilder = \OC::$server->getDatabaseConnection()->getQueryBuilder();
481
-		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT ' . $queryBuilder->getColumnName('uid') . ')'))
481
+		$queryBuilder->select($queryBuilder->createFunction('COUNT(DISTINCT '.$queryBuilder->getColumnName('uid').')'))
482 482
 			->from('preferences', 'p')
483 483
 			->innerJoin('p', 'group_user', 'g', $queryBuilder->expr()->eq('p.userid', 'g.uid'))
484 484
 			->where($queryBuilder->expr()->eq('appid', $queryBuilder->createNamedParameter('core')))
@@ -491,7 +491,7 @@  discard block
 block discarded – undo
491 491
 		$result->closeCursor();
492 492
 		
493 493
 		if ($count !== false) {
494
-			$count = (int)$count;
494
+			$count = (int) $count;
495 495
 		} else {
496 496
 			$count = 0;
497 497
 		}
@@ -515,7 +515,7 @@  discard block
 block discarded – undo
515 515
 
516 516
 		$query = $queryBuilder->execute();
517 517
 
518
-		$result = (int)$query->fetchColumn();
518
+		$result = (int) $query->fetchColumn();
519 519
 		$query->closeCursor();
520 520
 
521 521
 		return $result;
@@ -604,7 +604,7 @@  discard block
 block discarded – undo
604 604
 	}
605 605
 
606 606
 	private function verifyUid(string $uid): bool {
607
-		$appdata = 'appdata_' . $this->config->getSystemValueString('instanceid');
607
+		$appdata = 'appdata_'.$this->config->getSystemValueString('instanceid');
608 608
 
609 609
 		if ($uid === '.htaccess' || $uid === 'files_external' || $uid === '.ocdata' || $uid === 'owncloud.log' || $uid === 'nextcloud.log' || $uid === $appdata) {
610 610
 			return false;
Please login to merge, or discard this patch.