From cb55909167b16a6d13ebe41a4197dc4e1df58f35 Mon Sep 17 00:00:00 2001
From: Elmar Ludwig <elmar.ludwig@uni-osnabrueck.de>
Date: Tue, 17 Jan 2023 12:34:52 +0000
Subject: [PATCH] don't treat user_id as array, remove duplicate check, fixes
#2031
Closes #2031
Merge request studip/studip!1324
---
app/controllers/course/members.php | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/app/controllers/course/members.php b/app/controllers/course/members.php
index c3610a8c220..4e31df1ebb6 100644
--- a/app/controllers/course/members.php
+++ b/app/controllers/course/members.php
@@ -397,8 +397,8 @@ class Course_MembersController extends AuthenticatedController
$lecturers = $sem->getMembers();
foreach ($deputies as $deputy) {
// ..but only if not already set as lecturer or deputy.
- if (!isset($lecturers[$deputy['user_id']]) && !Deputy::isDeputy($deputy['user_id'], $this->course_id)) {
- Deputy::addDeputy($deputy['user_id'], $this->course_id);
+ if (!isset($lecturers[$deputy])) {
+ Deputy::addDeputy($deputy, $this->course_id);
}
}
}
--
GitLab