LotsLerum-160: show Utlånad for on-loan+hold copies; show Reserverad for hold-shelf items (per customer)

Author: root

languages/en.ini

@@ -1716,4 +1716,4 @@ search_aria_description = "Search in the library catalog (title, author, subject
 
 holds_descriptive_text = "Text description"
 select_page = "Page"
-On Holdshelf = "Not Available"
+On Holdshelf = "Reserved"

languages/sv.ini

@@ -1654,7 +1654,7 @@ Zip = "Postnummer"
 zoom = "Zooma"
 Category = "Medietyp"
 On Hold = "Reserverad"
-On Holdshelf = "Inte tillgänglig"
+On Holdshelf = "Reserverad"
 On Shelf = "Tillgänglig"
 Overdue = "Försenad"
 Swedish = Svenska

module/LOTS/src/LOTS/ILS/Driver/KohaRest.php

@@ -724,6 +724,25 @@ protected function getItemStatusesForBiblio($id, $patron = null, array $options
     {
         $results = parent::getItemStatusesForBiblio($id, $patron, $options);
 
+        // LotsLerum-160: on loan + hold should show "Utlånad", not "Reserverad".
+        // Parent collapses a copy that is both checked out and reserved into
+        // status "On Hold" but keeps both in status_array (["Borrowed","On Hold"]).
+        // When that combination is present, prefer the loan status.
+        // NOTE: ported from lots-8.0 (verified there on live data); verify on a
+        // running VuFind 11 instance during the upgrade, as the parent result
+        // shape may differ.
+        if (is_array($results)) {
+            foreach ($results as &$lotsEntry) {
+                if (is_array($lotsEntry)
+                    && ($lotsEntry['status'] ?? null) === 'On Hold'
+                    && in_array('Borrowed', $lotsEntry['status_array'] ?? [], true)
+                ) {
+                    $lotsEntry['status'] = 'Checked Out';
+                }
+            }
+            unset($lotsEntry);
+        }
+
         if (empty($results['holdings'])) {
             return $results;
         }