chamilo
/
chamilo-lms
mirror of https://github.com/chamilo/chamilo-lms/tree/1.11.x
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Minor - format code, remove unused function, variables.

Browse Source
pull/2487/head
jmontoyaa 8 years ago
parent 7b74817091
commit 5a3d83c218
1 changed files with 58 additions and 52 deletions
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Show Stats Download Patch File Download Diff File
  1. 110
      main/inc/lib/sessionmanager.lib.php

110
main/inc/lib/sessionmanager.lib.php
Unescape View File

@ -747,7 +747,8 @@ class SessionManager
AND v.user_id = %d
$sessionCond";
$sql_query = sprintf($sql,
$sql_query = sprintf(
$sql,
intval($courseId),
intval($user['user_id']),
$sessionId
@ -789,19 +790,22 @@ class SessionManager
* @todo fix the query
* @return array table with user name, lp name, progress
*/
public static function get_survey_overview($sessionId = 0, $courseId = 0, $surveyId = 0, $date_from, $date_to, $options)
{
public static function get_survey_overview(
$sessionId = 0,
$courseId = 0,
$surveyId = 0,
$date_from,
$date_to,
$options
) {
//escaping vars
$sessionId = intval($sessionId);
$courseId = intval($courseId);
$surveyId = intval($surveyId);
$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);
$user = Database::get_main_table(TABLE_MAIN_USER);
$tbl_course_lp_view = Database::get_course_table(TABLE_LP_VIEW);
$c_survey = Database::get_course_table(TABLE_SURVEY);
$c_survey_answer = Database::get_course_table(TABLE_SURVEY_ANSWER);
$c_survey_question = Database::get_course_table(TABLE_SURVEY_QUESTION);
@ -899,8 +903,13 @@ class SessionManager
*
* @return array table with user name, lp name, progress
*/
public static function get_session_progress($sessionId, $courseId, $date_from, $date_to, $options)
{
public static function get_session_progress(
$sessionId,
$courseId,
$date_from,
$date_to,
$options
) {
$sessionId = intval($sessionId);
$getAllSessions = false;
@ -981,7 +990,13 @@ class SessionManager
/**
* Exercises
*/
$exercises = ExerciseLib::get_all_exercises($course, $sessionId, false, '', $getAllSessions);
$exercises = ExerciseLib::get_all_exercises(
$course,
$sessionId,
false,
'',
$getAllSessions
);
$exercises_total = count($exercises);
/**
@ -1222,19 +1237,6 @@ class SessionManager
return $table;
}
/**
* @return int
*/
public static function get_number_of_tracking_access_overview()
{
$table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_COURSE_ACCESS);
$sql = "SELECT COUNT(course_access_id) count FROM $table";
$result = Database::query($sql);
$row = Database::fetch_assoc($result);
return $row['count'];
}
/**
* Get the ip, total of clicks, login date and time logged in for all user, in one session
* @todo track_e_course_access table should have ip so we dont have to look for it in track_e_login
@ -1576,10 +1578,10 @@ class SessionManager
/**
* Delete session
* @author Carlos Vargas from existing code
* @param array $id_checked an array to delete sessions
* @param boolean $from_ws optional, true if the function is called
* @param array $id_checked an array to delete sessions
* @param boolean $from_ws optional, true if the function is called
* by a webservice, false otherwise.
* @return void Nothing, or false on error
* @return bool
* */
public static function delete($id_checked, $from_ws = false)
{
@ -1717,7 +1719,8 @@ class SessionManager
}
/**
* Subscribes students to the given session and optionally (default) unsubscribes previous users
* Subscribes students to the given session and optionally (default)
* unsubscribes previous users
*
* @author Carlos Vargas from existing code
* @author Julio Montoya. Cleaning code.
@ -2359,7 +2362,6 @@ class SessionManager
if ($removeExistingCoursesWithUsers === true && !empty($existingCourses)) {
foreach ($existingCourses as $existingCourse) {
if (!in_array($existingCourse['c_id'], $courseList)) {
$sql = "DELETE FROM $tbl_session_rel_course
WHERE
c_id = ".$existingCourse['c_id']." AND
@ -2625,14 +2627,18 @@ class SessionManager
/**
* Creates a new extra field for a given session
* @param string $variable Field's internal variable name
* @param int $fieldType Field's type
* @param string $displayText Field's language var name
* @param string $default Field's default value
* @param string $variable Field's internal variable name
* @param int $fieldType Field's type
* @param string $displayText Field's language var name
* @param string $default Field's default value
* @return int new extra field id
*/
public static function create_session_extra_field($variable, $fieldType, $displayText, $default = '')
{
public static function create_session_extra_field(
$variable,
$fieldType,
$displayText,
$default = ''
) {
$extraField = new ExtraFieldModel('session');
$params = [
'variable' => $variable,
@ -2646,10 +2652,10 @@ class SessionManager
/**
* Update an extra field value for a given session
* @param integer Course ID
* @param string Field variable name
* @param string Field value
* @return boolean true if field updated, false otherwise
* @param integer Course ID
* @param string Field variable name
* @param string Field value
* @return boolean true if field updated, false otherwise
*/
public static function update_session_extra_field_value($sessionId, $variable, $value = '')
{
@ -2712,13 +2718,13 @@ class SessionManager
/**
* Create a session category
* @author Jhon Hinojosa <jhon.hinojosa@dokeos.com>, from existing code
* @param string name
* @param integer year_start
* @param integer month_start
* @param integer day_start
* @param integer year_end
* @param integer month_end
* @param integer day_end
* @param string name
* @param integer year_start
* @param integer month_start
* @param integer day_start
* @param integer year_end
* @param integer month_end
* @param integer day_end
* @return int session ID
* */
public static function create_category_session(
@ -2787,14 +2793,14 @@ class SessionManager
/**
* Edit a sessions category
* @author Jhon Hinojosa <jhon.hinojosa@dokeos.com>,from existing code
* @param integer id
* @param string name
* @param integer year_start
* @param integer month_start
* @param integer day_start
* @param integer year_end
* @param integer month_end
* @param integer day_end
* @param integer id
* @param string name
* @param integer year_start
* @param integer month_start
* @param integer day_start
* @param integer year_end
* @param integer month_end
* @param integer day_end
* @return bool
* The parameter id is a primary key
* */

Write Preview
Loading…
Cancel
Save