Passed
Push — search ( de462e...bdb480 )
by Simon
18:54 queued 15:52
created
includes/ConsoleTasks/ClearOAuthDataTask.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
         $users = UserSearchHelper::get($database)->inIds(
22 22
             $database->query('SELECT user FROM oauthtoken WHERE type = \'access\'')->fetchColumn());
23 23
 
24
-        foreach ($users as $u){
24
+        foreach ($users as $u) {
25 25
             $oauth = new OAuthUserHelper($u, $database, $this->getOAuthProtocolHelper(), $this->getSiteConfiguration());
26 26
             $oauth->detach();
27 27
         }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
         $users = UserSearchHelper::get($database)->inIds(
22 22
             $database->query('SELECT user FROM oauthtoken WHERE type = \'access\'')->fetchColumn());
23 23
 
24
-        foreach ($users as $u){
24
+        foreach ($users as $u) {
25 25
             $oauth = new OAuthUserHelper($u, $database, $this->getOAuthProtocolHelper(), $this->getSiteConfiguration());
26 26
             $oauth->detach();
27 27
         }
Please login to merge, or discard this patch.
includes/API/Actions/JsUsersAction.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -23,7 +23,7 @@
 block discarded – undo
23 23
         $userSearchHelper = UserSearchHelper::get($this->getDatabase());
24 24
 
25 25
         if (WebRequest::getString('all') === null) {
26
-           $userSearchHelper->byStatus(User::STATUS_ACTIVE);
26
+            $userSearchHelper->byStatus(User::STATUS_ACTIVE);
27 27
 
28 28
         }
29 29
 
Please login to merge, or discard this patch.
includes/DataObjects/Request.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -420,7 +420,7 @@
 block discarded – undo
420 420
             . '|' . $this->forwardedip  // } private data not known to those without access
421 421
             . '|' . $this->useragent    // }
422 422
             . '|' . $this->email        // }
423
-            . '|' . $this->status;      // to rudimentarily invalidate the token on status change
423
+            . '|' . $this->status; // to rudimentarily invalidate the token on status change
424 424
 
425 425
         return hash('sha256', $data);
426 426
     }
Please login to merge, or discard this patch.
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -235,7 +235,7 @@
 block discarded – undo
235 235
             }
236 236
         }
237 237
 
238
-		$this->forwardedip = implode(", ", $valid);
238
+        $this->forwardedip = implode(", ", $valid);
239 239
     }
240 240
 
241 241
     /**
Please login to merge, or discard this patch.
includes/Pages/PageUserManagement.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
         $userSearchRequest = WebRequest::getString('usersearch');
41 41
         if ($userSearchRequest !== null) {
42 42
             $searchedUser = User::getByUsername($userSearchRequest, $database);
43
-            if($searchedUser !== false) {
43
+            if ($searchedUser !== false) {
44 44
                 $this->redirect('statistics/users', 'detail', ['user' => $searchedUser->getId()]);
45 45
                 return;
46 46
             }
Please login to merge, or discard this patch.
includes/Fragments/RequestData.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -184,7 +184,7 @@
 block discarded – undo
184 184
         $emailDomain = explode("@", $request->getEmail())[1];
185 185
         $this->assign("emailurl", $emailDomain);
186 186
         $this->assign('commonEmailDomain', in_array(strtolower($emailDomain), $configuration->getCommonEmailDomains())
187
-            || $request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail() );
187
+            || $request->getEmail() === $this->getSiteConfiguration()->getDataClearEmail());
188 188
 
189 189
         $trustedIp = $xffProvider->getTrustedClientIp($request->getIp(), $request->getForwardedIp());
190 190
         $this->assign('requestTrustedIp', $trustedIp);
Please login to merge, or discard this patch.
includes/Pages/RequestAction/PageCloseRequest.php 1 patch
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -195,8 +195,7 @@
 block discarded – undo
195 195
         if (
196 196
         ($request->getEmail() != $this->getSiteConfiguration()->getDataClearEmail()) && 
197 197
         ($request->getIp() != $this->getSiteConfiguration()->getDataClearIp())
198
-        )
199
-        {
198
+        ) {
200 199
             $requestEmailHelper = new RequestEmailHelper($this->getEmailHelper());
201 200
             $requestEmailHelper->sendMail($request, $mailText, $currentUser, $ccMailingList);
202 201
             
Please login to merge, or discard this patch.
includes/Helpers/LogHelper.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -361,7 +361,7 @@
 block discarded – undo
361 361
                 }
362 362
 
363 363
                 $task = $job->getTask();
364
-                if (isset($taskDescriptions[$task])){
364
+                if (isset($taskDescriptions[$task])) {
365 365
                     $description = $taskDescriptions[$task];
366 366
                 } 
367 367
                 else {
Please login to merge, or discard this patch.
Braces   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -361,9 +361,9 @@
 block discarded – undo
361 361
                 }
362 362
 
363 363
                 $task = $job->getTask();
364
-                if (isset($taskDescriptions[$task])){
364
+                if (isset($taskDescriptions[$task])) {
365 365
                     $description = $taskDescriptions[$task];
366
-                } 
366
+                }
367 367
                 else {
368 368
                     $description = 'Unknown task';
369 369
                 }
Please login to merge, or discard this patch.
includes/Tasks/PageBase.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -321,7 +321,7 @@
 block discarded – undo
321 321
      */
322 322
     final protected function addJs($path)
323 323
     {
324
-        if (in_array($path, $this->extraJs)){
324
+        if (in_array($path, $this->extraJs)) {
325 325
             // nothing to do
326 326
             return;
327 327
         }
Please login to merge, or discard this patch.
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -321,7 +321,7 @@
 block discarded – undo
321 321
      */
322 322
     final protected function addJs($path)
323 323
     {
324
-        if (in_array($path, $this->extraJs)){
324
+        if (in_array($path, $this->extraJs)) {
325 325
             // nothing to do
326 326
             return;
327 327
         }
Please login to merge, or discard this patch.
includes/Tasks/JsonApiPageBase.php 1 patch
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@
 block discarded – undo
72 72
         if ($targetVar !== null && preg_match('/^[a-z]+$/', $targetVar)) {
73 73
             $data = $targetVar . ' = ' . $data . ';';
74 74
             header("Content-Type: text/javascript");
75
-        } 
75
+        }
76 76
         else {
77 77
             header("Content-Type: application/json");
78 78
         }
Please login to merge, or discard this patch.