Minor - cosmetic changes

main/inc/lib/tracking.lib.php

@@ -1202,22 +1202,14 @@ class Tracking {
 			$tbl_session = Database :: get_main_table(TABLE_MAIN_SESSION);
 
 			// At first, courses where $coach_id is coach of the course //
-
-			/*$sql = 'SELECT 1
-			FROM ' . $tbl_session_course_user . ' AS session_course_user
-			INNER JOIN ' . $tbl_session_course . ' AS session_course
-			ON session_course.course_code = session_course_user.course_code
-			AND id_coach=' . $coach_id . '
-			WHERE id_user=' . $student_id;*/
-
+            
 			$sql = 'SELECT 1 FROM ' . $tbl_session_course_user . ' WHERE id_user=' . $coach_id .' AND status=2';
-
 			$result = Database::query($sql);
 			if (Database::num_rows($result) > 0) {
 				return true;
 			}
 
-			// Then, courses where $coach_id is coach of the session    //
+			// Then, courses where $coach_id is coach of the session
 
 			$sql = 'SELECT session_course_user.id_user
                         FROM ' . $tbl_session_course_user . ' as session_course_user
@@ -1231,12 +1223,9 @@ class Tracking {
 			if (Database::num_rows($result) > 0) {
 				return true;
 			}
-
 			return false;
-
 		}
 
-
 		/**
 		 * Get courses followed by coach
 		 * @param     int        Coach id

main/mySpace/lp_tracking.php

@@ -35,7 +35,7 @@ if (isset($_GET['course'])) {
 $user_infos = UserManager :: get_user_info_by_id($user_id);
 $name = api_get_person_name($user_infos['firstname'], $user_infos['lastname']);
 
-if (!api_is_platform_admin(true) && !CourseManager :: is_course_teacher($_user['user_id'], $cidReq) && !Tracking :: is_allowed_to_coach_student($_user['user_id'],$_GET['student_id']) && !api_is_drh() && !api_is_course_tutor()) {	
+if (!api_is_platform_admin(true) && !CourseManager :: is_course_teacher(api_get_user_id(), $cidReq) && !Tracking :: is_allowed_to_coach_student(api_get_user_id(), $_GET['student_id']) && !api_is_drh() && !api_is_course_tutor()) {	
 	api_not_allowed();	
 }
 $course_exits = CourseManager::course_exists($cidReq);