Passed
Push — master ( 9034c4...6140da )
by Kris
01:44
created
lib/ApiHandler.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
      * @access protected
51 51
      * @var array $selfIps  
52 52
      */
53
-    protected $selfIps = []; 
53
+    protected $selfIps = [ ]; 
54 54
 
55 55
     /**
56 56
      * Constructor
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
      * @param array   $myIps      The Ips/domain name you dont want to display in report messages
62 62
      * 
63 63
      */
64
-    public function __construct(string $apiKey, string $userId, array $myIps = [])
64
+    public function __construct(string $apiKey, string $userId, array $myIps = [ ])
65 65
     {
66 66
         $this->aipdbApiKey = $apiKey;
67 67
         $this->aipdbUserId = $userId;
@@ -100,21 +100,21 @@  discard block
 block discarded – undo
100 100
     {
101 101
 
102 102
         // check file exists
103
-        if (!file_exists($configPath) || !is_file($configPath)){
103
+        if (!file_exists($configPath) || !is_file($configPath)) {
104 104
             throw new \InvalidArgumentException('The file [' . $configPath . '] does not exist.');
105 105
         }
106 106
 
107 107
         // check file is readable
108
-        if (!is_readable($configPath)){
108
+        if (!is_readable($configPath)) {
109 109
             throw new InvalidPermissionException('The file [' . $configPath . '] is not readable.');
110 110
         }
111 111
 
112 112
         $keyConfig = self::loadJsonFile($configPath);
113
-        $selfIps = [];
113
+        $selfIps = [ ];
114 114
         
115 115
         // Look for other optional config files in the same directory 
116 116
         $selfIpsConfigPath = pathinfo($configPath, PATHINFO_DIRNAME) . DIRECTORY_SEPARATOR . 'self_ips.json';
117
-        if (file_exists($selfIpsConfigPath)){
117
+        if (file_exists($selfIpsConfigPath)) {
118 118
             $selfIps = self::loadJsonFile($selfIpsConfigPath)->self_ips;
119 119
         }
120 120
 
@@ -157,17 +157,17 @@  discard block
 block discarded – undo
157 157
     public function report(string $ip = '', string $categories = '', string $message = '', bool $returnArray = false)
158 158
     {
159 159
          // ip must be set
160
-        if (empty($ip)){
160
+        if (empty($ip)) {
161 161
             throw new \InvalidArgumentException('Ip was empty');
162 162
         }
163 163
 
164 164
         // categories must be set
165
-        if (empty($categories)){
165
+        if (empty($categories)) {
166 166
             throw new \InvalidArgumentException('categories list was empty');
167 167
         }
168 168
 
169 169
         // message must be set
170
-          if (empty($message)){
170
+          if (empty($message)) {
171 171
             throw new \InvalidArgumentException('report message was empty');
172 172
         }
173 173
 
@@ -234,12 +234,12 @@  discard block
 block discarded – undo
234 234
     public function checkBlock(string $network = null, int $maxAge = 30, bool $returnArray = false)
235 235
     {
236 236
         // max age must be less or equal to 365
237
-        if ($maxAge > 365 || $maxAge < 1){
237
+        if ($maxAge > 365 || $maxAge < 1) {
238 238
             throw new \InvalidArgumentException('maxAge must be at least 1 and less than 365 (' . $maxAge . ' was given)');
239 239
         }
240 240
 
241 241
         // ip must be set
242
-        if (empty($network)){
242
+        if (empty($network)) {
243 243
             throw new \InvalidArgumentException('network argument must be set (null given)');
244 244
         }
245 245
 
@@ -249,7 +249,7 @@  discard block
 block discarded – undo
249 249
             'maxAgeInDays'  => $maxAge,  
250 250
         ];
251 251
 
252
-        $response = $this->apiRequest('check-block', $data, 'GET', $returnArray) ;
252
+        $response = $this->apiRequest('check-block', $data, 'GET', $returnArray);
253 253
 
254 254
         return json_decode($response, $returnArray);
255 255
     }
@@ -269,12 +269,12 @@  discard block
 block discarded – undo
269 269
     public function check(string $ip = null, int $maxAge = 30, bool $verbose = false, bool $returnArray = false)
270 270
     {
271 271
         // max age must be less or equal to 365
272
-        if ($maxAge > 365 || $maxAge < 1){
272
+        if ($maxAge > 365 || $maxAge < 1) {
273 273
             throw new \InvalidArgumentException('maxAge must be at least 1 and less than 365 (' . $maxAge . ' was given)');
274 274
         }
275 275
 
276 276
         // ip must be set
277
-        if (empty($ip)){
277
+        if (empty($ip)) {
278 278
             throw new \InvalidArgumentException('ip argument must be set (null given)');
279 279
         }
280 280
 
@@ -285,12 +285,12 @@  discard block
 block discarded – undo
285 285
         ];
286 286
 
287 287
         // option
288
-        if ($verbose){
289
-           $data['verbose'] = true;
288
+        if ($verbose) {
289
+           $data[ 'verbose' ] = true;
290 290
         }
291 291
 
292 292
         // check AbuseIPDB request
293
-        $response = $this->apiRequest('check', $data, 'GET', $returnArray) ;
293
+        $response = $this->apiRequest('check', $data, 'GET', $returnArray);
294 294
 
295 295
         return json_decode($response, $returnArray);
296 296
     }
@@ -310,7 +310,7 @@  discard block
 block discarded – undo
310 310
     public function getBlacklist(int $limit = 10000, bool $plainText = false, bool $returnArray = false)
311 311
     {
312 312
 
313
-        if ($limit < 1){
313
+        if ($limit < 1) {
314 314
             throw new \InvalidArgumentException('limit must be at least 1 (' . $limit . ' was given)');
315 315
         }
316 316
 
@@ -322,13 +322,13 @@  discard block
 block discarded – undo
322 322
 
323 323
         // plaintext paremeter has no value and must be added only when true 
324 324
         // (set plaintext=false won't work)
325
-        if ($plainText){
326
-            $data['plaintext'] = $plainText;
325
+        if ($plainText) {
326
+            $data[ 'plaintext' ] = $plainText;
327 327
         }
328 328
 
329 329
         $response = $this->apiRequest('blacklist', $data, 'GET');
330 330
 
331
-        if ($plainText){
331
+        if ($plainText) {
332 332
             return $response;
333 333
         } 
334 334
        
@@ -359,21 +359,21 @@  discard block
 block discarded – undo
359 359
         foreach ($cats as $cat) {
360 360
 
361 361
             // get index on our array of categories
362
-            $catIndex    = is_numeric($cat) ? $this->getCategoryIndex($cat, 1) : $this->getCategoryIndex($cat, 0);
362
+            $catIndex = is_numeric($cat) ? $this->getCategoryIndex($cat, 1) : $this->getCategoryIndex($cat, 0);
363 363
 
364 364
             // check if found
365
-            if ($catIndex === false ){
366
-                throw new \InvalidArgumentException('Invalid report category was given : ['. $cat .  ']');
365
+            if ($catIndex === false) {
366
+                throw new \InvalidArgumentException('Invalid report category was given : [' . $cat . ']');
367 367
             }
368 368
 
369 369
             // get Id
370
-            $catId = $this->aipdbApiCategories[$catIndex][1];
370
+            $catId = $this->aipdbApiCategories[ $catIndex ][ 1 ];
371 371
 
372 372
             // need another ?
373
-            if ($needAnother !== false){
373
+            if ($needAnother !== false) {
374 374
 
375 375
                 // is a standalone cat ?
376
-                if ($this->aipdbApiCategories[$catIndex][3] === false) {
376
+                if ($this->aipdbApiCategories[ $catIndex ][ 3 ] === false) {
377 377
                     $needAnother = true;
378 378
 
379 379
                 } else {
@@ -384,10 +384,10 @@  discard block
 block discarded – undo
384 384
             }
385 385
 
386 386
             // set or add to cats list 
387
-            $catsString = ($catsString === '') ? $catId : $catsString .','.$catId;
387
+            $catsString = ($catsString === '') ? $catId : $catsString . ',' . $catId;
388 388
         }
389 389
 
390
-        if ($needAnother !== false){
390
+        if ($needAnother !== false) {
391 391
             throw new \InvalidArgumentException('Invalid report category paremeter given: some categories can\'t be used alone');
392 392
         }
393 393
 
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
         $message = str_replace('\\', '', $message);
460 460
 
461 461
         // Remove self ips
462
-        foreach ($this->selfIps as $ip){
462
+        foreach ($this->selfIps as $ip) {
463 463
             $message = str_replace($ip, '*', $message);
464 464
         }
465 465
 
@@ -486,7 +486,7 @@  discard block
 block discarded – undo
486 486
     protected static function loadJsonFile(string $filePath, bool $throwError = true)
487 487
     {
488 488
         // check file exists
489
-        if (!file_exists($filePath) || !is_file($filePath)){
489
+        if (!file_exists($filePath) || !is_file($filePath)) {
490 490
            if ($throwError) {
491 491
                 throw new \Exception('Config file not found');
492 492
            }
@@ -498,7 +498,7 @@  discard block
 block discarded – undo
498 498
         $json    = json_decode(utf8_encode($content));
499 499
 
500 500
         // check for errors
501
-        if ($json == null && json_last_error() != JSON_ERROR_NONE){
501
+        if ($json == null && json_last_error() != JSON_ERROR_NONE) {
502 502
             if ($throwError) {
503 503
                 throw new \LogicException(sprintf("Failed to parse config file Error: '%s'", json_last_error_msg()));
504 504
             }
Please login to merge, or discard this patch.