terrylinooo / shieldon

src/Firewall/Log/ActionLogParser.php

Spacing +28 added lines, -28 removed lines

@@ -197,7 +197,7 @@ 
         ];
 
         if (empty($dataRange[$this->type])) {
-            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'));
                 $endDate = date('Ymd');
@@ -336,21 +336,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();
 
@@ -391,22 +391,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)) {
 
@@ -436,7 +436,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) . "'";
@@ -466,7 +466,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/Kernel.php

Spacing +3 added lines, -3 removed lines

@@ -614,9 +614,9 @@
         $this->initComponents();
 
         $processMethods = [
-            'isRuleExist',   // Stage 1 - Looking for rule table.
-            'isTrustedBot',  // Stage 2 - Detect popular search engine.
-            'isFakeRobot',   // Stage 3 - Reject fake search engine crawlers.
+            'isRuleExist', // Stage 1 - Looking for rule table.
+            'isTrustedBot', // Stage 2 - Detect popular search engine.
+            'isFakeRobot', // Stage 3 - Reject fake search engine crawlers.
             'isIpComponent', // Stage 4 - IP manager.
             'isComponents'   // Stage 5 - Check other components.
         ];

src/Firewall/Panel/BaseController.php

Spacing +3 added lines, -5 removed lines

@@ -169,7 +169,7 @@ 
             define('SHIELDON_VIEW', true);
         }
 
-        $viewFilePath =  __DIR__ . '/../../../templates/' . $page . '.php';
+        $viewFilePath = __DIR__ . '/../../../templates/' . $page . '.php';
     
         if (!empty($data)) {
             extract($data);
@@ -451,12 +451,10 @@ 
     protected function messengerAjaxStatus(string $moduleName, string $echoType = 'css'): void
     {
         $echo['css'] = $this->getConfig('messengers.' . $moduleName . '.confirm_test') ? 
-            'success' :
-            '';
+            'success' : '';
         
         $echo['icon'] = $this->getConfig('messengers.' . $moduleName . '.confirm_test') ?
-            '<i class="fas fa-check"></i>' :
-            '<i class="fas fa-exclamation"></i>';
+            '<i class="fas fa-check"></i>' : '<i class="fas fa-exclamation"></i>';
 
         echo $echo[$echoType];
     }

src/Firewall/Firewall/Captcha/CaptchaFactory.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
         $str = implode(
             '',
             array_map(
-                function ($word) {
+                function($word) {
                     return ucwords($word); 
                 },
                 $str

src/Firewall/Firewall/XssProtectionTrait.php

Spacing +4 added lines, -4 removed lines

@@ -79,7 +79,7 @@ 
 
             $this->kernel->setClosure(
                 'xss_post',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_POST)) {
                         foreach (array_keys($_POST) as $k) {
                             $_POST[$k] = $xss->clean($_POST[$k]);
@@ -104,7 +104,7 @@ 
 
             $this->kernel->setClosure(
                 'xss_get',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_GET)) {
                         foreach (array_keys($_GET) as $k) {
                             $_GET[$k] = $xss->clean($_GET[$k]);
@@ -129,7 +129,7 @@ 
 
             $this->kernel->setClosure(
                 'xss_cookie',
-                function () use ($xss) {
+                function() use ($xss) {
                     if (!empty($_COOKIE)) {
                         foreach (array_keys($_COOKIE) as $k) {
                             $_COOKIE[$k] = $xss->clean($_COOKIE[$k]);
@@ -154,7 +154,7 @@ 
 
             $this->kernel->setClosure(
                 'xss_protection', 
-                function () use ($xss, $protectedList) {
+                function() use ($xss, $protectedList) {
                     foreach ($protectedList as $v) {
                         $k = $v['variable'] ?? 'undefined';
         

templates/frontend/login.php

Spacing +2 added lines, -2 removed lines

@@ -129,7 +129,7 @@ 
                 <div class="card-body">
                     <form action="<?php echo $form ?>" method="post" autocomplete="off">
                         <div class="main-content">
-                            <?php if (! empty($error)) : ?>
+                            <?php if (!empty($error)) : ?>
                             <div class="error-notice">
                                 <?php echo $error; ?>
                             </div>
@@ -140,7 +140,7 @@ 
                             <div class="input-box">
                                 <input type="password" name="s_pass" placeholder="Password" class="form-input" />
                             </div>
-                            <?php if (! empty($captchas)) : ?>
+                            <?php if (!empty($captchas)) : ?>
                             <div class="input-box">
                                 <?php foreach ($captchas as $captcha) : ?>
                                     <?php echo $captcha->form(); ?>

templates/frontend/captcha.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
                             <p><input type="submit" value="<?php _e('core', 'stop_submit', 'Submit') ?>" /></p>
                         </form>
                     </div>
-                    <?php if (! empty($dialoguserinfo)) : ?>
+                    <?php if (!empty($dialoguserinfo)) : ?>
                         <div class="status-user-info">
                             <?php foreach ($dialoguserinfo as $key => $userinfo) : ?>
                                 <div class="row">

src/Firewall/Firewall/MainTrait.php

Spacing +4 added lines, -4 removed lines

@@ -279,7 +279,7 @@ 
 
         foreach ($ipList as $ip) {
 
-            if (0 === strpos($this->kernel->getCurrentUrl(), $ip['url']) ) {
+            if (0 === strpos($this->kernel->getCurrentUrl(), $ip['url'])) {
 
                 if ('allow' === $ip['rule']) {
                     $allowedList[] = $ip['ip'];
@@ -352,7 +352,7 @@ 
         $this->kernel->setProperty(
             'deny_attempt_enable',
             [
-                'data_circle'     => $setting['data_circle']['enable'],     // false   
+                'data_circle'     => $setting['data_circle']['enable'], // false   
                 'system_firewall' => $setting['system_firewall']['enable'], // false   
             ]
         );
@@ -360,7 +360,7 @@ 
         $this->kernel->setProperty(
             'deny_attempt_buffer',
             [
-                'data_circle'     => $setting['data_circle']['buffer'],     // 10
+                'data_circle'     => $setting['data_circle']['buffer'], // 10
                 'system_firewall' => $setting['system_firewall']['buffer'], // 10
             ]
         );
@@ -428,7 +428,7 @@ 
 
             $lastResetTime = $cronjobSetting['config']['last_update'];
 
-            if (!empty($lastResetTime) ) {
+            if (!empty($lastResetTime)) {
                 $lastResetTime = strtotime($lastResetTime);
             } else {
                 // @codeCoverageIgnoreStart

src/Firewall/Firewall/Messenger/MessengerFactory.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
         $str = implode(
             '',
             array_map(
-                function ($word) {
+                function($word) {
                     return ucwords($word); 
                 },
                 $str