Skip to content
Snippets Groups Projects

don't treat user_id as array, remove duplicate check, fixes #1743

Merged Elmar Ludwig requested to merge biest-1743 into main
1 file
+ 2
2
Compare changes
  • Side-by-side
  • Inline
@@ -625,8 +625,8 @@ class Course_BasicdataController extends AuthenticatedController
@@ -625,8 +625,8 @@ class Course_BasicdataController extends AuthenticatedController
$lecturers = $sem->getMembers();
$lecturers = $sem->getMembers();
foreach ($deputies as $deputy) {
foreach ($deputies as $deputy) {
// ..but only if not already set as lecturer or deputy.
// ..but only if not already set as lecturer or deputy.
if (!isset($lecturers[$deputy['user_id']]) && !Deputy::isDeputy($deputy['user_id'], $course_id)) {
if (!isset($lecturers[$deputy])) {
Deputy::addDeputy($deputy['user_id'], $course_id);
Deputy::addDeputy($deputy, $course_id);
}
}
}
}
}
}
Loading