Passed
Push — master ( 883394...78dd46 )
by Nils
06:23
created
sources/core.php 1 patch
Spacing   +24 added lines, -24 removed lines patch added patch discarded remove patch
@@ -53,16 +53,16 @@  discard block
 block discarded – undo
53 53
 {
54 54
     // Load AntiXSS
55 55
     $antiXss = new AntiXSS();
56
-    if (! headers_sent()) {    //If headers not sent yet... then do php redirect
57
-        header('Location: ' . $antiXss->xss_clean($url));
56
+    if (!headers_sent()) {    //If headers not sent yet... then do php redirect
57
+        header('Location: '.$antiXss->xss_clean($url));
58 58
     }
59 59
 
60 60
     //If headers are sent... do java redirect... if java disabled, do html redirect.
61 61
     echo '<script type="text/javascript">';
62
-    echo 'window.location.href="' . $antiXss->xss_clean($url) . '";';
62
+    echo 'window.location.href="'.$antiXss->xss_clean($url).'";';
63 63
     echo '</script>';
64 64
     echo '<noscript>';
65
-    echo '<meta http-equiv="refresh" content="0;url=' . $antiXss->xss_clean($url) . '" />';
65
+    echo '<meta http-equiv="refresh" content="0;url='.$antiXss->xss_clean($url).'" />';
66 66
     echo '</noscript>';
67 67
 }
68 68
 
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
88 88
     && isset($SETTINGS['enable_sts']) === true
89 89
     && (int) $SETTINGS['enable_sts'] === 1
90 90
 ) {
91
-    teampassRedirect('https://' . $server['http_host'] . $server['request_uri']);
91
+    teampassRedirect('https://'.$server['http_host'].$server['request_uri']);
92 92
 }
93 93
 
94 94
 // Load pwComplexity
@@ -111,11 +111,11 @@  discard block
 block discarded – undo
111 111
 // LOAD CPASSMAN SETTINGS
