Bug 11692: fix display of additional bib information in OPAC "most popular" pag

The template for the OPAC's "most popular" page includes several
variables which are not passed by the script, including
biblioitems.size. This patch adds the right columns to the query and
standardizes the output somewhat based on non-XSLT opac-detail.

To test, apply the patch and view the "most popular" page in both prog
and bootstrap. You should see correct publisher, copyright, pages, and
size information.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Tested in all three themes and works lovely

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Improves the bibliographic display on the 'Most popular' page
in the OPAC. Passes all tests and QA script.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
Owen Leonard 2014-02-05 12:16:10 -05:00 committed by Galen Charlton
parent b2d08f8359
commit 5aabe570f4
3 changed files with 18 additions and 10 deletions

View File

@ -64,11 +64,15 @@
[% FOREACH results_loo IN results_loop %]
<tr>
<td><a class="title" href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% results_loo.biblionumber %]">[% results_loo.title |html %]</a><p>[% results_loo.author %]
[% IF ( results_loo.publicationyear ) %] - [% results_loo.publicationyear %][% END %]
[% IF ( results_loo.publishercode ) %]- [% results_loo.publishercode %][% END %] [% IF ( results_loo.seriestitle ) %]([% results_loo.seriestitle %])[% END %]
[% IF ( results_loo.place ) %] ; [% results_loo.place %][% END %]
[% IF ( results_loo.place ) %][% results_loo.place %][% END %]
[% IF ( results_loo.publicationyear ) %]
[% results_loo.publicationyear %]
[% ELSIF ( results_loo.copyrightdate ) %]
[% results_loo.copyrightdate %]
[% END %]
[% IF ( results_loo.pages ) %] - [% results_loo.pages %][% END %]
[% IF ( results_loo.size ) %] ; [% results_loo.size %][% END %]</p>
[% IF ( results_loo.size ) %][% results_loo.size %][% END %]</p>
</td>
<td>
[% IF ( results_loo.description ) %]

View File

@ -72,11 +72,15 @@ function Dopop(link) {
<tbody>[% FOREACH results_loo IN results_loop %]
<tr>
<td><a href="/cgi-bin/koha/opac-detail.pl?biblionumber=[% results_loo.biblionumber %]">[% results_loo.title |html %]</a><p>[% results_loo.author %]
[% IF ( results_loo.publicationyear ) %] - [% results_loo.publicationyear %][% END %]
[% IF ( results_loo.publishercode ) %]- [% results_loo.publishercode %][% END %] [% IF ( results_loo.seriestitle ) %]([% results_loo.seriestitle %])[% END %]
[% IF ( results_loo.place ) %] ; [% results_loo.place %][% END %]
[% IF ( results_loo.place ) %][% results_loo.place %][% END %]
[% IF ( results_loo.publicationyear ) %]
[% results_loo.publicationyear %]
[% ELSIF ( results_loo.copyrightdate ) %]
[% results_loo.copyrightdate %]
[% END %]
[% IF ( results_loo.pages ) %] - [% results_loo.pages %][% END %]
[% IF ( results_loo.size ) %] ; [% results_loo.size %][% END %]</p>
[% IF ( results_loo.size ) %][% results_loo.size %][% END %]</p>
</td>
<td>[% results_loo.description %]</td>
<td>[% results_loo.tot %]</td>

View File

@ -81,8 +81,8 @@ if($advanced_search_types eq 'ccode'){
$whereclause .= ' AND authorised_values.authorised_value='.$dbh->quote($itemtype) if $itemtype;
$query = "SELECT datecreated, biblio.biblionumber, title,
author, sum( items.issues ) AS tot, biblioitems.itemtype,
biblioitems.publishercode,biblioitems.publicationyear,
authorised_values.lib as description
biblioitems.publishercode, biblioitems.place, biblioitems.publicationyear, biblio.copyrightdate,
authorised_values.lib as description, biblioitems.pages, biblioitems.size
FROM biblio
LEFT JOIN items USING (biblionumber)
LEFT JOIN biblioitems USING (biblionumber)
@ -107,8 +107,8 @@ if($advanced_search_types eq 'ccode'){
}
$query = "SELECT datecreated, biblio.biblionumber, title,
author, sum( items.issues ) AS tot, biblioitems.itemtype,
biblioitems.publishercode,biblioitems.publicationyear,
itemtypes.description
biblioitems.publishercode, biblioitems.place, biblioitems.publicationyear, biblio.copyrightdate,
itemtypes.description, biblioitems.pages, biblioitems.size
FROM biblio
LEFT JOIN items USING (biblionumber)
LEFT JOIN biblioitems USING (biblionumber)