Bug 2505: turning on "warnings" in catalogue/detail.pl
[koha_fer] / catalogue / detail.pl
index bc251a3..006552e 100755 (executable)
 
 
 use strict;
-require Exporter;
+use warnings;
+
 use CGI;
 use C4::Auth;
-use C4::Date qw/format_date/;
+use C4::Dates qw/format_date/;
 use C4::Koha;
 use C4::Serials;    #uses getsubscriptionfrom biblionumber
 use C4::Output;
@@ -53,53 +54,54 @@ my $fw = GetFrameworkCode($biblionumber);
 ## get notes and subjects from MARC record
 my $marcflavour      = C4::Context->preference("marcflavour");
 my $record           = GetMarcBiblio($biblionumber);
+
+unless (defined($record)) {
+    print $query->redirect("/cgi-bin/koha/errors/404.pl");
+       exit;
+}
+
 my $marcnotesarray   = GetMarcNotes( $record, $marcflavour );
 my $marcauthorsarray = GetMarcAuthors( $record, $marcflavour );
 my $marcsubjctsarray = GetMarcSubjects( $record, $marcflavour );
 my $marcseriesarray  = GetMarcSeries($record,$marcflavour);
+my $marcurlsarray    = GetMarcUrls    ($record,$marcflavour);
+my $subtitle         = C4::Biblio::get_koha_field_from_marc('bibliosubtitle', 'subtitle', $record, '');
 
 # Get Branches, Itemtypes and Locations
 my $branches = GetBranches();
 my $itemtypes = GetItemTypes();
 
-my %locations;
 # FIXME: move this to a pm, check waiting status for holds
 my $dbh = C4::Context->dbh;
-my $lsch = $dbh->prepare("SELECT authorised_value,lib FROM authorised_values WHERE category = 'LOC'");
-$lsch->execute();
-while (my $ldata = $lsch->fetchrow_hashref ) {
-    $locations{ $ldata->{'authorised_value'} } = $ldata->{'lib'};
-}
 
 # change back when ive fixed request.pl
 my @items = &GetItemsInfo( $biblionumber, 'intra' );
 my $dat = &GetBiblioData($biblionumber);
 
-if (!$dat) { 
-    print $query->redirect("/cgi-bin/koha/koha-tmpl/errors/404.pl");
-       exit;
-}
-
 #coping with subscriptions
 my $subscriptionsnumber = CountSubscriptionFromBiblionumber($biblionumber);
 my @subscriptions       = GetSubscriptions( $dat->{title}, $dat->{issn}, $biblionumber );
-
-
-
 my @subs;
+$dat->{'serial'}=1 if $subscriptionsnumber;
 foreach my $subscription (@subscriptions) {
     my %cell;
     $cell{subscriptionid}    = $subscription->{subscriptionid};
     $cell{subscriptionnotes} = $subscription->{notes};
-
+       $cell{branchcode}        = $subscription->{branchcode};
+       $cell{hasalert}          = $subscription->{hasalert};
     #get the three latest serials.
     $cell{latestserials} =
       GetLatestSerials( $subscription->{subscriptionid}, 3 );
     push @subs, \%cell;
 }
-$dat->{imageurl} = getitemtypeimagesrc() . "/".$itemtypes->{ $dat->{itemtype} }{imageurl};
-$dat->{'count'} = @items;
-my @itemloop;
+
+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);
+my (@itemloop, %itemfields);
 my $norequests = 1;
 foreach my $item (@items) {
 
@@ -107,19 +109,26 @@ foreach my $item (@items) {
     $norequests = 0 unless ( ( $item->{'notforloan'} > 0 ) || ( $item->{'itemnotforloan'} > 0 ) );
 
     # format some item fields for display
-    $item->{ $item->{'publictype'} } = 1;
-    $item->{imageurl} = getitemtypeimagesrc() . "/".$itemtypes->{ $item->{itype} }{imageurl};
-    $item->{datedue} = format_date($item->{datedue});
-    $item->{datelastseen} = format_date($item->{datelastseen});
-    $item->{onloan} = format_date($item->{onloan});
-    $item->{locationname} = $locations{$item->{location}};
+    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->{$_});
+       }
     # item damaged, lost, withdrawn loops
     $item->{itemlostloop}= GetAuthorisedValues(GetAuthValCode('items.itemlost',$fw),$item->{itemlost}) if GetAuthValCode('items.itemlost',$fw);
     if ($item->{damaged}) {
         $item->{itemdamagedloop}= GetAuthorisedValues(GetAuthValCode('items.damaged',$fw),$item->{damaged}) if GetAuthValCode('items.damaged',$fw);
     }
