Passed
Pull Request — master (#4432)
by Nils
05:45
created
sources/core.php 1 patch
Spacing   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -57,16 +57,16 @@  discard block
 block discarded – undo
57 57
 {
58 58
     // Load AntiXSS
59 59
     $antiXss = new AntiXSS();
60
-    if (! headers_sent()) {    //If headers not sent yet... then do php redirect
61
-        header('Location: ' . $antiXss->xss_clean($url));
60
+    if (!headers_sent()) {    //If headers not sent yet... then do php redirect
61
+        header('Location: '.$antiXss->xss_clean($url));
62 62
     }
63 63
 
64 64
     //If headers are sent... do java redirect... if java disabled, do html redirect.
65 65
     echo '<script type="text/javascript">';
66
-    echo 'window.location.href="' . $antiXss->xss_clean($url) . '";';
66
+    echo 'window.location.href="'.$antiXss->xss_clean($url).'";';
67 67
     echo '</script>';
68 68
     echo '<noscript>';
69
-    echo '<meta http-equiv="refresh" content="0;url=' . $antiXss->xss_clean($url) . '" />';
69
+    echo '<meta http-equiv="refresh" content="0;url='.$antiXss->xss_clean($url).'" />';
70 70
     echo '</noscript>';
71 71
 }
72 72
 
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
     && isset($SETTINGS['enable_sts']) === true
93 93
     && (int) $SETTINGS['enable_sts'] === 1
94 94
 ) {
95
-    teampassRedirect('https://' . $server['http_host'] . $server['request_uri']);
95
+    teampassRedirect('https://'.$server['http_host'].$server['request_uri']);
96 96
 }
97 97
 
98 98
 // Load pwComplexity
@@ -113,11 +113,11 @@  discard block
 block discarded – undo
113 113
 // LOAD CPASSMAN SETTINGS
114 114
 if (
115 115
     isset($SETTINGS['cpassman_dir']) === true
116
-    && is_dir($SETTINGS['cpassman_dir'] . '/install') === true
116
+    && is_dir($SETTINGS['cpassman_dir'].'/install') === true
117 117
 ) {
118 118
     // Should we delete folder INSTALL?
119 119
     $row = DB::queryFirstRow(
120
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s AND intitule=%s',
120
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s AND intitule=%s',
121 121
         'install',
122 122
         'clear_install_folder'
123 123
     );
@@ -135,11 +135,11 @@  discard block
 block discarded – undo
135 135
             if ($directories !== false) {
136 136
                 $files = array_diff($directories, ['.', '..']);
137 137
                 foreach ($files as $file) {
138
-                    if (is_dir($dir . '/' . $file)) {
139
-                        delTree($dir . '/' . $file);
138
+                    if (is_dir($dir.'/'.$file)) {
139
+                        delTree($dir.'/'.$file);
140 140
                     } else {
141 141
                         try {
142
-                            unlink($dir . '/' . $file);
142
+                            unlink($dir.'/'.$file);
143 143
                         } catch (Exception $e) {
144 144
                             // do nothing... php will ignore and continue
145 145
                         }
@@ -153,13 +153,13 @@  discard block
 block discarded – undo
153 153
             return false;
154 154
         }
155 155
 
156
-        if (is_dir($SETTINGS['cpassman_dir'] . '/install')) {
156
+        if (is_dir($SETTINGS['cpassman_dir'].'/install')) {
157 157
             // Set the permissions on the install directory and delete
158 158
             // is server Windows or Linux?
159 159
             if (strtoupper(substr(PHP_OS, 0, 3)) !== 'WIN') {
160
-                recursiveChmod($SETTINGS['cpassman_dir'] . '/install', 0755, 0440);
160
+                recursiveChmod($SETTINGS['cpassman_dir'].'/install', 0755, 0440);
161 161
             }
162
-            delTree($SETTINGS['cpassman_dir'] . '/install');
162
+            delTree($SETTINGS['cpassman_dir'].'/install');
163 163
         }
164 164
 
165 165
         // Delete temporary install table
@@ -177,10 +177,10 @@  discard block
 block discarded – undo
177 177
 // Load Languages stuff
178 178
 if (isset($languagesList) === false) {
179 179
     $languagesList = [];
180
-    $rows = DB::query('SELECT * FROM ' . prefixTable('languages') . ' GROUP BY name, label, code, flag, id ORDER BY name ASC');
180
+    $rows = DB::query('SELECT * FROM '.prefixTable('languages').' GROUP BY name, label, code, flag, id ORDER BY name ASC');
181 181
     foreach ($rows as $record) {
182 182
         array_push($languagesList, $record['name']);
183
-        if ($session->get('user-language') === $record['name'] ) {
183
+        if ($session->get('user-language') === $record['name']) {
184 184
             $session->set('user-language_flag', $record['flag']);
185 185
             $session->set('user-language_code', $record['code']);
186 186
             //$session->set('user-language_label', $record['label']);
@@ -237,7 +237,7 @@  discard block
 block discarded – undo
237 237
 // CHECK IF SESSION EXISTS AND IF SESSION IS VALID
238 238
 if (empty($session->get('user-session_duration')) === false) {
239 239
     $dataSession = DB::queryFirstRow(
240
-        'SELECT key_tempo FROM ' . prefixTable('users') . ' WHERE id=%i',
240
+        'SELECT key_tempo FROM '.prefixTable('users').' WHERE id=%i',
241 241
         $session->get('user-id')
242 242
     );
243 243
 } else {
@@ -292,7 +292,7 @@  discard block
 block discarded – undo
292 292
     && ($session->has('user-admin') && $session->get('user-admin') && null !== $session->get('user-admin') && $session->get('user-admin') === 1)
293 293
 ) {
294 294
     $row = DB::queryFirstRow(
295
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s_type AND intitule=%s_intitule',
295
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s_type AND intitule=%s_intitule',
296 296
         [
297 297
             'type' => 'admin',
298 298
             'intitule' => 'teampass_version',
@@ -333,7 +333,7 @@  discard block
 block discarded – undo
333 333
 
334 334
         syslog(
335 335
             LOG_WARNING,
336
-            'Unlog user: ' . date('Y/m/d H:i:s') . " {$server['remote_addr']} ({$server['http_user_agent']})"
336
+            'Unlog user: '.date('Y/m/d H:i:s')." {$server['remote_addr']} ({$server['http_user_agent']})"
337 337
         );
338 338
         // erase session table
339 339
         $session->invalidate();
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
 if ($session->has('user-timezone') && null !== $session->get('user-id') && empty($session->get('user-id')) === false) {
390 390
     // query on user
391 391
     $data = DB::queryfirstrow(
392
-        'SELECT login, admin, gestionnaire, can_manage_all_users, groupes_visibles, groupes_interdits, fonction_id, last_connexion, roles_from_ad_groups, auth_type, last_pw_change FROM ' . prefixTable('users') . ' WHERE id=%i',
392
+        'SELECT login, admin, gestionnaire, can_manage_all_users, groupes_visibles, groupes_interdits, fonction_id, last_connexion, roles_from_ad_groups, auth_type, last_pw_change FROM '.prefixTable('users').' WHERE id=%i',
393 393
         $session->get('user-id')
394 394
     );
395 395
     //Check if user has been deleted or unlogged
@@ -436,7 +436,7 @@  discard block
 block discarded – undo
436 436
             $data['groupes_visibles'],
437 437
             $data['groupes_interdits'],
438 438
             $data['admin'],
439
-            is_null($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : (empty($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : $data['fonction_id'] . ';' . $data['roles_from_ad_groups']),
439
+            is_null($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : (empty($data['roles_from_ad_groups']) === true ? $data['fonction_id'] : $data['fonction_id'].';'.$data['roles_from_ad_groups']),
440 440
             $SETTINGS
441 441
         );
442 442
         if ($session->has('user-can_create_root_folder') && (int) $session->get('user-can_create_root_folder') && null !== $session->get('user-can_create_root_folder') && (int) $session->get('user-can_create_root_folder') === 1) {
@@ -471,7 +471,7 @@  discard block
 block discarded – undo
471 471
     $session->set('system-item_fields', []);
472 472
     $rows = DB::query(
473 473
         'SELECT *
474
-            FROM ' . prefixTable('categories') . '
474
+            FROM ' . prefixTable('categories').'
475 475
             WHERE level=%i',
476 476
         '0'
477 477
     );
@@ -480,7 +480,7 @@  discard block
 block discarded – undo
480 480
         // get each field
481 481
         $rows2 = DB::query(
482 482
             'SELECT *
483
-            FROM ' . prefixTable('categories') . '
483
+            FROM ' . prefixTable('categories').'
484 484
             WHERE parent_id=%i
485 485
             ORDER BY `order` ASC',
486 486
             $record['id']
@@ -567,5 +567,5 @@  discard block
 block discarded – undo
567 567
 }
568 568
 
569 569
 /* CHECK NUMBER OF USER ONLINE */
570
-DB::query('SELECT * FROM ' . prefixTable('users') . ' WHERE timestamp>=%i', time() - 600);
570
+DB::query('SELECT * FROM '.prefixTable('users').' WHERE timestamp>=%i', time() - 600);
571 571
 $session->set('system-nb_users_online', DB::count());
Please login to merge, or discard this patch.