Passed
Push — dev ( 8b84d5...475726 )
by Nils
05:50
created
pages/statistics.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('statistics') === false) {
64 64
     // Not allowed page
65 65
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
66
-    include $SETTINGS['cpassman_dir'] . '/error.php';
66
+    include $SETTINGS['cpassman_dir'].'/error.php';
67 67
     exit;
68 68
 }
69 69
 
Please login to merge, or discard this patch.
pages/favourites.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('favourites') === false) {
63 63
     // Not allowed page
64 64
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
65
-    include $SETTINGS['cpassman_dir'] . '/error.php';
65
+    include $SETTINGS['cpassman_dir'].'/error.php';
66 66
     exit;
67 67
 }
68 68
 
@@ -114,12 +114,12 @@  discard block
 block discarded – undo
114 114
                             if (empty($fav) === false) {
115 115
                                 $data = DB::queryFirstRow(
116 116
                                     'SELECT i.label, i.description, i.id, i.id_tree, t.title
117
-                                    FROM ' . prefixTable('items') . ' as i
118
-                                    INNER JOIN ' . prefixTable('nested_tree') . ' as t ON (t.id = i.id_tree)
117
+                                    FROM ' . prefixTable('items').' as i
118
+                                    INNER JOIN ' . prefixTable('nested_tree').' as t ON (t.id = i.id_tree)
119 119
                                     WHERE i.id = %i',
120 120
                                     $fav
121 121
                                 );
122
-                                if (! empty($data['label'])) {
122
+                                if (!empty($data['label'])) {
123 123
                                     ?>
124 124
                                     <tr>
125 125
                                         <td>
Please login to merge, or discard this patch.
pages/export.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('export') === false) {
64 64
     // Not allowed page
65 65
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
66
-    include $SETTINGS['cpassman_dir'] . '/error.php';
66
+    include $SETTINGS['cpassman_dir'].'/error.php';
67 67
     exit;
68 68
 }
69 69
 
Please login to merge, or discard this patch.
pages/emails.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('emails') === false) {
64 64
     // Not allowed page
65 65
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
66
-    include $SETTINGS['cpassman_dir'] . '/error.php';
66
+    include $SETTINGS['cpassman_dir'].'/error.php';
67 67
     exit;
68 68
 }
69 69
 
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
                     <div class="card-body">
246 246
                         <div id="unsent-emails">
247 247
                             <?php
248
-                            DB::query('SELECT * FROM ' . prefixTable('emails') . ' WHERE status = %s OR status = %s', 'not_sent', '');
248
+                            DB::query('SELECT * FROM '.prefixTable('emails').' WHERE status = %s OR status = %s', 'not_sent', '');
249 249
 echo str_replace('#nb_emails#', (string) DB::count(), langHdl('email_send_backlog'));
250 250
                             ?>
251 251
                         </div>
Please login to merge, or discard this patch.
pages/users.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('users') === false) {
64 64
     // Not allowed page
65 65
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
66
-    include $SETTINGS['cpassman_dir'] . '/error.php';
66
+    include $SETTINGS['cpassman_dir'].'/error.php';
67 67
     exit;
68 68
 }
69 69
 
@@ -89,24 +89,24 @@  discard block
 block discarded – undo
89 89
 // If administrator then all roles are shown
90 90
 // else only the Roles the users is associated to.
91 91
 if ((int) $_SESSION['is_admin'] === 1) {
92
-    $optionsManagedBy .= '<option value="0">' . langHdl('administrators_only') . '</option>';
92
+    $optionsManagedBy .= '<option value="0">'.langHdl('administrators_only').'</option>';
93 93
 }
94 94
 
95 95
 $rows = DB::query(
96 96
     'SELECT id, title, creator_id
97
-    FROM ' . prefixTable('roles_title') . '
97
+    FROM ' . prefixTable('roles_title').'
98 98
     ORDER BY title ASC'
99 99
 );
