Passed
Pull Request — master (#198)
by
unknown
03:01
created
src/plugins/onTick/authCheck.php 1 patch
Braces   +23 added lines, -14 removed lines patch added patch discarded remove patch
@@ -177,8 +177,9 @@  discard block
 block discarded – undo
177 177
             for ($i=0; $i<3; $i++) {
178 178
                 $character = characterDetails($charID);
179 179
                 //if (isset($character['corporation_id']) && in_array($character['corporation_id'], $corpArray))
180
-                if (!is_null($character))
181
-                    break;
180
+                if (!is_null($character)) {
181
+                                    break;
182
+                }
182 183
                 //Postpone check if ESI is down to prevent timeouts
183 184
                 if (isset($character['error']) && $character['error'] === 'The datasource tranquility is temporarily unavailable') {
184 185
                     $this->logger->addInfo('AuthCheck: The datasource tranquility is temporarily unavailable, check canceled.');
@@ -197,8 +198,9 @@  discard block
 block discarded – undo
197 198
             // alliance membership check
198 199
             for ($i=0; $i<3; $i++) {
199 200
                 $corporationDetails = corpDetails($corporationID);
200
-                if (!is_null($corporationDetails))
201
-                    break;
201
+                if (!is_null($corporationDetails)) {
202
+                                    break;
203
+                }
202 204
             }
203 205
             if (is_null($corporationDetails) || isset($corporationDetails['error'])) {
204 206
                 $this->logger->addInfo('AuthCheck: corpDetails lookup failed.');
@@ -223,8 +225,10 @@  discard block
 block discarded – undo
223 225
                     $standings = 1;
224 226
                 }
225 227
             }
226
-            if ($standings)
227
-                continue; //keep the user based on standings
228
+            if ($standings) {
229
+                            continue;
230
+            }
231
+            //keep the user based on standings
228 232
 
229 233
             // User failed all checks, deactivate user in database
230 234
             $sql = "UPDATE authUsers SET active='no' WHERE discordID='$discordID'";
@@ -294,8 +298,10 @@  discard block
 block discarded – undo
294 298
 
295 299
             } while (!isset($character['corporation_id']));
296 300
             $corporationID = $character['corporation_id'];
297
-            if (in_array($corporationID, $corpArray))
298
-                continue; // user is in a valid corp, stop checking
301
+            if (in_array($corporationID, $corpArray)) {
302
+                            continue;
303
+            }
304
+            // user is in a valid corp, stop checking
299 305
 
300 306
             $timeout = 0;
301 307
             do {
@@ -306,8 +312,10 @@  discard block
 block discarded – undo
306 312
                 $timeout++;
307 313
                 $corporationDetails = corpDetails($corporationID);
308 314
             } while (!isset($corporationDetails));
309
-            if (isset($corporationDetails['alliance_id']) && in_array($corporationDetails['alliance_id'], $allianceArray))
310
-                continue; // user is in a valid alliance, stop checking
315
+            if (isset($corporationDetails['alliance_id']) && in_array($corporationDetails['alliance_id'], $allianceArray)) {
316
+                            continue;
317
+            }
318
+            // user is in a valid alliance, stop checking
311 319
 
312 320
             //check if user authed based on standings
313 321
             $standings = null;
@@ -324,8 +332,10 @@  discard block
 block discarded – undo
324 332
                     $standings = 1;
325 333
                 }
326 334
             }
327
-            if ($standings)
328
-                continue; //keep the user based on standings
335
+            if ($standings) {
336
+                            continue;
337
+            }
338
+            //keep the user based on standings
329 339
 
330 340
             // User failed all checks, deactivate user in database
331 341
             $sql = "UPDATE authUsers SET active='no' WHERE discordID='$discordID'";
@@ -467,8 +477,7 @@  discard block
 block discarded – undo
467 477
                     while (null === $character) { //try 10 times to pull characterDetails
468 478
                         if ($timeout > 9) {
469 479
                             continue;
470
-                        }
471
-                        else{
480
+                        } else{
472 481
                             $character = characterDetails($charID);
473 482
                             $timeout++;
474 483
                         }
Please login to merge, or discard this patch.