Resolve "MVV: Modul-Kurzdarstellung im VVZ"

Closes #6252

Merge request studip/studip!4745

8 files changed, 21 insertions, 12 deletions

diff --git a/app/views/course/details/index.php b/app/views/course/details/index.php
index 934d251..98d4eac 100644
--- a/app/views/course/details/index.php
+++ b/app/views/course/details/index.php
@@ -424,17 +424,19 @@ if (!empty($mvv_tree)) : ?>
     </article>
 <? endif; ?>
 
-<? if (!empty($mvv_pathes)) : ?>
+<? if (!empty($mvv_paths)) : ?>
     <article class="studip">
         <header>
             <h1><?= _('Modulzuordnungen') ?></h1>
         </header>
         <section>
             <ul class="list-unstyled">
-                <? foreach ($mvv_pathes as $mvv_path) : ?>
+                <? foreach ($mvv_paths as $mvv_path) : ?>
+                <? $types = array_keys($mvv_path); ?>
+                <? $values = array_values($mvv_path); ?>
                 <li>
-                    <a data-dialog href="<?= URLHelper::getScriptLink('dispatch.php/search/module/overview/' . reset(array_keys($mvv_path)) . '/' . $course->start_semester->id) ?>">
-                        <?= htmlReady(implode(' > ', reset(array_values($mvv_path)))) ?>
+                    <a data-dialog href="<?= URLHelper::getScriptLink('dispatch.php/search/module/overview/' . $types[0] . '/' . $course->start_semester->id) ?>">
+                        <?= htmlReady(implode(' > ', $values[0])) ?>
                     </a>
                 </li>
                 <? endforeach; ?>
diff --git a/app/views/lvgruppen/lvgruppen/export_xls.php b/app/views/lvgruppen/lvgruppen/export_xls.php
index 50a9090..e101bb9 100644
--- a/app/views/lvgruppen/lvgruppen/export_xls.php
+++ b/app/views/lvgruppen/lvgruppen/export_xls.php
@@ -8,7 +8,7 @@
         <th><?= _('Modul') ?></th>
         <th><?= _('Modulteil') ?></th>
     </tr>
