diff options
| author | David Siegfried <david.siegfried@uni-vechta.de> | 2024-08-27 13:49:19 +0000 |
|---|---|---|
| committer | David Siegfried <david.siegfried@uni-vechta.de> | 2024-08-27 13:49:19 +0000 |
| commit | 5ddf03c012a44b14b9585b27352b86e8487c1657 (patch) | |
| tree | 4e3e0715d14765e1cb49b50cb076ecd0b2f36160 /app/controllers/course/members.php | |
| parent | 8830544b03106eac39720268e65f1d2ff9455c6a (diff) | |
prevent php-warnings on members.php, fixes #4533
Closes #4533
Merge request studip/studip!3328
Diffstat (limited to 'app/controllers/course/members.php')
| -rw-r--r-- | app/controllers/course/members.php | 21 |
1 files changed, 10 insertions, 11 deletions
diff --git a/app/controllers/course/members.php b/app/controllers/course/members.php index 1ec1587..0ab18f8 100644 --- a/app/controllers/course/members.php +++ b/app/controllers/course/members.php @@ -1155,7 +1155,7 @@ class Course_MembersController extends AuthenticatedController // insert admission user to autorlist $msgs = $this->setMemberStatus($users, $status, $next_status, 'upgrade'); - if ($msgs['success']) { + if (!empty($msgs['success'])) { PageLayout::postSuccess(sprintf( _('Das Hochstufen auf den Status %s von %s wurde erfolgreich durchgeführt'), htmlReady($this->decoratedStatusGroups[$next_status]), @@ -1163,7 +1163,7 @@ class Course_MembersController extends AuthenticatedController )); } - if ($msgs['no_tutor']) { + if (!empty($msgs['no_tutor'])) { PageLayout::postError(sprintf( _('Das Hochstufen auf den Status %s von %s konnte nicht durchgeführt werden, weil die globale Rechtestufe "tutor" fehlt.') . ' ' . _('Bitte wenden Sie sich an den Support.'), htmlReady($this->decoratedStatusGroups[$next_status]), @@ -1208,7 +1208,7 @@ class Course_MembersController extends AuthenticatedController if (!empty($users)) { $msgs = $this->setMemberStatus($users, $status, $next_status, 'downgrade'); - if ($msgs['success']) { + if (!empty($msgs['success'])) { PageLayout::postSuccess(sprintf( _('Der/die %s %s wurde auf den Status %s heruntergestuft.'), htmlReady($this->decoratedStatusGroups[$status]), @@ -1948,12 +1948,15 @@ class Course_MembersController extends AuthenticatedController } } - private function setMemberStatus($members, $status, $next_status, $direction) + private function setMemberStatus($members, $status, $next_status, $direction): array { - $msgs = []; + $msgs = [ + 'success' => [], + 'no_tutor' => [] + ]; foreach ($members as $user_id) { $temp_user = User::find($user_id); - if ($next_status == 'tutor' && !$GLOBALS['perm']->have_perm('tutor', $user_id)) { + if ($next_status === 'tutor' && !$GLOBALS['perm']->have_perm('tutor', $user_id)) { $msgs['no_tutor'][$user_id] = $temp_user->getFullName(); } else { if ($temp_user) { @@ -1994,11 +1997,7 @@ class Course_MembersController extends AuthenticatedController } } - if (!empty($msgs)) { - return $msgs; - } else { - return false; - } + return $msgs; } public function addMember(string $user_id, bool $accepted = false, bool $consider_contingent = null, &$msg = []): bool |