Minor - format code

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 .= '<div id="'.$style_id.'">';