Test Setup Failed
Push — master ( 57ad8c...1e5da0 )
by Julito
91:54 queued 57:30
created
main/inc/lib/course.lib.php 2 patches
Spacing   +319 added lines, -320 removed lines patch added patch discarded remove patch
@@ -178,8 +178,8 @@  discard block
 block discarded – undo
178 178
     {
179 179
         return Database::fetch_array(
180 180
             Database::query(
181
-                "SELECT *, id as real_id FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . "
182
-                WHERE code='" . Database::escape_string($course_code) . "'"), 'ASSOC'
181
+                "SELECT *, id as real_id FROM ".Database::get_main_table(TABLE_MAIN_COURSE)."
182
+                WHERE code='" . Database::escape_string($course_code)."'"), 'ASSOC'
183 183
         );
184 184
     }
185 185
 
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
         $alsoSearchCode = false,
208 208
         $conditionsLike = array()
209 209
     ) {
210
-        $sql = "SELECT course.* FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . " course ";
210
+        $sql = "SELECT course.* FROM ".Database::get_main_table(TABLE_MAIN_COURSE)." course ";
211 211
 
212 212
         if (!empty($urlId)) {
213 213
             $table = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE);
@@ -215,9 +215,9 @@  discard block
 block discarded – undo
215 215
         }
216 216
 
217 217
         if (!empty($startwith)) {
218
-            $sql .= "WHERE (title LIKE '" . Database::escape_string($startwith) . "%' ";
218
+            $sql .= "WHERE (title LIKE '".Database::escape_string($startwith)."%' ";
219 219
             if ($alsoSearchCode) {
220
-                $sql .= "OR code LIKE '" . Database::escape_string($startwith) . "%' ";
220
+                $sql .= "OR code LIKE '".Database::escape_string($startwith)."%' ";
221 221
             }
222 222
             $sql .= ') ';
223 223
             if ($visibility !== -1 && $visibility == strval(intval($visibility))) {
@@ -251,19 +251,19 @@  discard block
 block discarded – undo
251 251
                 $value = Database::escape_string($value);
252 252
                 $simple_like = false;
253 253
                 if ($simple_like) {
254
-                    $temp_conditions[] = $field . " LIKE '$value%'";
254
+                    $temp_conditions[] = $field." LIKE '$value%'";
255 255
                 } else {
256
-                    $temp_conditions[] = $field . ' LIKE \'%' . $value . '%\'';
256
+                    $temp_conditions[] = $field.' LIKE \'%'.$value.'%\'';
257 257
                 }
258 258
             }
259 259
             $condition = ' AND ';
260 260
             if (!empty($temp_conditions)) {
261
-                $sql .= implode(' ' . $condition . ' ', $temp_conditions);
261
+                $sql .= implode(' '.$condition.' ', $temp_conditions);
262 262
             }
263 263
         }
264 264
 
265 265
         if (!empty($orderby)) {
266
-            $sql .= " ORDER BY " . Database::escape_string($orderby) . " ";
266
+            $sql .= " ORDER BY ".Database::escape_string($orderby)." ";
267 267
         } else {
268 268
             $sql .= " ORDER BY 1 ";
269 269
         }
@@ -275,13 +275,13 @@  discard block
 block discarded – undo
275 275
         }
276 276
 
277 277
         if (!empty($howmany) && is_int($howmany) and $howmany > 0) {
278
-            $sql .= ' LIMIT ' . Database::escape_string($howmany);
278
+            $sql .= ' LIMIT '.Database::escape_string($howmany);
279 279
         } else {
280 280
             $sql .= ' LIMIT 1000000'; //virtually no limit
281 281
         }
282 282
         if (!empty($from)) {
283 283
             $from = intval($from);
284
-            $sql .= ' OFFSET ' . intval($from);
284
+            $sql .= ' OFFSET '.intval($from);
285 285
         } else {
286 286
             $sql .= ' OFFSET 0';
287 287
         }
@@ -313,10 +313,10 @@  discard block
 block discarded – undo
313 313
 
314 314
         $result = Database::fetch_array(
315 315
             Database::query(
316
-                "SELECT status FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
316
+                "SELECT status FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)."
317 317
                 WHERE
318 318
                     c_id  = $courseId AND
319
-                    user_id = " . intval($userId)
319
+                    user_id = ".intval($userId)
320 320
             )
321 321
         );
322 322
 
@@ -334,9 +334,9 @@  discard block
 block discarded – undo
334 334
 
335 335
         $result = Database::fetch_array(
336 336
             Database::query("
337
-                SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
337
+                SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER)."
338 338
                 WHERE
339
-                    c_id  = '" . intval($courseId). "' AND
339
+                    c_id  = '" . intval($courseId)."' AND
340 340
                     user_id = " . intval($userId)
341 341
             )
342 342
         );
@@ -383,9 +383,9 @@  discard block
 block discarded – undo
383 383
         $result = Database::fetch_array(
384 384
             Database::query("
385 385
                 SELECT is_tutor
386
-                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
386
+                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER)."
387 387
                 WHERE
388
-                    c_id = '" . Database::escape_string($courseId) . "' AND
388
+                    c_id = '" . Database::escape_string($courseId)."' AND
389 389
                     user_id = " . intval($user_id)
390 390
             )
391 391
         );
@@ -438,17 +438,17 @@  discard block
 block discarded – undo
438 438
         $course_id = $course_info['real_id'];
439 439
 
440 440
         // Unsubscribe user from all groups in the course.
441
-        $sql = "DELETE FROM " . Database::get_course_table(TABLE_GROUP_USER) . "
442
-                WHERE c_id = $course_id AND user_id IN (" . $user_ids . ")";
441
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_GROUP_USER)."
442
+                WHERE c_id = $course_id AND user_id IN (".$user_ids.")";
443 443
         Database::query($sql);
444
-        $sql = "DELETE FROM " . Database::get_course_table(TABLE_GROUP_TUTOR) . "
445
-                WHERE c_id = $course_id AND user_id IN (" . $user_ids . ")";
444
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_GROUP_TUTOR)."
445
+                WHERE c_id = $course_id AND user_id IN (".$user_ids.")";
446 446
         Database::query($sql);
447 447
 
448 448
         // Erase user student publications (works) in the course - by André Boivin
449 449
 
450 450
         if (!empty($user_list)) {
451
-            require_once api_get_path(SYS_CODE_PATH) . 'work/work.lib.php';
451
+            require_once api_get_path(SYS_CODE_PATH).'work/work.lib.php';
452 452
             foreach ($user_list as $userId) {
453 453
                 // Getting all work from user
454 454
                 $workList = getWorkPerUser($userId);
@@ -467,74 +467,74 @@  discard block
 block discarded – undo
467 467
         }
468 468
 
469 469
         // Unsubscribe user from all blogs in the course.
470
-        Database::query("DELETE FROM " . Database::get_course_table(TABLE_BLOGS_REL_USER) . " WHERE c_id = $course_id AND  user_id IN (" . $user_ids . ")");
471
-        Database::query("DELETE FROM " . Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER) . " WHERE c_id = $course_id AND  user_id IN (" . $user_ids . ")");
470
+        Database::query("DELETE FROM ".Database::get_course_table(TABLE_BLOGS_REL_USER)." WHERE c_id = $course_id AND  user_id IN (".$user_ids.")");
471
+        Database::query("DELETE FROM ".Database::get_course_table(TABLE_BLOGS_TASKS_REL_USER)." WHERE c_id = $course_id AND  user_id IN (".$user_ids.")");
472 472
 
473 473
         // Deleting users in forum_notification and mailqueue course tables
474
-        $sql = "DELETE FROM  " . Database::get_course_table(TABLE_FORUM_NOTIFICATION) . "
475
-                WHERE c_id = $course_id AND user_id IN (" . $user_ids . ")";
474
+        $sql = "DELETE FROM  ".Database::get_course_table(TABLE_FORUM_NOTIFICATION)."
475
+                WHERE c_id = $course_id AND user_id IN (".$user_ids.")";
476 476
         Database::query($sql);
477 477
 
478
-        $sql = "DELETE FROM " . Database::get_course_table(TABLE_FORUM_MAIL_QUEUE) . "
479
-                WHERE c_id = $course_id AND user_id IN (" . $user_ids . ")";
478
+        $sql = "DELETE FROM ".Database::get_course_table(TABLE_FORUM_MAIL_QUEUE)."
479
+                WHERE c_id = $course_id AND user_id IN (".$user_ids.")";
480 480
         Database::query($sql);
481 481
 
482 482
         // Unsubscribe user from the course.
483 483
         if (!empty($session_id)) {
484 484
 
485 485
             // Delete in table session_rel_course_rel_user
486
-            $sql = "DELETE FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . "
486
+            $sql = "DELETE FROM ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)."
487 487
                     WHERE
488
-                        session_id ='" . $session_id . "' AND
489
-                        c_id = '" . $course_id . "' AND
488
+                        session_id ='" . $session_id."' AND
489
+                        c_id = '" . $course_id."' AND
490 490
                         user_id IN ($user_ids)";
491 491
             Database::query($sql);
492 492
 
493 493
             foreach ($user_list as $uid) {
494 494
                 // check if a user is register in the session with other course
495
-                $sql = "SELECT user_id FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . "
495
+                $sql = "SELECT user_id FROM ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)."
496 496
                         WHERE session_id='$session_id' AND user_id='$uid'";
497 497
                 $rs = Database::query($sql);
498 498
 
499 499
                 if (Database::num_rows($rs) == 0) {
500 500
                     // Delete in table session_rel_user
501
-                    $sql = "DELETE FROM " . Database::get_main_table(TABLE_MAIN_SESSION_USER) . "
501
+                    $sql = "DELETE FROM ".Database::get_main_table(TABLE_MAIN_SESSION_USER)."
502 502
                             WHERE
503
-                                session_id ='" . $session_id . "' AND
503
+                                session_id ='" . $session_id."' AND
504 504
                                 user_id = '$uid' AND
505
-                                relation_type<>" . SESSION_RELATION_TYPE_RRHH . "";
505
+                                relation_type<>".SESSION_RELATION_TYPE_RRHH."";
506 506
                     Database::query($sql);
507 507
                 }
508 508
             }
509 509
 
510 510
             // Update the table session
511
-            $sql = "SELECT COUNT(*) FROM " . Database::get_main_table(TABLE_MAIN_SESSION_USER) . "
512
-                    WHERE session_id = '" . $session_id . "' AND relation_type <> " . SESSION_RELATION_TYPE_RRHH;
511
+            $sql = "SELECT COUNT(*) FROM ".Database::get_main_table(TABLE_MAIN_SESSION_USER)."
512
+                    WHERE session_id = '" . $session_id."' AND relation_type <> ".SESSION_RELATION_TYPE_RRHH;
513 513
             $row = Database::fetch_array(Database::query($sql));
514 514
             $count = $row[0];
515 515
             // number of users by session
516
-            $sql = "UPDATE " . Database::get_main_table(TABLE_MAIN_SESSION) . " SET nbr_users = '$count'
517
-                    WHERE id = '" . $session_id . "'";
516
+            $sql = "UPDATE ".Database::get_main_table(TABLE_MAIN_SESSION)." SET nbr_users = '$count'
517
+                    WHERE id = '".$session_id."'";
518 518
             Database::query($sql);
519 519
 
520 520
             // Update the table session_rel_course
521
-            $sql = "SELECT COUNT(*) FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . "
521
+            $sql = "SELECT COUNT(*) FROM ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)."
522 522
                     WHERE session_id = '$session_id' AND c_id = '$course_id' AND status<>2";
523 523
             $row = Database::fetch_array(@Database::query($sql));
524 524
             $count = $row[0];
525 525
 
526 526
             // number of users by session and course
527
-            $sql = "UPDATE " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE) . "
527
+            $sql = "UPDATE ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE)."
528 528
                     SET nbr_users = '$count'
529 529
                     WHERE session_id = '$session_id' AND c_id = '$course_id'";
530 530
             Database::query($sql);
531 531
 
