terrylinooo / shieldon

src/Firewall/Firewall/Messenger/ItemSendgrid.php

Spacing +2 added lines, -2 removed lines

@@ -40,8 +40,8 @@
      */
     public static function get(array $setting): MessengerInterface
     {
-        $apiKey     = $setting['config']['api_key']    ?? '';
-        $sender     = $setting['config']['sender']     ?? '';
+        $apiKey     = $setting['config']['api_key'] ?? '';
+        $sender     = $setting['config']['sender'] ?? '';
         $recipients = $setting['config']['recipients'] ?? [];
 
         $instance = new Sendgrid($apiKey);

src/Firewall/Middleware/UserAgent.php

Spacing +10 added lines, -10 removed lines

@@ -50,19 +50,19 @@ 
     protected $deniedList = [
 
         // Backlink crawlers
-        'Ahrefs',     // http://ahrefs.com/robot/
-        'roger',      // rogerbot (SEOMOZ)
-        'moz.com',    // SEOMOZ crawlers
-        'MJ12bot',    // Majestic crawlers
-        'findlinks',  // http://wortschatz.uni-leipzig.de/findlinks
-        'Semrush',    // http://www.semrush.com/bot.html
+        'Ahrefs', // http://ahrefs.com/robot/
+        'roger', // rogerbot (SEOMOZ)
+        'moz.com', // SEOMOZ crawlers
+        'MJ12bot', // Majestic crawlers
+        'findlinks', // http://wortschatz.uni-leipzig.de/findlinks
+        'Semrush', // http://www.semrush.com/bot.html
 
         // Web information crawlers
-        'domain',     // Domain name information crawlers.
-        'copyright',  // Copyright information crawlers.
+        'domain', // Domain name information crawlers.
+        'copyright', // Copyright information crawlers.
 
         // Others
-        'archive',    // Wayback machine
+        'archive', // Wayback machine
     ];
 
     /**
@@ -96,7 +96,7 @@ 
         }
 
         if (!empty($this->deniedList)) {
-            if (preg_match('/(' . implode('|', $this->deniedList). ')/i', $userAgent)) {
+            if (preg_match('/(' . implode('|', $this->deniedList) . ')/i', $userAgent)) {
                 return (new Response)->withStatus(self::HTTP_STATUS_CODE);
             }
         }

src/Firewall/Driver/RedisDriver.php

Spacing +2 added lines, -2 removed lines

@@ -274,8 +274,8 @@
     {
         $table = [
             'filter'  => $this->tableFilterLogs . ':' . $ip,
-            'session' => $this->tableSessions   . ':' . $ip,
-            'rule'    => $this->tableRuleList   . ':' . $ip,
+            'session' => $this->tableSessions . ':' . $ip,
+            'rule'    => $this->tableRuleList . ':' . $ip,
         ];
         
         return $table[$type] ?? '';

src/Firewall/Driver/FileDriverTrait.php

Spacing +2 added lines, -2 removed lines

@@ -100,8 +100,8 @@
         $path = [];
 
         $path['filter']  = $this->directory . '/' . $this->tableFilterLogs . '/' . $ip . '.' . $this->extension;
-        $path['session'] = $this->directory . '/' . $this->tableSessions   . '/' . $ip . '.' . $this->extension;
-        $path['rule']    = $this->directory . '/' . $this->tableRuleList   . '/' . $ip . '.' . $this->extension;
+        $path['session'] = $this->directory . '/' . $this->tableSessions . '/' . $ip . '.' . $this->extension;
+        $path['rule']    = $this->directory . '/' . $this->tableRuleList . '/' . $ip . '.' . $this->extension;
 
         return $path[$type] ?? '';
     }

src/Firewall/Firewall/Captcha/ItemImage.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
      */
     public static function get(array $setting): CaptchaInterface
     {
-        $type   = $setting['config']['type']   ?? 'alnum';
+        $type   = $setting['config']['type'] ?? 'alnum';
         $length = $setting['config']['length'] ?? 8;
 
         $imageCaptchaConfig = [];

src/Firewall/Panel/BaseController.php

Spacing +3 added lines, -5 removed lines

@@ -164,7 +164,7 @@ 
             define('SHIELDON_VIEW', true);
         }
 
-        $viewFilePath =  __DIR__ . '/../../../templates/' . $page . '.php';
+        $viewFilePath = __DIR__ . '/../../../templates/' . $page . '.php';
     
         if (!empty($data)) {
             extract($data);
@@ -433,12 +433,10 @@ 
         $echo = [];
 
         $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];
     }

templates/panel/template.php

Braces +5 added lines, -2 removed lines

@@ -263,13 +263,16 @@
                                         <?php endif; ?>
                                      </span>
                                 </div>
-                            <?php else: ?>
+                            <?php else {
+    : ?>
                                 <div class="icon-box">
                                     <span class="message-icon">
                                         <i class="fas fa-exclamation-circle"></i>
                                     </span>
                                 </div>
-                            <?php endif; ?>
+                            <?php endif;
+}
+?>
                         </div>
                         <button type="button" class="close" data-dismiss="modal" aria-label="Close">
                             <span aria-hidden="true">&times;</span>

templates/panel/table_rules.php

Braces +5 added lines, -2 removed lines

@@ -67,7 +67,8 @@ 
             </tbdoy>
         </table>
     </div>
-    <?php else: ?>
+    <?php else {
+    : ?>
     <div id="so-table-loading" class="so-datatables">
         <div class="lds-css ng-scope">
             <div class="lds-ripple">
@@ -76,7 +77,9 @@ 
             </div>
         </div>
     </div>
-    <?php endif; ?>
+    <?php endif;
+}
+?>
     <?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%">

src/Firewall/Log/ActionLogParser.php

Spacing +26 added lines, -26 removed lines

@@ -207,7 +207,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');
@@ -350,18 +350,18 @@ 
 
         $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['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
 
         $ipdData = $this->getIpData();
 
@@ -406,18 +406,18 @@ 
         $results['label_chart_string']         = ''; // string
         $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['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
 
         if (!empty($periodData)) {
 
@@ -477,7 +477,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']++;