Skip to content
Snippets Groups Projects
Commit 82967ed6 authored by Jan-Hendrik Willms's avatar Jan-Hendrik Willms
Browse files

prevent messily hidden php8 warnings, fixes #2421

Closes #2421

Merge request studip/studip!1615
parent 953de28b
No related branches found
No related tags found
No related merge requests found
...@@ -121,16 +121,18 @@ class CourseDate extends SimpleORMap implements PrivacyObject ...@@ -121,16 +121,18 @@ class CourseDate extends SimpleORMap implements PrivacyObject
*/ */
public static function getConsecutiveNumber($date, $semester = null) public static function getConsecutiveNumber($date, $semester = null)
{ {
if (!isset(self::$numbered_dates[@$semester->id ?: 'all'])) { $semester_id = $semester ? $semester->id : 'all';
if (!isset(self::$numbered_dates[$semester_id])) {
$db = DBManager::get(); $db = DBManager::get();
$numbered = array_flip($db->fetchFirst("SELECT termin_id FROM termine WHERE range_id = ?" . $numbered = array_flip($db->fetchFirst("SELECT termin_id FROM termine WHERE range_id = ?" .
($semester ? " AND date BETWEEN ? AND ?" : "") . ($semester ? " AND date BETWEEN ? AND ?" : "") .
" ORDER BY date", " ORDER BY date",
$semester ? [$date->range_id, $semester->beginn, $semester->ende] : [$date->range_id])); $semester ? [$date->range_id, $semester->beginn, $semester->ende] : [$date->range_id]));
self::$numbered_dates[@$semester->id ?: 'all'] = $numbered; self::$numbered_dates[$semester_id] = $numbered;
} }
return isset(self::$numbered_dates[@$semester->id ?: 'all'][$date->termin_id]) return isset(self::$numbered_dates[$semester_id][$date->termin_id])
? self::$numbered_dates[@$semester->id ?: 'all'][$date->termin_id] + 1 ? self::$numbered_dates[$semester_id][$date->termin_id] + 1
: null; : null;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment