diff --git a/main/inc/lib/myspace.lib.php b/main/inc/lib/myspace.lib.php index cf250f55aa..501655a58d 100644 --- a/main/inc/lib/myspace.lib.php +++ b/main/inc/lib/myspace.lib.php @@ -202,11 +202,17 @@ class MySpace // getting all the courses of the user $sql = "SELECT * FROM $tbl_course_user - WHERE user_id = '".intval($user_id)."' AND relation_type<>".COURSE_RELATION_TYPE_RRHH." "; + WHERE + user_id = '".intval($user_id)."' AND + relation_type<>".COURSE_RELATION_TYPE_RRHH." "; $result = Database::query($sql); - while ($row = Database::fetch_row($result)) { - $courseCode = $row[0]; - $courseInfo = api_get_course_info($courseCode); + while ($row = Database::fetch_array($result)) { + $courseInfo = api_get_course_info_by_id($row['c_id']); + if (empty($courseInfo)) { + continue; + } + + $courseCode = $courseInfo['code']; $courseId = $courseInfo['real_id']; $return .= ''; diff --git a/main/tracking/course_session_report.php b/main/tracking/course_session_report.php index ac275ff150..295569fc22 100755 --- a/main/tracking/course_session_report.php +++ b/main/tracking/course_session_report.php @@ -115,9 +115,9 @@ foreach ($course_list as $current_course ) { $main_result[$current_course['code']] = $attempt_result; } -//var_dump($main_result); $total_average_score = 0; $total_average_score_count = 0; +$html_result = ''; if (!empty($users) && is_array($users)) { $html_result .= ''; @@ -188,7 +188,10 @@ if (!empty($users) && is_array($users)) { $counter = 0; foreach($course_list as $course_item) { if (!empty($course_average_counter[$course_item['code']])) { - $average_per_course = round($course_average[$course_item['code']]/($course_average_counter[$course_item['code']]*100)*100,2); + $average_per_course = round( + $course_average[$course_item['code']]/($course_average_counter[$course_item['code']]*100)*100, + 2 + ); } else { $average_per_course = '-'; } diff --git a/main/tracking/question_course_report.php b/main/tracking/question_course_report.php index c3cb486cfa..98361d2101 100755 --- a/main/tracking/question_course_report.php +++ b/main/tracking/question_course_report.php @@ -13,7 +13,7 @@ $this_section = "session_my_space"; $is_allowedToTrack = $is_courseAdmin || $is_platformAdmin || $is_courseCoach || $is_sessionAdmin; -if(!$is_allowedToTrack) { +if (!$is_allowedToTrack) { Display :: display_header(null); api_not_allowed(); Display :: display_footer(); @@ -32,6 +32,7 @@ if (api_is_platform_admin() ) { $global = true; $course_list = $course_select_list = array(); +$html_result = ''; $course_select_list[0] = get_lang('None'); $htmlHeadXtra[] = ' @@ -39,7 +40,7 @@ $htmlHeadXtra[] = ' function load_courses() { document.search_simple.submit(); } - '; +'; $session_id = isset($_REQUEST['session_id']) ? intval($_REQUEST['session_id']) : null; @@ -51,12 +52,6 @@ if (empty($session_id)) { foreach ($temp_course_list as $temp_course_item) { $course_item = CourseManager ::get_course_information($temp_course_item['code']); - $course_list[] = array( - 'db_name' => $course_item['db_name'], - 'code' => $course_item['code'], - 'title' => $course_item['title'], - 'visual_code' => $course_item['visual_code'] - ); $course_select_list[$temp_course_item['code']] = $course_item['title']; }