diff --git a/public/main/user/user.php b/public/main/user/user.php index 8055f55a67e..41327f33fb5 100644 --- a/public/main/user/user.php +++ b/public/main/user/user.php @@ -1025,91 +1025,77 @@ function active_filter($active, $urlParams, $row) */ function modify_filter($user_id, $row, $data) { - $canEditUsers = 'true' == api_get_setting('allow_user_course_subscription_by_course_admin') || api_is_platform_admin(); - + $canEditUsers = 'true' === api_get_setting('allow_user_course_subscription_by_course_admin') || api_is_platform_admin(); $is_allowed_to_track = api_is_allowed_to_edit(true, true); $user_id = $data[0]; $userInfo = api_get_user_info($user_id); - $isInvitee = INVITEE == $userInfo['status'] ? true : false; + $isInvitee = INVITEE == $userInfo['status']; $course_info = $_course = api_get_course_info(); $current_user_id = api_get_user_id(); $sessionId = api_get_session_id(); $courseId = $_course['id']; $type = isset($_REQUEST['type']) ? intval($_REQUEST['type']) : STUDENT; - $result = ''; + $result = '
'; + + // Reporting if ($is_allowed_to_track) { - $result .= ' - '.Display::getMdiIcon('statistics', 'ch-tool-icon', null, ICON_SIZE_SMALL, get_lang('Reporting')).' - '; + $result .= Display::url( + Display::getMdiIcon(ActionIcon::VIEW_DETAILS, 'ch-tool-icon', null, ICON_SIZE_MEDIUM, get_lang('Reporting')), + '../my_space/myStudents.php?'.api_get_cidreq().'&student='.$user_id.'&details=true&course='.$courseId.'&origin=user_course&id_session='.api_get_session_id() + ); } - // If platform admin, show the login_as icon (this drastically shortens - // time taken by support to test things out) + // Login as if (api_is_platform_admin()) { - $result .= ' '. - Display::getMdiIcon('login_as', 'ch-tool-icon', null, ICON_SIZE_SMALL, get_lang('Login as')).'  '; + $result .= Display::url( + Display::getMdiIcon(ActionIcon::LOGIN_AS, 'ch-tool-icon', null, ICON_SIZE_MEDIUM, get_lang('Login as')), + api_get_path(WEB_PATH).'?_switch_user='.$userInfo['username'] + ); } if (api_is_allowed_to_edit(null, true)) { if (empty($sessionId)) { $isTutor = isset($data['is_tutor']) ? (int) $data['is_tutor'] : 0; $isTutor = empty($isTutor) ? 1 : 0; + $text = $isTutor ? get_lang('Convert to assistant') : get_lang('Remove assistant role'); + $disabled = $isInvitee ? 'disabled' : ''; - $text = get_lang('Remove assistant role'); - if ($isTutor) { - $text = get_lang('Convert to assistant'); - } - - if ($isInvitee) { - $disabled = 'disabled'; - } else { - $disabled = ''; - } - - $allow = api_get_configuration_value('extra'); - if ($allow) { - $result .= ''. - Display::getMdiIcon(ActionIcon::EDIT, 'ch-tool-icon', null, ICON_SIZE_SMALL, get_lang('Edit')). - ' '; + // Edit + if (api_get_configuration_value('extra')) { + $result .= Display::url( + Display::getMdiIcon(ActionIcon::EDIT, 'ch-tool-icon', null, ICON_SIZE_MEDIUM, get_lang('Edit')), + api_get_path(WEB_CODE_PATH).'extra/userInfo.php?'.api_get_cidreq().'&editMainUserInfo='.$user_id + ); } + // Convert to assistant / Remove if (STUDENT == $data['user_status_in_course']) { $result .= Display::url( - $text, + Display::getMdiIcon($isTutor ? ActionIcon::ACCEPT : ActionIcon::REJECT, 'ch-tool-icon text-xl', ICON_SIZE_MEDIUM, null, $text), 'user.php?'.api_get_cidreq().'&action=set_tutor&is_tutor='.$isTutor.'&user_id='.$user_id.'&type='.$type, - ['class' => 'btn btn--plain '.$disabled] - ).' '; + ['class' => $disabled] + ); } } - // edit - if ($canEditUsers) { - // unregister - if ($user_id != $current_user_id || api_is_platform_admin()) { - $result .= ''. - get_lang('Unsubscribe').' '; - } + // Unsubscribe + if ($canEditUsers && ($user_id != $current_user_id || api_is_platform_admin())) { + $result .= Display::url( + Display::getMdiIcon(ActionIcon::EXIT, 'ch-tool-icon text-xl delete-swal', null, ICON_SIZE_MEDIUM, get_lang('Unsubscribe')), + api_get_self().'?'.api_get_cidreq().'&type='.$type.'&unregister=yes&user_id='.$user_id + ); } } else { - // Show buttons for unsubscribe - if (1 == $course_info['unsubscribe']) { - if ($user_id == $current_user_id) { - $result .= '