Passed
Pull Request — master (#4596)
by
unknown
07:11
created
pages/admin.php 1 patch
Spacing   +11 added lines, -11 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
 
@@ -199,7 +199,7 @@  discard block
 block discarded – undo
199 199
     // Get last cron execution timestamp
200 200
     DB::query(
201 201
         'SELECT valeur
202
-        FROM ' . prefixTable('misc') . '
202
+        FROM ' . prefixTable('misc').'
203 203
         WHERE type = %s AND intitule = %s and valeur >= %d',
204 204
         'admin',
205 205
         'last_cron_exec',
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
                     <div class="card-body">
271 271
                         <?php
272 272
                         // Display information about server
273
-                        $dbSize = DB::queryFirstRow("SELECT ROUND(SUM(data_length + index_length) / 1024 / 1024, 2) AS 'size' FROM information_schema.TABLES WHERE table_schema = '" . DB_NAME . "'");
273
+                        $dbSize = DB::queryFirstRow("SELECT ROUND(SUM(data_length + index_length) / 1024 / 1024, 2) AS 'size' FROM information_schema.TABLES WHERE table_schema = '".DB_NAME."'");
274 274
 
275 275
                         // Get OS
276 276
                         $uname = php_uname('s');
@@ -326,17 +326,17 @@  discard block
 block discarded – undo
326 326
                         }
327 327
 
328 328
                         echo 
329
-                        '<p>' . $os.
329
+                        '<p>'.$os.
330 330
                             '<br><span class="ml-4"></span>'.
331 331
                         '</p>'.
332 332
                         $internetAccess.
333
-                        '<p><i class="fa-brands fa-php mr-2"></i>PHP version: ' . phpversion().
333
+                        '<p><i class="fa-brands fa-php mr-2"></i>PHP version: '.phpversion().
334 334
                             '<br><span class="ml-4">Memory limit: '.(ini_get('memory_limit')).'</span>'.
335 335
                             '<br><span class="ml-4">Memory usage: '.formatSizeUnits(memory_get_usage()).'</span>'.
336 336
                             '<br><span class="ml-4">Maximum time execution: '.ini_get('max_execution_time').'</span>'.
337 337
                             '<br><span class="ml-4">Maximum file size upload: '.ini_get('upload_max_filesize').'</span>'.
338 338
                         '</p>'.
339
-                        '<p><i class="fa-solid fa-server mr-2"></i>Server version: ' . DB::serverVersion().
339
+                        '<p><i class="fa-solid fa-server mr-2"></i>Server version: '.DB::serverVersion().
340 340
                             '<br><span class="ml-4">Database size: '.($dbSize['size']).'MB</span>'.
341 341
                         '</p>';
342 342
 
@@ -344,11 +344,11 @@  discard block
 block discarded – undo
344 344
                         $serverTime = localtime(time(), true);
345 345
                         echo '<div class="row">'.
346 346
                             '<div class="col-6"><i class="fa-solid fa-clock mr-2"></i>Server time:</div>'.
347
-                            '<div class="col-6"><span class="badge badge-info">' . 
348
-                            str_pad(strval($serverTime['tm_hour']), 2, "0", STR_PAD_LEFT) . ':' .
349
-                            str_pad(strval($serverTime['tm_min']), 2, "0", STR_PAD_LEFT) . ':' .
350
-                            str_pad(strval($serverTime['tm_sec']), 2, "0", STR_PAD_LEFT) .
351
-                            '</span></div>' .
347
+                            '<div class="col-6"><span class="badge badge-info">'. 
348
+                            str_pad(strval($serverTime['tm_hour']), 2, "0", STR_PAD_LEFT).':'.
349
+                            str_pad(strval($serverTime['tm_min']), 2, "0", STR_PAD_LEFT).':'.
350
+                            str_pad(strval($serverTime['tm_sec']), 2, "0", STR_PAD_LEFT).
351
+                            '</span></div>'.
352 352
                         '</div>'.
353 353
                         '<div class="row">'.
354 354
                             '<div class="col-6"><span class="ml-4">Timezone:</span></div>'.
Please login to merge, or discard this patch.
pages/tasks.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('tasks') === false) {
65 65
     // Not allowed page
66 66
     $session->set('system-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
 
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
     // Get last cron execution timestamp
127 127
     DB::query(
128 128
         'SELECT valeur
129
-        FROM ' . prefixTable('misc') . '
129
+        FROM ' . prefixTable('misc').'
130 130
         WHERE type = %s AND intitule = %s and valeur >= %d',
131 131
         'admin',
132 132
         'last_cron_exec',
@@ -234,7 +234,7 @@  discard block
 block discarded – undo
234 234
                                             $task = isset($SETTINGS['users_personal_folder_task']) === true ? explode(";", $SETTINGS['users_personal_folder_task']) : [];
235 235
                                             ?>
236 236
                                             <input type='text' disabled class='form-control form-control-sm' id='users_personal_folder_task_parameter' value='<?php echo isset($task[0]) === true && empty($task[0]) === false ? $lang->get($task[0])." ".(isset($task[2]) === true ? strtolower($lang->get('day')).' '.$task[2].' ' : '').$lang->get('at')." ".(isset($task[1]) === true ? $task[1] : '') : $lang->get('not_defined') ?>'>
237
-                                            <input type='hidden' disabled class='form-control form-control-sm' id='users_personal_folder_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : '';?>'>
237
+                                            <input type='hidden' disabled class='form-control form-control-sm' id='users_personal_folder_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : ''; ?>'>
238 238
                                         </div>
239 239
                                         <div class='col-2'>
240 240
                                             <button class="btn btn-primary task-define" data-task="users_personal_folder_task">
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
                                             $task = isset($SETTINGS['clean_orphan_objects_task']) === true ? explode(";", $SETTINGS['clean_orphan_objects_task']) : [];
257 257
                                             ?>
258 258
                                             <input type='text' disabled class='form-control form-control-sm' id='clean_orphan_objects_task_parameter' value='<?php echo isset($task[0]) === true && empty($task[0]) === false ? $lang->get($task[0])." ".(isset($task[2]) === true ? strtolower($lang->get('day')).' '.$task[2].' ' : '').$lang->get('at')." ".(isset($task[1]) === true ? $task[1] : '') : $lang->get('not_defined') ?>'>
259
-                                            <input type='hidden' disabled class='form-control form-control-sm' id='clean_orphan_objects_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : '';?>'>
259
+                                            <input type='hidden' disabled class='form-control form-control-sm' id='clean_orphan_objects_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : ''; ?>'>
260 260
                                         </div>
261 261
                                         <div class='col-2'>
262 262
                                             <button class="btn btn-primary task-define" data-task="clean_orphan_objects_task">
@@ -278,7 +278,7 @@  discard block
 block discarded – undo
278 278
                                             $task = isset($SETTINGS['purge_temporary_files_task']) === true ? explode(";", $SETTINGS['purge_temporary_files_task']) : [];
279 279
                                             ?>
280 280
                                             <input type='text' disabled class='form-control form-control-sm' id='purge_temporary_files_task_parameter' value='<?php echo isset($task[0]) === true && empty($task[0]) === false ? $lang->get($task[0])." ".(isset($task[2]) === true ? strtolower($lang->get('day')).' '.$task[2].' ' : '').$lang->get('at')." ".(isset($task[1]) === true ? $task[1] : '') : $lang->get('not_defined') ?>'>
281
-                                            <input type='hidden' disabled class='form-control form-control-sm' id='purge_temporary_files_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : '';?>'>
281
+                                            <input type='hidden' disabled class='form-control form-control-sm' id='purge_temporary_files_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : ''; ?>'>
282 282
                                         </div>
283 283
                                         <div class='col-2'>
284 284
                                             <button class="btn btn-primary task-define" data-task="purge_temporary_files_task">
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
                                             $task = isset($SETTINGS['reload_cache_table_task']) === true ? explode(";", $SETTINGS['reload_cache_table_task']) : [];
301 301
                                             ?>
302 302
                                             <input type='text' disabled class='form-control form-control-sm' id='reload_cache_table_task_parameter' value='<?php echo isset($task[0]) === true && empty($task[0]) === false ? $lang->get($task[0])." ".(isset($task[2]) === true ? strtolower($lang->get('day')).' '.$task[2].' ' : '').$lang->get('at')." ".(isset($task[1]) === true ? $task[1] : '') : $lang->get('not_defined') ?>'>
303
-                                            <input type='hidden' disabled class='form-control form-control-sm' id='reload_cache_table_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : '';?>'>
303
+                                            <input type='hidden' disabled class='form-control form-control-sm' id='reload_cache_table_task_parameter_value' value='<?php echo isset($task[0]) === true ? $task[0].";".(isset($task[1]) === true ? $task[1] : '').(isset($task[2]) === true ? $task[2] : '') : ''; ?>'>
304 304
                                         </div>
305 305
                                         <div class='col-2'>
306 306
                                             <button class="btn btn-primary task-define" data-task="reload_cache_table_task">
@@ -517,7 +517,7 @@  discard block
 block discarded – undo
517 517
                         <h5><?php echo $lang->get('day_of_month'); ?></h5>              
518 518
                         <select class='form-control form-control-sm no-save' id='task-define-modal-parameter-monthly-value' style="width:100%;">
519 519
                             <?php
520
-                            for ($i=1; $i<=31; $i++) {
520
+                            for ($i = 1; $i <= 31; $i++) {
521 521
                                 echo '<option value="'.$i.'">'.$lang->get('day').' '.$i.'</option>';
522 522
                             }
523 523
                             ?>
Please login to merge, or discard this patch.
scripts/background_tasks___user_task.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
     // launched several times.
70 70
     $record = DB::queryFirstRow(
71 71
         'SELECT *
72
-        FROM ' . prefixTable('background_tasks') . '
72
+        FROM ' . prefixTable('background_tasks').'
73 73
         WHERE is_in_progress = %i AND process_type = %s
74 74
         ORDER BY increment_id ASC LIMIT 1',
75 75
         0,
Please login to merge, or discard this patch.
pages/oauth.js.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('oauth') === false) {
72 72
     // Not allowed page
73 73
     $session->set('system-error_code', ERR_NOT_ALLOWED);
74
-    include $SETTINGS['cpassman_dir'] . '/error.php';
74
+    include $SETTINGS['cpassman_dir'].'/error.php';
75 75
     exit;
76 76
 }
77 77
 ?>
@@ -215,7 +215,7 @@  discard block
 block discarded – undo
215 215
             }).fail(function(error) {
216 216
                 toastr.remove();
217 217
                 toastr.error(
218
-                    '<?php echo $lang->get('server_answer_error') . '<br />' . $lang->get('server_returned_data') . ':<br />'; ?>' + error,
218
+                    '<?php echo $lang->get('server_answer_error').'<br />'.$lang->get('server_returned_data').':<br />'; ?>' + error,
219 219
                     '', {
220 220
                         closeButton: true,
221 221
                         positionClass: 'toast-bottom-right'
Please login to merge, or discard this patch.
sources/oauth.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2,7 +2,7 @@
 block discarded – undo
2 2
 use TeampassClasses\OAuth2Controller\OAuth2Controller;
3 3
 use TeampassClasses\SessionManager\SessionManager;
4 4
 
5
-require_once __DIR__. '/../includes/config/include.php';
5
+require_once __DIR__.'/../includes/config/include.php';
6 6
 require_once __DIR__.'/../sources/main.functions.php';
7 7
 
8 8
 // init
Please login to merge, or discard this patch.
includes/core/login.oauth2.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@
 block discarded – undo
32 32
 use TeampassClasses\OAuth2Controller\OAuth2Controller;
33 33
 use TeampassClasses\SessionManager\SessionManager;
34 34
 
35
-require_once __DIR__. '/../../includes/config/include.php';
35
+require_once __DIR__.'/../../includes/config/include.php';
36 36
 require_once __DIR__.'/../../sources/main.functions.php';
37 37
 
38 38
 // init
Please login to merge, or discard this patch.
scripts/task_maintenance_purge_old_files.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
 function purgeTemporaryFiles(): void
72 72
 {
73 73
     // Load expected files
74
-    require_once __DIR__. '/../sources/main.functions.php';
74
+    require_once __DIR__.'/../sources/main.functions.php';
75 75
 
76 76
     // $SETTINGS is set then read folder
77 77
     if (is_dir($SETTINGS['path_to_files_folder']) === true) {
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
             //delete file FILES
81 81
             while (false !== ($f = readdir($dir))) {
82 82
                 if ($f !== '.' && $f !== '..' && $f !== '.htaccess') {
83
-                    $filePath = $SETTINGS['path_to_files_folder'] . '/' . $f;
83
+                    $filePath = $SETTINGS['path_to_files_folder'].'/'.$f;
84 84
                     if (file_exists($filePath) && ((time() - filectime($filePath)) > 604800)) {
85 85
                         fileDelete($filePath, $SETTINGS);
86 86
                     }
@@ -101,7 +101,7 @@  discard block
 block discarded – undo
101 101
             while (false !== ($f = readdir($dir))) {
102 102
                 if ($f !== '.' && $f !== '..') {
103 103
                     if (strpos($f, '_delete.') > 0) {
104
-                        fileDelete($SETTINGS['path_to_upload_folder'] . '/' . $f, $SETTINGS);
104
+                        fileDelete($SETTINGS['path_to_upload_folder'].'/'.$f, $SETTINGS);
105 105
                     }
106 106
                 }
107 107
             }
Please login to merge, or discard this patch.
sources/tree.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@  discard block
 block discarded – undo
64 64
 ) {
65 65
     // Not allowed page
66 66
     $session->set('system-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
 
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
 );
126 126
 
127 127
 $lastFolderChange = DB::queryfirstrow(
128
-    'SELECT valeur FROM ' . prefixTable('misc') . '
128
+    'SELECT valeur FROM '.prefixTable('misc').'
129 129
     WHERE type = %s AND intitule = %s',
130 130
     'timestamp',
131 131
     'last_folder_change'
@@ -346,7 +346,7 @@  discard block
 block discarded – undo
346 346
         $nbItemsInSubfolders = (int) $node->nb_items_in_subfolders;
347 347
         $nbItemsInFolder = (int) $node->nb_items_in_folder;
348 348
         $nbSubfolders = (int) $node->nb_subfolders;
349
-        break;  // Get out as soon as we find a valid node.
349
+        break; // Get out as soon as we find a valid node.
350 350
     }
351 351
     
352 352
     if ($displayThisNode === true) {
@@ -458,7 +458,7 @@  discard block
 block discarded – undo
458 458
         foreach ($currentNode->children as $child) {
459 459
             recursiveTree(
460 460
                 (int) $child,
461
-                $tree->getNode($child),// get node info for this child
461
+                $tree->getNode($child), // get node info for this child
462 462
                 /** @scrutinizer ignore-type */ $tree,
463 463
                 $listFoldersLimitedKeys,
464 464
                 $listRestrictedFoldersForItemsKeys,
@@ -505,7 +505,7 @@  discard block
 block discarded – undo
505 505
     $lang = new Language($session->get('user-language') ?? 'english');
506 506
 
507 507
     // prepare json return for current node
508
-    $parent = $currentNode->parent_id === '0' ? '#' : 'li_' . $currentNode->parent_id;
508
+    $parent = $currentNode->parent_id === '0' ? '#' : 'li_'.$currentNode->parent_id;
509 509
 
510 510
     // handle displaying
511 511
     if (isKeyExistingAndEqual('show_only_accessible_folders', 1, $SETTINGS) === true) {
@@ -522,17 +522,17 @@  discard block
 block discarded – undo
522 522
         array_push(
523 523
             $ret_json,
524 524
             array(
525
-                'id' => 'li_' . $nodeId,
525
+                'id' => 'li_'.$nodeId,
526 526
                 'parent' => $last_visible_parent === -1 ? $parent : $last_visible_parent,
527 527
                 'text' => '<i class="'.$currentNode->fa_icon.' tree-folder mr-2" data-folder="'.$currentNode->fa_icon.'"  data-folder-selected="'.$currentNode->fa_icon_selected.'"></i>'.$text.htmlspecialchars($currentNode->title).$nodeData['html'],
528 528
                 'li_attr' => array(
529 529
                     'class' => 'jstreeopen',
530
-                    'title' => 'ID [' . $nodeId . '] ' . $nodeData['title'],
530
+                    'title' => 'ID ['.$nodeId.'] '.$nodeData['title'],
531 531
                 ),
532 532
                 'a_attr' => array(
533
-                    'id' => 'fld_' . $nodeId,
533
+                    'id' => 'fld_'.$nodeId,
534 534
                     'class' => $nodeData['folderClass'],
535
-                    'onclick' => 'ListerItems(' . $nodeId . ', ' . $nodeData['restricted'] . ', 0, 1)',
535
+                    'onclick' => 'ListerItems('.$nodeId.', '.$nodeData['restricted'].', 0, 1)',
536 536
                     'data-title' => htmlspecialchars($currentNode->title),
537 537
                 ),
538 538
                 'is_pf' => in_array($nodeId, $inputData['personalFolders']) === true ? 1 : 0,
@@ -548,12 +548,12 @@  discard block
 block discarded – undo
548 548
         array_push(
549 549
             $ret_json,
550 550
             array(
551
-                'id' => 'li_' . $nodeId,
551
+                'id' => 'li_'.$nodeId,
552 552
                 'parent' => $last_visible_parent === -1 ? $parent : $last_visible_parent,
553 553
                 'text' => '<i class="'.$currentNode->fa_icon.' tree-folder mr-2" data-folder="'.$currentNode->fa_icon.'"  data-folder-selected="'.$currentNode->fa_icon_selected.'"></i>'.'<i class="fas fa-times fa-xs text-danger mr-1 ml-1"></i>'.$text.htmlspecialchars($currentNode->title).$nodeData['html'],
554 554
                 'li_attr' => array(
555 555
                     'class' => '',
556
-                    'title' => 'ID [' . $nodeId . '] ' . $lang->get('no_access'),
556
+                    'title' => 'ID ['.$nodeId.'] '.$lang->get('no_access'),
557 557
                 ),
558 558
             )
559 559
         );
@@ -613,7 +613,7 @@  discard block
 block discarded – undo
613 613
         if (in_array($nodeId, $session_read_only_folders) === true) {
614 614
             return [
615 615
                 'html' => '<i class="far fa-eye fa-xs mr-1 ml-1"></i>'.
616
-                    ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_' . $nodeId . '">' . $nbItemsInFolder .'/'.$nbItemsInSubfolders .'/'.$nbSubfolders. '</span>'  : ''),
616
+                    ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_'.$nodeId.'">'.$nbItemsInFolder.'/'.$nbItemsInSubfolders.'/'.$nbSubfolders.'</span>' : ''),
617 617
                 'title' => $lang->get('read_only_account'),
618 618
                 'restricted' => 1,
619 619
                 'folderClass' => 'folder_not_droppable',
@@ -628,7 +628,7 @@  discard block
 block discarded – undo
628 628
         ) {
629 629
             return [
630 630
                 'html' => '<i class="far fa-eye fa-xs mr-1"></i>'.
631
-                    ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_' . $nodeId . '">' . $nbItemsInFolder .'/'.$nbItemsInSubfolders .'/'.$nbSubfolders. '</span>'  : ''),
631
+                    ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_'.$nodeId.'">'.$nbItemsInFolder.'/'.$nbItemsInSubfolders.'/'.$nbSubfolders.'</span>' : ''),
632 632
                 'title' => $lang->get('read_only_account'),
633 633
                 'restricted' => 0,
634 634
                 'folderClass' => 'folder',
@@ -639,7 +639,7 @@  discard block
 block discarded – undo
639 639
         }
640 640
         
641 641
         return [
642
-            'html' => ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_' . $nodeId . '">' . $nbItemsInFolder .'/'.$nbItemsInSubfolders .'/'.$nbSubfolders. '</span>'  : ''),
642
+            'html' => ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_'.$nodeId.'">'.$nbItemsInFolder.'/'.$nbItemsInSubfolders.'/'.$nbSubfolders.'</span>' : ''),
643 643
             'title' => '',
644 644
             'restricted' => 0,
645 645
             'folderClass' => 'folder',
@@ -650,8 +650,8 @@  discard block
 block discarded – undo
650 650
 
651 651
     } elseif (in_array($nodeId, $listFoldersLimitedKeys) === true) {
652 652
         return [
653
-            'html' => ($session_user_read_only === true ? '<i class="far fa-eye fa-xs mr-1"></i>' : '') .
654
-                ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_' . $nodeId . '">' . count($session_list_folders_limited[$nodeId]) . '</span>' : ''),
653
+            'html' => ($session_user_read_only === true ? '<i class="far fa-eye fa-xs mr-1"></i>' : '').
654
+                ($tree_counters === 1 ? '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_'.$nodeId.'">'.count($session_list_folders_limited[$nodeId]).'</span>' : ''),
655 655
             'title' => '',
656 656
             'restricted' => 1,
657 657
             'folderClass' => 'folder',
@@ -662,8 +662,8 @@  discard block
 block discarded – undo
662 662
 
663 663
     } elseif (in_array($nodeId, $listRestrictedFoldersForItemsKeys) === true) {
664 664
         return [
665
-            'html' => $session_user_read_only === true ? '<i class="far fa-eye fa-xs mr-1"></i>' : '' .
666
-                '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_' . $nodeId . '">' . count($session_list_restricted_folders_for_items[$nodeId]) . '</span>',
665
+            'html' => $session_user_read_only === true ? '<i class="far fa-eye fa-xs mr-1"></i>' : ''.
666
+                '<span class="badge badge-pill badge-light ml-2 items_count" id="itcount_'.$nodeId.'">'.count($session_list_restricted_folders_for_items[$nodeId]).'</span>',
667 667
             'title' => '',
668 668
             'restricted' => 1,
669 669
             'folderClass' => 'folder',
@@ -773,7 +773,7 @@  discard block
 block discarded – undo
773 773
     // Does this user has a tree cache
774 774
     $userCacheTree = DB::queryfirstrow(
775 775
         'SELECT data
776
-        FROM ' . prefixTable('cache_tree') . '
776
+        FROM ' . prefixTable('cache_tree').'
777 777
         WHERE user_id = %i',
778 778
         $userId
779 779
     );
Please login to merge, or discard this patch.
sources/users.queries.php 1 patch
Spacing   +80 added lines, -80 removed lines patch added patch discarded remove patch
@@ -73,7 +73,7 @@  discard block
 block discarded – undo
73 73
 if ($checkUserAccess->checkSession() === false || $checkUserAccess->userAccessPage('profile') === false) {
74 74
     // Not allowed page
75 75
     $session->set('system-error_code', ERR_NOT_ALLOWED);
76
-    include $SETTINGS['cpassman_dir'] . '/error.php';
76
+    include $SETTINGS['cpassman_dir'].'/error.php';
77 77
     exit;
78 78
 }
79 79
 
@@ -144,7 +144,7 @@  discard block
 block discarded – undo
144 144
 
145 145
         // Get info about user to modify
146 146
         $targetUserInfos = DB::queryfirstrow(
147
-            'SELECT admin, gestionnaire, can_manage_all_users, isAdministratedByRole FROM ' . prefixTable('users') . '
147
+            'SELECT admin, gestionnaire, can_manage_all_users, isAdministratedByRole FROM '.prefixTable('users').'
148 148
             WHERE id = %i',
149 149
             (int) $dataReceived['user_id']
150 150
         );
@@ -264,7 +264,7 @@  discard block
 block discarded – undo
264 264
             // Check if user already exists
265 265
             $data = DB::query(
266 266
                 'SELECT id, fonction_id, groupes_interdits, groupes_visibles
267
-                FROM ' . prefixTable('users') . '
267
+                FROM ' . prefixTable('users').'
268 268
                 WHERE login = %s
269 269
                 AND deleted_at IS NULL',
270 270
                 $login
@@ -483,7 +483,7 @@  discard block
 block discarded – undo
483 483
 
484 484
             // Get info about user to delete
485 485
             $data_user = DB::queryfirstrow(
486
-                'SELECT login, admin, isAdministratedByRole FROM ' . prefixTable('users') . '
486
+                'SELECT login, admin, isAdministratedByRole FROM '.prefixTable('users').'
487 487
                 WHERE id = %i',
488 488
                 $post_id
489 489
             );
@@ -517,7 +517,7 @@  discard block
 block discarded – undo
517 517
                 );
518 518
                 // delete personal folder and subfolders
519 519
                 $data = DB::queryfirstrow(
520
-                    'SELECT id FROM ' . prefixTable('nested_tree') . '
520
+                    'SELECT id FROM '.prefixTable('nested_tree').'
521 521
                     WHERE title = %s AND personal_folder = %i',
522 522
                     $post_id,
523 523
                     '1'
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
                         DB::delete(prefixTable('nested_tree'), 'id = %i AND personal_folder = %i', $folder->id, '1');
531 531
                         // delete items & logs
532 532
                         $items = DB::query(
533
-                            'SELECT id FROM ' . prefixTable('items') . '
533
+                            'SELECT id FROM '.prefixTable('items').'
534 534
                             WHERE id_tree=%i AND perso = %i',
535 535
                             $folder->id,
536 536
                             '1'
@@ -554,7 +554,7 @@  discard block
 block discarded – undo
554 554
                 // Delete any process related to user
555 555
                 $processes = DB::query(
556 556
                     'SELECT increment_id
557
-                    FROM ' . prefixTable('background_tasks') . '
557
+                    FROM ' . prefixTable('background_tasks').'
558 558
                     WHERE JSON_EXTRACT(arguments, "$.new_user_id") = %i',
559 559
                     $post_id
560 560
                 );
@@ -607,7 +607,7 @@  discard block
 block discarded – undo
607 607
             $return = array();
608 608
             // Check if folder exists
609 609
             $data = DB::query(
610
-                'SELECT * FROM ' . prefixTable('nested_tree') . '
610
+                'SELECT * FROM '.prefixTable('nested_tree').'
611 611
                 WHERE title = %s AND parent_id = %i',
612 612
                 filter_input(INPUT_POST, 'domain', FILTER_SANITIZE_FULL_SPECIAL_CHARS),
613 613
                 '0'
@@ -620,7 +620,7 @@  discard block
 block discarded – undo
620 620
             }
621 621
             // Check if role exists
622 622
             $data = DB::query(
623
-                'SELECT * FROM ' . prefixTable('roles_title') . '
623
+                'SELECT * FROM '.prefixTable('roles_title').'
624 624
                 WHERE title = %s',
625 625
                 filter_input(INPUT_POST, 'domain', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
626 626
             );
@@ -649,7 +649,7 @@  discard block
 block discarded – undo
649 649
             // Get info about user to delete
650 650
             $data_user = DB::queryfirstrow(
651 651
                 'SELECT admin, isAdministratedByRole, gestionnaire
652
-                FROM ' . prefixTable('users') . '
652
+                FROM ' . prefixTable('users').'
653 653
                 WHERE id = %i',
654 654
                 $post_user_id
655 655
             );
@@ -705,7 +705,7 @@  discard block
 block discarded – undo
705 705
             // Get info about user
706 706
             $rowUser = DB::queryfirstrow(
707 707
                 'SELECT *
708
-                FROM ' . prefixTable('users') . '
708
+                FROM ' . prefixTable('users').'
709 709
                 WHERE id = %i',
710 710
                 $post_id
711 711
             );
@@ -730,7 +730,7 @@  discard block
 block discarded – undo
730 730
                 $selected = '';
731 731
                 $users_functions = array_filter(array_unique(explode(';', empty($rowUser['fonction_id'].';'.$rowUser['roles_from_ad_groups']) === true ? '' : $rowUser['fonction_id'].';'.$rowUser['roles_from_ad_groups'])));
732 732
 
733
-                $rows = DB::query('SELECT id,title,creator_id FROM ' . prefixTable('roles_title'));
733
+                $rows = DB::query('SELECT id,title,creator_id FROM '.prefixTable('roles_title'));
734 734
                 foreach ($rows as $record) {
735 735
                     if (
736 736
                         (int) $session->get('user-admin') === 1
@@ -765,7 +765,7 @@  discard block
 block discarded – undo
765 765
                 $rolesList = array();
766 766
                 $managedBy = array();
767 767
                 $selected = '';
768
-                $rows = DB::query('SELECT id,title FROM ' . prefixTable('roles_title') . ' ORDER BY title ASC');
768
+                $rows = DB::query('SELECT id,title FROM '.prefixTable('roles_title').' ORDER BY title ASC');
769 769
                 foreach ($rows as $reccord) {
770 770
                     $rolesList[$reccord['id']] = array('id' => $reccord['id'], 'title' => $reccord['title']);
771 771
                 }
@@ -796,7 +796,7 @@  discard block
 block discarded – undo
796 796
                         array_push(
797 797
                             $managedBy,
798 798
                             array(
799
-                                'title' => $lang->get('managers_of') . ' ' . $fonction['title'],
799
+                                'title' => $lang->get('managers_of').' '.$fonction['title'],
800 800
                                 'id' => $fonction['id'],
801 801
                                 'selected' => $selected,
802 802
                             )
@@ -878,9 +878,9 @@  discard block
 block discarded – undo
878 878
 
879 879
                 // get USER STATUS
880 880
                 if ($rowUser['disabled'] == 1) {
881
-                    $arrData['info'] = $lang->get('user_info_locked') . '<br><input type="checkbox" value="unlock" name="1" class="chk">&nbsp;<label for="1">' . $lang->get('user_info_unlock_question') . '</label><br><input type="checkbox"  value="delete" id="account_delete" class="chk mr-2" name="2" onclick="confirmDeletion()">label for="2">' . $lang->get('user_info_delete_question') . '</label>';
881
+                    $arrData['info'] = $lang->get('user_info_locked').'<br><input type="checkbox" value="unlock" name="1" class="chk">&nbsp;<label for="1">'.$lang->get('user_info_unlock_question').'</label><br><input type="checkbox"  value="delete" id="account_delete" class="chk mr-2" name="2" onclick="confirmDeletion()">label for="2">'.$lang->get('user_info_delete_question').'</label>';
882 882
                 } else {
883
-                    $arrData['info'] = $lang->get('user_info_active') . '<br><input type="checkbox" value="lock" class="chk">&nbsp;' . $lang->get('user_info_lock_question');
883
+                    $arrData['info'] = $lang->get('user_info_active').'<br><input type="checkbox" value="lock" class="chk">&nbsp;'.$lang->get('user_info_lock_question');
884 884
                 }
885 885
 
886 886
                 $arrData['error'] = false;
@@ -967,7 +967,7 @@  discard block
 block discarded – undo
967 967
 
968 968
             // Get info about user to modify
969 969
             $data_user = DB::queryfirstrow(
970
-                'SELECT admin, gestionnaire, can_manage_all_users, isAdministratedByRole FROM ' . prefixTable('users') . '
970
+                'SELECT admin, gestionnaire, can_manage_all_users, isAdministratedByRole FROM '.prefixTable('users').'
971 971
                 WHERE id = %i',
972 972
                 $post_id
973 973
             );
@@ -978,7 +978,7 @@  discard block
 block discarded – undo
978 978
                 // count number of admins
979 979
                 $users = DB::query(
980 980
                     'SELECT id
981
-                    FROM ' . prefixTable('users') . '
981
+                    FROM ' . prefixTable('users').'
982 982
                     WHERE admin = 1 AND email != "" AND pw != "" AND id != %i',
983 983
                     $post_id
984 984
                 );
@@ -1001,7 +1001,7 @@  discard block
 block discarded – undo
1001 1001
             // Exclude roles from AD - PR #3635
1002 1002
             $adRoles = DB::query(
1003 1003
                 'SELECT roles_from_ad_groups
1004
-                FROM ' . prefixTable('users') . '
1004
+                FROM ' . prefixTable('users').'
1005 1005
                 WHERE id = %i',
1006 1006
                 $post_id
1007 1007
             )[0]['roles_from_ad_groups'];
@@ -1105,7 +1105,7 @@  discard block
 block discarded – undo
1105 1105
                     );
1106 1106
                     // delete personal folder and subfolders
1107 1107
                     $data = DB::queryfirstrow(
1108
-                        'SELECT id FROM ' . prefixTable('nested_tree') . '
1108
+                        'SELECT id FROM '.prefixTable('nested_tree').'
1109 1109
                         WHERE title = %s AND personal_folder = %i',
1110 1110
                         $post_id,
1111 1111
                         '1'
@@ -1118,7 +1118,7 @@  discard block
 block discarded – undo
1118 1118
                             DB::delete(prefixTable('nested_tree'), 'id = %i AND personal_folder = %i', $folder->id, '1');
1119 1119
                             // delete items & logs
1120 1120
                             $items = DB::query(
1121
-                                'SELECT id FROM ' . prefixTable('items') . '
1121
+                                'SELECT id FROM '.prefixTable('items').'
1122 1122
                                 WHERE id_tree=%i AND perso = %i',
1123 1123
                                 $folder->id,
1124 1124
                                 '1'
@@ -1139,7 +1139,7 @@  discard block
 block discarded – undo
1139 1139
                 } else {
1140 1140
                     // Get old data about user
1141 1141
                     $oldData = DB::queryfirstrow(
1142
-                        'SELECT * FROM ' . prefixTable('users') . '
1142
+                        'SELECT * FROM '.prefixTable('users').'
1143 1143
                         WHERE id = %i',
1144 1144
                         $post_id
1145 1145
                     );
@@ -1176,7 +1176,7 @@  discard block
 block discarded – undo
1176 1176
 
1177 1177
                     // update LOG
1178 1178
                     if ($oldData['email'] !== $post_email) {
1179
-                        logEvents($SETTINGS, 'user_mngt', 'at_user_email_changed:' . $oldData['email'], (string) $session->get('user-id'), $session->get('user-login'), $post_id);
1179
+                        logEvents($SETTINGS, 'user_mngt', 'at_user_email_changed:'.$oldData['email'], (string) $session->get('user-id'), $session->get('user-login'), $post_id);
1180 1180
                     }
1181 1181
                 }
1182 1182
                 echo prepareExchangedData(
@@ -1224,7 +1224,7 @@  discard block
 block discarded – undo
1224 1224
             }
1225 1225
 
1226 1226
             DB::queryfirstrow(
1227
-                'SELECT * FROM ' . prefixTable('users') . '
1227
+                'SELECT * FROM '.prefixTable('users').'
1228 1228
                 WHERE login = %s
1229 1229
                 AND deleted_at IS NULL',
1230 1230
                 filter_input(INPUT_POST, 'login', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
@@ -1276,7 +1276,7 @@  discard block
 block discarded – undo
1276 1276
             // get User info
1277 1277
             $rowUser = DB::queryFirstRow(
1278 1278
                 'SELECT login, name, lastname, email, disabled, fonction_id, groupes_interdits, groupes_visibles, isAdministratedByRole, avatar_thumb, roles_from_ad_groups
1279
-                FROM ' . prefixTable('users') . '
1279
+                FROM ' . prefixTable('users').'
1280 1280
                 WHERE id = %i',
1281 1281
                 $post_id
1282 1282
             );
@@ -1286,7 +1286,7 @@  discard block
 block discarded – undo
1286 1286
             $html = '';
1287 1287
 
1288 1288
             if (isset($SETTINGS['ldap_mode']) === true && (int) $SETTINGS['ldap_mode'] === 1 && isset($SETTINGS['enable_ad_users_with_ad_groups']) === true && (int) $SETTINGS['enable_ad_users_with_ad_groups'] === 1) {
1289
-                $rowUser['fonction_id'] = empty($rowUser['fonction_id'])  === true ? $rowUser['roles_from_ad_groups'] : $rowUser['fonction_id']. ';' . $rowUser['roles_from_ad_groups'];
1289
+                $rowUser['fonction_id'] = empty($rowUser['fonction_id']) === true ? $rowUser['roles_from_ad_groups'] : $rowUser['fonction_id'].';'.$rowUser['roles_from_ad_groups'];
1290 1290
             }
1291 1291
             $arrData['functions'] = array_filter(explode(';', $rowUser['fonction_id']));
1292 1292
             $arrData['allowed_folders'] = array_filter(explode(';', $rowUser['groupes_visibles']));
@@ -1297,7 +1297,7 @@  discard block
 block discarded – undo
1297 1297
                 // refine folders based upon roles
1298 1298
                 $rows = DB::query(
1299 1299
                     'SELECT folder_id, type
1300
-                    FROM ' . prefixTable('roles_values') . '
1300
+                    FROM ' . prefixTable('roles_values').'
1301 1301
                     WHERE role_id IN %ls
1302 1302
                     ORDER BY folder_id ASC',
1303 1303
                     $arrData['functions']
@@ -1320,7 +1320,7 @@  discard block
 block discarded – undo
1320 1320
                 }
1321 1321
 
1322 1322
                 // add allowed folders
1323
-                foreach($arrData['allowed_folders'] as $Fld) {
1323
+                foreach ($arrData['allowed_folders'] as $Fld) {
1324 1324
                     array_push($arrFolders, array('id' => $Fld, 'type' => 'W', 'special' => true));
1325 1325
                 }
1326 1326
                 
@@ -1331,7 +1331,7 @@  discard block
 block discarded – undo
1331 1331
                             // get folder name
1332 1332
                             $row = DB::queryFirstRow(
1333 1333
                                 'SELECT title, nlevel, id
1334
-                                FROM ' . prefixTable('nested_tree') . '
1334
+                                FROM ' . prefixTable('nested_tree').'
1335 1335
                                 WHERE id = %i',
1336 1336
                                 $fld['id']
1337 1337
                             );
@@ -1344,38 +1344,38 @@  discard block
 block discarded – undo
1344 1344
 
1345 1345
                             // manage right icon
1346 1346
                             if ($fld['type'] == 'W') {
1347
-                                $label = '<i class="fas fa-indent infotip text-success mr-2" title="' . $lang->get('write') . '"></i>' .
1348
-                                    '<i class="fas fa-edit infotip text-success mr-2" title="' . $lang->get('edit') . '"></i>' .
1349
-                                    '<i class="fas fa-eraser infotip text-success" title="' . $lang->get('delete') . '"></i>';
1347
+                                $label = '<i class="fas fa-indent infotip text-success mr-2" title="'.$lang->get('write').'"></i>'.
1348
+                                    '<i class="fas fa-edit infotip text-success mr-2" title="'.$lang->get('edit').'"></i>'.
1349
+                                    '<i class="fas fa-eraser infotip text-success" title="'.$lang->get('delete').'"></i>';
1350 1350
                             } elseif ($fld['type'] == 'ND') {
1351
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1352
-                                    '<i class="fas fa-edit infotip text-success mr-2" title="' . $lang->get('edit') . '"></i>' .
1353
-                                    '<i class="fas fa-eraser infotip text-danger" title="' . $lang->get('no_delete') . '"></i>';
1351
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1352
+                                    '<i class="fas fa-edit infotip text-success mr-2" title="'.$lang->get('edit').'"></i>'.
1353
+                                    '<i class="fas fa-eraser infotip text-danger" title="'.$lang->get('no_delete').'"></i>';
1354 1354
                             } elseif ($fld['type'] == 'NE') {
1355
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1356
-                                    '<i class="fas fa-edit infotip text-danger mr-2" title="' . $lang->get('no_edit') . '"></i>' .
1357
-                                    '<i class="fas fa-eraser infotip text-success" title="' . $lang->get('delete') . '"></i>';
1355
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1356
+                                    '<i class="fas fa-edit infotip text-danger mr-2" title="'.$lang->get('no_edit').'"></i>'.
1357
+                                    '<i class="fas fa-eraser infotip text-success" title="'.$lang->get('delete').'"></i>';
1358 1358
                             } elseif ($fld['type'] == 'NDNE') {
1359
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1360
-                                    '<i class="fas fa-edit infotip text-danger mr-2" title="' . $lang->get('no_edit') . '"></i>' .
1361
-                                    '<i class="fas fa-eraser infotip text-danger" title="' . $lang->get('no_delete') . '"></i>';
1359
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1360
+                                    '<i class="fas fa-edit infotip text-danger mr-2" title="'.$lang->get('no_edit').'"></i>'.
1361
+                                    '<i class="fas fa-eraser infotip text-danger" title="'.$lang->get('no_delete').'"></i>';
1362 1362
                             } elseif ($fld['type'] == '') {
1363
-                                $label = '<i class="fas fa-eye-slash infotip text-danger mr-2" title="' . $lang->get('no_access') . '"></i>';
1363
+                                $label = '<i class="fas fa-eye-slash infotip text-danger mr-2" title="'.$lang->get('no_access').'"></i>';
1364 1364
                             } else {
1365
-                                $label = '<i class="fas fa-eye infotip text-info mr-2" title="' . $lang->get('read') . '"></i>';
1365
+                                $label = '<i class="fas fa-eye infotip text-info mr-2" title="'.$lang->get('read').'"></i>';
1366 1366
                             }
1367 1367
 
1368
-                            $html .= '<tr><td>' . $ident . $row['title'] .
1369
-                                ' <small class="text-info">[' . $row['id'] . ']</small>'.
1370
-                                ($fld['special'] === true ? '<i class="fas fa-user-tag infotip text-primary ml-5" title="' . $lang->get('user_specific_right') . '"></i>' : '').
1371
-                                '</td><td>' . $label . '</td></tr>';
1368
+                            $html .= '<tr><td>'.$ident.$row['title'].
1369
+                                ' <small class="text-info">['.$row['id'].']</small>'.
1370
+                                ($fld['special'] === true ? '<i class="fas fa-user-tag infotip text-primary ml-5" title="'.$lang->get('user_specific_right').'"></i>' : '').
1371
+                                '</td><td>'.$label.'</td></tr>';
1372 1372
                             break;
1373 1373
                         }
1374 1374
                     }
1375 1375
                 }
1376 1376
 
1377
-                $html_full = '<table id="table-folders" class="table table-bordered table-striped dt-responsive nowrap" style="width:100%"><tbody>' .
1378
-                    $html . '</tbody></table>';
1377
+                $html_full = '<table id="table-folders" class="table table-bordered table-striped dt-responsive nowrap" style="width:100%"><tbody>'.
1378
+                    $html.'</tbody></table>';
1379 1379
             } else {
1380 1380
                 $html_full = '';
1381 1381
             }
@@ -1421,7 +1421,7 @@  discard block
 block discarded – undo
1421 1421
             if ((int) $session->get('user-admin') === 0 && (int) $session->get('user-can_manage_all_users') === 0) {
1422 1422
                 $rows = DB::query(
1423 1423
                     'SELECT *
1424
-                    FROM ' . prefixTable('users') . '
1424
+                    FROM ' . prefixTable('users').'
1425 1425
                     WHERE admin = %i AND isAdministratedByRole IN %ls',
1426 1426
                     '0',
1427 1427
                     array_filter($session->get('user-roles_array'))
@@ -1429,7 +1429,7 @@  discard block
 block discarded – undo
1429 1429
             } else {
1430 1430
                 $rows = DB::query(
1431 1431
                     'SELECT *
1432
-                    FROM ' . prefixTable('users') . '
1432
+                    FROM ' . prefixTable('users').'
1433 1433
                     WHERE admin = %i',
1434 1434
                     '0'
1435 1435
                 );
@@ -1441,7 +1441,7 @@  discard block
 block discarded – undo
1441 1441
                 $groupIds = [];
1442 1442
                 foreach (explode(';', $record['fonction_id']) as $group) {
1443 1443
                     $tmp = DB::queryfirstrow(
1444
-                        'SELECT id, title FROM ' . prefixTable('roles_title') . '
1444
+                        'SELECT id, title FROM '.prefixTable('roles_title').'
1445 1445
                         WHERE id = %i',
1446 1446
                         $group
1447 1447
                     );
@@ -1453,7 +1453,7 @@  discard block
 block discarded – undo
1453 1453
 
1454 1454
                 // Get managed_by
1455 1455
                 $managedBy = DB::queryfirstrow(
1456
-                    'SELECT id, title FROM ' . prefixTable('roles_title') . '
1456
+                    'SELECT id, title FROM '.prefixTable('roles_title').'
1457 1457
                     WHERE id = %i',
1458 1458
                     $record['isAdministratedByRole']
1459 1459
                 );
@@ -1463,7 +1463,7 @@  discard block
 block discarded – undo
1463 1463
                 $foldersAllowedIds = [];
1464 1464
                 foreach (explode(';', $record['groupes_visibles']) as $role) {
1465 1465
                     $tmp = DB::queryfirstrow(
1466
-                        'SELECT id, title FROM ' . prefixTable('nested_tree') . '
1466
+                        'SELECT id, title FROM '.prefixTable('nested_tree').'
1467 1467
                         WHERE id = %i',
1468 1468
                         $role
1469 1469
                     );
@@ -1476,7 +1476,7 @@  discard block
 block discarded – undo
1476 1476
                 $foldersForbiddenIds = [];
1477 1477
                 foreach (explode(';', $record['groupes_interdits']) as $role) {
1478 1478
                     $tmp = DB::queryfirstrow(
1479
-                        'SELECT id, title FROM ' . prefixTable('nested_tree') . '
1479
+                        'SELECT id, title FROM '.prefixTable('nested_tree').'
1480 1480
                         WHERE id = %i',
1481 1481
                         $role
1482 1482
                     );
@@ -1494,7 +1494,7 @@  discard block
 block discarded – undo
1494 1494
                         'login' => $record['login'],
1495 1495
                         'groups' => implode(', ', $groups),
1496 1496
                         'groupIds' => $groupIds,
1497
-                        'managedBy' => $managedBy=== null ? $lang->get('administrator') : $managedBy['title'],
1497
+                        'managedBy' => $managedBy === null ? $lang->get('administrator') : $managedBy['title'],
1498 1498
                         'managedById' => $managedBy === null ? 0 : $managedBy['id'],
1499 1499
                         'foldersAllowed' => implode(', ', $foldersAllowed),
1500 1500
                         'foldersAllowedIds' => $foldersAllowedIds,
@@ -1595,7 +1595,7 @@  discard block
 block discarded – undo
1595 1595
 
1596 1596
             // Get info about user
1597 1597
             $data_user = DB::queryfirstrow(
1598
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
1598
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
1599 1599
                 WHERE id = %i',
1600 1600
                 $inputData['source_id']
1601 1601
             );
@@ -1797,14 +1797,14 @@  discard block
 block discarded – undo
1797 1797
             if (empty($post_context) === false && $post_context === 'add_one_role_to_user') {
1798 1798
                 $data_user = DB::queryfirstrow(
1799 1799
                     'SELECT fonction_id, public_key
1800
-                    FROM ' . prefixTable('users') . '
1800
+                    FROM ' . prefixTable('users').'
1801 1801
                     WHERE id = %i',
1802 1802
                     $post_user_id
1803 1803
                 );
1804 1804
 
1805 1805
                 if ($data_user) {
1806 1806
                     // Ensure array is unique
1807
-                    $post_new_value = str_replace(',', ';', $data_user['fonction_id']) . ';' . $post_new_value;
1807
+                    $post_new_value = str_replace(',', ';', $data_user['fonction_id']).';'.$post_new_value;
1808 1808
                     $post_new_value = implode(';', array_unique(explode(';', $post_new_value)));
1809 1809
                 } else {
1810 1810
                     // User not found
@@ -1820,14 +1820,14 @@  discard block
 block discarded – undo
1820 1820
             }
1821 1821
 
1822 1822
             // Manage specific case of api key
1823
-            if($post_field === 'user_api_key') {
1823
+            if ($post_field === 'user_api_key') {
1824 1824
                 $encrypted_key = encryptUserObjectKey(base64_encode($post_new_value), $session->get('user-public_key'));
1825 1825
                 $session->set('user-api_key', $post_new_value);
1826 1826
 
1827 1827
                 // test if user has an api key
1828 1828
                 $data_user = DB::queryfirstrow(
1829 1829
                     'SELECT value
1830
-                    FROM ' . prefixTable('api') . '
1830
+                    FROM ' . prefixTable('api').'
1831 1831
                     WHERE user_id = %i',
1832 1832
                     $post_user_id
1833 1833
                 );
@@ -1947,7 +1947,7 @@  discard block
 block discarded – undo
1947 1947
             } catch (\LdapRecord\Auth\BindException $e) {
1948 1948
                 $error = $e->getDetailedError();
1949 1949
                 if ($error && defined('LOG_TO_SERVER') && LOG_TO_SERVER === true) {
1950
-                    error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
1950
+                    error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
1951 1951
                 }
1952 1952
                 // deepcode ignore ServerLeak: No important data is sent and it is encrypted before sending
1953 1953
                 echo prepareExchangedData(
@@ -1977,7 +1977,7 @@  discard block
 block discarded – undo
1977 1977
             } catch (\LdapRecord\Auth\BindException $e) {
1978 1978
                 $error = $e->getDetailedError();
1979 1979
                 if ($error && defined('LOG_TO_SERVER') && LOG_TO_SERVER === true) {
1980
-                    error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
1980
+                    error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
1981 1981
                 }
1982 1982
                 // deepcode ignore ServerLeak: No important data is sent and it is encrypted before sending
1983 1983
                 echo prepareExchangedData(
@@ -1994,7 +1994,7 @@  discard block
 block discarded – undo
1994 1994
                 if (isset($adUser[$SETTINGS['ldap_user_attribute']][0]) === false) continue;
1995 1995
                 // Build the list of all groups in AD
1996 1996
                 if (isset($adUser['memberof']) === true) {
1997
-                    foreach($adUser['memberof'] as $j => $adUserGroup) {
1997
+                    foreach ($adUser['memberof'] as $j => $adUserGroup) {
1998 1998
                         if (empty($adUserGroup) === false && $j !== "count") {
1999 1999
                             $adGroup = substr($adUserGroup, 3, strpos($adUserGroup, ',') - 3);
2000 2000
                             if (in_array($adGroup, $adRoles) === false && empty($adGroup) === false) {
@@ -2010,7 +2010,7 @@  discard block
 block discarded – undo
2010 2010
                     // Get his ID
2011 2011
                     $userInfo = DB::queryfirstrow(
2012 2012
                         'SELECT id, login, fonction_id, auth_type
2013
-                        FROM ' . prefixTable('users') . '
2013
+                        FROM ' . prefixTable('users').'
2014 2014
                         WHERE login = %s',
2015 2015
                         $userLogin
2016 2016
                     );
@@ -2046,7 +2046,7 @@  discard block
 block discarded – undo
2046 2046
 
2047 2047
             // Get all groups in Teampass
2048 2048
             $teampassRoles = array();
2049
-            $rows = DB::query('SELECT id,title FROM ' . prefixTable('roles_title'));
2049
+            $rows = DB::query('SELECT id,title FROM '.prefixTable('roles_title'));
2050 2050
             foreach ($rows as $record) {
2051 2051
                 array_push(
2052 2052
                     $teampassRoles,
@@ -2110,7 +2110,7 @@  discard block
 block discarded – undo
2110 2110
             // Check if user already exists
2111 2111
             $data = DB::query(
2112 2112
                 'SELECT id, fonction_id, groupes_interdits, groupes_visibles
2113
-                FROM ' . prefixTable('users') . '
2113
+                FROM ' . prefixTable('users').'
2114 2114
                 WHERE login = %s',
2115 2115
                 $post_login
2116 2116
             );
@@ -2287,7 +2287,7 @@  discard block
 block discarded – undo
2287 2287
             // Check if user already exists
2288 2288
             DB::query(
2289 2289
                 'SELECT id
2290
-                FROM ' . prefixTable('users') . '
2290
+                FROM ' . prefixTable('users').'
2291 2291
                 WHERE id = %i',
2292 2292
                 $post_id
2293 2293
             );
@@ -2373,7 +2373,7 @@  discard block
 block discarded – undo
2373 2373
             // Check if user already exists
2374 2374
             DB::query(
2375 2375
                 'SELECT id
2376
-                FROM ' . prefixTable('users') . '
2376
+                FROM ' . prefixTable('users').'
2377 2377
                 WHERE id = %i',
2378 2378
                 $post_id
2379 2379
             );
@@ -2440,7 +2440,7 @@  discard block
 block discarded – undo
2440 2440
             // Search TP_USER in db        
2441 2441
             $userTP = DB::queryFirstRow(
2442 2442
                 'SELECT pw
2443
-                FROM ' . prefixTable('users') . '
2443
+                FROM ' . prefixTable('users').'
2444 2444
                 WHERE id = %i',
2445 2445
                 TP_USER_ID
2446 2446
             );
@@ -2462,8 +2462,8 @@  discard block
 block discarded – undo
2462 2462
                     'process_type' => 'create_user_keys',
2463 2463
                     'arguments' => json_encode([
2464 2464
                         'new_user_id' => (int) $post_user_id,
2465
-                        'new_user_pwd' => empty($post_user_pwd) === true ? '' : cryption($post_user_pwd, '','encrypt', $SETTINGS)['string'],
2466
-                        'new_user_code' => cryption($post_user_code, '','encrypt', $SETTINGS)['string'],
2465
+                        'new_user_pwd' => empty($post_user_pwd) === true ? '' : cryption($post_user_pwd, '', 'encrypt', $SETTINGS)['string'],
2466
+                        'new_user_code' => cryption($post_user_code, '', 'encrypt', $SETTINGS)['string'],
2467 2467
                         'owner_id' => (int) TP_USER_ID,
2468 2468
                         'creator_pwd' => $userTP['pw'],
2469 2469
                         'email_body' => $lang->get('email_body_user_config_5'),
@@ -2638,8 +2638,8 @@  discard block
 block discarded – undo
2638 2638
             // get user info
2639 2639
             $processesProgress = DB::query(
2640 2640
                 'SELECT u.ongoing_process_id, pt.task, pt.updated_at, pt.finished_at, pt.is_in_progress
2641
-                FROM ' . prefixTable('users') . ' AS u
2642
-                INNER JOIN ' . prefixTable('background_subtasks') . ' AS pt ON (pt.task_id = u.ongoing_process_id)
2641
+                FROM ' . prefixTable('users').' AS u
2642
+                INNER JOIN ' . prefixTable('background_subtasks').' AS pt ON (pt.task_id = u.ongoing_process_id)
2643 2643
                 WHERE u.id = %i',
2644 2644
                 $user_id
2645 2645
             );
@@ -2647,9 +2647,9 @@  discard block
 block discarded – undo
2647 2647
             //print_r($processesProgress);
2648 2648
             $finished_steps = 0;
2649 2649
             $nb_steps = count($processesProgress);
2650
-            foreach($processesProgress as $process) {
2650
+            foreach ($processesProgress as $process) {
2651 2651
                 if ((int) $process['is_in_progress'] === -1) {
2652
-                    $finished_steps ++;
2652
+                    $finished_steps++;
2653 2653
                 }
2654 2654
             }
2655 2655
 
@@ -2658,7 +2658,7 @@  discard block
 block discarded – undo
2658 2658
                     'error' => false,
2659 2659
                     'message' => '',
2660 2660
                     'user_id' => $user_id,
2661
-                    'status' => $finished_steps === $nb_steps ? 'finished' : number_format($finished_steps/$nb_steps*100, 0).'%',
2661
+                    'status' => $finished_steps === $nb_steps ? 'finished' : number_format($finished_steps / $nb_steps * 100, 0).'%',
2662 2662
                     'debug' => $finished_steps.",".$nb_steps,
2663 2663
                 ),
2664 2664
                 'encode'
@@ -2722,7 +2722,7 @@  discard block
 block discarded – undo
2722 2722
 
2723 2723
     // Get info about user
2724 2724
     $data_user = DB::queryfirstrow(
2725
-        'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
2725
+        'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
2726 2726
         WHERE id = %i',
2727 2727
         $value[1]
2728 2728
     );
@@ -2756,7 +2756,7 @@  discard block
 block discarded – undo
2756 2756
             logEvents(
2757 2757
                 $SETTINGS,
2758 2758
                 'user_mngt',
2759
-                'at_user_new_' . $value[0] . ':' . $value[1],
2759
+                'at_user_new_'.$value[0].':'.$value[1],
2760 2760
                 (string) $session->get('user-id'),
2761 2761
                 $session->get('user-login'),
2762 2762
                 filter_input(INPUT_POST, 'id', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
@@ -2790,7 +2790,7 @@  discard block
 block discarded – undo
2790 2790
 
2791 2791
     // Get info about user
2792 2792
     $data_user = DB::queryfirstrow(
2793
-        'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
2793
+        'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
2794 2794
         WHERE id = %i',
2795 2795
         $value[1]
2796 2796
     );
Please login to merge, or discard this patch.