Minor - fix PHP warning.

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)) {
+        while ($row = Database::fetch_array($result)) {
-            $courseCode = $row[0];
+            $courseInfo = api_get_course_info_by_id($row['c_id']);
-            $courseInfo = api_get_course_info($courseCode);
+            if (empty($courseInfo)) {
+                continue;
+            }
+
+            $courseCode = $courseInfo['code'];
             $courseId = $courseInfo['real_id'];
 
             $return .= '<tr>';

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 .= '<table  class="data_table">';
@@ -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 = '-';
 		}

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();
 }
-</script>	';
+</script>';
 
 $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'];
 }