terrylinooo / shieldon

src/Firewall/Driver/SqlDriverProvider.php

Spacing +3 added lines, -3 removed lines

@@ -302,7 +302,7 @@ 
 
         // @codeCoverageIgnoreStart
         
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             return false;
         }
 
@@ -361,7 +361,7 @@ 
 
         // @codeCoverageIgnoreStart
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             return false;
         }
 
@@ -414,7 +414,7 @@ 
 
         // @codeCoverageIgnoreStart
 
-        } catch(Exception $e) {
+        } catch (Exception $e) {
             return false;
         }
 

src/Firewall/Log/ActionLogParser.php

Spacing +28 added lines, -28 removed lines

@@ -208,7 +208,7 @@ 
             default:
 
                 // We also accept querying N days data from logs. For example: `past_365_days`.
-                if (preg_match('/past_([0-9]+)_days/', $this->type, $matches) ) {
+                if (preg_match('/past_([0-9]+)_days/', $this->type, $matches)) {
 
                     $dayCount = $matches[1];
                     $startDate = date('Ymd', strtotime('-' . $dayCount . ' days'));
@@ -331,21 +331,21 @@ 
             return [];
         }
 
-        $results['captcha_chart_string']  = '';     // string
-        $results['pageview_chart_string'] = '';     // string
-        $results['captcha_success_count'] = 0;      // integer
-        $results['captcha_failure_count'] = 0;      // integer
-        $results['captcha_count'] = 0;              // integer
-        $results['pageview_count'] = 0;             // integer
-        $results['captcha_percentageage'] = 0;      // integer
+        $results['captcha_chart_string']  = ''; // string
+        $results['pageview_chart_string'] = ''; // string
+        $results['captcha_success_count'] = 0; // integer
+        $results['captcha_failure_count'] = 0; // integer
+        $results['captcha_count'] = 0; // integer
+        $results['pageview_count'] = 0; // integer
+        $results['captcha_percentageage'] = 0; // integer
         $results['captcha_failure_percentage'] = 0; // integer
         $results['captcha_success_percentage'] = 0; // integer
 
-        $results['action_ban_count'] = 0;           // integer
-        $results['action_temp_ban_count'] = 0;      // integer
-        $results['action_unban_count'] = 0;         // integer
-        $results['blacklist_count'] = 0;            // integer
-        $results['session_limit_count'] = 0;        // integer
+        $results['action_ban_count'] = 0; // integer
+        $results['action_temp_ban_count'] = 0; // integer
+        $results['action_unban_count'] = 0; // integer
+        $results['blacklist_count'] = 0; // integer
+        $results['session_limit_count'] = 0; // integer
 
         $ipdData = $this->getIpData();
 
@@ -386,22 +386,22 @@ 
     {
         $periodData = $this->getPeriodData();
 
-        $results['captcha_chart_string']  = '';     // string
-        $results['pageview_chart_string'] = '';     // string
-        $results['label_chart_string'] = '';        // string
-        $results['captcha_success_count'] = 0;      // integer
-        $results['captcha_failure_count'] = 0;      // integer
-        $results['captcha_count'] = 0;              // integer
-        $results['pageview_count'] = 0;             // integer
-        $results['captcha_percentageage'] = 0;      // integer
+        $results['captcha_chart_string']  = ''; // string
+        $results['pageview_chart_string'] = ''; // string
+        $results['label_chart_string'] = ''; // string
+        $results['captcha_success_count'] = 0; // integer
+        $results['captcha_failure_count'] = 0; // integer
+        $results['captcha_count'] = 0; // integer
+        $results['pageview_count'] = 0; // integer
+        $results['captcha_percentageage'] = 0; // integer
         $results['captcha_failure_percentage'] = 0; // integer
         $results['captcha_success_percentage'] = 0; // integer
 
-        $results['action_ban_count'] = 0;           // integer
-        $results['action_temp_ban_count'] = 0;      // integer
-        $results['action_unban_count'] = 0;         // integer
-        $results['blacklist_count'] = 0;            // integer
-        $results['session_limit_count'] = 0;        // integer
+        $results['action_ban_count'] = 0; // integer
+        $results['action_temp_ban_count'] = 0; // integer
+        $results['action_unban_count'] = 0; // integer
+        $results['blacklist_count'] = 0; // integer
+        $results['session_limit_count'] = 0; // integer
 
         if (!empty($periodData)) {
 
@@ -431,7 +431,7 @@ 
             }
 
             $results['captcha_chart_string'] = implode(',', $chartCaptcha);
-            $results['pageview_chart_string']= implode(',', $chartPageview);
+            $results['pageview_chart_string'] = implode(',', $chartPageview);
             $results['captcha_success_chart_string'] = implode(',', $chartCaptchaSuccess);
             $results['captcha_failure_chart_string'] = implode(',', $chartCaptchaFailure);
             $results['label_chart_string'] = "'" . implode("','", $labels) . "'";
@@ -461,7 +461,7 @@ 
         $ip = $log['ip'];
         $sessionId = $log['session_id'];
 
-        $this->ipDetail[$t][$ip]['session_id'][$sessionId ] = 1;
+        $this->ipDetail[$t][$ip]['session_id'][$sessionId] = 1;
 
         if ($logActionCode === self::LOG_TEMPORARILY_BAN) {
             $this->periodDetail[$t][$k]['action_temp_ban_count']++;

src/Firewall/Log/ActionLogger.php

Spacing +3 added lines, -3 removed lines

@@ -100,10 +100,10 @@
             $record['session_id'] = substr($record['session_id'], 0, 4);
         }
 
-        $data[0] = $record['ip']          ?? 'null';
-        $data[1] = $record['session_id']  ?? 'null';
+        $data[0] = $record['ip'] ?? 'null';
+        $data[1] = $record['session_id'] ?? 'null';
         $data[2] = $record['action_code'] ?? 'null';
-        $data[3] = $record['timesamp']    ?? 'null';
+        $data[3] = $record['timesamp'] ?? 'null';
 
         file_put_contents($this->filePath, implode(',', $data) . "\n", FILE_APPEND | LOCK_EX);
     }

templates/panel/table_sessions.php

Spacing +5 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
             </div>
             <div class="board-field right">
                 <div class="heading"><?php _e('panel', 'table_heading_period', 'Period'); ?></div>
-                <div class="nums"><?php echo number_format($session_limit_period ); ?></div>
+                <div class="nums"><?php echo number_format($session_limit_period); ?></div>
                 <div class="note"><?php _e('panel', 'table_note_period', 'Keep-alive period. (minutes)'); ?></div>
             </div>
         </div>
@@ -49,7 +49,7 @@ 
             </div>
             <div class="board-field right">
                 <div class="heading"><?php _e('panel', 'table_heading_online', 'Online'); ?></div>
-                <div class="nums"><?php echo number_format($online_count ); ?></div>
+                <div class="nums"><?php echo number_format($online_count); ?></div>
                 <div class="note"><?php _e('panel', 'table_note_online', 'Online session amount.'); ?></div>
             </div>
         </div>
@@ -89,14 +89,14 @@ 
 
                         $remainsTime = $expires - (time() - $sessionInfo['time']);
 
-                        if ($remainsTime < 1 ) {
+                        if ($remainsTime < 1) {
                             $remainsTime = 0;
                         }
 
-                        if ($i < $session_limit_count ) {
+                        if ($i < $session_limit_count) {
                             $satusName = __('panel', 'table_text_allowable', 'Allowable');
 
-                            if ($remainsTime < 1 ) {
+                            if ($remainsTime < 1) {
                                 $satusName = __('panel', 'table_text_expired', 'Expired');
                             }
                         } else {

templates/panel/action_log_today.php

Spacing +4 added lines, -4 removed lines

@@ -18,7 +18,7 @@ 
 ?>
 
 <div class="so-dashboard">
-    <?php if (! empty($period_data)) : ?>
+    <?php if (!empty($period_data)) : ?>
     <div class="so-flex">
         <div class="so-board">
             <div class="board-field left">
@@ -88,7 +88,7 @@ 
                 </tr>
             </thead>
             <tbody>
-                <?php if (! empty($ip_details)) : ?>
+                <?php if (!empty($ip_details)) : ?>
                 <?php foreach ($ip_details as $ip => $ipInfo) : ?>
                 <tr>
                     <td>
@@ -111,7 +111,7 @@ 
         </table>
     </div>
     <div class="so-timezone">
-        <?php if (! empty($last_cached_time)) : ?>
+        <?php if (!empty($last_cached_time)) : ?>
             <?php _e('panel', 'log_label_cache_time', 'Report generated time'); ?>: <strong class="text-info"><?php echo $last_cached_time; ?></strong>
             &nbsp;&nbsp;&nbsp;&nbsp; 
         <?php endif; ?>
@@ -121,7 +121,7 @@ 
 
 <?php
 
-    if (! empty($period_data)) {
+    if (!empty($period_data)) {
         $data['period_data'] = $period_data;
         $data['past_seven_hours'] = $past_seven_hours;
         $this->_include('panel/js/chart', $data);

templates/panel/ip_manager.php

Spacing +3 added lines, -3 removed lines

@@ -61,7 +61,7 @@ 
         </div>
     </div>
     <br />
-    <?php if ( empty($ip_list)) : ?>
+    <?php if (empty($ip_list)) : ?>
     <div id="so-table-container" class="so-datatables">
         <table id="so-datalog" class="cell-border compact stripe responsive" cellspacing="0" width="100%">
             <tbody>
@@ -96,7 +96,7 @@ 
                 </tr>
             </thead>
             <tbody>
-                <?php if (! empty($ip_list)) : ?>
+                <?php if (!empty($ip_list)) : ?>
                 <?php foreach ($ip_list as $i => $ipInfo) : ?>
                 <tr>
                     <td><?php echo $i + 1; ?></td>
@@ -119,7 +119,7 @@ 
     </div>
 </div>
 
-<?php if (! empty($ip_list)) : ?>
+<?php if (!empty($ip_list)) : ?>
 
 <script>
 

templates/panel/authentication.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
                 </tr>
             </thead>
             <tbody>
-                <?php if (! empty($authentication_list)) : ?>
+                <?php if (!empty($authentication_list)) : ?>
                 <?php foreach ($authentication_list as $i => $authInfo) : ?>
                 <tr>
                     <td><?php echo $authInfo['url']; ?></td>

templates/panel/table_filter_logs.php

Spacing +4 added lines, -4 removed lines

@@ -58,20 +58,20 @@
             </thead>
             <tbody>
                 <?php foreach ($ip_log_list as $ipInfo) : ?>
-                    <?php $logData = is_array($ipInfo['log_data']) ? $ipInfo['log_data'] : json_decode($ipInfo['log_data'], true ); ?>
+                    <?php $logData = is_array($ipInfo['log_data']) ? $ipInfo['log_data'] : json_decode($ipInfo['log_data'], true); ?>
                     <?php
 
                         $text_warning = '';
 
-                        if ($logData['pageviews_m'] > 6 || $logData['pageviews_h'] > 50 || $logData['pageviews_d'] > 100 ) {
+                        if ($logData['pageviews_m'] > 6 || $logData['pageviews_h'] > 50 || $logData['pageviews_d'] > 100) {
                             $text_warning = '<span class="so-text-warning"><i class="fas fa-exclamation-triangle"></i></span>';
                         }
 
-                        if ($logData['flag_js_cookie'] > 2 || $logData['flag_multi_session'] > 2 || $logData['flag_empty_referer'] > 2 ) {
+                        if ($logData['flag_js_cookie'] > 2 || $logData['flag_multi_session'] > 2 || $logData['flag_empty_referer'] > 2) {
                             $text_warning = '<span class="so-text-warning"><i class="fas fa-exclamation-triangle"></i></span>';
                         }
 
-                        if ($logData['flag_js_cookie'] > 3 || $logData['flag_multi_session'] > 3 || $logData['flag_empty_referer'] > 3 ) {
+                        if ($logData['flag_js_cookie'] > 3 || $logData['flag_multi_session'] > 3 || $logData['flag_empty_referer'] > 3) {
                             $text_warning = '<span class="so-text-danger"><i class="fas fa-exclamation-triangle"></i></span>';
                         }
                     ?>

templates/panel/table_rules.php

Spacing +5 added lines, -5 removed lines

@@ -67,7 +67,7 @@ 
         </div>
     </div>
     <?php endif; ?>
-    <?php if (! empty($rule_list)) : ?>
+    <?php if (!empty($rule_list)) : ?>
     <div id="so-table-container" class="so-datatables" style="display: none;">
         <table id="so-datalog" class="cell-border compact stripe responsive" cellspacing="0" width="100%">
             <thead>
@@ -92,15 +92,15 @@ 
                     <td><?php echo $ipInfo['ip_resolve']; ?></td>
                     <td>
                         <?php 
-                            if (! empty($type_mapping[$ipInfo['type'] ]) ) {
-                                echo $type_mapping[$ipInfo['type'] ];
+                            if (!empty($type_mapping[$ipInfo['type']])) {
+                                echo $type_mapping[$ipInfo['type']];
                             }
                         ?>
                     </td>
                     <td>
                         <?php
-                            if (! empty($reason_mapping[$ipInfo['reason'] ]) ) {
-                                echo $reason_mapping[$ipInfo['reason'] ];
+                            if (!empty($reason_mapping[$ipInfo['reason']])) {
+                                echo $reason_mapping[$ipInfo['reason']];
                             }
                         ?>
                     </td>