[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