diff options
| author | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2024-12-04 08:48:02 +0000 |
|---|---|---|
| committer | Jan-Hendrik Willms <tleilax+studip@gmail.com> | 2024-12-04 10:05:33 +0100 |
| commit | d89b2751b7fb6cfecd7ccf2ec05ed9b1697b9cbc (patch) | |
| tree | e60ca81765229a3575caf4397a345f7fbf353ff7 /lib/modules | |
| parent | c93229dade9d9a34c56a4e7801ee67b1c089071e (diff) | |
fixes #613
Closes #613
Merge request studip/studip!1801
Diffstat (limited to 'lib/modules')
| -rw-r--r-- | lib/modules/CoreParticipants.class.php | 27 |
1 files changed, 4 insertions, 23 deletions
diff --git a/lib/modules/CoreParticipants.class.php b/lib/modules/CoreParticipants.class.php index b2f1948..cdb0984 100644 --- a/lib/modules/CoreParticipants.class.php +++ b/lib/modules/CoreParticipants.class.php @@ -34,21 +34,6 @@ class CoreParticipants extends CorePlugin implements StudipModule $course = Course::find($course_id); - // Is the participants page hidden for students? - if (!$GLOBALS['perm']->have_studip_perm('tutor', $course_id, $user_id) && $course->config->COURSE_MEMBERS_HIDE) { - $tab_navigation = $this->getTabNavigation($course_id); - if ($tab_navigation && count($tab_navigation['members']->getSubNavigation()) > 0) { - $sub_nav = $tab_navigation['members']->getSubNavigation(); - $first_nav = reset($sub_nav); - - $navigation = new Navigation($first_nav->getTitle(), $first_nav->getURL()); - $navigation->setImage(Icon::create('persons', Icon::ROLE_CLICKABLE)); - return $navigation; - - } - return null; - } - // Determine url to redirect to if (!$course->getSemClass()->isGroup()) { $url = 'dispatch.php/course/members/index'; @@ -138,14 +123,10 @@ class CoreParticipants extends CorePlugin implements StudipModule // Only courses without children have a regular member list and statusgroups. if (!$course->getSemClass()->isGroup()) { - if ($GLOBALS['perm']->have_studip_perm('tutor', $course_id) || !$course->config->COURSE_MEMBERS_HIDE) { - $navigation->addSubNavigation('view', new Navigation(_('Teilnehmende'), 'dispatch.php/course/members')); - $navigation->addSubNavigation('statusgroups', new Navigation(_('Gruppen'), 'dispatch.php/course/statusgroups')); - } - } else { - if ($GLOBALS['perm']->have_studip_perm('tutor', $course_id)) { - $navigation->addSubNavigation('children', new Navigation(_('Teilnehmende in Unterveranstaltungen'), 'dispatch.php/course/grouping/members')); - } + $navigation->addSubNavigation('view', new Navigation(_('Teilnehmende'), 'dispatch.php/course/members')); + $navigation->addSubNavigation('statusgroups', new Navigation(_('Gruppen'), 'dispatch.php/course/statusgroups')); + } elseif ($GLOBALS['perm']->have_studip_perm('tutor', $course_id)) { + $navigation->addSubNavigation('children', new Navigation(_('Teilnehmende in Unterveranstaltungen'), 'dispatch.php/course/grouping/members')); } if ($course->aux_lock_rule) { |