100 100
 foreach ($rows as $record) {
101 101
     if ((int) $_SESSION['is_admin'] === 1 || in_array($record['id'], $_SESSION['user_roles']) === true) {
102
-        $optionsManagedBy .= '<option value="' . $record['id'] . '">' . langHdl('managers_of') . ' ' . addslashes($record['title']) . '</option>';
102
+        $optionsManagedBy .= '<option value="'.$record['id'].'">'.langHdl('managers_of').' '.addslashes($record['title']).'</option>';
103 103
     }
104 104
     if (
105 105
         (int) $_SESSION['is_admin'] === 1
106 106
         || (((int) $superGlobal->get('user_manager', 'SESSION') === 1 || (int) $_SESSION['user_can_manage_all_users'] === 1)
107 107
             && (in_array($record['id'], $userRoles) === true) || (int) $record['creator_id'] === (int) $_SESSION['user_id'])
108 108
     ) {
109
-        $optionsRoles .= '<option value="' . $record['id'] . '">' . addslashes($record['title']) . '</option>';
109
+        $optionsRoles .= '<option value="'.$record['id'].'">'.addslashes($record['title']).'</option>';
110 110
     }
111 111
 }
112 112
 
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
         for ($y = 1; $y < $t->nlevel; ++$y) {
122 122
             $ident .= '&nbsp;&nbsp;';
123 123
         }
124
-        $foldersList .= '<option value="' . $t->id . '">' . $ident . htmlspecialchars($t->title, ENT_COMPAT, 'UTF-8') . '</option>';
124
+        $foldersList .= '<option value="'.$t->id.'">'.$ident.htmlspecialchars($t->title, ENT_COMPAT, 'UTF-8').'</option>';
125 125
     }
126 126
 }
127 127
 
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
                         </button><?php
159 159
                                     echo isset($SETTINGS['ldap_mode']) === true && (int) $SETTINGS['ldap_mode'] === 1 && (int) $_SESSION['is_admin'] === 1 ?
160 160
                                         '<button type="button" class="btn btn-primary btn-sm tp-action mr-2" data-action="ldap-sync">
161
-                            <i class="fa-solid fa-address-card mr-2"></i>' . langHdl('ldap_synchronization') . '
161
+                            <i class="fa-solid fa-address-card mr-2"></i>' . langHdl('ldap_synchronization').'
162 162
                         </button>' : '';
163 163
                                     ?>
164 164
                     </h3>
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
230 230
                                             <?php
231 231
                                             foreach (TP_PW_COMPLEXITY as $entry) {
232 232
                                                 echo '
233
-                                            <option value="' . $entry[0] . '">' . addslashes($entry[1]) . '</option>';
233
+                                            <option value="' . $entry[0].'">'.addslashes($entry[1]).'</option>';
234 234
                                             }
235 235
                                             ?>
236 236
                                         </select>
Please login to merge, or discard this patch.
pages/tasks.js.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@  discard block
 block discarded – undo
70 70
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('tasks') === false) {
71 71
     // Not allowed page
72 72
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
73
-    include $SETTINGS['cpassman_dir'] . '/error.php';
73
+    include $SETTINGS['cpassman_dir'].'/error.php';
74 74
     exit;
75 75
 }
76 76
 ?>
@@ -423,7 +423,7 @@  discard block
 block discarded – undo
423 423
                         // error
424 424
                         toastr.remove();
