[BUGFIX] Fix runtime error for OAI-PMH access without verb (kitodo#1426)

stweil · sebastian-meyer · web-flow · commit e09862626436 · 2025-02-01T02:17:56.000+01:00
Signed-off-by: Stefan Weil &lt;sw@weilnetz.de&gt;
Co-authored-by: Sebastian Meyer &lt;sebastian.meyer@opencultureconsulting.com&gt;
diff --git a/Classes/Controller/OaiPmhController.php b/Classes/Controller/OaiPmhController.php
@@ -273,7 +273,7 @@ public function mainAction()
         // Delete expired resumption tokens.
         $this->deleteExpiredTokens();
 
-        switch ($this->parameters['verb']) {
+        switch ($this->parameters['verb'] ?? null) {
             case 'GetRecord':
                 $this->verbGetRecord();
                 break;
diff --git a/Classes/Domain/Repository/CollectionRepository.php b/Classes/Domain/Repository/CollectionRepository.php
@@ -97,11 +97,11 @@ public function findCollectionsBySettings(array $settings = []): QueryResultInte
 
         $constraints = [];
 
-        if ($settings['collections']) {
+        if ($settings['collections'] ?? false) {
             $constraints[] = $query->in('uid', GeneralUtility::intExplode(',', $settings['collections']));
         }
 
-        if ($settings['index_name']) {
+        if ($settings['index_name'] ?? false) {
             $constraints[] = $query->in('index_name', $settings['index_name']);
         }
 

Original file line number	Diff line number	Diff line change
`@@ -97,11 +97,11 @@ public function findCollectionsBySettings(array $settings = []): QueryResultInte`
`97`	`97`
`98`	`98`	`$constraints = [];`
`99`	`99`
`100`		`- if ($settings['collections']) {`
	`100`	`+ if ($settings['collections'] ?? false) {`
`101`	`101`	`$constraints[] = $query->in('uid', GeneralUtility::intExplode(',', $settings['collections']));`
`102`	`102`	`}`
`103`	`103`
`104`		`- if ($settings['index_name']) {`
	`104`	`+ if ($settings['index_name'] ?? false) {`
`105`	`105`	`$constraints[] = $query->in('index_name', $settings['index_name']);`
`106`	`106`	`}`
`107`	`107`