Passed
Push — master ( 6b6536...139055 )
by Roeland
15:14 queued 05:29
created
lib/private/SubAdmin.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -117,9 +117,9 @@  discard block
 block discarded – undo
117 117
 			->execute();
118 118
 
119 119
 		$groups = [];
120
-		while($row = $result->fetch()) {
120
+		while ($row = $result->fetch()) {
121 121
 			$group = $this->groupManager->get($row['gid']);
122
-			if(!is_null($group)) {
122
+			if (!is_null($group)) {
123 123
 				$groups[$group->getGID()] = $group;
124 124
 			}
125 125
 		}
@@ -153,9 +153,9 @@  discard block
 block discarded – undo
153 153
 			->execute();
154 154
 
155 155
 		$users = [];
156
-		while($row = $result->fetch()) {
156
+		while ($row = $result->fetch()) {
157 157
 			$user = $this->userManager->get($row['uid']);
158
-			if(!is_null($user)) {
158
+			if (!is_null($user)) {
159 159
 				$users[] = $user;
160 160
 			}
161 161
 		}
@@ -176,10 +176,10 @@  discard block
 block discarded – undo
176 176
 			->execute();
177 177
 
178 178
 		$subadmins = [];
179
-		while($row = $result->fetch()) {
179
+		while ($row = $result->fetch()) {
180 180
 			$user = $this->userManager->get($row['uid']);
181 181
 			$group = $this->groupManager->get($row['gid']);
182
-			if(!is_null($user) && !is_null($group)) {
182
+			if (!is_null($user) && !is_null($group)) {
183 183
 				$subadmins[] = [
184 184
 					'user'  => $user,
185 185
 					'group' => $group
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
 			->andWhere($qb->expr()->eq('uid', $qb->createNamedParameter($user->getUID())))
210 210
 			->execute();
211 211
 
212
-		$fetch =  $result->fetch();
212
+		$fetch = $result->fetch();
213 213
 		$result->closeCursor();
214 214
 		$result = !empty($fetch) ? true : false;
215 215
 
@@ -248,15 +248,15 @@  discard block
 block discarded – undo
248 248
 	 * @return bool
249 249
 	 */
250 250
 	public function isUserAccessible(IUser $subadmin, IUser $user): bool {
251
-		if(!$this->isSubAdmin($subadmin)) {
251
+		if (!$this->isSubAdmin($subadmin)) {
252 252
 			return false;
253 253
 		}
254
-		if($this->groupManager->isAdmin($user->getUID())) {
254
+		if ($this->groupManager->isAdmin($user->getUID())) {
255 255
 			return false;
256 256
 		}
257 257
 		$accessibleGroups = $this->getSubAdminsGroups($subadmin);
258
-		foreach($accessibleGroups as $accessibleGroup) {
259
-			if($accessibleGroup->inGroup($user)) {
258
+		foreach ($accessibleGroups as $accessibleGroup) {
259
+			if ($accessibleGroup->inGroup($user)) {
260 260
 				return true;
261 261
 			}
262 262
 		}
Please login to merge, or discard this patch.