Passed
Pull Request — master (#5009)
by
unknown
05:57
created
sources/main.functions.php 1 patch
Spacing   +224 added lines, -226 removed lines patch added patch discarded remove patch
@@ -99,19 +99,19 @@  discard block
 block discarded – undo
99 99
             $text = Crypto::decrypt($message, $key);
100 100
         }
101 101
     } catch (CryptoException\WrongKeyOrModifiedCiphertextException $ex) {
102
-        error_log('TEAMPASS-Error-Wrong key or modified ciphertext: ' . $ex->getMessage());
102
+        error_log('TEAMPASS-Error-Wrong key or modified ciphertext: '.$ex->getMessage());
103 103
         $err = 'wrong_key_or_modified_ciphertext';
104 104
     } catch (CryptoException\BadFormatException $ex) {
105
-        error_log('TEAMPASS-Error-Bad format exception: ' . $ex->getMessage());
105
+        error_log('TEAMPASS-Error-Bad format exception: '.$ex->getMessage());
106 106
         $err = 'bad_format';
107 107
     } catch (CryptoException\EnvironmentIsBrokenException $ex) {
108
-        error_log('TEAMPASS-Error-Environment: ' . $ex->getMessage());
108
+        error_log('TEAMPASS-Error-Environment: '.$ex->getMessage());
109 109
         $err = 'environment_error';
110 110
     } catch (CryptoException\IOException $ex) {
111
-        error_log('TEAMPASS-Error-IO: ' . $ex->getMessage());
111
+        error_log('TEAMPASS-Error-IO: '.$ex->getMessage());
112 112
         $err = 'io_error';
113 113
     } catch (Exception $ex) {
114
-        error_log('TEAMPASS-Error-Unexpected exception: ' . $ex->getMessage());
114
+        error_log('TEAMPASS-Error-Unexpected exception: '.$ex->getMessage());
115 115
         $err = 'unexpected_error';
116 116
     }
117 117
 
@@ -196,7 +196,7 @@  discard block
 block discarded – undo
196 196
  */
197 197
 function trimElement($chaine, string $element): string
