From bc1cd0a656269ec0503f11ca17e955b09d44c456 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 26 Oct 2012 09:14:15 -0400 Subject: [PATCH] Bug 8710 - Don't show the images tab in the OPAC if the record has no local images For unknown reasons, having ListImagesForBiblio return undef when there are no images still results in a variable being passed to the template which evaluates as true, with a size of 1. This patch alters ListImagesForBiblio to remove the "return undef" condition, allowing the template to evaluate images as false and show no tab. To test, turn on local cover images and view records in the OPAC which do and do not have cover images attached. Images should display as expected when present, and no images tab should appear on records which have none. Signed-off-by: Melia Meggs Signed-off-by: Jonathan Druart Signed-off-by: Jared Camins-Esakov --- C4/Images.pm | 12 +++--------- koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt | 6 +++--- 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/C4/Images.pm b/C4/Images.pm index 8b17f1cd2f..4bae00ca7a 100644 --- a/C4/Images.pm +++ b/C4/Images.pm @@ -131,16 +131,10 @@ sub ListImagesForBiblio { my $query = 'SELECT imagenumber FROM biblioimages WHERE biblionumber = ?'; my $sth = $dbh->prepare($query); $sth->execute($biblionumber); - warn "Database error!" if $sth->errstr; - if ( !$sth->errstr && $sth->rows > 0 ) { - while ( my $row = $sth->fetchrow_hashref ) { - push @imagenumbers, $row->{'imagenumber'}; - } - return @imagenumbers; - } - else { - return undef; + while ( my $row = $sth->fetchrow_hashref ) { + push @imagenumbers, $row->{'imagenumber'}; } + return @imagenumbers; } =head2 DelImage diff --git a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt index 3ed3144c94..17658b9f3b 100644 --- a/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt +++ b/koha-tmpl/opac-tmpl/prog/en/modules/opac-detail.tt @@ -712,9 +712,9 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %] -[% IF ( OPACLocalCoverImages ) %][% IF ( localimages ) %] +[% IF ( OPACLocalCoverImages && localimages.size ) %]
  • Images
  • -[% END %][% END %] +[% END %] [% IF ( serialcollection ) %] @@ -1149,7 +1149,7 @@ YAHOO.util.Event.onContentReady("furtherm", function () { [% END %][% END %] -[% IF ( OPACLocalCoverImages ) %] +[% IF ( OPACLocalCoverImages && localimages.size ) %]

    Click on an image to view it in the image viewer

    [% FOREACH image IN localimages %] -- 2.39.5