Bug 13238: Improve heading on vendor search when searching for all vendors
authorAleisha <aleishaamohia@hotmail.com>
Wed, 2 Mar 2016 00:36:49 +0000 (00:36 +0000)
committerBrendan A Gallagher <brendan@bywatersolutions.com>
Wed, 23 Mar 2016 23:06:19 +0000 (23:06 +0000)
To test:
1) Go to Acquisitions and do an empty vendor search
2) Should now read "Vendor search: X results found"
3) Enter a search term and search for vendor
4) Should now read "Vendor search: Y results found for 'search term'"

Sponsored-by: Catalyst IT
NOTE: two nested if/else/end's could make it result/results,
      as suggested in comment #3. However, this meets my understanding
      of the problem as defined in comment #0.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Brendan A Gallagher <brendan@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt

index edb26d9..20e505b 100644 (file)
@@ -52,7 +52,11 @@ $(document).ready(function() {
     
 [% INCLUDE 'acquisitions-toolbar.inc' %]
 [% UNLESS ( count == 1 ) %]
-<h1>You searched on <b>vendor [% supplier %],</b> [% count %] results found</h1>
+    [% IF ( supplier.length < 1 ) %]
+        <h1>Vendor search: [% count %] results found</h1>
+    [% ELSE %]
+        <h1>Vendor search: [% count %] results found for '[% supplier %]'</h1>
+    [% END %]
 [% END %]
 [% IF ( loop_suppliers.size ) %]
     [% UNLESS (count == 1) %]