kristuff / abuseipdb-cli

src/ShellUtils.php

Spacing +22 added lines, -22 removed lines

@@ -37,17 +37,17 @@ 
     /**
      * @var string      
      */
-    const OUTPUT_JSON       = 'json';
+    const OUTPUT_JSON = 'json';
 
     /**
      * @var string      
      */
-    const OUTPUT_DEFAULT    = 'default';
+    const OUTPUT_DEFAULT = 'default';
 
     /**
      * @var string      
      */
-    const OUTPUT_PLAINTEXT  = 'plaintext';
+    const OUTPUT_PLAINTEXT = 'plaintext';
     
     /**
      * @var string      $outputFormat
@@ -92,7 +92,7 @@ 
     protected static function printTempMessage(): void
     {
         if (self::isDefaultOuput()) {
-            Console::reLog(Console::text('   ? ', 'green') . Console::text('waiting for api response', 'white') . Console::text(' ... ', 'green'));
+            Console::reLog(Console::text('   ? ', 'green').Console::text('waiting for api response', 'white').Console::text(' ... ', 'green'));
         }
     }
 
@@ -146,11 +146,11 @@ 
         self::printLogo();
 
         Console::log();
-        Console::log(Console::text('  Kristuff/AbuseIPDB Client version: ', 'darkgray') . Console::text(AbuseIPDBClient::VERSION, 'lightgray'));
-        Console::log(Console::text('  Kristuff/AbuseIPDB Core version:   ', 'darkgray') . Console::text(ApiHandler::VERSION, 'lightgray')); 
+        Console::log(Console::text('  Kristuff/AbuseIPDB Client version: ', 'darkgray').Console::text(AbuseIPDBClient::VERSION, 'lightgray'));
+        Console::log(Console::text('  Kristuff/AbuseIPDB Core version:   ', 'darkgray').Console::text(ApiHandler::VERSION, 'lightgray')); 
         Console::log(Console::text('  --------------------------------------------------', 'darkgray'));    
         Console::log(Console::text('  Released under the MIT licence', 'darkgray'));
-        Console::log(Console::text('  Made with ', 'darkgray') . Console::text('♥', 'red') . Console::text(' in France', 'darkgray'));
+        Console::log(Console::text('  Made with ', 'darkgray').Console::text('♥', 'red').Console::text(' in France', 'darkgray'));
         Console::log(
             Console::text('  © 2020-2021 Kristuff (', 'darkgray').
             Console::text('https://github.com/kristuff', 'darkgray', 'underlined').
@@ -172,8 +172,8 @@ 
     {
         if (self::isDefaultOuput()) {
             Console::log();    
-            Console::log( Console::text(' Kristuff/AbuseIPDB-client ', 'darkgray') . Console::text(' ' . AbuseIPDBClient::VERSION . ' ', 'white', 'blue')); 
-            Console::log(Console::text(' Made with ', 'darkgray') . Console::text('♥', 'red') . Console::text(' in France', 'darkgray')); 
+            Console::log(Console::text(' Kristuff/AbuseIPDB-client ', 'darkgray').Console::text(' '.AbuseIPDBClient::VERSION.' ', 'white', 'blue')); 
+            Console::log(Console::text(' Made with ', 'darkgray').Console::text('♥', 'red').Console::text(' in France', 'darkgray')); 
             Console::log(Console::text(' © 2020-2021 Kristuff (', 'darkgray').
                 Console::text('https://github.com/kristuff', 'darkgray', 'underlined').
                 Console::text(')', 'darkgray')
@@ -193,19 +193,19 @@ 
     protected static function printFooter($requestTime = ''): void
     {
         if (self::isDefaultOuput()) {
-            if (!empty($requestTime)){
+            if (!empty($requestTime)) {
                 $date_utc = new \DateTime("now", new \DateTimeZone("UTC"));
                 Console::log(
-                    Console::text('  Request time: ', 'darkgray') . Console::text($requestTime . 's', 'lightgray'). 
-                    Console::text(' | UTC time: ', 'darkgray') . Console::text($date_utc->format('Y-m-d H:i:s'), 'lightgray')
+                    Console::text('  Request time: ', 'darkgray').Console::text($requestTime.'s', 'lightgray'). 
+                    Console::text(' | UTC time: ', 'darkgray').Console::text($date_utc->format('Y-m-d H:i:s'), 'lightgray')
                 );
             }
             Console::log(Console::text('  ------------------------------------------------------------------------------------------------------', 'darkgray')); 
             Console::log(
-                Console::text('  Kristuff\AbuseIPDB ', 'darkgray') . 
-                Console::text(AbuseIPDBClient::VERSION, 'lightgray') . 
-                Console::text(' | Made with ', 'darkgray') . 
-                Console::text('♥', 'red') .
+                Console::text('  Kristuff\AbuseIPDB ', 'darkgray'). 
+                Console::text(AbuseIPDBClient::VERSION, 'lightgray'). 
+                Console::text(' | Made with ', 'darkgray'). 
+                Console::text('♥', 'red').
                 Console::text(' in France | © 2020-2021 Kristuff (https://github.com/kristuff)', 'darkgray')
             ); 
             Console::log(); 
@@ -223,9 +223,9 @@ 
     protected static function printResult($text, $value, string $foregroundColor = 'lightred', string $backgroundColor = '', bool $print = true): string
     {
         // do not print null/blank values
-        if (isset($value)){
-            $line = Console::text($text, 'white') . Console::text($value, $foregroundColor, $backgroundColor); 
-            if ($print && self::isDefaultOuput()){
+        if (isset($value)) {
+            $line = Console::text($text, 'white').Console::text($value, $foregroundColor, $backgroundColor); 
+            if ($print && self::isDefaultOuput()) {
                 Console::log($line);
             }
             return $line;
@@ -249,16 +249,16 @@ 
         $scoreforegroundColor = 'white';
         $scoreBackgroundColor = 'green';
 
-        if (intval($score) > 0 ){
+        if (intval($score) > 0) {
             $scoreforegroundColor = 'black';
             $scoreBackgroundColor = 'yellow';
         } 
-        if (intval($score) > 50 ){
+        if (intval($score) > 50) {
             $scoreforegroundColor = 'white';
             $scoreBackgroundColor = 'red';
         } 
   
-        $badge = str_pad(strval($score), 3, ' ',STR_PAD_LEFT); 
+        $badge = str_pad(strval($score), 3, ' ', STR_PAD_LEFT); 
         return Console::text($padding.$badge.$padding, $scoreforegroundColor, $scoreBackgroundColor);
     }
 }
\ No newline at end of file

src/AbuseIPDBClient.php

Spacing +113 added lines, -114 removed lines

@@ -48,7 +48,7 @@ 
     {
         // required at least one valid argument
         self::$keyPath = $keyPath; 
-        self::validate( !empty($arguments), 'No valid arguments given. Run abuseipdb --help to get help.');
+        self::validate(!empty($arguments), 'No valid arguments given. Run abuseipdb --help to get help.');
         if (!self::parseCommand($arguments, $keyPath)) {
             self::error('Invalid arguments. Run abuseipdb --help to get help.');
             self::printFooter();
@@ -69,9 +69,9 @@ 
     {
         self::printTitle(Console::text('  ► Register API key ', 'darkgray'));
         
-        $key = self::getArgumentValue($arguments,'S', 'save-key');
+        $key = self::getArgumentValue($arguments, 'S', 'save-key');
         
-        if (empty($key)){
+        if (empty($key)) {
             self::error('Null or invalid key argument.');
             self::printFooter();
             Program::exit(1);
@@ -79,12 +79,12 @@ 
 
         $data = json_encode(['api_key' => $key]);
        
-        if (file_put_contents(self::$keyPath, $data, LOCK_EX) === false){
+        if (file_put_contents(self::$keyPath, $data, LOCK_EX) === false) {
             self::error('An error occurred when writing config file. Make sure to give the appropriate permissions to the config directory.');
             self::printFooter();
             Program::exit(1);
         }
-        Console::log(Console::text('  ✓ ', 'green') . Console::text('Your config key file has been successfully created.', 'white'));
+        Console::log(Console::text('  ✓ ', 'green').Console::text('Your config key file has been successfully created.', 'white'));
         Console::log();   
         self::printFooter();
         Program::exit();
@@ -102,61 +102,61 @@ 
     {
         self::printBanner();
 
-        Console::log(' ' . Console::text('SYNOPSIS:', 'white', 'underline')); 
-        Console::log(' ' . Console::text('    abuseipdb -C ') . 
-                           Console::text('IP', 'yellow') . 
-                           Console::text(' [-d ') . 
-                           Console::text('DAYS', 'yellow') . 
-                           Console::text('] [-v] [-l ') . 
-                           Console::text('LIMIT', 'yellow') . 
-                           Console::text('] [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+        Console::log(' '.Console::text('SYNOPSIS:', 'white', 'underline')); 
+        Console::log(' '.Console::text('    abuseipdb -C '). 
+                           Console::text('IP', 'yellow'). 
+                           Console::text(' [-d '). 
+                           Console::text('DAYS', 'yellow'). 
+                           Console::text('] [-v] [-l '). 
+                           Console::text('LIMIT', 'yellow'). 
+                           Console::text('] [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
 
-        Console::log(' ' . Console::text('    abuseipdb -K ') . 
-                           Console::text('NETWORK', 'yellow') . 
-                           Console::text(' [-d ') . 
-                           Console::text('DAYS', 'yellow') . 
-                           Console::text('] [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+        Console::log(' '.Console::text('    abuseipdb -K '). 
+                           Console::text('NETWORK', 'yellow'). 
+                           Console::text(' [-d '). 
+                           Console::text('DAYS', 'yellow'). 
+                           Console::text('] [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
 
-        Console::log(' ' . Console::text('    abuseipdb -R ') .
-                           Console::text('IP', 'yellow') . ' -c ' .
-                           Console::text('CATEGORIES', 'yellow') . ' -m ' .
-                           Console::text('MESSAGE', 'yellow') .
-                           Console::text(' [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+        Console::log(' '.Console::text('    abuseipdb -R ').
+                           Console::text('IP', 'yellow').' -c '.
+                           Console::text('CATEGORIES', 'yellow').' -m '.
+                           Console::text('MESSAGE', 'yellow').
+                           Console::text(' [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
 
-        Console::log(' ' . Console::text('    abuseipdb -V ') .
-                           Console::text('FILE', 'yellow') .
-                           Console::text(' [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+        Console::log(' '.Console::text('    abuseipdb -V ').
+                           Console::text('FILE', 'yellow').
+                           Console::text(' [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
 
-        Console::log(' ' . Console::text('    abuseipdb -E ') .
+        Console::log(' '.Console::text('    abuseipdb -E ').
                            Console::text('IP', 'yellow').
-                           Console::text(' [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+                           Console::text(' [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
                            
-        Console::log(' ' . Console::text('    abuseipdb -B ') . 
-                           Console::text('[-l ') . 
-                           Console::text('LIMIT', 'yellow') . 
-                           Console::text('] [-s ') . 
-                           Console::text('SCORE', 'yellow') . 
-                           Console::text('] [-o ') . 
-                           Console::text('FORMAT', 'yellow') . 
+        Console::log(' '.Console::text('    abuseipdb -B '). 
+                           Console::text('[-l '). 
+                           Console::text('LIMIT', 'yellow'). 
+                           Console::text('] [-s '). 
+                           Console::text('SCORE', 'yellow'). 
+                           Console::text('] [-o '). 
+                           Console::text('FORMAT', 'yellow'). 
                            Console::text(']')); 
 
-        Console::log(' ' . Console::text('    abuseipdb -S ' .
+        Console::log(' '.Console::text('    abuseipdb -S '.
                            Console::text('KEY', 'yellow')));
 
-        Console::log(' ' . Console::text('    abuseipdb -L | -G | -h | --version'));
+        Console::log(' '.Console::text('    abuseipdb -L | -G | -h | --version'));
                            
         Console::log();    
-        Console::log(' ' . Console::text('OPTIONS:', 'white', 'underline')); 
+        Console::log(' '.Console::text('OPTIONS:', 'white', 'underline')); 
         Console::log();
         Console::log(Console::text('   -h, --help', 'white')); 
         Console::log('       Prints the current help. If given, all next arguments are ignored.', 'lightgray');
@@ -167,46 +167,46 @@ 
         Console::log(Console::text('   -L, --list', 'white')); 
         Console::log('       Prints the list report categories. If given, all next arguments are ignored.', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -C, --check ', 'white') . Console::text('IP', 'yellow', 'underline')); 
+        Console::log(Console::text('   -C, --check ', 'white').Console::text('IP', 'yellow', 'underline')); 
         Console::log('       Performs a check request for the given IP address. A valid IPv4 or IPv6 address is required.', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -K, --check-block ', 'white') . Console::text('NETWORK', 'yellow', 'underline')); 
+        Console::log(Console::text('   -K, --check-block ', 'white').Console::text('NETWORK', 'yellow', 'underline')); 
         Console::log('       Performs a check-block request for the given network. A valid subnet (v4 or v6) denoted with ', 'lightgray');
         Console::log('       CIDR notation is required.', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -d, --days ', 'white') . Console::text('DAYS', 'yellow', 'underline')); 
+        Console::log(Console::text('   -d, --days ', 'white').Console::text('DAYS', 'yellow', 'underline')); 
         Console::log('       For a check or check-block request, defines the maxAgeDays. Min is 1, max is 365, default is 30.', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -R, --report ', 'white') . Console::text('IP', 'yellow', 'underline')); 
+        Console::log(Console::text('   -R, --report ', 'white').Console::text('IP', 'yellow', 'underline')); 
         Console::log('       Performs a report request for the given IP address. A valid IPv4 or IPv6 address is required.', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -V, --bulk-report ', 'white') . Console::text('FILE', 'yellow', 'underline')); 
+        Console::log(Console::text('   -V, --bulk-report ', 'white').Console::text('FILE', 'yellow', 'underline')); 
         Console::log('       Performs a bulk-report request sending a csv file. A valid file name or full path is required.', 'lightgray');
         Console::log();    
         Console::log(Console::text('   -E, --clear ', 'white')); 
         Console::log('       Remove own reports for the given IP address. A valid IPv4 or IPv6 address is required.', 'lightgray');
         Console::log();
-        Console::log(Console::text('   -c, --categories ', 'white') . Console::text('CATEGORIES', 'yellow', 'underline')); 
+        Console::log(Console::text('   -c, --categories ', 'white').Console::text('CATEGORIES', 'yellow', 'underline')); 
         Console::log('       For a report request, defines the report category(ies). Categories must be separate by a comma.', 'lightgray');
         Console::log('       Some categories cannot be used alone. A category can be represented by its shortname or by its', 'lightgray');
-        Console::log(Console::text('       id. Use ','lightgray')  . Console::text('abuseipdb -L', 'white') . Console::text(' to print the categories list.','lightgray'));
+        Console::log(Console::text('       id. Use ', 'lightgray').Console::text('abuseipdb -L', 'white').Console::text(' to print the categories list.', 'lightgray'));
         Console::log();    
-        Console::log(Console::text('   -m, --message ', 'white') . Console::text('MESSAGE', 'yellow', 'underline')); 
+        Console::log(Console::text('   -m, --message ', 'white').Console::text('MESSAGE', 'yellow', 'underline')); 
         Console::log('       For a report request, defines the message to send with report. Message is required for all', 'lightgray');
         Console::log('       report requests.', 'lightgray');
         Console::log();
         Console::log(Console::text('   -B, --blacklist ', 'white')); 
         Console::log('       Performs a blacklist request. Default limit is 1000. This limit can ne changed with the', 'lightgray');
-        Console::log('       ' . Console::text('--limit', 'white') . Console::text(' parameter. ', 'lightgray'));
+        Console::log('       '.Console::text('--limit', 'white').Console::text(' parameter. ', 'lightgray'));
         Console::log();    
-        Console::log(Console::text('   -l, --limit ', 'white') . Console::text('LIMIT', 'yellow', 'underline')); 
+        Console::log(Console::text('   -l, --limit ', 'white').Console::text('LIMIT', 'yellow', 'underline')); 
         Console::log('       For a blacklist request, defines the limit.', 'lightgray');
         Console::log('       For a check request with verbose flag, sets the max number of last reports displayed. Default is 10', 'lightgray');
         Console::log('       For a check-block request, sets the max number of IPs displayed. Default is 0 (no limit).', 'lightgray');
         Console::log();    
-        Console::log(Console::text('   -o, --output ', 'white') . Console::text('FORMAT', 'yellow', 'underline')); 
+        Console::log(Console::text('   -o, --output ', 'white').Console::text('FORMAT', 'yellow', 'underline')); 
         Console::log('       Defines the output format for API requests. Default is a colorized report, possible formats are', 'lightgray');
-        Console::log('       '. Console::text('json', 'yellow', 'underline') . ' or ' . Console::text('plaintext', 'yellow', 'underline') . '. Plaintext option prints partial response (blacklist: IPs list, ');
+        Console::log('       '.Console::text('json', 'yellow', 'underline').' or '.Console::text('plaintext', 'yellow', 'underline').'. Plaintext option prints partial response (blacklist: IPs list, ');
         Console::log('       check or report: confidence score only, check-block: reported IPs list with confidence score, ', 'lightgray');
         Console::log('       bulk-report: number of saved reports, clear: number of deleted reports).', 'lightgray');
         Console::log();    
@@ -217,12 +217,12 @@ 
         Console::log(Console::text('   -v, --verbose ', 'white')); 
         Console::log('       For a check request, display additional fields like the x last reports. This increases ', 'lightgray');
         Console::log(Console::text('       request time and response size. Max number of last reports displayed can be changed with the ', 'lightgray'));
-        Console::log('       ' . Console::text('--limit', 'white') . Console::text(' parameter. ', 'lightgray'));
+        Console::log('       '.Console::text('--limit', 'white').Console::text(' parameter. ', 'lightgray'));
         Console::log();    
         Console::log(Console::text('   --version', 'white')); 
         Console::log('       Prints the current version. If given, all next arguments are ignored.', 'lightgray');
         Console::log(); 
-        Console::log(Console::text('   -S, --save-key ', 'white') . Console::text('KEY', 'yellow', 'underline')); 
+        Console::log(Console::text('   -S, --save-key ', 'white').Console::text('KEY', 'yellow', 'underline')); 
         Console::log('       Save the given API key in the configuration file. Requires writing permissions on the config directory. ', 'lightgray');
         Console::log(); 
     }
@@ -241,11 +241,11 @@ 
 
         self::printTitle(Console::text('  ► Current configuration ', 'darkgray'));
         
-        Console::log(Console::text('  api_key:[', 'white') . Console::text($conf['apiKey'], 'green') . Console::text(']', 'white'));
+        Console::log(Console::text('  api_key:[', 'white').Console::text($conf['apiKey'], 'green').Console::text(']', 'white'));
         Console::log(Console::text('  self_ips:', 'white'));
         
         foreach ($conf['selfIps'] as $ip) {
-            Console::log(Console::text('    [', 'white') . Console::text($ip, 'green') . Console::text(']', 'white'));   
+            Console::log(Console::text('    [', 'white').Console::text($ip, 'green').Console::text(']', 'white'));   
         }
 
         Console::log();   
@@ -266,10 +266,10 @@ 
 
         $categories = ApiHandler::getCategories();
         $rowHeaders = [
-            Console::text('ShortName',      'darkgray') => 15, 
-            Console::text('Id',             'darkgray') => 2, 
-            Console::text('Full name',      'darkgray') => 18,
-            Console::text('Can be alone?',  'darkgray') => 15
+            Console::text('ShortName', 'darkgray') => 15, 
+            Console::text('Id', 'darkgray') => 2, 
+            Console::text('Full name', 'darkgray') => 18,
+            Console::text('Can be alone?', 'darkgray') => 15
         ];
         Console::$verticalSeparator = '  ';
         Console::$verticalInnerSeparator = '  ';
@@ -279,17 +279,16 @@ 
         
         foreach ($categories as $cat) {
             $id = Console::text($cat[1], 'white');
-            $standalone = $cat[3] ? Console::text('✓', 'green') . Console::text(' true ', 'lightgray') : 
-                                    Console::text('✗', 'red')   . Console::text(' false', 'darkgray');
-            $shortName =  Console::text($cat[0], 'white');
-            $fullName =   Console::text($cat[2], 'lightgray');
+            $standalone = $cat[3] ? Console::text('✓', 'green').Console::text(' true ', 'lightgray') : Console::text('✗', 'red').Console::text(' false', 'darkgray');
+            $shortName = Console::text($cat[0], 'white');
+            $fullName = Console::text($cat[2], 'lightgray');
 
             Console::log(
                 Console::TableRowStart().  
-                Console::TableRowCell( $shortName , 15).  
-                Console::TableRowCell( $id , 2, Console::ALIGN_CENTER).  
-                Console::TableRowCell( $fullName , 18).  
-                Console::TableRowCell( $standalone , 15,  Console::ALIGN_CENTER)  
+                Console::TableRowCell($shortName, 15).  
+                Console::TableRowCell($id, 2, Console::ALIGN_CENTER).  
+                Console::TableRowCell($fullName, 18).  
+                Console::TableRowCell($standalone, 15, Console::ALIGN_CENTER)  
             );
         }
         //Console::log(Console::tableRowSeparator($rowHeaders), 'darkgray');
@@ -308,11 +307,11 @@ 
      */
     protected static function reportIP(array $arguments): void
     {
-        $ip      = self::getArgumentValue($arguments,'R', 'report');
-        $cats    = self::getArgumentValue($arguments,'c', 'categories');
-        $message = self::getArgumentValue($arguments,'m', 'message');
+        $ip      = self::getArgumentValue($arguments, 'R', 'report');
+        $cats    = self::getArgumentValue($arguments, 'c', 'categories');
+        $message = self::getArgumentValue($arguments, 'm', 'message');
         
-        self::printTitle(Console::text('  ► Report IP: ', 'darkgray') . Console::text(escapeshellcmd($ip), 'white'));
+        self::printTitle(Console::text('  ► Report IP: ', 'darkgray').Console::text(escapeshellcmd($ip), 'white'));
         self::printTempMessage();
 
         // Peforms request 
@@ -323,7 +322,7 @@ 
         self::clearTempMessage();
         
         // check for errors / empty response
-        if (self::hasErrors($report)){
+        if (self::hasErrors($report)) {
             self::printFooter();
             Program::exit(1);
         }
@@ -332,14 +331,14 @@ 
         $score = empty($report->data->abuseConfidenceScore) ? 0 : $report->data->abuseConfidenceScore;
         $scoreColor = self::getScoreColor($score);
 
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($report, JSON_PRETTY_PRINT);
                 break;
        
             case self::OUTPUT_DEFAULT:  
                 Console::log(
-                    Console::text('   ✓', 'green').Console::text(' IP: [', 'white') .
+                    Console::text('   ✓', 'green').Console::text(' IP: [', 'white').
                     Console::text($ip, $scoreColor).Console::text('] successfully reported', 'white')
                 );
                 Console::log(Console::text('     Confidence score: ', 'white').self::getScoreBadge($score));
@@ -365,26 +364,26 @@ 
      */
     protected static function bulkReport(array $arguments): void
     {
-        $fileName = self::getArgumentValue($arguments,'V', 'bulk-report');
+        $fileName = self::getArgumentValue($arguments, 'V', 'bulk-report');
 
-        self::printTitle(Console::text('  ► Bulk report for file: ', 'darkgray') . Console::text(escapeshellcmd($fileName), 'white'));
+        self::printTitle(Console::text('  ► Bulk report for file: ', 'darkgray').Console::text(escapeshellcmd($fileName), 'white'));
         self::printTempMessage();
 
         // Peforms request 
         $timeStart = microtime(true);  
         $response = self::$api->bulkReport($fileName)->getObject();     
         $timeEnd = microtime(true);      
-        $time = $timeEnd - $timeStart;  // request time
+        $time = $timeEnd - $timeStart; // request time
         self::clearTempMessage();
 
         // check for errors / empty response
-        if (self::hasErrors($response)){
+        if (self::hasErrors($response)) {
             self::printFooter();
             Program::exit(1);
         }
 
         // ✓ Done
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($response, JSON_PRETTY_PRINT);
                 break;
@@ -399,7 +398,7 @@ 
 
             case self::OUTPUT_PLAINTEXT:
                 $nbSavedReports = isset($response->data->savedReports) ? $response->data->savedReports : 0;
-                echo $nbSavedReports . PHP_EOL;
+                echo $nbSavedReports.PHP_EOL;
                 break;
 
         }
@@ -416,8 +415,8 @@ 
      */
     protected static function clearIP(array $arguments): void
     {
-        $ip = self::getArgumentValue($arguments,'E', 'clear');
-        self::printTitle(Console::text('  ► Clear reports for IP: ', 'darkgray') . Console::text(escapeshellcmd($ip), 'white'));
+        $ip = self::getArgumentValue($arguments, 'E', 'clear');
+        self::printTitle(Console::text('  ► Clear reports for IP: ', 'darkgray').Console::text(escapeshellcmd($ip), 'white'));
 
         // Peforms request 
         self::printTempMessage();
@@ -428,22 +427,22 @@ 
         self::clearTempMessage();
 
         // check for errors / empty response
-        if (self::hasErrors($response)){
+        if (self::hasErrors($response)) {
             self::printFooter($time);
             Program::exit(1);
         }
         
         // ✓ Done: print deleted report number 
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($response, JSON_PRETTY_PRINT);
                 break;
        
             case self::OUTPUT_DEFAULT:  
                 Console::log(
-                    Console::text('   ✓', 'green') . 
-                    Console::text(' Successfull clear request for IP: [', 'white') .
-                    Console::text($ip, 'lightyellow') .
+                    Console::text('   ✓', 'green'). 
+                    Console::text(' Successfull clear request for IP: [', 'white').
+                    Console::text($ip, 'lightyellow').
                     Console::text(']', 'white')
                 );
                 self::printResult('     Deleted reports: ', $response->data->numReportsDeleted ?? 0, 'lightyellow');
@@ -452,7 +451,7 @@ 
                 break;
 
             case self::OUTPUT_PLAINTEXT:
-                echo ($response->data->numReportsDeleted ?? 0) . PHP_EOL;
+                echo ($response->data->numReportsDeleted ?? 0).PHP_EOL;
                 break;
 
         }
@@ -472,8 +471,8 @@ 
         self::printTitle(Console::text('  ► Get Blacklist ', 'darkgray'));
 
         $plainText  = (self::$outputFormat === self::OUTPUT_PLAINTEXT); 
-        $limit      = self::getNumericParameter($arguments,'l', 'limit', 1000);
-        $scoreMin   = self::getNumericParameter($arguments,'s', 'score', 100);
+        $limit      = self::getNumericParameter($arguments, 'l', 'limit', 1000);
+        $scoreMin   = self::getNumericParameter($arguments, 's', 'score', 100);
         
         self::printTempMessage();
         
@@ -487,13 +486,13 @@ 
     
         // response could be json on error, while plaintext flag is set
         $decodedResponse = $response->getObject();
-        if (self::hasErrors($decodedResponse, false)){
+        if (self::hasErrors($decodedResponse, false)) {
             self::printFooter($time);
             Program::exit(1);
         }
 
         // ✓ Done: print deleted report number 
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($decodedResponse, JSON_PRETTY_PRINT);
                 break;
@@ -502,7 +501,7 @@ 
                  // print list
                 self::printResult('  List generated at: ', self::getDate($decodedResponse->meta->generatedAt), 'lightyellow', '');
                 Console::log();
-                foreach ($decodedResponse->data as $report){
+                foreach ($decodedResponse->data as $report) {
                     $score = empty($report->abuseConfidenceScore) ? 0 : $report->abuseConfidenceScore;
                     $defaultColor = self::getScoreColor($score);
 
@@ -537,12 +536,12 @@ 
      */
     protected static function checkBlock(array $arguments): void
     {
-        $network  = self::getArgumentValue($arguments,'K', 'check-block');
+        $network = self::getArgumentValue($arguments, 'K', 'check-block');
 
-        self::printTitle(Console::text('  ► Check network: ', 'darkgray') . Console::text(escapeshellcmd($network), 'white') . Console::text('', 'darkgray'));
+        self::printTitle(Console::text('  ► Check network: ', 'darkgray').Console::text(escapeshellcmd($network), 'white').Console::text('', 'darkgray'));
 
         $maxAge   = self::getNumericParameter($arguments, 'd', 'days', 30);
-        $limit    = self::getNumericParameter($arguments,'l', 'limit', 0); // 0 mean no limit
+        $limit    = self::getNumericParameter($arguments, 'l', 'limit', 0); // 0 mean no limit
 
         self::printTempMessage();
 
@@ -553,19 +552,19 @@ 
         self::clearTempMessage();
 
         // check for errors / empty response
-        if (self::hasErrors($check)){
+        if (self::hasErrors($check)) {
             self::printFooter($time);
             Program::exit(1);
         }
 
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($check, JSON_PRETTY_PRINT);
                 break;
        
             case self::OUTPUT_DEFAULT:  
                 self::printCheckBlockDetail($check);
-                self::printCheckBlockReportedIP($check,$maxAge,$limit);
+                self::printCheckBlockReportedIP($check, $maxAge, $limit);
                 Console::log();
                 self::printFooter($time);
                 break;
@@ -574,8 +573,8 @@ 
                 $nbReports = isset($check->data->reportedAddress) ? count($check->data->reportedAddress) : 0;
                 if ($nbReports > 0) {
                     $numberDiplayedReports = 0;
-                    foreach ($check->data->reportedAddress as $report){
-                        echo ($report->ipAddress) . ' ' . $report->abuseConfidenceScore . PHP_EOL;
+                    foreach ($check->data->reportedAddress as $report) {
+                        echo ($report->ipAddress).' '.$report->abuseConfidenceScore.PHP_EOL;
 
                         // counter
                         $numberDiplayedReports++;
@@ -599,14 +598,14 @@ 
      */
     protected static function checkIP(array $arguments): void
     {
-        $ip = self::getArgumentValue($arguments,'C', 'check');
+        $ip = self::getArgumentValue($arguments, 'C', 'check');
         
-        self::printTitle(Console::text('  ► Check IP: ', 'darkgray') . Console::text(escapeshellcmd($ip), 'white') . Console::text('', 'darkgray'));
+        self::printTitle(Console::text('  ► Check IP: ', 'darkgray').Console::text(escapeshellcmd($ip), 'white').Console::text('', 'darkgray'));
         
-        $verbose            = self::inArguments($arguments,'v', 'verbose');
+        $verbose            = self::inArguments($arguments, 'v', 'verbose');
         $maxAge             = self::getNumericParameter($arguments, 'd', 'days', 30);
-        $maxReportsNumber   = self::getNumericParameter($arguments,'l', 'limit', 10);
-        $ip                 = self::getArgumentValue($arguments,'C', 'check');
+        $maxReportsNumber   = self::getNumericParameter($arguments, 'l', 'limit', 10);
+        $ip                 = self::getArgumentValue($arguments, 'C', 'check');
 
         self::printTempMessage();
         $timeStart = microtime(true);                                           
@@ -616,7 +615,7 @@ 
         self::clearTempMessage();
 
         // check for errors / empty response
-        if (self::hasErrors($check)){
+        if (self::hasErrors($check)) {
             self::printFooter($time);
             Program::exit(1);
         }
@@ -624,7 +623,7 @@ 
         // score and data color (depending of abuseConfidenceScore)
         $score = empty($check->data->abuseConfidenceScore) ? 0 : $check->data->abuseConfidenceScore;
 
-        switch (self::$outputFormat){
+        switch (self::$outputFormat) {
             case self::OUTPUT_JSON:
                 echo json_encode($check, JSON_PRETTY_PRINT);
                 break;
@@ -634,7 +633,7 @@ 
                 self::printCheckScore($check);
                 self::printCheckDetail($check, $defaultColor);
                 self::printCheckReports($check, $maxAge, $defaultColor);
-                if ($verbose){
+                if ($verbose) {
                     self::printCheckLastReports($check, $maxReportsNumber);
                 }
                 Console::log();
@@ -642,7 +641,7 @@ 
                 break;
 
             case self::OUTPUT_PLAINTEXT:
-                echo ($check->data->abuseConfidenceScore ?? 0) . PHP_EOL;
+                echo ($check->data->abuseConfidenceScore ?? 0).PHP_EOL;
                 break;
 
         }

src/ShellErrorHandler.php

Spacing +21 added lines, -21 removed lines

@@ -56,8 +56,8 @@ 
      */
     private static function parseErrors(?object $response = null): bool
     {
-        if (isset($response) && isset($response->errors)){
-            switch (self::$outputFormat){
+        if (isset($response) && isset($response->errors)) {
+            switch (self::$outputFormat) {
                 case self::OUTPUT_DEFAULT:
                     self::printFormattedErrors($response);
                     break;
@@ -86,24 +86,24 @@ 
     protected static function printFormattedErrors(object $response): void
     {
         // top error badge    
-        Console::log('  ' .   Console::text(' ERROR ','white', 'red'));
+        Console::log('  '.Console::text(' ERROR ', 'white', 'red'));
 
         $num = 0;
         // errors is an array, could have more than one error..
-        foreach ($response->errors as $err){
+        foreach ($response->errors as $err) {
             $num++;
 
-            Console::log(Console::text('   ✗', 'red') .  self::printResult(' Number:    ', $num, 'lightyellow','', false));
-            self::printResult('     Status:    ', $err->status ?? null, 'lightyellow','');    
+            Console::log(Console::text('   ✗', 'red').self::printResult(' Number:    ', $num, 'lightyellow', '', false));
+            self::printResult('     Status:    ', $err->status ?? null, 'lightyellow', '');    
             
-            if (!empty($err->source) && !empty($err->source->parameter)){
+            if (!empty($err->source) && !empty($err->source->parameter)) {
                 self::printResult('     Parameter: ', $err->source->parameter, 'lightyellow');    
             }
             self::printResult('     Title:     ', $err->title ?? null, 'lightyellow');    
             self::printResult('     Detail:    ', $err->detail ?? null, 'lightyellow');    
 
             // separate errors
-            if (count($response->errors) > 1){
+            if (count($response->errors) > 1) {
                 Console::log('   ---');
             }
         }
@@ -123,11 +123,11 @@ 
     {
         if (self::isDefaultOuput()) {
             // ✗
-            Console::log('  ' .   Console::text(' ERROR ','white', 'red'));
+            Console::log('  '.Console::text(' ERROR ', 'white', 'red'));
             Console::log(
-                Console::text('   ✗', 'red') . 
-                Console::text(' Detail:    ', 'white') . 
-                Console::text($error, 'lightyellow') . 
+                Console::text('   ✗', 'red'). 
+                Console::text(' Detail:    ', 'white'). 
+                Console::text($error, 'lightyellow'). 
                 Console::text('', 'white')
             );    
             Console::log();
@@ -147,7 +147,7 @@ 
      */
     protected static function validate(bool $condition, string $message, bool $print = true): void
     {
-        if ( !$condition ){
+        if (!$condition) {
             if ($print && self::isDefaultOuput()) {
                 Console::log();
                 self::error($message);
@@ -171,10 +171,10 @@ 
      */
     protected static function getNumericParameter(array $arguments, string $shortArg, string $longArg, int $defaultValue): int
     {
-         if (self::inArguments($arguments,$shortArg, $longArg)){
-            $val = self::getArgumentValue($arguments,$shortArg, $longArg);
+         if (self::inArguments($arguments, $shortArg, $longArg)) {
+            $val = self::getArgumentValue($arguments, $shortArg, $longArg);
 
-            if (!is_numeric($val)){
+            if (!is_numeric($val)) {
                 self::error("Invalid parameter: $longArg must be a numeric value.");
                 self::printFooter();
                 Program::exit(1);
@@ -195,7 +195,7 @@ 
      */
     protected static function printPlainTextErrors(object $response): void
     {
-        foreach ($response->errors as $err){
+        foreach ($response->errors as $err) {
             $text = 'Error: ';
             $text .= self::getErrorDetail($err, 'title');
             $text .= self::getErrorDetail($err, 'statuts');
@@ -219,11 +219,11 @@ 
      */
     private static function getErrorDetail(object $error, string $field, ?string $parent = null): string
     {
-        if (!empty($parent)){
-            return  !empty($error->$parent) && !empty($error->$parent->$field) ? ' ' . $field . ': ' . $error->$parent->$field : '';
+        if (!empty($parent)) {
+            return  !empty($error->$parent) && !empty($error->$parent->$field) ? ' '.$field.': '.$error->$parent->$field : '';
         }
 
-        return !empty($error->$field) ? ' ' . $field . ': ' . $error->$field : '';
+        return !empty($error->$field) ? ' '.$field.': '.$error->$field : '';
     }
 
     /**
@@ -238,7 +238,7 @@ 
     protected static function checkForEmpty(object $response): bool
     {
         // check for empty response ?
-        if ( empty($response) || empty($response->data) ){
+        if (empty($response) || empty($response->data)) {
             self::error('An unexpected error occurred.');
             return true;
         }