Skip to content

Commit d697192

Browse files
committed
Check access to "who is online in session"
bd93988#commitcomment-28898680
1 parent 1ccb779 commit d697192

File tree

1 file changed

+11
-7
lines changed

1 file changed

+11
-7
lines changed

main/inc/lib/banner.lib.php

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -248,13 +248,17 @@ function returnNotificationMenu()
248248
.' '.$number_online_in_course.' </a></li>';
249249
}
250250

251-
if (isset($user_id) && $sessionId != 0) {
252-
$numberOnlineInSession = getOnlineUsersInSessionCount($sessionId);
253-
254-
$html .= '<li class="user-online-session"><a href="'.api_get_path(WEB_PATH).'whoisonlinesession.php?id_coach='.$user_id.'" target="_self">'
255-
.Display::return_icon('session.png', get_lang('UsersConnectedToMySessions'), [], ICON_SIZE_TINY)
256-
.' '.$numberOnlineInSession
257-
.'</a></li>';
251+
if (!empty($sessionId)) {
252+
$allow = api_is_platform_admin(true) ||
253+
api_is_coach($sessionId, null, false) ||
254+
SessionManager::isUserSubscribedAsStudent($sessionId, api_get_user_id());
255+
if ($allow) {
256+
$numberOnlineInSession = getOnlineUsersInSessionCount($sessionId);
257+
$html .= '<li class="user-online-session">
258+
<a href="'.api_get_path(WEB_PATH).'whoisonlinesession.php" target="_self">'
259+
.Display::return_icon('session.png', get_lang('UsersConnectedToMySessions'), [], ICON_SIZE_TINY)
260+
.' '.$numberOnlineInSession.'</a></li>';
261+
}
258262
}
259263
}
260264

0 commit comments

Comments
 (0)