[BUGFIX] Fix several runtime issues (kitodo#1468)

Signed-off-by: Stefan Weil <sw@weilnetz.de>
Co-authored-by: Sebastian Meyer <sebastian.meyer@opencultureconsulting.com>

Author: stweil

Classes/Command/BaseCommand.php

@@ -247,7 +247,7 @@ protected function saveToDatabase(Document $document, bool $softCommit = false):
             $document->setRightsInfo($metadata['rights_info'][0] ?? '');
             $document->setStatus(0);
 
-            $this->setOwner($metadata['owner'][0]);
+            $this->setOwner($metadata['owner'][0] ?? '');
             $document->setOwner($this->owner);
 
             // set volume data

Classes/Common/Indexer.php

@@ -331,7 +331,7 @@ protected static function processLogical(Document $document, array $logicalUnit)
         $doc = $document->getCurrentDocument();
         $doc->cPid = $document->getPid();
         // Get metadata for logical unit.
-        $metadata = $doc->metadataArray[$logicalUnit['id']];
+        $metadata = $doc->metadataArray[$logicalUnit['id']] ?? [];
         if (!empty($metadata)) {
             $extConf = GeneralUtility::makeInstance(ExtensionConfiguration::class)->get(self::$extKey, 'general');
             $validator = new DocumentValidator($metadata, explode(',', $extConf['requiredMetadataFields']));
@@ -356,19 +356,19 @@ protected static function processLogical(Document $document, array $logicalUnit)
                 // There can be only one toplevel unit per UID, independently of backend configuration
                 $solrDoc->setField('toplevel', $logicalUnit['id'] == $doc->toplevelId ? true : false);
                 $solrDoc->setField('title', $metadata['title'][0]);
-                $solrDoc->setField('volume', $metadata['volume'][0]);
+                $solrDoc->setField('volume', $metadata['volume'][0] ?? '');
                 // verify date formatting
                 if(strtotime($metadata['date'][0])) {
                     $solrDoc->setField('date', self::getFormattedDate($metadata['date'][0]));
                 }
                 $solrDoc->setField('record_id', $metadata['record_id'][0]);
-                $solrDoc->setField('purl', $metadata['purl'][0]);
+                $solrDoc->setField('purl', $metadata['purl'][0] ?? '');
                 $solrDoc->setField('location', $document->getLocation());
                 $solrDoc->setField('urn', $metadata['urn']);
                 $solrDoc->setField('license', $metadata['license']);
                 $solrDoc->setField('terms', $metadata['terms']);
                 $solrDoc->setField('restrictions', $metadata['restrictions']);
-                $coordinates = json_decode($metadata['coordinates'][0]);
+                $coordinates = json_decode($metadata['coordinates'][0] ?? '');
                 if (is_object($coordinates)) {
                     $feature = (array) $coordinates->features[0];
                     $geometry = (array) $feature['geometry'];
@@ -518,7 +518,8 @@ private static function processMetadata($document, $metadata, &$solrDoc): array
                 && substr($indexName, -8) !== '_sorting'
             ) {
                 $solrDoc->setField(self::getIndexFieldName($indexName, $document->getPid()), $data);
-                if (in_array($indexName, self::$fields['sortables'])) {
+                if (in_array($indexName, self::$fields['sortables']) &&
+                    in_array($indexName . '_sorting', $metadata)) {
                     // Add sortable fields to index.
                     $solrDoc->setField($indexName . '_sorting', $metadata[$indexName . '_sorting'][0]);
                 }

Classes/Common/MetsDocument.php

@@ -1602,7 +1602,7 @@ protected function magicGetThumbnail(): string
                     ) {
                         $this->thumbnail = $this->getFileLocation($this->physicalStructureInfo[$this->smLinks['l2p'][$strctId][0]]['files'][$useGroup]);
                         break;
-                    } elseif (!empty($this->physicalStructureInfo[$this->physicalStructure[1]]['files'][$useGroup])) {
+                    } elseif (isset($this->physicalStructure[1]) && !empty($this->physicalStructureInfo[$this->physicalStructure[1]]['files'][$useGroup])) {
                         $this->thumbnail = $this->getFileLocation($this->physicalStructureInfo[$this->physicalStructure[1]]['files'][$useGroup]);
                         break;
                     }