prevent php8 warnings, fixes #3732

Closes #3732

Merge request studip/studip!2594

app/controllers/module/download.php

@@ -82,7 +82,7 @@ class Module_DownloadController extends MVVController
 
         foreach ($modul->modulteile as $modulTeil) {
             $deskriptor = $modulTeil->getDeskriptor($language);
-            $num_bezeichnung = $GLOBALS['MVV_MODULTEIL']['NUM_BEZEICHNUNG']['values'][$modulTeil->num_bezeichnung]['name'];
+            $num_bezeichnung = $GLOBALS['MVV_MODULTEIL']['NUM_BEZEICHNUNG']['values'][$modulTeil->num_bezeichnung]['name'] ?? '';
 
             $name_kurz = sprintf('%s %d', $num_bezeichnung, $modulTeil->nummer);
             $modulTeilData[$modulTeil->id] = [
@@ -97,7 +97,7 @@ class Module_DownloadController extends MVVController
                 'wl_bereitung'           => $modulTeil->wl_bereitung,
                 'wl_selbst'              => $modulTeil->wl_selbst,
                 'wl_pruef'               => $modulTeil->wl_pruef,
-                'turnus'                 => $GLOBALS['MVV_NAME_SEMESTER']['values'][$modulTeil->semester]['name'],
+                'turnus'                 => $GLOBALS['MVV_NAME_SEMESTER']['values'][$modulTeil->semester]['name'] ?? '',
                 'kommentar_wl_preasenz'  => $deskriptor->kommentar_wl_praesenz,
                 'kommentar_wl_bereitung' => $deskriptor->kommentar_wl_bereitung,
                 'kommentar_wl_selbst'    => $deskriptor->kommentar_wl_selbst,
@@ -133,7 +133,7 @@ class Module_DownloadController extends MVVController
             ));
         $this->semesterSelector = Semester::GetSemesterSelector(null, $currentSemester->getId(), 'semester_id', false);
         $this->modul = $modul;
