[Koha-patches] [PATCH] Bug 8913 [Follow-up] Improve acquisitions navigation

Owen Leonard oleonard at myacpl.org
Thu Oct 18 21:40:20 CEST 2012


This follow up adds the vendor menu and toolbar to booksellers.pl
when there is only one "search result" (i.e. a vendor id is passed).

- Menu appears when booksellerid variable is present
- Redundant heading removed
- Additional variables added to enable proper display of the toolbar
---
 acqui/booksellers.pl                               |    3 ++-
 .../prog/en/modules/acqui/booksellers.tt           |    7 ++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/acqui/booksellers.pl b/acqui/booksellers.pl
index 5a1c248..1c66ba0 100755
--- a/acqui/booksellers.pl
+++ b/acqui/booksellers.pl
@@ -87,7 +87,8 @@ my $supplier_count = @suppliers;
 if ( $supplier_count == 1 ) {
     $template->param(
         supplier_name => $suppliers[0]->{'name'},
-        booksellerid  => $suppliers[0]->{'booksellerid'}
+        booksellerid  => $suppliers[0]->{'id'},
+        basketcount   => $suppliers[0]->{'basketcount'}
     );
 }
 
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt
index 4077b5d..b0e4549 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/booksellers.tt
@@ -47,9 +47,7 @@ $(document).ready(function() {
     <div class="yui-b">
     
 [% INCLUDE 'acquisitions-toolbar.inc' %]
-[% IF ( count == 1 ) %]	
-<h1> [% supplier_name %] </h1>
-[% ELSE %]
+[% UNLESS ( count == 1 ) %]
 <h1>You searched on <b>vendor [% supplier %],</b> [% count %] results found</h1>
 [% END %]
 [% IF ( loop_suppliers.size ) %]
@@ -139,6 +137,9 @@ $(document).ready(function() {
 </div>
 </div>
 <div class="yui-b">
+[% IF ( booksellerid ) %]
+[% INCLUDE 'vendor-menu.inc' %]
+[% END %]
 [% INCLUDE 'acquisitions-menu.inc' %]
 </div>
 </div>
-- 
1.7.9.5


More information about the Koha-patches mailing list