112 112
 if (
113 113
     isset($SETTINGS['cpassman_dir']) === true
114
-    && is_dir($SETTINGS['cpassman_dir'] . '/install') === true
114
+    && is_dir($SETTINGS['cpassman_dir'].'/install') === true
115 115
 ) {
116 116
     // Should we delete folder INSTALL?
117 117
     $row = DB::queryFirstRow(
118
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s AND intitule=%s',
118
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s AND intitule=%s',
119 119
         'install',
120 120
         'clear_install_folder'
121 121
     );
@@ -133,11 +133,11 @@  discard block
 block discarded – undo
133 133
             if ($directories !== false) {
134 134
                 $files = array_diff($directories, ['.', '..']);
135 135
                 foreach ($files as $file) {
136
-                    if (is_dir($dir . '/' . $file)) {
137
-                        delTree($dir . '/' . $file);
136
+                    if (is_dir($dir.'/'.$file)) {
137
+                        delTree($dir.'/'.$file);
138 138
                     } else {
139 139
                         try {
140
-                            unlink($dir . '/' . $file);
140
+                            unlink($dir.'/'.$file);
141 141
                         } catch (Exception $e) {
142 142
                             // do nothing... php will ignore and continue
143 143
                         }
@@ -151,13 +151,13 @@  discard block
 block discarded – undo
151 151
             return false;
152 152
         }
153 153
 
154
-        if (is_dir($SETTINGS['cpassman_dir'] . '/install')) {
154
+        if (is_dir($SETTINGS['cpassman_dir'].'/install')) {
155 155
             // Set the permissions on the install directory and delete
156 156
             // is server Windows or Linux?
157 157
             if (strtoupper(substr(PHP_OS, 0, 3)) !== 'WIN') {
158
-                recursiveChmod($SETTINGS['cpassman_dir'] . '/install', 0755, 0440);
158
+                recursiveChmod($SETTINGS['cpassman_dir'].'/install', 0755, 0440);
159 159
             }
160
-            delTree($SETTINGS['cpassman_dir'] . '/install');
160
+            delTree($SETTINGS['cpassman_dir'].'/install');
161 161
         }
162 162
 
163 163
         // Delete temporary install table
@@ -175,10 +175,10 @@  discard block
 block discarded – undo
175 175
 // Load Languages stuff
176 176
 if (isset($languagesList) === false) {
177 177
     $languagesList = [];
178
-    $rows = DB::query('SELECT * FROM ' . prefixTable('languages') . ' GROUP BY name, label, code, flag, id ORDER BY name ASC');
178
+    $rows = DB::query('SELECT * FROM '.prefixTable('languages').' GROUP BY name, label, code, flag, id ORDER BY name ASC');
179 179
     foreach ($rows as $record) {
180 180
         array_push($languagesList, $record['name']);
181
-        if ($superGlobal->get('user_language', 'SESSION', 'user') === $record['name'] ) {
181
+        if ($superGlobal->get('user_language', 'SESSION', 'user') === $record['name']) {
182 182
             $superGlobal->put('user_language_flag', $record['flag'], 'SESSION');
183 183
             $superGlobal->put('user_language_code', $record['code'], 'SESSION');
184 184
             $superGlobal->put('user_language_label', $record['label'], 'SESSION');
@@ -233,7 +233,7 @@  discard block
 block discarded – undo
233 233
 // CHECK IF SESSION EXISTS AND IF SESSION IS VALID
234 234
 if (empty($_SESSION['sessionDuration']) === false) {
235 235
     $dataSession = DB::queryFirstRow(
236
-        'SELECT key_tempo FROM ' . prefixTable('users') . ' WHERE id=%i',
236
+        'SELECT key_tempo FROM '.prefixTable('users').' WHERE id=%i',
237 237
         $_SESSION['user_id']
238 238
     );
239 239
 } else {
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
     && (isset($_SESSION['user_admin']) === true && $_SESSION['user_admin'] === 1)
298 298
 ) {
299 299
     $row = DB::queryFirstRow(
300
-        'SELECT valeur FROM ' . prefixTable('misc') . ' WHERE type=%s_type AND intitule=%s_intitule',
300
+        'SELECT valeur FROM '.prefixTable('misc').' WHERE type=%s_type AND intitule=%s_intitule',
301 301
         [
302 302
             'type' => 'admin',
303 303
             'intitule' => 'teampass_version',
@@ -338,7 +338,7 @@  discard block
 block discarded – undo
338 338
 
339 339
         syslog(
340 340
             LOG_WARNING,
341
-            'Unlog user: ' . date('Y/m/d H:i:s') . " {$server['remote_addr']} ({$server['http_user_agent']})"
341
+            'Unlog user: '.date('Y/m/d H:i:s')." {$server['remote_addr']} ({$server['http_user_agent']})"
342 342
         );
343 343
         // erase session table
344 344
         $_SESSION = [];
@@ -389,7 +389,7 @@  discard block
 block discarded – undo
389 389
 if (isset($_SESSION['user_id']) === true && empty($_SESSION['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 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 FROM '.prefixTable('users').' WHERE id=%i',
393 393
         $_SESSION['user_id']
394 394
     );
395 395
     //Check if user has been deleted or unlogged
@@ -435,7 +435,7 @@  discard block
 block discarded – undo
435 435
             $data['groupes_visibles'],
436 436
             $data['groupes_interdits'],
437 437
             $data['admin'],
438
-            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']),
438
+            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 439
             $SETTINGS
440 440
         );
441 441
         if (isset($_SESSION['can_create_root_folder']) === true && (int) $_SESSION['can_create_root_folder'] === 1) {
@@ -470,7 +470,7 @@  discard block
 block discarded – undo
470 470
     $_SESSION['item_fields'] = [];
471 471
     $rows = DB::query(
472 472
         'SELECT *
473
-            FROM ' . prefixTable('categories') . '
473
+            FROM ' . prefixTable('categories').'
474 474
             WHERE level=%i',
475 475
         '0'
476 476
     );
@@ -479,7 +479,7 @@  discard block
 block discarded – undo
479 479
         // get each field
480 480
         $rows2 = DB::query(
481 481
             'SELECT *
482
-            FROM ' . prefixTable('categories') . '
482
+            FROM ' . prefixTable('categories').'
483 483
             WHERE parent_id=%i
484 484
             ORDER BY `order` ASC',
485 485
             $record['id']
@@ -557,7 +557,7 @@  discard block
 block discarded – undo
557 557
 if (
558 558
     isset($SETTINGS['roles_allowed_to_print']) === true
559 559
     && isset($_SESSION['user_roles']) === true
560
-    && (! isset($_SESSION['temporary']['user_can_printout']) || empty($_SESSION['temporary']['user_can_printout']))
560
+    && (!isset($_SESSION['temporary']['user_can_printout']) || empty($_SESSION['temporary']['user_can_printout']))
561 561
 ) {
562 562
     foreach (explode(';', $SETTINGS['roles_allowed_to_print']) as $role) {
563 563
         if (in_array($role, $_SESSION['user_roles']) === true) {
@@ -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['nb_users_online'] = DB::count();
Please login to merge, or discard this patch.