diff --git a/main/inc/lib/social.lib.php b/main/inc/lib/social.lib.php index 9b41041798..84731b8f38 100644 --- a/main/inc/lib/social.lib.php +++ b/main/inc/lib/social.lib.php @@ -794,72 +794,75 @@ class SocialManager extends UserManager /** * Displays a sortable table with the list of online users. - * @param array $user_list + * @param array $user_list The list of users to be shown + * @return string HTML block or null if and ID was defined + * @assert (null) === false */ public static function display_user_list($user_list) { - if (!isset($_GET['id'])) { - $column_size = '9'; - $add_row = false; - if (api_is_anonymous()) { - $column_size = '12'; - $add_row = true; - } + $html = null; + if (isset($_GET['id']) or count($user_list) < 1) { + return false; + } + $column_size = '9'; + $add_row = false; + if (api_is_anonymous()) { + $column_size = '12'; + $add_row = true; + } - $extra_params = array(); - $course_url = ''; - if (isset($_GET['cidReq']) && strlen($_GET['cidReq']) > 0) { - $extra_params['cidReq'] = Security::remove_XSS($_GET['cidReq']); - $course_url = '&cidReq='.Security::remove_XSS($_GET['cidReq']); - } + $extra_params = array(); + $course_url = ''; + if (isset($_GET['cidReq']) && strlen($_GET['cidReq']) > 0) { + $extra_params['cidReq'] = Security::remove_XSS($_GET['cidReq']); + $course_url = '&cidReq='.Security::remove_XSS($_GET['cidReq']); + } - $html = null; - if ($add_row) { - $html .='