Passed
Push — 2.x ( c675a1...c39d11 )
by Terry
02:00
created
src/Firewall/Driver/RedisDriver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -274,8 +274,8 @@
 block discarded – undo
274 274
     {
275 275
         $table = [
276 276
             'filter'  => $this->tableFilterLogs . ':' . $ip,
277
-            'session' => $this->tableSessions   . ':' . $ip,
278
-            'rule'    => $this->tableRuleList   . ':' . $ip,
277
+            'session' => $this->tableSessions . ':' . $ip,
278
+            'rule'    => $this->tableRuleList . ':' . $ip,
279 279
         ];
280 280
         
281 281
         return $table[$type] ?? '';
Please login to merge, or discard this patch.
src/Firewall/Driver/FileDriverTrait.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -100,8 +100,8 @@
 block discarded – undo
100 100
         $path = [];
101 101
 
102 102
         $path['filter']  = $this->directory . '/' . $this->tableFilterLogs . '/' . $ip . '.' . $this->extension;
103
-        $path['session'] = $this->directory . '/' . $this->tableSessions   . '/' . $ip . '.' . $this->extension;
104
-        $path['rule']    = $this->directory . '/' . $this->tableRuleList   . '/' . $ip . '.' . $this->extension;
103
+        $path['session'] = $this->directory . '/' . $this->tableSessions . '/' . $ip . '.' . $this->extension;
104
+        $path['rule']    = $this->directory . '/' . $this->tableRuleList . '/' . $ip . '.' . $this->extension;
105 105
 
106 106
         return $path[$type] ?? '';
107 107
     }
Please login to merge, or discard this patch.
src/Firewall/Firewall/Captcha/ItemImage.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@
 block discarded – undo
39 39
      */
40 40
     public static function get(array $setting): CaptchaInterface
41 41
     {
42
-        $type   = $setting['config']['type']   ?? 'alnum';
42
+        $type   = $setting['config']['type'] ?? 'alnum';
43 43
         $length = $setting['config']['length'] ?? 8;
44 44
 
45 45
         $imageCaptchaConfig = [];
Please login to merge, or discard this patch.
src/Firewall/Panel/BaseController.php 1 patch
Spacing   +3 added lines, -5 removed lines patch added patch discarded remove patch
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
             define('SHIELDON_VIEW', true);
165 165
         }
166 166
 
167
-        $viewFilePath =  __DIR__ . '/../../../templates/' . $page . '.php';
167
+        $viewFilePath = __DIR__ . '/../../../templates/' . $page . '.php';
168 168
     
169 169
         if (!empty($data)) {
170 170
             extract($data);
@@ -433,12 +433,10 @@  discard block
 block discarded – undo
433 433
         $echo = [];
434 434
 
435 435
         $echo['css'] = $this->getConfig('messengers.' . $moduleName . '.confirm_test') ? 
436
-            'success' :
437
-            '';
436
+            'success' : '';
438 437
         
439 438
         $echo['icon'] = $this->getConfig('messengers.' . $moduleName . '.confirm_test') ?
440
-            '<i class="fas fa-check"></i>' :
441
-            '<i class="fas fa-exclamation"></i>';
439
+            '<i class="fas fa-check"></i>' : '<i class="fas fa-exclamation"></i>';
442 440
 
443 441
         echo $echo[$echoType];
444 442
     }
Please login to merge, or discard this patch.
templates/panel/iptables_manager.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
                     <select name="subnet" class="regular" id="subnet">
60 60
                         <option value="null">---</option>
61 61
                         <?php for ($i = 32; $i > 0; $i--) : ?>
62
-                            <?php $label = $i;  ?>
62
+                            <?php $label = $i; ?>
63 63
                             <?php if ($i === 8) $label = $i . ' (A)'; ?>
64 64
                             <?php if ($i === 16) $label = $i . ' (B)'; ?>