-    <? $trail_classes = words('Modulteil Modul StgteilAbschnitt StgteilVersion '
+    <? $trail_classes = words('Modulteil StgteilabschnittModul StgteilAbschnitt StgteilVersion '
                 . 'Studiengang Fachbereich'); ?>
     <? foreach ($lvgruppen as $lvgruppe) : ?>
         <? $trails = $lvgruppe->getTrails($trail_classes, MvvTreeItem::TRAIL_SHOW_INCOMPLETE); ?>
@@ -18,6 +18,9 @@
                     <td><?= htmlReady($lvgruppe->getDisplayName()) ?></td>
                 <? foreach (array_reverse($trail_classes) as $trail_class) : ?>
                     <? if ($trail[$trail_class]) : ?>
+                        <? if ($trail_class === 'StgteilabschnittModul') : ?>
+                            <? $trail['Modulteil']->setReplaceDfAbschnitt($trail[$trail_class]->abschnitt) ?>
+                        <? endif; ?>
                         <td><?= htmlReady($trail[$trail_class]->getDisplayName()) ?></td>
                     <? else : ?>
                         <td></td>
diff --git a/app/views/lvgruppen/lvgruppen/trails_table_lvgruppe.php b/app/views/lvgruppen/lvgruppen/trails_table_lvgruppe.php
index cd150be..13180b5 100644
--- a/app/views/lvgruppen/lvgruppen/trails_table_lvgruppe.php
+++ b/app/views/lvgruppen/lvgruppen/trails_table_lvgruppe.php
@@ -9,6 +9,9 @@
     <tr>
         <? foreach (array_reverse($trail_classes) as $trail_class) : ?>
             <? if (isset($trail[$trail_class])) : ?>
+                <? if ($trail_class === 'StgteilabschnittModul') : ?>
+                    <? $trail['Modulteil']->setReplaceDfAbschnitt($trail[$trail_class]->abschnitt) ?>
+                <? endif; ?>
             <td style="vertical-align: top;">
                 <?= htmlReady($trail[$trail_class]->getDisplayName()) ?>
             </td>
@@ -23,4 +26,4 @@
 <span class="mvv-no-entry">
 <?= _('Diese Lehrveranstaltungsgruppe wurde für das ausgewählte Semester keinen Modulteilen zugeordnet.') ?>
 </span>
-<? endif; ?>
\ No newline at end of file
+<? endif; ?>
diff --git a/app/views/search/studiengaenge/verlauf.php b/app/views/search/studiengaenge/verlauf.php
index 107f15c..b3657fc 100644
--- a/app/views/search/studiengaenge/verlauf.php
+++ b/app/views/search/studiengaenge/verlauf.php
@@ -8,6 +8,7 @@
  * @var array $abschnitte_data
  * @var Search_StudiengaengeController $controller
  * @var int $close_sections
+ * @var Semester $active_sem
  */
 ?>
 <div>
diff --git a/app/views/shared/modul/_modullvs.php b/app/views/shared/modul/_modullvs.php
index 97ac3e2..8a86879 100644
--- a/app/views/shared/modul/_modullvs.php
+++ b/app/views/shared/modul/_modullvs.php
@@ -27,7 +27,7 @@
             <? $modulteil_deskriptor = $modulteil->getDeskriptor();
             // 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-
+            // Ist keine Nummer vorhanden, dann durchnummerieren und Standard-
             // Bezeichnung verwenden.
             if (trim($modulteil->nummer)) {
                 $num_bezeichnung = $GLOBALS['MVV_MODULTEIL']['NUM_BEZEICHNUNG']['values'][$modulteil->num_bezeichnung]['name'];
diff --git a/app/views/shared/modul/description.php b/app/views/shared/modul/description.php
index 05eddbf..d104822 100644
--- a/app/views/shared/modul/description.php
+++ b/app/views/shared/modul/description.php
@@ -15,7 +15,7 @@
             href="<?= $controller->action_link('description/' . $modul->id . '/',
                 [
                     'display_language' => $language,
-                    'abschnitt_id' => $abschnitt_modul->abschnitt_id,
+                    'abschnitt_id' => empty($abschnitt_modul) ? '' : $abschnitt_modul->abschnitt_id,
                 ]) ?>">
             <?= Assets::img(MVV::getContentLanguageImagePath($language), ['alt' => $lang['name'], 'size' => 24]) ?>
         </a>
diff --git a/app/views/shared/modul/overview.php b/app/views/shared/modul/overview.php
index c6eadca..7e07ee8 100644
--- a/app/views/shared/modul/overview.php
+++ b/app/views/shared/modul/overview.php
@@ -127,7 +127,7 @@
             <tr>
                 <? if ($type === 1): ?>
                 <td>
-                    <b> <?= htmlReady($modulteil_deskriptor->getReplacedValue('bezeichnung')) ?> </b>
+                    <b> <?= htmlReady($modulteil->getDisplayName()) ?> </b>
                     <? if (mb_strlen($modulteil_deskriptor->getReplacedValue('kommentar')) > 0): ?>
                     <?= trim($modulteil_deskriptor->getReplacedValue('kommentar')) ? '<br>(' . formatReady($modulteil_deskriptor->getReplacedValue('kommentar')) . ')' : '' ?>
                     <? endif; ?>
diff --git a/app/views/shared/mvv_tree.php b/app/views/shared/mvv_tree.php
index 6a6c82d..38858b6 100644
--- a/app/views/shared/mvv_tree.php
+++ b/app/views/shared/mvv_tree.php
@@ -1,9 +1,9 @@
 <? foreach ($tree[$node] as $current) : ?>
 <li>
     <? if ($current['class'] != 'Modulteil' && $current['id'] != 'root') : ?>
-        <input id="<?= htmlReady($current['id'] . $id_sfx->c) ?>" type="checkbox"<?= $current['class'] != 'StgteilabschnittModul' ? 'checked' : ''?>>
+        <input id="<?= htmlReady($current['tree_id'] . $id_sfx->c) ?>" type="checkbox"<?= $current['class'] != 'StgteilabschnittModul' ? 'checked' : ''?>>
     <? endif; ?>
-        <label for="<?= htmlReady($current['id'] . $id_sfx->c++) ?>"></label>
+        <label for="<?= htmlReady($current['tree_id'] . $id_sfx->c++) ?>"></label>
     <? if ($current['class'] == 'StgteilabschnittModul') : ?>
         <a data-dialog title="<?= htmlReady($current['name']) ?>" href="<?= URLHelper::getLink('dispatch.php/shared/modul/overview/' . $current['id'] . '/' . $course->start_semester->id) ?>">
             <?= htmlReady($current['name']) ?>
@@ -16,7 +16,7 @@
     <? endif; ?>
     <? if ($current['class'] != 'Modulteil') : ?>
         <ul>
-            <?= $this->render_partial('shared/mvv_tree.php', ['tree' => $tree, 'node' => $current['id']]) ?>
+            <?= $this->render_partial('shared/mvv_tree.php', ['tree' => $tree, 'node' => $current['tree_id']]) ?>
         </ul>
     <? endif; ?>
 </li>