425 425
                         toastr.error(
426
-                            '<?php echo langHdl('server_answer_error') . '<br />' . langHdl('server_returned_data') . ':<br />'; ?>' + data.error,
426
+                            '<?php echo langHdl('server_answer_error').'<br />'.langHdl('server_returned_data').':<br />'; ?>' + data.error,
427 427
                             '', {
428 428
                                 closeButton: true,
429 429
                                 positionClass: 'toastr-top-right'
@@ -446,11 +446,11 @@  discard block
 block discarded – undo
446 446
                         $('#'+field+'_parameter_value').val(frequency === null ? '' : frequency + ';' +value,);
447 447
                         param = value.split(';');
448 448
                         if (param.length === 1) {
449
-                            txt = ' <?php echo langHdl('at');?> ' + param[0];
449
+                            txt = ' <?php echo langHdl('at'); ?> ' + param[0];
450 450
                         } else {
451
-                            txt = ' <?php echo langHdl('day');?> ' + param[1] + ' <?php echo langHdl('at');?> ' + param[0];
451
+                            txt = ' <?php echo langHdl('day'); ?> ' + param[1] + ' <?php echo langHdl('at'); ?> ' + param[0];
452 452
                         }
453
-                        $('#'+field+'_parameter').val(frequency === null ? '<?php echo langHdl('not_defined');?>' : (data.message + txt));
453
+                        $('#'+field+'_parameter').val(frequency === null ? '<?php echo langHdl('not_defined'); ?>' : (data.message + txt));
454 454
                         $("#task-define-modal").modal('hide');
455 455
                         $('#task-define-modal-type, #task-define-modal-parameter-hourly-value, #task-define-modal-parameter-daily-value, #task-define-modal-frequency').val('');
456 456
                     }
@@ -556,7 +556,7 @@  discard block
 block discarded – undo
556 556
                         // error
557 557
                         toastr.remove();
558 558
                         toastr.error(
559
-                            '<?php echo langHdl('server_answer_error') . '<br />' . langHdl('server_returned_data') . ':<br />'; ?>' + data.error,
559
+                            '<?php echo langHdl('server_answer_error').'<br />'.langHdl('server_returned_data').':<br />'; ?>' + data.error,
560 560
                             '', {
561 561
                                 closeButton: true,
562 562
                                 positionClass: 'toastr-top-right'
@@ -607,7 +607,7 @@  discard block
 block discarded – undo
607 607
                     // error
608 608
                     toastr.remove();
609 609
                     toastr.error(
610
-                        '<?php echo langHdl('server_answer_error') . '<br />' . langHdl('server_returned_data') . ':<br />'; ?>' + data.error,
610
+                        '<?php echo langHdl('server_answer_error').'<br />'.langHdl('server_returned_data').':<br />'; ?>' + data.error,
611 611
                         '', {
612 612
                             closeButton: true,
613 613
                             positionClass: 'toastr-top-right'
Please login to merge, or discard this patch.
pages/ldap.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('ldap') === false) {
65 65
     // Not allowed page
66 66
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
67
-    include $SETTINGS['cpassman_dir'] . '/error.php';
67
+    include $SETTINGS['cpassman_dir'].'/error.php';
68 68
     exit;
69 69
 }
70 70
 
Please login to merge, or discard this patch.
pages/favorites.js.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -70,7 +70,7 @@
 block discarded – undo
70 70
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('favourites') === false) {
71 71
     // Not allowed page
72 72
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
73
-    include $SETTINGS['cpassman_dir'] . '/error.php';
73
+    include $SETTINGS['cpassman_dir'].'/error.php';
74 74
     exit;
75 75
 }
76 76
 ?>
Please login to merge, or discard this patch.
pages/utilities.logs.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('utilities.logs') === false) {
64 64
     // Not allowed page
65 65
     $_SESSION['error']['code'] = ERR_NOT_ALLOWED;
66
-    include $SETTINGS['cpassman_dir'] . '/error.php';
66
+    include $SETTINGS['cpassman_dir'].'/error.php';
67 67
     exit;
68 68
 }
69 69
 
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
                                             echo isset($_SESSION['user_admin']) && (int) $_SESSION['user_admin'] === 1 ? '' : ' hidden';
204 204
                                             ?>">
205 205
                         <div class="form-group">
206
-                            <h5><i class="fas fa-broom mr-2"></i><?php echo langHdl('purge') . ' ' . langHdl('date_range'); ?></h5>
206
+                            <h5><i class="fas fa-broom mr-2"></i><?php echo langHdl('purge').' '.langHdl('date_range'); ?></h5>
207 207
                             <div class="input-group">
208 208
                                 <div class="input-group-prepend">
209 209
                                     <span class="input-group-text">
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
226 226
                                     <select class="form-control" id="purge-filter-user">
227 227
                                         <option value="-1"><?php echo langHdl('all'); ?></option>
228 228
                                     <?php
229
-                                    $rows = DB::query('SELECT id, name, lastname FROM ' . prefixTable('users') . ' WHERE admin = 0');
229
+                                    $rows = DB::query('SELECT id, name, lastname FROM '.prefixTable('users').' WHERE admin = 0');
230 230
 foreach ($rows as $record) {
231 231
     echo '
232 232
                                         <option value="'.$record['id'].'">'.$record['name'].' '.$record['lastname'].'</option>';
Please login to merge, or discard this patch.