65 65
                             <?php if ($i === 24) $label = $i . ' (C)'; ?>
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
                     <select name="subnet" class="regular" id="subnet">
72 72
                         <option value="null">---</option>
73 73
                         <?php for ($i = 128; $i > 0; $i--) : ?>
74
-                            <?php $label = $i;  ?>
74
+                            <?php $label = $i; ?>
75 75
                             <option value="<?php echo $i; ?>"><?php echo $label; ?></option>
76 76
                         <?php endfor; ?>
77 77
                     </select>
Please login to merge, or discard this patch.
templates/panel/xss_protection.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -128,7 +128,7 @@
 block discarded – undo
128 128
         </div>
129 129
     </div>
130 130
     <br />
131
-    <?php if ( empty($xss_protected_list)) : ?>
131
+    <?php if (empty($xss_protected_list)) : ?>
132 132
     <div id="so-table-container" class="so-datatables">
133 133
         <table id="so-datalog" class="cell-border compact stripe responsive" cellspacing="0" width="100%">
134 134
             <tbody>
Please login to merge, or discard this patch.
templates/panel/ip_manager.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@
 block discarded – undo
71 71
         </div>
72 72
     </div>
73 73
     <br />
74
-    <?php if ( empty($ip_list)) : ?>
74
+    <?php if (empty($ip_list)) : ?>
75 75
     <div id="so-table-container" class="so-datatables">
76 76
         <table id="so-datalog" class="cell-border compact stripe responsive" cellspacing="0" width="100%">
77 77
             <tbody>
Please login to merge, or discard this patch.
src/Firewall/Log/ActionLogParser.php 1 patch
Spacing   +26 added lines, -26 removed lines patch added patch discarded remove patch
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
         ];
208 208
 
