Browse Source

Minor - format code, remove unused code, add int casting

Julio 6 years ago
parent
commit
2f58153199
2 changed files with 36 additions and 616 deletions
  1. 9 372
      main/inc/lib/course.lib.php
  2. 27 244
      main/inc/lib/sessionmanager.lib.php

+ 9 - 372
main/inc/lib/course.lib.php

@@ -1071,11 +1071,10 @@ class CourseManager
         $session_id = 0
     ) {
         $user_id = (int) $user_id;
+        $session_id = (int) $session_id;
 
         if (empty($session_id)) {
             $session_id = api_get_session_id();
-        } else {
-            $session_id = (int) $session_id;
         }
 
         $condition_course = '';
@@ -1108,21 +1107,21 @@ class CourseManager
 
         $tableSessionCourseUser = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
         $sql = 'SELECT 1 FROM '.$tableSessionCourseUser.'
-            WHERE user_id = '.$user_id.' AND session_id = '.$session_id.' '.$condition_course;
+                WHERE user_id = '.$user_id.' AND session_id = '.$session_id.' '.$condition_course;
 
         if (Database::num_rows(Database::query($sql)) > 0) {
             return true;
         }
 
         $sql = 'SELECT 1 FROM '.$tableSessionCourseUser.'
-            WHERE user_id = '.$user_id.' AND session_id = '.$session_id.' AND status = 2 '.$condition_course;
+                WHERE user_id = '.$user_id.' AND session_id = '.$session_id.' AND status = 2 '.$condition_course;
 
         if (Database::num_rows(Database::query($sql)) > 0) {
             return true;
         }
 
         $sql = 'SELECT 1 FROM '.Database::get_main_table(TABLE_MAIN_SESSION).
-            ' WHERE id = '.$session_id.' AND id_coach='.$user_id;
+              ' WHERE id = '.$session_id.' AND id_coach='.$user_id;
 
         if (Database::num_rows(Database::query($sql)) > 0) {
             return true;
@@ -1679,7 +1678,7 @@ class CourseManager
         $status = null
     ) {
         // variable initialisation
-        $session_id = intval($session_id);
+        $session_id = (int) $session_id;
         $course_code = Database::escape_string($course_code);
         $tblUser = Database::get_main_table(TABLE_MAIN_USER);
         $tblSessionCourseUser = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
@@ -1755,7 +1754,7 @@ class CourseManager
         $course_code = Database::escape_string($course_code);
         $courseInfo = api_get_course_info($course_code);
         $courseId = $courseInfo['real_id'];
-        $session_id = intval($session_id);
+        $session_id = (int) $session_id;
         $users = [];
 
         // We get the coach for the given course in a given session.
@@ -3052,27 +3051,6 @@ class CourseManager
         return $extraField->save($params);
     }
 
-    /**
-     * Updates course attribute. Note that you need to check that your
-     * attribute is valid before you use this function.
-     *
-     * @param int    $id    Course id
-     * @param string $name  Attribute name
-     * @param string $value Attribute value
-     *
-     * @return Doctrine\DBAL\Driver\Statement|null True if attribute was successfully updated,
-     *                                             false if course was not found or attribute name is invalid
-     */
-    public static function update_attribute($id, $name, $value)
-    {
-        $id = (int) $id;
-        $table = Database::get_main_table(TABLE_MAIN_COURSE);
-        $sql = "UPDATE $table SET $name = '".Database::escape_string($value)."'
-                WHERE id = $id";
-
-        return Database::query($sql);
-    }
-
     /**
      * Update course attributes. Will only update attributes with a non-empty value.
      * Note that you NEED to check that your attributes are valid before using this function.
@@ -3559,89 +3537,6 @@ class CourseManager
         }
     }
 
-    /**
-     * Builds the course block in user_portal.php.
-     *
-     * @todo use Twig
-     *
-     * @param array $params
-     *
-     * @return string
-     */
-    public static function course_item_html_no_icon($params)
-    {
-        $html = '<div class="course_item">';
-        $html .= '<div class="row">';
-        $html .= '<div class="col-md-7">';
-
-        $notifications = isset($params['notifications']) ? $params['notifications'] : null;
-
-        $html .= '<h3>'.$params['title'].$notifications.'</h3> ';
-
-        if (isset($params['description'])) {
-            $html .= '<p>'.$params['description'].'</p>';
-        }
-        if (!empty($params['subtitle'])) {
-            $html .= '<small>'.$params['subtitle'].'</small>';
-        }
-        if (!empty($params['teachers'])) {
-            $html .= '<h5 class="teacher">'.
-                Display::return_icon(
-                    'teacher.png',
-                    get_lang('Teacher'),
-                    [],
-                    ICON_SIZE_TINY
-                ).
-                $params['teachers'].'</h5>';
-        }
-        if (!empty($params['coaches'])) {
-            $html .= '<h5 class="teacher">'.
-                Display::return_icon(
-                    'teacher.png',
-                    get_lang('Coach'),
-                    [],
-                    ICON_SIZE_TINY
-                ).
-                $params['coaches'].'</h5>';
-        }
-
-        $html .= '</div>';
-        $params['right_actions'] = isset($params['right_actions']) ? $params['right_actions'] : null;
-        $html .= '<div class="pull-right course-box-actions">'.$params['right_actions'].'</div>';
-        $html .= '</div>';
-        $html .= '</div>';
-
-        return $html;
-    }
-
-    /**
-     * @param $params
-     * @param bool|false $is_sub_content
-     *
-     * @return string
-     */
-    public static function session_items_html($params, $is_sub_content = false)
-    {
-        $html = '';
-        $html .= '<div class="row">';
-        $html .= '<div class="col-md-2">';
-        if (!empty($params['link'])) {
-            $html .= '<a class="thumbnail" href="'.$params['link'].'">';
-            $html .= $params['icon'];
-            $html .= '</a>';
-        } else {
-            $html .= $params['icon'];
-        }
-        $html .= '</div>';
-        $html .= '<div class="col-md-10">';
-        $html .= $params['title'];
-        $html .= $params['coaches'];
-        $html .= '</div>';
-        $html .= '</div>';
-
-        return $html;
-    }
-
     /**
      * Display special courses (and only these) as several HTML divs of class userportal-course-item.
      *
@@ -3659,7 +3554,7 @@ class CourseManager
         $load_dirs = false,
         $useUserLanguageFilterIfAvailable = true
     ) {
-        $user_id = intval($user_id);
+        $user_id = (int) $user_id;
         $table = Database::get_main_table(TABLE_MAIN_COURSE);
         $specialCourseList = self::get_special_course_list();
 
@@ -4061,150 +3956,6 @@ class CourseManager
         return $courseList;
     }
 
-    /**
-     *  Display courses inside a category (without special courses) as HTML dics of
-     *  class userportal-course-item.
-     *
-     * @deprecated use self::returnCoursesCategories(0);
-     *
-     * @param int      User category id
-     * @param bool      Whether to show the document quick-loader or not
-     *
-     * @return array
-     */
-    public static function returnCoursesWithoutCategories($user_category_id, $load_dirs = false)
-    {
-        $user_id = api_get_user_id();
-        // Table definitions
-        $TABLECOURS = Database::get_main_table(TABLE_MAIN_COURSE);
-        $TABLECOURSUSER = Database::get_main_table(TABLE_MAIN_COURSE_USER);
-        $TABLE_ACCESS_URL_REL_COURSE = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_COURSE);
-        $current_url_id = api_get_current_access_url_id();
-        $courseList = [];
-
-        // Get course list auto-register
-        $special_course_list = self::get_special_course_list();
-
-        $without_special_courses = '';
-        if (!empty($special_course_list)) {
-            $without_special_courses = ' AND course.id NOT IN ("'.implode('","', $special_course_list).'")';
-        }
-
-        $sql = "SELECT
-                    course.id,
-                    course.title,
-                    course.code,
-                    course.subscribe subscr,
-                    course.unsubscribe unsubscr,
-                    course_rel_user.status status,
-                    course_rel_user.sort sort,
-                    course_rel_user.user_course_cat user_course_cat
-                FROM $TABLECOURS course 
-                INNER JOIN $TABLECOURSUSER course_rel_user
-                ON (course.id = course_rel_user.c_id)
-                INNER JOIN $TABLE_ACCESS_URL_REL_COURSE url
-                ON (url.c_id = course.id)
-                WHERE
-                    course_rel_user.user_id = $user_id AND
-                    course_rel_user.user_course_cat = $user_category_id
-                    $without_special_courses ";
-
-        // If multiple URL access mode is enabled, only fetch courses
-        // corresponding to the current URL.
-        if (api_get_multiple_access_url() && $current_url_id != -1) {
-            $sql .= " AND url.c_id = course.id AND access_url_id = $current_url_id ";
-        }
-        // Use user's classification for courses (if any).
-        $sql .= " ORDER BY course_rel_user.user_course_cat, course_rel_user.sort ASC";
-        $result = Database::query($sql);
-
-        $showCustomIcon = api_get_setting('course_images_in_courses_list');
-        // Browse through all courses.
-        while ($row = Database::fetch_array($result)) {
-            $course_info = api_get_course_info($row['code']);
-            if (isset($course_info['visibility']) &&
-                $course_info['visibility'] == COURSE_VISIBILITY_HIDDEN
-            ) {
-                continue;
-            }
-            $course_info['id_session'] = null;
-            $course_info['status'] = $row['status'];
-
-            // For each course, get if there is any notification icon to show
-            // (something that would have changed since the user's last visit).
-            $showNotification = !api_get_configuration_value('hide_course_notification')
-                ? Display::show_notification($course_info)
-                : '';
-
-            $thumbnails = null;
-            $image = null;
-            $iconName = basename($course_info['course_image']);
-            if ($showCustomIcon === 'true' && $iconName != 'course.png') {
-                $thumbnails = $course_info['course_image'];
-                $image = $course_info['course_image_large'];
-            } else {
-                $image = Display::return_icon(
-                    'session_default.png',
-                    null,
-                    null,
-                    null,
-                    null,
-                    true
-                );
-            }
-
-            $params = [];
-            //Param (course_code) needed to get the student process
-            $params['course_code'] = $row['code'];
-            $params['edit_actions'] = '';
-            $params['document'] = '';
-            if (api_is_platform_admin()) {
-                $params['edit_actions'] .= api_get_path(WEB_CODE_PATH).'course_info/infocours.php?cidReq='.$course_info['code'];
-                if ($load_dirs) {
-                    $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
-                               .Display::returnFontAwesomeIcon('folder-open').'</a>';
-                    $params['document'] .= Display::div('', ['id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container']);
-                }
-            }
-            if ($load_dirs) {
-                $params['document'] = '<a id="document_preview_'.$course_info['real_id'].'_0" class="document_preview btn btn-default btn-sm" href="javascript:void(0);">'
-                    .Display::returnFontAwesomeIcon('folder-open').'</a>';
-                $params['document'] .= Display::div('', ['id' => 'document_result_'.$course_info['real_id'].'_0', 'class' => 'document_preview_container']);
-            }
-
-            $course_title_url = api_get_path(WEB_COURSE_PATH).$course_info['path'].'/index.php?id_session=0';
-
-            $teachers = [];
-            if (api_get_setting('display_teacher_in_courselist') === 'true') {
-                $teachers = self::getTeachersFromCourse(
-                    $course_info['real_id'],
-                    false
-                );
-            }
-            $params['status'] = $row['status'];
-
-            if (api_get_setting('display_coursecode_in_courselist') == 'true') {
-                $params['code_course'] = '('.$course_info['visual_code'].') ';
-            }
-
-            $params['visibility'] = $course_info['visibility'];
-            $params['link'] = $course_title_url;
-            $params['thumbnails'] = $thumbnails;
-            $params['image'] = $image;
-            $params['title'] = $course_info['title'];
-            $params['category'] = $course_info['categoryName'];
-            $params['teachers'] = $teachers;
-
-            if ($course_info['visibility'] != COURSE_VISIBILITY_CLOSED) {
-                $params['notifications'] = $showNotification;
-            }
-
-            $courseList[] = $params;
-        }
-
-        return $courseList;
-    }
-
     /**
      * Retrieves the user defined course categories.
      *
@@ -5214,27 +4965,6 @@ class CourseManager
         return $hotCourses;
     }
 
-    /**
-     * @param int $limit
-     *
-     * @return array
-     */
-    public static function return_most_accessed_courses($limit = 5)
-    {
-        $table = Database::get_main_table(TABLE_STATISTIC_TRACK_COURSE_RANKING);
-        $params['url_id'] = api_get_current_access_url_id();
-
-        $result = Database::select(
-            'c_id, accesses, total_score, users',
-            $table,
-            ['where' => ['url_id = ?' => $params], 'order' => 'accesses DESC', 'limit' => $limit],
-            'all',
-            true
-        );
-
-        return $result;
-    }
-
     /**
      * Get courses count.
      *
@@ -5697,7 +5427,7 @@ class CourseManager
     public static function hasCourseSetting($variable, $courseId)
     {
         $courseSetting = Database::get_course_table(TABLE_COURSE_SETTING);
-        $courseId = intval($courseId);
+        $courseId = (int) $courseId;
         $variable = Database::escape_string($variable);
         $sql = "SELECT variable FROM $courseSetting
                 WHERE c_id = $courseId AND variable = '$variable'";
@@ -5706,34 +5436,6 @@ class CourseManager
         return Database::num_rows($result) > 0;
     }
 
-    /**
-     * Get information from the track_e_course_access table.
-     *
-     * @param int $sessionId
-     * @param int $userId
-     * @param int $limit
-     *
-     * @return array
-     */
-    public static function getCourseAccessPerSessionAndUser($sessionId, $userId, $limit = 0)
-    {
-        $table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
-
-        $sessionId = intval($sessionId);
-        $userId = intval($userId);
-
-        $sql = "SELECT * FROM $table
-                WHERE session_id = $sessionId AND user_id = $userId";
-
-        if (!empty($limit)) {
-            $limit = intval($limit);
-            $sql .= " LIMIT $limit";
-        }
-        $result = Database::query($sql);
-
-        return Database::store_result($result);
-    }
-
     /**
      * Get information from the track_e_course_access table.
      *
@@ -5939,27 +5641,6 @@ class CourseManager
         );
     }
 
-    /**
-     * @param $courseId
-     *
-     * @return array
-     */
-    public static function getCourseSettings($courseId)
-    {
-        $settingTable = Database::get_course_table(TABLE_COURSE_SETTING);
-        $courseId = intval($courseId);
-        $sql = "SELECT * FROM $settingTable WHERE c_id = $courseId";
-        $result = Database::query($sql);
-        $settings = [];
-        if (Database::num_rows($result)) {
-            while ($row = Database::fetch_array($result, 'ASSOC')) {
-                $settings[$row['variable']] = $row;
-            }
-        }
-
-        return $settings;
-    }
-
     /**
      * this function gets all the users of the course,
      * including users from linked courses.
@@ -6191,7 +5872,7 @@ class CourseManager
      */
     public static function get_course_by_category($category_id)
     {
-        $category_id = intval($category_id);
+        $category_id = (int) $category_id;
         $info = Database::fetch_array(
             Database::query(
                 'SELECT course_code 
@@ -6722,50 +6403,6 @@ class CourseManager
         return api_get_path(WEB_COURSE_PATH).$course->getDirectory().'/course-pic85x85.png';
     }
 
-    /**
-     * @return int
-     */
-    public static function getCountOpenCourses()
-    {
-        $visibility = [
-            COURSE_VISIBILITY_REGISTERED,
-            COURSE_VISIBILITY_OPEN_PLATFORM,
-            COURSE_VISIBILITY_OPEN_WORLD,
-        ];
-
-        $table = Database::get_main_table(TABLE_MAIN_COURSE);
-        $sql = "SELECT count(id) count 
-                FROM $table 
-                WHERE visibility IN (".implode(',', $visibility).")";
-        $result = Database::query($sql);
-        $row = Database::fetch_array($result);
-
-        return (int) $row['count'];
-    }
-
-    /**
-     * @return int
-     */
-    public static function getCountExercisesFromOpenCourse()
-    {
-        $visibility = [
-            COURSE_VISIBILITY_REGISTERED,
-            COURSE_VISIBILITY_OPEN_PLATFORM,
-            COURSE_VISIBILITY_OPEN_WORLD,
-        ];
-
-        $table = Database::get_main_table(TABLE_MAIN_COURSE);
-        $tableExercise = Database::get_course_table(TABLE_QUIZ_TEST);
-        $sql = "SELECT count(e.iid) count 
-                FROM $table c INNER JOIN $tableExercise e
-                ON (c.id = e.c_id)
-                WHERE e.active <> -1 AND visibility IN (".implode(',', $visibility).")";
-        $result = Database::query($sql);
-        $row = Database::fetch_array($result);
-
-        return (int) $row['count'];
-    }
-
     /**
      * Check if a specific access-url-related setting is a problem or not.
      *

+ 27 - 244
main/inc/lib/sessionmanager.lib.php

@@ -727,35 +727,6 @@ class SessionManager
         return $formattedSessions;
     }
 
-    /**
-     *  Get total of records for progress of learning paths in the given session.
-     *
-     *  @param int session id
-     *
-     *  @return int
-     */
-    public static function get_count_session_lp_progress($sessionId = 0)
-    {
-        $tbl_lp = Database::get_course_table(TABLE_LP_MAIN);
-        $tbl_lp_view = Database::get_course_table(TABLE_LP_VIEW);
-        $tbl_user = Database::get_main_table(TABLE_MAIN_USER);
-        $tbl_course = Database::get_main_table(TABLE_MAIN_COURSE);
-
-        $sessionId = intval($sessionId);
-
-        $sql = "SELECT  count(*) as total_rows
-                FROM $tbl_lp_view v
-                INNER JOIN $tbl_lp l ON l.id = v.lp_id
-                INNER JOIN $tbl_user u ON u.user_id = v.user_id
-                INNER JOIN $tbl_course c
-                WHERE v.session_id = ".$sessionId;
-        $result_rows = Database::query($sql);
-        $row = Database::fetch_array($result_rows);
-        $num = $row['total_rows'];
-
-        return $num;
-    }
-
     /**
      * Gets the progress of learning paths in the given session.
      *
@@ -777,8 +748,6 @@ class SessionManager
         //escaping vars
         $sessionId = $sessionId == 'T' ? 'T' : intval($sessionId);
         $courseId = intval($courseId);
-        $date_from = Database::escape_string($date_from);
-        $date_to = Database::escape_string($date_to);
 
         //tables
         $session_course_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
@@ -786,18 +755,6 @@ class SessionManager
         $tbl_course_lp_view = Database::get_course_table(TABLE_LP_VIEW);
 
         $course = api_get_course_info_by_id($courseId);
-
-        //getting all the students of the course
-        //we are not using this because it only returns user ids
-        /* if (empty($sessionId)
-          {
-          // Registered students in a course outside session.
-          $users = CourseManager::get_student_list_from_course_code($course_code);
-          } else {
-          // Registered students in session.
-          $users = CourseManager::get_student_list_from_course_code($course_code, true, $sessionId);
-          } */
-
         $sessionCond = 'and session_id = %s';
         if ($sessionId == 'T') {
             $sessionCond = '';
@@ -2136,7 +2093,7 @@ class SessionManager
         $courseInfo,
         $status = null
     ) {
-        $sessionId = intval($sessionId);
+        $sessionId = (int) $sessionId;
         $courseId = $courseInfo['real_id'];
 
         if (empty($sessionId) || empty($courseId)) {
@@ -2145,7 +2102,7 @@ class SessionManager
 
         $statusCondition = null;
         if (isset($status) && !is_null($status)) {
-            $status = intval($status);
+            $status = (int) $status;
             $statusCondition = " AND status = $status";
         }
 
@@ -2526,7 +2483,7 @@ class SessionManager
         $removeExistingCoursesWithUsers = true,
         $copyEvaluation = false
     ) {
-        $sessionId = intval($sessionId);
+        $sessionId = (int) $sessionId;
 
         if (empty($sessionId) || empty($courseList)) {
             return false;
@@ -2796,7 +2753,6 @@ class SessionManager
 
         // Get course code
         $course_code = CourseManager::get_course_code_from_course_id($course_id);
-        $course_id = intval($course_id);
 
         if (empty($course_code)) {
             return false;
@@ -2944,6 +2900,8 @@ class SessionManager
     {
         $table = Database::get_main_table(TABLE_MAIN_SESSION);
         $name = Database::escape_string(trim($name));
+        $sessionId = (int) $sessionId;
+
         if (empty($name)) {
             return false;
         }
@@ -3295,7 +3253,7 @@ class SessionManager
     public static function get_session_category($id)
     {
         $table = Database::get_main_table(TABLE_MAIN_SESSION_CATEGORY);
-        $id = intval($id);
+        $id = (int) $id;
         $sql = "SELECT id, name, date_start, date_end
                 FROM $table
                 WHERE id= $id";
@@ -3315,8 +3273,9 @@ class SessionManager
      */
     public static function getSessionImage($id)
     {
+        $id = (int) $id;
         $extraFieldValuesTable = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES);
-        $sql = "SELECT value  FROM extra_field_values WHERE field_id = 16 AND item_id = ".intval($id);
+        $sql = "SELECT value  FROM $extraFieldValuesTable WHERE field_id = 16 AND item_id = ".$id;
         $result = Database::query($sql);
         if (Database::num_rows($result) > 0) {
             while ($row = Database::fetch_array($result, 'ASSOC')) {
@@ -3414,9 +3373,9 @@ class SessionManager
             $data = Database::store_result($result, 'ASSOC');
 
             return $data;
-        } else {
-            return false;
         }
+
+        return false;
     }
 
     /**
@@ -3668,8 +3627,8 @@ class SessionManager
         $tbl_session_rel_user = Database::get_main_table(TABLE_MAIN_SESSION_USER);
         $tbl_session_rel_access_url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
 
-        $userId = intval($userId);
-        $sessionId = intval($sessionId);
+        $userId = (int) $userId;
+        $sessionId = (int) $sessionId;
 
         $select = " SELECT * ";
         if (api_is_multiple_url_enabled()) {
@@ -3681,7 +3640,7 @@ class SessionManager
                         sru.session_id = '$sessionId' AND
                         sru.relation_type = '".SESSION_RELATION_TYPE_RRHH."' AND
                         access_url_id = ".api_get_current_access_url_id()."
-                        ";
+                    ";
         } else {
             $sql = "$select FROM $tbl_session s
                      INNER JOIN $tbl_session_rel_user sru
@@ -4090,7 +4049,7 @@ class SessionManager
         $tbl_course = Database::get_main_table(TABLE_MAIN_COURSE);
         $tbl_session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
 
-        $session_id = intval($session_id);
+        $session_id = (int) $session_id;
         $course_name = Database::escape_string($course_name);
 
         // select the courses
@@ -4132,7 +4091,7 @@ class SessionManager
     {
         $tbl_course = Database::get_main_table(TABLE_MAIN_COURSE);
         $tbl_session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
-        $session_id = intval($session_id);
+        $session_id = (int) $session_id;
 
         // select the courses
         $sql = "SELECT COUNT(c.code) count
@@ -4263,7 +4222,7 @@ class SessionManager
     public static function get_sessions_by_general_coach($user_id, $asPlatformAdmin = false)
     {
         $session_table = Database::get_main_table(TABLE_MAIN_SESSION);
-        $user_id = intval($user_id);
+        $user_id = (int) $user_id;
 
         // Session where we are general coach
         $sql = "SELECT DISTINCT *
@@ -4398,7 +4357,7 @@ class SessionManager
         $params['promotion_id'] = $promotion_id;
         if (!empty($list)) {
             foreach ($list as $session_id) {
-                $session_id = intval($session_id);
+                $session_id = (int) $session_id;
                 Database::update($table, $params, ['id = ?' => $session_id]);
             }
         }
@@ -4769,7 +4728,7 @@ SQL;
         $table_session_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
         $table_session = Database::get_main_table(TABLE_MAIN_SESSION);
         $url = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
-        $courseId = intval($courseId);
+        $courseId = (int) $courseId;
         $urlId = api_get_current_access_url_id();
 
         if (empty($courseId)) {
@@ -5796,8 +5755,8 @@ SQL;
     public static function getCoachesByCourseSession($sessionId, $courseId)
     {
         $table = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
-        $sessionId = intval($sessionId);
-        $courseId = intval($courseId);
+        $sessionId = (int) $sessionId;
+        $courseId = (int) $courseId;
 
         $sql = "SELECT user_id FROM $table
                 WHERE
@@ -6521,64 +6480,6 @@ SQL;
         }
     }
 
-    /**
-     * @param int $sessionId
-     * @param int $courseId
-     */
-    public static function addCourseIntroduction($sessionId, $courseId)
-    {
-        // @todo create a tool intro lib
-        $sessionId = intval($sessionId);
-        $courseId = intval($courseId);
-
-        $TBL_INTRODUCTION = Database::get_course_table(TABLE_TOOL_INTRO);
-        $sql = "SELECT * FROM $TBL_INTRODUCTION WHERE c_id = $courseId";
-        $result = Database::query($sql);
-        $result = Database::store_result($result, 'ASSOC');
-
-        if (!empty($result)) {
-            foreach ($result as $result) {
-                // @todo check if relation exits.
-                $result['session_id'] = $sessionId;
-                Database::insert($TBL_INTRODUCTION, $result);
-            }
-        }
-    }
-
-    /**
-     * @param int $sessionId
-     * @param int $courseId
-     */
-    public static function removeCourseIntroduction($sessionId, $courseId)
-    {
-        $sessionId = intval($sessionId);
-        $courseId = intval($courseId);
-        $TBL_INTRODUCTION = Database::get_course_table(TABLE_TOOL_INTRO);
-        $sql = "DELETE FROM $TBL_INTRODUCTION
-                WHERE c_id = $courseId AND session_id = $sessionId";
-        Database::query($sql);
-    }
-
-    /**
-     * @param int $sessionId
-     * @param int $courseId
-     */
-    public static function addCourseDescription($sessionId, $courseId)
-    {
-        /* $description = new CourseDescription();
-          $descriptions = $description->get_descriptions($courseId);
-          foreach ($descriptions as $description) {
-          } */
-    }
-
-    /**
-     * @param int $sessionId
-     * @param int $courseId
-     */
-    public static function removeCourseDescription($sessionId, $courseId)
-    {
-    }
-
     /**
      * @param array $userSessionList        format see self::importSessionDrhCSV()
      * @param bool  $sendEmail
@@ -7034,7 +6935,7 @@ SQL;
      */
     public static function sessionHasCourse($sessionId, $courseCode)
     {
-        $sessionId = intval($sessionId);
+        $sessionId = (int) $sessionId;
         $courseCode = Database::escape_string($courseCode);
         $courseTable = Database::get_main_table(TABLE_MAIN_COURSE);
         $sessionRelCourseTable = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
@@ -7059,55 +6960,6 @@ SQL;
         return false;
     }
 
-    /**
-     * Get the list of course coaches.
-     *
-     * @return array The list
-     */
-    public static function getAllCourseCoaches()
-    {
-        $coaches = [];
-
-        $scuTable = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
-        $userTable = Database::get_main_table(TABLE_MAIN_USER);
-
-        $idResult = Database::select('DISTINCT user_id', $scuTable, [
-            'where' => [
-                'status = ?' => 2,
-            ],
-        ]);
-
-        if ($idResult != false) {
-            foreach ($idResult as $idData) {
-                $userResult = Database::select(
-                    'user_id, lastname, firstname, username',
-                    $userTable,
-                    [
-                        'where' => [
-                            'user_id = ?' => $idData['user_id'],
-                        ],
-                    ],
-                    'first'
-                );
-
-                if ($userResult != false) {
-                    $coaches[] = [
-                        'id' => $userResult['user_id'],
-                        'lastname' => $userResult['lastname'],
-                        'firstname' => $userResult['firstname'],
-                        'username' => $userResult['username'],
-                        'completeName' => api_get_person_name(
-                            $userResult['firstname'],
-                            $userResult['lastname']
-                        ),
-                    ];
-                }
-            }
-        }
-
-        return $coaches;
-    }
-
     /**
      * Calculate the total user time in the platform.
      *
@@ -7119,7 +6971,7 @@ SQL;
      */
     public static function getTotalUserTimeInPlatform($userId, $from = '', $until = '')
     {
-        $userId = intval($userId);
+        $userId = (int) $userId;
         $trackLoginTable = Database::get_main_table(TABLE_STATISTIC_TRACK_E_LOGIN);
         $whereConditions = [
             'login_user_id = ? ' => $userId,
@@ -7179,6 +7031,8 @@ SQL;
         $tableUser = Database::get_main_table(TABLE_MAIN_USER);
         $table = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
 
+        $sessionId = (int) $sessionId;
+
         if (empty($sessionId)) {
             return [];
         }
@@ -7199,7 +7053,7 @@ SQL;
                 ON scu.user_id = u.id
                 WHERE 
                   $courseCondition
-                  scu.session_id = ".intval($sessionId)."
+                  scu.session_id = ".$sessionId."
                 GROUP BY u.id";
 
         $result = Database::query($sql);
@@ -7517,7 +7371,7 @@ SQL;
     public static function getDescriptionFromSessionId($sessionId)
     {
         // Init variables
-        $sessionId = intval($sessionId);
+        $sessionId = (int) $sessionId;
         $description = '';
         // Check if session id is valid
         if ($sessionId > 0) {
@@ -8980,77 +8834,6 @@ SQL;
         return $listResults;
     }
 
-    /**
-     * Return HTML code for displaying session_course_for_coach.
-     *
-     * @param $userId
-     *
-     * @return string
-     */
-    public static function getHtmlNamedSessionCourseForCoach($userId)
-    {
-        $htmlRes = '';
-        $listInfo = self::getNamedSessionCourseForCoach($userId);
-        foreach ($listInfo as $i => $listCoursesInfo) {
-            $courseInfo = $listCoursesInfo['course'];
-            $courseCode = $listCoursesInfo['course']['code'];
-
-            $listParamsCourse = [];
-            $listParamsCourse['icon'] = '<div style="float:left">
-                <input style="border:none;" type="button" onclick="$(\'#course-'.$courseCode.'\').toggle(\'fast\')" value="+" /></div>'.
-                Display::return_icon('blackboard.png', $courseInfo['title'], [], ICON_SIZE_LARGE);
-            $listParamsCourse['link'] = '';
-            $listParamsCourse['title'] = Display::tag(
-                'a',
-                $courseInfo['title'],
-                ['href' => $listParamsCourse['link']]
-            );
-            $htmlCourse = '<div class="well" style="border-color:#27587D">'.
-                CourseManager::course_item_html($listParamsCourse, true);
-            // for each category of session
-            $htmlCatSessions = '';
-            foreach ($listCoursesInfo['sessionCatList'] as $j => $listCatSessionsInfo) {
-                // we got an array of session categories
-                $catSessionId = $listCoursesInfo['sessionCatList'][$j]['catSessionId'];
-                $catSessionName = $listCoursesInfo['sessionCatList'][$j]['catSessionName'];
-
-                $listParamsCatSession['icon'] = Display::return_icon('folder_blue.png', $catSessionName, [], ICON_SIZE_LARGE);
-                $listParamsCatSession['link'] = '';
-                $listParamsCatSession['title'] = $catSessionName;
-
-                $marginShift = 20;
-                if ($catSessionName != '') {
-                    $htmlCatSessions .= '<div style="margin-left:'.$marginShift.'px;">'.
-                        CourseManager::course_item_html($listParamsCatSession, true).'</div>';
-                    $marginShift = 40;
-                }
-
-                // for each sessions
-                $listCatSessionSessionList = $listCoursesInfo['sessionCatList'][$j]['sessionList'];
-                $htmlSession = '';
-                foreach ($listCatSessionSessionList as $k => $listSessionInfo) {
-                    // we got an array of session info
-                    $sessionId = $listSessionInfo['sessionId'];
-                    $sessionName = $listSessionInfo['sessionName'];
-
-                    $listParamsSession['icon'] = Display::return_icon('blackboard_blue.png', $sessionName, [], ICON_SIZE_LARGE);
-                    $listParamsSession['link'] = '';
-                    $linkToCourseSession = $courseInfo['course_public_url'].'?id_session='.$sessionId;
-                    $listParamsSession['title'] =
-                        $sessionName.'<div style="font-weight:normal; font-style:italic">
-                            <a href="'.$linkToCourseSession.'">'.get_lang('GoToCourseInsideSession').'</a>
-                            </div>';
-                    $htmlSession .= '<div style="margin-left:'.$marginShift.'px;">'.
-                        CourseManager::course_item_html($listParamsSession, true).'</div>';
-                }
-                $htmlCatSessions .= $htmlSession;
-            }
-            $htmlRes .= $htmlCourse.'<div style="display:none" id="course-'.$courseCode.'">'.$htmlCatSessions.'</div></div>';
-        }
-
-        return $htmlRes;
-    }
-
     /**
      * @param int $userId
      * @param int $courseId
@@ -9078,7 +8861,7 @@ SQL;
      */
     public static function redirectToSession()
     {
-        $sessionId = ChamiloSession::read('session_redirect');
+        $sessionId = (int) ChamiloSession::read('session_redirect');
         $onlyOneCourseSessionToRedirect = ChamiloSession::read('only_one_course_session_redirect');
         if ($sessionId) {
             $sessionInfo = api_get_session_info($sessionId);