Fix errors caused by missing keys in entry array

kitodo · Sep 24, 2024 · 7d5f515 · 7d5f515
1 parent c5d13ea
commit 7d5f515
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Classes/Controller/TableOfContentsController.php b/Classes/Controller/TableOfContentsController.php
@@ -131,7 +131,7 @@ private function getMenuEntry(array $entry, bool $recursive = false): array
         $entryArray['doNotLinkIt'] = 1;
         $entryArray['ITEM_STATE'] = 'NO';
 
-        $this->buildMenuLinks($entryArray, $entry['id'], $entry['points'], $entry['targetUid']);
+        $this->buildMenuLinks($entryArray, $entry['id'], $entry['points'] ?? null, $entry['targetUid'] ?? null);
 
         // Set "ITEM_STATE" to "CUR" if this entry points to current page.
         if (in_array($entry['id'], $this->activeEntries)) {
@@ -148,7 +148,7 @@ private function getMenuEntry(array $entry, bool $recursive = false): array
             // 3. Current menu node has no corresponding images
             if (
                 $entryArray['ITEM_STATE'] == 'CUR'
-                || is_string($entry['points'])
+                || (array_key_exists('points', $entry) && is_string($entry['points']))
                 || empty($this->document->getCurrentDocument()->smLinks['l2p'][$entry['id']])
             ) {
                 $entryArray['_SUB_MENU'] = [];
@@ -241,7 +241,7 @@ private function resolveMenuEntry(array $entry): array
         $doc = $this->document->getCurrentDocument();
         if (
             $doc instanceof MetsDocument
-            && ($entry['points'] === $doc->parentHref || $this->isMultiElement($entry['type']))
+            && ((array_key_exists('points', $entry) && $entry['points'] === $doc->parentHref) || $this->isMultiElement($entry['type']))
             && !empty($this->document->getPartof())
         ) {
             unset($entry['points']);