Passed
Push — master ( 5d5912...cd15f0 )
by Nils
04:50
created
api/Model/Operation.php 2 patches
Braces   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -38,7 +38,7 @@
 block discarded – undo
38 38
                 "error" => '',
39 39
             );
40 40
     
41
-        }catch (Exception $e) {    
41
+        } catch (Exception $e) {    
42 42
             return false;
43 43
         }
44 44
     }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -40,7 +40,7 @@
 block discarded – undo
40 40
                 "error" => '',
41 41
             );
42 42
     
43
-        }catch (Exception $e) {    
43
+        } catch (Exception $e) {    
44 44
             return false;
45 45
         }
46 46
     }
Please login to merge, or discard this patch.
api/Model/UserModel.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
  *
22 22
  * @see       https://www.teampass.net
23 23
  */
24
-require_once API_ROOT_PATH . "/Model/Database.php";
24
+require_once API_ROOT_PATH."/Model/Database.php";
25 25
 
26 26
 class UserModel extends Database
27 27
 {
Please login to merge, or discard this patch.
api/Model/FolderModel.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -34,7 +34,7 @@  discard block
 block discarded – undo
34 34
         $ret = [];
35 35
 
36 36
         foreach ($rows as $row) {
37
-			$isVisible = in_array((int) $row['id'], $foldersId);
37
+            $isVisible = in_array((int) $row['id'], $foldersId);
38 38
             $childrens = $this->getFoldersChildren($row['id'], $foldersId);
39 39
 
40 40
             if ($isVisible || count($childrens) > 0) {
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
                     [
44 44
                         'id' => (int) $row['id'],
45 45
                         'title' => $row['title'],
46
-						'isVisible' => $isVisible,
46
+                        'isVisible' => $isVisible,
47 47
                         'childrens' => $childrens
48 48
                     ]
49 49
                 );
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 
64 64
         if ( count($childrens) > 0) {
65 65
             foreach ($childrens as $children) {
66
-				$isVisible = in_array((int) $children['id'], $foldersId);
66
+                $isVisible = in_array((int) $children['id'], $foldersId);
67 67
                 $childs = $this->getFoldersChildren($children['id'], $foldersId);
68 68
 
69 69
                 if (in_array((int) $children['id'], $foldersId) || count($childs) > 0) {
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
                         [
73 73
                             'id' => (int) $children['id'],
74 74
                             'title' => $children['title'],
75
-							'isVisible' => $isVisible,
75
+                            'isVisible' => $isVisible,
76 76
                             'childrens' => $childs
77 77
                         ]
78 78
                     );
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -22,13 +22,13 @@  discard block
 block discarded – undo
22 22
  *
23 23
  * @see       https://www.teampass.net
24 24
  */
25
-require_once API_ROOT_PATH . "/Model/Database.php";
25
+require_once API_ROOT_PATH."/Model/Database.php";
26 26
  
27 27
 class FolderModel extends Database
28 28
 {
29 29
     public function getFoldersInfo(array $foldersId): array
30 30
     {
31
-        $rows = $this->select( "SELECT id, title FROM " . prefixTable('nested_tree') . " WHERE nlevel=1" );
31
+        $rows = $this->select("SELECT id, title FROM ".prefixTable('nested_tree')." WHERE nlevel=1");
32 32
 
33 33
         $ret = [];
34 34
 
@@ -55,9 +55,9 @@  discard block
 block discarded – undo
55 55
     private function getFoldersChildren(int $parentId, array $foldersId): array
56 56
     {
57 57
         $ret = [];
58
-        $childrens = $this->select('SELECT id, title FROM ' . prefixTable('nested_tree') . ' WHERE parent_id=' . $parentId);
58
+        $childrens = $this->select('SELECT id, title FROM '.prefixTable('nested_tree').' WHERE parent_id='.$parentId);
59 59
 
60
-        if ( count($childrens) > 0) {
60
+        if (count($childrens) > 0) {
61 61
             foreach ($childrens as $children) {
62 62
 				$isVisible = in_array((int) $children['id'], $foldersId);
63 63
                 $childs = $this->getFoldersChildren($children['id'], $foldersId);
Please login to merge, or discard this patch.
api/Controller/Api/ItemController.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
             // SQL where clause with folders list
53 53
             if (isset($arrQueryStringParams['folders']) === true) {
54 54
                 // convert the folders to an array
55
-                $arrQueryStringParams['folders'] = explode(',', str_replace( array('[',']') , ''  , $arrQueryStringParams['folders']));
55
+                $arrQueryStringParams['folders'] = explode(',', str_replace(array('[', ']'), '', $arrQueryStringParams['folders']));
56 56
 
57 57
                 // ensure to only use the intersection
58 58
                 $foldersList = implode(',', array_intersect($arrQueryStringParams['folders'], $userData['folders_list']));
@@ -126,10 +126,10 @@  discard block
 block discarded – undo
126 126
             // SQL where clause with item id
127 127
             if (isset($arrQueryStringParams['id']) === true) {
128 128
                 // build sql where clause by ID
129
-                $sqlExtra = ' WHERE i.id = '.$arrQueryStringParams['id'] . $sql_constraint;
129
+                $sqlExtra = ' WHERE i.id = '.$arrQueryStringParams['id'].$sql_constraint;
130 130
             } else if (isset($arrQueryStringParams['label']) === true) {
131 131
                 // build sql where clause by LABEL
132
-                $sqlExtra = ' WHERE i.label '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE '.$arrQueryStringParams['label'] : ' = '.$arrQueryStringParams['label']) . $sql_constraint;
132
+                $sqlExtra = ' WHERE i.label '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE '.$arrQueryStringParams['label'] : ' = '.$arrQueryStringParams['label']).$sql_constraint;
133 133
             } else if (isset($arrQueryStringParams['description']) === true) {
134 134
                 // build sql where clause by LABEL
135 135
                 $sqlExtra = ' WHERE i.description '.(isset($arrQueryStringParams['like']) === true && (int) $arrQueryStringParams['like'] === 1 ? ' LIKE '.$arrQueryStringParams['description'] : ' = '.$arrQueryStringParams['description']).$sql_constraint;
Please login to merge, or discard this patch.
api/Controller/Api/FolderController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
                     $arrFolders = $folderModel->getFoldersInfo(explode(",", $userData['folders_list']));
51 51
                     $responseData = json_encode($arrFolders);
52 52
                 } catch (Error $e) {
53
-                    $strErrorDesc = $e->getMessage() . ' Something went wrong! Please contact support.';
53
+                    $strErrorDesc = $e->getMessage().' Something went wrong! Please contact support.';
54 54
                     $strErrorHeader = 'HTTP/1.1 500 Internal Server Error';
55 55
                 }
56 56
             }
Please login to merge, or discard this patch.
sources/users.queries.php 2 patches
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -2603,7 +2603,9 @@
 block discarded – undo
2603 2603
             }
2604 2604
             
2605 2605
             foreach ($results as $adUser) {
2606
-                if (isset($adUser[$SETTINGS['ldap_user_attribute']][0]) === false) continue;
2606
+                if (isset($adUser[$SETTINGS['ldap_user_attribute']][0]) === false) {
2607
+                    continue;
2608
+                }
2607 2609
                 // Build the list of all groups in AD
2608 2610
                 if (isset($adUser['memberof']) === true) {
2609 2611
                     foreach($adUser['memberof'] as $j => $adUserGroup) {
Please login to merge, or discard this patch.
Spacing   +116 added lines, -116 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
 
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
             // Prepare variables
134 134
             $login = filter_var($dataReceived['login'], FILTER_SANITIZE_FULL_SPECIAL_CHARS);
135 135
             $email = filter_var($dataReceived['email'], FILTER_SANITIZE_EMAIL);
136
-            $password = '';//filter_var($dataReceived['pw'], FILTER_SANITIZE_FULL_SPECIAL_CHARS);
136
+            $password = ''; //filter_var($dataReceived['pw'], FILTER_SANITIZE_FULL_SPECIAL_CHARS);
137 137
             $lastname = filter_var($dataReceived['lastname'], FILTER_SANITIZE_FULL_SPECIAL_CHARS);
138 138
             $name = filter_var($dataReceived['name'], FILTER_SANITIZE_FULL_SPECIAL_CHARS);
139 139
             $is_admin = filter_var($dataReceived['admin'], FILTER_SANITIZE_NUMBER_INT);
@@ -164,7 +164,7 @@  discard block
 block discarded – undo
164 164
             // Check if user already exists
165 165
             $data = DB::query(
166 166
                 'SELECT id, fonction_id, groupes_interdits, groupes_visibles
167
-                FROM ' . prefixTable('users') . '
167
+                FROM ' . prefixTable('users').'
168 168
                 WHERE login = %s
169 169
                 AND deleted_at IS NULL',
170 170
                 $login
@@ -400,7 +400,7 @@  discard block
 block discarded – undo
400 400
 
401 401
             // Get info about user to delete
402 402
             $data_user = DB::queryfirstrow(
403
-                'SELECT login, admin, isAdministratedByRole FROM ' . prefixTable('users') . '
403
+                'SELECT login, admin, isAdministratedByRole FROM '.prefixTable('users').'
404 404
                 WHERE id = %i',
405 405
                 $post_id
406 406
             );
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
434 434
                 );
435 435
                 // delete personal folder and subfolders
436 436
                 $data = DB::queryfirstrow(
437
-                    'SELECT id FROM ' . prefixTable('nested_tree') . '
437
+                    'SELECT id FROM '.prefixTable('nested_tree').'
438 438
                     WHERE title = %s AND personal_folder = %i',
439 439
                     $post_id,
440 440
                     '1'
@@ -447,7 +447,7 @@  discard block
 block discarded – undo
447 447
                         DB::delete(prefixTable('nested_tree'), 'id = %i AND personal_folder = %i', $folder->id, '1');
448 448
                         // delete items & logs
449 449
                         $items = DB::query(
450
-                            'SELECT id FROM ' . prefixTable('items') . '
450
+                            'SELECT id FROM '.prefixTable('items').'
451 451
                             WHERE id_tree=%i AND perso = %i',
452 452
                             $folder->id,
453 453
                             '1'
@@ -471,7 +471,7 @@  discard block
 block discarded – undo
471 471
                 // Delete any process related to user
472 472
                 $processes = DB::query(
473 473
                     'SELECT increment_id
474
-                    FROM ' . prefixTable('background_tasks') . '
474
+                    FROM ' . prefixTable('background_tasks').'
475 475
                     WHERE JSON_EXTRACT(arguments, "$.new_user_id") = %i',
476 476
                     $post_id
477 477
                 );
@@ -531,7 +531,7 @@  discard block
 block discarded – undo
531 531
 
532 532
             // Get info about user to delete
533 533
             $data_user = DB::queryfirstrow(
534
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
534
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
535 535
                 WHERE id = %i',
536 536
                 $post_id
537 537
             );
@@ -573,7 +573,7 @@  discard block
 block discarded – undo
573 573
 
574 574
             // Get info about user to delete
575 575
             $data_user = DB::queryfirstrow(
576
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
576
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
577 577
                 WHERE id = %i',
578 578
                 $post_id
579 579
             );
@@ -616,7 +616,7 @@  discard block
 block discarded – undo
616 616
             // Get info about user to delete
617 617
             $data_user = DB::queryfirstrow(
618 618
                 'SELECT admin, isAdministratedByRole, can_manage_all_users, gestionnaire
619
-                FROM ' . prefixTable('users') . '
619
+                FROM ' . prefixTable('users').'
620 620
                 WHERE id = %i',
621 621
                 $post_id
622 622
             );
@@ -660,7 +660,7 @@  discard block
 block discarded – undo
660 660
 
661 661
             // Get info about user to delete
662 662
             $data_user = DB::queryfirstrow(
663
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
663
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
664 664
                 WHERE id = %i',
665 665
                 $post_id
666 666
             );
@@ -703,7 +703,7 @@  discard block
 block discarded – undo
703 703
             // Get info about user to delete
704 704
             $data_user = DB::queryfirstrow(
705 705
                 'SELECT admin, isAdministratedByRole, gestionnaire
706
-                FROM ' . prefixTable('users') . '
706
+                FROM ' . prefixTable('users').'
707 707
                 WHERE id = %i',
708 708
                 $post_id
709 709
             );
@@ -746,7 +746,7 @@  discard block
 block discarded – undo
746 746
             // Get info about user to delete
747 747
             $data_user = DB::queryfirstrow(
748 748
                 'SELECT admin, isAdministratedByRole, gestionnaire
749
-                FROM ' . prefixTable('users') . '
749
+                FROM ' . prefixTable('users').'
750 750
                 WHERE id = %i',
751 751
                 $post_id
752 752
             );
@@ -786,7 +786,7 @@  discard block
 block discarded – undo
786 786
             // Get info about user to delete
787 787
             $data_user = DB::queryfirstrow(
788 788
                 'SELECT admin, isAdministratedByRole, gestionnaire
789
-                FROM ' . prefixTable('users') . '
789
+                FROM ' . prefixTable('users').'
790 790
                 WHERE id = %i',
791 791
                 $post_id
792 792
             );
@@ -825,7 +825,7 @@  discard block
 block discarded – undo
825 825
             $return = array();
826 826
             // Check if folder exists
827 827
             $data = DB::query(
828
-                'SELECT * FROM ' . prefixTable('nested_tree') . '
828
+                'SELECT * FROM '.prefixTable('nested_tree').'
829 829
                 WHERE title = %s AND parent_id = %i',
830 830
                 filter_input(INPUT_POST, 'domain', FILTER_SANITIZE_FULL_SPECIAL_CHARS),
831 831
                 '0'
@@ -838,7 +838,7 @@  discard block
 block discarded – undo
838 838
             }
839 839
             // Check if role exists
840 840
             $data = DB::query(
841
-                'SELECT * FROM ' . prefixTable('roles_title') . '
841
+                'SELECT * FROM '.prefixTable('roles_title').'
842 842
                 WHERE title = %s',
843 843
                 filter_input(INPUT_POST, 'domain', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
844 844
             );
@@ -858,7 +858,7 @@  discard block
 block discarded – undo
858 858
         case 'user_log_items':
859 859
             $nb_pages = 1;
860 860
             $logs = $sql_filter = '';
861
-            $pages = '<table style=\'border-top:1px solid #969696;\'><tr><td>' . $lang->get('pages') . '&nbsp;:&nbsp;</td>';
861
+            $pages = '<table style=\'border-top:1px solid #969696;\'><tr><td>'.$lang->get('pages').'&nbsp;:&nbsp;</td>';
862 862
 
863 863
             // Prepare POST variables
864 864
             $post_nb_items_by_page = filter_input(INPUT_POST, 'nb_items_by_page', FILTER_SANITIZE_NUMBER_INT);
@@ -870,14 +870,14 @@  discard block
 block discarded – undo
870 870
                     && !empty(filter_input(INPUT_POST, 'filter', FILTER_SANITIZE_FULL_SPECIAL_CHARS))
871 871
                     && filter_input(INPUT_POST, 'filter', FILTER_SANITIZE_FULL_SPECIAL_CHARS) !== 'all'
872 872
                 ) {
873
-                    $sql_filter = " AND l.action = '" . filter_input(INPUT_POST, 'filter', FILTER_SANITIZE_FULL_SPECIAL_CHARS) . "'";
873
+                    $sql_filter = " AND l.action = '".filter_input(INPUT_POST, 'filter', FILTER_SANITIZE_FULL_SPECIAL_CHARS)."'";
874 874
                 }
875 875
                 // get number of pages
876 876
                 DB::query(
877 877
                     'SELECT *
878
-                    FROM ' . prefixTable('log_items') . ' as l
879
-                    INNER JOIN ' . prefixTable('items') . ' as i ON (l.id_item=i.id)
880
-                    INNER JOIN ' . prefixTable('users') . ' as u ON (l.id_user=u.id)
878
+                    FROM ' . prefixTable('log_items').' as l
879
+                    INNER JOIN ' . prefixTable('items').' as i ON (l.id_item=i.id)
880
+                    INNER JOIN ' . prefixTable('users').' as u ON (l.id_user=u.id)
881 881
                     WHERE l.id_user = %i ' . $sql_filter,
882 882
                     filter_input(INPUT_POST, 'id', FILTER_SANITIZE_NUMBER_INT)
883 883
                 );
@@ -895,19 +895,19 @@  discard block
 block discarded – undo
895 895
                 // launch query
896 896
                 $rows = DB::query(
897 897
                     'SELECT l.date as date, u.login as login, i.label as label, l.action as action
898
-                    FROM ' . prefixTable('log_items') . ' as l
899
-                    INNER JOIN ' . prefixTable('items') . ' as i ON (l.id_item=i.id)
900
-                    INNER JOIN ' . prefixTable('users') . ' as u ON (l.id_user=u.id)
901
-                    WHERE l.id_user = %i ' . $sql_filter . '
898
+                    FROM ' . prefixTable('log_items').' as l
899
+                    INNER JOIN ' . prefixTable('items').' as i ON (l.id_item=i.id)
900
+                    INNER JOIN ' . prefixTable('users').' as u ON (l.id_user=u.id)
901
+                    WHERE l.id_user = %i ' . $sql_filter.'
902 902
                     ORDER BY date DESC
903
-                    LIMIT ' . intval($start) . ',' . intval($post_nb_items_by_page),
903
+                    LIMIT ' . intval($start).','.intval($post_nb_items_by_page),
904 904
                     filter_input(INPUT_POST, 'id', FILTER_SANITIZE_NUMBER_INT)
905 905
                 );
906 906
             } else {
907 907
                 // get number of pages
908 908
                 DB::query(
909 909
                     'SELECT *
910
-                    FROM ' . prefixTable('log_system') . '
910
+                    FROM ' . prefixTable('log_system').'
911 911
                     WHERE type = %s AND field_1=%i',
912 912
                     'user_mngt',
913 913
                     filter_input(INPUT_POST, 'id', FILTER_SANITIZE_NUMBER_INT)
@@ -926,7 +926,7 @@  discard block
 block discarded – undo
926 926
                 // launch query
927 927
                 $rows = DB::query(
928 928
                     'SELECT *
929
-                    FROM ' . prefixTable('log_system') . '
929
+                    FROM ' . prefixTable('log_system').'
930 930
                     WHERE type = %s AND field_1 = %i
931 931
                     ORDER BY date DESC
932 932
                     LIMIT %i, %i',
@@ -940,7 +940,7 @@  discard block
 block discarded – undo
940 940
             if (isset($counter) && $counter != 0) {
941 941
                 $nb_pages = ceil($counter / intval($post_nb_items_by_page));
942 942
                 for ($i = 1; $i <= $nb_pages; ++$i) {
943
-                    $pages .= '<td onclick=\'displayLogs(' . $i . ',\"' . $post_scope . '\")\'><span style=\'cursor:pointer;' . (filter_input(INPUT_POST, 'page', FILTER_SANITIZE_NUMBER_INT) === $i ? 'font-weight:bold;font-size:18px;\'>' . $i : '\'>' . $i) . '</span></td>';
943
+                    $pages .= '<td onclick=\'displayLogs('.$i.',\"'.$post_scope.'\")\'><span style=\'cursor:pointer;'.(filter_input(INPUT_POST, 'page', FILTER_SANITIZE_NUMBER_INT) === $i ? 'font-weight:bold;font-size:18px;\'>'.$i : '\'>'.$i).'</span></td>';
944 944
                 }
945 945
             }
946 946
             $pages .= '</tr></table>';
@@ -949,7 +949,7 @@  discard block
 block discarded – undo
949 949
                     if ($post_scope === 'user_mngt') {
950 950
                         $user = DB::queryfirstrow(
951 951
                             'SELECT login
952
-                            from ' . prefixTable('users') . '
952
+                            from ' . prefixTable('users').'
953 953
                             WHERE id=%i',
954 954
                             $record['qui']
955 955
                         );
@@ -959,7 +959,7 @@  discard block
 block discarded – undo
959 959
                         if ($tmp[0] == 'at_user_initial_pwd_changed') {
960 960
                             $label = $lang->get('log_user_initial_pwd_changed');
961 961
                         } elseif ($tmp[0] == 'at_user_email_changed') {
962
-                            $label = $lang->get('log_user_email_changed') . $tmp[1];
962
+                            $label = $lang->get('log_user_email_changed').$tmp[1];
963 963
                         } elseif ($tmp[0] == 'at_user_added') {
964 964
                             $label = $lang->get('log_user_created');
965 965
                         } elseif ($tmp[0] == 'at_user_locked') {
@@ -970,14 +970,14 @@  discard block
 block discarded – undo
970 970
                             $label = $lang->get('log_user_pwd_changed');
971 971
                         }
972 972
                         // prepare log
973
-                        $logs .= '<tr><td>' . date($SETTINGS['date_format'] . ' ' . $SETTINGS['time_format'], (int) $record['date']) . '</td><td align=\"center\">' . $label . '</td><td align=\"center\">' . $user['login'] . '</td><td align=\"center\"></td></tr>';
973
+                        $logs .= '<tr><td>'.date($SETTINGS['date_format'].' '.$SETTINGS['time_format'], (int) $record['date']).'</td><td align=\"center\">'.$label.'</td><td align=\"center\">'.$user['login'].'</td><td align=\"center\"></td></tr>';
974 974
                     } else {
975
-                        $logs .= '<tr><td>' . date($SETTINGS['date_format'] . ' ' . $SETTINGS['time_format'], (int) $record['date']) . '</td><td align=\"center\">' . str_replace('"', '\"', $record['label']) . '</td><td align=\"center\">' . $record['login'] . '</td><td align=\"center\">' . $lang->get($record['action']) . '</td></tr>';
975
+                        $logs .= '<tr><td>'.date($SETTINGS['date_format'].' '.$SETTINGS['time_format'], (int) $record['date']).'</td><td align=\"center\">'.str_replace('"', '\"', $record['label']).'</td><td align=\"center\">'.$record['login'].'</td><td align=\"center\">'.$lang->get($record['action']).'</td></tr>';
976 976
                     }
977 977
                 }
978 978
             }
979 979
 
980
-            echo '[ { "table_logs": "' . ($logs) . '", "pages": "' . ($pages) . '", "error" : "no" } ]';
980
+            echo '[ { "table_logs": "'.($logs).'", "pages": "'.($pages).'", "error" : "no" } ]';
981 981
             break;
982 982
 
983 983
             /*
@@ -1000,7 +1000,7 @@  discard block
 block discarded – undo
1000 1000
             } else {
1001 1001
                 // Get folder id for Admin
1002 1002
                 $admin_folder = DB::queryFirstRow(
1003
-                    'SELECT id FROM ' . prefixTable('nested_tree') . '
1003
+                    'SELECT id FROM '.prefixTable('nested_tree').'
1004 1004
                     WHERE title = %i AND personal_folder = %i',
1005 1005
                     (int) $session->get('user-id'),
1006 1006
                     '1'
@@ -1011,15 +1011,15 @@  discard block
 block discarded – undo
1011 1011
                     // Get each Items in PF
1012 1012
                     $rows = DB::query(
1013 1013
                         'SELECT i.pw, i.label, l.id_user
1014
-                        FROM ' . prefixTable('items') . ' as i
1015
-                        LEFT JOIN ' . prefixTable('log_items') . ' as l ON (l.id_item=i.id)
1014
+                        FROM ' . prefixTable('items').' as i
1015
+                        LEFT JOIN ' . prefixTable('log_items').' as l ON (l.id_item=i.id)
1016 1016
                         WHERE l.action = %s AND i.perso=%i AND i.id_tree=%i',
1017 1017
                         'at_creation',
1018 1018
                         '1',
1019 1019
                         intval($folder->id)
1020 1020
                     );
1021 1021
                     foreach ($rows as $record) {
1022
-                        echo $record['label'] . ' - ';
1022
+                        echo $record['label'].' - ';
1023 1023
                         // Change user
1024 1024
                         DB::update(
1025 1025
                             prefixTable('log_items'),
@@ -1054,7 +1054,7 @@  discard block
 block discarded – undo
1054 1054
             // Get info about user to delete
1055 1055
             $data_user = DB::queryfirstrow(
1056 1056
                 'SELECT admin, isAdministratedByRole, gestionnaire
1057
-                FROM ' . prefixTable('users') . '
1057
+                FROM ' . prefixTable('users').'
1058 1058
                 WHERE id = %i',
1059 1059
                 $post_user_id
1060 1060
             );
@@ -1091,7 +1091,7 @@  discard block
 block discarded – undo
1091 1091
 
1092 1092
             // Do
1093 1093
             $rows = DB::query(
1094
-                'SELECT id FROM ' . prefixTable('users') . '
1094
+                'SELECT id FROM '.prefixTable('users').'
1095 1095
                 WHERE timestamp != %s AND admin != %i',
1096 1096
                 '',
1097 1097
                 '1'
@@ -1100,7 +1100,7 @@  discard block
 block discarded – undo
1100 1100
                 // Get info about user to delete
1101 1101
                 $data_user = DB::queryfirstrow(
1102 1102
                     'SELECT admin, isAdministratedByRole, gestionnaire
1103
-                    FROM ' . prefixTable('users') . '
1103
+                    FROM ' . prefixTable('users').'
1104 1104
                     WHERE id = %i',
1105 1105
                     $record['id']
1106 1106
                 );
@@ -1161,7 +1161,7 @@  discard block
 block discarded – undo
1161 1161
             // Get info about user
1162 1162
             $rowUser = DB::queryfirstrow(
1163 1163
                 'SELECT *
1164
-                FROM ' . prefixTable('users') . '
1164
+                FROM ' . prefixTable('users').'
1165 1165
                 WHERE id = %i',
1166 1166
                 $post_id
1167 1167
             );
@@ -1188,7 +1188,7 @@  discard block
 block discarded – undo
1188 1188
                 // array of roles for actual user
1189 1189
                 //$my_functions = explode(';', $rowUser['fonction_id']);
1190 1190
 
1191
-                $rows = DB::query('SELECT id,title,creator_id FROM ' . prefixTable('roles_title'));
1191
+                $rows = DB::query('SELECT id,title,creator_id FROM '.prefixTable('roles_title'));
1192 1192
                 foreach ($rows as $record) {
1193 1193
                     if (
1194 1194
                         (int) $session->get('user-admin') === 1
@@ -1225,7 +1225,7 @@  discard block
 block discarded – undo
1225 1225
                 $rolesList = array();
1226 1226
                 $managedBy = array();
1227 1227
                 $selected = '';
1228
-                $rows = DB::query('SELECT id,title FROM ' . prefixTable('roles_title') . ' ORDER BY title ASC');
1228
+                $rows = DB::query('SELECT id,title FROM '.prefixTable('roles_title').' ORDER BY title ASC');
1229 1229
                 foreach ($rows as $reccord) {
1230 1230
                     $rolesList[$reccord['id']] = array('id' => $reccord['id'], 'title' => $reccord['title']);
1231 1231
                 }
@@ -1256,7 +1256,7 @@  discard block
 block discarded – undo
1256 1256
                         array_push(
1257 1257
                             $managedBy,
1258 1258
                             array(
1259
-                                'title' => $lang->get('managers_of') . ' ' . $fonction['title'],
1259
+                                'title' => $lang->get('managers_of').' '.$fonction['title'],
1260 1260
                                 'id' => $fonction['id'],
1261 1261
                                 'selected' => $selected,
1262 1262
                             )
@@ -1338,9 +1338,9 @@  discard block
 block discarded – undo
1338 1338
 
1339 1339
                 // get USER STATUS
1340 1340
                 if ($rowUser['disabled'] == 1) {
1341
-                    $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>';
1341
+                    $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>';
1342 1342
                 } else {
1343
-                    $arrData['info'] = $lang->get('user_info_active') . '<br><input type="checkbox" value="lock" class="chk">&nbsp;' . $lang->get('user_info_lock_question');
1343
+                    $arrData['info'] = $lang->get('user_info_active').'<br><input type="checkbox" value="lock" class="chk">&nbsp;'.$lang->get('user_info_lock_question');
1344 1344
                 }
1345 1345
 
1346 1346
                 $arrData['error'] = false;
@@ -1437,7 +1437,7 @@  discard block
 block discarded – undo
1437 1437
                 // count number of admins
1438 1438
                 $users = DB::query(
1439 1439
                     'SELECT id
1440
-                    FROM ' . prefixTable('users') . '
1440
+                    FROM ' . prefixTable('users').'
1441 1441
                     WHERE admin = 1 AND email != "" AND pw != "" AND id != %i',
1442 1442
                     $post_id
1443 1443
                 );
@@ -1460,7 +1460,7 @@  discard block
 block discarded – undo
1460 1460
             // Exclude roles from AD - PR #3635
1461 1461
             $adRoles = DB::query(
1462 1462
                 'SELECT roles_from_ad_groups
1463
-                FROM ' . prefixTable('users') . '
1463
+                FROM ' . prefixTable('users').'
1464 1464
                 WHERE id = '. $dataReceived['user_id']
1465 1465
             )[0]['roles_from_ad_groups'];
1466 1466
             $fonctions = [];
@@ -1539,7 +1539,7 @@  discard block
 block discarded – undo
1539 1539
 
1540 1540
             // Get info about user to delete
1541 1541
             $data_user = DB::queryfirstrow(
1542
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
1542
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
1543 1543
                 WHERE id = %i',
1544 1544
                 $post_id
1545 1545
             );
@@ -1560,7 +1560,7 @@  discard block
 block discarded – undo
1560 1560
                     );
1561 1561
                     // delete personal folder and subfolders
1562 1562
                     $data = DB::queryfirstrow(
1563
-                        'SELECT id FROM ' . prefixTable('nested_tree') . '
1563
+                        'SELECT id FROM '.prefixTable('nested_tree').'
1564 1564
                         WHERE title = %s AND personal_folder = %i',
1565 1565
                         $post_id,
1566 1566
                         '1'
@@ -1573,7 +1573,7 @@  discard block
 block discarded – undo
1573 1573
                             DB::delete(prefixTable('nested_tree'), 'id = %i AND personal_folder = %i', $folder->id, '1');
1574 1574
                             // delete items & logs
1575 1575
                             $items = DB::query(
1576
-                                'SELECT id FROM ' . prefixTable('items') . '
1576
+                                'SELECT id FROM '.prefixTable('items').'
1577 1577
                                 WHERE id_tree=%i AND perso = %i',
1578 1578
                                 $folder->id,
1579 1579
                                 '1'
@@ -1594,7 +1594,7 @@  discard block
 block discarded – undo
1594 1594
                 } else {
1595 1595
                     // Get old data about user
1596 1596
                     $oldData = DB::queryfirstrow(
1597
-                        'SELECT * FROM ' . prefixTable('users') . '
1597
+                        'SELECT * FROM '.prefixTable('users').'
1598 1598
                         WHERE id = %i',
1599 1599
                         $post_id
1600 1600
                     );
@@ -1631,7 +1631,7 @@  discard block
 block discarded – undo
1631 1631
 
1632 1632
                     // update LOG
1633 1633
                     if ($oldData['email'] !== $post_email) {
1634
-                        logEvents($SETTINGS, 'user_mngt', 'at_user_email_changed:' . $oldData['email'], (string) $session->get('user-id'), $session->get('user-login'), $post_id);
1634
+                        logEvents($SETTINGS, 'user_mngt', 'at_user_email_changed:'.$oldData['email'], (string) $session->get('user-id'), $session->get('user-login'), $post_id);
1635 1635
                     }
1636 1636
                 }
1637 1637
                 echo prepareExchangedData(
@@ -1689,7 +1689,7 @@  discard block
 block discarded – undo
1689 1689
 
1690 1690
             // Get info about user to delete
1691 1691
             $data_user = DB::queryfirstrow(
1692
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
1692
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
1693 1693
                 WHERE id = %i',
1694 1694
                 $post_id
1695 1695
             );
@@ -1739,7 +1739,7 @@  discard block
 block discarded – undo
1739 1739
             }
1740 1740
 
1741 1741
             DB::queryfirstrow(
1742
-                'SELECT * FROM ' . prefixTable('users') . '
1742
+                'SELECT * FROM '.prefixTable('users').'
1743 1743
                 WHERE login = %s
1744 1744
                 AND deleted_at IS NULL',
1745 1745
                 filter_input(INPUT_POST, 'login', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
@@ -1791,7 +1791,7 @@  discard block
 block discarded – undo
1791 1791
             // get User info
1792 1792
             $rowUser = DB::queryFirstRow(
1793 1793
                 'SELECT login, name, lastname, email, disabled, fonction_id, groupes_interdits, groupes_visibles, isAdministratedByRole, avatar_thumb, roles_from_ad_groups
1794
-                FROM ' . prefixTable('users') . '
1794
+                FROM ' . prefixTable('users').'
1795 1795
                 WHERE id = %i',
1796 1796
                 $post_id
1797 1797
             );
@@ -1801,7 +1801,7 @@  discard block
 block discarded – undo
1801 1801
             $html = '';
1802 1802
 
1803 1803
             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) {
1804
-                $rowUser['fonction_id'] = empty($rowUser['fonction_id'])  === true ? $rowUser['roles_from_ad_groups'] : $rowUser['fonction_id']. ';' . $rowUser['roles_from_ad_groups'];
1804
+                $rowUser['fonction_id'] = empty($rowUser['fonction_id']) === true ? $rowUser['roles_from_ad_groups'] : $rowUser['fonction_id'].';'.$rowUser['roles_from_ad_groups'];
1805 1805
             }
1806 1806
             $arrData['functions'] = array_filter(explode(';', $rowUser['fonction_id']));
1807 1807
             $arrData['allowed_folders'] = array_filter(explode(';', $rowUser['groupes_visibles']));
@@ -1812,7 +1812,7 @@  discard block
 block discarded – undo
1812 1812
                 // refine folders based upon roles
1813 1813
                 $rows = DB::query(
1814 1814
                     'SELECT folder_id, type
1815
-                    FROM ' . prefixTable('roles_values') . '
1815
+                    FROM ' . prefixTable('roles_values').'
1816 1816
                     WHERE role_id IN %ls
1817 1817
                     ORDER BY folder_id ASC',
1818 1818
                     $arrData['functions']
@@ -1835,7 +1835,7 @@  discard block
 block discarded – undo
1835 1835
                 }
1836 1836
 
1837 1837
                 // add allowed folders
1838
-                foreach($arrData['allowed_folders'] as $Fld) {
1838
+                foreach ($arrData['allowed_folders'] as $Fld) {
1839 1839
                     array_push($arrFolders, array('id' => $Fld, 'type' => 'W', 'special' => true));
1840 1840
                 }
1841 1841
                 
@@ -1846,7 +1846,7 @@  discard block
 block discarded – undo
1846 1846
                             // get folder name
1847 1847
                             $row = DB::queryFirstRow(
1848 1848
                                 'SELECT title, nlevel, id
1849
-                                FROM ' . prefixTable('nested_tree') . '
1849
+                                FROM ' . prefixTable('nested_tree').'
1850 1850
                                 WHERE id = %i',
1851 1851
                                 $fld['id']
1852 1852
                             );
@@ -1859,38 +1859,38 @@  discard block
 block discarded – undo
1859 1859
 
1860 1860
                             // manage right icon
1861 1861
                             if ($fld['type'] == 'W') {
1862
-                                $label = '<i class="fas fa-indent infotip text-success mr-2" title="' . $lang->get('write') . '"></i>' .
1863
-                                    '<i class="fas fa-edit infotip text-success mr-2" title="' . $lang->get('edit') . '"></i>' .
1864
-                                    '<i class="fas fa-eraser infotip text-success" title="' . $lang->get('delete') . '"></i>';
1862
+                                $label = '<i class="fas fa-indent infotip text-success mr-2" title="'.$lang->get('write').'"></i>'.
1863
+                                    '<i class="fas fa-edit infotip text-success mr-2" title="'.$lang->get('edit').'"></i>'.
1864
+                                    '<i class="fas fa-eraser infotip text-success" title="'.$lang->get('delete').'"></i>';
1865 1865
                             } elseif ($fld['type'] == 'ND') {
1866
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1867
-                                    '<i class="fas fa-edit infotip text-success mr-2" title="' . $lang->get('edit') . '"></i>' .
1868
-                                    '<i class="fas fa-eraser infotip text-danger" title="' . $lang->get('no_delete') . '"></i>';
1866
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1867
+                                    '<i class="fas fa-edit infotip text-success mr-2" title="'.$lang->get('edit').'"></i>'.
1868
+                                    '<i class="fas fa-eraser infotip text-danger" title="'.$lang->get('no_delete').'"></i>';
1869 1869
                             } elseif ($fld['type'] == 'NE') {
1870
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1871
-                                    '<i class="fas fa-edit infotip text-danger mr-2" title="' . $lang->get('no_edit') . '"></i>' .
1872
-                                    '<i class="fas fa-eraser infotip text-success" title="' . $lang->get('delete') . '"></i>';
1870
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1871
+                                    '<i class="fas fa-edit infotip text-danger mr-2" title="'.$lang->get('no_edit').'"></i>'.
1872
+                                    '<i class="fas fa-eraser infotip text-success" title="'.$lang->get('delete').'"></i>';
1873 1873
                             } elseif ($fld['type'] == 'NDNE') {
1874
-                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="' . $lang->get('write') . '"></i>' .
1875
-                                    '<i class="fas fa-edit infotip text-danger mr-2" title="' . $lang->get('no_edit') . '"></i>' .
1876
-                                    '<i class="fas fa-eraser infotip text-danger" title="' . $lang->get('no_delete') . '"></i>';
1874
+                                $label = '<i class="fas fa-indent infotip text-warning mr-2" title="'.$lang->get('write').'"></i>'.
1875
+                                    '<i class="fas fa-edit infotip text-danger mr-2" title="'.$lang->get('no_edit').'"></i>'.
1876
+                                    '<i class="fas fa-eraser infotip text-danger" title="'.$lang->get('no_delete').'"></i>';
1877 1877
                             } elseif ($fld['type'] == '') {
1878
-                                $label = '<i class="fas fa-eye-slash infotip text-danger mr-2" title="' . $lang->get('no_access') . '"></i>';
1878
+                                $label = '<i class="fas fa-eye-slash infotip text-danger mr-2" title="'.$lang->get('no_access').'"></i>';
1879 1879
                             } else {
1880
-                                $label = '<i class="fas fa-eye infotip text-info mr-2" title="' . $lang->get('read') . '"></i>';
1880
+                                $label = '<i class="fas fa-eye infotip text-info mr-2" title="'.$lang->get('read').'"></i>';
1881 1881
                             }
1882 1882
 
1883
-                            $html .= '<tr><td>' . $ident . $row['title'] .
1884
-                                ' <small class="text-info">[' . $row['id'] . ']</small>'.
1885
-                                ($fld['special'] === true ? '<i class="fas fa-user-tag infotip text-primary ml-5" title="' . $lang->get('user_specific_right') . '"></i>' : '').
1886
-                                '</td><td>' . $label . '</td></tr>';
1883
+                            $html .= '<tr><td>'.$ident.$row['title'].
1884
+                                ' <small class="text-info">['.$row['id'].']</small>'.
1885
+                                ($fld['special'] === true ? '<i class="fas fa-user-tag infotip text-primary ml-5" title="'.$lang->get('user_specific_right').'"></i>' : '').
1886
+                                '</td><td>'.$label.'</td></tr>';
1887 1887
                             break;
1888 1888
                         }
1889 1889
                     }
1890 1890
                 }
1891 1891
 
1892
-                $html_full = '<table id="table-folders" class="table table-bordered table-striped dt-responsive nowrap" style="width:100%"><tbody>' .
1893
-                    $html . '</tbody></table>';
1892
+                $html_full = '<table id="table-folders" class="table table-bordered table-striped dt-responsive nowrap" style="width:100%"><tbody>'.
1893
+                    $html.'</tbody></table>';
1894 1894
             } else {
1895 1895
                 $html_full = '';
1896 1896
             }
@@ -1936,7 +1936,7 @@  discard block
 block discarded – undo
1936 1936
             if ((int) $session->get('user-admin') === 0 && (int) $session->get('user-can_manage_all_users') === 0) {
1937 1937
                 $rows = DB::query(
1938 1938
                     'SELECT *
1939
-                    FROM ' . prefixTable('users') . '
1939
+                    FROM ' . prefixTable('users').'
1940 1940
                     WHERE admin = %i AND isAdministratedByRole IN %ls',
1941 1941
                     '0',
1942 1942
                     array_filter($session->get('user-roles_array'))
@@ -1944,7 +1944,7 @@  discard block
 block discarded – undo
1944 1944
             } else {
1945 1945
                 $rows = DB::query(
1946 1946
                     'SELECT *
1947
-                    FROM ' . prefixTable('users') . '
1947
+                    FROM ' . prefixTable('users').'
1948 1948
                     WHERE admin = %i',
1949 1949
                     '0'
1950 1950
                 );
@@ -1956,7 +1956,7 @@  discard block
 block discarded – undo
1956 1956
                 $groupIds = [];
1957 1957
                 foreach (explode(';', $record['fonction_id']) as $group) {
1958 1958
                     $tmp = DB::queryfirstrow(
1959
-                        'SELECT id, title FROM ' . prefixTable('roles_title') . '
1959
+                        'SELECT id, title FROM '.prefixTable('roles_title').'
1960 1960
                         WHERE id = %i',
1961 1961
                         $group
1962 1962
                     );
@@ -1968,7 +1968,7 @@  discard block
 block discarded – undo
1968 1968
 
1969 1969
                 // Get managed_by
1970 1970
                 $managedBy = DB::queryfirstrow(
1971
-                    'SELECT id, title FROM ' . prefixTable('roles_title') . '
1971
+                    'SELECT id, title FROM '.prefixTable('roles_title').'
1972 1972
                     WHERE id = %i',
1973 1973
                     $record['isAdministratedByRole']
1974 1974
                 );
@@ -1978,7 +1978,7 @@  discard block
 block discarded – undo
1978 1978
                 $foldersAllowedIds = [];
1979 1979
                 foreach (explode(';', $record['groupes_visibles']) as $role) {
1980 1980
                     $tmp = DB::queryfirstrow(
1981
-                        'SELECT id, title FROM ' . prefixTable('nested_tree') . '
1981
+                        'SELECT id, title FROM '.prefixTable('nested_tree').'
1982 1982
                         WHERE id = %i',
1983 1983
                         $role
1984 1984
                     );
@@ -1991,7 +1991,7 @@  discard block
 block discarded – undo
1991 1991
                 $foldersForbiddenIds = [];
1992 1992
                 foreach (explode(';', $record['groupes_interdits']) as $role) {
1993 1993
                     $tmp = DB::queryfirstrow(
1994
-                        'SELECT id, title FROM ' . prefixTable('nested_tree') . '
1994
+                        'SELECT id, title FROM '.prefixTable('nested_tree').'
1995 1995
                         WHERE id = %i',
1996 1996
                         $role
1997 1997
                     );
@@ -2009,7 +2009,7 @@  discard block
 block discarded – undo
2009 2009
                         'login' => $record['login'],
2010 2010
                         'groups' => implode(', ', $groups),
2011 2011
                         'groupIds' => $groupIds,
2012
-                        'managedBy' => $managedBy=== null ? $lang->get('administrator') : $managedBy['title'],
2012
+                        'managedBy' => $managedBy === null ? $lang->get('administrator') : $managedBy['title'],
2013 2013
                         'managedById' => $managedBy === null ? 0 : $managedBy['id'],
2014 2014
                         'foldersAllowed' => implode(', ', $foldersAllowed),
2015 2015
                         'foldersAllowedIds' => $foldersAllowedIds,
@@ -2090,7 +2090,7 @@  discard block
 block discarded – undo
2090 2090
 
2091 2091
             // Get info about user
2092 2092
             $data_user = DB::queryfirstrow(
2093
-                'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
2093
+                'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
2094 2094
                 WHERE id = %i',
2095 2095
                 $post_source_id
2096 2096
             );
@@ -2283,14 +2283,14 @@  discard block
 block discarded – undo
2283 2283
             if (empty($post_context) === false && $post_context === 'add_one_role_to_user') {
2284 2284
                 $data_user = DB::queryfirstrow(
2285 2285
                     'SELECT fonction_id, public_key
2286
-                    FROM ' . prefixTable('users') . '
2286
+                    FROM ' . prefixTable('users').'
2287 2287
                     WHERE id = %i',
2288 2288
                     $post_user_id
2289 2289
                 );
2290 2290
 
2291 2291
                 if ($data_user) {
2292 2292
                     // Ensure array is unique
2293
-                    $post_new_value = str_replace(',', ';', $data_user['fonction_id']) . ';' . $post_new_value;
2293
+                    $post_new_value = str_replace(',', ';', $data_user['fonction_id']).';'.$post_new_value;
2294 2294
                     $post_new_value = implode(';', array_unique(explode(';', $post_new_value)));
2295 2295
                 } else {
2296 2296
                     // User not found
@@ -2306,7 +2306,7 @@  discard block
 block discarded – undo
2306 2306
             }
2307 2307
 
2308 2308
             // Manage specific case of api key
2309
-            if($post_field === 'user_api_key') {
2309
+            if ($post_field === 'user_api_key') {
2310 2310
                 $encrypted_key = encryptUserObjectKey(base64_encode($post_new_value), $session->get('user-public_key'));
2311 2311
                 $session->set('user-api_key', $post_new_value);
2312 2312
 
@@ -2411,7 +2411,7 @@  discard block
 block discarded – undo
2411 2411
             
2412 2412
             } catch (\LdapRecord\Auth\BindException $e) {
2413 2413
                 $error = $e->getDetailedError();
2414
-                error_log('TEAMPASS Error - Users - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
2414
+                error_log('TEAMPASS Error - Users - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
2415 2415
                 // deepcode ignore ServerLeak: No important data is sent and it is encrypted before sending
2416 2416
                 echo prepareExchangedData(
2417 2417
                     array(
@@ -2439,7 +2439,7 @@  discard block
 block discarded – undo
2439 2439
                     ->paginate(100);
2440 2440
             } catch (\LdapRecord\Auth\BindException $e) {
2441 2441
                 $error = $e->getDetailedError();
2442
-                error_log('TEAMPASS Error - Users - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
2442
+                error_log('TEAMPASS Error - Users - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
2443 2443
                 // deepcode ignore ServerLeak: No important data is sent and it is encrypted before sending
2444 2444
                 echo prepareExchangedData(
2445 2445
                     array(
@@ -2455,7 +2455,7 @@  discard block
 block discarded – undo
2455 2455
                 if (isset($adUser[$SETTINGS['ldap_user_attribute']][0]) === false) continue;
2456 2456
                 // Build the list of all groups in AD
2457 2457
                 if (isset($adUser['memberof']) === true) {
2458
-                    foreach($adUser['memberof'] as $j => $adUserGroup) {
2458
+                    foreach ($adUser['memberof'] as $j => $adUserGroup) {
2459 2459
                         if (empty($adUserGroup) === false && $j !== "count") {
2460 2460
                             $adGroup = substr($adUserGroup, 3, strpos($adUserGroup, ',') - 3);
2461 2461
                             if (in_array($adGroup, $adRoles) === false && empty($adGroup) === false) {
@@ -2471,7 +2471,7 @@  discard block
 block discarded – undo
2471 2471
                     // Get his ID
2472 2472
                     $userInfo = DB::queryfirstrow(
2473 2473
                         'SELECT id, login, fonction_id, auth_type
2474
-                        FROM ' . prefixTable('users') . '
2474
+                        FROM ' . prefixTable('users').'
2475 2475
                         WHERE login = %s',
2476 2476
                         $userLogin
2477 2477
                     );
@@ -2507,7 +2507,7 @@  discard block
 block discarded – undo
2507 2507
 
2508 2508
             // Get all groups in Teampass
2509 2509
             $teampassRoles = array();
2510
-            $rows = DB::query('SELECT id,title FROM ' . prefixTable('roles_title'));
2510
+            $rows = DB::query('SELECT id,title FROM '.prefixTable('roles_title'));
2511 2511
             foreach ($rows as $record) {
2512 2512
                 array_push(
2513 2513
                     $teampassRoles,
@@ -2577,7 +2577,7 @@  discard block
 block discarded – undo
2577 2577
             // Check if user already exists
2578 2578
             $data = DB::query(
2579 2579
                 'SELECT id, fonction_id, groupes_interdits, groupes_visibles
2580
-                FROM ' . prefixTable('users') . '
2580
+                FROM ' . prefixTable('users').'
2581 2581
                 WHERE login = %s',
2582 2582
                 $post_login
2583 2583
             );
@@ -2757,12 +2757,12 @@  discard block
 block discarded – undo
2757 2757
             // Send mail to user with new OTP
2758 2758
             $userInfo = DB::queryFirstRow(
2759 2759
                 'SELECT email
2760
-                FROM ' . prefixTable('users') . '
2760
+                FROM ' . prefixTable('users').'
2761 2761
                 WHERE id = %i',
2762 2762
                 $post_userId
2763 2763
             );
2764 2764
             sendEmail(
2765
-                'TEAMPASS - ' . $lang->get('temporary_encryption_code'),
2765
+                'TEAMPASS - '.$lang->get('temporary_encryption_code'),
2766 2766
                 str_replace(
2767 2767
                     array('#enc_code#'),
2768 2768
                     array($post_otp),
@@ -2824,7 +2824,7 @@  discard block
 block discarded – undo
2824 2824
             // Check if user already exists
2825 2825
             DB::query(
2826 2826
                 'SELECT id
2827
-                FROM ' . prefixTable('users') . '
2827
+                FROM ' . prefixTable('users').'
2828 2828
                 WHERE id = %i',
2829 2829
                 $post_id
2830 2830
             );
@@ -2903,7 +2903,7 @@  discard block
 block discarded – undo
2903 2903
             // Check if user already exists
2904 2904
             $userInfo = DB::queryfirstrow(
2905 2905
                 'SELECT id, private_key, public_key
2906
-                FROM ' . prefixTable('users') . '
2906
+                FROM ' . prefixTable('users').'
2907 2907
                 WHERE id = %i',
2908 2908
                 $post_userid
2909 2909
             );
@@ -2950,7 +2950,7 @@  discard block
 block discarded – undo
2950 2950
             if (count($session->get('user-personal_folders')) > 0) {
2951 2951
                 DB::query(
2952 2952
                     'SELECT id
2953
-                    FROM ' . prefixTable('items') . '
2953
+                    FROM ' . prefixTable('items').'
2954 2954
                     WHERE id_tree IN %ls',
2955 2955
                     $session->get('user-personal_folders')
2956 2956
                 );
@@ -3031,7 +3031,7 @@  discard block
 block discarded – undo
3031 3031
             // Check if user already exists
3032 3032
             DB::query(
3033 3033
                 'SELECT id
3034
-                FROM ' . prefixTable('users') . '
3034
+                FROM ' . prefixTable('users').'
3035 3035
                 WHERE id = %i',
3036 3036
                 $post_id
3037 3037
             );
@@ -3109,10 +3109,10 @@  discard block
 block discarded – undo
3109 3109
                     'process_type' => 'create_user_keys',
3110 3110
                     'arguments' => json_encode([
3111 3111
                         'new_user_id' => (int) $post_user_id,
3112
-                        'new_user_pwd' => empty($post_user_pwd) === true ? '' : cryption($post_user_pwd, '','encrypt', $SETTINGS)['string'],
3113
-                        'new_user_code' => cryption($post_user_code, '','encrypt', $SETTINGS)['string'],
3112
+                        'new_user_pwd' => empty($post_user_pwd) === true ? '' : cryption($post_user_pwd, '', 'encrypt', $SETTINGS)['string'],
3113
+                        'new_user_code' => cryption($post_user_code, '', 'encrypt', $SETTINGS)['string'],
3114 3114
                         'owner_id' => (int) $session->get('user-id'),
3115
-                        'creator_pwd' => cryption($session->get('user-password'), '','encrypt', $SETTINGS)['string'],
3115
+                        'creator_pwd' => cryption($session->get('user-password'), '', 'encrypt', $SETTINGS)['string'],
3116 3116
                         'email_body' => $lang->get('email_body_user_config_5'),
3117 3117
                         'send_email' => 1,
3118 3118
                     ]),
@@ -3275,7 +3275,7 @@  discard block
 block discarded – undo
3275 3275
             // get user info
3276 3276
             $userInfo = DB::queryFirstRow(
3277 3277
                 'SELECT *
3278
-                FROM ' . prefixTable('users') . '
3278
+                FROM ' . prefixTable('users').'
3279 3279
                 WHERE id = %i',
3280 3280
                 $user_id
3281 3281
             );
@@ -3368,8 +3368,8 @@  discard block
 block discarded – undo
3368 3368
             // get user info
3369 3369
             $processesProgress = DB::query(
3370 3370
                 'SELECT u.ongoing_process_id, pt.task, pt.updated_at, pt.finished_at, pt.is_in_progress
3371
-                FROM ' . prefixTable('users') . ' AS u
3372
-                INNER JOIN ' . prefixTable('background_subtasks') . ' AS pt ON (pt.task_id = u.ongoing_process_id)
3371
+                FROM ' . prefixTable('users').' AS u
3372
+                INNER JOIN ' . prefixTable('background_subtasks').' AS pt ON (pt.task_id = u.ongoing_process_id)
3373 3373
                 WHERE u.id = %i',
3374 3374
                 $user_id
3375 3375
             );
@@ -3377,9 +3377,9 @@  discard block
 block discarded – undo
3377 3377
             //print_r($processesProgress);
3378 3378
             $finished_steps = 0;
3379 3379
             $nb_steps = count($processesProgress);
3380
-            foreach($processesProgress as $process) {
3380
+            foreach ($processesProgress as $process) {
3381 3381
                 if ((int) $process['is_in_progress'] === -1) {
3382
-                    $finished_steps ++;
3382
+                    $finished_steps++;
3383 3383
                 }
3384 3384
             }
3385 3385
 
@@ -3388,7 +3388,7 @@  discard block
 block discarded – undo
3388 3388
                     'error' => false,
3389 3389
                     'message' => '',
3390 3390
                     'user_id' => $user_id,
3391
-                    'status' => $finished_steps === $nb_steps ? 'finished' : number_format($finished_steps/$nb_steps*100, 0).'%',
3391
+                    'status' => $finished_steps === $nb_steps ? 'finished' : number_format($finished_steps / $nb_steps * 100, 0).'%',
3392 3392
                     'debug' => $finished_steps.",".$nb_steps,
3393 3393
                 ),
3394 3394
                 'encode'
@@ -3453,7 +3453,7 @@  discard block
 block discarded – undo
3453 3453
 
3454 3454
     // Get info about user
3455 3455
     $data_user = DB::queryfirstrow(
3456
-        'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
3456
+        'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
3457 3457
         WHERE id = %i',
3458 3458
         $value[1]
3459 3459
     );
@@ -3486,7 +3486,7 @@  discard block
 block discarded – undo
3486 3486
             logEvents(
3487 3487
                 $SETTINGS,
3488 3488
                 'user_mngt',
3489
-                'at_user_new_' . $value[0] . ':' . $value[1],
3489
+                'at_user_new_'.$value[0].':'.$value[1],
3490 3490
                 (string) $session->get('user-id'),
3491 3491
                 $session->get('user-login'),
3492 3492
                 filter_input(INPUT_POST, 'id', FILTER_SANITIZE_FULL_SPECIAL_CHARS)
@@ -3517,7 +3517,7 @@  discard block
 block discarded – undo
3517 3517
 
3518 3518
     // Get info about user
3519 3519
     $data_user = DB::queryfirstrow(
3520
-        'SELECT admin, isAdministratedByRole FROM ' . prefixTable('users') . '
3520
+        'SELECT admin, isAdministratedByRole FROM '.prefixTable('users').'
3521 3521
         WHERE id = %i',
3522 3522
         $value[1]
3523 3523
     );
Please login to merge, or discard this patch.
scripts/task_maintenance_purge_old_files.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -71,8 +71,8 @@  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';
75
-    include __DIR__. '/../includes/config/tp.config.php';
74
+    require_once __DIR__.'/../sources/main.functions.php';
75
+    include __DIR__.'/../includes/config/tp.config.php';
76 76
 
77 77
     if (isset($SETTINGS) === true) {
78 78
         //read folder
@@ -82,8 +82,8 @@  discard block
 block discarded – undo
82 82
                 //delete file FILES
83 83
                 while (false !== ($f = readdir($dir))) {
84 84
                     if ($f !== '.' && $f !== '..' && $f !== '.htaccess') {
85
-                        if (file_exists($dir . $f) && ((time() - filectime($dir . $f)) > 604800)) {
86
-                            fileDelete($dir . '/' . $f, $SETTINGS);
85
+                        if (file_exists($dir.$f) && ((time() - filectime($dir.$f)) > 604800)) {
86
+                            fileDelete($dir.'/'.$f, $SETTINGS);
87 87
                         }
88 88
                     }
89 89
                 }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
                 while (false !== ($f = readdir($dir))) {
103 103
                     if ($f !== '.' && $f !== '..') {
104 104
                         if (strpos($f, '_delete.') > 0) {
105
-                            fileDelete($SETTINGS['path_to_upload_folder'] . '/' . $f, $SETTINGS);
105
+                            fileDelete($SETTINGS['path_to_upload_folder'].'/'.$f, $SETTINGS);
106 106
                         }
107 107
                     }
108 108
                 }
Please login to merge, or discard this patch.
scripts/task_maintenance_rebuild_config_file.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -72,8 +72,8 @@
 block discarded – undo
72 72
 function rebuildConfigFile(): void
73 73
 {
74 74
     // Load expected files
75
-    require_once __DIR__. '/../sources/main.functions.php';
76
-    include __DIR__. '/../includes/config/tp.config.php';
75
+    require_once __DIR__.'/../sources/main.functions.php';
76
+    include __DIR__.'/../includes/config/tp.config.php';
77 77
 
78 78
     if (isset($SETTINGS) === true) {
79 79
         handleConfigFile('rebuild', $SETTINGS);
Please login to merge, or discard this patch.
api/Controller/Api/AuthController.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
         $arrQueryStringParams = $this->getQueryStringParams();
40 40
 
41 41
         if (strtoupper($requestMethod) === 'POST') {
42
-            require API_ROOT_PATH . "/Model/AuthModel.php";
42
+            require API_ROOT_PATH."/Model/AuthModel.php";
43 43
             try {
44 44
                 $authModel = new AuthModel();
45 45
                 $arrUser = $authModel->getUserAuth(
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
                 if (array_key_exists("token", $arrUser)) {
51 51
                     $responseData = json_encode($arrUser);
52 52
                 } else {
53
-                    $strErrorDesc = $arrUser['error'] . " (" . $arrUser['info'] . ")";
53
+                    $strErrorDesc = $arrUser['error']." (".$arrUser['info'].")";
54 54
                     $strErrorHeader = 'HTTP/1.1 401 Unauthorized';
55 55
                 }
56 56
             } catch (Error $e) {
Please login to merge, or discard this patch.