prevent messily hidden php8 warnings, fixes #2421issue-2373

Closes #2421

Merge request studip/studip!1615

1 files changed, 6 insertions, 4 deletions

diff --git a/lib/models/CourseDate.class.php b/lib/models/CourseDate.class.php
index 7f7d172..1465fa1 100644
--- a/lib/models/CourseDate.class.php
+++ b/lib/models/CourseDate.class.php
@@ -121,16 +121,18 @@ class CourseDate extends SimpleORMap implements PrivacyObject
      */
     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();
             $numbered = array_flip($db->fetchFirst("SELECT termin_id FROM termine WHERE range_id = ?" .
                 ($semester ? " AND date BETWEEN ? AND ?" : "") .
                 " ORDER BY date",
                 $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])
-             ? self::$numbered_dates[@$semester->id ?: 'all'][$date->termin_id] + 1
+        return isset(self::$numbered_dates[$semester_id][$date->termin_id])
+             ? self::$numbered_dates[$semester_id][$date->termin_id] + 1
              : null;
     }