Passed
Pull Request — master (#4928)
by
unknown
06:18
created
pages/admin.js.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -68,7 +68,7 @@  discard block
 block discarded – undo
68 68
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('admin') === false) {
69 69
     // Not allowed page
70 70
     $session->set('system-error_code', ERR_NOT_ALLOWED);
71
-    include $SETTINGS['cpassman_dir'] . '/error.php';
71
+    include $SETTINGS['cpassman_dir'].'/error.php';
72 72
     exit;
73 73
 }
74 74
 
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
                     // error
154 154
                     toastr.remove();
155 155
                     toastr.error(
156
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
156
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
157 157
                         '', {
158 158
                             closeButton: true,
159 159
                             positionClass: 'toast-bottom-right'
@@ -275,7 +275,7 @@  discard block
 block discarded – undo
275 275
                     // error
276 276
                     toastr.remove();
277 277
                     toastr.error(
278
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
278
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
279 279
                         '', {
280 280
                             closeButton: true,
281 281
                             positionClass: 'toast-bottom-right'
@@ -340,7 +340,7 @@  discard block
 block discarded – undo
340 340
             // error
341 341
             toastr.remove();
342 342
             toastr.error(
343
-                '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
343
+                '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
344 344
                 '', {
345 345
                     closeButton: true,
346 346
                     positionClass: 'toast-bottom-right'
@@ -551,7 +551,7 @@  discard block
 block discarded – undo
551 551
                     // error
552 552
                     toastr.remove();
553 553
                     toastr.error(
554
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
554
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
555 555
                         '', {
556 556
                             closeButton: true,
557 557
                             positionClass: 'toast-bottom-right'
@@ -650,7 +650,7 @@  discard block
 block discarded – undo
650 650
                     // error
651 651
                     toastr.remove();
652 652
                     toastr.error(
653
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
653
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
654 654
                         '', {
655 655
                             closeButton: true,
656 656
                             positionClass: 'toast-bottom-right'
@@ -679,7 +679,7 @@  discard block
 block discarded – undo
679 679
                     
680 680
                     //const className = result.success ? 'success' : 'error';
681 681
                     const icon = result.success ? '<i class="fa-solid fa-check text-success mr-1"></i>' : '<i class="fa-solid fa-xmark text-danger mr-1"></i>';
682
-                    const message = result.success ? '<?php echo $lang->get('server_returned_data');?>' : 'Error: ' + result.error;
682
+                    const message = result.success ? '<?php echo $lang->get('server_returned_data'); ?>' : 'Error: ' + result.error;
683 683
                     
684 684
                     output += '<li>' + icon + '<b>' + file + '</b><br/>' + message + '</li>';
685 685
                 });
@@ -710,7 +710,7 @@  discard block
 block discarded – undo
710 710
                     // error
711 711
                     toastr.remove();
712 712
                     toastr.error(
713
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
713
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
714 714
                         '', {
715 715
                             closeButton: true,
716 716
                             positionClass: 'toast-bottom-right'
@@ -774,7 +774,7 @@  discard block
 block discarded – undo
774 774
                     // error
775 775
                     toastr.remove();
776 776
                     toastr.error(
777
-                        '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + data.error,
777
+                        '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + data.error,
778 778
                         '', {
779 779
                             closeButton: true,
780 780
                             positionClass: 'toast-bottom-right'
Please login to merge, or discard this patch.
sources/utilities.queries.php 1 patch
Spacing   +30 added lines, -30 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 ) {
78 78
     // Not allowed page
79 79
     $session->set('system-error_code', ERR_NOT_ALLOWED);
80
-    include $SETTINGS['cpassman_dir'] . '/error.php';
80
+    include $SETTINGS['cpassman_dir'].'/error.php';
81 81
     exit;
82 82
 }
83 83
 
@@ -128,7 +128,7 @@  discard block
 block discarded – undo
128 128
             $arrFolders = array();
129 129
             $rows = DB::query(
130 130
                 'SELECT valeur, intitule
131
-                FROM ' . prefixTable('misc') . '
131
+                FROM ' . prefixTable('misc').'
132 132
                 WHERE type  = %s',
133 133
                 'folder_deleted'
134 134
             );
@@ -151,11 +151,11 @@  discard block
 block discarded – undo
151 151
                 i.id as id, i.label as label,
152 152
                 i.id_tree as id_tree, l.date as date, n.title as folder_title,
153 153
                 a.del_enabled as del_enabled, a.del_value as del_value, a.del_type as del_type
154
-                FROM ' . prefixTable('log_items') . ' as l
155
-                INNER JOIN ' . prefixTable('items') . ' as i ON (l.id_item=i.id)
156
-                LEFT JOIN ' . prefixTable('users') . ' as u ON (l.id_user=u.id)
157
-                LEFT JOIN ' . prefixTable('nested_tree') . ' as n ON (i.id_tree=n.id)
158
-                LEFT JOIN ' . prefixTable('automatic_del') . ' as a ON (l.id_item = a.item_id)
154
+                FROM ' . prefixTable('log_items').' as l
155
+                INNER JOIN ' . prefixTable('items').' as i ON (l.id_item=i.id)
156
+                LEFT JOIN ' . prefixTable('users').' as u ON (l.id_user=u.id)
157
+                LEFT JOIN ' . prefixTable('nested_tree').' as n ON (i.id_tree=n.id)
158
+                LEFT JOIN ' . prefixTable('automatic_del').' as a ON (l.id_item = a.item_id)
159 159
                 WHERE l.action = %s
160 160
                 AND i.perso = %i
161 161
                 AND TRIM(COALESCE(i.deleted_at, "")) <> ""
@@ -179,7 +179,7 @@  discard block
 block discarded – undo
179 179
                             'label' => $record['label'],
180 180
                             'date' => date($SETTINGS['date_format'], (int) $record['date']),
181 181
                             'login' => $record['login'],
182
-                            'name' => $record['name'] . ' ' . $record['lastname'],
182
+                            'name' => $record['name'].' '.$record['lastname'],
183 183
                             'folder_label' => $record['folder_title'],
184 184
                             'folder_deleted' => $thisFolder,
185 185
                             'del_enabled' => (bool) $record['del_enabled'],
@@ -240,10 +240,10 @@  discard block
 block discarded – undo
240 240
             foreach ($post_folders as $folderId) {
241 241
                 $data = DB::queryFirstRow(
242 242
                     'SELECT valeur
243
-                    FROM ' . prefixTable('misc') . "
243
+                    FROM ' . prefixTable('misc')."
244 244
                     WHERE type = 'folder_deleted'
245 245
                     AND intitule = %s",
246
-                    'f' . $folderId
246
+                    'f'.$folderId
247 247
                 );
248 248
                 if ((int) $data['valeur'] !== 0) {
249 249
                     $folderData = explode(', ', $data['valeur']);
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
                         prefixTable('misc'),
271 271
                         'type = %s AND intitule = %s',
272 272
                         'folder_deleted',
273
-                        'f' . $folderId
273
+                        'f'.$folderId
274 274
                     );
275 275
 
276 276
                     // Restore all items in this folder
@@ -287,7 +287,7 @@  discard block
 block discarded – undo
287 287
                     // Get list of all items in thos folder
288 288
                     $items = DB::query(
289 289
                         'SELECT id
290
-                        FROM ' . prefixTable('items') . '
290
+                        FROM ' . prefixTable('items').'
291 291
                         WHERE id_tree = %i',
292 292
                         $folderId
293 293
                     );
@@ -377,10 +377,10 @@  discard block
 block discarded – undo
377 377
             foreach ($post_folders as $folderId) {
378 378
                 $data = DB::queryFirstRow(
379 379
                     'SELECT valeur
380
-                    FROM ' . prefixTable('misc') . "
380
+                    FROM ' . prefixTable('misc')."
381 381
                     WHERE type = 'folder_deleted'
382 382
                     AND intitule = %s",
383
-                    'f' . $folderId
383
+                    'f'.$folderId
384 384
                 );
385 385
                 if ((int) $data['valeur'] !== 0) {
386 386
                     $exploded = explode(',', $data['valeur']);
@@ -391,7 +391,7 @@  discard block
 block discarded – undo
391 391
                         prefixTable('misc'),
392 392
                         'type = %s AND intitule = %s',
393 393
                         'folder_deleted',
394
-                        'f' . $folderData[0]
394
+                        'f'.$folderData[0]
395 395
                     );
396 396
 
397 397
                     // Delete all items in this folder
@@ -405,7 +405,7 @@  discard block
 block discarded – undo
405 405
                     // Get list of all items in thos folder
406 406
                     $items = DB::query(
407 407
                         'SELECT id
408
-                        FROM ' . prefixTable('items') . '
408
+                        FROM ' . prefixTable('items').'
409 409
                         WHERE id_tree = %i',
410 410
                         $folderData[0]
411 411
                     );
@@ -422,7 +422,7 @@  discard block
 block discarded – undo
422 422
                         // Delete all fields
423 423
                         $fields = DB::query(
424 424
                             'SELECT id
425
-                            FROM ' . prefixTable('categories_items') . '
425
+                            FROM ' . prefixTable('categories_items').'
426 426
                             WHERE item_id = %i',
427 427
                             $folderData[0]
428 428
                         );
@@ -442,7 +442,7 @@  discard block
 block discarded – undo
442 442
                         // Delete all files
443 443
                         $files = DB::query(
444 444
                             'SELECT id
445
-                            FROM ' . prefixTable('files') . '
445
+                            FROM ' . prefixTable('files').'
446 446
                             WHERE id_item = %i',
447 447
                             $folderData[0]
448 448
                         );
@@ -499,7 +499,7 @@  discard block
 block discarded – undo
499 499
                 // Delete sharekey fields
500 500
                 $itemFields = DB::query(
501 501
                     'SELECT id
502
-                    FROM ' . prefixTable('categories_items') . '
502
+                    FROM ' . prefixTable('categories_items').'
503 503
                     WHERE item_id = %i',
504 504
                     $itemId
505 505
                 );
@@ -514,7 +514,7 @@  discard block
 block discarded – undo
514 514
                 // Delete sharekey files
515 515
                 $itemFiles = DB::query(
516 516
                     'SELECT id
517
-                    FROM ' . prefixTable('files') . '
517
+                    FROM ' . prefixTable('files').'
518 518
                     WHERE id_item = %i',
519 519
                     $itemId
520 520
                 );
@@ -591,7 +591,7 @@  discard block
 block discarded – undo
591 591
             ) {
592 592
                 if ($post_log_type === 'items') {
593 593
                     DB::query(
594
-                        'SELECT * FROM ' . prefixTable('log_items') . '
594
+                        'SELECT * FROM '.prefixTable('log_items').'
595 595
                         WHERE (date BETWEEN %i AND %i)'
596 596
                         . ($post_filter_action === 'all' ? '' : ' AND action = "'.$post_filter_action.'"')
597 597
                         . ((int) $post_filter_user === -1 ? '' : ' AND id_user = '.(int) $post_filter_user),
@@ -611,7 +611,7 @@  discard block
 block discarded – undo
611 611
                 } elseif ($post_log_type === 'connections') {
612 612
                     //db::debugmode(true);
613 613
                     DB::query(
614
-                        'SELECT * FROM ' . prefixTable('log_system') . '
614
+                        'SELECT * FROM '.prefixTable('log_system').'
615 615
                         WHERE type=%s '
616 616
                         . 'AND (date BETWEEN %i AND %i)'
617 617
                         . ($post_filter_action === 'all' ? '' : ' AND action = '.$post_filter_action)
@@ -634,7 +634,7 @@  discard block
 block discarded – undo
634 634
                     );
635 635
                 } elseif ($post_log_type === 'errors') {
636 636
                     DB::query(
637
-                        'SELECT * FROM ' . prefixTable('log_system') . ' WHERE type=%s ' .
637
+                        'SELECT * FROM '.prefixTable('log_system').' WHERE type=%s '.
638 638
                             'AND (date BETWEEN %i AND %i)',
639 639
                         'error',
640 640
                         $post_date_from,
@@ -651,7 +651,7 @@  discard block
 block discarded – undo
651 651
                     );
652 652
                 } elseif ($post_log_type === 'copy') {
653 653
                     DB::query(
654
-                        'SELECT * FROM ' . prefixTable('log_items') . ' WHERE action=%s ' .
654
+                        'SELECT * FROM '.prefixTable('log_items').' WHERE action=%s '.
655 655
                             'AND (date BETWEEN %i AND %i)',
656 656
                         'at_copy',
657 657
                         $post_date_from,
@@ -668,7 +668,7 @@  discard block
 block discarded – undo
668 668
                     );
669 669
                 } elseif ($post_log_type === 'admin') {
670 670
                     DB::query(
671
-                        'SELECT * FROM ' . prefixTable('log_system') . ' WHERE type=%s ' .
671
+                        'SELECT * FROM '.prefixTable('log_system').' WHERE type=%s '.
672 672
                             'AND (date BETWEEN %i AND %i)',
673 673
                         'admin_action',
674 674
                         $post_date_from,
@@ -685,7 +685,7 @@  discard block
 block discarded – undo
685 685
                     );
686 686
                 } elseif ($post_log_type === 'failed') {
687 687
                     DB::query(
688
-                        'SELECT * FROM ' . prefixTable('log_system') . ' WHERE type=%s ' .
688
+                        'SELECT * FROM '.prefixTable('log_system').' WHERE type=%s '.
689 689
                             'AND (date BETWEEN %i AND %i)',
690 690
                         'failed_auth',
691 691
                         $post_date_from,
@@ -756,19 +756,19 @@  discard block
 block discarded – undo
756 756
             // Get info about task
757 757
             $taskInfo = DB::queryFirstRow(
758 758
                 'SELECT p.process_type as process_type
759
-                FROM ' . prefixTable('background_tasks') . ' as p
759
+                FROM ' . prefixTable('background_tasks').' as p
760 760
                 WHERE p.increment_id = %i',
761 761
                 $post_id
762 762
             );
763 763
             if ($taskInfo !== null) {
764 764
                 // delete task
765 765
                 DB::query(
766
-                    'DELETE FROM ' . prefixTable('background_subtasks') . '
766
+                    'DELETE FROM '.prefixTable('background_subtasks').'
767 767
                     WHERE task_id = %i',
768 768
                     $post_id
769 769
                 );
770 770
                 DB::query(
771
-                    'DELETE FROM ' . prefixTable('background_tasks') . '
771
+                    'DELETE FROM '.prefixTable('background_tasks').'
772 772
                     WHERE increment_id = %i',
773 773
                     $post_id
774 774
                 );
@@ -835,7 +835,7 @@  discard block
 block discarded – undo
835 835
                         ->setDayOfMonth('*')
836 836
                         ->setMonths('*')
837 837
                         ->setDayOfWeek('*')
838
-                        ->setTaskCommandLine($phpBinaryPath . ' ' . $SETTINGS['cpassman_dir'] . '/sources/scheduler.php')
838
+                        ->setTaskCommandLine($phpBinaryPath.' '.$SETTINGS['cpassman_dir'].'/sources/scheduler.php')
839 839
                         ->setComments('Teampass scheduler');
840 840
                     
841 841
                     $crontabRepository->addJob($crontabJob);
Please login to merge, or discard this patch.
pages/users.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('users') === false) {
70 70
     // Not allowed page
71 71
     $session->set('system-error_code', ERR_NOT_ALLOWED);
72
-    include $SETTINGS['cpassman_dir'] . '/error.php';
72
+    include $SETTINGS['cpassman_dir'].'/error.php';
73 73
     exit;
74 74
 }
75 75
 
@@ -92,24 +92,24 @@  discard block
 block discarded – undo
92 92
 // If administrator then all roles are shown
93 93
 // else only the Roles the users is associated to.
94 94
 if ((int) $session->get('user-admin') === 1) {
95
-    $optionsManagedBy .= '<option value="0">' . $lang->get('administrators_only') . '</option>';
95
+    $optionsManagedBy .= '<option value="0">'.$lang->get('administrators_only').'</option>';
96 96
 }
97 97
 
98 98
 $rows = DB::query(
99 99
     'SELECT id, title, creator_id
100
-    FROM ' . prefixTable('roles_title') . '
100
+    FROM ' . prefixTable('roles_title').'
101 101
     ORDER BY title ASC'
102 102
 );
103 103
 foreach ($rows as $record) {
104 104
     if ((int) $session->get('user-admin') === 1 || in_array($record['id'], $session->get('user-roles_array')) === true) {
105
-        $optionsManagedBy .= '<option value="' . $record['id'] . '">' . $lang->get('managers_of') . ' ' . addslashes($record['title']) . '</option>';
105
+        $optionsManagedBy .= '<option value="'.$record['id'].'">'.$lang->get('managers_of').' '.addslashes($record['title']).'</option>';
106 106
     }
107 107
     if (
108 108
         (int) $session->get('user-admin') === 1
109 109
         || (((int) $session->get('user-manager') === 1 || (int) $session->get('user-can_manage_all_users') === 1)
110 110
             && (in_array($record['id'], $userRoles) === true) || (int) $record['creator_id'] === (int) $session->get('user-id'))
111 111
     ) {
112
-        $optionsRoles .= '<option value="' . $record['id'] . '">' . addslashes($record['title']) . '</option>';
112
+        $optionsRoles .= '<option value="'.$record['id'].'">'.addslashes($record['title']).'</option>';
113 113
     }
114 114
 }
115 115
 
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
         for ($y = 1; $y < $t->nlevel; ++$y) {
125 125
             $ident .= '&nbsp;&nbsp;';
126 126
         }
127
-        $foldersList .= '<option value="' . $t->id . '">' . $ident . htmlspecialchars($t->title, ENT_COMPAT, 'UTF-8') . '</option>';
127
+        $foldersList .= '<option value="'.$t->id.'">'.$ident.htmlspecialchars($t->title, ENT_COMPAT, 'UTF-8').'</option>';
128 128
     }
129 129
 }
130 130
 
@@ -161,13 +161,13 @@  discard block
 block discarded – undo
161 161
                         </button><?php
162 162
                                     echo isset($SETTINGS['ldap_mode']) === true && (int) $SETTINGS['ldap_mode'] === 1 && (int) $session->get('user-admin') === 1 ?
163 163
                                         '<button type="button" class="btn btn-primary btn-sm tp-action mr-2" data-action="ldap-sync">
164
-                            <i class="fa-solid fa-address-card mr-2"></i>' . $lang->get('ldap_synchronization') . '
164
+                            <i class="fa-solid fa-address-card mr-2"></i>' . $lang->get('ldap_synchronization').'
165 165
                         </button>' : '';
166 166
                                     ?>
167 167
                         </button><?php
168 168
                                     echo isset($SETTINGS['oauth2_enabled']) === true && (int) $SETTINGS['oauth2_enabled'] === 1 && (int) $session->get('user-admin') === 1 ?
169 169
                                         '<button type="button" class="btn btn-primary btn-sm tp-action mr-2" data-action="oauth2-sync">
170
-                            <i class="fa-solid fa-plug mr-2"></i>' . $lang->get('oauth2_synchronization') . '
170
+                            <i class="fa-solid fa-plug mr-2"></i>' . $lang->get('oauth2_synchronization').'
171 171
                         </button>' : '';
172 172
                                     ?>
173 173
                         <button type="button" class="btn btn-primary btn-sm tp-action mr-2" data-action="deleted-users">
@@ -177,7 +177,7 @@  discard block
 block discarded – undo
177 177
 
178 178
                 <!-- /.card-header -->
179 179
                 <div class="card-body form user-content with-header-menu" id="users-list" data-content="refresh">
180
-                    <label><input type="checkbox" id="warnings_display" class="tp-action pointer" data-action="refresh"><span class="ml-2 pointer"><?php echo $lang->get('display_warning_icons');?></span></label>
180
+                    <label><input type="checkbox" id="warnings_display" class="tp-action pointer" data-action="refresh"><span class="ml-2 pointer"><?php echo $lang->get('display_warning_icons'); ?></span></label>
181 181
                     <table id="table-users" class="table table-striped nowrap table-responsive-sm">
182 182
                         <thead>
183 183
                             <tr>
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
                                             <?php
242 242
                                             foreach (TP_PW_COMPLEXITY as $entry) {
243 243
                                                 echo '
244
-                                            <option value="' . $entry[0] . '">' . addslashes($entry[1]) . '</option>';
244
+                                            <option value="' . $entry[0].'">'.addslashes($entry[1]).'</option>';
245 245
                                             }
246 246
                                             ?>
247 247
                                         </select>
@@ -318,7 +318,7 @@  discard block
 block discarded – undo
318 318
                                             <?php
319 319
                                             foreach (TP_PW_COMPLEXITY as $entry) {
320 320
                                                 echo '
321
-                                            <option value="' . $entry[0] . '">' . addslashes($entry[1]) . '</option>';
321
+                                            <option value="' . $entry[0].'">'.addslashes($entry[1]).'</option>';
322 322
                                             }
323 323
                                             ?>
324 324
                                         </select>
Please login to merge, or discard this patch.
pages/items.js.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('items') === false) {
73 73
     // Not allowed page
74 74
     $session->set('system-error_code', ERR_NOT_ALLOWED);
75
-    include $SETTINGS['cpassman_dir'] . '/error.php';
75
+    include $SETTINGS['cpassman_dir'].'/error.php';
76 76
     exit;
77 77
 }
78 78
 
@@ -1076,7 +1076,7 @@  discard block
 block discarded – undo
1076 1076
             //
1077 1077
         } else if ($(this).data('item-action') === 'link') {
1078 1078
             // Add link to clipboard.
1079
-            navigator.clipboard.writeText("<?php echo $SETTINGS['cpassman_url'];?>/index.php?page=items&group="+store.get('teampassItem').folderId+"&id="+store.get('teampassItem').id);
1079
+            navigator.clipboard.writeText("<?php echo $SETTINGS['cpassman_url']; ?>/index.php?page=items&group="+store.get('teampassItem').folderId+"&id="+store.get('teampassItem').id);
1080 1080
 
1081 1081
             // Display message.
1082 1082
             toastr.remove();
@@ -2904,7 +2904,7 @@  discard block
 block discarded – undo
2904 2904
                 progress: "#form-item-password-strength",
2905 2905
                 score: "#form-item-password-strength"
2906 2906
             },
2907
-            scores: [<?php echo TP_PW_STRENGTH_1;?>, <?php echo TP_PW_STRENGTH_2;?>, <?php echo TP_PW_STRENGTH_3;?>, <?php echo TP_PW_STRENGTH_4;?>, <?php echo TP_PW_STRENGTH_5;?>],
2907
+            scores: [<?php echo TP_PW_STRENGTH_1; ?>, <?php echo TP_PW_STRENGTH_2; ?>, <?php echo TP_PW_STRENGTH_3; ?>, <?php echo TP_PW_STRENGTH_4; ?>, <?php echo TP_PW_STRENGTH_5; ?>],
2908 2908
         },
2909 2909
         i18n : {
2910 2910
             t: function (key) {
@@ -2931,7 +2931,7 @@  discard block
 block discarded – undo
2931 2931
     <?php
2932 2932
     $max_file_size = '';
2933 2933
     if (strrpos($SETTINGS['upload_maxfilesize'], 'mb') === false) {
2934
-        $max_file_size = $SETTINGS['upload_maxfilesize'] . 'mb';
2934
+        $max_file_size = $SETTINGS['upload_maxfilesize'].'mb';
2935 2935
     } else {
2936 2936
         $max_file_size = $SETTINGS['upload_maxfilesize'];
2937 2937
     }
@@ -3921,7 +3921,7 @@  discard block
 block discarded – undo
3921 3921
     function displaySubfolders(folders, parentId) {
3922 3922
         // Manage case where no folders
3923 3923
         if (folders === '' || folders === undefined) {
3924
-            $('#teampass_subfolders_list').html('<tr><td colspan="2" class="text-center text-muted"><?php echo $lang->get('no_folder_selected');?></td></tr>');
3924
+            $('#teampass_subfolders_list').html('<tr><td colspan="2" class="text-center text-muted"><?php echo $lang->get('no_folder_selected'); ?></td></tr>');
3925 3925
             return false;
3926 3926
         }
3927 3927
         
@@ -4912,7 +4912,7 @@  discard block
 block discarded – undo
4912 4912
                         (retData.edition_locked_delay === null ? 
4913 4913
                         '' 
4914 4914
                         :
4915
-                        ' : ' + retData.edition_locked_delay + ' <?php echo $lang->get('seconds');?>'),
4915
+                        ' : ' + retData.edition_locked_delay + ' <?php echo $lang->get('seconds'); ?>'),
4916 4916
                     '', {
4917 4917
                         timeOut: 5000,
4918 4918
                         progressBar: true
@@ -5069,7 +5069,7 @@  discard block
 block discarded – undo
5069 5069
 
5070 5070
                             // Show passwords inputs and form
5071 5071
                             $('#dialog-ldap-user-change-password-info')
5072
-                                .html('<i class="icon fa-solid fa-info mr-2"></i><?php echo $lang->get('ldap_user_has_changed_his_password');?>')
5072
+                                .html('<i class="icon fa-solid fa-info mr-2"></i><?php echo $lang->get('ldap_user_has_changed_his_password'); ?>')
5073 5073
                                 .removeClass('hidden');
5074 5074
                             $('#dialog-ldap-user-change-password').removeClass('hidden');
5075 5075
                         } else if (data.error_type !== 'undefined') {
@@ -5088,7 +5088,7 @@  discard block
 block discarded – undo
5088 5088
 
5089 5089
                                 // Show passwords inputs and form
5090 5090
                                 $('#dialog-ldap-user-change-password-info')
5091
-                                    .html('<i class="icon fa-solid fa-info mr-2"></i><?php echo $lang->get('ldap_user_has_changed_his_password');?>')
5091
+                                    .html('<i class="icon fa-solid fa-info mr-2"></i><?php echo $lang->get('ldap_user_has_changed_his_password'); ?>')
5092 5092
                                     .removeClass('hidden');
5093 5093
                                 $('#dialog-ldap-user-change-password').removeClass('hidden');
5094 5094
                             });
Please login to merge, or discard this patch.
sources/core.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -58,16 +58,16 @@  discard block
 block discarded – undo
58 58
 {
59 59
     // Load AntiXSS
60 60
     $antiXss = new AntiXSS();
61
-    if (! headers_sent()) {    //If headers not sent yet... then do php redirect
62
-        header('Location: ' . $antiXss->xss_clean($url));
61
+    if (!headers_sent()) {    //If headers not sent yet... then do php redirect
62
+        header('Location: '.$antiXss->xss_clean($url));
63 63
     }
64 64
 
65 65
     //If headers are sent... do java redirect... if java disabled, do html redirect.
66 66
     echo '<script type="text/javascript">';
67
-    echo 'window.location.href="' . $antiXss->xss_clean($url) . '";';
67
+    echo 'window.location.href="'.$antiXss->xss_clean($url).'";';
68 68
     echo '</script>';
69 69
     echo '<noscript>';
70
-    echo '<meta http-equiv="refresh" content="0;url=' . $antiXss->xss_clean($url) . '" />';
70
+    echo '<meta http-equiv="refresh" content="0;url='.$antiXss->xss_clean($url).'" />';
71 71
     echo '</noscript>';
72 72
 }
73 73
 
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
     && isset($SETTINGS['enable_sts']) === true
94 94
     && (int) $SETTINGS['enable_sts'] === 1
95 95
 ) {
96
-    teampassRedirect('https://' . $server['http_host'] . $server['request_uri']);
96
+    teampassRedirect('https://'.$server['http_host'].$server['request_uri']);
97 97
 }
98 98
 
99 99
 // Load pwComplexity
@@ -114,11 +114,11 @@  discard block
 block discarded – undo
114 114
 // LOAD CPASSMAN SETTINGS
115 115
 if (
116 116
     isset($SETTINGS['cpassman_dir']) === true
117
-    && is_dir($SETTINGS['cpassman_dir'] . '/install') === true
117
+    && is_dir($SETTINGS['cpassman_dir'].'/install') === true
118 118
 ) {
119 119
     // Should we delete folder INSTALL?
120 120
     $row = DB::queryFirstRow(
121
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s AND intitule=%s',
121
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s AND intitule=%s',
122 122
         'install',
123 123
         'clear_install_folder'
124 124
     );
@@ -136,11 +136,11 @@  discard block
 block discarded – undo
136 136
             if ($directories !== false) {
137 137
                 $files = array_diff($directories, ['.', '..']);
138 138
                 foreach ($files as $file) {
139
-                    if (is_dir($dir . '/' . $file)) {
140
-                        delTree($dir . '/' . $file);
139
+                    if (is_dir($dir.'/'.$file)) {
140
+                        delTree($dir.'/'.$file);
141 141
                     } else {
142 142
                         try {
143
-                            unlink($dir . '/' . $file);
143
+                            unlink($dir.'/'.$file);
144 144
                         } catch (Exception $e) {
145 145
                             // do nothing... php will ignore and continue
146 146
                         }
@@ -154,13 +154,13 @@  discard block
 block discarded – undo
154 154
             return false;
155 155
         }
156 156
 
157
-        if (is_dir($SETTINGS['cpassman_dir'] . '/install')) {
157
+        if (is_dir($SETTINGS['cpassman_dir'].'/install')) {
158 158
             // Set the permissions on the install directory and delete
159 159
             // is server Windows or Linux?
160 160
             if (strtoupper(substr(PHP_OS, 0, 3)) !== 'WIN') {
161
-                recursiveChmod($SETTINGS['cpassman_dir'] . '/install', 0755, 0440);
161
+                recursiveChmod($SETTINGS['cpassman_dir'].'/install', 0755, 0440);
162 162
             }
163
-            delTree($SETTINGS['cpassman_dir'] . '/install');
163
+            delTree($SETTINGS['cpassman_dir'].'/install');
164 164
         }
165 165
 
166 166
         // Delete temporary install table
@@ -178,10 +178,10 @@  discard block
 block discarded – undo
178 178
 // Load Languages stuff
179 179
 if (isset($languagesList) === false) {
180 180
     $languagesList = [];
181
-    $rows = DB::query('SELECT * FROM ' . prefixTable('languages') . ' GROUP BY name, label, code, flag, id ORDER BY name ASC');
181
+    $rows = DB::query('SELECT * FROM '.prefixTable('languages').' GROUP BY name, label, code, flag, id ORDER BY name ASC');
182 182
     foreach ($rows as $record) {
183 183
         array_push($languagesList, $record['name']);
184
-        if ($session->get('user-language') === $record['name'] ) {
184
+        if ($session->get('user-language') === $record['name']) {
185 185
             $session->set('user-language_flag', $record['flag']);
186 186
             $session->set('user-language_code', $record['code']);
187 187
             //$session->set('user-language_label', $record['label']);
@@ -238,7 +238,7 @@  discard block
 block discarded – undo
238 238
 // CHECK IF SESSION EXISTS AND IF SESSION IS VALID
239 239
 if (empty($session->get('user-session_duration')) === false) {
240 240
     $dataSession = DB::queryFirstRow(
241
-        'SELECT key_tempo, deleted_at FROM ' . prefixTable('users') . ' WHERE id=%i',
241
+        'SELECT key_tempo, deleted_at FROM '.prefixTable('users').' WHERE id=%i',
242 242
         $session->get('user-id')
243 243
     );
244 244
 } else {
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
     && ($session->has('user-admin') && $session->get('user-admin') && null !== $session->get('user-admin') && $session->get('user-admin') === 1)
296 296
 ) {
297 297
     $row = DB::queryFirstRow(
298
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s_type AND intitule=%s_intitule',
298
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s_type AND intitule=%s_intitule',
299 299
         [
300 300
             'type' => 'admin',
301 301
             'intitule' => 'teampass_version',
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
 
337 337
         syslog(
338 338
             LOG_WARNING,
339
-            'Unlog user: ' . date('Y/m/d H:i:s') . " {$server['remote_addr']} ({$server['http_user_agent']})"
339
+            'Unlog user: '.date('Y/m/d H:i:s')." {$server['remote_addr']} ({$server['http_user_agent']})"
340 340
         );
341 341
         // erase session table
342 342
         $session->invalidate();
@@ -390,7 +390,7 @@  discard block
 block discarded – undo
390 390
 if ($session->has('user-timezone') && null !== $session->get('user-id') && empty($session->get('user-id')) === false) {
391 391
     // query on user
392 392
     $data = DB::queryFirstRow(
393
-        'SELECT login, admin, gestionnaire, can_manage_all_users, groupes_visibles, groupes_interdits, fonction_id, last_connexion, roles_from_ad_groups, auth_type, last_pw_change FROM ' . prefixTable('users') . ' WHERE id=%i',
393
+        'SELECT login, admin, gestionnaire, can_manage_all_users, groupes_visibles, groupes_interdits, fonction_id, last_connexion, roles_from_ad_groups, auth_type, last_pw_change FROM '.prefixTable('users').' WHERE id=%i',
394 394
         $session->get('user-id')
395 395
     );
396 396
     //Check if user has been deleted or unlogged
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
             $data['groupes_visibles'],
438 438
             $data['groupes_interdits'],
439 439
             $data['admin'],
440
-            is_null($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : (empty($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : $data['fonction_id'] . ';' . $data['roles_from_ad_groups']),
440
+            is_null($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : (empty($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : $data['fonction_id'].';'.$data['roles_from_ad_groups']),
441 441
             $SETTINGS
442 442
         );
443 443
         if ($session->has('user-can_create_root_folder') && (int) $session->get('user-can_create_root_folder') && null !== $session->get('user-can_create_root_folder') && (int) $session->get('user-can_create_root_folder') === 1) {
@@ -472,7 +472,7 @@  discard block
 block discarded – undo
472 472
     $session->set('system-item_fields', []);
473 473
     $rows = DB::query(
474 474
         'SELECT *
475
-            FROM ' . prefixTable('categories') . '
475
+            FROM ' . prefixTable('categories').'
476 476
             WHERE level=%i',
477 477
         '0'
478 478
     );
@@ -481,7 +481,7 @@  discard block
 block discarded – undo
481 481
         // get each field
482 482
         $rows2 = DB::query(
483 483
             'SELECT *
484
-            FROM ' . prefixTable('categories') . '
484
+            FROM ' . prefixTable('categories').'
485 485
             WHERE parent_id=%i
486 486
             ORDER BY `order` ASC',
487 487
             $record['id']
@@ -568,5 +568,5 @@  discard block
 block discarded – undo
568 568
 }
569 569
 
570 570
 /* CHECK NUMBER OF USER ONLINE */
571
-DB::query('SELECT * FROM ' . prefixTable('users') . ' WHERE timestamp>=%i', time() - 600);
571
+DB::query('SELECT * FROM '.prefixTable('users').' WHERE timestamp>=%i', time() - 600);
572 572
 $session->set('system-nb_users_online', DB::count());
Please login to merge, or discard this patch.