From a0b20de83919351edea45464432a31e13fe3243d Mon Sep 17 00:00:00 2001
From: Nick Clemens <nick@bywatersolutions.com>
Date: Tue, 11 Apr 2017 16:09:27 -0400
Subject: [PATCH] Bug 18367 - Fix untranslatable string from Bug 18264

Restore display of message when no barcode found

To test:
1 - Go to course reserves
2 - Add an item to a course
3 - Try adding an invalid barcode - should get an alert
4 - Try force adding an item with itemnumber:
    http://localhost:8081/cgi-bin/koha/course_reserves/add_items.pl?op=lookpup&course_id=11&itemnumber=1234523546
5 - Should recieve an error
---
 course_reserves/add_items.pl                                     | 8 +++++---
 .../prog/en/modules/course_reserves/add_items-step1.tt           | 9 +++++++--
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/course_reserves/add_items.pl b/course_reserves/add_items.pl
index 89cbb47..ed37654 100755
--- a/course_reserves/add_items.pl
+++ b/course_reserves/add_items.pl
@@ -54,9 +54,11 @@ my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
         flagsrequired   => { coursereserves => 'add_reserves' },
     }
 );
-my $inumber = $itemnumber ? "(blank) (itemnumber:$itemnumber)" : "";
-$template->param( ERROR_BARCODE_NOT_FOUND => $barcode . $inumber )
-  unless ( $barcode && !$itemnumber && $item && $action eq 'lookup' );
+
+unless ( $item && $action eq 'lookup' ){
+    $template->param( ERROR_ITEM_NOT_FOUND => 1 );
+    $template->param( UNKNOWN_BARCODE => $barcode ) if $barcode;
+}
 
 $template->param( course => GetCourse($course_id) );
 
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step1.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step1.tt
index b503636..daf1602 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step1.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/add_items-step1.tt
@@ -11,8 +11,13 @@
 
 <div id="doc" class="yui-t7">
     <div id="bd">
-        [% IF ERROR_BARCODE_NOT_FOUND %]
-            <div class="dialog alert">No item found with barcode [% ERROR_BARCODE_NOT_FOUND %]</div>
+
+        [% IF ERROR_ITEM_NOT_FOUND %]
+            [% IF UNKNOWN_BARCODE %]
+                <div class="dialog alert">No item found with barcode [% UNKNOWN_BARCODE %]</div>
+            [% ELSE %]
+                 <div class="dialog alert">No item found</div>
+            [% END %]
         [% END %]
 
         <form method="post" action="/cgi-bin/koha/course_reserves/add_items.pl">
-- 
2.1.4