532 532
         } else {
533
-            $sql = "DELETE FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
533
+            $sql = "DELETE FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)."
534 534
                     WHERE
535
-                        user_id IN (" . $user_ids . ") AND
536
-                        relation_type<>" . COURSE_RELATION_TYPE_RRHH . " AND
537
-                        c_id = '" . $course_id . "'";
535
+                        user_id IN (" . $user_ids.") AND
536
+                        relation_type<>" . COURSE_RELATION_TYPE_RRHH." AND
537
+                        c_id = '" . $course_id."'";
538 538
             Database::query($sql);
539 539
 
540 540
             // add event to system log
@@ -604,17 +604,17 @@  discard block
 block discarded – undo
604 604
         $status = ($status == STUDENT || $status == COURSEMANAGER) ? $status : STUDENT;
605 605
 
606 606
         // A preliminary check whether the user has bben already registered on the platform.
607
-        $sql = "SELECT status FROM " . Database::get_main_table(TABLE_MAIN_USER) . "
607
+        $sql = "SELECT status FROM ".Database::get_main_table(TABLE_MAIN_USER)."
608 608
                 WHERE user_id = '$user_id' ";
609 609
         if (Database::num_rows(Database::query($sql)) == 0) {
610 610
             return false; // The user has not been registered to the platform.
611 611
         }
612 612
 
613 613
         // Check whether the user has not been already subscribed to the course.
614
-        $sql = "SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "                    
614
+        $sql = "SELECT * FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)."                    
615 615
                 WHERE 
616 616
                     user_id = '$user_id' AND 
617
-                    relation_type <> " . COURSE_RELATION_TYPE_RRHH . " AND 
617
+                    relation_type <> ".COURSE_RELATION_TYPE_RRHH." AND 
618 618
                     c_id = $courseId
619 619
                 ";
620 620
         if (empty($session_id)) {
@@ -749,7 +749,7 @@  discard block
 block discarded – undo
749 749
         $courseId = $courseInfo['real_id'];
750 750
 
751 751
         // Check in advance whether the user has already been registered on the platform.
752
-        $sql = "SELECT status FROM " . $user_table . " WHERE user_id = $user_id ";
752
+        $sql = "SELECT status FROM ".$user_table." WHERE user_id = $user_id ";
753 753
         if (Database::num_rows(Database::query($sql)) == 0) {
754 754
             if ($debug) {
755 755
                 error_log('The user has not been registered to the platform');
@@ -761,7 +761,7 @@  discard block
 block discarded – undo
761 761
         $sql = "SELECT * FROM $course_user_table
762 762
                 WHERE
763 763
                     user_id = $user_id AND
764
-                    relation_type <> " . COURSE_RELATION_TYPE_RRHH . " AND
764
+                    relation_type <> ".COURSE_RELATION_TYPE_RRHH." AND
765 765
                     c_id = $courseId";
766 766
         if (Database::num_rows(Database::query($sql)) > 0) {
767 767
             if ($debug) {
@@ -773,7 +773,7 @@  discard block
 block discarded – undo
773 773
         if (!api_is_course_admin()) {
774 774
             // Check in advance whether subscription is allowed or not for this course.
775 775
             $sql = "SELECT code, visibility FROM $course_table
776
-                    WHERE id = $courseId AND subscribe = '" . SUBSCRIBE_NOT_ALLOWED . "'";
776
+                    WHERE id = $courseId AND subscribe = '".SUBSCRIBE_NOT_ALLOWED."'";
777 777
             if (Database::num_rows(Database::query($sql)) > 0) {
778 778
                 if ($debug) {
779 779
                     error_log('Subscription is not allowed for this course');
@@ -822,7 +822,7 @@  discard block
 block discarded – undo
822 822
         $courseId = $courseInfo['real_id'];
823 823
 
824 824
         // Check in advance whether the user has already been registered on the platform.
825
-        $sql = "SELECT status FROM " . $userTable . " WHERE user_id = $userId ";
825
+        $sql = "SELECT status FROM ".$userTable." WHERE user_id = $userId ";
826 826
         if (Database::num_rows(Database::query($sql)) == 0) {
827 827
             if ($debug) {
828 828
                 error_log('The user has not been registered to the platform');
@@ -834,7 +834,7 @@  discard block
 block discarded – undo
834 834
         $sql = "SELECT * FROM $courseUserTable
835 835
                 WHERE
836 836
                     user_id = $userId AND
837
-                    visible = " . $visible . " AND
837
+                    visible = ".$visible." AND
838 838
                     c_id = $courseId";
839 839
         if (Database::num_rows(Database::query($sql)) > 0) {
840 840
             if ($debug) {
@@ -881,7 +881,7 @@  discard block
 block discarded – undo
881 881
         $sql = "SELECT * FROM $courseUserTable
882 882
                 WHERE
883 883
                     user_id = $userId AND
884
-                    visible = " . $visible . " AND
884
+                    visible = ".$visible." AND
885 885
                     c_id = $courseId";
886 886
         if (Database::num_rows(Database::query($sql)) > 0) {
887 887
             $cond = array(
@@ -905,7 +905,7 @@  discard block
 block discarded – undo
905 905
     {
906 906
         $wanted_course_code = Database::escape_string($wanted_course_code);
907 907
         $sql = "SELECT COUNT(*) as number
908
-                FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . "
908
+                FROM " . Database::get_main_table(TABLE_MAIN_COURSE)."
909 909
                 WHERE code = '$wanted_course_code' OR visual_code = '$wanted_course_code'";
910 910
         $result = Database::fetch_array(Database::query($sql));
911 911
 
@@ -1101,7 +1101,7 @@  discard block
 block discarded – undo
1101 1101
         $sql = "SELECT * FROM $table
1102 1102
                 WHERE
1103 1103
                     user_id = $userId AND
1104
-                    relation_type = " . COURSE_RELATION_TYPE_RRHH . " AND
1104
+                    relation_type = ".COURSE_RELATION_TYPE_RRHH." AND
1105 1105
                     c_id = $courseId";
1106 1106
 
1107 1107
         $result = Database::fetch_array(Database::query($sql));
@@ -1140,13 +1140,13 @@  discard block
 block discarded – undo
1140 1140
                 return false;
1141 1141
             }
1142 1142
             $courseId = $courseInfo['real_id'];
1143
-            $condition_course = ' AND c_id = ' . $courseId;
1143
+            $condition_course = ' AND c_id = '.$courseId;
1144 1144
         }
1145 1145
 
1146
-        $sql = "SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
1146
+        $sql = "SELECT * FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)."
1147 1147
                 WHERE
1148 1148
                     user_id = $user_id AND
1149
-                    relation_type<>" . COURSE_RELATION_TYPE_RRHH . "
1149
+                    relation_type<>".COURSE_RELATION_TYPE_RRHH."
1150 1150
                     $condition_course ";
1151 1151
 
1152 1152
         $result = Database::fetch_array(Database::query($sql));
@@ -1162,20 +1162,20 @@  discard block
 block discarded – undo
1162 1162
         }
1163 1163
 
1164 1164
         $tableSessionCourseUser = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
1165
-        $sql = 'SELECT 1 FROM ' . $tableSessionCourseUser .
1166
-            ' WHERE user_id = ' . $user_id . ' ' . $condition_course;
1165
+        $sql = 'SELECT 1 FROM '.$tableSessionCourseUser.
1166
+            ' WHERE user_id = '.$user_id.' '.$condition_course;
1167 1167
         if (Database::num_rows(Database::query($sql)) > 0) {
1168 1168
             return true;
1169 1169
         }
1170 1170
 
1171
-        $sql = 'SELECT 1 FROM ' . $tableSessionCourseUser .
1172
-            ' WHERE user_id = ' . $user_id . ' AND status=2 ' . $condition_course;
1171
+        $sql = 'SELECT 1 FROM '.$tableSessionCourseUser.
1172
+            ' WHERE user_id = '.$user_id.' AND status=2 '.$condition_course;
1173 1173
         if (Database::num_rows(Database::query($sql)) > 0) {
1174 1174
             return true;
1175 1175
         }
1176 1176
 
1177
-        $sql = 'SELECT 1 FROM ' . Database::get_main_table(TABLE_MAIN_SESSION) .
1178
-            ' WHERE id = ' . $session_id . ' AND id_coach=' . $user_id;
1177
+        $sql = 'SELECT 1 FROM '.Database::get_main_table(TABLE_MAIN_SESSION).
1178
+            ' WHERE id = '.$session_id.' AND id_coach='.$user_id;
1179 1179
 
1180 1180
         if (Database::num_rows(Database::query($sql)) > 0) {
1181 1181
             return true;
@@ -1202,8 +1202,8 @@  discard block
 block discarded – undo
1202 1202
         $courseId = $courseInfo['real_id'];
1203 1203
 
1204 1204
         $result = Database::query(
1205
-            'SELECT status FROM ' . Database::get_main_table(TABLE_MAIN_COURSE_USER) .
1206
-            ' WHERE c_id = ' . $courseId . ' AND user_id = ' . $user_id . ''
1205
+            'SELECT status FROM '.Database::get_main_table(TABLE_MAIN_COURSE_USER).
1206
+            ' WHERE c_id = '.$courseId.' AND user_id = '.$user_id.''
1207 1207
         );
1208 1208
 
1209 1209
         if (Database::num_rows($result) > 0) {
@@ -1233,12 +1233,12 @@  discard block
 block discarded – undo
1233 1233
             $result = Database::fetch_array(
1234 1234
                 Database::query(
1235 1235
                     "SELECT *
1236
-                    FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . " course
1237
-                    LEFT JOIN " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . " course_user
1236
+                    FROM " . Database::get_main_table(TABLE_MAIN_COURSE)." course
1237
+                    LEFT JOIN " . Database::get_main_table(TABLE_MAIN_COURSE_USER)." course_user
1238 1238
                     ON course.id = course_user.c_id
1239 1239
                     WHERE
1240 1240
                         course_user.user_id = '$user_id' AND
1241
-                        course_user.relation_type<>" . COURSE_RELATION_TYPE_RRHH . " AND
1241
+                        course_user.relation_type<>".COURSE_RELATION_TYPE_RRHH." AND
1242 1242
                         ( course.id = '$courseId')"
1243 1243
                 )
1244 1244
             );
@@ -1252,8 +1252,8 @@  discard block
 block discarded – undo
1252 1252
 
1253 1253
         // A user?
1254 1254
         if (Database::num_rows(Database::query("SELECT user_id
1255
-                FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . "
1256
-                WHERE session_id='" . $session_id . "'
1255
+                FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)."
1256
+                WHERE session_id='" . $session_id."'
1257 1257
                 AND user_id ='$user_id'"))
1258 1258
         ) {
1259 1259
             return true;
@@ -1261,8 +1261,8 @@  discard block
 block discarded – undo
1261 1261
 
1262 1262
         // A course coach?
1263 1263
         if (Database::num_rows(Database::query("SELECT user_id
1264
-                FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . "
1265
-                WHERE session_id='" . $session_id . "'
1264
+                FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)."
1265
+                WHERE session_id='" . $session_id."'
1266 1266
                 AND user_id = '$user_id' AND status = 2
1267 1267
                 AND c_id ='$courseId'"))
1268 1268
         ) {
@@ -1271,8 +1271,8 @@  discard block
 block discarded – undo
1271 1271
 
1272 1272
         // A session coach?
1273 1273
         if (Database::num_rows(Database::query("SELECT id_coach
1274
-                FROM " . Database::get_main_table(TABLE_MAIN_SESSION) . " AS session
1275
-                WHERE session.id='" . $session_id . "'
1274
+                FROM " . Database::get_main_table(TABLE_MAIN_SESSION)." AS session
1275
+                WHERE session.id='" . $session_id."'
1276 1276
                 AND id_coach='$user_id'"))
1277 1277
         ) {
1278 1278
             return true;
@@ -1340,7 +1340,7 @@  discard block
 block discarded – undo
1340 1340
         // we have to check if it is a valid field that can be sorted on
1341 1341
         if (!strstr($order_by, 'ORDER BY')) {
1342 1342
             if (!empty($order_by)) {
1343
-                $order_by = 'ORDER BY ' . $order_by;
1343
+                $order_by = 'ORDER BY '.$order_by;
1344 1344
             } else {
1345 1345
                 $order_by = '';
1346 1346
             }
@@ -1372,11 +1372,11 @@  discard block
 block discarded – undo
1372 1372
             if (!empty($courseCodeList)) {
1373 1373
                 $courseCodeListForSession = array_map(array('Database', 'escape_string'), $courseCodeList);
1374 1374
                 $courseCodeListForSession = implode('","', $courseCodeListForSession);
1375
-                $courseCondition = ' course.code IN ("' . $courseCodeListForSession . '")  ';
1375
+                $courseCondition = ' course.code IN ("'.$courseCodeListForSession.'")  ';
1376 1376
             }
1377 1377
 
1378
-            $sql .= ' FROM ' . Database::get_main_table(TABLE_MAIN_USER) . ' as user ';
1379
-            $sql .= " LEFT JOIN ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . " as session_course_user
1378
+            $sql .= ' FROM '.Database::get_main_table(TABLE_MAIN_USER).' as user ';
1379
+            $sql .= " LEFT JOIN ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)." as session_course_user
1380 1380
                       ON
1381 1381
                         user.id = session_course_user.user_id AND
1382 1382
                         $sessionCondition
@@ -1417,15 +1417,15 @@  discard block
 block discarded – undo
1417 1417
                 }
1418 1418
             }
1419 1419
 
1420
-            $sql .= ' FROM ' . Database::get_main_table(TABLE_MAIN_USER) . ' as user '
1421
-                  . ' LEFT JOIN ' . Database::get_main_table(TABLE_MAIN_COURSE_USER) . ' as course_rel_user
1420
+            $sql .= ' FROM '.Database::get_main_table(TABLE_MAIN_USER).' as user '
1421
+                  . ' LEFT JOIN '.Database::get_main_table(TABLE_MAIN_COURSE_USER).' as course_rel_user
1422 1422
                       ON 
1423 1423
                         user.id = course_rel_user.user_id AND
1424
-                        course_rel_user.relation_type <> ' . COURSE_RELATION_TYPE_RRHH . '  '
1424
+                        course_rel_user.relation_type <> ' . COURSE_RELATION_TYPE_RRHH.'  '
1425 1425
                   . " INNER JOIN $course_table course ON course_rel_user.c_id = course.id ";
1426 1426
 
1427 1427
             if (!empty($course_code)) {
1428
-                $sql .= ' AND course_rel_user.c_id = "' . $courseId . '"';
1428
+                $sql .= ' AND course_rel_user.c_id = "'.$courseId.'"';
1429 1429
             }
1430 1430
             $where[] = ' course_rel_user.c_id IS NOT NULL ';
1431 1431
 
@@ -1437,7 +1437,7 @@  discard block
 block discarded – undo
1437 1437
 
1438 1438
         $multiple_access_url = api_get_multiple_access_url();
1439 1439
         if ($multiple_access_url) {
1440
-            $sql .= ' LEFT JOIN ' . Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER) . ' au
1440
+            $sql .= ' LEFT JOIN '.Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER).' au
1441 1441
                       ON (au.user_id = user.id) ';
1442 1442
         }
1443 1443
 
@@ -1457,7 +1457,7 @@  discard block
 block discarded – undo
1457 1457
             }
1458 1458
         }
1459 1459
 
1460
-        $sql .= ' WHERE ' . $filter_by_status_condition . ' ' . implode(' OR ', $where);
1460
+        $sql .= ' WHERE '.$filter_by_status_condition.' '.implode(' OR ', $where);
1461 1461
 
1462 1462
         if ($multiple_access_url) {
1463 1463
             $current_access_url_id = api_get_current_access_url_id();
@@ -1479,15 +1479,15 @@  discard block
 block discarded – undo
1479 1479
         if (!empty($userIdList)) {
1480 1480
             $userIdList = array_map('intval', $userIdList);
1481 1481
             $userIdList = implode('","', $userIdList);
1482
-            $sql .= ' AND user.id IN ("' . $userIdList . '")';
1482
+            $sql .= ' AND user.id IN ("'.$userIdList.'")';
1483 1483
         }
1484 1484
 
1485 1485
         if (isset($filterByActive)) {
1486 1486
             $filterByActive = intval($filterByActive);
1487
-            $sql .= ' AND user.active = ' . $filterByActive;
1487
+            $sql .= ' AND user.active = '.$filterByActive;
1488 1488
         }
1489 1489
 
1490
-        $sql .= ' ' . $order_by . ' ' . $limit;
1490
+        $sql .= ' '.$order_by.' '.$limit;
1491 1491
 
1492 1492
         $rs = Database::query($sql);
1493 1493
         $users = array();
@@ -1633,7 +1633,7 @@  discard block
 block discarded – undo
1633 1633
                             )
1634 1634
                         );
1635 1635
                         $report_info['progress_100'] = $progress == 100 ? Display::label(get_lang('Yes'), 'success') : Display::label(get_lang('No'));
1636
-                        $report_info['progress'] = $progress . "%";
1636
+                        $report_info['progress'] = $progress."%";
1637 1637
 
1638 1638
                         foreach ($extra_fields as $extra) {
1639 1639
                             $user_data = UserManager::get_extra_user_data_by_field($user['user_id'], $extra['1']);
@@ -1703,32 +1703,32 @@  discard block
 block discarded – undo
1703 1703
         $courseId = $courseInfo['real_id'];
1704 1704
 
1705 1705
         $sql = 'SELECT DISTINCT count(user.id) as count  
1706
-                FROM ' . Database::get_main_table(TABLE_MAIN_USER) . ' as user ';
1706
+                FROM ' . Database::get_main_table(TABLE_MAIN_USER).' as user ';
1707 1707
         $where = array();
1708 1708
         if (!empty($session_id)) {
1709
-            $sql .= ' LEFT JOIN ' . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . ' as session_course_user
1709
+            $sql .= ' LEFT JOIN '.Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER).' as session_course_user
1710 1710
                       ON
1711 1711
                         user.user_id = session_course_user.user_id AND
1712
-                        session_course_user.c_id = "' . $courseId . '" AND
1712
+                        session_course_user.c_id = "' . $courseId.'" AND
1713 1713
                         session_course_user.session_id  = ' . $session_id;
1714 1714
 
1715 1715
             $where[] = ' session_course_user.c_id IS NOT NULL ';
1716 1716
         } else {
1717
-            $sql .= ' LEFT JOIN ' . Database::get_main_table(TABLE_MAIN_COURSE_USER) . ' as course_rel_user
1717
+            $sql .= ' LEFT JOIN '.Database::get_main_table(TABLE_MAIN_COURSE_USER).' as course_rel_user
1718 1718
                         ON
1719 1719
                             user.user_id = course_rel_user.user_id AND
1720
-                            course_rel_user.relation_type<>' . COURSE_RELATION_TYPE_RRHH . ' AND
1721
-                            course_rel_user.c_id = ' . $courseId ;
1720
+                            course_rel_user.relation_type<>' . COURSE_RELATION_TYPE_RRHH.' AND
1721
+                            course_rel_user.c_id = ' . $courseId;
1722 1722
             $where[] = ' course_rel_user.c_id IS NOT NULL ';
1723 1723
         }
1724 1724
 
1725 1725
         $multiple_access_url = api_get_multiple_access_url();
1726 1726
         if ($multiple_access_url) {
1727
-            $sql .= ' LEFT JOIN ' . Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER) . '  au
1727
+            $sql .= ' LEFT JOIN '.Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_USER).'  au
1728 1728
                       ON (au.user_id = user.user_id) ';
1729 1729
         }
1730 1730
 
1731
-        $sql .= ' WHERE ' . implode(' OR ', $where);
1731
+        $sql .= ' WHERE '.implode(' OR ', $where);
1732 1732
 
1733 1733
         if ($multiple_access_url) {
1734 1734
             $current_access_url_id = api_get_current_access_url_id();
@@ -1765,8 +1765,8 @@  discard block
 block discarded – undo
1765 1765
         $users = array();
1766 1766
 
1767 1767
         // We get the coach for the given course in a given session.
1768
-        $sql = 'SELECT user_id FROM ' . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) .
1769
-               ' WHERE session_id ="' . $session_id . '" AND c_id="' . $courseId . '" AND status = 2';
1768
+        $sql = 'SELECT user_id FROM '.Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER).
1769
+               ' WHERE session_id ="'.$session_id.'" AND c_id="'.$courseId.'" AND status = 2';
1770 1770
         $rs = Database::query($sql);
1771 1771
         while ($user = Database::fetch_array($rs)) {
1772 1772
             $userInfo = api_get_user_info($user['user_id']);
@@ -1853,11 +1853,11 @@  discard block
 block discarded – undo
1853 1853
             $joinSession = "";
1854 1854
             //Session creation date
1855 1855
             if (!empty($date_from) && !empty($date_to)) {
1856
-                $joinSession = "INNER JOIN " . Database::get_main_table(TABLE_MAIN_SESSION) . " s";
1856
+                $joinSession = "INNER JOIN ".Database::get_main_table(TABLE_MAIN_SESSION)." s";
1857 1857
             }
1858 1858
 
1859 1859
             $sql_query = "SELECT *
1860
-                          FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . " scu
1860
+                          FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)." scu
1861 1861
                           $joinSession
1862 1862
                           INNER JOIN $userTable u ON scu.user_id = u.user_id
1863 1863
                           WHERE scu.c_id = '$courseId' AND scu.status <> 2";
@@ -1869,11 +1869,11 @@  discard block
 block discarded – undo
1869 1869
             }
1870 1870
 
1871 1871
             if ($session_id != 0) {
1872
-                $sql_query .= ' AND scu.session_id = ' . $session_id;
1872
+                $sql_query .= ' AND scu.session_id = '.$session_id;
1873 1873
             }
1874 1874
 
1875 1875
             if (!$includeInvitedUsers) {
1876
-                $sql_query .= " AND u.status != " . INVITEE;
1876
+                $sql_query .= " AND u.status != ".INVITEE;
1877 1877
             }
1878 1878
 
1879 1879
             $rs = Database::query($sql_query);
@@ -1907,8 +1907,8 @@  discard block
 block discarded – undo
1907 1907
                     u.email,
1908 1908
                     u.username,
1909 1909
                     u.status
1910
-                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . " cu
1911
-                INNER JOIN " . Database::get_main_table(TABLE_MAIN_USER) . " u
1910
+                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER)." cu
1911
+                INNER JOIN " . Database::get_main_table(TABLE_MAIN_USER)." u
1912 1912
                 ON (cu.user_id = u.id)
1913 1913
                 WHERE
1914 1914
                     cu.c_id = $courseId AND
@@ -1946,8 +1946,8 @@  discard block
 block discarded – undo
1946 1946
                     u.email,
1947 1947
                     u.username,
1948 1948
                     u.status
1949
-                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . " cu
1950
-                INNER JOIN " . Database::get_main_table(TABLE_MAIN_USER) . " u
1949
+                FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER)." cu
1950
+                INNER JOIN " . Database::get_main_table(TABLE_MAIN_USER)." u
1951 1951
                 ON (cu.user_id = u.id)
1952 1952
                 WHERE
1953 1953
                     cu.c_id = $courseId AND
@@ -1955,7 +1955,7 @@  discard block
 block discarded – undo
1955 1955
         $rs = Database::query($sql);
1956 1956
         $listTeachers = array();
1957 1957
         $teachers = array();
1958
-        $url = api_get_path(WEB_AJAX_PATH) . 'user_manager.ajax.php?a=get_user_popup';
1958
+        $url = api_get_path(WEB_AJAX_PATH).'user_manager.ajax.php?a=get_user_popup';
1959 1959
         while ($teacher = Database::fetch_array($rs)) {
1960 1960
             $teachers['id'] = $teacher['user_id'];
1961 1961
             $teachers['lastname'] = $teacher['lastname'];
@@ -1968,7 +1968,7 @@  discard block
 block discarded – undo
1968 1968
                 $userPicture = UserManager::getUserPicture($teacher['user_id'], USER_IMAGE_SIZE_SMALL);
1969 1969
                 $teachers['avatar'] = $userPicture;
1970 1970
             }
1971
-            $teachers['url'] = $url.'&user_id='. $teacher['user_id'];
1971
+            $teachers['url'] = $url.'&user_id='.$teacher['user_id'];
1972 1972
             $listTeachers[] = $teachers;
1973 1973
         }
1974 1974
 
@@ -1998,7 +1998,7 @@  discard block
 block discarded – undo
1998 1998
                     $teacher['lastname']
1999 1999
                 );
2000 2000
                 if ($add_link_to_profile) {
2001
-                    $url = api_get_path(WEB_AJAX_PATH) . 'user_manager.ajax.php?a=get_user_popup&user_id=' . $teacher['user_id'];
2001
+                    $url = api_get_path(WEB_AJAX_PATH).'user_manager.ajax.php?a=get_user_popup&user_id='.$teacher['user_id'];
2002 2002
                     $teacher_name = Display::url(
2003 2003
                         $teacher_name,
2004 2004
                         $url,
@@ -2014,8 +2014,8 @@  discard block
 block discarded – undo
2014 2014
             if (!empty($list)) {
2015 2015
                 if ($orderList === true) {
2016 2016
                     $html .= '<ul class="user-teacher">';
2017
-                    foreach ($list as $teacher){
2018
-                        $html .= Display::tag('li', Display::return_icon('teacher.png', $teacher, null, ICON_SIZE_TINY) . ' ' . $teacher);
2017
+                    foreach ($list as $teacher) {
2018
+                        $html .= Display::tag('li', Display::return_icon('teacher.png', $teacher, null, ICON_SIZE_TINY).' '.$teacher);
2019 2019
                     }
2020 2020
                     $html .= '</ul>';
2021 2021
                 } else {
@@ -2096,7 +2096,7 @@  discard block
 block discarded – undo
2096 2096
             foreach ($coachList as $coach_course) {
2097 2097
                 $coach_name = $coach_course['full_name'];
2098 2098
                 if ($add_link_to_profile) {
2099
-                    $url = api_get_path(WEB_AJAX_PATH) . 'user_manager.ajax.php?a=get_user_popup&user_id=' . $coach_course['user_id'];
2099
+                    $url = api_get_path(WEB_AJAX_PATH).'user_manager.ajax.php?a=get_user_popup&user_id='.$coach_course['user_id'];
2100 2100
                     $coach_name = Display::url(
2101 2101
                         $coach_name,
2102 2102
                         $url,
@@ -2117,7 +2117,7 @@  discard block
 block discarded – undo
2117 2117
                 foreach ($course_coachs as $coachs) {
2118 2118
                     $html .= Display::tag(
2119 2119
                         'li',
2120
-                        Display::return_icon('teacher.png', $coachs, null, ICON_SIZE_TINY) . ' ' . $coachs
2120
+                        Display::return_icon('teacher.png', $coachs, null, ICON_SIZE_TINY).' '.$coachs
2121 2121
                     );
2122 2122
                 }
2123 2123
                 $html .= '</ul>';
@@ -2170,20 +2170,20 @@  discard block
 block discarded – undo
2170 2170
         }
2171 2171
 
2172 2172
         $group_list = array();
2173
-        $session_id != 0 ? $session_condition = ' WHERE g.session_id IN(1,' . intval($session_id) . ')' : $session_condition = ' WHERE g.session_id = 0';
2173
+        $session_id != 0 ? $session_condition = ' WHERE g.session_id IN(1,'.intval($session_id).')' : $session_condition = ' WHERE g.session_id = 0';
2174 2174
 
2175 2175
         if ($in_get_empty_group == 0) {
2176 2176
             // get only groups that are not empty
2177 2177
             $sql = "SELECT DISTINCT g.id, g.iid, g.name
2178
-                    FROM " . Database::get_course_table(TABLE_GROUP) . " AS g
2179
-                    INNER JOIN " . Database::get_course_table(TABLE_GROUP_USER) . " gu
2178
+                    FROM " . Database::get_course_table(TABLE_GROUP)." AS g
2179
+                    INNER JOIN " . Database::get_course_table(TABLE_GROUP_USER)." gu
2180 2180
                     ON (g.id = gu.group_id AND g.c_id = $course_id AND gu.c_id = $course_id)
2181 2181
                     $session_condition
2182 2182
                     ORDER BY g.name";
2183 2183
         } else {
2184 2184
             // get all groups even if they are empty
2185 2185
             $sql = "SELECT g.id, g.name, g.iid 
2186
-                    FROM " . Database::get_course_table(TABLE_GROUP) . " AS g
2186
+                    FROM " . Database::get_course_table(TABLE_GROUP)." AS g
2187 2187
                     $session_condition
2188 2188
                     AND c_id = $course_id";
2189 2189
         }
@@ -2239,7 +2239,7 @@  discard block
 block discarded – undo
2239 2239
         $table_stats_uploads = Database::get_main_table(TABLE_STATISTIC_TRACK_E_UPLOADS);
2240 2240
 
2241 2241
         $codeFiltered = Database::escape_string($code);
2242
-        $sql = "SELECT * FROM $table_course WHERE code='" . $codeFiltered . "'";
2242
+        $sql = "SELECT * FROM $table_course WHERE code='".$codeFiltered."'";
2243 2243
         $res = Database::query($sql);
2244 2244
 
2245 2245
         if (Database::num_rows($res) == 0) {
@@ -2247,7 +2247,7 @@  discard block
 block discarded – undo
2247 2247
         }
2248 2248
 
2249 2249
         $sql = "SELECT * FROM $table_course
2250
-                WHERE code = '" . $codeFiltered . "'";
2250
+                WHERE code = '".$codeFiltered."'";
2251 2251
         $res = Database::query($sql);
2252 2252
         $course = Database::fetch_array($res);
2253 2253
 
@@ -2294,34 +2294,34 @@  discard block
 block discarded – undo
2294 2294
                 }
2295 2295
             }
2296 2296
 
2297
-            $course_dir = api_get_path(SYS_COURSE_PATH) . $course['directory'];
2298
-            $archive_dir = api_get_path(SYS_ARCHIVE_PATH) . $course['directory'] . '_' . time();
2297
+            $course_dir = api_get_path(SYS_COURSE_PATH).$course['directory'];
2298
+            $archive_dir = api_get_path(SYS_ARCHIVE_PATH).$course['directory'].'_'.time();
2299 2299
             if (is_dir($course_dir)) {
2300 2300
                 rename($course_dir, $archive_dir);
2301 2301
             }
2302 2302
 
2303 2303
             // Unsubscribe all users from the course
2304
-            $sql = "DELETE FROM $table_course_user WHERE c_id='" . $courseId . "'";
2304
+            $sql = "DELETE FROM $table_course_user WHERE c_id='".$courseId."'";
2305 2305
             Database::query($sql);
2306 2306
             // Delete the course from the sessions tables
2307
-            $sql = "DELETE FROM $table_session_course WHERE c_id='" . $courseId . "'";
2307
+            $sql = "DELETE FROM $table_session_course WHERE c_id='".$courseId."'";
2308 2308
             Database::query($sql);
2309
-            $sql = "DELETE FROM $table_session_course_user WHERE c_id='" . $courseId . "'";
2309
+            $sql = "DELETE FROM $table_session_course_user WHERE c_id='".$courseId."'";
2310 2310
             Database::query($sql);
2311 2311
 
2312 2312
             // Delete from Course - URL
2313
-            $sql = "DELETE FROM $table_course_rel_url WHERE c_id = '" . $courseId. "'";
2313
+            $sql = "DELETE FROM $table_course_rel_url WHERE c_id = '".$courseId."'";
2314 2314
             Database::query($sql);
2315 2315
 
2316
-            $sql = 'SELECT survey_id FROM ' . $table_course_survey . ' WHERE course_code="' . $codeFiltered . '"';
2316
+            $sql = 'SELECT survey_id FROM '.$table_course_survey.' WHERE course_code="'.$codeFiltered.'"';
2317 2317
             $result_surveys = Database::query($sql);
2318 2318
             while ($surveys = Database::fetch_array($result_surveys)) {
2319 2319
                 $survey_id = $surveys[0];
2320
-                $sql = 'DELETE FROM ' . $table_course_survey_question . ' WHERE survey_id="' . $survey_id . '"';
2320
+                $sql = 'DELETE FROM '.$table_course_survey_question.' WHERE survey_id="'.$survey_id.'"';
2321 2321
                 Database::query($sql);
2322
-                $sql = 'DELETE FROM ' . $table_course_survey_question_option . ' WHERE survey_id="' . $survey_id . '"';
2322
+                $sql = 'DELETE FROM '.$table_course_survey_question_option.' WHERE survey_id="'.$survey_id.'"';
2323 2323
                 Database::query($sql);
2324
-                $sql = 'DELETE FROM ' . $table_course_survey . ' WHERE survey_id="' . $survey_id . '"';
2324
+                $sql = 'DELETE FROM '.$table_course_survey.' WHERE survey_id="'.$survey_id.'"';
2325 2325
                 Database::query($sql);
2326 2326
             }
2327 2327
 
@@ -2373,11 +2373,11 @@  discard block
 block discarded – undo
2373 2373
             Category::deleteCategoryFromCourse($courseId);
2374 2374
 
2375 2375
             // Delete the course from the database
2376
-            $sql = "DELETE FROM gradebook_link WHERE c_id = '" . $courseId . "'";
2376
+            $sql = "DELETE FROM gradebook_link WHERE c_id = '".$courseId."'";
2377 2377
             Database::query($sql);
2378 2378
 
2379 2379
             // Delete the course from the database
2380
-            $sql = "DELETE FROM $table_course WHERE code = '" . $codeFiltered . "'";
2380
+            $sql = "DELETE FROM $table_course WHERE code = '".$codeFiltered."'";
2381 2381
             Database::query($sql);
2382 2382
 
2383 2383
             // delete extra course fields
@@ -2423,15 +2423,15 @@  discard block
 block discarded – undo
2423 2423
                 while ($row = Database::fetch_array($res_table, 'ASSOC')) {
2424 2424
                     $row_to_save = array();
2425 2425
                     foreach ($row as $key => $value) {
2426
-                        $row_to_save[$key] = $key . "='" . Database::escape_string($row[$key]) . "'";
2426
+                        $row_to_save[$key] = $key."='".Database::escape_string($row[$key])."'";
2427 2427
                     }
2428
-                    $sql_dump .= "\nINSERT INTO $table SET " . implode(', ', $row_to_save) . ';';
2428
+                    $sql_dump .= "\nINSERT INTO $table SET ".implode(', ', $row_to_save).';';
2429 2429
                 }
2430 2430
             }
2431 2431
         }
2432 2432
 
2433
-        if (is_dir(api_get_path(SYS_COURSE_PATH) . $course['directory'])) {
2434
-            $file_name = api_get_path(SYS_COURSE_PATH) . $course['directory'] . '/mysql_dump.sql';
2433
+        if (is_dir(api_get_path(SYS_COURSE_PATH).$course['directory'])) {
2434
+            $file_name = api_get_path(SYS_COURSE_PATH).$course['directory'].'/mysql_dump.sql';
2435 2435
             $handle = fopen($file_name, 'a+');
2436 2436
             if ($handle !== false) {
2437 2437
                 fwrite($handle, $sql_dump);
@@ -2460,13 +2460,13 @@  discard block
 block discarded – undo
2460 2460
         $TABLECOURSE = Database::get_main_table(TABLE_MAIN_COURSE);
2461 2461
         $TABLECOURSUSER = Database::get_main_table(TABLE_MAIN_COURSE_USER);
2462 2462
 
2463
-        $course_title = Database::result(Database::query('SELECT title FROM ' . $TABLECOURSE . ' WHERE code="' . $course_code . '"'),
2463
+        $course_title = Database::result(Database::query('SELECT title FROM '.$TABLECOURSE.' WHERE code="'.$course_code.'"'),
2464 2464
             0, 0);
2465 2465
 
2466 2466
         $sql = 'SELECT course.code as code, course.title as title, cu.sort as sort
2467
-                FROM ' . $TABLECOURSUSER . ' as cu, ' . $TABLECOURSE . ' as course
2468
-                WHERE   course.id = cu.c_id AND user_id = "' . $user_id . '" AND
2469
-                        cu.relation_type<>' . COURSE_RELATION_TYPE_RRHH . ' AND
2467
+                FROM ' . $TABLECOURSUSER.' as cu, '.$TABLECOURSE.' as course
2468
+                WHERE   course.id = cu.c_id AND user_id = "' . $user_id.'" AND
2469
+                        cu.relation_type<>' . COURSE_RELATION_TYPE_RRHH.' AND
2470 2470
                         user_course_cat = 0
2471 2471
                 ORDER BY cu.sort';
2472 2472
         $result = Database::query($sql);
@@ -2485,21 +2485,21 @@  discard block
 block discarded – undo
2485 2485
                     $course_found = true;
2486 2486
                     $course_sort = $courses['sort'];
2487 2487
                     if ($counter == 0) {
2488
-                        $sql = 'UPDATE ' . $TABLECOURSUSER . '
2488
+                        $sql = 'UPDATE '.$TABLECOURSUSER.'
2489 2489
                                 SET sort = sort+1
2490 2490
                                 WHERE
2491
-                                    user_id= "' . $user_id . '" AND
2492
-                                    relation_type<>' . COURSE_RELATION_TYPE_RRHH . '
2491
+                                    user_id= "' . $user_id.'" AND
2492
+                                    relation_type<>' . COURSE_RELATION_TYPE_RRHH.'
2493 2493
                                     AND user_course_cat="0"
2494
-                                    AND sort > "' . $course_sort . '"';
2494
+                                    AND sort > "' . $course_sort.'"';
2495 2495
                         $course_sort++;
2496 2496
                     } else {
2497
-                        $sql = 'UPDATE ' . $TABLECOURSUSER . ' SET sort = sort+1
2497
+                        $sql = 'UPDATE '.$TABLECOURSUSER.' SET sort = sort+1
2498 2498
                                 WHERE
2499
-                                    user_id= "' . $user_id . '" AND
2500
-                                    relation_type<>' . COURSE_RELATION_TYPE_RRHH . ' AND
2499
+                                    user_id= "' . $user_id.'" AND
2500
+                                    relation_type<>' . COURSE_RELATION_TYPE_RRHH.' AND
2501 2501
                                     user_course_cat="0" AND
2502
-                                    sort >= "' . $course_sort . '"';
2502
+                                    sort >= "' . $course_sort.'"';
2503 2503
                     }
2504 2504
                     Database::query($sql);
2505 2505
                     break;
@@ -2512,9 +2512,9 @@  discard block
 block discarded – undo
2512 2512
 
2513 2513
             // We must register the course in the beginning of the list
2514 2514
             if (!$course_found) {
2515
-                $course_sort = Database::result(Database::query('SELECT min(sort) as min_sort FROM ' . $TABLECOURSUSER . ' WHERE user_id="' . $user_id . '" AND user_course_cat="0"'),
2515
+                $course_sort = Database::result(Database::query('SELECT min(sort) as min_sort FROM '.$TABLECOURSUSER.' WHERE user_id="'.$user_id.'" AND user_course_cat="0"'),
2516 2516
                     0, 0);
2517
-                Database::query('UPDATE ' . $TABLECOURSUSER . ' SET sort = sort+1 WHERE user_id= "' . $user_id . '" AND user_course_cat="0"');
2517
+                Database::query('UPDATE '.$TABLECOURSUSER.' SET sort = sort+1 WHERE user_id= "'.$user_id.'" AND user_course_cat="0"');
2518 2518
             }
2519 2519
         }
2520 2520
         return $course_sort;
@@ -2527,8 +2527,8 @@  discard block
 block discarded – undo
2527 2527
      */
2528 2528
     public static function course_exists($course_code)
2529 2529
     {
2530
-        $sql = 'SELECT 1 FROM ' . Database::get_main_table(TABLE_MAIN_COURSE) . '
2531
-                WHERE code="' . Database::escape_string($course_code) . '"';
2530
+        $sql = 'SELECT 1 FROM '.Database::get_main_table(TABLE_MAIN_COURSE).'
2531
+                WHERE code="' . Database::escape_string($course_code).'"';
2532 2532
 
2533 2533
         return Database::num_rows(Database::query($sql));
2534 2534
     }
@@ -2553,8 +2553,8 @@  discard block
 block discarded – undo
2553 2553
         $student = api_get_user_info($user_id);
2554 2554
 
2555 2555
         $name_course = $information['title'];
2556
-        $sql = "SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . " 
2557
-                WHERE c_id ='" . $courseId . "'";
2556
+        $sql = "SELECT * FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)." 
2557
+                WHERE c_id ='" . $courseId."'";
2558 2558
 
2559 2559
         // TODO: Ivan: This is a mistake, please, have a look at it. Intention here is diffcult to be guessed.
2560 2560
         //if ($send_to_tutor_also = true)
@@ -2569,18 +2569,18 @@  discard block
 block discarded – undo
2569 2569
         while ($row = Database::fetch_array($result)) {
2570 2570
             $tutor = api_get_user_info($row['user_id']);
2571 2571
             $emailto = $tutor['email'];
2572
-            $emailsubject = get_lang('NewUserInTheCourse') . ': ' . $name_course;
2573
-            $emailbody = get_lang('Dear') . ': ' . api_get_person_name($tutor['firstname'], $tutor['lastname']) . "\n";
2574
-            $emailbody .= get_lang('MessageNewUserInTheCourse') . ': ' . $name_course . "\n";
2575
-            $emailbody .= get_lang('UserName') . ': ' . $student['username'] . "\n";
2572
+            $emailsubject = get_lang('NewUserInTheCourse').': '.$name_course;
2573
+            $emailbody = get_lang('Dear').': '.api_get_person_name($tutor['firstname'], $tutor['lastname'])."\n";
2574
+            $emailbody .= get_lang('MessageNewUserInTheCourse').': '.$name_course."\n";
2575
+            $emailbody .= get_lang('UserName').': '.$student['username']."\n";
2576 2576
             if (api_is_western_name_order()) {
2577
-                $emailbody .= get_lang('FirstName') . ': ' . $student['firstname'] . "\n";
2578
-                $emailbody .= get_lang('LastName') . ': ' . $student['lastname'] . "\n";
2577
+                $emailbody .= get_lang('FirstName').': '.$student['firstname']."\n";
2578
+                $emailbody .= get_lang('LastName').': '.$student['lastname']."\n";
2579 2579
             } else {
2580
-                $emailbody .= get_lang('LastName') . ': ' . $student['lastname'] . "\n";
2581
-                $emailbody .= get_lang('FirstName') . ': ' . $student['firstname'] . "\n";
2580
+                $emailbody .= get_lang('LastName').': '.$student['lastname']."\n";
2581
+                $emailbody .= get_lang('FirstName').': '.$student['firstname']."\n";
2582 2582
             }
2583
-            $emailbody .= get_lang('Email') . ': <a href="mailto:' . $student['email'] . '">' . $student['email'] ."</a>\n\n";
2583
+            $emailbody .= get_lang('Email').': <a href="mailto:'.$student['email'].'">'.$student['email']."</a>\n\n";
2584 2584
             $recipient_name = api_get_person_name($tutor['firstname'], $tutor['lastname'], null,
2585 2585
                 PERSON_NAME_EMAIL_ADDRESS);
2586 2586
             $sender_name = api_get_person_name(api_get_setting('administratorName'),
@@ -2819,7 +2819,7 @@  discard block
 block discarded – undo
2819 2819
 
2820 2820
         if ($include_sessions === true) {
2821 2821
             $sql = "SELECT DISTINCT(c.code), c.id as real_id
2822
-                    FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER) . " s,
2822
+                    FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER)." s,
2823 2823
                     $tbl_course c
2824 2824
                     WHERE user_id = $user_id AND s.c_id = c.id";
2825 2825
             $r = Database::query($sql);
@@ -2847,7 +2847,7 @@  discard block
 block discarded – undo
2847 2847
     public static function get_course_id_from_path($path)
2848 2848
     {
2849 2849
         $path = Database::escape_string(str_replace('.', '', str_replace('/', '', $path)));
2850
-        $res = Database::query("SELECT code FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . "
2850
+        $res = Database::query("SELECT code FROM ".Database::get_main_table(TABLE_MAIN_COURSE)."
2851 2851
                 WHERE directory LIKE BINARY '$path'");
2852 2852
         if ($res === false) {
2853 2853
             return false;
@@ -2869,8 +2869,8 @@  discard block
 block discarded – undo
2869 2869
     public static function get_courses_info_from_visual_code($code)
2870 2870
     {
2871 2871
         $result = array();
2872
-        $sql_result = Database::query("SELECT * FROM " . Database::get_main_table(TABLE_MAIN_COURSE) . "
2873
-                WHERE visual_code = '" . Database::escape_string($code) . "'");
2872
+        $sql_result = Database::query("SELECT * FROM ".Database::get_main_table(TABLE_MAIN_COURSE)."
2873
+                WHERE visual_code = '" . Database::escape_string($code)."'");
2874 2874
         while ($virtual_course = Database::fetch_array($sql_result)) {
2875 2875
             $result[] = $virtual_course;
2876 2876
         }
@@ -2887,10 +2887,10 @@  discard block
 block discarded – undo
2887 2887
     public static function get_emails_of_tutors_to_course($courseId)
2888 2888
     {
2889 2889
         $list = array();
2890
-        $res = Database::query("SELECT user_id FROM " . Database::get_main_table(TABLE_MAIN_COURSE_USER) . "
2891
-                WHERE c_id ='" . intval($courseId) . "' AND status=1");
2890
+        $res = Database::query("SELECT user_id FROM ".Database::get_main_table(TABLE_MAIN_COURSE_USER)."
2891
+                WHERE c_id ='" . intval($courseId)."' AND status=1");
2892 2892
         while ($list_users = Database::fetch_array($res)) {
2893
-            $result = Database::query("SELECT * FROM " . Database::get_main_table(TABLE_MAIN_USER) . "
2893
+            $result = Database::query("SELECT * FROM ".Database::get_main_table(TABLE_MAIN_USER)."
2894 2894
                     WHERE user_id=" . $list_users['user_id']);
2895 2895
             while ($row_user = Database::fetch_array($result)) {
2896 2896
                 $name_teacher = api_get_person_name($row_user['firstname'], $row_user['lastname']);
@@ -2934,7 +2934,7 @@  discard block
 block discarded – undo
2934 2934
             }
2935 2935
 
2936 2936
             $sql = "SELECT firstname, lastname, email FROM $tbl_user
2937
-                    WHERE user_id IN (" . implode(",", $user_ids) . ")";
2937
+                    WHERE user_id IN (".implode(",", $user_ids).")";
2938 2938
             $rs_user = Database::query($sql);
2939 2939
 
2940 2940
             while ($row_emails = Database::fetch_array($rs_user)) {
@@ -2983,9 +2983,9 @@  discard block
 block discarded – undo
2983 2983
      */
2984 2984
     public static function update_attribute($id, $name, $value)
2985 2985
     {
2986
-        $id = (int)$id;
2986
+        $id = (int) $id;
2987 2987
         $table = Database::get_main_table(TABLE_MAIN_COURSE);
2988
-        $sql = "UPDATE $table SET $name = '" . Database::escape_string($value) . "'
2988
+        $sql = "UPDATE $table SET $name = '".Database::escape_string($value)."'
2989 2989
                 WHERE id = '$id'";
2990 2990
 
2991 2991
         return Database::query($sql);
@@ -3001,7 +3001,7 @@  discard block
 block discarded – undo
3001 3001
      */
3002 3002
     public static function update_attributes($id, $attributes)
3003 3003
     {
3004
-        $id = (int)$id;
3004
+        $id = (int) $id;
3005 3005
         $table = Database::get_main_table(TABLE_MAIN_COURSE);
3006 3006
         $sql = "UPDATE $table SET ";
3007 3007
         $i = 0;
@@ -3010,7 +3010,7 @@  discard block
 block discarded – undo
3010 3010
                 if ($i > 0) {
3011 3011
                     $sql .= ", ";
3012 3012
                 }
3013
-                $sql .= " $name = '" . Database::escape_string($value) . "'";
3013
+                $sql .= " $name = '".Database::escape_string($value)."'";
3014 3014
                 $i++;
3015 3015
             }
3016 3016
         }
@@ -3052,10 +3052,10 @@  discard block
 block discarded – undo
3052 3052
         }
3053 3053
 
3054 3054
         $sql = 'SELECT sc.id session_category
3055
-                FROM ' . Database::get_main_table(TABLE_MAIN_SESSION_CATEGORY) . ' sc
3056
-                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_SESSION) . ' s
3055
+                FROM ' . Database::get_main_table(TABLE_MAIN_SESSION_CATEGORY).' sc
3056
+                INNER JOIN ' . Database::get_main_table(TABLE_MAIN_SESSION).' s
3057 3057
                 ON sc.id = s.session_category_id 
3058
-                WHERE s.id="' . intval($session_id) . '"';
3058
+                WHERE s.id="' . intval($session_id).'"';
3059 3059
 
3060 3060
         return Database::result(
3061 3061
             Database::query($sql),
@@ -3101,13 +3101,13 @@  discard block
 block discarded – undo
3101 3101
                 $data .= '<div class="sectiontitle">';
3102 3102
                 if (api_is_allowed_to_edit() && $action_show) {
3103 3103
                     //delete
3104
-                    $data .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&action=delete&description_id=' . $description->id . '" onclick="javascript:if(!confirm(\'' . addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),
3105
-                                ENT_QUOTES, $charset)) . '\')) return false;">';
3104
+                    $data .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&action=delete&description_id='.$description->id.'" onclick="javascript:if(!confirm(\''.addslashes(api_htmlentities(get_lang('ConfirmYourChoice'),
3105
+                                ENT_QUOTES, $charset)).'\')) return false;">';
3106 3106
                     $data .= Display::return_icon('delete.gif', get_lang('Delete'),
3107 3107
                         array('style' => 'vertical-align:middle;float:right;'));
3108 3108
                     $data .= '</a> ';
3109 3109
                     //edit
3110
-                    $data .= '<a href="' . api_get_self() . '?' . api_get_cidreq() . '&description_id=' . $description->id . '">';
3110
+                    $data .= '<a href="'.api_get_self().'?'.api_get_cidreq().'&description_id='.$description->id.'">';
3111 3111
                     $data .= Display::return_icon('edit.png', get_lang('Edit'),
3112 3112
                         array('style' => 'vertical-align:middle;float:right; padding-right:4px;'), ICON_SIZE_SMALL);
3113 3113
                     $data .= '</a> ';
@@ -3119,7 +3119,7 @@  discard block
 block discarded – undo
3119 3119
                 $data .= '</div>';
3120 3120
             }
3121 3121
         } else {
3122
-            $data .= '<em>' . get_lang('ThisCourseDescriptionIsEmpty') . '</em>';
3122
+            $data .= '<em>'.get_lang('ThisCourseDescriptionIsEmpty').'</em>';
3123 3123
         }
3124 3124
 
3125 3125
         return $data;
@@ -3178,11 +3178,11 @@  discard block
 block discarded – undo
3178 3178
                     ON (a.c_id = s.c_id)
3179 3179
                     WHERE
3180 3180
                         user_id = $hr_manager_id AND
3181
-                        relation_type=" . COURSE_RELATION_TYPE_RRHH . " AND
3182
-                        access_url_id = " . api_get_current_access_url_id() . "";
3181
+                        relation_type=".COURSE_RELATION_TYPE_RRHH." AND
3182
+                        access_url_id = " . api_get_current_access_url_id()."";
3183 3183
         } else {
3184 3184
             $sql = "SELECT c_id FROM $tbl_course_rel_user
3185
-                    WHERE user_id = $hr_manager_id AND relation_type=" . COURSE_RELATION_TYPE_RRHH . " ";
3185
+                    WHERE user_id = $hr_manager_id AND relation_type=".COURSE_RELATION_TYPE_RRHH." ";
3186 3186
         }
3187 3187
         $result = Database::query($sql);
3188 3188
         if (Database::num_rows($result) > 0) {
@@ -3191,7 +3191,7 @@  discard block
 block discarded – undo
3191 3191
                         WHERE
3192 3192
                             c_id = '{$row['c_id']}' AND
3193 3193
                             user_id = $hr_manager_id AND
3194
-                            relation_type=" . COURSE_RELATION_TYPE_RRHH . " ";
3194
+                            relation_type=".COURSE_RELATION_TYPE_RRHH." ";
3195 3195
                 Database::query($sql);
3196 3196
             }
3197 3197
         }
@@ -3202,7 +3202,7 @@  discard block
 block discarded – undo
3202 3202
                 $courseInfo = api_get_course_info($course_code);
3203 3203
                 $courseId = $courseInfo['real_id'];
3204 3204
                 $sql = "INSERT IGNORE INTO $tbl_course_rel_user(c_id, user_id, status, relation_type)
3205
-                        VALUES($courseId, $hr_manager_id, '" . DRH . "', '" . COURSE_RELATION_TYPE_RRHH . "')";
3205
+                        VALUES($courseId, $hr_manager_id, '".DRH."', '".COURSE_RELATION_TYPE_RRHH."')";
3206 3206
                 $result = Database::query($sql);
3207 3207
                 if (Database::affected_rows($result)) {
3208 3208
                     $affected_rows++;
@@ -3288,16 +3288,16 @@  discard block
 block discarded – undo
3288 3288
             case COURSEMANAGER:
3289 3289
                 $whereConditions .= " AND cru.user_id = '$user_id'";
3290 3290
                 if (!$showAllAssignedCourses) {
3291
-                    $whereConditions .= " AND cru.status = " . COURSEMANAGER;
3291
+                    $whereConditions .= " AND cru.status = ".COURSEMANAGER;
3292 3292
                 } else {
3293
-                    $whereConditions .= " AND relation_type = " . COURSE_RELATION_TYPE_COURSE_MANAGER;
3293
+                    $whereConditions .= " AND relation_type = ".COURSE_RELATION_TYPE_COURSE_MANAGER;
3294 3294
                 }
3295 3295
                 break;
3296 3296
             case DRH:
3297 3297
                 $whereConditions .= " AND
3298 3298
                     cru.user_id = '$user_id' AND
3299
-                    cru.status = " . DRH . " AND
3300
-                    relation_type = '" . COURSE_RELATION_TYPE_RRHH . "'
3299
+                    cru.status = ".DRH." AND
3300
+                    relation_type = '" . COURSE_RELATION_TYPE_RRHH."'
3301 3301
                 ";
3302 3302
                 break;
3303 3303
         }
@@ -3314,14 +3314,14 @@  discard block
 block discarded – undo
3314 3314
         if (!empty($sessionId)) {
3315 3315
             if ($status == COURSEMANAGER) {
3316 3316
                 // Teacher of course or teacher inside session
3317
-                $whereConditions = " AND (cru.status = " . COURSEMANAGER." OR srcru.status = 2) ";
3317
+                $whereConditions = " AND (cru.status = ".COURSEMANAGER." OR srcru.status = 2) ";
3318 3318
             }
3319 3319
             $courseList = SessionManager::get_course_list_by_session_id(
3320 3320
                 $sessionId
3321 3321
             );
3322 3322
             if (!empty($courseList)) {
3323 3323
                 $courseListToString = implode("','", array_keys($courseList));
3324
-                $whereConditions .= " AND c.id IN ('" . $courseListToString . "')";
3324
+                $whereConditions .= " AND c.id IN ('".$courseListToString."')";
3325 3325
             }
3326 3326
             $tableSessionRelCourse = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
3327 3327
             $tableSessionRelCourseRelUser = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
@@ -3342,7 +3342,7 @@  discard block
 block discarded – undo
3342 3342
                 ON (a.c_id = c.id)
3343 3343
                 $extraInnerJoin
3344 3344
                 WHERE
3345
-                    access_url_id = " . api_get_current_access_url_id() . "
3345
+                    access_url_id = ".api_get_current_access_url_id()."
3346 3346
                     $whereConditions
3347 3347
                 $orderBy
3348 3348
                 ";
@@ -3406,10 +3406,10 @@  discard block
 block discarded – undo
3406 3406
         }
3407 3407
 
3408 3408
         // course path
3409
-        $store_path = api_get_path(SYS_COURSE_PATH) . $courseInfo['path'];
3409
+        $store_path = api_get_path(SYS_COURSE_PATH).$courseInfo['path'];
3410 3410
         // image name for courses
3411
-        $course_image = $store_path . '/course-pic.png';
3412
-        $course_medium_image = $store_path . '/course-pic85x85.png';
3411
+        $course_image = $store_path.'/course-pic.png';
3412
+        $course_medium_image = $store_path.'/course-pic85x85.png';
3413 3413
 
3414 3414
         if (file_exists($course_image)) {
3415 3415
             unlink($course_image);
@@ -3443,11 +3443,11 @@  discard block
 block discarded – undo
3443 3443
     {
3444 3444
         $course_info = api_get_course_info($courseCode);
3445 3445
         // course path
3446
-        $storePath = api_get_path(SYS_COURSE_PATH) . $course_info['path'];
3446
+        $storePath = api_get_path(SYS_COURSE_PATH).$course_info['path'];
3447 3447
         // image name for courses
3448
-        $courseImage = $storePath . '/course-pic.png';
3449
-        $courseMediumImage = $storePath . '/course-pic85x85.png';
3450
-        $courseSmallImage = $storePath . '/course-pic32.png';
3448
+        $courseImage = $storePath.'/course-pic.png';
3449
+        $courseMediumImage = $storePath.'/course-pic85x85.png';
3450
+        $courseSmallImage = $storePath.'/course-pic32.png';
3451 3451
 
3452 3452
         if (file_exists($courseImage)) {
3453 3453
             unlink($courseImage);
@@ -3475,26 +3475,26 @@  discard block
 block discarded – undo
3475 3475
 
3476 3476
         $notifications = isset($params['notifications']) ? $params['notifications'] : null;
3477 3477
 
3478
-        $html .= '<h3>' . $params['title'] . $notifications . '</h3> ';
3478
+        $html .= '<h3>'.$params['title'].$notifications.'</h3> ';
3479 3479
 
3480 3480
         if (isset($params['description'])) {
3481
-            $html .= '<p>' . $params['description'] . '</p>';
3481
+            $html .= '<p>'.$params['description'].'</p>';
3482 3482
         }
3483 3483
         if (!empty($params['subtitle'])) {
3484
-            $html .= '<small>' . $params['subtitle'] . '</small>';
3484
+            $html .= '<small>'.$params['subtitle'].'</small>';
3485 3485
         }
3486 3486
         if (!empty($params['teachers'])) {
3487
-            $html .= '<h5 class="teacher">' . Display::return_icon('teacher.png', get_lang('Teacher'), array(),
3488
-                    ICON_SIZE_TINY) . $params['teachers'] . '</h5>';
3487
+            $html .= '<h5 class="teacher">'.Display::return_icon('teacher.png', get_lang('Teacher'), array(),
3488
+                    ICON_SIZE_TINY).$params['teachers'].'</h5>';
3489 3489
         }
3490 3490
         if (!empty($params['coaches'])) {
3491
-            $html .= '<h5 class="teacher">' . Display::return_icon('teacher.png', get_lang('Coach'), array(),
3492
-                    ICON_SIZE_TINY) . $params['coaches'] . '</h5>';
3491
+            $html .= '<h5 class="teacher">'.Display::return_icon('teacher.png', get_lang('Coach'), array(),
3492
+                    ICON_SIZE_TINY).$params['coaches'].'</h5>';
3493 3493
         }
3494 3494
 
3495 3495
         $html .= '</div>';
3496 3496
         $params['right_actions'] = isset($params['right_actions']) ? $params['right_actions'] : null;
3497
-        $html .= '<div class="pull-right course-box-actions">' . $params['right_actions'] . '</div>';
3497
+        $html .= '<div class="pull-right course-box-actions">'.$params['right_actions'].'</div>';
3498 3498
         $html .= '</div>';
3499 3499
         $html .= '</div>';
3500 3500
 
@@ -3511,11 +3511,11 @@  discard block
 block discarded – undo
3511 3511
         $html = '';
3512 3512
         $html .= '<div class="row">';
3513 3513
         $html .= '<div class="col-md-2">';
3514
-        if (!empty($params['link'])){
3514
+        if (!empty($params['link'])) {
3515 3515
             $html .= '<a class="thumbnail" href="'.$params['link'].'">';
3516 3516
             $html .= $params['icon'];
3517 3517
             $html .= '</a>';
3518
-        }else{
3518
+        } else {
3519 3519
             $html .= $params['icon'];
3520 3520
         }
3521 3521
         $html .= '</div>';
@@ -3588,27 +3588,27 @@  discard block
 block discarded – undo
3588 3588
                 $params['edit_actions'] = '';
3589 3589
                 $params['document'] = '';
3590 3590
                 if (api_is_platform_admin()) {
3591
-                    $params['edit_actions'] .= api_get_path(WEB_CODE_PATH) . 'course_info/infocours.php?cidReq=' . $course['code'];
3591
+                    $params['edit_actions'] .= api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'];
3592 3592
                     if ($load_dirs) {
3593
-                        $params['document'] = '<a id="document_preview_' . $courseId . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3594
-                           . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3595
-                        $params['document'] .= Display::div('', ['id' => 'document_result_' . $courseId . '_0', 'class' => 'document_preview_container']);
3593
+                        $params['document'] = '<a id="document_preview_'.$courseId.'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3594
+                           . Display::returnFontAwesomeIcon('folder-open').'</a>';
3595
+                        $params['document'] .= Display::div('', ['id' => 'document_result_'.$courseId.'_0', 'class' => 'document_preview_container']);
3596 3596
                     }
3597 3597
                 } else {
3598 3598
                     if ($course_info['visibility'] != COURSE_VISIBILITY_CLOSED && $load_dirs) {
3599
-                        $params['document'] = '<a id="document_preview_' . $courseId . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3600
-                           . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3601
-                        $params['document'] .= Display::div('', ['id' => 'document_result_' . $courseId . '_0', 'class' => 'document_preview_container']);
3599
+                        $params['document'] = '<a id="document_preview_'.$courseId.'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3600
+                           . Display::returnFontAwesomeIcon('folder-open').'</a>';
3601
+                        $params['document'] .= Display::div('', ['id' => 'document_result_'.$courseId.'_0', 'class' => 'document_preview_container']);
3602 3602
                     }
3603 3603
                 }
3604 3604
 
3605 3605
                 $params['visibility'] = $course_info['visibility'];
3606 3606
                 $params['status'] = $course_info['status'];
3607 3607
                 $params['category'] = $course_info['categoryName'];
3608
-                $params['icon'] = Display::return_icon('drawing-pin.png',null, null, ICON_SIZE_LARGE, null);
3608
+                $params['icon'] = Display::return_icon('drawing-pin.png', null, null, ICON_SIZE_LARGE, null);
3609 3609
 
3610 3610
                 if (api_get_setting('display_coursecode_in_courselist') == 'true') {
3611
-                    $params['code_course']  = '(' . $course_info['visual_code'] . ')';
3611
+                    $params['code_course'] = '('.$course_info['visual_code'].')';
3612 3612
                 }
3613 3613
 
3614 3614
                 $params['title'] = $course_info['title'];
@@ -3653,7 +3653,7 @@  discard block
 block discarded – undo
3653 3653
         // Step 1: We get all the categories of the user
3654 3654
         $table = Database::get_main_table(TABLE_USER_COURSE_CATEGORY);
3655 3655
         $sql = "SELECT id, title FROM $table
3656
-                WHERE user_id = '" . $user_id . "'
3656
+                WHERE user_id = '".$user_id."'
3657 3657
                 ORDER BY sort ASC";
3658 3658
 
3659 3659
         $result = Database::query($sql);
@@ -3708,7 +3708,7 @@  discard block
 block discarded – undo
3708 3708
         $special_course_list = self::get_special_course_list();
3709 3709
         $without_special_courses = '';
3710 3710
         if (!empty($special_course_list)) {
3711
-            $without_special_courses = ' AND course.id NOT IN ("' . implode('","', $special_course_list) . '")';
3711
+            $without_special_courses = ' AND course.id NOT IN ("'.implode('","', $special_course_list).'")';
3712 3712
         }
3713 3713
 
3714 3714
         $userCategoryCondition = " (course_rel_user.user_course_cat = $user_category_id) ";
@@ -3726,14 +3726,14 @@  discard block
 block discarded – undo
3726 3726
                 INNER JOIN $TABLE_ACCESS_URL_REL_COURSE url
3727 3727
                 ON (url.c_id = course.id)
3728 3728
                 WHERE
3729
-                    course_rel_user.user_id = '" . $user_id . "' AND
3729
+                    course_rel_user.user_id = '".$user_id."' AND
3730 3730
                     $userCategoryCondition
3731 3731
                     $without_special_courses ";
3732 3732
 
3733 3733
         // If multiple URL access mode is enabled, only fetch courses
3734 3734
         // corresponding to the current URL.
3735 3735
         if (api_get_multiple_access_url() && $current_url_id != -1) {
3736
-            $sql .= " AND access_url_id='" . $current_url_id . "'";
3736
+            $sql .= " AND access_url_id='".$current_url_id."'";
3737 3737
         }
3738 3738
         // Use user's classification for courses (if any).
3739 3739
         $sql .= " ORDER BY course_rel_user.user_course_cat, course_rel_user.sort ASC";
@@ -3779,28 +3779,28 @@  discard block
 block discarded – undo
3779 3779
             if ($showCustomIcon === 'true' && $iconName != 'course.png') {
3780 3780
                 $thumbnails = $course_info['course_image'];
3781 3781
                 $image = $course_info['course_image_large'];
3782
-            }else{
3783
-                $image = Display::return_icon('session_default.png', null, null, null,null, true);
3782
+            } else {
3783
+                $image = Display::return_icon('session_default.png', null, null, null, null, true);
3784 3784
             }
3785 3785
 
3786 3786
             $params['course_id'] = $course_info['real_id'];
3787 3787
             $params['edit_actions'] = '';
3788 3788
             $params['document'] = '';
3789 3789
             if (api_is_platform_admin()) {
3790
-                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH) . 'course_info/infocours.php?cidReq=' . $course_info['code'];
3790
+                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course_info['code'];
3791 3791
                 if ($load_dirs) {
3792
-                    $params['document'] = '<a id="document_preview_' . $course_info['real_id'] . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3793
-                               . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3794
-                    $params['document'] .= Display::div('', array('id' => 'document_result_' . $course_info['real_id'] . '_0', 'class' => 'document_preview_container'));
3792
+                    $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3793
+                               . Display::returnFontAwesomeIcon('folder-open').'</a>';
3794
+                    $params['document'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container'));
3795 3795
                 }
3796 3796
             }
3797 3797
             if ($load_dirs) {
3798
-                $params['document'] = '<a id="document_preview_' . $course_info['real_id'] . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3799
-                    . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3800
-                $params['document'] .= Display::div('', array('id' => 'document_result_' . $course_info['real_id'] . '_0', 'class' => 'document_preview_container'));
3798
+                $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3799
+                    . Display::returnFontAwesomeIcon('folder-open').'</a>';
3800
+                $params['document'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container'));
3801 3801
             }
3802 3802
 
3803
-            $courseUrl = api_get_path(WEB_COURSE_PATH) . $course_info['path'] . '/index.php?id_session=0';
3803
+            $courseUrl = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/index.php?id_session=0';
3804 3804
             $teachers = [];
3805 3805
             if (api_get_setting('display_teacher_in_courselist') === 'true') {
3806 3806
                 $teachers = self::getTeachersFromCourse($course_info['real_id'], false);
@@ -3808,7 +3808,7 @@  discard block
 block discarded – undo
3808 3808
 
3809 3809
             $params['status'] = $row['status'];
3810 3810
             if (api_get_setting('display_coursecode_in_courselist') == 'true') {
3811
-                $params['code_course'] = '(' . $course_info['visual_code'] . ') ';
3811
+                $params['code_course'] = '('.$course_info['visual_code'].') ';
3812 3812
             }
3813 3813
 
3814 3814
             $params['current_user_is_teacher'] = false;
@@ -3864,7 +3864,7 @@  discard block
 block discarded – undo
3864 3864
 
3865 3865
         $without_special_courses = '';
3866 3866
         if (!empty($special_course_list)) {
3867
-            $without_special_courses = ' AND course.id NOT IN ("' . implode('","', $special_course_list) . '")';
3867
+            $without_special_courses = ' AND course.id NOT IN ("'.implode('","', $special_course_list).'")';
3868 3868
         }
3869 3869
 
3870 3870
         $sql = "SELECT
@@ -3882,14 +3882,14 @@  discard block
 block discarded – undo
3882 3882
                 INNER JOIN $TABLE_ACCESS_URL_REL_COURSE url
3883 3883
                 ON (url.c_id = course.id)
3884 3884
                 WHERE
3885
-                    course_rel_user.user_id = '" . $user_id . "' AND
3886
-                    course_rel_user.user_course_cat = '" . $user_category_id . "'
3885
+                    course_rel_user.user_id = '".$user_id."' AND
3886
+                    course_rel_user.user_course_cat = '" . $user_category_id."'
3887 3887
                     $without_special_courses ";
3888 3888
 
3889 3889
         // If multiple URL access mode is enabled, only fetch courses
3890 3890
         // corresponding to the current URL.
3891 3891
         if (api_get_multiple_access_url() && $current_url_id != -1) {
3892
-            $sql .= " AND url.c_id = course.id AND access_url_id='" . $current_url_id . "'";
3892
+            $sql .= " AND url.c_id = course.id AND access_url_id='".$current_url_id."'";
3893 3893
         }
3894 3894
         // Use user's classification for courses (if any).
3895 3895
         $sql .= " ORDER BY course_rel_user.user_course_cat, course_rel_user.sort ASC";
@@ -3920,8 +3920,8 @@  discard block
 block discarded – undo
3920 3920
             if ($showCustomIcon === 'true' && $iconName != 'course.png') {
3921 3921
                 $thumbnails = $course_info['course_image'];
3922 3922
                 $image = $course_info['course_image_large'];
3923
-            }else{
3924
-                $image = Display::return_icon('session_default.png', null, null, null,null, true);
3923
+            } else {
3924
+                $image = Display::return_icon('session_default.png', null, null, null, null, true);
3925 3925
             }
3926 3926
 
3927 3927
             $params = array();
@@ -3930,20 +3930,20 @@  discard block
 block discarded – undo
3930 3930
             $params['edit_actions'] = '';
3931 3931
             $params['document'] = '';
3932 3932
             if (api_is_platform_admin()) {
3933
-                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH) . 'course_info/infocours.php?cidReq=' . $course_info['code'];
3934
-                if($load_dirs){
3935
-                    $params['document'] = '<a id="document_preview_' . $course_info['real_id'] . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3936
-                               . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3937
-                    $params['document'] .= Display::div('', array('id' => 'document_result_' . $course_info['real_id'] . '_0', 'class' => 'document_preview_container'));
3933
+                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course_info['code'];
3934
+                if ($load_dirs) {
3935
+                    $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3936
+                               . Display::returnFontAwesomeIcon('folder-open').'</a>';
3937
+                    $params['document'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container'));
3938 3938
                 }
3939 3939
             }
3940 3940
             if ($load_dirs) {
3941
-                $params['document'] = '<a id="document_preview_' . $course_info['real_id'] . '_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3942
-                    . Display::returnFontAwesomeIcon('folder-open') . '</a>';
3943
-                $params['document'] .= Display::div('', array('id' => 'document_result_' . $course_info['real_id'] . '_0', 'class' => 'document_preview_container'));
3941
+                $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
3942
+                    . Display::returnFontAwesomeIcon('folder-open').'</a>';
3943
+                $params['document'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container'));
3944 3944
             }
3945 3945
 
3946
-            $course_title_url = api_get_path(WEB_COURSE_PATH) . $course_info['path'] . '/index.php?id_session=0';
3946
+            $course_title_url = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/index.php?id_session=0';
3947 3947
 
3948 3948
             $teachers = [];
3949 3949
             if (api_get_setting('display_teacher_in_courselist') === 'true') {
@@ -3952,7 +3952,7 @@  discard block
 block discarded – undo
3952 3952
             $params['status'] = $row['status'];
3953 3953
 
3954 3954
             if (api_get_setting('display_coursecode_in_courselist') == 'true') {
3955
-                $params['code_course'] = '(' . $course_info['visual_code'] . ') ';
3955
+                $params['code_course'] = '('.$course_info['visual_code'].') ';
3956 3956
             }
3957 3957
 
3958 3958
             $params['visibility'] = $course_info['visibility'];
@@ -4138,15 +4138,15 @@  discard block
 block discarded – undo
4138 4138
                 }
4139 4139
 
4140 4140
                 if ($userInCourseStatus == COURSEMANAGER || $sessionCourseAvailable) {
4141
-                    $session_url = $course_info['course_public_url'] . '?id_session=' . $course_info['id_session'];
4142
-                    $session_title = '<a href="' . $session_url. '">'. $course_info['name'] . '</a>'.$notifications;
4141
+                    $session_url = $course_info['course_public_url'].'?id_session='.$course_info['id_session'];
4142
+                    $session_title = '<a href="'.$session_url.'">'.$course_info['name'].'</a>'.$notifications;
4143 4143
                 } else {
4144 4144
                     $session_title = $course_info['name'];
4145 4145
                 }
4146 4146
 
4147 4147
             } else {
4148 4148
                 $session_title =
4149
-                    $course_info['name'] . ' ' .
4149
+                    $course_info['name'].' '.
4150 4150
                     Display::tag('span', get_lang('CourseClosed'), array('class' => 'item_closed'));
4151 4151
             }
4152 4152
         } else {
@@ -4162,7 +4162,7 @@  discard block
 block discarded – undo
4162 4162
             $thumbnails = $course_info['course_image'];
4163 4163
             $image = $course_info['course_image_large'];
4164 4164
         } else {
4165
-            $image = Display::return_icon('session_default.png', null, null, null,null, true);
4165
+            $image = Display::return_icon('session_default.png', null, null, null, null, true);
4166 4166
         }
4167 4167
         $params['thumbnails'] = $thumbnails;
4168 4168
         $params['image'] = $image;
@@ -4176,12 +4176,12 @@  discard block
 block discarded – undo
4176 4176
             $course_visibility != COURSE_VISIBILITY_HIDDEN
4177 4177
         ) {
4178 4178
             if (api_is_platform_admin()) {
4179
-                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH) . 'course_info/infocours.php?cidReq=' . $course_info['code'];
4179
+                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course_info['code'];
4180 4180
                 if ($load_dirs) {
4181
-                    $params['document'] .= '<a id="document_preview_' . $course_info['real_id'] . '_' . $course_info['id_session'] . '" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">' .
4182
-                        Display::returnFontAwesomeIcon('folder-open') . '</a>';
4181
+                    $params['document'] .= '<a id="document_preview_'.$course_info['real_id'].'_'.$course_info['id_session'].'" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'.
4182
+                        Display::returnFontAwesomeIcon('folder-open').'</a>';
4183 4183
                     $params['document'] .= Display::div('', array(
4184
-                        'id' => 'document_result_' . $course_info['real_id'] . '_' . $course_info['id_session'],
4184
+                        'id' => 'document_result_'.$course_info['real_id'].'_'.$course_info['id_session'],
4185 4185
                         'class' => 'document_preview_container'
4186 4186
                     ));
4187 4187
                 }
@@ -4189,7 +4189,7 @@  discard block
 block discarded – undo
4189 4189
         }
4190 4190
 
4191 4191
         if (api_get_setting('display_coursecode_in_courselist') === 'true') {
4192
-            $session_title .= ' (' . $course_info['visual_code'] . ') ';
4192
+            $session_title .= ' ('.$course_info['visual_code'].') ';
4193 4193
         }
4194 4194
 
4195 4195
         if (api_get_setting('display_teacher_in_courselist') === 'true') {
@@ -4211,7 +4211,7 @@  discard block
 block discarded – undo
4211 4211
             }
4212 4212
         }
4213 4213
 
4214
-        $session_title .= isset($course['special_course']) ? ' ' .
4214
+        $session_title .= isset($course['special_course']) ? ' '.
4215 4215
                           Display::return_icon('klipper.png', get_lang('CourseAutoRegister')) : '';
4216 4216
 
4217 4217
         $params['title'] = $session_title;
@@ -4238,13 +4238,13 @@  discard block
 block discarded – undo
4238 4238
                 ) {
4239 4239
                     $session['dates'] = '';
4240 4240
                     if (api_get_setting('show_session_coach') === 'true') {
4241
-                        $session['coach'] = get_lang('GeneralCoach') . ': ' . $sessionCoachName;
4241
+                        $session['coach'] = get_lang('GeneralCoach').': '.$sessionCoachName;
4242 4242
                     }
4243 4243
                     $active = true;
4244 4244
                 } else {
4245
-                    $session ['dates'] = ' - ' . get_lang('From') . ' ' . $session['access_start_date'] . ' ' . get_lang('To') . ' ' . $session['access_end_date'];
4245
+                    $session ['dates'] = ' - '.get_lang('From').' '.$session['access_start_date'].' '.get_lang('To').' '.$session['access_end_date'];
4246 4246
                     if (api_get_setting('show_session_coach') === 'true') {
4247
-                        $session['coach'] = get_lang('GeneralCoach') . ': ' . $sessionCoachName;
4247
+                        $session['coach'] = get_lang('GeneralCoach').': '.$sessionCoachName;
4248 4248
                     }
4249 4249
                     $date_start = $session['access_start_date'];
4250 4250
                     $date_end = $session['access_end_date'];
@@ -4377,7 +4377,7 @@  discard block
 block discarded – undo
4377 4377
             if (Database::num_rows($result) > 0) {
4378 4378
                 $row = Database::fetch_array($result);
4379 4379
                 $count = $row['count'] + 1;
4380
-                $wanted_code = $wanted_code . '_' . $count;
4380
+                $wanted_code = $wanted_code.'_'.$count;
4381 4381
                 $result = api_get_course_info($wanted_code);
4382 4382
                 if (empty($result)) {
4383 4383
                     return $wanted_code;
@@ -4411,7 +4411,7 @@  discard block
 block discarded – undo
4411 4411
         $enabled = api_get_plugin_setting('courselegal', 'tool_enable');
4412 4412
 
4413 4413
         if ($enabled == 'true') {
4414
-            require_once api_get_path(SYS_PLUGIN_PATH) . 'courselegal/config.php';
4414
+            require_once api_get_path(SYS_PLUGIN_PATH).'courselegal/config.php';
4415 4415
             $plugin = CourseLegalPlugin::create();
4416 4416
             return $plugin->isUserAcceptedLegal($user_id, $course_code, $session_id);
4417 4417
         }
@@ -4456,7 +4456,7 @@  discard block
 block discarded – undo
4456 4456
         $enabled = api_get_plugin_setting('courselegal', 'tool_enable');
4457 4457
 
4458 4458
         if ($enabled == 'true') {
4459
-            require_once api_get_path(SYS_PLUGIN_PATH) . 'courselegal/config.php';
4459
+            require_once api_get_path(SYS_PLUGIN_PATH).'courselegal/config.php';
4460 4460
             $plugin = CourseLegalPlugin::create();
4461 4461
             return $plugin->saveUserLegal($user_id, $course_code, $session_id);
4462 4462
         }
@@ -4798,10 +4798,10 @@  discard block
 block discarded – undo
4798 4798
                 INNER JOIN $table_course_url u
4799 4799
                 ON u.c_id = c.id
4800 4800
                 WHERE
4801
-                    u.access_url_id = " . api_get_current_access_url_id() . " AND
4801
+                    u.access_url_id = ".api_get_current_access_url_id()." AND
4802 4802
                     login_course_date <= '$now' AND
4803 4803
                     login_course_date > DATE_SUB('$now', INTERVAL $days DAY) AND
4804
-                    visibility <> '" . COURSE_VISIBILITY_CLOSED . "' AND visibility <> '" . COURSE_VISIBILITY_HIDDEN . "'
4804
+                    visibility <> '".COURSE_VISIBILITY_CLOSED."' AND visibility <> '".COURSE_VISIBILITY_HIDDEN."'
4805 4805
                 GROUP BY a.c_id
4806 4806
                 ORDER BY course_count DESC
4807 4807
                 LIMIT $limit
@@ -4826,7 +4826,7 @@  discard block
 block discarded – undo
4826 4826
     public static function process_hot_course_item($courses, $my_course_code_list = array())
4827 4827
     {
4828 4828
         $hotCourses = [];
4829
-        $ajax_url = api_get_path(WEB_AJAX_PATH) . 'course.ajax.php?a=add_course_vote';
4829
+        $ajax_url = api_get_path(WEB_AJAX_PATH).'course.ajax.php?a=add_course_vote';
4830 4830
         $stok = Security::get_existing_token();
4831 4831
         $user_id = api_get_user_id();
4832 4832
 
@@ -4884,10 +4884,10 @@  discard block
 block discarded – undo
4884 4884
             //Course visibility
4885 4885
             if ($access_link && in_array('register', $access_link)) {
4886 4886
                 $my_course['register_button'] = Display::url(
4887
-                    get_lang('Subscribe') . ' ' .
4887
+                    get_lang('Subscribe').' '.
4888 4888
                     Display::returnFontAwesomeIcon('sign-in'),
4889
-                    api_get_path(WEB_COURSE_PATH) . $course_info['path'] .
4890
-                     '/index.php?action=subscribe&sec_token=' . $stok,
4889
+                    api_get_path(WEB_COURSE_PATH).$course_info['path'].
4890
+                     '/index.php?action=subscribe&sec_token='.$stok,
4891 4891
                     array(
4892 4892
                         'class' => 'btn btn-success btn-sm',
4893 4893
                         'title' => get_lang('Subscribe'),
@@ -4901,9 +4901,9 @@  discard block
 block discarded – undo
4901 4901
                     $access_link) || $course_info['visibility'] == COURSE_VISIBILITY_OPEN_WORLD
4902 4902
             ) {
4903 4903
                 $my_course['go_to_course_button'] = Display::url(
4904
-                    get_lang('GoToCourse'). ' ' .
4904
+                    get_lang('GoToCourse').' '.
4905 4905
                     Display::returnFontAwesomeIcon('share'),
4906
-                    api_get_path(WEB_COURSE_PATH) . $course_info['path'] . '/index.php',
4906
+                    api_get_path(WEB_COURSE_PATH).$course_info['path'].'/index.php',
4907 4907
                     array(
4908 4908
                         'class' => 'btn btn-default btn-sm',
4909 4909
                         'title' => get_lang('GoToCourse'),
@@ -4915,10 +4915,10 @@  discard block
 block discarded – undo
4915 4915
 
4916 4916
             if ($access_link && in_array('unsubscribe', $access_link)) {
4917 4917
                 $my_course['unsubscribe_button'] = Display::url(
4918
-                    get_lang('Unreg') . ' ' .
4918
+                    get_lang('Unreg').' '.
4919 4919
                     Display::returnFontAwesomeIcon('sign-out'),
4920
-                    api_get_path(WEB_CODE_PATH) . 'auth/courses.php?action=unsubscribe&unsubscribe=' . $courseCode
4921
-                    . '&sec_token=' . $stok . '&category_code=' . $categoryCode,
4920
+                    api_get_path(WEB_CODE_PATH).'auth/courses.php?action=unsubscribe&unsubscribe='.$courseCode
4921
+                    . '&sec_token='.$stok.'&category_code='.$categoryCode,
4922 4922
                     array(
4923 4923
                         'class' => 'btn btn-danger btn-sm',
4924 4924
                         'title' => get_lang('Unreg'),
@@ -4952,7 +4952,7 @@  discard block
 block discarded – undo
4952 4952
             ) { */
4953 4953
                 $my_course['description_button'] = Display::url(
4954 4954
                     Display::returnFontAwesomeIcon('info-circle'),
4955
-                    api_get_path(WEB_AJAX_PATH) . 'course_home.ajax.php?a=show_course_information&code=' . $course_info['code'],
4955
+                    api_get_path(WEB_AJAX_PATH).'course_home.ajax.php?a=show_course_information&code='.$course_info['code'],
4956 4956
                     [
4957 4957
                         'class' => 'btn btn-default btn-sm ajax',
4958 4958
                         'data-title' => get_lang('Description'),
@@ -5049,9 +5049,9 @@  discard block
 block discarded – undo
5049 5049
                     WHERE
5050 5050
                         c.id = u.c_id AND
5051 5051
                         u.access_url_id = $urlId AND
5052
-                        visibility <> " . COURSE_VISIBILITY_HIDDEN;
5052
+                        visibility <> ".COURSE_VISIBILITY_HIDDEN;
5053 5053
         } else {
5054
-            $sql .= " WHERE visibility <> " . COURSE_VISIBILITY_HIDDEN;
5054
+            $sql .= " WHERE visibility <> ".COURSE_VISIBILITY_HIDDEN;
5055 5055
         }
5056 5056
         $res = Database::query($sql);
5057 5057
         $row = Database::fetch_row($res);
@@ -5072,7 +5072,7 @@  discard block
 block discarded – undo
5072 5072
             $visibilityCondition .= ' AND '.$courseTableAlias.'.visibility <> '.COURSE_VISIBILITY_REGISTERED;
5073 5073
         }
5074 5074
         if ($hideClosed) {
5075
-            $visibilityCondition .= ' AND ' . $courseTableAlias . '.visibility NOT IN (' . COURSE_VISIBILITY_CLOSED .','. COURSE_VISIBILITY_HIDDEN .')';
5075
+            $visibilityCondition .= ' AND '.$courseTableAlias.'.visibility NOT IN ('.COURSE_VISIBILITY_CLOSED.','.COURSE_VISIBILITY_HIDDEN.')';
5076 5076
         }
5077 5077
 
5078 5078
         // Check if course have users allowed to see it in the catalogue, then show only if current user is allowed to see it
@@ -5080,16 +5080,16 @@  discard block
 block discarded – undo
5080 5080
         $restrictedCourses = self::getCatalogueCourseList(true);
5081 5081
         $allowedCoursesToCurrentUser = self::getCatalogueCourseList(true, $currentUserId);
5082 5082
         if (!empty($restrictedCourses)) {
5083
-            $visibilityCondition .= ' AND ('.$courseTableAlias.'.code NOT IN ("' . implode('","', $restrictedCourses) . '")';
5084
-            $visibilityCondition .= ' OR '.$courseTableAlias.'.code IN ("' . implode('","', $allowedCoursesToCurrentUser) . '"))';
5083
+            $visibilityCondition .= ' AND ('.$courseTableAlias.'.code NOT IN ("'.implode('","', $restrictedCourses).'")';
5084
+            $visibilityCondition .= ' OR '.$courseTableAlias.'.code IN ("'.implode('","', $allowedCoursesToCurrentUser).'"))';
5085 5085
         }
5086 5086
 
5087 5087
         // Check if course have users denied to see it in the catalogue, then show only if current user is not denied to see it
5088 5088
         $restrictedCourses = self::getCatalogueCourseList(false);
5089 5089
         $notAllowedCoursesToCurrentUser = self::getCatalogueCourseList(false, $currentUserId);
5090 5090
         if (!empty($restrictedCourses)) {
5091
-            $visibilityCondition .= ' AND ('.$courseTableAlias.'.code NOT IN ("' . implode('","', $restrictedCourses) . '")';
5092
-            $visibilityCondition .= ' OR '.$courseTableAlias.'.code NOT IN ("' . implode('","', $notAllowedCoursesToCurrentUser) . '"))';
5091
+            $visibilityCondition .= ' AND ('.$courseTableAlias.'.code NOT IN ("'.implode('","', $restrictedCourses).'")';
5092
+            $visibilityCondition .= ' OR '.$courseTableAlias.'.code NOT IN ("'.implode('","', $notAllowedCoursesToCurrentUser).'"))';
5093 5093
         }
5094 5094
 
5095 5095
         return $visibilityCondition;
@@ -5109,7 +5109,7 @@  discard block
 block discarded – undo
5109 5109
 
5110 5110
         $withoutSpecialCourses = '';
5111 5111
         if (!empty($specialCourseList)) {
5112
-            $withoutSpecialCourses = ' AND c.id NOT IN ("' . implode('","', $specialCourseList) . '")';
5112
+            $withoutSpecialCourses = ' AND c.id NOT IN ("'.implode('","', $specialCourseList).'")';
5113 5113
         }
5114 5114
 
5115 5115
         $visibilityCondition = self::getCourseVisibilitySQLCondition('c', true);
@@ -5243,13 +5243,13 @@  discard block
 block discarded – undo
5243 5243
             if (count($teachers) > 0) {
5244 5244
                 foreach ($teachers as $key) {
5245 5245
                     $key = Database::escape_string($key);
5246
-                    $cond .= " AND user_id <> '" . $key . "'";
5246
+                    $cond .= " AND user_id <> '".$key."'";
5247 5247
                 }
5248 5248
             }
5249 5249
 
5250 5250
             // Recover user categories
5251
-            $sql = 'SELECT * FROM ' . $course_user_table . '
5252
-                    WHERE c_id ="' . $courseId . '" AND status="1" AND relation_type = 0 ' . $cond;
5251
+            $sql = 'SELECT * FROM '.$course_user_table.'
5252
+                    WHERE c_id ="' . $courseId.'" AND status="1" AND relation_type = 0 '.$cond;
5253 5253
             $result = Database::query($sql);
5254 5254
             if (Database::num_rows($result)) {
5255 5255
                 $teachersToDelete = Database::store_result($result, 'ASSOC');
@@ -5259,8 +5259,8 @@  discard block
 block discarded – undo
5259 5259
                 }
5260 5260
             }
5261 5261
 
5262
-            $sql = 'DELETE FROM ' . $course_user_table . '
5263
-                    WHERE c_id ="' . $courseId . '" AND status="1" AND relation_type = 0 ' . $cond;
5262
+            $sql = 'DELETE FROM '.$course_user_table.'
5263
+                    WHERE c_id ="' . $courseId.'" AND status="1" AND relation_type = 0 '.$cond;
5264 5264
 
5265 5265
             Database::query($sql);
5266 5266
         }
@@ -5269,13 +5269,13 @@  discard block
 block discarded – undo
5269 5269
             foreach ($teachers as $userId) {
5270 5270
                 $userId = intval($userId);
5271 5271
                 // We check if the teacher is already subscribed in this course
5272
-                $sql = 'SELECT 1 FROM ' . $course_user_table . '
5273
-                        WHERE user_id = "' . $userId . '" AND c_id = "' . $courseId . '" ';
5272
+                $sql = 'SELECT 1 FROM '.$course_user_table.'
5273
+                        WHERE user_id = "' . $userId.'" AND c_id = "'.$courseId.'" ';
5274 5274
                 $result = Database::query($sql);
5275 5275
                 if (Database::num_rows($result)) {
5276
-                    $sql = 'UPDATE ' . $course_user_table . ' 
5276
+                    $sql = 'UPDATE '.$course_user_table.' 
5277 5277
                             SET status = "1"
5278
-                            WHERE c_id = "' . $courseId . '" AND user_id = "' . $userId . '"  ';
5278
+                            WHERE c_id = "' . $courseId.'" AND user_id = "'.$userId.'"  ';
5279 5279
                 } else {
5280 5280
                     $userCourseCategory = '0';
5281 5281
                     if (isset($teacherBackup[$userId]) &&
@@ -5654,8 +5654,8 @@  discard block
 block discarded – undo
5654 5654
     {
5655 5655
         $tbl_main_categories = Database::get_main_table(TABLE_MAIN_COURSE);
5656 5656
         $sql = 'SELECT title
5657
-                FROM ' . $tbl_main_categories . '
5658
-                WHERE code = "' . Database::escape_string($code) . '"';
5657
+                FROM ' . $tbl_main_categories.'
5658
+                WHERE code = "' . Database::escape_string($code).'"';
5659 5659
         $result = Database::query($sql);
5660 5660
         if ($col = Database::fetch_array($result)) {
5661 5661
             return $col['title'];
@@ -5836,7 +5836,7 @@  discard block
 block discarded – undo
5836 5836
             foreach ($group_list as $this_group) {
5837 5837
                 if (is_array($to_already_selected)) {
5838 5838
                     if (!in_array(
5839
-                        "GROUP:" . $this_group['id'],
5839
+                        "GROUP:".$this_group['id'],
5840 5840
                         $to_already_selected
5841 5841
                     )
5842 5842
                     ) { // $to_already_selected is the array containing the groups (and users) that are already selected
@@ -5844,8 +5844,8 @@  discard block
 block discarded – undo
5844 5844
                         $user_disabled = ($this_group['userNb'] > 0) ? "" : "disabled=disabled";
5845 5845
                         $result[] = array(
5846 5846
                             'disabled' => $user_disabled,
5847
-                            'value' => "GROUP:" . $this_group['id'],
5848
-                            'content' => "G: " . $this_group['name'] . " - " . $this_group['userNb'] . " " . $user_label
5847
+                            'value' => "GROUP:".$this_group['id'],
5848
+                            'content' => "G: ".$this_group['name']." - ".$this_group['userNb']." ".$user_label
5849 5849
                         );
5850 5850
                     }
5851 5851
                 }
@@ -5857,13 +5857,13 @@  discard block
 block discarded – undo
5857 5857
             foreach ($user_list as $user) {
5858 5858
                 if (is_array($to_already_selected)) {
5859 5859
                     if (!in_array(
5860
-                        "USER:" . $user['user_id'],
5860
+                        "USER:".$user['user_id'],
5861 5861
                         $to_already_selected
5862 5862
                     )
5863 5863
                     ) { // $to_already_selected is the array containing the users (and groups) that are already selected
5864 5864
 
5865 5865
                         $result[] = array(
5866
-                            'value' => "USER:" . $user['user_id'],
5866
+                            'value' => "USER:".$user['user_id'],
5867 5867
                             'content' => api_get_person_name($user['firstname'], $user['lastname'])
5868 5868
                         );
5869 5869
                     }
@@ -5892,7 +5892,7 @@  discard block
 block discarded – undo
5892 5892
     {
5893 5893
         $category_id = intval($category_id);
5894 5894
         $info = Database::fetch_array(
5895
-            Database::query('SELECT c_id FROM ' . Database::get_main_table(TABLE_MAIN_GRADEBOOK_CATEGORY) . '
5895
+            Database::query('SELECT c_id FROM '.Database::get_main_table(TABLE_MAIN_GRADEBOOK_CATEGORY).'
5896 5896
             WHERE id=' . $category_id), 'ASSOC'
5897 5897
         );
5898 5898
         return $info ? $info['c_id'] : false;
@@ -5908,15 +5908,14 @@  discard block
 block discarded – undo
5908 5908
     public static function getCoursesWithoutSession($startDate = null, $endDate = null, $includeClosed = false)
5909 5909
     {
5910 5910
         $dateConditional = ($startDate && $endDate) ?
5911
-            " WHERE session_id IN (SELECT id FROM " . Database::get_main_table(TABLE_MAIN_SESSION) .
5912
-            " WHERE access_start_date = '$startDate' AND access_end_date = '$endDate')" :
5913
-            null;
5911
+            " WHERE session_id IN (SELECT id FROM ".Database::get_main_table(TABLE_MAIN_SESSION).
5912
+            " WHERE access_start_date = '$startDate' AND access_end_date = '$endDate')" : null;
5914 5913
         $visibility = ($includeClosed ? '' : 'visibility NOT IN (0, 4) AND ');
5915 5914
 
5916 5915
         $query = "SELECT id, code, title
5917
-                FROM " . Database::get_main_table(TABLE_MAIN_COURSE). "
5916
+                FROM " . Database::get_main_table(TABLE_MAIN_COURSE)."
5918 5917
                 WHERE $visibility code NOT IN (
5919
-                    SELECT DISTINCT course_code FROM " . Database::get_main_table(TABLE_MAIN_SESSION_COURSE) . $dateConditional . ")
5918
+                    SELECT DISTINCT course_code FROM ".Database::get_main_table(TABLE_MAIN_SESSION_COURSE).$dateConditional.")
5920 5919
                 ORDER BY id";
5921 5920
 
5922 5921
         $result = Database::query($query);
@@ -5949,7 +5948,7 @@  discard block
 block discarded – undo
5949 5948
                 FROM $courseTable c
5950 5949
                 INNER JOIN $courseUserTable cru ON c.id = cru.c_id
5951 5950
                 WHERE (
5952
-                    cru.user_id IN (" . implode(', ', $userIdList) . ")
5951
+                    cru.user_id IN (".implode(', ', $userIdList).")
5953 5952
                     AND cru.relation_type = 0
5954 5953
                 )";
5955 5954
 
@@ -5964,7 +5963,7 @@  discard block
 block discarded – undo
5964 5963
                         INNER JOIN $courseAccessUrlTable crau ON c.id = crau.c_id
5965 5964
                         WHERE crau.access_url_id = $accessUrlId
5966 5965
                             AND (
5967
-                            cru.id_user IN (" . implode(', ', $userIdList) . ") AND
5966
+                            cru.id_user IN (".implode(', ', $userIdList).") AND
5968 5967
                             cru.relation_type = 0
5969 5968
                         )";
5970 5969
             }
@@ -6017,7 +6016,7 @@  discard block
 block discarded – undo
6017 6016
                         $result = $objExercise->read($exercise_redirect);
6018 6017
 
6019 6018
                         if (!empty($exercise_redirect) && !empty($result)) {
6020
-                            $form_data['action'] = api_get_path(WEB_CODE_PATH) . 'exercise/overview.php?exerciseId='.$exercise_redirect.'&cidReq='.$course_info['code'];
6019
+                            $form_data['action'] = api_get_path(WEB_CODE_PATH).'exercise/overview.php?exerciseId='.$exercise_redirect.'&cidReq='.$course_info['code'];
6021 6020
                             $form_data['message'] .= '<br />'.get_lang('YouCanAccessTheExercise');
6022 6021
                             $form_data['button'] = Display::button(
6023 6022
                                 'next',
@@ -6073,7 +6072,7 @@  discard block
 block discarded – undo
6073 6072
 
6074 6073
         $without_special_courses = '';
6075 6074
         if (!empty($special_course_list)) {
6076
-            $without_special_courses = ' AND course.id NOT IN ("'.implode('","',$special_course_list).'")';
6075
+            $without_special_courses = ' AND course.id NOT IN ("'.implode('","', $special_course_list).'")';
6077 6076
         }
6078 6077
 
6079 6078
         //AND course_rel_user.relation_type<>".COURSE_RELATION_TYPE_RRHH."
@@ -6139,11 +6138,11 @@  discard block
 block discarded – undo
6139 6138
 
6140 6139
         if (api_is_platform_admin()) {
6141 6140
             if ($loadDirs) {
6142
-                $params['right_actions'] .= '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview" href="javascript:void(0);">'.Display::return_icon('folder.png', get_lang('Documents'), array('align' => 'absmiddle'),ICON_SIZE_SMALL).'</a>';
6143
-                $params['right_actions'] .= '<a href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::return_icon('edit.png', get_lang('Edit'), array('align' => 'absmiddle'),ICON_SIZE_SMALL).'</a>';
6141
+                $params['right_actions'] .= '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview" href="javascript:void(0);">'.Display::return_icon('folder.png', get_lang('Documents'), array('align' => 'absmiddle'), ICON_SIZE_SMALL).'</a>';
6142
+                $params['right_actions'] .= '<a href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::return_icon('edit.png', get_lang('Edit'), array('align' => 'absmiddle'), ICON_SIZE_SMALL).'</a>';
6144 6143
                 $params['right_actions'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class'=>'document_preview_container'));
6145 6144
             } else {
6146
-                $params['right_actions'].= '<a class="btn btn-default btn-sm" title="'.get_lang('Edit').'" href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::returnFontAwesomeIcon('pencil').'</a>';
6145
+                $params['right_actions'] .= '<a class="btn btn-default btn-sm" title="'.get_lang('Edit').'" href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::returnFontAwesomeIcon('pencil').'</a>';
6147 6146
             }
6148 6147
 
6149 6148
             if ($course_info['status'] == COURSEMANAGER) {
@@ -6152,11 +6151,11 @@  discard block
 block discarded – undo
6152 6151
         } else {
6153 6152
             if ($course_info['visibility'] != COURSE_VISIBILITY_CLOSED) {
6154 6153
                 if ($loadDirs) {
6155
-                    $params['right_actions'] .= '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview" href="javascript:void(0);">'.Display::return_icon('folder.png', get_lang('Documents'), array('align' => 'absmiddle'),ICON_SIZE_SMALL).'</a>';
6154
+                    $params['right_actions'] .= '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview" href="javascript:void(0);">'.Display::return_icon('folder.png', get_lang('Documents'), array('align' => 'absmiddle'), ICON_SIZE_SMALL).'</a>';
6156 6155
                     $params['right_actions'] .= Display::div('', array('id' => 'document_result_'.$course_info['real_id'].'_0', 'class'=>'document_preview_container'));
6157 6156
                 } else {
6158 6157
                     if ($course_info['status'] == COURSEMANAGER) {
6159
-                        $params['right_actions'].= '<a class="btn btn-default btn-sm" title="'.get_lang('Edit').'" href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::returnFontAwesomeIcon('pencil').'</a>';
6158
+                        $params['right_actions'] .= '<a class="btn btn-default btn-sm" title="'.get_lang('Edit').'" href="'.api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course['code'].'">'.Display::returnFontAwesomeIcon('pencil').'</a>';
6160 6159
                     }
6161 6160
                 }
6162 6161
             }
@@ -6167,7 +6166,7 @@  discard block
 block discarded – undo
6167 6166
             $course_title_url = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/?id_session=0';
6168 6167
             $course_title = Display::url($course_info['title'], $course_title_url);
6169 6168
         } else {
6170
-            $course_title = $course_info['title'].' '.Display::tag('span',get_lang('CourseClosed'), array('class'=>'item_closed'));
6169
+            $course_title = $course_info['title'].' '.Display::tag('span', get_lang('CourseClosed'), array('class'=>'item_closed'));
6171 6170
         }
6172 6171
 
6173 6172
         // Start displaying the course block itself
Please login to merge, or discard this patch.
Upper-Lower-Casing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
         $courseId = intval($courseId);
359 359
         $isTutor = intval($isTutor);
360 360
 
361
-        $sql = "UPDATE $table SET is_tutor = '".$isTutor."'
361
+        $sql = "update $table SET is_tutor = '".$isTutor."'
362 362
 			    WHERE
363 363
 				    user_id = '".$userId."' AND
364 364
 				    c_id = '".$courseId."'";
@@ -2354,7 +2354,7 @@  discard block
 block discarded – undo
2354 2354
 
2355 2355
             // Update ticket
2356 2356
             $table = Database::get_main_table(TABLE_TICKET_TICKET);
2357
-            $sql = "UPDATE $table SET course_id = NULL WHERE course_id = $courseId";
2357
+            $sql = "update $table SET course_id = NULL WHERE course_id = $courseId";
2358 2358
             Database::query($sql);
2359 2359
 
2360 2360
             // Class
@@ -2367,7 +2367,7 @@  discard block
 block discarded – undo
2367 2367
             $table = Database::get_main_table(TABLE_MAIN_SKILL_REL_USER);
2368 2368
             $argumentation = Database::escape_string(sprintf(get_lang('SkillFromCourseXDeletedSinceThen'), $course['code']));
2369 2369
 
2370
-            $sql = "UPDATE $table SET course_id = NULL, session_id = NULL, argumentation = '$argumentation' WHERE course_id = $courseId";
2370
+            $sql = "update $table SET course_id = NULL, session_id = NULL, argumentation = '$argumentation' WHERE course_id = $courseId";
2371 2371
             Database::query($sql);
2372 2372
 
2373 2373
             Category::deleteCategoryFromCourse($courseId);
@@ -2985,7 +2985,7 @@  discard block
 block discarded – undo
2985 2985
     {
2986 2986
         $id = (int)$id;
2987 2987
         $table = Database::get_main_table(TABLE_MAIN_COURSE);
2988
-        $sql = "UPDATE $table SET $name = '" . Database::escape_string($value) . "'
2988
+        $sql = "update $table SET $name = '" . Database::escape_string($value) . "'
2989 2989
                 WHERE id = '$id'";
2990 2990
 
2991 2991
         return Database::query($sql);
@@ -3003,7 +3003,7 @@  discard block
 block discarded – undo
3003 3003
     {
3004 3004
         $id = (int)$id;
3005 3005
         $table = Database::get_main_table(TABLE_MAIN_COURSE);
3006
-        $sql = "UPDATE $table SET ";
3006
+        $sql = "update $table SET ";
3007 3007
         $i = 0;
3008 3008
         foreach ($attributes as $name => $value) {
3009 3009
             if ($value != '') {
@@ -4470,12 +4470,12 @@  discard block
 block discarded – undo
4470 4470
 
4471 4471
         if (empty($session_id)) {
4472 4472
             $table = Database::get_main_table(TABLE_MAIN_COURSE_USER);
4473
-            $sql = "UPDATE $table SET legal_agreement = '1'
4473
+            $sql = "update $table SET legal_agreement = '1'
4474 4474
                     WHERE user_id = $user_id AND c_id  = $courseId ";
4475 4475
             Database::query($sql);
4476 4476
         } else {
4477 4477
             $table = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
4478
-            $sql = "UPDATE  $table SET legal_agreement = '1'
4478
+            $sql = "update  $table SET legal_agreement = '1'
4479 4479
                     WHERE user_id = $user_id AND c_id = $courseId AND session_id = $session_id";
4480 4480
             Database::query($sql);
4481 4481
         }
Please login to merge, or discard this patch.