Skip to content

Commit 63041f8

Browse files
[BUGFIX] Fix errors caused by missing keys in entry array (#1347)
Co-authored-by: Sebastian Meyer <sebastian.meyer@opencultureconsulting.com>
1 parent 0baa144 commit 63041f8

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

Classes/Controller/TableOfContentsController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ private function getMenuEntry(array $entry, bool $recursive = false): array
131131
$entryArray['doNotLinkIt'] = 1;
132132
$entryArray['ITEM_STATE'] = 'NO';
133133

134-
$this->buildMenuLinks($entryArray, $entry['id'], $entry['points'], $entry['targetUid']);
134+
$this->buildMenuLinks($entryArray, $entry['id'], $entry['points'] ?? null, $entry['targetUid'] ?? null);
135135

136136
// Set "ITEM_STATE" to "CUR" if this entry points to current page.
137137
if (in_array($entry['id'], $this->activeEntries)) {
@@ -148,7 +148,7 @@ private function getMenuEntry(array $entry, bool $recursive = false): array
148148
// 3. Current menu node has no corresponding images
149149
if (
150150
$entryArray['ITEM_STATE'] == 'CUR'
151-
|| is_string($entry['points'])
151+
|| (array_key_exists('points', $entry) && is_string($entry['points']))
152152
|| empty($this->document->getCurrentDocument()->smLinks['l2p'][$entry['id']])
153153
) {
154154
$entryArray['_SUB_MENU'] = [];
@@ -241,7 +241,7 @@ private function resolveMenuEntry(array $entry): array
241241
$doc = $this->document->getCurrentDocument();
242242
if (
243243
$doc instanceof MetsDocument
244-
&& ($entry['points'] === $doc->parentHref || $this->isMultiElement($entry['type']))
244+
&& ((array_key_exists('points', $entry) && $entry['points'] === $doc->parentHref) || $this->isMultiElement($entry['type']))
245245
&& !empty($this->document->getPartof())
246246
) {
247247
unset($entry['points']);

0 commit comments

Comments
 (0)