Completed
Push — 1.11.x ( 1039fe...2dc406 )
by José
159:48 queued 115:41
created
main/mySpace/myStudents.php 1 patch
Spacing   +69 added lines, -69 removed lines patch added patch discarded remove patch
@@ -67,52 +67,52 @@  discard block
 block discarded – undo
67 67
 if (isset($_GET['details'])) {
68 68
     if ($origin === 'user_course') {
69 69
         if (empty ($cidReq)) {
70
-            $interbreadcrumb[] = array (
71
-                "url" => api_get_path(WEB_COURSE_PATH) . $courseInfo['directory'],
70
+            $interbreadcrumb[] = array(
71
+                "url" => api_get_path(WEB_COURSE_PATH).$courseInfo['directory'],
72 72
                 'name' => $courseInfo['title']
73 73
             );
74 74
         }
75
-        $interbreadcrumb[] = array (
76
-            "url" => "../user/user.php?cidReq=" . $course_code,
75
+        $interbreadcrumb[] = array(
76
+            "url" => "../user/user.php?cidReq=".$course_code,
77 77
             "name" => get_lang("Users")
78 78
         );
79 79
     } else
80 80
         if ($origin === 'tracking_course') {
81
-            $interbreadcrumb[] = array (
82
-                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session=' . api_get_session_id(),
81
+            $interbreadcrumb[] = array(
82
+                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session='.api_get_session_id(),
83 83
                 "name" => get_lang("Tracking")
84 84
             );
85 85
         } else
86 86
             if ($origin === 'resume_session') {
87
-                $interbreadcrumb[] = array (
87
+                $interbreadcrumb[] = array(
88 88
                     'url' => "../session/session_list.php",
89 89
                     "name" => get_lang('SessionList')
90 90
                 );
91
-                $interbreadcrumb[] = array (
92
-                    'url' => "../session/resume_session.php?id_session=" . $sessionId,
91
+                $interbreadcrumb[] = array(
92
+                    'url' => "../session/resume_session.php?id_session=".$sessionId,
93 93
                     "name" => get_lang('SessionOverview')
94 94
                 );
95 95
             } else {
96
-                $interbreadcrumb[] = array (
97
-                    "url" => api_is_student_boss()?"#":"index.php",
96
+                $interbreadcrumb[] = array(
97
+                    "url" => api_is_student_boss() ? "#" : "index.php",
98 98
                     "name" => get_lang('MySpace')
99 99
                 );
100 100
                 if (isset ($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
101
-                    $interbreadcrumb[] = array (
102
-                        "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
101
+                    $interbreadcrumb[] = array(
102
+                        "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
103 103
                         "name" => get_lang("CoachStudents")
104 104
                     );
105
-                    $interbreadcrumb[] = array (
106
-                        "url" => "myStudents.php?student=" . $student_id. '&id_coach=' . Security :: remove_XSS($_GET['id_coach']),
105
+                    $interbreadcrumb[] = array(
106
+                        "url" => "myStudents.php?student=".$student_id.'&id_coach='.Security :: remove_XSS($_GET['id_coach']),
107 107
                         "name" => get_lang("StudentDetails")
108 108
                     );
109 109
                 } else {
110
-                    $interbreadcrumb[] = array (
110
+                    $interbreadcrumb[] = array(
111 111
                         "url" => "student.php",
112 112
                         "name" => get_lang("MyStudents")
113 113
                     );
114
-                    $interbreadcrumb[] = array (
115
-                        "url" => "myStudents.php?student=" . $student_id,
114
+                    $interbreadcrumb[] = array(
115
+                        "url" => "myStudents.php?student=".$student_id,
116 116
                         "name" => get_lang("StudentDetails")
117 117
                     );
118 118
                 }
@@ -120,7 +120,7 @@  discard block
 block discarded – undo
120 120
     $nameTools = get_lang("DetailsStudentInCourse");
121 121
 } else {
122 122
     if ($origin == 'resume_session') {
123
-        $interbreadcrumb[] = array (
123
+        $interbreadcrumb[] = array(
124 124
             'url' => "../session/session_list.php",
125 125
             "name" => get_lang('SessionList')
126 126
         );
@@ -131,24 +131,24 @@  discard block
 block discarded – undo
131 131
             );
132 132
         }
133 133
     } else {
134
-        $interbreadcrumb[] = array (
135
-            "url" => api_is_student_boss()?"#":"index.php",
134
+        $interbreadcrumb[] = array(
135
+            "url" => api_is_student_boss() ? "#" : "index.php",
136 136
             "name" => get_lang('MySpace')
137 137
         );
138 138
         if (isset ($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
139 139
             if ($sessionId) {
140
-                $interbreadcrumb[] = array (
141
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']) . "&id_session=" . $sessionId,
140
+                $interbreadcrumb[] = array(
141
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach'])."&id_session=".$sessionId,
142 142
                     "name" => get_lang("CoachStudents")
143 143
                 );
144 144
             } else {
145
-                $interbreadcrumb[] = array (
146
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
145
+                $interbreadcrumb[] = array(
146
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
147 147
                     "name" => get_lang("CoachStudents")
148 148
                 );
149 149
             }
150 150
         } else {
151
-            $interbreadcrumb[] = array (
151
+            $interbreadcrumb[] = array(
152 152
                 "url" => "student.php",
153 153
                 "name" => get_lang("MyStudents")
154 154
             );
@@ -330,28 +330,28 @@  discard block
 block discarded – undo
330 330
     // Actions bar
331 331
     echo '<div class="actions">';
332 332
     echo '<a href="javascript: window.history.go(-1);">'.
333
-            Display::return_icon('back.png', get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
333
+            Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
334 334
 
335 335
     echo '<a href="javascript: void(0);" onclick="javascript: window.print();">'.
336
-            Display::return_icon('printer.png', get_lang('Print'),'',ICON_SIZE_MEDIUM).'</a>';
336
+            Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM).'</a>';
337 337
 
338
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=csv">'.
339
-            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a> ';
338
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=csv">'.
339
+            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a> ';
340 340
 
341
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=xls">'.
342
-    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'),'',ICON_SIZE_MEDIUM).'</a> ';
341
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=xls">'.
342
+    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), '', ICON_SIZE_MEDIUM).'</a> ';
343 343
 
344 344
     if (!empty ($user_info['email'])) {
345 345
         $send_mail = '<a href="mailto:'.$user_info['email'].'">'.
346
-            Display :: return_icon('mail_send.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM).'</a>';
346
+            Display :: return_icon('mail_send.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM).'</a>';
347 347
     } else {
348
-        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM);
348
+        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM);
349 349
     }
350 350
     echo $send_mail;
351 351
     if (!empty($student_id) && !empty($course_code)) {
352 352
         // Only show link to connection details if course and student were defined in the URL
353
-        echo '<a href="access_details.php?student=' . $student_id . '&course=' . $course_code . '&origin=' . $origin. '&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
354
-            Display :: return_icon('statistics.png', get_lang('AccessDetails'),'',ICON_SIZE_MEDIUM).'</a>';
353
+        echo '<a href="access_details.php?student='.$student_id.'&course='.$course_code.'&origin='.$origin.'&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
354
+            Display :: return_icon('statistics.png', get_lang('AccessDetails'), '', ICON_SIZE_MEDIUM).'</a>';
355 355
     }
356 356
     if (api_can_login_as($student_id)) {
357 357
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'admin/user_list.php?action=login_as&user_id='.$student_id.'&sec_token='.$token.'">'.
@@ -360,7 +360,7 @@  discard block
 block discarded – undo
360 360
 
361 361
     echo Display::url(
362 362
         Display::return_icon('skill-badges.png', get_lang('AssignSkill'), null, ICON_SIZE_MEDIUM),
363
-        api_get_path(WEB_CODE_PATH) . 'badge/assign.php?' . http_build_query(['user' => $student_id])
363
+        api_get_path(WEB_CODE_PATH).'badge/assign.php?'.http_build_query(['user' => $student_id])
364 364
     );
365 365
 
366 366
 
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
421 421
     $csv_content[] = array(
422 422
         get_lang('Information', '')
423 423
     );
424
-    $csv_content[] = array (
424
+    $csv_content[] = array(
425 425
         get_lang('Name', ''),
426 426
         get_lang('Email', ''),
427 427
         get_lang('Tel', '')
@@ -449,7 +449,7 @@  discard block
 block discarded – undo
449 449
         strip_tags($first_connection_date),
450 450
         strip_tags($last_connection_date),
451 451
         $time_spent_on_the_course,
452
-        $avg_student_progress . '%',
452
+        $avg_student_progress.'%',
453 453
         $avg_student_score
454 454
     );
455 455
 
@@ -479,19 +479,19 @@  discard block
 block discarded – undo
479 479
             </thead>
480 480
             <tbody>
481 481
             <tr>
482
-                <td><?php echo get_lang('Name') . ' : '.$user_info['complete_name']; ?></td>
482
+                <td><?php echo get_lang('Name').' : '.$user_info['complete_name']; ?></td>
483 483
             </tr>
484 484
             <tr>
485
-                <td><?php echo get_lang('Email') . ' : ';
485
+                <td><?php echo get_lang('Email').' : ';
486 486
                     if (!empty ($user_info['email'])) {
487
-                        echo '<a href="mailto:' . $user_info['email'] . '">' . $user_info['email'] . '</a>';
487
+                        echo '<a href="mailto:'.$user_info['email'].'">'.$user_info['email'].'</a>';
488 488
                     } else {
489 489
                         echo get_lang('NoEmail');
490 490
                     } ?>
491 491
                 </td>
492 492
             </tr>
493 493
             <tr>
494
-                <td> <?php echo get_lang('Tel') . ' : ';
494
+                <td> <?php echo get_lang('Tel').' : ';
495 495
                     if (!empty ($user_info['phone'])) {
496 496
                         echo $user_info['phone'];
497 497
                     } else {
@@ -501,7 +501,7 @@  discard block
 block discarded – undo
501 501
                 </td>
502 502
             </tr>
503 503
             <tr>
504
-                <td> <?php echo get_lang('OfficialCode') . ' : ';
504
+                <td> <?php echo get_lang('OfficialCode').' : ';
505 505
                     if (!empty ($user_info['official_code'])) {
506 506
                         echo $user_info['official_code'];
507 507
                     } else {
@@ -511,7 +511,7 @@  discard block
 block discarded – undo
511 511
                 </td>
512 512
             </tr>
513 513
             <tr>
514
-                <td><?php echo get_lang('OnLine') . ' : '.$online; ?> </td>
514
+                <td><?php echo get_lang('OnLine').' : '.$online; ?> </td>
515 515
             </tr>
516 516
             <?php
517 517
 
@@ -525,7 +525,7 @@  discard block
 block discarded – undo
525 525
             if ($timezone !== null) {
526 526
                 ?>
527 527
                 <tr>
528
-                    <td> <?php echo get_lang('Timezone') . ' : '.$timezone; ?> </td>
528
+                    <td> <?php echo get_lang('Timezone').' : '.$timezone; ?> </td>
529 529
                 </tr>
530 530
             <?php
531 531
             }
@@ -672,9 +672,9 @@  discard block
 block discarded – undo
672 672
                 }
673 673
 
674 674
                 if (!empty($access_start_date) && !empty($access_end_date)) {
675
-                    $date_session = get_lang('From') . ' ' . $access_start_date . ' ' . get_lang('Until') . ' ' . $access_end_date;
675
+                    $date_session = get_lang('From').' '.$access_start_date.' '.get_lang('Until').' '.$access_end_date;
676 676
                 }
677
-                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session?' ('.$date_session.')':'');
677
+                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session ? ' ('.$date_session.')' : '');
678 678
             }
679 679
 
680 680
             // Courses
@@ -729,16 +729,16 @@  discard block
 block discarded – undo
729 729
                         $scoretotal = array();
730 730
                         if (isset($cats) && isset($cats[0])) {
731 731
                             if (!empty($sessionId)) {
732
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem, $sessionId);
732
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem, $sessionId);
733 733
                             } else {
734
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem);
734
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem);
735 735
                             }
736 736
                         }
737 737
 
738 738
                         $scoretotal_display = '0/0 (0%)';
739 739
                         if (!empty($scoretotal)) {
740 740
                             $scoretotal_display =
741
-                                round($scoretotal[0], 1 ).'/'.
741
+                                round($scoretotal[0], 1).'/'.
742 742
                                 round($scoretotal[1], 1).
743 743
                                 ' ('.round(($scoretotal[0] / $scoretotal[1]) * 100, 2).' %)';
744 744
                         }
@@ -759,9 +759,9 @@  discard block
 block discarded – undo
759 759
                         );
760 760
 
761 761
                         echo '<tr>
762
-                        <td ><a href="' .$courseInfoItem['course_public_url'] .'?id_session=' . $sessionId . '">'.
762
+                        <td ><a href="' .$courseInfoItem['course_public_url'].'?id_session='.$sessionId.'">'.
763 763
                             $courseInfoItem['title'].'</a></td>
764
-                        <td >'.$time_spent_on_course .'</td>
764
+                        <td >'.$time_spent_on_course.'</td>
765 765
                         <td >'.$progress.'</td>
766 766
                         <td >'.$score.'</td>
767 767
                         <td >'.$attendances_faults_avg.'</td>
@@ -829,7 +829,7 @@  discard block
 block discarded – undo
829 829
                 <table class="table table-striped table-hover">
830 830
                 <thead>
831 831
                 <tr>
832
-                    <th><?php echo get_lang('LearningPath');?></th>
832
+                    <th><?php echo get_lang('LearningPath'); ?></th>
833 833
                     <th>
834 834
                         <?php
835 835
                         echo get_lang('Time').' ';
@@ -931,7 +931,7 @@  discard block
 block discarded – undo
931 931
                     if (!empty($start_time)) {
932 932
                         $start_time = api_convert_and_format_date($start_time, DATE_TIME_FORMAT_LONG);
933 933
                     } else {
934
-                        $start_time =  '-';
934
+                        $start_time = '-';
935 935
                     }
936 936
 
937 937
                     if (!empty($total_time)) {
@@ -968,8 +968,8 @@  discard block
 block discarded – undo
968 968
                     $csv_content[] = array(
969 969
                         api_html_entity_decode(stripslashes($lp_name), ENT_QUOTES, $charset),
970 970
                         api_time_to_hms($total_time),
971
-                        $score . '%',
972
-                        $score_latest . '%',
971
+                        $score.'%',
972
+                        $score_latest.'%',
973 973
                         $progress.'%',
974 974
                         $start_time
975 975
                     );
@@ -1008,7 +1008,7 @@  discard block
 block discarded – undo
1008 1008
                     if ($any_result === true) {
1009 1009
                         $from = '';
1010 1010
                         if ($from_myspace) {
1011
-                            $from ='&from=myspace';
1011
+                            $from = '&from=myspace';
1012 1012
                         }
1013 1013
                         $link = Display::url(
1014 1014
                             Display::return_icon('2rightarrow.png', get_lang('Details')),
@@ -1021,14 +1021,14 @@  discard block
 block discarded – undo
1021 1021
                         echo '<td>';
1022 1022
                         if ($any_result === true) {
1023 1023
                             echo '<a href="myStudents.php?action=reset_lp&sec_token='.$token.'&cidReq='.$course_code.'&course='.$course_code.'&details='.Security::remove_XSS($_GET['details']).'&origin='.$origin.'&lp_id='.$learnpath->getId().'&student='.$user_info['user_id'].'&details=true&id_session='.$sessionId.'">';
1024
-                            echo Display::return_icon('clean.png', get_lang('Clean'),'',ICON_SIZE_SMALL).'</a>';
1024
+                            echo Display::return_icon('clean.png', get_lang('Clean'), '', ICON_SIZE_SMALL).'</a>';
1025 1025
                             echo '</a>';
1026 1026
                         }
1027 1027
                         echo '</td>';
1028 1028
                         echo '</tr>';
1029 1029
                     }
1030 1030
                     $data_learnpath[$i][] = $lp_name;
1031
-                    $data_learnpath[$i][] = $progress . '%';
1031
+                    $data_learnpath[$i][] = $progress.'%';
1032 1032
                 }
1033 1033
                 ?>
1034 1034
                 </tbody>
@@ -1044,7 +1044,7 @@  discard block
 block discarded – undo
1044 1044
         <thead>
1045 1045
         <tr>
1046 1046
             <th><?php echo get_lang('Exercises'); ?></th>
1047
-            <th><?php echo get_lang('LearningPath');?></th>
1047
+            <th><?php echo get_lang('LearningPath'); ?></th>
1048 1048
             <th><?php echo get_lang('AvgCourseScore').' '.Display :: return_icon('info3.gif', get_lang('AverageScore'), array('align' => 'absmiddle', 'hspace' => '3px')) ?></th>
1049 1049
             <th><?php echo get_lang('Attempts'); ?></th>
1050 1050
             <th><?php echo get_lang('LatestAttempt'); ?></th>
@@ -1063,7 +1063,7 @@  discard block
 block discarded – undo
1063 1063
         );
1064 1064
 
1065 1065
         $t_quiz = Database :: get_course_table(TABLE_QUIZ_TEST);
1066
-        $sql = "SELECT quiz.title, id FROM " . $t_quiz . " AS quiz
1066
+        $sql = "SELECT quiz.title, id FROM ".$t_quiz." AS quiz
1067 1067
                 WHERE
1068 1068
                     quiz.c_id = ".$courseInfo['real_id']." AND
1069 1069
                     (quiz.session_id = $sessionId OR quiz.session_id = 0) AND
@@ -1128,7 +1128,7 @@  discard block
 block discarded – undo
1128 1128
                 echo '<td>';
1129 1129
 
1130 1130
                 if ($count_attempts > 0) {
1131
-                    echo $score_percentage . '%';
1131
+                    echo $score_percentage.'%';
1132 1132
                 } else {
1133 1133
                     echo '-';
1134 1134
                     $score_percentage = 0;
@@ -1138,7 +1138,7 @@  discard block
 block discarded – undo
1138 1138
                 echo '<td>'.$count_attempts.'</td>';
1139 1139
                 echo '<td>';
1140 1140
 
1141
-                $sql = 'SELECT exe_id FROM ' . $tbl_stats_exercices . '
1141
+                $sql = 'SELECT exe_id FROM '.$tbl_stats_exercices.'
1142 1142
                          WHERE
1143 1143
                             exe_exo_id = "'.$exercise_id.'" AND
1144 1144
                             exe_user_id ="'.$student_id.'" AND
@@ -1151,7 +1151,7 @@  discard block
 block discarded – undo
1151 1151
                 if (Database :: num_rows($result_last_attempt) > 0) {
1152 1152
                     $id_last_attempt = Database :: result($result_last_attempt, 0, 0);
1153 1153
                     if ($count_attempts > 0)
1154
-                        echo '<a href="../exercise/exercise_show.php?id=' . $id_last_attempt . '&cidReq='.$course_code.'&session_id='.$sessionId.'&student='.$student_id.'&origin='.(empty($origin)?'tracking':$origin).'">
1154
+                        echo '<a href="../exercise/exercise_show.php?id='.$id_last_attempt.'&cidReq='.$course_code.'&session_id='.$sessionId.'&student='.$student_id.'&origin='.(empty($origin) ? 'tracking' : $origin).'">
1155 1155
                         '.Display::return_icon('quiz.gif').'
1156 1156
                      </a>';
1157 1157
                 }
@@ -1159,11 +1159,11 @@  discard block
 block discarded – undo
1159 1159
 
1160 1160
                 echo '<td>';
1161 1161
                 $all_attempt_url = "../exercise/exercise_report.php?exerciseId=$exercise_id&cidReq=$course_code&filter_by_user=$student_id&id_session=$sessionId";
1162
-                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url );
1162
+                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url);
1163 1163
 
1164 1164
                 echo '</td></tr>';
1165 1165
                 $data_exercices[$i][] = $exercices['title'];
1166
-                $data_exercices[$i][] = $score_percentage . '%';
1166
+                $data_exercices[$i][] = $score_percentage.'%';
1167 1167
                 $data_exercices[$i][] = $count_attempts;
1168 1168
 
1169 1169
                 $csv_content[] = array(
@@ -1191,7 +1191,7 @@  discard block
 block discarded – undo
1191 1191
             $survey_list = SurveyManager::get_surveys($course_code, $sessionId);
1192 1192
 
1193 1193
             $survey_data = array();
1194
-            foreach($survey_list as $survey) {
1194
+            foreach ($survey_list as $survey) {
1195 1195
                 $user_list = SurveyManager::get_people_who_filled_survey($survey['survey_id'], false, $courseInfo['real_id']);
1196 1196
                 $survey_done = Display::return_icon("accept_na.png", get_lang('NoAnswer'), array(), ICON_SIZE_SMALL);
1197 1197
                 if (in_array($student_id, $user_list)) {
@@ -1216,7 +1216,7 @@  discard block
 block discarded – undo
1216 1216
                         $column = 0;
1217 1217
                         $table->setCellContents($row, $column, $data);
1218 1218
                         $class = 'class="row_odd"';
1219
-                        if($row % 2) {
1219
+                        if ($row % 2) {
1220 1220
                             $class = 'class="row_even"';
1221 1221
                         }
1222 1222
                         $table->setRowAttributes($row, $class, true);
Please login to merge, or discard this patch.