From 4a57d5c7542922598a70bf650feeaae310ad6526 Mon Sep 17 00:00:00 2001 From: jmontoyaa Date: Wed, 9 May 2018 13:07:55 +0200 Subject: [PATCH] Minor - flint fixes --- main/inc/lib/api.lib.php | 3 +-- whoisonlinesession.php | 3 +++ 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/main/inc/lib/api.lib.php b/main/inc/lib/api.lib.php index e26f9d61bf..c8471bd0b7 100644 --- a/main/inc/lib/api.lib.php +++ b/main/inc/lib/api.lib.php @@ -5272,7 +5272,6 @@ function copyr($source, $dest, $exclude = [], $copied_files = []) } /** - * * @todo: Using DIRECTORY_SEPARATOR is not recommended, this is an obsolete approach. * Documentation header to be added here. * @@ -5281,7 +5280,7 @@ function copyr($source, $dest, $exclude = [], $copied_files = []) * @param int $session_id * * @return mixed True if directory already exists, false if a file already exists at - * the destination and null if everything goes according to plan + * the destination and null if everything goes according to plan */ function copy_folder_course_session( $pathname, diff --git a/whoisonlinesession.php b/whoisonlinesession.php index 3b67d4910b..7023b1a78d 100755 --- a/whoisonlinesession.php +++ b/whoisonlinesession.php @@ -76,6 +76,7 @@ function getUsers( $urlCondition"; $result = Database::query($sql); $result = Database::fetch_array($result); + return $result['count']; } @@ -98,6 +99,7 @@ function getUsers( while ($user_list = Database::fetch_array($result)) { $studentsOnline[$user_list['login_user_id']] = $user_list; } + return $studentsOnline; } @@ -131,6 +133,7 @@ function course_filter($courseId, $urlParams, $row) { $sessionId = api_get_session_id(); $courseInfo = api_get_course_info_by_id($courseId); + return Display::url( $courseInfo['title'], $courseInfo['course_public_url'].'?id_session='.$sessionId,