KohaRest: Fix getConfig for holdings.

There was a mistake that dropped everything else but itemLimit from the Holdings configuration.
EreMaijala · Sep 9, 2024 · ba0e9dc · ba0e9dc
1 parent 7c7c9d0
commit ba0e9dc
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/module/VuFind/src/VuFind/ILS/Driver/KohaRest.php b/module/VuFind/src/VuFind/ILS/Driver/KohaRest.php
@@ -1807,17 +1807,15 @@ public function getConfig($function, $params = [])
                 'default_sort' => '+due_date',
             ];
         } elseif ('Holdings' === $function) {
-            $limitByType = $this->config['Holdings']['itemLimitByType'] ?? [];
-            $type = '';
-            if ($limitByType) {
+            $config = $this->config['Holdings'] ?? [];
+            if ($limitByType = $this->config['Holdings']['itemLimitByType'] ?? null) {
                 $biblio = $this->getBiblio($params['id']);
                 $type   = $biblio['item_type'];
+                if ($typeLimit = $limitByType[$type] ?? null) {
+                    $config['itemLimit'] = $typeLimit;
+                }
             }
-            return [
-                'itemLimit' => $limitByType[$type]
-                    ?? $this->config['Holdings']['itemLimit']
-                    ?? null,
-            ];
+            return $config;
         }
 
         return $this->config[$function] ?? false;