-        $this->pruefungsEbene = $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name'];
+        $this->pruefungsEbene = $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name'] ?? null;
         $this->modulDeskriptor = $modul->getDeskriptor($language);
         $this->startSemester = Semester::find($modul->start);
         if ($modul->responsible_institute->institute) {

app/controllers/module/module.php

@@ -5,8 +5,6 @@
  * @since       3.5
  */
 
-require_once 'lib/classes/exportdocument/ExportPDF.class.php';
-
 class Module_ModuleController extends MVVController
 {
     public $filter = [];

app/controllers/search/studiengaenge.php

@@ -402,7 +402,7 @@ class Search_StudiengaengeController extends MVVController
                             [
                                 'name' => $file->getDisplayName(),
                                 'url'  => $mvv_file_ref->file_ref->getDownloadURL(),
-                                'metadata_url' => $mvv_file_ref->file_ref->file->metadata['url'],
+                                'metadata_url' => $mvv_file_ref->file_ref->file->metadata['url'] ?? null,
                                 'extension' => $mvv_file_ref->file_ref->file->getExtension(),
                                 'is_link' => ($filetype instanceof URLFile)
                             ];

app/controllers/shared/modul.php

@@ -63,7 +63,7 @@ class Shared_ModulController extends AuthenticatedController
 
                 $sws += (int) $modulTeil->sws;
 
-                $num_bezeichnung = $GLOBALS['MVV_MODULTEIL']['NUM_BEZEICHNUNG']['values'][$modulTeil->num_bezeichnung]['name'];
+                $num_bezeichnung = $GLOBALS['MVV_MODULTEIL']['NUM_BEZEICHNUNG']['values'][$modulTeil->num_bezeichnung]['name'] ?? '';
 
                 $name_kurz = sprintf('%s %d', $num_bezeichnung, $modulTeil->nummer);
 
@@ -94,7 +94,7 @@ class Shared_ModulController extends AuthenticatedController
             $this->semester = $currentSemester;
             $this->sws = $sws;
 
-            $this->pruef_ebene = $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name'];
+            $this->pruef_ebene = $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name'] ?? null;
             $this->modul = $modul;
             $this->type = $type;
             $this->self_url = $this->url_for('modul/show/' . $modul_id);
@@ -133,16 +133,19 @@ class Shared_ModulController extends AuthenticatedController
 
         $this->semesterSelector = Semester::getSemesterSelector(null, $currentSemester['semester_id'], 'semester_id', false);
         $this->modul = $modul;
-        $this->pruefungsEbene = $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name'];
+        $this->pruefungsEbene = isset($GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene])
+                              ? $GLOBALS['MVV_MODUL']['PRUEF_EBENE']['values'][$modul->pruef_ebene]['name']
+                              : null;
         $this->modulDeskriptor = $modul->getDeskriptor($display_language);
         $this->startSemester = Semester::findByTimestamp($modul->start);
-        if ($modul->responsible_institute) {
-            if ($modul->responsible_institute->institute) {
-                $this->instituteName = $modul->responsible_institute->institute->getValue('name');
+
+        if (!$modul->responsible_institute) {
+            $this->instituteName = null;
+        } elseif ($modul->responsible_institute->institute) {
+            $this->instituteName = $modul->responsible_institute->institute->name;
         } else {
             $this->instituteName = _('Unbekannte Einrichtung');
         }
-        }
         $this->type = $type;
         $this->semester = $currentSemester;
         $this->display_language = $display_language;

app/views/search/studiengaenge/info.php

@@ -172,7 +172,7 @@
         </header>
         <section>
             <? foreach ($all_documents as $category => $files) : ?>
-                <? if ($GLOBALS['MVV_DOCUMENTS']['CATEGORY']['values'][$category]['visible']) : ?>
+                <? if (!empty($GLOBALS['MVV_DOCUMENTS']['CATEGORY']['values'][$category]['visible'])) : ?>
                     <strong><?= htmlReady($GLOBALS['MVV_DOCUMENTS']['CATEGORY']['values'][$category]['name']) ?></strong>
                     <ul>
                         <? foreach ($files as $file) : ?>

app/views/search/studiengaenge/mehrfach.php

@@ -3,8 +3,8 @@
     <caption>
         <?= _('Studiengang') ?>: <?= htmlReady($studiengang->getDisplayName()) ?>
         <? if (Config::get()->ENABLE_STUDYCOURSE_INFO_PAGE) : ?>
-            <a href="<?= $controller->url_for('search/studiengaenge/info', $studiengang->id)?>" data-dialog>
-                <?= Icon::create('infopage2', Icon::ROLE_CLICKABLE, ['title' => _('Informationen zum Studiengang')]) ?>
+            <a href="<?= $controller->link_for('search/studiengaenge/info', $studiengang->id)?>" data-dialog>
+                <?= Icon::create('infopage2')->asImg(['title' => _('Informationen zum Studiengang')]) ?>
             </a>
         <? endif; ?>
     </caption>
@@ -25,8 +25,8 @@
         <? foreach ($studiengangTeilBezeichnungen as $teil_bezeichnung): ?>
             <td style="text-align: center;">
             <? if (isset($fach[$teil_bezeichnung->id])) : ?>
-                    <a href="<?= $controller->url_for($verlauf_url, $fach[$teil_bezeichnung->id], $teil_bezeichnung->id, $studiengang_id) ?>">
-                        <?= Icon::create('info-circle-full', 'clickable', ['title' => _('Studienverlaufsplan anzeigen')]); ?>
+                <a href="<?= $controller->link_for($verlauf_url, $fach[$teil_bezeichnung->id], $teil_bezeichnung->id, $studiengang_id ?? null) ?>">
+                    <?= Icon::create('info-circle-full')->asImg(['title' => _('Studienverlaufsplan anzeigen')]) ?>
                 </a>
             <? endif; ?>
             </td>

app/views/search/studiengaenge/verlauf.php

@@ -84,10 +84,22 @@
                                 </td>
                             <? endif ?>
                             <td><?= htmlReady($modulTeil['name']) ?> </td>
-                            <? for ($i = 1; $i <= $max_fachsemester; $i++) : ?>
-                                <? $fachsemester_typ = $GLOBALS['MVV_MODULTEIL_STGABSCHNITT']['STATUS']['values'][$modulTeil['fachsemester'][$fachsemesterData[$i]]] ?>
-                                <? if ($fachsemester_typ['visible']) : ?>
-                            <td class="mvv-type-<?= $modulTeil['fachsemester'][$fachsemesterData[$i]] ?>"><span title="<? printf(_('%s Semester (%s)'), $i . ModuleManagementModel::getLocaleOrdinalNumberSuffix($i), $fachsemester_typ['name']) ?>"><?= $fachsemester_typ['icon'] ?></span></td>
+                        <? for ($i = 1; $i <= $max_fachsemester; $i++) :
+                            $fachsemester_typ = null;
+                            if (isset(
+                                $fachsemesterData[$i],
+                                $modulTeil['fachsemester'][$fachsemesterData[$i]],
+                                $GLOBALS['MVV_MODULTEIL_STGABSCHNITT']['STATUS']['values'][$modulTeil['fachsemester'][$fachsemesterData[$i]]]
+                            )) {
+                                $fachsemester_typ = $GLOBALS['MVV_MODULTEIL_STGABSCHNITT']['STATUS']['values'][$modulTeil['fachsemester'][$fachsemesterData[$i]]];
+                            }
+                        ?>
+                            <? if (!empty($fachsemester_typ['visible'])) : ?>
+                                <td class="mvv-type-<?= $modulTeil['fachsemester'][$fachsemesterData[$i]] ?? '' ?>">
+                                    <span title="<? printf(_('%s Semester (%s)'), $i . ModuleManagementModel::getLocaleOrdinalNumberSuffix($i), $fachsemester_typ['name']) ?>">
+                                        <?= $fachsemester_typ['icon'] ?>
+                                    </span>
+                                </td>
                             <? else : ?>
                                 <td class="mvv-type">&nbsp;</td>
                             <? endif ?>

app/views/shared/modul/_modul.php

-<? $modulDeskriptor = $modul->getDeskriptor($display_language); ?>
+<? $modulDeskriptor = $modul->getDeskriptor($display_language ?? null); ?>
 <table class="mvv-modul-details default nohover" data-mvv-id="<?= $modul->getId(); ?>" data-mvv-type="modul">
     <colgroup>
         <col width="30%">
@@ -66,14 +66,14 @@
                     <? endif; ?>
                     <ul>
                     <? foreach ($pathes as $i => $path) : ?>
-                    <? $version = $trails[$i]['StgteilVersion']['version_id'];?>
+                    <? $version = $trails[$i]['StgteilVersion']['version_id'] ?? null;?>
                     <? $statement = DBManager::get()->prepare(
                              'SELECT `mvv_stgteilabschnitt_modul`.`abschnitt_id`  '
                            . 'FROM mvv_stgteilabschnitt_modul LEFT JOIN mvv_stgteilabschnitt USING(abschnitt_id) '
                            . 'WHERE modul_id = ? AND version_id = ?');
                        $statement->execute([$modul->getId(), $version]);
                        $res = $statement->fetchOne();
-                       $affect_id = $res['abschnitt_id'];
+                       $affect_id = $res['abschnitt_id'] ?? null;
                     ?>
                     <li data-mvv-field="mvv_stgteilabschnitt_modul" data-mvv-id="<?= $affect_id; ?>" data-mvv-cooid="<?= $modul->getId(); ?>">
                         <?= htmlReady($path)?>

app/views/shared/modul/_modullv.php

 <? $modulTeil = $modul->modulteile->first(); ?>
-<? $modulTeilDeskriptor = $modulTeil->getDeskriptor($display_language); ?>
+<? $modulTeilDeskriptor = $modulTeil->getDeskriptor($display_language ?? null); ?>
 <table class="mvv-modul-details default nohover" data-mvv-id="<?= $modulTeil->id; ?>" data-mvv-type="modulteil">
     <tbody>
         <? $modulTeilSumme = $modulTeil->wl_praesenz + $modulTeil->wl_bereitung + $modulTeil->wl_selbst + $modulTeil->wl_pruef ?>
@@ -73,7 +73,7 @@
     <tbody>
         <tr>
             <td style="width: 30%;"><strong><?= _('Angebotsrhythmus') ?></strong></td>
-            <td style="width: 70%;" data-mvv-field="mvv_modulteil.semester"><?= $GLOBALS['MVV_NAME_SEMESTER']['values'][$modulTeil->semester]['name'] ?></td>
+            <td style="width: 70%;" data-mvv-field="mvv_modulteil.semester"><?= $GLOBALS['MVV_NAME_SEMESTER']['values'][$modulTeil->semester]['name'] ?? '' ?></td>
         </tr>
         <tr>
             <td style="width: 30%;"><strong><?= _('Aufnahmekapazität') ?></strong></td>

app/views/shared/modul/_modullvs.php

@@ -36,7 +36,7 @@
             <? $wlPruef += $modulTeil->wl_pruef; ?>
             <? $modulTeilSumme = $modulTeil->wl_praesenz + $modulTeil->wl_bereitung + $modulTeil->wl_selbst + $modulTeil->wl_pruef; ?>
             <? $modulSumme += $modulTeilSumme; ?>
-            <? if ($show_synopse) : ?>
+            <? if (!empty($show_synopse)) : ?>
             <tr data-mvv-id="<?= $modulTeil->getId(); ?>" data-mvv-type="modulteil">
                 <td data-mvv-field="mvv_modulteil.nummer mvv_modulteil.num_bezeichnung"><strong><?= htmlReady($name_kurz) ?></strong></td>
                 <td data-mvv-field="mvv_modulteil.lernlehrform"><?= $GLOBALS['MVV_MODULTEIL']['LERNLEHRFORM']['values'][$modulTeil->lernlehrform]['name'] ?></td>

app/views/shared/modul/_pruefungen.php

@@ -9,7 +9,7 @@
     <tbody>
         <? foreach ($modul->modulteile as $modulTeil): ?>
             <?
-            $modulTeilDeskriptor = $modulTeil->getDeskriptor($display_language);
+            $modulTeilDeskriptor = $modulTeil->getDeskriptor($display_language ?? null);
             // Für die Kenntlichmachung der Modulteile in Listen die Nummer des
             // Modulteils und den ausgewählten Namen verwenden.
             // Ist keine Nummer vorhanden, dann Durchnummerieren und Standard-

lib/classes/exportdocument/ExportPDF.class.php

@@ -27,8 +27,8 @@ define('K_PATH_IMAGES', $GLOBALS['STUDIP_BASE_PATH'] . '/public/assets/images/')
  *  //cancels all other output.
  *
  */
-class ExportPDF extends TCPDF implements ExportDocument {
-
+class ExportPDF extends TCPDF implements ExportDocument
+{
     private $media_proxy = NULL;
     private $config;
     private $defaults = false;