Completed
Push — master ( aa100c...0ecf7a )
by Angel Fernando Quiroz
48:16 queued 20:19
created
main/mySpace/myStudents.php 1 patch
Spacing   +69 added lines, -69 removed lines patch added patch discarded remove patch
@@ -76,52 +76,52 @@  discard block
 block discarded – undo
76 76
 if (isset($_GET['details'])) {
77 77
     if ($origin === 'user_course') {
78 78
         if (empty($cidReq)) {
79
-            $interbreadcrumb[] = array (
80
-                "url" => api_get_path(WEB_COURSE_PATH) . $courseInfo['directory'],
79
+            $interbreadcrumb[] = array(
80
+                "url" => api_get_path(WEB_COURSE_PATH).$courseInfo['directory'],
81 81
                 'name' => $courseInfo['title']
82 82
             );
83 83
         }
84
-        $interbreadcrumb[] = array (
85
-            "url" => "../user/user.php?cidReq=" . $course_code,
84
+        $interbreadcrumb[] = array(
85
+            "url" => "../user/user.php?cidReq=".$course_code,
86 86
             "name" => get_lang("Users")
87 87
         );
88 88
     } else {
89 89
         if ($origin === 'tracking_course') {
90
-            $interbreadcrumb[] = array (
91
-                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session=' . api_get_session_id(),
90
+            $interbreadcrumb[] = array(
91
+                "url" => "../tracking/courseLog.php?cidReq=".$course_code.'&id_session='.api_get_session_id(),
92 92
                 "name" => get_lang("Tracking")
93 93
             );
94 94
         } else {
95 95
             if ($origin === 'resume_session') {
96
-                $interbreadcrumb[] = array (
96
+                $interbreadcrumb[] = array(
97 97
                     'url' => "../session/session_list.php",
98 98
                     "name" => get_lang('SessionList')
99 99
                 );
100
-                $interbreadcrumb[] = array (
101
-                    'url' => "../session/resume_session.php?id_session=" . $sessionId,
100
+                $interbreadcrumb[] = array(
101
+                    'url' => "../session/resume_session.php?id_session=".$sessionId,
102 102
                     "name" => get_lang('SessionOverview')
103 103
                 );
104 104
             } else {
105
-                $interbreadcrumb[] = array (
106
-                    "url" => api_is_student_boss()?"#":"index.php",
105
+                $interbreadcrumb[] = array(
106
+                    "url" => api_is_student_boss() ? "#" : "index.php",
107 107
                     "name" => get_lang('MySpace')
108 108
                 );
109 109
                 if (isset ($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
110
-                    $interbreadcrumb[] = array (
111
-                        "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
110
+                    $interbreadcrumb[] = array(
111
+                        "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
112 112
                         "name" => get_lang("CoachStudents")
113 113
                     );
114
-                    $interbreadcrumb[] = array (
115
-                        "url" => "myStudents.php?student=" . $student_id. '&id_coach=' . Security :: remove_XSS($_GET['id_coach']),
114
+                    $interbreadcrumb[] = array(
115
+                        "url" => "myStudents.php?student=".$student_id.'&id_coach='.Security :: remove_XSS($_GET['id_coach']),
116 116
                         "name" => get_lang("StudentDetails")
117 117
                     );
118 118
                 } else {
119
-                    $interbreadcrumb[] = array (
119
+                    $interbreadcrumb[] = array(
120 120
                         "url" => "student.php",
121 121
                         "name" => get_lang("MyStudents")
122 122
                     );
123
-                    $interbreadcrumb[] = array (
124
-                        "url" => "myStudents.php?student=" . $student_id,
123
+                    $interbreadcrumb[] = array(
124
+                        "url" => "myStudents.php?student=".$student_id,
125 125
                         "name" => get_lang("StudentDetails")
126 126
                     );
127 127
                 }
@@ -131,7 +131,7 @@  discard block
 block discarded – undo
131 131
     $nameTools = get_lang("DetailsStudentInCourse");
132 132
 } else {
133 133
     if ($origin == 'resume_session') {
134
-        $interbreadcrumb[] = array (
134
+        $interbreadcrumb[] = array(
135 135
             'url' => "../session/session_list.php",
136 136
             "name" => get_lang('SessionList')
137 137
         );
@@ -142,24 +142,24 @@  discard block
 block discarded – undo
142 142
             );
143 143
         }
144 144
     } else {
145
-        $interbreadcrumb[] = array (
146
-            "url" => api_is_student_boss()?"#":"index.php",
145
+        $interbreadcrumb[] = array(
146
+            "url" => api_is_student_boss() ? "#" : "index.php",
147 147
             "name" => get_lang('MySpace')
148 148
         );
149 149
         if (isset($_GET['id_coach']) && intval($_GET['id_coach']) != 0) {
150 150
             if ($sessionId) {
151
-                $interbreadcrumb[] = array (
152
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']) . "&id_session=" . $sessionId,
151
+                $interbreadcrumb[] = array(
152
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach'])."&id_session=".$sessionId,
153 153
                     "name" => get_lang("CoachStudents")
154 154
                 );
155 155
             } else {
156
-                $interbreadcrumb[] = array (
157
-                    "url" => "student.php?id_coach=" . Security :: remove_XSS($_GET['id_coach']),
156
+                $interbreadcrumb[] = array(
157
+                    "url" => "student.php?id_coach=".Security :: remove_XSS($_GET['id_coach']),
158 158
                     "name" => get_lang("CoachStudents")
159 159
                 );
160 160
             }
161 161
         } else {
162
-            $interbreadcrumb[] = array (
162
+            $interbreadcrumb[] = array(
163 163
                 "url" => "student.php",
164 164
                 "name" => get_lang("MyStudents")
165 165
             );
@@ -339,28 +339,28 @@  discard block
 block discarded – undo
339 339
     // Actions bar
340 340
     echo '<div class="actions">';
341 341
     echo '<a href="javascript: window.history.go(-1);">'.
342
-            Display::return_icon('back.png', get_lang('Back'),'',ICON_SIZE_MEDIUM).'</a>';
342
+            Display::return_icon('back.png', get_lang('Back'), '', ICON_SIZE_MEDIUM).'</a>';
343 343
 
344 344
     echo '<a href="javascript: void(0);" onclick="javascript: window.print();">'.
345
-            Display::return_icon('printer.png', get_lang('Print'),'',ICON_SIZE_MEDIUM).'</a>';
345
+            Display::return_icon('printer.png', get_lang('Print'), '', ICON_SIZE_MEDIUM).'</a>';
346 346
 
347
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=csv">'.
348
-            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'),'',ICON_SIZE_MEDIUM).'</a> ';
347
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=csv">'.
348
+            Display::return_icon('export_csv.png', get_lang('ExportAsCSV'), '', ICON_SIZE_MEDIUM).'</a> ';
349 349
 
350
-    echo '<a href="' . api_get_self() . '?' . Security :: remove_XSS($_SERVER['QUERY_STRING']) . '&export=xls">'.
351
-    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'),'',ICON_SIZE_MEDIUM).'</a> ';
350
+    echo '<a href="'.api_get_self().'?'.Security :: remove_XSS($_SERVER['QUERY_STRING']).'&export=xls">'.
351
+    Display::return_icon('export_excel.png', get_lang('ExportAsXLS'), '', ICON_SIZE_MEDIUM).'</a> ';
352 352
 
353 353
     if (!empty ($user_info['email'])) {
354 354
         $send_mail = '<a href="mailto:'.$user_info['email'].'">'.
355
-            Display :: return_icon('mail_send.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM).'</a>';
355
+            Display :: return_icon('mail_send.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM).'</a>';
356 356
     } else {
357
-        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'),'',ICON_SIZE_MEDIUM);
357
+        $send_mail = Display :: return_icon('mail_send_na.png', get_lang('SendMail'), '', ICON_SIZE_MEDIUM);
358 358
     }
359 359
     echo $send_mail;
360 360
     if (!empty($student_id) && !empty($course_code)) {
361 361
         // Only show link to connection details if course and student were defined in the URL
362
-        echo '<a href="access_details.php?student=' . $student_id . '&course=' . $course_code . '&origin=' . $origin. '&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
363
-            Display :: return_icon('statistics.png', get_lang('AccessDetails'),'',ICON_SIZE_MEDIUM).'</a>';
362
+        echo '<a href="access_details.php?student='.$student_id.'&course='.$course_code.'&origin='.$origin.'&cidReq='.$course_code.'&id_session='.$sessionId.'">'.
363
+            Display :: return_icon('statistics.png', get_lang('AccessDetails'), '', ICON_SIZE_MEDIUM).'</a>';
364 364
     }
365 365
     if (api_can_login_as($student_id)) {
366 366
         echo '<a href="'.api_get_path(WEB_CODE_PATH).'admin/user_list.php?action=login_as&user_id='.$student_id.'&sec_token='.$token.'">'.
@@ -370,7 +370,7 @@  discard block
 block discarded – undo
370 370
     if (api_is_platform_admin(false, true) || api_is_student_boss()) {
371 371
         echo Display::url(
372 372
             Display::return_icon('skill-badges.png', get_lang('AssignSkill'), null, ICON_SIZE_MEDIUM),
373
-            api_get_path(WEB_CODE_PATH) . 'badge/assign.php?' . http_build_query(['user' => $student_id])
373
+            api_get_path(WEB_CODE_PATH).'badge/assign.php?'.http_build_query(['user' => $student_id])
374 374
         );
375 375
     }
376 376
 
@@ -431,7 +431,7 @@  discard block
 block discarded – undo
431 431
     $csv_content[] = array(
432 432
         get_lang('Information')
433 433
     );
434
-    $csv_content[] = array (
434
+    $csv_content[] = array(
435 435
         get_lang('Name'),
436 436
         get_lang('Email'),
437 437
         get_lang('Tel')
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
459 459
         strip_tags($first_connection_date),
460 460
         strip_tags($last_connection_date),
461 461
         $time_spent_on_the_course,
462
-        $avg_student_progress . '%',
462
+        $avg_student_progress.'%',
463 463
         $avg_student_score
464 464
     );
465 465
 
@@ -489,19 +489,19 @@  discard block
 block discarded – undo
489 489
             </thead>
490 490
             <tbody>
491 491
             <tr>
492
-                <td><?php echo get_lang('Name') . ' : '.$user_info['complete_name']; ?></td>
492
+                <td><?php echo get_lang('Name').' : '.$user_info['complete_name']; ?></td>
493 493
             </tr>
494 494
             <tr>
495
-                <td><?php echo get_lang('Email') . ' : ';
495
+                <td><?php echo get_lang('Email').' : ';
496 496
                     if (!empty ($user_info['email'])) {
497
-                        echo '<a href="mailto:' . $user_info['email'] . '">' . $user_info['email'] . '</a>';
497
+                        echo '<a href="mailto:'.$user_info['email'].'">'.$user_info['email'].'</a>';
498 498
                     } else {
499 499
                         echo get_lang('NoEmail');
500 500
                     } ?>
501 501
                 </td>
502 502
             </tr>
503 503
             <tr>
504
-                <td> <?php echo get_lang('Tel') . ' : ';
504
+                <td> <?php echo get_lang('Tel').' : ';
505 505
                     if (!empty ($user_info['phone'])) {
506 506
                         echo $user_info['phone'];
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('OfficialCode') . ' : ';
514
+                <td> <?php echo get_lang('OfficialCode').' : ';
515 515
                     if (!empty ($user_info['official_code'])) {
516 516
                         echo $user_info['official_code'];
517 517
                     } else {
@@ -521,7 +521,7 @@  discard block
 block discarded – undo
521 521
                 </td>
522 522
             </tr>
523 523
             <tr>
524
-                <td><?php echo get_lang('OnLine') . ' : '.$online; ?> </td>
524
+                <td><?php echo get_lang('OnLine').' : '.$online; ?> </td>
525 525
             </tr>
526 526
             <?php
527 527
 
@@ -535,7 +535,7 @@  discard block
 block discarded – undo
535 535
             if ($timezone !== null) {
536 536
                 ?>
537 537
                 <tr>
538
-                    <td> <?php echo get_lang('Timezone') . ' : '.$timezone; ?> </td>
538
+                    <td> <?php echo get_lang('Timezone').' : '.$timezone; ?> </td>
539 539
                 </tr>
540 540
             <?php
541 541
             }
@@ -682,9 +682,9 @@  discard block
 block discarded – undo
682 682
                 }
683 683
 
684 684
                 if (!empty($access_start_date) && !empty($access_end_date)) {
685
-                    $date_session = get_lang('From') . ' ' . $access_start_date . ' ' . get_lang('Until') . ' ' . $access_end_date;
685
+                    $date_session = get_lang('From').' '.$access_start_date.' '.get_lang('Until').' '.$access_end_date;
686 686
                 }
687
-                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session?' ('.$date_session.')':'');
687
+                $title = Display::return_icon('session.png', get_lang('Session'), array(), ICON_SIZE_SMALL).' '.$session_name.($date_session ? ' ('.$date_session.')' : '');
688 688
             }
689 689
 
690 690
             // Courses
@@ -739,16 +739,16 @@  discard block
 block discarded – undo
739 739
                         $scoretotal = array();
740 740
                         if (isset($cats) && isset($cats[0])) {
741 741
                             if (!empty($sId)) {
742
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem, $sId);
742
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem, $sId);
743 743
                             } else {
744
-                                $scoretotal= $cats[0]->calc_score($student_id, null, $courseCodeItem);
744
+                                $scoretotal = $cats[0]->calc_score($student_id, null, $courseCodeItem);
745 745
                             }
746 746
                         }
747 747
 
748 748
                         $scoretotal_display = '0/0 (0%)';
749 749
                         if (!empty($scoretotal) && !empty($scoretotal[1])) {
750 750
                             $scoretotal_display =
751
-                                round($scoretotal[0], 1 ).'/'.
751
+                                round($scoretotal[0], 1).'/'.
752 752
                                 round($scoretotal[1], 1).
753 753
                                 ' ('.round(($scoretotal[0] / $scoretotal[1]) * 100, 2).' %)';
754 754
                         }
@@ -769,9 +769,9 @@  discard block
 block discarded – undo
769 769
                         );
770 770
 
771 771
                         echo '<tr>
772
-                        <td ><a href="' .$courseInfoItem['course_public_url'] .'?id_session=' . $sId . '">'.
772
+                        <td ><a href="' .$courseInfoItem['course_public_url'].'?id_session='.$sId.'">'.
773 773
                             $courseInfoItem['title'].'</a></td>
774
-                        <td >'.$time_spent_on_course .'</td>
774
+                        <td >'.$time_spent_on_course.'</td>
775 775
                         <td >'.$progress.'</td>
776 776
                         <td >'.$score.'</td>
777 777
                         <td >'.$attendances_faults_avg.'</td>
@@ -839,7 +839,7 @@  discard block
 block discarded – undo
839 839
                 <table class="table table-striped table-hover">
840 840
                 <thead>
841 841
                 <tr>
842
-                    <th><?php echo get_lang('LearningPath');?></th>
842
+                    <th><?php echo get_lang('LearningPath'); ?></th>
843 843
                     <th>
844 844
                         <?php
845 845
                         echo get_lang('Time').' ';
@@ -941,7 +941,7 @@  discard block
 block discarded – undo
941 941
                     if (!empty($start_time)) {
942 942
                         $start_time = api_convert_and_format_date($start_time, DATE_TIME_FORMAT_LONG);
943 943
                     } else {
944
-                        $start_time =  '-';
944
+                        $start_time = '-';
945 945
                     }
946 946
 
947 947
                     if (!empty($total_time)) {
@@ -978,8 +978,8 @@  discard block
 block discarded – undo
978 978
                     $csv_content[] = array(
979 979
                         api_html_entity_decode(stripslashes($lp_name), ENT_QUOTES, $charset),
980 980
                         api_time_to_hms($total_time),
981
-                        $score . '%',
982
-                        $score_latest . '%',
981
+                        $score.'%',
982
+                        $score_latest.'%',
983 983
                         $progress.'%',
984 984
                         $start_time
985 985
                     );
@@ -1018,7 +1018,7 @@  discard block
 block discarded – undo
1018 1018
                     if ($any_result === true) {
1019 1019
                         $from = '';
1020 1020
                         if ($from_myspace) {
1021
-                            $from ='&from=myspace';
1021
+                            $from = '&from=myspace';
1022 1022
                         }
1023 1023
                         $link = Display::url(
1024 1024
                             Display::return_icon('2rightarrow.png', get_lang('Details')),
@@ -1031,14 +1031,14 @@  discard block
 block discarded – undo
1031 1031
                         echo '<td>';
1032 1032
                         if ($any_result === true) {
1033 1033
                             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.'">';
1034
-                            echo Display::return_icon('clean.png', get_lang('Clean'),'',ICON_SIZE_SMALL).'</a>';
1034
+                            echo Display::return_icon('clean.png', get_lang('Clean'), '', ICON_SIZE_SMALL).'</a>';
1035 1035
                             echo '</a>';
1036 1036
                         }
1037 1037
                         echo '</td>';
1038 1038
                         echo '</tr>';
1039 1039
                     }
1040 1040
                     $data_learnpath[$i][] = $lp_name;
1041
-                    $data_learnpath[$i][] = $progress . '%';
1041
+                    $data_learnpath[$i][] = $progress.'%';
1042 1042
                 }
1043 1043
                 ?>
1044 1044
                 </tbody>
@@ -1054,7 +1054,7 @@  discard block
 block discarded – undo
1054 1054
         <thead>
1055 1055
         <tr>
1056 1056
             <th><?php echo get_lang('Exercises'); ?></th>
1057
-            <th><?php echo get_lang('LearningPath');?></th>
1057
+            <th><?php echo get_lang('LearningPath'); ?></th>
1058 1058
             <th><?php echo get_lang('AvgCourseScore').' '.Display :: return_icon('info3.gif', get_lang('AverageScore'), array('align' => 'absmiddle', 'hspace' => '3px')) ?></th>
1059 1059
             <th><?php echo get_lang('Attempts'); ?></th>
1060 1060
             <th><?php echo get_lang('LatestAttempt'); ?></th>
@@ -1075,7 +1075,7 @@  discard block
 block discarded – undo
1075 1075
         $t_quiz = Database :: get_course_table(TABLE_QUIZ_TEST);
1076 1076
         $sessionCondition = api_get_session_condition($sessionId, true, true, 'quiz.session_id');
1077 1077
 
1078
-        $sql = "SELECT quiz.title, id FROM " . $t_quiz . " AS quiz
1078
+        $sql = "SELECT quiz.title, id FROM ".$t_quiz." AS quiz
1079 1079
                 WHERE
1080 1080
                     quiz.c_id = ".$courseInfo['real_id']." AND
1081 1081
                     active IN (0, 1)
@@ -1140,7 +1140,7 @@  discard block
 block discarded – undo
1140 1140
                 echo '<td>';
1141 1141
 
1142 1142
                 if ($count_attempts > 0) {
1143
-                    echo $score_percentage . '%';
1143
+                    echo $score_percentage.'%';
1144 1144
                 } else {
1145 1145
                     echo '-';
1146 1146
                     $score_percentage = 0;
@@ -1150,7 +1150,7 @@  discard block
 block discarded – undo
1150 1150
                 echo '<td>'.$count_attempts.'</td>';
1151 1151
                 echo '<td>';
1152 1152
 
1153
-                $sql = 'SELECT exe_id FROM ' . $tbl_stats_exercices . '
1153
+                $sql = 'SELECT exe_id FROM '.$tbl_stats_exercices.'
1154 1154
                          WHERE
1155 1155
                             exe_exo_id = "'.$exercise_id.'" AND
1156 1156
                             exe_user_id ="'.$student_id.'" AND
@@ -1163,7 +1163,7 @@  discard block
 block discarded – undo
1163 1163
                 if (Database :: num_rows($result_last_attempt) > 0) {
1164 1164
                     $id_last_attempt = Database :: result($result_last_attempt, 0, 0);
1165 1165
                     if ($count_attempts > 0)
1166
-                        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).'">
1166
+                        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).'">
1167 1167
                         '.Display::return_icon('quiz.png').'
1168 1168
                      </a>';
1169 1169
                 }
@@ -1171,11 +1171,11 @@  discard block
 block discarded – undo
1171 1171
 
1172 1172
                 echo '<td>';
1173 1173
                 $all_attempt_url = "../exercise/exercise_report.php?exerciseId=$exercise_id&cidReq=$course_code&filter_by_user=$student_id&id_session=$sessionId";
1174
-                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url );
1174
+                echo Display::url(Display::return_icon('test_results.png', get_lang('AllAttempts'), array(), ICON_SIZE_SMALL), $all_attempt_url);
1175 1175
 
1176 1176
                 echo '</td></tr>';
1177 1177
                 $data_exercices[$i][] = $exercices['title'];
1178
-                $data_exercices[$i][] = $score_percentage . '%';
1178
+                $data_exercices[$i][] = $score_percentage.'%';
1179 1179
                 $data_exercices[$i][] = $count_attempts;
1180 1180
 
1181 1181
                 $csv_content[] = array(
@@ -1203,7 +1203,7 @@  discard block
 block discarded – undo
1203 1203
             $survey_list = SurveyManager::get_surveys($course_code, $sessionId);
1204 1204
 
1205 1205
             $survey_data = array();
1206
-            foreach($survey_list as $survey) {
1206
+            foreach ($survey_list as $survey) {
1207 1207
                 $user_list = SurveyManager::get_people_who_filled_survey($survey['survey_id'], false, $courseInfo['real_id']);
1208 1208
                 $survey_done = Display::return_icon("accept_na.png", get_lang('NoAnswer'), array(), ICON_SIZE_SMALL);
1209 1209
                 if (in_array($student_id, $user_list)) {
@@ -1228,7 +1228,7 @@  discard block
 block discarded – undo
1228 1228
                         $column = 0;
1229 1229
                         $table->setCellContents($row, $column, $data);
1230 1230
                         $class = 'class="row_odd"';
1231
-                        if($row % 2) {
1231
+                        if ($row % 2) {
1232 1232
                             $class = 'class="row_even"';
1233 1233
                         }
1234 1234
                         $table->setRowAttributes($row, $class, true);
Please login to merge, or discard this patch.