diff --git a/main/inc/lib/course_home.lib.php b/main/inc/lib/course_home.lib.php index 01565dc984..cb5581ce1a 100755 --- a/main/inc/lib/course_home.lib.php +++ b/main/inc/lib/course_home.lib.php @@ -1090,18 +1090,18 @@ class CourseHome * @param bool $include_admin_tools * @return array */ - static function get_navigation_items($include_admin_tools = false) + public static function get_navigation_items($include_admin_tools = false) { $navigation_items = array(); $course_id = api_get_course_int_id(); $courseInfo = api_get_course_info(); + $sessionId = api_get_session_id(); if (!empty($course_id)) { $course_tools_table = Database :: get_course_table(TABLE_TOOL_LIST); /* Link to the Course homepage */ - $navigation_items['home']['image'] = 'home.gif'; $navigation_items['home']['link'] = $courseInfo['course_public_url']; $navigation_items['home']['name'] = get_lang('CourseHomepageLink'); @@ -1123,12 +1123,12 @@ class CourseHome - Course rights (roles & rights overview) */ if ($include_admin_tools) { - $course_settings_sql = "SELECT name,image FROM $course_tools_table - WHERE c_id = $course_id AND link='course_info/infocours.php'"; - $sql_result = Database::query($course_settings_sql); + $sql = "SELECT name,image FROM $course_tools_table + WHERE c_id = $course_id AND link='course_info/infocours.php'"; + $sql_result = Database::query($sql); $course_setting_info = Database::fetch_array($sql_result); $course_setting_visual_name = CourseHome::translate_tool_name($course_setting_info); - if (api_get_session_id() == 0) { + if ($sessionId == 0) { // course settings item $navigation_items['course_settings']['image'] = $course_setting_info['image']; $navigation_items['course_settings']['link'] = api_get_path(WEB_CODE_PATH).'course_info/infocours.php'; @@ -1146,7 +1146,7 @@ class CourseHome $parameter_separator = '?'; } //$navigation_items[$key]['link'] .= $parameter_separator.api_get_cidreq(); - $navigation_items[$key]['link'] .= $parameter_separator.'cidReq='.api_get_course_id().'&gidReq=0&id_session='.api_get_session_id(); + $navigation_items[$key]['link'] .= $parameter_separator.'cidReq='.api_get_course_id().'&gidReq=0&id_session='.$sessionId; } return $navigation_items; @@ -1213,9 +1213,9 @@ class CourseHome $navigation_items = self::get_navigation_items(false); $html = ''; if (!empty($navigation_items)) { - if ($orientation == SHORTCUTS_HORIZONTAL) + if ($orientation == SHORTCUTS_HORIZONTAL) { $style_id = "toolshortcuts_horizontal"; - else { + } else { $style_id = "toolshortcuts_vertical"; } $html .= '