Bugzilla – Attachment 12918 Details for
Bug 8913
Improve acquisitions navigation
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 8913 [Follow-up] Improve acquisitions navigation
Bug-8913-Follow-up-Improve-acquisitions-navigation.patch (text/plain), 2.03 KB, created by
Owen Leonard
on 2012-10-18 19:40:49 UTC
(
hide
)
Description:
Bug 8913 [Follow-up] Improve acquisitions navigation
Filename:
MIME Type:
Creator:
Owen Leonard
Created:
2012-10-18 19:40:49 UTC
Size:
2.03 KB
patch
obsolete
>From 24f835b153841355174d1e089d43868a1de6b190 Mon Sep 17 00:00:00 2001 >From: Owen Leonard <oleonard@myacpl.org> >Date: Thu, 18 Oct 2012 15:34:23 -0400 >Subject: [PATCH] Bug 8913 [Follow-up] Improve acquisitions navigation >Content-Type: text/plain; charset="utf-8" > >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
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 8913
:
12890
|
12892
|
12914
|
12918
|
12928
|
12929
|
15036
|
15057
|
15246
|
15281