-    #get collection code description, too
-    $item->{'ccode'}  = GetAuthorisedValueDesc('','',   $item->{'ccode'} ,'','','ccode');
+    #get shelf location and collection code description if they are authorised value.
+    my $shelfcode = $item->{'location'};
+    $item->{'location'} = $shelflocations->{$shelfcode} if ( defined( $shelfcode ) && defined($shelflocations) && exists( $shelflocations->{$shelfcode} ) );
+    my $ccode = $item->{'ccode'};
+    $item->{'ccode'} = $collections->{$ccode} if ( defined( $ccode ) && defined($collections) && exists( $collections->{$ccode} ) );
+    foreach (qw(ccode enumchron copynumber)) {
+        $itemfields{$_} = 1 if ( $item->{$_} );
+    }
 
     # checking for holds
     my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($item->{itemnumber});
@@ -130,16 +139,16 @@ foreach my $item (@items) {
         $item->{reservedate}     = format_date($reservedate);
         $item->{ReservedForBorrowernumber}     = $reservedfor;
         $item->{ReservedForSurname}     = $ItemBorrowerReserveInfo->{'surname'};
-        $item->{ReservedForFirstname}     = $ItemBorrowerReserveInfo->{'firstname'};
+        $item->{ReservedForFirstname}   = $ItemBorrowerReserveInfo->{'firstname'};
         $item->{ExpectedAtLibrary}     = $branches->{$expectedAt}{branchname};
     }
 
        # 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};
+        $item->{transfertto}   = $branches->{$transfertto}{branchname};
         $item->{nocancel} = 1;
     }
 
@@ -154,17 +163,22 @@ foreach my $item (@items) {
 }
 
 $template->param( norequests => $norequests );
-
-    $template->param(
-        MARCNOTES   => $marcnotesarray,
-        MARCSUBJCTS => $marcsubjctsarray,
-        MARCAUTHORS => $marcauthorsarray,
-        MARCSERIES  => $marcseriesarray
-    );
+$template->param(
+       MARCNOTES   => $marcnotesarray,
+       MARCSUBJCTS => $marcsubjctsarray,
+       MARCAUTHORS => $marcauthorsarray,
+       MARCSERIES  => $marcseriesarray,
+       MARCURLS => $marcurlsarray,
+       subtitle    => $subtitle,
+       itemdata_ccode      => $itemfields{ccode},
+       itemdata_enumchron  => $itemfields{enumchron},
+       itemdata_copynumber => $itemfields{copynumber},
+       volinfo                         => $itemfields{enumchron} || $dat->{'serial'} ,
+);
 
 my @results = ( $dat, );
 foreach ( keys %{$dat} ) {
-    $template->param( "$_" => $dat->{$_} . "" );
+    $template->param( "$_" => defined $dat->{$_} ? $dat->{$_} : '' );
 }
 
 $template->param(
@@ -176,10 +190,12 @@ $template->param(
     subscriptiontitle   => $dat->{title},
 );
 
+# $debug and $template->param(debug_display => 1);
+
 # XISBN Stuff
 my $xisbn=$dat->{'isbn'};
-$xisbn =~ s/(p|-| |:)//g;
-$template->param(amazonisbn => $xisbn);
+$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 {
         $template->param(
@@ -191,7 +207,7 @@ if (C4::Context->preference("FRBRizeEditions")==1) {
 }
 if ( C4::Context->preference("AmazonContent") == 1 ) {
     my $similar_products_exist;
-    my $amazon_details = &get_amazon_details( $xisbn );
+    my $amazon_details = &get_amazon_details( $xisbn, $record, $marcflavour );
     my $item_attributes = \%{$amazon_details->{Items}->{Item}->{ItemAttributes}};
     my $customer_reviews = \@{$amazon_details->{Items}->{Item}->{CustomerReviews}->{Review}};
     my @similar_products;
@@ -199,8 +215,10 @@ if ( C4::Context->preference("AmazonContent") == 1 ) {
         # do we have any of these isbns in our collection?
         my $similar_biblionumbers = get_biblionumber_from_isbn($similar_product->{ASIN});
         # verify that there is at least one similar item
-        $similar_products_exist++ if ${@$similar_biblionumbers}[0];
-        push @similar_products, +{ similar_biblionumbers => $similar_biblionumbers, title => $similar_product->{Title}, ASIN => $similar_product->{ASIN}  };
+               if (scalar(@$similar_biblionumbers)){            
+                       $similar_products_exist++ if ($similar_biblionumbers && $similar_biblionumbers->[0]);
+            push @similar_products, +{ similar_biblionumbers => $similar_biblionumbers, title => $similar_product->{Title}, ASIN => $similar_product->{ASIN}  };
+        }
     }
     my $editorial_reviews = \@{$amazon_details->{Items}->{Item}->{EditorialReviews}->{EditorialReview}};
     my $average_rating = $amazon_details->{Items}->{Item}->{CustomerReviews}->{AverageRating};