Skip to content
Snippets Groups Projects
Commit 2ffa7291 authored by David Siegfried's avatar David Siegfried Committed by Jan-Hendrik Willms
Browse files

prevent php-warnings, closes #2356

Closes #2356

Merge request studip/studip!1540
parent 57d455a7
No related branches found
No related tags found
No related merge requests found
...@@ -1193,7 +1193,9 @@ class Admin_CoursesController extends AuthenticatedController ...@@ -1193,7 +1193,9 @@ class Admin_CoursesController extends AuthenticatedController
$filter->filterBySemester($this->semester->getId()); $filter->filterBySemester($this->semester->getId());
} }
if ($active_elements['courseType'] && $params['typeFilter'] && $params['typeFilter'] !== "all") { if ($active_elements['courseType'] && $params['typeFilter'] && $params['typeFilter'] !== "all") {
list($class_filter,$type_filter) = explode('_', $params['typeFilter']); $parts = explode('_', $params['typeFilter']);
$class_filter = $parts[0];
$type_filter = $parts[1] ?? null;
if (!$type_filter && !empty($GLOBALS['SEM_CLASS'][$class_filter])) { if (!$type_filter && !empty($GLOBALS['SEM_CLASS'][$class_filter])) {
$type_filter = array_keys($GLOBALS['SEM_CLASS'][$class_filter]->getSemTypes()); $type_filter = array_keys($GLOBALS['SEM_CLASS'][$class_filter]->getSemTypes());
} }
......
...@@ -243,7 +243,7 @@ class Semester extends SimpleORMap ...@@ -243,7 +243,7 @@ class Semester extends SimpleORMap
$semesters = self::getAllAsArray($with_before_first); $semesters = self::getAllAsArray($with_before_first);
} }
foreach ($semesters as $index => $semester) { foreach ($semesters as $index => $semester) {
if (@$semester['semester_id'] === $semester_id) { if (isset($semester['semester_id']) && $semester['semester_id'] === $semester_id) {
return $index; return $index;
} }
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment