From 2513b4f596f9cd9185204cf8abb3ba865f373e7a Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Tue, 5 Nov 2019 16:32:45 +0000
Subject: [PATCH] Bug 7611: (follow-up) Fix display of undefined statuses and
 add a class

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
---
 C4/XSLT.pm                                                  | 3 +--
 .../opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACResults.xsl  | 6 +++++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/C4/XSLT.pm b/C4/XSLT.pm
index e0bfde7776..1cc5ebe479 100644
--- a/C4/XSLT.pm
+++ b/C4/XSLT.pm
@@ -344,8 +344,7 @@ sub buildKohaItemsNamespace {
         }
         elsif ( $item->notforloan ) {
                 $status = "reallynotforloan";
-                $substatus = $descs{$item->{notforloan}} || '';
-                $substatus = $substatus->{opac_description} if $substatus;
+                $substatus = exists $descs{$item->{notforloan}} ? $descs{$item->{notforloan}}->{opac_description} : "Not for loan_$item->{notforloan}";
         }
         elsif ( exists $itemtypes->{ $item->effective_itemtype }
             && $itemtypes->{ $item->effective_itemtype }->{notforloan} == 1 )
diff --git a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACResults.xsl b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACResults.xsl
index c403a4675c..edff548f2d 100644
--- a/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACResults.xsl
+++ b/koha-tmpl/opac-tmpl/bootstrap/en/xslt/MARC21slim2OPACResults.xsl
@@ -1343,7 +1343,11 @@
                               <br />
                              <xsl:variable name="reference_items" select="key('item-by-status', 'reallynotforloan')"/>
                              <xsl:for-each select="$reference_items[generate-id() = generate-id(key('item-by-substatus-and-branch', concat(items:substatus, ' ', items:homebranch))[1])]">
-                                <span class="ItemSummary">
+                                <span>
+                                    <xsl:attribute name="class">
+                                        ItemSummary
+                                        <xsl:value-of select="translate(items:substatus,' ','_')"/>
+                                    </xsl:attribute>
                                     <xsl:if test="$singleBranchMode=0">
 					    <span class="ItemBranch"><xsl:value-of select="items:homebranch"/><xsl:text> </xsl:text></span>
                                     </xsl:if>
-- 
2.39.5