prevent php-warnings, fixes #4823

Closes #4823

Merge request studip/studip!3608

1 files changed, 4 insertions, 4 deletions

diff --git a/lib/classes/coursewizardsteps/BasicDataWizardStep.php b/lib/classes/coursewizardsteps/BasicDataWizardStep.php
index 4944611..8f7a96c 100644
--- a/lib/classes/coursewizardsteps/BasicDataWizardStep.php
+++ b/lib/classes/coursewizardsteps/BasicDataWizardStep.php
@@ -350,7 +350,7 @@ class BasicDataWizardStep implements CourseWizardStep
         $values = $values[__CLASS__];
         $ok = true;
         $errors = [];
-        if (!trim($values['name'])) {
+        if (empty($values['name'])) {
             $errors[] = _('Bitte geben Sie den Namen der Veranstaltung an.');
         }
         if (isset($values['number']) && $values['number'] != '') {
@@ -362,7 +362,7 @@ class BasicDataWizardStep implements CourseWizardStep
         if (empty($values['lecturers'])) {
             $errors[] = sprintf(
                 _('Bitte tragen Sie mindestens eine Person als %s ein.'),
-                htmlReady(get_title_for_status('dozent', 1, $values['coursetype']))
+                htmlReady(get_title_for_status('dozent', 1, $values['coursetype'] ?? null))
             );
         }
         if (empty($values['lecturers'][$GLOBALS['user']->id]) && !$GLOBALS['perm']->have_perm('admin')) {
@@ -370,13 +370,13 @@ class BasicDataWizardStep implements CourseWizardStep
                 if (empty($values['deputies'][$GLOBALS['user']->id])) {
                     $errors[] = sprintf(
                         _('Sie selbst müssen entweder als %s oder als Vertretung eingetragen sein.'),
-                        htmlReady(get_title_for_status('dozent', 1, $values['coursetype']))
+                        htmlReady(get_title_for_status('dozent', 1, $values['coursetype'] ?? null))
                     );
                 }
             } else {
                 $errors[] = sprintf(
                     _('Sie müssen selbst als %s eingetragen sein.'),
-                    htmlReady(get_title_for_status('dozent', 1, $values['coursetype']))
+                    htmlReady(get_title_for_status('dozent', 1, $values['coursetype'] ?? null))
                 );
             }
         }