Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 2505: turning on "warnings" in catalogue/detail.pl

Browse source 
Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
This commit is contained in:
Andrew Moore 2008-12-17 16:32:46 -06:00 committed by Galen Charlton
parent 20a609b587
commit a5ca358d04
1 changed files with 18 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
catalogue/detail.pl
View file

@ -17,6 +17,7 @@
use strict;
use warnings;
use CGI;
use C4::Auth;
@ -93,7 +94,10 @@ foreach my $subscription (@subscriptions) {
GetLatestSerials( $subscription->{subscriptionid}, 3 );
push @subs, \%cell;
}
if ( defined $dat->{'itemtype'} ) {
$dat->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }{imageurl} );
}
$dat->{'count'} = scalar @items;
my $shelflocations = GetKohaAuthorisedValues('items.location', $fw);
my $collections = GetKohaAuthorisedValues('items.ccode' , $fw);
@ -105,7 +109,9 @@ foreach my $item (@items) {
$norequests = 0 unless ( ( $item->{'notforloan'} > 0 ) || ( $item->{'itemnotforloan'} > 0 ) );
# format some item fields for display
if ( defined $item->{'publictype'} ) {
$item->{ $item->{'publictype'} } = 1;
}
$item->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $item->{itype} }{imageurl} );
foreach (qw(datedue datelastseen onloan)) {
$item->{$_} = format_date($item->{$_});
@ -117,9 +123,9 @@ foreach my $item (@items) {
}
#get shelf location and collection code description if they are authorised value.
my $shelfcode = $item->{'location'};
$item->{'location'} = $shelflocations->{$shelfcode} if(defined($shelflocations) && exists($shelflocations->{$shelfcode}));
$item->{'location'} = $shelflocations->{$shelfcode} if ( defined( $shelfcode ) && defined($shelflocations) && exists( $shelflocations->{$shelfcode} ) );
my $ccode = $item->{'ccode'};
$item->{'ccode'} = $collections->{$ccode} if(defined($collections) && exists($collections->{$ccode}));
$item->{'ccode'} = $collections->{$ccode} if ( defined( $ccode ) && defined($collections) && exists( $collections->{$ccode} ) );
foreach (qw(ccode enumchron copynumber)) {
$itemfields{$_} = 1 if ( $item->{$_} );
}
@ -139,7 +145,7 @@ foreach my $item (@items) {
# Check the transit status
my ( $transfertwhen, $transfertfrom, $transfertto ) = GetTransfers($item->{itemnumber});
if ( $transfertwhen ne '' ) {
if ( defined( $transfertwhen ) && ( $transfertwhen ne '' ) ) {
$item->{transfertwhen} = format_date($transfertwhen);
$item->{transfertfrom} = $branches->{$transfertfrom}{branchname};
$item->{transfertto} = $branches->{$transfertto}{branchname};
@ -172,7 +178,7 @@ $template->param(
my @results = ( $dat, );
foreach ( keys %{$dat} ) {
$template->param( "$_" => $dat->{$_} . "" );
$template->param( "$_" => defined $dat->{$_} ? $dat->{$_} : '' );
}
$template->param(
@ -188,7 +194,7 @@ $template->param(
# XISBN Stuff
my $xisbn=$dat->{'isbn'};
$xisbn =~ /(\d*[X]*)/;
$xisbn =~ /(\d*[X]*)/ if ( $xisbn );
$template->param(amazonisbn => $1); # FIXME: so it is OK if the ISBN = 'XXXXX' ?
if (C4::Context->preference("FRBRizeEditions")==1) {
eval {