nextcloud / server

settings/Controller/GroupsController.php

Spacing +5 added lines, -5 removed lines

@@ -101,16 +101,16 @@ 
 	 * @return DataResponse
 	 */
 	public function create($id) {
-		if($this->groupManager->groupExists($id)) {
+		if ($this->groupManager->groupExists($id)) {
 			return new DataResponse(
 				array(
-					'message' => (string)$this->l10n->t('Group already exists.')
+					'message' => (string) $this->l10n->t('Group already exists.')
 				),
 				Http::STATUS_CONFLICT
 			);
 		}
 		$group = $this->groupManager->createGroup($id);
-		if($group instanceof IGroup) {
+		if ($group instanceof IGroup) {
 			return new DataResponse(['groupname' => $group->getDisplayName()], Http::STATUS_CREATED);
 		}
 
@@ -118,7 +118,7 @@ 
 			array(
 				'status' => 'error',
 				'data' => array(
-					'message' => (string)$this->l10n->t('Unable to add group.')
+					'message' => (string) $this->l10n->t('Unable to add group.')
 				)
 			),
 			Http::STATUS_FORBIDDEN
@@ -147,7 +147,7 @@ 
 			array(
 				'status' => 'error',
 				'data' => array(
-					'message' => (string)$this->l10n->t('Unable to delete group.')
+					'message' => (string) $this->l10n->t('Unable to delete group.')
 				),
 			),
 			Http::STATUS_FORBIDDEN

settings/templates/users/part.grouplist.php

Spacing +8 added lines, -8 removed lines

@@ -26,12 +26,12 @@ 
 	</li>
 
 	<!-- The Admin Group -->
-	<?php foreach($_["adminGroup"] as $adminGroup): ?>
-		<li data-gid="admin" data-usercount="<?php if($adminGroup['usercount'] > 0) { p($adminGroup['usercount']); } ?>" class="isgroup">
+	<?php foreach ($_["adminGroup"] as $adminGroup): ?>
+		<li data-gid="admin" data-usercount="<?php if ($adminGroup['usercount'] > 0) { p($adminGroup['usercount']); } ?>" class="isgroup">
 			<a href="#"><span class="groupname"><?php p($l->t('Admins')); ?></span></a>
 			<div class="app-navigation-entry-utils">
 				<ul>
-					<li class="app-navigation-entry-utils-counter"><?php if($adminGroup['usercount'] > 0) { p($adminGroup['usercount']); } ?></li>
+					<li class="app-navigation-entry-utils-counter"><?php if ($adminGroup['usercount'] > 0) { p($adminGroup['usercount']); } ?></li>
 				</ul>
 			</div>
 		</li>
@@ -39,25 +39,25 @@ 
 
 	<!-- Disabled Users -->
 	<?php $disabledUsersGroup = $_["disabledUsersGroup"] ?>
-	<li data-gid="_disabledUsers" data-usercount="<?php if($disabledUsersGroup['usercount'] > 0) { p($disabledUsersGroup['usercount']); } ?>" class="isgroup">
+	<li data-gid="_disabledUsers" data-usercount="<?php if ($disabledUsersGroup['usercount'] > 0) { p($disabledUsersGroup['usercount']); } ?>" class="isgroup">
 		<a href="#"><span class="groupname"><?php p($l->t('Disabled')); ?></span></a>
 		<div class="app-navigation-entry-utils">
 			<ul>
-				<li class="app-navigation-entry-utils-counter"><?php if($disabledUsersGroup['usercount'] > 0) { p($disabledUsersGroup['usercount']); } ?></li>
+				<li class="app-navigation-entry-utils-counter"><?php if ($disabledUsersGroup['usercount'] > 0) { p($disabledUsersGroup['usercount']); } ?></li>
 			</ul>
 		</div>
 	</li>
 
 	<!--List of Groups-->
-	<?php foreach($_["groups"] as $group): ?>
+	<?php foreach ($_["groups"] as $group): ?>
 		<li data-gid="<?php p($group['id']) ?>" data-usercount="<?php p($group['usercount']) ?>" class="isgroup">
 			<a href="#" class="dorename">
 				<span class="groupname"><?php p($group['name']); ?></span>
 			</a>
 			<div class="app-navigation-entry-utils">
 				<ul>
-					<li class="app-navigation-entry-utils-counter"><?php if($group['usercount'] > 0) { p($group['usercount']); } ?></li>
-				 	<?php if($_['isAdmin']): ?>
+					<li class="app-navigation-entry-utils-counter"><?php if ($group['usercount'] > 0) { p($group['usercount']); } ?></li>
+				 	<?php if ($_['isAdmin']): ?>
 				 		<li class="app-navigation-entry-utils-menu-button delete">
 							<button class="icon-delete"></button>
 						</li>

settings/templates/users/main.php

Spacing +4 added lines, -4 removed lines

@@ -19,11 +19,11 @@ 
 style('settings', 'settings');
 
 $userlistParams = array();
-$allGroups=array();
-foreach($_["adminGroup"] as $group) {
+$allGroups = array();
+foreach ($_["adminGroup"] as $group) {
 	$allGroups[$group['id']] = array('displayName' => $group['name']);
 }
-foreach($_["groups"] as $group) {
+foreach ($_["groups"] as $group) {
 	$allGroups[$group['id']] = array('displayName' => $group['name']);
 }
 $userlistParams['subadmingroups'] = $allGroups;
@@ -40,7 +40,7 @@ 
 	<?php print_unescaped($this->inc('users/part.grouplist')); ?>
 	<div id="app-settings">
 		<div id="app-settings-header">
-			<button class="settings-button" tabindex="0" data-apps-slide-toggle="#app-settings-content"><?php p($l->t('Settings'));?></button>
+			<button class="settings-button" tabindex="0" data-apps-slide-toggle="#app-settings-content"><?php p($l->t('Settings')); ?></button>
 		</div>
 		<div id="app-settings-content">
 			<?php print_unescaped($this->inc('users/part.setquota')); ?>

lib/private/SubAdmin.php

Spacing +11 added lines, -11 removed lines

@@ -120,9 +120,9 @@ 
 			->execute();
 
 		$groups = [];
-		while($row = $result->fetch()) {
+		while ($row = $result->fetch()) {
 			$group = $this->groupManager->get($row['gid']);
-			if(!is_null($group)) {
+			if (!is_null($group)) {
 				$groups[$group->getGID()] = $group;
 			}
 		}
@@ -156,9 +156,9 @@ 
 			->execute();
 
 		$users = [];
-		while($row = $result->fetch()) {
+		while ($row = $result->fetch()) {
 			$user = $this->userManager->get($row['uid']);
-			if(!is_null($user)) {
+			if (!is_null($user)) {
 				$users[] = $user;
 			}
 		}
@@ -179,10 +179,10 @@ 
 			->execute();
 
 		$subadmins = [];
-		while($row = $result->fetch()) {
+		while ($row = $result->fetch()) {
 			$user = $this->userManager->get($row['uid']);
 			$group = $this->groupManager->get($row['gid']);
-			if(!is_null($user) && !is_null($group)) {
+			if (!is_null($user) && !is_null($group)) {
 				$subadmins[] = [
 					'user'  => $user,
 					'group' => $group
@@ -212,7 +212,7 @@ 
 			->andWhere($qb->expr()->eq('uid', $qb->createNamedParameter($user->getUID())))
 			->execute();
 
-		$fetch =  $result->fetch();
+		$fetch = $result->fetch();
 		$result->closeCursor();
 		$result = !empty($fetch) ? true : false;
 
@@ -251,15 +251,15 @@ 
 	 * @return bool
 	 */
 	public function isUserAccessible($subadmin, $user) {
-		if(!$this->isSubAdmin($subadmin)) {
+		if (!$this->isSubAdmin($subadmin)) {
 			return false;
 		}
-		if($this->groupManager->isAdmin($user->getUID())) {
+		if ($this->groupManager->isAdmin($user->getUID())) {
 			return false;
 		}
 		$accessibleGroups = $this->getSubAdminsGroups($subadmin);
-		foreach($accessibleGroups as $accessibleGroup) {
-			if($accessibleGroup->inGroup($user)) {
+		foreach ($accessibleGroups as $accessibleGroup) {
+			if ($accessibleGroup->inGroup($user)) {
 				return true;
 			}
 		}