209 209
         if (empty($dataRange[$this->type])) {
210
-            if (preg_match('/past_([0-9]+)_days/', $this->type, $matches) ) {
210
+            if (preg_match('/past_([0-9]+)_days/', $this->type, $matches)) {
211 211
                 $dayCount = $matches[1];
212 212
                 $startDate = date('Ymd', strtotime('-' . $dayCount . ' days'));
213 213
                 $endDate = date('Ymd');
@@ -350,18 +350,18 @@  discard block
 block discarded – undo
350 350
 
351 351
         $results['captcha_chart_string']       = ''; // string
352 352
         $results['pageview_chart_string']      = ''; // string
353
-        $results['captcha_success_count']      = 0;  // integer
354
-        $results['captcha_failure_count']      = 0;  // integer
355
-        $results['captcha_count']              = 0;  // integer
356
-        $results['pageview_count']             = 0;  // integer
357
-        $results['captcha_percentageage']      = 0;  // integer
358
-        $results['captcha_failure_percentage'] = 0;  // integer
359
-        $results['captcha_success_percentage'] = 0;  // integer
360
-        $results['action_ban_count']           = 0;  // integer
361
-        $results['action_temp_ban_count']      = 0;  // integer
362
-        $results['action_unban_count']         = 0;  // integer
363
-        $results['blacklist_count']            = 0;  // integer
364
-        $results['session_limit_count']        = 0;  // integer
353
+        $results['captcha_success_count']      = 0; // integer
354
+        $results['captcha_failure_count']      = 0; // integer
355
+        $results['captcha_count']              = 0; // integer
356
+        $results['pageview_count']             = 0; // integer
357
+        $results['captcha_percentageage']      = 0; // integer
358
+        $results['captcha_failure_percentage'] = 0; // integer
359
+        $results['captcha_success_percentage'] = 0; // integer
360
+        $results['action_ban_count']           = 0; // integer
361
+        $results['action_temp_ban_count']      = 0; // integer
362
+        $results['action_unban_count']         = 0; // integer
363
+        $results['blacklist_count']            = 0; // integer
364
+        $results['session_limit_count']        = 0; // integer
365 365
 
366 366
         $ipdData = $this->getIpData();
367 367
 
@@ -406,18 +406,18 @@  discard block
 block discarded – undo
406 406
         $results['label_chart_string']         = ''; // string
407 407
         $results['captcha_chart_string']       = ''; // string
408 408
         $results['pageview_chart_string']      = ''; // string
409
-        $results['captcha_success_count']      = 0;  // integer
410
-        $results['captcha_failure_count']      = 0;  // integer
411
-        $results['captcha_count']              = 0;  // integer
412
-        $results['pageview_count']             = 0;  // integer
413
-        $results['captcha_percentageage']      = 0;  // integer
414
-        $results['captcha_failure_percentage'] = 0;  // integer
415
-        $results['captcha_success_percentage'] = 0;  // integer
416
-        $results['action_ban_count']           = 0;  // integer
417
-        $results['action_temp_ban_count']      = 0;  // integer
418
-        $results['action_unban_count']         = 0;  // integer
419
-        $results['blacklist_count']            = 0;  // integer
420
-        $results['session_limit_count']        = 0;  // integer
409
+        $results['captcha_success_count']      = 0; // integer
410
+        $results['captcha_failure_count']      = 0; // integer
411
+        $results['captcha_count']              = 0; // integer
412
+        $results['pageview_count']             = 0; // integer
413
+        $results['captcha_percentageage']      = 0; // integer
414
+        $results['captcha_failure_percentage'] = 0; // integer
415
+        $results['captcha_success_percentage'] = 0; // integer
416
+        $results['action_ban_count']           = 0; // integer
417
+        $results['action_temp_ban_count']      = 0; // integer
418
+        $results['action_unban_count']         = 0; // integer
419
+        $results['blacklist_count']            = 0; // integer
420
+        $results['session_limit_count']        = 0; // integer
421 421
 
422 422
         if (!empty($periodData)) {
423 423
 
@@ -477,7 +477,7 @@  discard block
 block discarded – undo
477 477
         $ip = $log['ip'];
478 478
         $sessionId = $log['session_id'];
479 479
 
480
-        $this->ipDetail[$t][$ip]['session_id'][$sessionId ] = 1;
480
+        $this->ipDetail[$t][$ip]['session_id'][$sessionId] = 1;
481 481
 
482 482
         if ($logActionCode === self::LOG_TEMPORARILY_BAN) {
483 483
             $this->periodDetail[$t][$k]['action_temp_ban_count']++;
Please login to merge, or discard this patch.
src/Firewall/Panel/Report.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             $data = array_merge($data, $result);
138 138
         }
139 139
 
140
-        $data['page_availability'] = $this->pageAvailability['logs'];;
140
+        $data['page_availability'] = $this->pageAvailability['logs']; ;
141 141
 
142 142
         $data['page_url'] = $this->url('report/actionLog');
143 143
 
@@ -209,11 +209,11 @@  discard block
 block discarded – undo
209 209
     private function operationTemplateVarsOfComponents(array $data = []): array
210 210
     {
211 211
         $data['components'] = [
212
-            'Ip'         => (!empty($this->kernel->component['Ip']))         ? true : false,
212
+            'Ip'         => (!empty($this->kernel->component['Ip'])) ? true : false,
213 213
             'TrustedBot' => (!empty($this->kernel->component['TrustedBot'])) ? true : false,
214
-            'Header'     => (!empty($this->kernel->component['Header']))     ? true : false,
215
-            'Rdns'       => (!empty($this->kernel->component['Rdns']))       ? true : false,
216
-            'UserAgent'  => (!empty($this->kernel->component['UserAgent']))  ? true : false,
214
+            'Header'     => (!empty($this->kernel->component['Header'])) ? true : false,
215
+            'Rdns'       => (!empty($this->kernel->component['Rdns'])) ? true : false,
216
+            'UserAgent'  => (!empty($this->kernel->component['UserAgent'])) ? true : false,
217 217
         ];
218 218
 
219 219
         return $data;
Please login to merge, or discard this patch.