198 198
 {
199
-    if (! empty($chaine)) {
199
+    if (!empty($chaine)) {
200 200
         if (is_array($chaine) === true) {
201 201
             $chaine = implode(';', $chaine);
202 202
         }
@@ -244,8 +244,8 @@  discard block
 block discarded – undo
244 244
  */
245 245
 function db_error_handler(array $params): void
246 246
 {
247
-    echo 'Error: ' . $params['error'] . "<br>\n";
248
-    echo 'Query: ' . $params['query'] . "<br>\n";
247
+    echo 'Error: '.$params['error']."<br>\n";
248
+    echo 'Query: '.$params['query']."<br>\n";
249 249
     throw new Exception('Error - Query', 1);
250 250
 }
251 251
 
@@ -319,7 +319,7 @@  discard block
 block discarded – undo
319 319
     $session->set('user-forbiden_personal_folders', []);
320 320
     
321 321
     // Get list of Folders
322
-    $rows = DB::query('SELECT id FROM ' . prefixTable('nested_tree') . ' WHERE personal_folder = %i', 0);
322
+    $rows = DB::query('SELECT id FROM '.prefixTable('nested_tree').' WHERE personal_folder = %i', 0);
323 323
     foreach ($rows as $record) {
324 324
         array_push($groupesVisibles, $record['id']);
325 325
     }
@@ -328,20 +328,20 @@  discard block
 block discarded – undo
328 328
     // get complete list of ROLES
329 329
     $tmp = array_filter(explode(';', $idFonctions !== null ? $idFonctions : ''));
330 330
     $rows = DB::query(
331
-        'SELECT * FROM ' . prefixTable('roles_title') . '
331
+        'SELECT * FROM '.prefixTable('roles_title').'
332 332
         ORDER BY title ASC'
333 333
     );
334 334
     foreach ($rows as $record) {
335
-        if (! empty($record['id']) && ! in_array($record['id'], $tmp)) {
335
+        if (!empty($record['id']) && !in_array($record['id'], $tmp)) {
336 336
             array_push($tmp, $record['id']);
337 337
         }
338 338
     }
339 339
     $session->set('user-roles', implode(';', $tmp));
340 340
     $session->set('user-admin', 1);
341 341
     // Check if admin has created Folders and Roles
342
-    DB::query('SELECT * FROM ' . prefixTable('nested_tree') . '');
342
+    DB::query('SELECT * FROM '.prefixTable('nested_tree').'');
343 343
     $session->set('user-nb_folders', DB::count());
344
-    DB::query('SELECT * FROM ' . prefixTable('roles_title'));
344
+    DB::query('SELECT * FROM '.prefixTable('roles_title'));
345 345
     $session->set('user-nb_roles', DB::count());
346 346
 
347 347
     return true;
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
424 424
     // Does this user is allowed to see other items
425 425
     $inc = 0;
426 426
     $rows = DB::query(
427
-        'SELECT id, id_tree FROM ' . prefixTable('items') . '
427
+        'SELECT id, id_tree FROM '.prefixTable('items').'
428 428
             WHERE restricted_to LIKE %ss AND inactif = %s'.
429 429
             (count($allowedFolders) > 0 ? ' AND id_tree NOT IN ('.implode(',', $allowedFolders).')' : ''),
430 430
         $globalsUserId,
@@ -441,8 +441,8 @@  discard block
 block discarded – undo
441 441
     // Check for the users roles if some specific rights exist on items
442 442
     $rows = DB::query(
443 443
         'SELECT i.id_tree, r.item_id
444
-        FROM ' . prefixTable('items') . ' as i
445
-        INNER JOIN ' . prefixTable('restriction_to_roles') . ' as r ON (r.item_id=i.id)
444
+        FROM ' . prefixTable('items').' as i
445
+        INNER JOIN ' . prefixTable('restriction_to_roles').' as r ON (r.item_id=i.id)
446 446
         WHERE i.id_tree <> "" '.
447 447
         (count($userRoles) > 0 ? 'AND r.role_id IN %li ' : '').
448 448
         'ORDER BY i.id_tree ASC',
@@ -486,18 +486,18 @@  discard block
 block discarded – undo
486 486
     $session->set('system-list_restricted_folders_for_items', $restrictedFoldersForItems);
487 487
     $session->set('user-forbiden_personal_folders', $noAccessPersonalFolders);
488 488
     // Folders and Roles numbers
489
-    DB::queryFirstRow('SELECT id FROM ' . prefixTable('nested_tree') . '');
490
-    DB::queryFirstRow('SELECT id FROM ' . prefixTable('nested_tree') . '');
489
+    DB::queryFirstRow('SELECT id FROM '.prefixTable('nested_tree').'');
490
+    DB::queryFirstRow('SELECT id FROM '.prefixTable('nested_tree').'');
491 491
     $session->set('user-nb_folders', DB::count());
492
-    DB::queryFirstRow('SELECT id FROM ' . prefixTable('roles_title'));
493
-    DB::queryFirstRow('SELECT id FROM ' . prefixTable('roles_title'));
492
+    DB::queryFirstRow('SELECT id FROM '.prefixTable('roles_title'));
493
+    DB::queryFirstRow('SELECT id FROM '.prefixTable('roles_title'));
494 494
     $session->set('user-nb_roles', DB::count());
495 495
     // check if change proposals on User's items
496 496
     if (isset($SETTINGS['enable_suggestion']) === true && (int) $SETTINGS['enable_suggestion'] === 1) {
497 497
         $countNewItems = DB::query(
498 498
             'SELECT COUNT(*)
499
-            FROM ' . prefixTable('items_change') . ' AS c
500
-            LEFT JOIN ' . prefixTable('log_items') . ' AS i ON (c.item_id = i.id_item)
499
+            FROM ' . prefixTable('items_change').' AS c
500
+            LEFT JOIN ' . prefixTable('log_items').' AS i ON (c.item_id = i.id_item)
501 501
             WHERE i.action = %s AND i.id_user = %i',
502 502
             'at_creation',
503 503
             $globalsUserId
@@ -524,7 +524,7 @@  discard block
 block discarded – undo
524 524
 {
525 525
     $rows = DB::query(
526 526
         'SELECT *
527
-        FROM ' . prefixTable('roles_values') . '
527
+        FROM ' . prefixTable('roles_values').'
528 528
         WHERE type IN %ls'.(count($userRoles) > 0 ? ' AND role_id IN %li' : ''),
529 529
         ['W', 'ND', 'NE', 'NDNE', 'R'],
530 530
         $userRoles,
@@ -591,7 +591,7 @@  discard block
 block discarded – undo
591 591
     ) {
592 592
         $persoFld = DB::queryFirstRow(
593 593
             'SELECT id
594
-            FROM ' . prefixTable('nested_tree') . '
594
+            FROM ' . prefixTable('nested_tree').'
595 595
             WHERE title = %s AND personal_folder = %i'.
596 596
             (count($allowedFolders) > 0 ? ' AND id NOT IN ('.implode(',', $allowedFolders).')' : ''),
597 597
             $globalsUserId,
@@ -624,7 +624,7 @@  discard block
 block discarded – undo
624 624
     }
625 625
     $persoFlds = DB::query(
626 626
         'SELECT id
627
-        FROM ' . prefixTable('nested_tree') . '
627
+        FROM ' . prefixTable('nested_tree').'
628 628
         WHERE %l',
629 629
         $where
630 630
     );
@@ -692,12 +692,12 @@  discard block
 block discarded – undo
692 692
     //Load Tree
693 693
     $tree = new NestedTree(prefixTable('nested_tree'), 'id', 'parent_id', 'title');
694 694
     // truncate table
695
-    DB::query('TRUNCATE TABLE ' . prefixTable('cache'));
695
+    DB::query('TRUNCATE TABLE '.prefixTable('cache'));
696 696
     // reload date
697 697
     $rows = DB::query(
698 698
         'SELECT *
699
-        FROM ' . prefixTable('items') . ' as i
700
-        INNER JOIN ' . prefixTable('log_items') . ' as l ON (l.id_item = i.id)
699
+        FROM ' . prefixTable('items').' as i
700
+        INNER JOIN ' . prefixTable('log_items').' as l ON (l.id_item = i.id)
701 701
         AND l.action = %s
702 702
         AND i.inactif = %i',
703 703
         'at_creation',
@@ -709,18 +709,18 @@  discard block
 block discarded – undo
709 709
             $tags = '';
710 710
             $itemTags = DB::query(
711 711
                 'SELECT tag
712
-                FROM ' . prefixTable('tags') . '
712
+                FROM ' . prefixTable('tags').'
713 713
                 WHERE item_id = %i AND tag != ""',
714 714
                 $record['id']
715 715
             );
716 716
             foreach ($itemTags as $itemTag) {
717
-                $tags .= $itemTag['tag'] . ' ';
717
+                $tags .= $itemTag['tag'].' ';
718 718
             }
719 719
 
720 720
             // Get renewal period
721 721
             $resNT = DB::queryFirstRow(
722 722
                 'SELECT renewal_period
723
-                FROM ' . prefixTable('nested_tree') . '
723
+                FROM ' . prefixTable('nested_tree').'
724 724
                 WHERE id = %i',
725 725
                 $record['id_tree']
726 726
             );
@@ -733,7 +733,7 @@  discard block
 block discarded – undo
733 733
                     // Is this a User id?
734 734
                     $user = DB::queryFirstRow(
735 735
                         'SELECT login
736
-                        FROM ' . prefixTable('users') . '
736
+                        FROM ' . prefixTable('users').'
737 737
                         WHERE id = %i',
738 738
                         $elem->title
739 739
                     );
@@ -751,11 +751,11 @@  discard block
 block discarded – undo
751 751
                     'id' => $record['id'],
752 752
                     'label' => $record['label'],
753 753
                     'description' => $record['description'] ?? '',
754
-                    'url' => isset($record['url']) && ! empty($record['url']) ? $record['url'] : '0',
754
+                    'url' => isset($record['url']) && !empty($record['url']) ? $record['url'] : '0',
755 755
                     'tags' => $tags,
756 756
                     'id_tree' => $record['id_tree'],
757 757
                     'perso' => $record['perso'],
758
-                    'restricted_to' => isset($record['restricted_to']) && ! empty($record['restricted_to']) ? $record['restricted_to'] : '0',
758
+                    'restricted_to' => isset($record['restricted_to']) && !empty($record['restricted_to']) ? $record['restricted_to'] : '0',
759 759
                     'login' => $record['login'] ?? '',
760 760
                     'folder' => implode(' » ', $folder),
761 761
                     'author' => $record['id_user'],
@@ -784,7 +784,7 @@  discard block
 block discarded – undo
784 784
     // get new value from db
785 785
     $data = DB::queryFirstRow(
786 786
         'SELECT label, description, id_tree, perso, restricted_to, login, url
787
-        FROM ' . prefixTable('items') . '
787
+        FROM ' . prefixTable('items').'
788 788
         WHERE id=%i',
789 789
         $ident
790 790
     );
@@ -792,12 +792,12 @@  discard block
 block discarded – undo
792 792
     $tags = '';
793 793
     $itemTags = DB::query(
794 794
         'SELECT tag
795
-            FROM ' . prefixTable('tags') . '
795
+            FROM ' . prefixTable('tags').'
796 796
             WHERE item_id = %i AND tag != ""',
797 797
         $ident
798 798
     );
799 799
     foreach ($itemTags as $itemTag) {
800
-        $tags .= $itemTag['tag'] . ' ';
800
+        $tags .= $itemTag['tag'].' ';
801 801
     }
802 802
     // form id_tree to full foldername
803 803
     $folder = [];
@@ -808,7 +808,7 @@  discard block
 block discarded – undo
808 808
             // Is this a User id?
809 809
             $user = DB::queryFirstRow(
810 810
                 'SELECT id, login
811
-                FROM ' . prefixTable('users') . '
811
+                FROM ' . prefixTable('users').'
812 812
                 WHERE id = %i',
813 813
                 $elem->title
814 814
             );
@@ -826,10 +826,10 @@  discard block
 block discarded – undo
826 826
             'label' => $data['label'],
827 827
             'description' => $data['description'],
828 828
             'tags' => $tags,
829
-            'url' => isset($data['url']) && ! empty($data['url']) ? $data['url'] : '0',
829
+            'url' => isset($data['url']) && !empty($data['url']) ? $data['url'] : '0',
830 830
             'id_tree' => $data['id_tree'],
831 831
             'perso' => $data['perso'],
832
-            'restricted_to' => isset($data['restricted_to']) && ! empty($data['restricted_to']) ? $data['restricted_to'] : '0',
832
+            'restricted_to' => isset($data['restricted_to']) && !empty($data['restricted_to']) ? $data['restricted_to'] : '0',
833 833
             'login' => $data['login'] ?? '',
834 834
             'folder' => implode(' » ', $folder),
835 835
             'author' => $session->get('user-id'),
@@ -859,8 +859,8 @@  discard block
 block discarded – undo
859 859
     // get new value from db
860 860
     $data = DB::queryFirstRow(
861 861
         'SELECT i.label, i.description, i.id_tree as id_tree, i.perso, i.restricted_to, i.id, i.login, i.url, l.date
862
-        FROM ' . prefixTable('items') . ' as i
863
-        INNER JOIN ' . prefixTable('log_items') . ' as l ON (l.id_item = i.id)
862
+        FROM ' . prefixTable('items').' as i
863
+        INNER JOIN ' . prefixTable('log_items').' as l ON (l.id_item = i.id)
864 864
         WHERE i.id = %i
865 865
         AND l.action = %s',
866 866
         $ident,
@@ -870,12 +870,12 @@  discard block
 block discarded – undo
870 870
     $tags = '';
871 871
     $itemTags = DB::query(
872 872
         'SELECT tag
873
-            FROM ' . prefixTable('tags') . '
873
+            FROM ' . prefixTable('tags').'
874 874
             WHERE item_id = %i AND tag != ""',
875 875
         $ident
876 876
     );
877 877
     foreach ($itemTags as $itemTag) {
878
-        $tags .= $itemTag['tag'] . ' ';
878
+        $tags .= $itemTag['tag'].' ';
879 879
     }
880 880
     // form id_tree to full foldername
881 881
     $folder = [];
@@ -886,7 +886,7 @@  discard block
 block discarded – undo
886 886
             // Is this a User id?
887 887
             $user = DB::queryFirstRow(
888 888
                 'SELECT id, login
889
-                FROM ' . prefixTable('users') . '
889
+                FROM ' . prefixTable('users').'
890 890
                 WHERE id = %i',
891 891
                 $elem->title
892 892
             );
@@ -905,7 +905,7 @@  discard block
 block discarded – undo
905 905
             'label' => $data['label'],
906 906
             'description' => $data['description'],
907 907
             'tags' => empty($tags) === false ? $tags : 'None',
908
-            'url' => isset($data['url']) && ! empty($data['url']) ? $data['url'] : '0',
908
+            'url' => isset($data['url']) && !empty($data['url']) ? $data['url'] : '0',
909 909
             'id_tree' => $data['id_tree'],
910 910
             'perso' => isset($data['perso']) && empty($data['perso']) === false && $data['perso'] !== 'None' ? $data['perso'] : '0',
911 911
             'restricted_to' => isset($data['restricted_to']) && empty($data['restricted_to']) === false ? $data['restricted_to'] : '0',
@@ -927,53 +927,53 @@  discard block
 block discarded – undo
927 927
 function getStatisticsData(array $SETTINGS): array
928 928
 {
929 929
     DB::query(
930
-        'SELECT id FROM ' . prefixTable('nested_tree') . ' WHERE personal_folder = %i',
930
+        'SELECT id FROM '.prefixTable('nested_tree').' WHERE personal_folder = %i',
931 931
         0
932 932
     );
933 933
     $counter_folders = DB::count();
934 934
     DB::query(
935
-        'SELECT id FROM ' . prefixTable('nested_tree') . ' WHERE personal_folder = %i',
935
+        'SELECT id FROM '.prefixTable('nested_tree').' WHERE personal_folder = %i',
936 936
         1
937 937
     );
938 938
     $counter_folders_perso = DB::count();
939 939
     DB::query(
940
-        'SELECT id FROM ' . prefixTable('items') . ' WHERE perso = %i',
940
+        'SELECT id FROM '.prefixTable('items').' WHERE perso = %i',
941 941
         0
942 942
     );
943 943
     $counter_items = DB::count();
944 944
         DB::query(
945
-        'SELECT id FROM ' . prefixTable('items') . ' WHERE perso = %i',
945
+        'SELECT id FROM '.prefixTable('items').' WHERE perso = %i',
946 946
         1
947 947
     );
948 948
     $counter_items_perso = DB::count();
949 949
         DB::query(
950
-        'SELECT id FROM ' . prefixTable('users') . ' WHERE login NOT IN (%s, %s, %s)',
950
+        'SELECT id FROM '.prefixTable('users').' WHERE login NOT IN (%s, %s, %s)',
951 951
         'OTV', 'TP', 'API'
952 952
     );
953 953
     $counter_users = DB::count();
954 954
         DB::query(
955
-        'SELECT id FROM ' . prefixTable('users') . ' WHERE admin = %i',
955
+        'SELECT id FROM '.prefixTable('users').' WHERE admin = %i',
956 956
         1
957 957
     );
958 958
     $admins = DB::count();
959 959
     DB::query(
960
-        'SELECT id FROM ' . prefixTable('users') . ' WHERE gestionnaire = %i',
960
+        'SELECT id FROM '.prefixTable('users').' WHERE gestionnaire = %i',
961 961
         1
962 962
     );
963 963
     $managers = DB::count();
964 964
     DB::query(
965
-        'SELECT id FROM ' . prefixTable('users') . ' WHERE read_only = %i',
965
+        'SELECT id FROM '.prefixTable('users').' WHERE read_only = %i',
966 966
         1
967 967
     );
968 968
     $readOnly = DB::count();
969 969
     // list the languages
970 970
     $usedLang = [];
971 971
     $tp_languages = DB::query(
972
-        'SELECT name FROM ' . prefixTable('languages')
972
+        'SELECT name FROM '.prefixTable('languages')
973 973
     );
974 974
     foreach ($tp_languages as $tp_language) {
975 975
         DB::query(
976
-            'SELECT * FROM ' . prefixTable('users') . ' WHERE user_language = %s',
976
+            'SELECT * FROM '.prefixTable('users').' WHERE user_language = %s',
977 977
             $tp_language['name']
978 978
         );
979 979
         $usedLang[$tp_language['name']] = round((DB::count() * 100 / $counter_users), 0);
@@ -982,12 +982,12 @@  discard block
 block discarded – undo
982 982
     // get list of ips
983 983
     $usedIp = [];
984 984
     $tp_ips = DB::query(
985
-        'SELECT user_ip FROM ' . prefixTable('users')
985
+        'SELECT user_ip FROM '.prefixTable('users')
986 986
     );
987 987
     foreach ($tp_ips as $ip) {
988 988
         if (array_key_exists($ip['user_ip'], $usedIp)) {
989 989
             $usedIp[$ip['user_ip']] += $usedIp[$ip['user_ip']];
990
-        } elseif (! empty($ip['user_ip']) && $ip['user_ip'] !== 'none') {
990
+        } elseif (!empty($ip['user_ip']) && $ip['user_ip'] !== 'none') {
991 991
             $usedIp[$ip['user_ip']] = 1;
992 992
         }
993 993
     }
@@ -1082,7 +1082,7 @@  discard block
 block discarded – undo
1082 1082
         <table width="600" cellpadding="0" cellspacing="0" border="0" class="container" bgcolor="#ffffff" style="border-spacing: 0; border-bottom: 1px solid #e0e0e0; box-shadow: 0 0 3px #ddd; color: #434343; font-family: Helvetica, Verdana, sans-serif;">
1083 1083
         <tr><td class="container-padding" bgcolor="#ffffff" style="border-collapse: collapse; border-left: 1px solid #e0e0e0; background-color: #ffffff; padding-left: 30px; padding-right: 30px;">
1084 1084
         <br><div style="float:right;">' .
1085
-        $textMail .
1085
+        $textMail.
1086 1086
         '<br><br></td></tr></table>
1087 1087
     </td></tr></table>
1088 1088
     <br></body></html>';
@@ -1163,7 +1163,7 @@  discard block
 block discarded – undo
1163 1163
 {
1164 1164
     array_walk_recursive(
1165 1165
         $array,
1166
-        static function (&$item): void {
1166
+        static function(&$item): void {
1167 1167
             if (mb_detect_encoding((string) $item, 'utf-8', true) === false) {
1168 1168
                 $item = mb_convert_encoding($item, 'ISO-8859-1', 'UTF-8');
1169 1169
             }
@@ -1274,7 +1274,7 @@  discard block
 block discarded – undo
1274 1274
  */
1275 1275
 function prefixTable(string $table): string
1276 1276
 {
1277
-    $safeTable = htmlspecialchars(DB_PREFIX . $table);
1277
+    $safeTable = htmlspecialchars(DB_PREFIX.$table);
1278 1278
     return $safeTable;
1279 1279
 }
1280 1280
 
@@ -1391,7 +1391,7 @@  discard block
 block discarded – undo
1391 1391
 function send_syslog($message, $host, $port, $component = 'teampass'): void
1392 1392
 {
1393 1393
     $sock = socket_create(AF_INET, SOCK_DGRAM, SOL_UDP);
1394
-    $syslog_message = '<123>' . date('M d H:i:s ') . $component . ': ' . $message;
1394
+    $syslog_message = '<123>'.date('M d H:i:s ').$component.': '.$message;
1395 1395
     socket_sendto($sock, (string) $syslog_message, strlen($syslog_message), 0, (string) $host, (int) $port);
1396 1396
     socket_close($sock);
1397 1397
 }
@@ -1438,14 +1438,14 @@  discard block
 block discarded – undo
1438 1438
     if (isset($SETTINGS['syslog_enable']) === true && (int) $SETTINGS['syslog_enable'] === 1) {
1439 1439
         if ($type === 'user_mngt') {
1440 1440
             send_syslog(
1441
-                'action=' . str_replace('at_', '', $label) . ' attribute=user user=' . $who . ' userid="' . $login . '" change="' . $field_1 . '" ',
1441
+                'action='.str_replace('at_', '', $label).' attribute=user user='.$who.' userid="'.$login.'" change="'.$field_1.'" ',
1442 1442
                 $SETTINGS['syslog_host'],
1443 1443
                 $SETTINGS['syslog_port'],
1444 1444
                 'teampass'
1445 1445
             );
1446 1446
         } else {
1447 1447
             send_syslog(
1448
-                'action=' . $type . ' attribute=' . $label . ' user=' . $who . ' userid="' . $login . '" ',
1448
+                'action='.$type.' attribute='.$label.' user='.$who.' userid="'.$login.'" ',
1449 1449
                 $SETTINGS['syslog_host'],
1450 1450
                 $SETTINGS['syslog_port'],
1451 1451
                 'teampass'
@@ -1520,7 +1520,7 @@  discard block
 block discarded – undo
1520 1520
         if (empty($item_label) === true) {
1521 1521
             $dataItem = DB::queryFirstRow(
1522 1522
                 'SELECT id, id_tree, label
1523
-                FROM ' . prefixTable('items') . '
1523
+                FROM ' . prefixTable('items').'
1524 1524
                 WHERE id = %i',
1525 1525
                 $item_id
1526 1526
             );
@@ -1528,11 +1528,11 @@  discard block
 block discarded – undo
1528 1528
         }
1529 1529
 
1530 1530
         send_syslog(
1531
-            'action=' . str_replace('at_', '', $action) .
1532
-                ' attribute=' . str_replace('at_', '', $attribute[0]) .
1533
-                ' itemno=' . $item_id .
1534
-                ' user=' . (is_null($login) === true ? '' : addslashes((string) $login)) .
1535
-                ' itemname="' . addslashes($item_label) . '"',
1531
+            'action='.str_replace('at_', '', $action).
1532
+                ' attribute='.str_replace('at_', '', $attribute[0]).
1533
+                ' itemno='.$item_id.
1534
+                ' user='.(is_null($login) === true ? '' : addslashes((string) $login)).
1535
+                ' itemname="'.addslashes($item_label).'"',
1536 1536
             $SETTINGS['syslog_host'],
1537 1537
             $SETTINGS['syslog_port'],
1538 1538
             'teampass'
@@ -1563,8 +1563,8 @@  discard block
 block discarded – undo
1563 1563
     // send email to user that what to be notified
1564 1564
     $notification = DB::queryFirstField(
1565 1565
         'SELECT email
1566
-        FROM ' . prefixTable('notification') . ' AS n
1567
-        INNER JOIN ' . prefixTable('users') . ' AS u ON (n.user_id = u.id)
1566
+        FROM ' . prefixTable('notification').' AS n
1567
+        INNER JOIN ' . prefixTable('users').' AS u ON (n.user_id = u.id)
1568 1568
         WHERE n.item_id = %i AND n.user_id != %i',
1569 1569
         $item_id,
1570 1570
         $globalsUserId
@@ -1575,7 +1575,7 @@  discard block
 block discarded – undo
1575 1575
         // Get list of changes
1576 1576
         $htmlChanges = '<ul>';
1577 1577
         foreach ($changes as $change) {
1578
-            $htmlChanges .= '<li>' . $change . '</li>';
1578
+            $htmlChanges .= '<li>'.$change.'</li>';
1579 1579
         }
1580 1580
         $htmlChanges .= '</ul>';
1581 1581
         // send email
@@ -1612,15 +1612,15 @@  discard block
 block discarded – undo
1612 1612
     $path = '';
1613 1613
     foreach ($arbo as $elem) {
1614 1614
         if (empty($path) === true) {
1615
-            $path = htmlspecialchars(stripslashes(htmlspecialchars_decode($elem->title, ENT_QUOTES)), ENT_QUOTES) . ' ';
1615
+            $path = htmlspecialchars(stripslashes(htmlspecialchars_decode($elem->title, ENT_QUOTES)), ENT_QUOTES).' ';
1616 1616
         } else {
1617
-            $path .= '&#8594; ' . htmlspecialchars(stripslashes(htmlspecialchars_decode($elem->title, ENT_QUOTES)), ENT_QUOTES);
1617
+            $path .= '&#8594; '.htmlspecialchars(stripslashes(htmlspecialchars_decode($elem->title, ENT_QUOTES)), ENT_QUOTES);
1618 1618
         }
1619 1619
     }
1620 1620
 
1621 1621
     // Build text to show user
1622 1622
     if (empty($label) === false) {
1623
-        return empty($path) === true ? addslashes($label) : addslashes($label) . ' (' . $path . ')';
1623
+        return empty($path) === true ? addslashes($label) : addslashes($label).' ('.$path.')';
1624 1624
     }
1625 1625
     return empty($path) === true ? '' : $path;
1626 1626
 }
@@ -1676,7 +1676,7 @@  discard block
 block discarded – undo
1676 1676
 {
1677 1677
     // Perform a copy if the file exists
1678 1678
     if (file_exists($configFilePath)) {
1679
-        $backupFilePath = $configFilePath . '.' . date('Y_m_d_His', time());
1679
+        $backupFilePath = $configFilePath.'.'.date('Y_m_d_His', time());
1680 1680
         if (!copy($configFilePath, $backupFilePath)) {
1681 1681
             return "ERROR: Could not copy file '$configFilePath'";
1682 1682
         }
@@ -1684,10 +1684,10 @@  discard block
 block discarded – undo
1684 1684
 
1685 1685
     // Regenerate the config file
1686 1686
     $data = ["<?php\n", "global \$SETTINGS;\n", "\$SETTINGS = array (\n"];
1687
-    $rows = DB::query('SELECT * FROM ' . prefixTable('misc') . ' WHERE type=%s', 'admin');
1687
+    $rows = DB::query('SELECT * FROM '.prefixTable('misc').' WHERE type=%s', 'admin');
1688 1688
     foreach ($rows as $record) {
1689 1689
         $value = getEncryptedValue($record['valeur'], $record['is_encrypted']);
1690
-        $data[] = "    '{$record['intitule']}' => '". htmlspecialchars_decode($value, ENT_COMPAT) . "',\n";
1690
+        $data[] = "    '{$record['intitule']}' => '".htmlspecialchars_decode($value, ENT_COMPAT)."',\n";
1691 1691
     }
1692 1692
     $data[] = ");\n";
1693 1693
     $data = array_unique($data);
@@ -1732,7 +1732,7 @@  discard block
 block discarded – undo
1732 1732
 {
1733 1733
     global $SETTINGS;
1734 1734
     /* LOAD CPASSMAN SETTINGS */
1735
-    if (! isset($SETTINGS['loaded']) || $SETTINGS['loaded'] !== 1) {
1735
+    if (!isset($SETTINGS['loaded']) || $SETTINGS['loaded'] !== 1) {
1736 1736
         $SETTINGS = [];
1737 1737
         $SETTINGS['duplicate_folder'] = 0;
1738 1738
         //by default, this is set to 0;
@@ -1742,7 +1742,7 @@  discard block
 block discarded – undo
1742 1742
         //by default, this value is set to 5;
1743 1743
         $settings = [];
1744 1744
         $rows = DB::query(
1745
-            'SELECT * FROM ' . prefixTable('misc') . ' WHERE type=%s_type OR type=%s_type2',
1745
+            'SELECT * FROM '.prefixTable('misc').' WHERE type=%s_type OR type=%s_type2',
1746 1746
             [
1747 1747
                 'type' => 'admin',
1748 1748
                 'type2' => 'settings',
@@ -1774,7 +1774,7 @@  discard block
 block discarded – undo
1774 1774
     $source_cf = [];
1775 1775
     $rows = DB::query(
1776 1776
         'SELECT id_category
1777
-            FROM ' . prefixTable('categories_folders') . '
1777
+            FROM ' . prefixTable('categories_folders').'
1778 1778
             WHERE id_folder = %i',
1779 1779
         $source_id
1780 1780
     );
@@ -1785,7 +1785,7 @@  discard block
 block discarded – undo
1785 1785
     $target_cf = [];
1786 1786
     $rows = DB::query(
1787 1787
         'SELECT id_category
1788
-            FROM ' . prefixTable('categories_folders') . '
1788
+            FROM ' . prefixTable('categories_folders').'
1789 1789
             WHERE id_folder = %i',
1790 1790
         $target_id
1791 1791
     );
@@ -1880,10 +1880,10 @@  discard block
 block discarded – undo
1880 1880
     } catch (CryptoException\WrongKeyOrModifiedCiphertextException $ex) {
1881 1881
         $err = 'wrong_key';
1882 1882
     } catch (CryptoException\EnvironmentIsBrokenException $ex) {
1883
-        error_log('TEAMPASS-Error-Environment: ' . $ex->getMessage());
1883
+        error_log('TEAMPASS-Error-Environment: '.$ex->getMessage());
1884 1884
         $err = 'environment_error';
1885 1885
     } catch (CryptoException\IOException $ex) {
1886
-        error_log('TEAMPASS-Error-General: ' . $ex->getMessage());
1886
+        error_log('TEAMPASS-Error-General: '.$ex->getMessage());
1887 1887
         $err = 'general_error';
1888 1888
     }
1889 1889
 
@@ -1915,10 +1915,10 @@  discard block
 block discarded – undo
1915 1915
     } catch (CryptoException\WrongKeyOrModifiedCiphertextException $ex) {
1916 1916
         $err = 'wrong_key';
1917 1917
     } catch (CryptoException\EnvironmentIsBrokenException $ex) {
1918
-        error_log('TEAMPASS-Error-Environment: ' . $ex->getMessage());
1918
+        error_log('TEAMPASS-Error-Environment: '.$ex->getMessage());
1919 1919
         $err = 'environment_error';
1920 1920
     } catch (CryptoException\IOException $ex) {
1921
-        error_log('TEAMPASS-Error-General: ' . $ex->getMessage());
1921
+        error_log('TEAMPASS-Error-General: '.$ex->getMessage());
1922 1922
         $err = 'general_error';
1923 1923
     }
1924 1924
 
@@ -2004,7 +2004,7 @@  discard block
 block discarded – undo
2004 2004
 ) {
2005 2005
     // Check if the path exists
2006 2006
     $path = basename($path);
2007
-    if (! file_exists($path)) {
2007
+    if (!file_exists($path)) {
2008 2008
         return false;
2009 2009
     }
2010 2010
 
@@ -2057,7 +2057,7 @@  discard block
 block discarded – undo
2057 2057
     // Load item data
2058 2058
     $data = DB::queryFirstRow(
2059 2059
         'SELECT id_tree
2060
-        FROM ' . prefixTable('items') . '
2060
+        FROM ' . prefixTable('items').'
2061 2061
         WHERE id = %i',
2062 2062
         $item_id
2063 2063
     );
@@ -2120,7 +2120,7 @@  discard block
 block discarded – undo
2120 2120
         }
2121 2121
         $host .= substr(explode(".", $email[1])[0], -1, 1);
2122 2122
     }
2123
-    $email = $name . "@" . $host . "." . explode(".", $email[1])[1];
2123
+    $email = $name."@".$host.".".explode(".", $email[1])[1];
2124 2124
     return $email;
2125 2125
 }
2126 2126
 
@@ -2151,11 +2151,11 @@  discard block
 block discarded – undo
2151 2151
 function formatSizeUnits(int $bytes): string
2152 2152
 {
2153 2153
     if ($bytes >= 1073741824) {
2154
-        $bytes = number_format($bytes / 1073741824, 2) . ' GB';
2154
+        $bytes = number_format($bytes / 1073741824, 2).' GB';
2155 2155
     } elseif ($bytes >= 1048576) {
2156
-        $bytes = number_format($bytes / 1048576, 2) . ' MB';
2156
+        $bytes = number_format($bytes / 1048576, 2).' MB';
2157 2157
     } elseif ($bytes >= 1024) {
2158
-        $bytes = number_format($bytes / 1024, 2) . ' KB';
2158
+        $bytes = number_format($bytes / 1024, 2).' KB';
2159 2159
     } elseif ($bytes > 1) {
2160 2160
         $bytes .= ' bytes';
2161 2161
     } elseif ($bytes === 1) {
@@ -2322,7 +2322,7 @@  discard block
 block discarded – undo
2322 2322
  */
2323 2323
 function generateKeyIntegrityHash(string $userSeed, string $publicKey, string $serverSecret): string
2324 2324
 {
2325
-    return hash_hmac('sha256', $userSeed . $publicKey, $serverSecret);
2325
+    return hash_hmac('sha256', $userSeed.$publicKey, $serverSecret);
2326 2326
 }
2327 2327
 
2328 2328
 /**
@@ -2397,7 +2397,7 @@  discard block
 block discarded – undo
2397 2397
                 'security_alert',
2398 2398
                 'key_integrity_check_failed',
2399 2399
                 (string) $userInfo['id'],
2400
-                'User: ' . $userInfo['login']
2400
+                'User: '.$userInfo['login']
2401 2401
             );
2402 2402
             return [
2403 2403
                 'success' => false,
@@ -2444,7 +2444,7 @@  discard block
 block discarded – undo
2444 2444
             'user_connection',
2445 2445
             'auto_reencryption_success',
2446 2446
             (string) $userInfo['id'],
2447
-            'User: ' . $userInfo['login']
2447
+            'User: '.$userInfo['login']
2448 2448
         );
2449 2449
 
2450 2450
         // Store in session for immediate use
@@ -2463,12 +2463,12 @@  discard block
 block discarded – undo
2463 2463
             'security_alert',
2464 2464
             'auto_reencryption_failed',
2465 2465
             (string) $userInfo['id'],
2466
-            'User: ' . $userInfo['login'] . ' - Error: ' . $e->getMessage()
2466
+            'User: '.$userInfo['login'].' - Error: '.$e->getMessage()
2467 2467
         );
2468 2468
 
2469 2469
         return [
2470 2470
             'success' => false,
2471
-            'error' => 'decryption_failed: ' . $e->getMessage(),
2471
+            'error' => 'decryption_failed: '.$e->getMessage(),
2472 2472
             'private_key_clear' => '',
2473 2473
         ];
2474 2474
     }
@@ -2539,7 +2539,7 @@  discard block
 block discarded – undo
2539 2539
         'security_alert',
2540 2540
         'auto_reencryption_critical_failure',
2541 2541
         (string) $userId,
2542
-        'User: ' . $userInfo['login'] . ' - disabled due to key recovery failure'
2542
+        'User: '.$userInfo['login'].' - disabled due to key recovery failure'
2543 2543
     );
2544 2544
 
2545 2545
     return false;
@@ -2697,7 +2697,7 @@  discard block
 block discarded – undo
2697 2697
     $cipher->enableContinuousBuffer();
2698 2698
 
2699 2699
     // Encrypt the file content
2700
-    $filePath = filter_var($fileInPath . '/' . $fileInName, FILTER_SANITIZE_URL);
2700
+    $filePath = filter_var($fileInPath.'/'.$fileInName, FILTER_SANITIZE_URL);
2701 2701
     $fileContent = file_get_contents($filePath);
2702 2702
     $plaintext = $fileContent;
2703 2703
     $ciphertext = $cipher->encrypt($plaintext);
@@ -2705,9 +2705,9 @@  discard block
 block discarded – undo
2705 2705
     // Save new file
2706 2706
     // deepcode ignore InsecureHash: is simply used to get a unique name
2707 2707
     $hash = uniqid('', true);
2708
-    $fileOut = $fileInPath . '/' . TP_FILE_PREFIX . $hash;
2708
+    $fileOut = $fileInPath.'/'.TP_FILE_PREFIX.$hash;
2709 2709
     file_put_contents($fileOut, $ciphertext);
2710
-    unlink($fileInPath . '/' . $fileInName);
2710
+    unlink($fileInPath.'/'.$fileInName);
2711 2711
     return [
2712 2712
         'fileHash' => base64_encode($hash),
2713 2713
         'objectKey' => base64_encode($objectKey),
@@ -2723,9 +2723,9 @@  discard block
 block discarded – undo
2723 2723
  *
2724 2724
  * @return string|array
2725 2725
  */
2726
-function decryptFile(string $fileName, string $filePath, string $key): string|array
2726
+function decryptFile(string $fileName, string $filePath, string $key): string | array
2727 2727
 {
2728
-    if (! defined('FILE_BUFFER_SIZE')) {
2728
+    if (!defined('FILE_BUFFER_SIZE')) {
2729 2729
         define('FILE_BUFFER_SIZE', 128 * 1024);
2730 2730
     }
2731 2731
     
@@ -2742,7 +2742,7 @@  discard block
 block discarded – undo
2742 2742
     $cipher->enableContinuousBuffer();
2743 2743
     $cipher->disablePadding();
2744 2744
     // Get file content
2745
-    $safeFilePath = realpath($filePath . '/' . TP_FILE_PREFIX . $safeFileName);
2745
+    $safeFilePath = realpath($filePath.'/'.TP_FILE_PREFIX.$safeFileName);
2746 2746
     if ($safeFilePath !== false && file_exists($safeFilePath)) {
2747 2747
         $ciphertext = file_get_contents(filter_var($safeFilePath, FILTER_SANITIZE_URL));
2748 2748
     } else {
@@ -2838,7 +2838,7 @@  discard block
 block discarded – undo
2838 2838
     }
2839 2839
     $users = DB::query(
2840 2840
         'SELECT id, public_key
2841
-        FROM ' . prefixTable('users') . '
2841
+        FROM ' . prefixTable('users').'
2842 2842
         WHERE id NOT IN %li
2843 2843
         AND public_key != ""',
2844 2844
         $user_ids
@@ -2847,7 +2847,7 @@  discard block
 block discarded – undo
2847 2847
         // Insert in DB the new object key for this item by user
2848 2848
         if (count($objectKeyArray) === 0) {
2849 2849
             if (WIP === true) {
2850
-                error_log('TEAMPASS Debug - storeUsersShareKey case1 - ' . $object_name . ' - ' . $post_object_id . ' - ' . $user['id']);
2850
+                error_log('TEAMPASS Debug - storeUsersShareKey case1 - '.$object_name.' - '.$post_object_id.' - '.$user['id']);
2851 2851
             }
2852 2852
             
2853 2853
             insertOrUpdateSharekey(
@@ -2859,7 +2859,7 @@  discard block
 block discarded – undo
2859 2859
         } else {
2860 2860
             foreach ($objectKeyArray as $object) {
2861 2861
                 if (WIP === true) {
2862
-                    error_log('TEAMPASS Debug - storeUsersShareKey case2 - ' . $object_name . ' - ' . $object['objectId'] . ' - ' . $user['id']);
2862
+                    error_log('TEAMPASS Debug - storeUsersShareKey case2 - '.$object_name.' - '.$object['objectId'].' - '.$user['id']);
2863 2863
                 }
2864 2864
                 
2865 2865
                 insertOrUpdateSharekey(
@@ -2891,7 +2891,7 @@  discard block
 block discarded – undo
2891 2891
 ): bool {
2892 2892
     try {
2893 2893
         DB::query(
2894
-            'INSERT INTO ' . $tableName . ' 
2894
+            'INSERT INTO '.$tableName.' 
2895 2895
             (object_id, user_id, share_key) 
2896 2896
             VALUES (%i, %i, %s)
2897 2897
             ON DUPLICATE KEY UPDATE share_key = VALUES(share_key)',
@@ -2901,7 +2901,7 @@  discard block
 block discarded – undo
2901 2901
         );
2902 2902
         return true;
2903 2903
     } catch (Exception $e) {
2904
-        error_log('TEAMPASS Error - insertOrUpdateSharekey: ' . $e->getMessage());
2904
+        error_log('TEAMPASS Error - insertOrUpdateSharekey: '.$e->getMessage());
2905 2905
         return false;
2906 2906
     }
2907 2907
 }
@@ -2916,7 +2916,7 @@  discard block
 block discarded – undo
2916 2916
 function isBase64(string $str): bool
2917 2917
 {
2918 2918
     $str = (string) trim($str);
2919
-    if (! isset($str[0])) {
2919
+    if (!isset($str[0])) {
2920 2920
         return false;
2921 2921
     }
2922 2922
 
@@ -2990,7 +2990,7 @@  discard block
 block discarded – undo
2990 2990
     } catch (\LdapRecord\Auth\BindException $e) {
2991 2991
         $error = $e->getDetailedError();
2992 2992
         if ($error && defined('LOG_TO_SERVER') && LOG_TO_SERVER === true) {
2993
-            error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
2993
+            error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
2994 2994
         }
2995 2995
         // deepcode ignore ServerLeak: No important data is sent
2996 2996
         echo 'An error occurred.';
@@ -3007,7 +3007,7 @@  discard block
 block discarded – undo
3007 3007
     } catch (\LdapRecord\Auth\BindException $e) {
3008 3008
         $error = $e->getDetailedError();
3009 3009
         if ($error && defined('LOG_TO_SERVER') && LOG_TO_SERVER === true) {
3010
-            error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage(). " - ".$error->getDiagnosticMessage());
3010
+            error_log('TEAMPASS Error - LDAP - '.$error->getErrorCode()." - ".$error->getErrorMessage()." - ".$error->getDiagnosticMessage());
3011 3011
         }
3012 3012
         // deepcode ignore ServerLeak: No important data is sent
3013 3013
         echo 'An error occurred.';
@@ -3043,7 +3043,7 @@  discard block
 block discarded – undo
3043 3043
     // expect if personal item
3044 3044
     DB::delete(
3045 3045
         prefixTable('sharekeys_items'),
3046
-        'user_id = %i',// AND object_id NOT IN (SELECT i.id FROM ' . prefixTable('items') . ' AS i WHERE i.perso = 1)'',
3046
+        'user_id = %i', // AND object_id NOT IN (SELECT i.id FROM ' . prefixTable('items') . ' AS i WHERE i.perso = 1)'',
3047 3047
         $userId
3048 3048
     );
3049 3049
     // Remove all item sharekeys files
@@ -3067,7 +3067,7 @@  discard block
 block discarded – undo
3067 3067
     // Remove all item sharekeys suggestions
3068 3068
     DB::delete(
3069 3069
         prefixTable('sharekeys_suggestions'),
3070
-        'user_id = %i',// AND object_id NOT IN (SELECT i.id FROM ' . prefixTable('items') . ' AS i WHERE i.perso = 1)',
3070
+        'user_id = %i', // AND object_id NOT IN (SELECT i.id FROM ' . prefixTable('items') . ' AS i WHERE i.perso = 1)',
3071 3071
         $userId
3072 3072
     );
3073 3073
     return false;
@@ -3088,7 +3088,7 @@  discard block
 block discarded – undo
3088 3088
         foreach (DateTimeZone::listIdentifiers() as $timezone) {
3089 3089
             $now->setTimezone(new DateTimeZone($timezone));
3090 3090
             $offsets[] = $offset = $now->getOffset();
3091
-            $timezones[$timezone] = '(' . format_GMT_offset($offset) . ') ' . format_timezone_name($timezone);
3091
+            $timezones[$timezone] = '('.format_GMT_offset($offset).') '.format_timezone_name($timezone);
3092 3092
         }
3093 3093
 
3094 3094
         array_multisort($offsets, $timezones);
@@ -3108,7 +3108,7 @@  discard block
 block discarded – undo
3108 3108
 {
3109 3109
     $hours = intval($offset / 3600);
3110 3110
     $minutes = abs(intval($offset % 3600 / 60));
3111
-    return 'GMT' . ($offset ? sprintf('%+03d:%02d', $hours, $minutes) : '');
3111
+    return 'GMT'.($offset ? sprintf('%+03d:%02d', $hours, $minutes) : '');
3112 3112
 }
3113 3113
 
3114 3114
 /**
@@ -3207,8 +3207,7 @@  discard block
 block discarded – undo
3207 3207
 {
3208 3208
     if (isset($array[$key]) === true
3209 3209
         && (is_int($value) === true ?
3210
-            (int) $array[$key] === $value :
3211
-            (string) $array[$key] === $value)
3210
+            (int) $array[$key] === $value : (string) $array[$key] === $value)
3212 3211
     ) {
3213 3212
         return true;
3214 3213
     }
@@ -3230,8 +3229,7 @@  discard block
 block discarded – undo
3230 3229
 {
3231 3230
     if (isset($var) === false
3232 3231
         || (is_int($value) === true ?
3233
-            (int) $var === $value :
3234
-            (string) $var === $value)
3232
+            (int) $var === $value : (string) $var === $value)
3235 3233
     ) {
3236 3234
         return true;
3237 3235
     }
@@ -3282,7 +3280,7 @@  discard block
 block discarded – undo
3282 3280
  */
3283 3281
 function isSetArrayOfValues(array $arrayOfValues): bool
3284 3282
 {
3285
-    foreach($arrayOfValues as $value) {
3283
+    foreach ($arrayOfValues as $value) {
3286 3284
         if (isset($value) === false) {
3287 3285
             return false;
3288 3286
         }
@@ -3304,7 +3302,7 @@  discard block
 block discarded – undo
3304 3302
     /*PHP8 - integer|string*/$value
3305 3303
 ) : bool
3306 3304
 {
3307
-    foreach($arrayOfVars as $variable) {
3305
+    foreach ($arrayOfVars as $variable) {
3308 3306
         if ($variable !== $value) {
3309 3307
             return false;
3310 3308
         }
@@ -3324,7 +3322,7 @@  discard block
 block discarded – undo
3324 3322
     /*PHP8 - integer|string*/$value
3325 3323
 ) : bool
3326 3324
 {
3327
-    foreach($arrayOfVars as $variable) {
3325
+    foreach ($arrayOfVars as $variable) {
3328 3326
         if ($variable === $value) {
3329 3327
             return true;
3330 3328
         }
@@ -3338,7 +3336,7 @@  discard block
 block discarded – undo
3338 3336
  * @param string|int|null $value
3339 3337
  * @return boolean
3340 3338
  */
3341
-function isValueSetNullEmpty(string|int|null $value) : bool
3339
+function isValueSetNullEmpty(string | int | null $value) : bool
3342 3340
 {
3343 3341
     if (is_null($value) === true || empty($value) === true) {
3344 3342
         return true;
@@ -3393,7 +3391,7 @@  discard block
 block discarded – undo
3393 3391
  * @param array     $filters
3394 3392
  * @return array|string
3395 3393
  */
3396
-function dataSanitizer(array $data, array $filters): array|string
3394
+function dataSanitizer(array $data, array $filters): array | string
3397 3395
 {
3398 3396
     // Load Sanitizer library
3399 3397
     $sanitizer = new Sanitizer($data, $filters);
@@ -3422,7 +3420,7 @@  discard block
 block discarded – undo
3422 3420
     // Exists ?
3423 3421
     $userCacheId = DB::queryFirstRow(
3424 3422
         'SELECT increment_id
3425
-        FROM ' . prefixTable('cache_tree') . '
3423
+        FROM ' . prefixTable('cache_tree').'
3426 3424
         WHERE user_id = %i',
3427 3425
         $user_id
3428 3426
     );
@@ -3473,7 +3471,7 @@  discard block
 block discarded – undo
3473 3471
  */
3474 3472
 function pourcentage(float $nombre, float $total, float $pourcentage): float
3475 3473
 { 
3476
-    $resultat = ($nombre/$total) * $pourcentage;
3474
+    $resultat = ($nombre / $total) * $pourcentage;
3477 3475
     return round($resultat);
3478 3476
 }
3479 3477
 
@@ -3503,7 +3501,7 @@  discard block
 block discarded – undo
3503 3501
 
3504 3502
     // Get last folder update
3505 3503
     $lastFolderChange = DB::queryFirstRow(
3506
-        'SELECT valeur FROM ' . prefixTable('misc') . '
3504
+        'SELECT valeur FROM '.prefixTable('misc').'
3507 3505
         WHERE type = %s AND intitule = %s',
3508 3506
         'timestamp',
3509 3507
         'last_folder_change'
@@ -3524,7 +3522,7 @@  discard block
 block discarded – undo
3524 3522
     // Does this user has a tree cache
3525 3523
     $userCacheTree = DB::queryFirstRow(
3526 3524
         'SELECT '.$fieldName.'
3527
-        FROM ' . prefixTable('cache_tree') . '
3525
+        FROM ' . prefixTable('cache_tree').'
3528 3526
         WHERE user_id = %i',
3529 3527
         $session->get('user-id')
3530 3528
     );
@@ -3562,7 +3560,7 @@  discard block
 block discarded – undo
3562 3560
     if (count($folderIds) === 0) {
3563 3561
         $folderIds = DB::queryFirstColumn(
3564 3562
             'SELECT id
3565
-            FROM ' . prefixTable('nested_tree') . '
3563
+            FROM ' . prefixTable('nested_tree').'
3566 3564
             WHERE personal_folder=%i',
3567 3565
             0
3568 3566
         );
@@ -3579,8 +3577,8 @@  discard block
 block discarded – undo
3579 3577
         $rows_tmp = DB::query(
3580 3578
             'SELECT c.id, c.title, c.level, c.type, c.masked, c.order, c.encrypted_data, c.role_visibility, c.is_mandatory,
3581 3579
             f.id_category AS category_id
3582
-            FROM ' . prefixTable('categories_folders') . ' AS f
3583
-            INNER JOIN ' . prefixTable('categories') . ' AS c ON (f.id_category = c.parent_id)
3580
+            FROM ' . prefixTable('categories_folders').' AS f
3581
+            INNER JOIN ' . prefixTable('categories').' AS c ON (f.id_category = c.parent_id)
3584 3582
             WHERE id_folder=%i',
3585 3583
             $folder
3586 3584
         );
@@ -3606,7 +3604,7 @@  discard block
 block discarded – undo
3606 3604
         $valTemp = '';
3607 3605
         $data = DB::queryFirstRow(
3608 3606
             'SELECT valeur
3609
-            FROM ' . prefixTable('misc') . '
3607
+            FROM ' . prefixTable('misc').'
3610 3608
             WHERE type = %s AND intitule=%i',
3611 3609
             'complex',
3612 3610
             $folder
@@ -3623,14 +3621,14 @@  discard block
 block discarded – undo
3623 3621
         $valTemp = '';
3624 3622
         $rows_tmp = DB::query(
3625 3623
             'SELECT t.title
3626
-            FROM ' . prefixTable('roles_values') . ' as v
3627
-            INNER JOIN ' . prefixTable('roles_title') . ' as t ON (v.role_id = t.id)
3624
+            FROM ' . prefixTable('roles_values').' as v
3625
+            INNER JOIN ' . prefixTable('roles_title').' as t ON (v.role_id = t.id)
3628 3626
             WHERE v.folder_id = %i
3629 3627
             GROUP BY title',
3630 3628
             $folder
3631 3629
         );
3632 3630
         foreach ($rows_tmp as $record) {
3633
-            $valTemp .= (empty($valTemp) === true ? '' : ' - ') . $record['title'];
3631
+            $valTemp .= (empty($valTemp) === true ? '' : ' - ').$record['title'];
3634 3632
         }
3635 3633
         $arr_data['visibilityRoles'] = $valTemp;
3636 3634
 
@@ -3667,8 +3665,8 @@  discard block
 block discarded – undo
3667 3665
                         CASE WHEN ur.source = \'manual\' THEN ur.role_id END
3668 3666
                         ORDER BY ur.role_id SEPARATOR ";"
3669 3667
                     ) AS fonction_id
3670
-             FROM ' . prefixTable('users') . ' AS u
3671
-             LEFT JOIN ' . prefixTable('users_roles') . ' AS ur ON ur.user_id = u.id
3668
+             FROM ' . prefixTable('users').' AS u
3669
+             LEFT JOIN ' . prefixTable('users_roles').' AS ur ON ur.user_id = u.id
3672 3670
              WHERE u.id != %i AND u.admin = 0
3673 3671
              GROUP BY u.id
3674 3672
              HAVING fonction_id IS NOT NULL AND fonction_id != ""',
@@ -3702,7 +3700,7 @@  discard block
 block discarded – undo
3702 3700
 
3703 3701
     $val = DB::queryFirstRow(
3704 3702
         'SELECT *
3705
-        FROM ' . prefixTable('users') . '
3703
+        FROM ' . prefixTable('users').'
3706 3704
         WHERE id = %i',
3707 3705
         $userId
3708 3706
     );
@@ -3718,12 +3716,12 @@  discard block
 block discarded – undo
3718 3716
 function upgradeRequired(): bool
3719 3717
 {
3720 3718
     // Get settings.php
3721
-    include_once __DIR__. '/../includes/config/settings.php';
3719
+    include_once __DIR__.'/../includes/config/settings.php';
3722 3720
 
3723 3721
     // Get timestamp in DB
3724 3722
     $val = DB::queryFirstRow(
3725 3723
         'SELECT valeur
3726
-        FROM ' . prefixTable('misc') . '
3724
+        FROM ' . prefixTable('misc').'
3727 3725
         WHERE type = %s AND intitule = %s',
3728 3726
         'admin',
3729 3727
         'upgrade_timestamp'
@@ -3776,7 +3774,7 @@  discard block
 block discarded – undo
3776 3774
     // prepapre background tasks for item keys generation        
3777 3775
     $userTP = DB::queryFirstRow(
3778 3776
         'SELECT pw, public_key, private_key
3779
-        FROM ' . prefixTable('users') . '
3777
+        FROM ' . prefixTable('users').'
3780 3778
         WHERE id = %i',
3781 3779
         TP_USER_ID
3782 3780
     );
@@ -3837,7 +3835,7 @@  discard block
 block discarded – undo
3837 3835
     }
3838 3836
 
3839 3837
     // Generate new keys
3840
-    if ($user_self_change === true && empty($recovery_public_key) === false && empty($recovery_private_key) === false){
3838
+    if ($user_self_change === true && empty($recovery_public_key) === false && empty($recovery_private_key) === false) {
3841 3839
         $userKeys = [
3842 3840
             'public_key' => $recovery_public_key,
3843 3841
             'private_key_clear' => $recovery_private_key,
@@ -3888,8 +3886,8 @@  discard block
 block discarded – undo
3888 3886
             'process_type' => 'create_user_keys',
3889 3887
             'arguments' => json_encode([
3890 3888
                 'new_user_id' => (int) $userId,
3891
-                'new_user_pwd' => cryption($passwordClear, '','encrypt')['string'],
3892
-                'new_user_code' => cryption(empty($encryptionKey) === true ? uniqidReal(20) : $encryptionKey, '','encrypt')['string'],
3889
+                'new_user_pwd' => cryption($passwordClear, '', 'encrypt')['string'],
3890
+                'new_user_code' => cryption(empty($encryptionKey) === true ? uniqidReal(20) : $encryptionKey, '', 'encrypt')['string'],
3893 3891
                 'owner_id' => (int) TP_USER_ID,
3894 3892
                 'creator_pwd' => $userTP['pw'],
3895 3893
                 'send_email' => $sendEmailToUser === true ? 1 : 0,
@@ -3961,19 +3959,19 @@  discard block
 block discarded – undo
3961 3959
 
3962 3960
     // Prepare the subtask queries
3963 3961
     $queries = [
3964
-        'step20' => 'SELECT * FROM ' . prefixTable('items'),
3962
+        'step20' => 'SELECT * FROM '.prefixTable('items'),
3965 3963
 
3966
-        'step30' => 'SELECT * FROM ' . prefixTable('log_items') . 
3964
+        'step30' => 'SELECT * FROM '.prefixTable('log_items'). 
3967 3965
                     ' WHERE raison LIKE "at_pw :%" AND encryption_type = "teampass_aes"',
3968 3966
 
3969
-        'step40' => 'SELECT * FROM ' . prefixTable('categories_items') . 
3967
+        'step40' => 'SELECT * FROM '.prefixTable('categories_items'). 
3970 3968
                     ' WHERE encryption_type = "teampass_aes"',
3971 3969
 
3972
-        'step50' => 'SELECT * FROM ' . prefixTable('suggestion'),
3970
+        'step50' => 'SELECT * FROM '.prefixTable('suggestion'),
3973 3971
 
3974
-        'step60' => 'SELECT * FROM ' . prefixTable('files') . ' AS f
3975
-                        INNER JOIN ' . prefixTable('items') . ' AS i ON i.id = f.id_item
3976
-                        WHERE f.status = "' . TP_ENCRYPTION_NAME . '"'
3972
+        'step60' => 'SELECT * FROM '.prefixTable('files').' AS f
3973
+                        INNER JOIN ' . prefixTable('items').' AS i ON i.id = f.id_item
3974
+                        WHERE f.status = "' . TP_ENCRYPTION_NAME.'"'
3977 3975
     ];
3978 3976
 
3979 3977
     // Perform loop on $queries to create sub-tasks
@@ -4163,7 +4161,7 @@  discard block
 block discarded – undo
4163 4161
  */
4164 4162
 function createTaskForItem(
4165 4163
     string $processType,
4166
-    string|array $taskName,
4164
+    string | array $taskName,
4167 4165
     int $itemId,
4168 4166
     int $userId,
4169 4167
     string $objectKey,
@@ -4187,7 +4185,7 @@  discard block
 block discarded – undo
4187 4185
                 'object_key' => $objectKey,
4188 4186
                 'author' => (int) $userId,
4189 4187
             ]),
4190
-            'item_id' => (int) $parentId !== -1 ?  $parentId : null,
4188
+            'item_id' => (int) $parentId !== -1 ? $parentId : null,
4191 4189
         )
4192 4190
     );
4193 4191
     $processId = DB::insertId();
@@ -4197,7 +4195,7 @@  discard block
 block discarded – undo
4197 4195
     if (is_array($taskName) === false) {
4198 4196
         $taskName = [$taskName];
4199 4197
     }
4200
-    foreach($taskName as $task) {
4198
+    foreach ($taskName as $task) {
4201 4199
         if (WIP === true) error_log('createTaskForItem - task: '.$task);
4202 4200
         switch ($task) {
4203 4201
             case 'item_password':
@@ -4294,7 +4292,7 @@  discard block
 block discarded – undo
4294 4292
  * @param integer $user_id
4295 4293
  * @return void
4296 4294
  */
4297
-function purgeUnnecessaryKeys(bool $allUsers = true, int $user_id=0)
4295
+function purgeUnnecessaryKeys(bool $allUsers = true, int $user_id = 0)
4298 4296
 {
4299 4297
     if ($allUsers === true) {
4300 4298
         // Load class DB
@@ -4304,7 +4302,7 @@  discard block
 block discarded – undo
4304 4302
 
4305 4303
         $users = DB::query(
4306 4304
             'SELECT id
4307
-            FROM ' . prefixTable('users') . '
4305
+            FROM ' . prefixTable('users').'
4308 4306
             WHERE id NOT IN ('.OTV_USER_ID.', '.TP_USER_ID.', '.SSH_USER_ID.', '.API_USER_ID.')
4309 4307
             ORDER BY login ASC'
4310 4308
         );
@@ -4322,7 +4320,7 @@  discard block
 block discarded – undo
4322 4320
  * @param integer $user_id
4323 4321
  * @return void
4324 4322
  */
4325
-function purgeUnnecessaryKeysForUser(int $user_id=0)
4323
+function purgeUnnecessaryKeysForUser(int $user_id = 0)
4326 4324
 {
4327 4325
     if ($user_id === 0) {
4328 4326
         return;
@@ -4333,8 +4331,8 @@  discard block
 block discarded – undo
4333 4331
 
4334 4332
     $personalItems = DB::queryFirstColumn(
4335 4333
         'SELECT id
4336
-        FROM ' . prefixTable('items') . ' AS i
4337
-        INNER JOIN ' . prefixTable('log_items') . ' AS li ON li.id_item = i.id
4334
+        FROM ' . prefixTable('items').' AS i
4335
+        INNER JOIN ' . prefixTable('log_items').' AS li ON li.id_item = i.id
4338 4336
         WHERE i.perso = 1 AND li.action = "at_creation" AND li.id_user IN (%i, '.TP_USER_ID.')',
4339 4337
         $user_id
4340 4338
     );
@@ -4344,28 +4342,28 @@  discard block
 block discarded – undo
4344 4342
             prefixTable('sharekeys_items'),
4345 4343
             'object_id IN %li AND user_id NOT IN %ls',
4346 4344
             $personalItems,
4347
-            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4345
+            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4348 4346
         );
4349 4347
         // Files keys
4350 4348
         DB::delete(
4351 4349
             prefixTable('sharekeys_files'),
4352 4350
             'object_id IN %li AND user_id NOT IN %ls',
4353 4351
             $personalItems,
4354
-            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4352
+            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4355 4353
         );
4356 4354
         // Fields keys
4357 4355
         DB::delete(
4358 4356
             prefixTable('sharekeys_fields'),
4359 4357
             'object_id IN %li AND user_id NOT IN %ls',
4360 4358
             $personalItems,
4361
-            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4359
+            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4362 4360
         );
4363 4361
         // Logs keys
4364 4362
         DB::delete(
4365 4363
             prefixTable('sharekeys_logs'),
4366 4364
             'object_id IN %li AND user_id NOT IN %ls',
4367 4365
             $personalItems,
4368
-            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4366
+            [$user_id, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4369 4367
         );
4370 4368
     }
4371 4369
 }
@@ -4383,7 +4381,7 @@  discard block
 block discarded – undo
4383 4381
     // Check if user exists
4384 4382
     $userInfo = DB::queryFirstRow(
4385 4383
         'SELECT login
4386
-        FROM ' . prefixTable('users') . '
4384
+        FROM ' . prefixTable('users').'
4387 4385
         WHERE id = %i',
4388 4386
         $userId
4389 4387
     );
@@ -4392,7 +4390,7 @@  discard block
 block discarded – undo
4392 4390
         $now = (int) time();
4393 4391
         // Prepare file content
4394 4392
         $export_value = file_get_contents(__DIR__."/../includes/core/teampass_ascii.txt")."\n".
4395
-            "Generation date: ".date($SETTINGS['date_format'] . ' ' . $SETTINGS['time_format'], $now)."\n\n".
4393
+            "Generation date: ".date($SETTINGS['date_format'].' '.$SETTINGS['time_format'], $now)."\n\n".
4396 4394
             "RECOVERY KEYS - Not to be shared - To be store safely\n\n".
4397 4395
             "Public Key:\n".$session->get('user-public_key')."\n\n".
4398 4396
             "Private Key:\n".$session->get('user-private_key')."\n\n";
@@ -4415,7 +4413,7 @@  discard block
 block discarded – undo
4415 4413
         return prepareExchangedData(
4416 4414
             array(
4417 4415
                 'error' => false,
4418
-                'datetime' => date($SETTINGS['date_format'] . ' ' . $SETTINGS['time_format'], $now),
4416
+                'datetime' => date($SETTINGS['date_format'].' '.$SETTINGS['time_format'], $now),
4419 4417
                 'timestamp' => $now,
4420 4418
                 'content' => base64_encode($export_value),
4421 4419
                 'login' => $userInfo['login'],
@@ -4441,8 +4439,8 @@  discard block
 block discarded – undo
4441 4439
  */
4442 4440
 function loadClasses(string $className = ''): void
4443 4441
 {
4444
-    require_once __DIR__. '/../includes/config/include.php';
4445
-    require_once __DIR__. '/../includes/config/settings.php';
4442
+    require_once __DIR__.'/../includes/config/include.php';
4443
+    require_once __DIR__.'/../includes/config/settings.php';
4446 4444
     require_once __DIR__.'/../vendor/autoload.php';
4447 4445
 
4448 4446
     if (defined('DB_PASSWD_CLEAR') === false) {
@@ -4655,7 +4653,7 @@  discard block
 block discarded – undo
4655 4653
 
4656 4654
     // Get current user hash
4657 4655
     $userHash = DB::queryFirstRow(
4658
-        "SELECT pw FROM " . prefixtable('users') . " WHERE id = %d;",
4656
+        "SELECT pw FROM ".prefixtable('users')." WHERE id = %d;",
4659 4657
         $session->get('user-id')
4660 4658
     )['pw'];
4661 4659
 
@@ -4703,11 +4701,11 @@  discard block
 block discarded – undo
4703 4701
         }
4704 4702
         
4705 4703
         // Alternative: serialize ou json selon le contexte
4706
-        return get_class($value) . (method_exists($value, 'getId') ? '#' . $value->getId() : '');
4704
+        return get_class($value).(method_exists($value, 'getId') ? '#'.$value->getId() : '');
4707 4705
     }
4708 4706
     
4709 4707
     if (is_resource($value)) {
4710
-        return 'Resource#' . get_resource_id($value) . ' of type ' . get_resource_type($value);
4708
+        return 'Resource#'.get_resource_id($value).' of type '.get_resource_type($value);
4711 4709
     }
4712 4710
     
4713 4711
     // Cas par défaut
@@ -4727,7 +4725,7 @@  discard block
 block discarded – undo
4727 4725
     // Refuse access if user does not exist and/or is admin
4728 4726
     $user = DB::queryFirstRow(
4729 4727
         'SELECT admin
4730
-        FROM ' . prefixTable('users') . '
4728
+        FROM ' . prefixTable('users').'
4731 4729
         WHERE id = %i',
4732 4730
         $userId
4733 4731
     );
@@ -4738,8 +4736,8 @@  discard block
 block discarded – undo
4738 4736
     // Get file info
4739 4737
     $file = DB::queryFirstRow(
4740 4738
         'SELECT f.id_item, i.id_tree
4741
-        FROM ' . prefixTable('files') . ' as f
4742
-        INNER JOIN ' . prefixTable('items') . ' AS i ON i.id = f.id_item
4739
+        FROM ' . prefixTable('files').' as f
4740
+        INNER JOIN ' . prefixTable('items').' AS i ON i.id = f.id_item
4743 4741
         WHERE f.id = %i',
4744 4742
         $fileId
4745 4743
     );
@@ -4748,7 +4746,7 @@  discard block
 block discarded – undo
4748 4746
     }
4749 4747
 
4750 4748
     // Check if user has access to the item
4751
-    include_once __DIR__. '/items.queries.php';
4749
+    include_once __DIR__.'/items.queries.php';
4752 4750
     $itemAccess = getCurrentAccessRights(
4753 4751
         (int) filter_var($userId, FILTER_SANITIZE_NUMBER_INT),
4754 4752
         (int) filter_var($file['id_item'], FILTER_SANITIZE_NUMBER_INT),
@@ -4781,7 +4779,7 @@  discard block
 block discarded – undo
4781 4779
     // Refuse access if user does not exist and/or is not admin
4782 4780
     $user = DB::queryFirstRow(
4783 4781
         'SELECT admin
4784
-        FROM ' . prefixTable('users') . '
4782
+        FROM ' . prefixTable('users').'
4785 4783
         WHERE id = %i',
4786 4784
         $userId
4787 4785
     );
@@ -4792,7 +4790,7 @@  discard block
 block discarded – undo
4792 4790
     // Ensure that user has performed the backup
4793 4791
     DB::queryFirstRow(
4794 4792
         'SELECT f.id
4795
-        FROM ' . prefixTable('log_system') . ' as f
4793
+        FROM ' . prefixTable('log_system').' as f
4796 4794
         WHERE f.type = %s AND f.label = %s AND f.qui = %i AND f.field_1 = %s',
4797 4795
         'admin_action',
4798 4796
         'dataBase backup',
@@ -4819,7 +4817,7 @@  discard block
 block discarded – undo
4819 4817
     // Refuse access if user does not exist and/or is admin
4820 4818
     $user = DB::queryFirstRow(
4821 4819
         'SELECT admin
4822
-        FROM ' . prefixTable('users') . '
4820
+        FROM ' . prefixTable('users').'
4823 4821
         WHERE id = %i',
4824 4822
         $userId
4825 4823
     );
@@ -4830,7 +4828,7 @@  discard block
 block discarded – undo
4830 4828
     // Get item info
4831 4829
     $item = DB::queryFirstRow(
4832 4830
         'SELECT i.perso, i.id_tree
4833
-        FROM ' . prefixTable('items') . ' as i
4831
+        FROM ' . prefixTable('items').' as i
4834 4832
         WHERE i.id = %i',
4835 4833
         $itemId
4836 4834
     );
@@ -4841,7 +4839,7 @@  discard block
 block discarded – undo
4841 4839
     // Get item owner
4842 4840
     $itemOwner = DB::queryFirstRow(
4843 4841
         'SELECT li.id_user
4844
-        FROM ' . prefixTable('log_items') . ' as li
4842
+        FROM ' . prefixTable('log_items').' as li
4845 4843
         WHERE li.id_item = %i AND li.action = %s',
4846 4844
         $itemId,
4847 4845
         'at_creation'
@@ -4855,25 +4853,25 @@  discard block
 block discarded – undo
4855 4853
         prefixTable('sharekeys_items'),
4856 4854
         'object_id = %i AND user_id NOT IN %ls',
4857 4855
         $itemId,
4858
-        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4856
+        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4859 4857
     );
4860 4858
     DB::delete(
4861 4859
         prefixTable('sharekeys_files'),
4862 4860
         'object_id IN (SELECT id FROM '.prefixTable('files').' WHERE id_item = %i) AND user_id NOT IN %ls',
4863 4861
         $itemId,
4864
-        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4862
+        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4865 4863
     );
4866 4864
     DB::delete(
4867 4865
         prefixTable('sharekeys_fields'),
4868 4866
         'object_id IN (SELECT id FROM '.prefixTable('fields').' WHERE id_item = %i) AND user_id NOT IN %ls',
4869 4867
         $itemId,
4870
-        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4868
+        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4871 4869
     );
4872 4870
     DB::delete(
4873 4871
         prefixTable('sharekeys_logs'),
4874 4872
         'object_id IN (SELECT id FROM '.prefixTable('log_items').' WHERE id_item = %i) AND user_id NOT IN %ls',
4875 4873
         $itemId,
4876
-        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID,SSH_USER_ID]
4874
+        [$userId, TP_USER_ID, API_USER_ID, OTV_USER_ID, SSH_USER_ID]
4877 4875
     );
4878 4876
 
4879 4877
     return true;
@@ -4945,7 +4943,7 @@  discard block
 block discarded – undo
4945 4943
     
4946 4944
     // 2. Check if user actually has personal items to migrate
4947 4945
     $personalFolderId = DB::queryFirstField(
4948
-        "SELECT id FROM ".prefixTable('nested_tree') ."
4946
+        "SELECT id FROM ".prefixTable('nested_tree')."
4949 4947
          WHERE personal_folder = 1 
4950 4948
          AND title = %s",
4951 4949
         $userId
@@ -5025,8 +5023,8 @@  discard block
 block discarded – undo
5025 5023
             'process_type' => 'migrate_user_personal_items',
5026 5024
             'arguments' => json_encode([
5027 5025
                 'user_id' => (int) $userId,
5028
-                'user_pwd' => cryption($passwordClear, '','encrypt')['string'],
5029
-                'user_private_key' => cryption($privateKeyDecrypted, '','encrypt')['string'],
5026
+                'user_pwd' => cryption($passwordClear, '', 'encrypt')['string'],
5027
+                'user_private_key' => cryption($privateKeyDecrypted, '', 'encrypt')['string'],
5030 5028
                 'personal_folders_ids' => $personalFolderIds,
5031 5029
             ]),
5032 5030
             'is_in_progress' => 0,
@@ -5055,19 +5053,19 @@  discard block
 block discarded – undo
5055 5053
 {
5056 5054
     // Prepare the subtask queries
5057 5055
     $queries = [
5058
-        'user-personal-items-migration-step10' => 'SELECT * FROM ' . prefixTable('items'),
5056
+        'user-personal-items-migration-step10' => 'SELECT * FROM '.prefixTable('items'),
5059 5057
 
5060
-        'user-personal-items-migration-step20' => 'SELECT * FROM ' . prefixTable('log_items') . 
5058
+        'user-personal-items-migration-step20' => 'SELECT * FROM '.prefixTable('log_items'). 
5061 5059
                     ' WHERE raison LIKE "at_pw :%" AND encryption_type = "teampass_aes"',
5062 5060
 
5063
-        'user-personal-items-migration-step30' => 'SELECT * FROM ' . prefixTable('categories_items') . 
5061
+        'user-personal-items-migration-step30' => 'SELECT * FROM '.prefixTable('categories_items'). 
5064 5062
                     ' WHERE encryption_type = "teampass_aes"',
5065 5063
 
5066
-        'user-personal-items-migration-step40' => 'SELECT * FROM ' . prefixTable('suggestion'),
5064
+        'user-personal-items-migration-step40' => 'SELECT * FROM '.prefixTable('suggestion'),
5067 5065
 
5068
-        'user-personal-items-migration-step50' => 'SELECT * FROM ' . prefixTable('files') . ' AS f
5069
-                        INNER JOIN ' . prefixTable('items') . ' AS i ON i.id = f.id_item
5070
-                        WHERE f.status = "' . TP_ENCRYPTION_NAME . '"'
5066
+        'user-personal-items-migration-step50' => 'SELECT * FROM '.prefixTable('files').' AS f
5067
+                        INNER JOIN ' . prefixTable('items').' AS i ON i.id = f.id_item
5068
+                        WHERE f.status = "' . TP_ENCRYPTION_NAME.'"'
5071 5069
     ];
5072 5070
 
5073 5071
     // Perform loop on $queries to create sub-tasks
@@ -5100,7 +5098,7 @@  discard block
 block discarded – undo
5100 5098
 {
5101 5099
     // 1. Insert or update the item with current timestamp
5102 5100
     DB::query(
5103
-        'INSERT INTO ' . prefixTable('users_latest_items') . ' (user_id, item_id, accessed_at)
5101
+        'INSERT INTO '.prefixTable('users_latest_items').' (user_id, item_id, accessed_at)
5104 5102
         VALUES (%i, %i, NOW())
5105 5103
         ON DUPLICATE KEY UPDATE accessed_at = NOW()',
5106 5104
         $userId,
@@ -5109,12 +5107,12 @@  discard block
 block discarded – undo
5109 5107
     
5110 5108
     // 2. Keep only the 20 most recent items (delete older ones)
5111 5109
     DB::query(
5112
-        'DELETE FROM ' . prefixTable('users_latest_items') . '
5110
+        'DELETE FROM '.prefixTable('users_latest_items').'
5113 5111
         WHERE user_id = %i
5114 5112
         AND increment_id NOT IN (
5115 5113
             SELECT increment_id FROM (
5116 5114
                 SELECT increment_id 
5117
-                FROM ' . prefixTable('users_latest_items') . '
5115
+                FROM ' . prefixTable('users_latest_items').'
5118 5116
                 WHERE user_id = %i
5119 5117
                 ORDER BY accessed_at DESC
5120 5118
                 LIMIT 20
@@ -5157,14 +5155,14 @@  discard block
 block discarded – undo
5157 5155
          GROUP_CONCAT(DISTINCT CASE WHEN ur.source = "ad" THEN ur.role_id END ORDER BY ur.role_id SEPARATOR ";") AS roles_from_ad_groups,
5158 5156
          GROUP_CONCAT(DISTINCT uf.item_id ORDER BY uf.created_at SEPARATOR ";") AS favourites,
5159 5157
          GROUP_CONCAT(DISTINCT ul.item_id ORDER BY ul.accessed_at DESC SEPARATOR ";") AS latest_items
5160
-        FROM ' . prefixTable('users') . ' AS u
5161
-        LEFT JOIN ' . prefixTable('api') . ' AS a ON (u.id = a.user_id)
5162
-        LEFT JOIN ' . prefixTable('users_groups') . ' AS ug ON (u.id = ug.user_id)
5163
-        LEFT JOIN ' . prefixTable('users_groups_forbidden') . ' AS ugf ON (u.id = ugf.user_id)
5164
-        LEFT JOIN ' . prefixTable('users_roles') . ' AS ur ON (u.id = ur.user_id)
5165
-        LEFT JOIN ' . prefixTable('users_favorites') . ' AS uf ON (u.id = uf.user_id)
5166
-        LEFT JOIN ' . prefixTable('users_latest_items') . ' AS ul ON (u.id = ul.user_id)
5167
-        WHERE ' . $whereClause . '
5158
+        FROM ' . prefixTable('users').' AS u
5159
+        LEFT JOIN ' . prefixTable('api').' AS a ON (u.id = a.user_id)
5160
+        LEFT JOIN ' . prefixTable('users_groups').' AS ug ON (u.id = ug.user_id)
5161
+        LEFT JOIN ' . prefixTable('users_groups_forbidden').' AS ugf ON (u.id = ugf.user_id)
5162
+        LEFT JOIN ' . prefixTable('users_roles').' AS ur ON (u.id = ur.user_id)
5163
+        LEFT JOIN ' . prefixTable('users_favorites').' AS uf ON (u.id = uf.user_id)
5164
+        LEFT JOIN ' . prefixTable('users_latest_items').' AS ul ON (u.id = ul.user_id)
5165
+        WHERE ' . $whereClause.'
5168 5166
         GROUP BY u.id',
5169 5167
         $whereParam
5170 5168
     );
@@ -5194,7 +5192,7 @@  discard block
 block discarded – undo
5194 5192
 function addUserGroup(int $userId, int $groupId): void
5195 5193
 {
5196 5194
     DB::query(
5197
-        'INSERT IGNORE INTO ' . prefixTable('users_groups') . ' (user_id, group_id)
5195
+        'INSERT IGNORE INTO '.prefixTable('users_groups').' (user_id, group_id)
5198 5196
         VALUES (%i, %i)',
5199 5197
         $userId,
5200 5198
         $groupId
@@ -5207,7 +5205,7 @@  discard block
 block discarded – undo
5207 5205
 function removeUserGroup(int $userId, int $groupId): void
5208 5206
 {
5209 5207
     DB::query(
5210
-        'DELETE FROM ' . prefixTable('users_groups') . '
5208
+        'DELETE FROM '.prefixTable('users_groups').'
5211 5209
         WHERE user_id = %i AND group_id = %i',
5212 5210
         $userId,
5213 5211
         $groupId
@@ -5224,7 +5222,7 @@  discard block
 block discarded – undo
5224 5222
 {
5225 5223
     // Delete all existing groups
5226 5224
     DB::query(
5227
-        'DELETE FROM ' . prefixTable('users_groups') . ' WHERE user_id = %i',
5225
+        'DELETE FROM '.prefixTable('users_groups').' WHERE user_id = %i',
5228 5226
         $userId
5229 5227
     );
5230 5228
     
@@ -5244,7 +5242,7 @@  discard block
 block discarded – undo
5244 5242
 function getUserGroups(int $userId): array
5245 5243
 {
5246 5244
     $result = DB::query(
5247
-        'SELECT group_id FROM ' . prefixTable('users_groups') . ' 
5245
+        'SELECT group_id FROM '.prefixTable('users_groups').' 
5248 5246
         WHERE user_id = %i ORDER BY group_id',
5249 5247
         $userId
5250 5248
     );
@@ -5261,7 +5259,7 @@  discard block
 block discarded – undo
5261 5259
 function addUserForbiddenGroup(int $userId, int $groupId): void
5262 5260
 {
5263 5261
     DB::query(
5264
-        'INSERT IGNORE INTO ' . prefixTable('users_groups_forbidden') . ' (user_id, group_id)
5262
+        'INSERT IGNORE INTO '.prefixTable('users_groups_forbidden').' (user_id, group_id)
5265 5263
         VALUES (%i, %i)',
5266 5264
         $userId,
5267 5265
         $groupId
@@ -5274,7 +5272,7 @@  discard block
 block discarded – undo
5274 5272
 function removeUserForbiddenGroup(int $userId, int $groupId): void
5275 5273
 {
5276 5274
     DB::query(
5277
-        'DELETE FROM ' . prefixTable('users_groups_forbidden') . '
5275
+        'DELETE FROM '.prefixTable('users_groups_forbidden').'
5278 5276
         WHERE user_id = %i AND group_id = %i',
5279 5277
         $userId,
5280 5278
         $groupId
@@ -5287,7 +5285,7 @@  discard block
 block discarded – undo
5287 5285
 function setUserForbiddenGroups(int $userId, array $groupIds): void
5288 5286
 {
5289 5287
     DB::query(
5290
-        'DELETE FROM ' . prefixTable('users_groups_forbidden') . ' WHERE user_id = %i',
5288
+        'DELETE FROM '.prefixTable('users_groups_forbidden').' WHERE user_id = %i',
5291 5289
         $userId
5292 5290
     );
5293 5291
     
@@ -5304,7 +5302,7 @@  discard block
 block discarded – undo
5304 5302
 function getUserForbiddenGroups(int $userId): array
5305 5303
 {
5306 5304
     $result = DB::query(
5307
-        'SELECT group_id FROM ' . prefixTable('users_groups_forbidden') . ' 
5305
+        'SELECT group_id FROM '.prefixTable('users_groups_forbidden').' 
5308 5306
         WHERE user_id = %i ORDER BY group_id',
5309 5307
         $userId
5310 5308
     );
@@ -5323,7 +5321,7 @@  discard block
 block discarded – undo
5323 5321
 function addUserRole(int $userId, int $roleId, string $source = 'manual'): void
5324 5322
 {
5325 5323
     DB::query(
5326
-        'INSERT IGNORE INTO ' . prefixTable('users_roles') . ' (user_id, role_id, source)
5324
+        'INSERT IGNORE INTO '.prefixTable('users_roles').' (user_id, role_id, source)
5327 5325
         VALUES (%i, %i, %s)',
5328 5326
         $userId,
5329 5327
         $roleId,
@@ -5337,7 +5335,7 @@  discard block
 block discarded – undo
5337 5335
 function removeUserRole(int $userId, int $roleId, string $source = 'manual'): void
5338 5336
 {
5339 5337
     DB::query(
5340
-        'DELETE FROM ' . prefixTable('users_roles') . '
5338
+        'DELETE FROM '.prefixTable('users_roles').'
5341 5339
         WHERE user_id = %i AND role_id = %i AND source = %s',
5342 5340
         $userId,
5343 5341
         $roleId,
@@ -5351,7 +5349,7 @@  discard block
 block discarded – undo
5351 5349
 function removeUserRolesBySource(int $userId, string $source): void
5352 5350
 {
5353 5351
     DB::query(
5354
-        'DELETE FROM ' . prefixTable('users_roles') . '
5352
+        'DELETE FROM '.prefixTable('users_roles').'
5355 5353
         WHERE user_id = %i AND source = %s',
5356 5354
         $userId,
5357 5355
         $source
@@ -5385,14 +5383,14 @@  discard block
 block discarded – undo
5385 5383
 {
5386 5384
     if ($source !== null) {
5387 5385
         $result = DB::query(
5388
-            'SELECT role_id FROM ' . prefixTable('users_roles') . ' 
5386
+            'SELECT role_id FROM '.prefixTable('users_roles').' 
5389 5387
             WHERE user_id = %i AND source = %s ORDER BY role_id',
5390 5388
             $userId,
5391 5389
             $source
5392 5390
         );
5393 5391
     } else {
5394 5392
         $result = DB::query(
5395
-            'SELECT role_id FROM ' . prefixTable('users_roles') . ' 
5393
+            'SELECT role_id FROM '.prefixTable('users_roles').' 
5396 5394
             WHERE user_id = %i ORDER BY role_id',
5397 5395
             $userId
5398 5396
         );
@@ -5410,7 +5408,7 @@  discard block
 block discarded – undo
5410 5408
 function addUserFavorite(int $userId, int $itemId): void
5411 5409
 {
5412 5410
     DB::query(
5413
-        'INSERT IGNORE INTO ' . prefixTable('users_favorites') . ' (user_id, item_id)
5411
+        'INSERT IGNORE INTO '.prefixTable('users_favorites').' (user_id, item_id)
5414 5412
         VALUES (%i, %i)',
5415 5413
         $userId,
5416 5414
         $itemId
@@ -5423,7 +5421,7 @@  discard block
 block discarded – undo
5423 5421
 function removeUserFavorite(int $userId, int $itemId): void
5424 5422
 {
5425 5423
     DB::query(
5426
-        'DELETE FROM ' . prefixTable('users_favorites') . '
5424
+        'DELETE FROM '.prefixTable('users_favorites').'
5427 5425
         WHERE user_id = %i AND item_id = %i',
5428 5426
         $userId,
5429 5427
         $itemId
@@ -5436,7 +5434,7 @@  discard block
 block discarded – undo
5436 5434
 function toggleUserFavorite(int $userId, int $itemId): bool
5437 5435
 {
5438 5436
     $exists = DB::queryFirstRow(
5439
-        'SELECT increment_id FROM ' . prefixTable('users_favorites') . '
5437
+        'SELECT increment_id FROM '.prefixTable('users_favorites').'
5440 5438
         WHERE user_id = %i AND item_id = %i',
5441 5439
         $userId,
5442 5440
         $itemId
@@ -5457,7 +5455,7 @@  discard block
 block discarded – undo
5457 5455
 function setUserFavorites(int $userId, array $itemIds): void
5458 5456
 {
5459 5457
     DB::query(
5460
-        'DELETE FROM ' . prefixTable('users_favorites') . ' WHERE user_id = %i',
5458
+        'DELETE FROM '.prefixTable('users_favorites').' WHERE user_id = %i',
5461 5459
         $userId
5462 5460
     );
5463 5461
     
@@ -5474,7 +5472,7 @@  discard block
 block discarded – undo
5474 5472
 function getUserFavorites(int $userId): array
5475 5473
 {
5476 5474
     $result = DB::query(
5477
-        'SELECT item_id FROM ' . prefixTable('users_favorites') . ' 
5475
+        'SELECT item_id FROM '.prefixTable('users_favorites').' 
5478 5476
         WHERE user_id = %i ORDER BY created_at DESC',
5479 5477
         $userId
5480 5478
     );
@@ -5487,7 +5485,7 @@  discard block
 block discarded – undo
5487 5485
 function isUserFavorite(int $userId, int $itemId): bool
5488 5486
 {
5489 5487
     $result = DB::queryFirstRow(
5490
-        'SELECT increment_id FROM ' . prefixTable('users_favorites') . '
5488
+        'SELECT increment_id FROM '.prefixTable('users_favorites').'
5491 5489
         WHERE user_id = %i AND item_id = %i',
5492 5490
         $userId,
5493 5491
         $itemId
Please login to merge, or discard this patch.