From be94393d8ef22e526b07dd6092768fc05f4adb19 Mon Sep 17 00:00:00 2001
From: Kyle M Hall <kyle@bywatersolutions.com>
Date: Wed, 20 Sep 2023 06:40:08 -0400
Subject: [PATCH] Bug 34820: (QA follow-up) add filter, update CSV output

---
 koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt | 4 ++--
 tools/inventory.pl                                         | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
index f7c4084a6b3..1937604441d 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tools/inventory.tt
@@ -149,7 +149,7 @@
 
     <fieldset class="rows" id="optionalfilters">
             <legend>Optional filters for inventory list or comparing barcodes</legend>
-            <span class="hint">Scanned items are expected to match one of the selected <em>not for loan</em> critera if any are checked.</span>
+            <span class="hint">Scanned items are expected to match one of the selected <em>not for loan</em> criteria if any are checked.</span>
             <br/>
             <div id="statuses" style="display: block;">
                   [% FOREACH status IN statuses %]
@@ -313,7 +313,7 @@
                     <span>Found in wrong place</span><br/>
                 [% ELSIF problem.key == 'changestatus' %]
                     [% IF result.notforloan %]
-                        <span>Item has unselected <em>Not for loan</em> status <em>[% result.notforloan %]</em></span><br/>
+                        <span>Item has unselected <em>Not for loan</em> status <em>[% result.notforloan | html %]</em></span><br/>
                     [% ELSE %]
                         <span>Items has no <em>not for loan</em> status</span><br/>
                     [% END %]
diff --git a/tools/inventory.pl b/tools/inventory.pl
index ca52b8faba0..65f261feb73 100755
--- a/tools/inventory.pl
+++ b/tools/inventory.pl
@@ -411,7 +411,7 @@ if (defined $input->param('CSVexport') && $input->param('CSVexport') eq 'on'){
             if( $key eq 'wrongplace' ) {
                 $errstr .= "wrong place,";
             } elsif( $key eq 'changestatus' ) {
-                $errstr .= "unknown notforloan status,";
+                $errstr .= "unselected notforloan status $item->{notforloan},";
             } elsif( $key eq 'not_scanned' ) {
                 $errstr .= "missing,";
             } elsif( $key eq 'no_barcode' ) {
-- 
2.30.2