X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=catalogue%2Fdetail.pl;h=e0f5694cf9301a072d2e32b00b38c686d7f5b9a4;hb=faf7c886f29d5c50fd747ea0627aea794f847bf5;hp=ac596706dfa4cc627bd724441b2a6a0a58b4b752;hpb=9222f7a6ca80d2a74cf09872e362b116835fc02b;p=koha_gimpoz diff --git a/catalogue/detail.pl b/catalogue/detail.pl index ac596706df..e0f5694cf9 100755 --- a/catalogue/detail.pl +++ b/catalogue/detail.pl @@ -17,10 +17,11 @@ 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; @@ -31,12 +32,12 @@ use C4::Branch; use C4::Reserves; use C4::Members; use C4::Serials; -use C4::XISBN qw(get_xisbns get_biblio_from_xisbn); -use C4::Amazon; +use C4::XISBN qw(get_xisbns get_biblionumber_from_isbn); +use C4::External::Amazon; # use Smart::Comments; -my $query = new CGI; +my $query = CGI->new(); my ( $template, $borrowernumber, $cookie ) = get_template_and_user( { template_name => "catalogue/detail.tmpl", @@ -53,73 +54,102 @@ my $fw = GetFrameworkCode($biblionumber); ## get notes and subjects from MARC record my $marcflavour = C4::Context->preference("marcflavour"); my $record = GetMarcBiblio($biblionumber); + +# some useful variables for enhanced content; +# in each case, we're grabbing the first value we find in +# the record and normalizing it +my $upc = GetNormalizedUPC($record,$marcflavour); +my $ean = GetNormalizedEAN($record,$marcflavour); +my $oclc = GetNormalizedOCLCNumber($record,$marcflavour); +my $isbn = GetNormalizedISBN(undef,$record,$marcflavour); + +$template->param( + normalized_upc => $upc, + normalized_ean => $ean, + normalized_oclc => $oclc, + normalized_isbn => $isbn, +); + +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; + my $serials_to_display; $cell{subscriptionid} = $subscription->{subscriptionid}; $cell{subscriptionnotes} = $subscription->{notes}; - + $cell{branchcode} = $subscription->{branchcode}; + $cell{hasalert} = $subscription->{hasalert}; #get the three latest serials. + $serials_to_display = $subscription->{staffdisplaycount}; + $serials_to_display = C4::Context->preference('StaffSerialIssueDisplayCount') unless $serials_to_display; + $cell{staffdisplaycount} = $serials_to_display; $cell{latestserials} = - GetLatestSerials( $subscription->{subscriptionid}, 3 ); + GetLatestSerials( $subscription->{subscriptionid}, $serials_to_display ); 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; +my $authvalcode_items_itemlost = GetAuthValCode('items.itemlost',$fw); +my $authvalcode_items_damaged = GetAuthValCode('items.damaged', $fw); foreach my $item (@items) { # can place holds defaults to yes $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} = defined $item->{itype} ? 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); + $item->{itemlostloop} = GetAuthorisedValues($authvalcode_items_itemlost, $item->{itemlost}) if $authvalcode_items_itemlost; if ($item->{damaged}) { - $item->{itemdamagedloop}= GetAuthorisedValues(GetAuthValCode('items.damaged',$fw),$item->{damaged}) if GetAuthValCode('items.damaged',$fw); + $item->{itemdamagedloop} = GetAuthorisedValues($authvalcode_items_damaged, $item->{damaged}) if $authvalcode_items_damaged; + } + #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->{$_} ); } - #get collection code description, too - $item->{'ccode'} = GetAuthorisedValueDesc('','', $item->{'ccode'} ,'','','ccode'); # checking for holds my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($item->{itemnumber}); @@ -130,16 +160,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 +184,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,59 +211,38 @@ $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); if (C4::Context->preference("FRBRizeEditions")==1) { eval { $template->param( - xisbn => $xisbn, - XISBNS => get_xisbns($xisbn) + XISBNS => get_xisbns($isbn) ); }; if ($@) { warn "XISBN Failed $@"; } } -if ( C4::Context->preference("AmazonContent") == 1 ) { - my $amazon_details = &get_amazon_details( $xisbn ); - foreach my $result ( @{ $amazon_details->{Details} } ) { - $template->param( item_description => $result->{ProductDescription} ); - $template->param( image => $result->{ImageUrlMedium} ); - $template->param( list_price => $result->{ListPrice} ); - $template->param( amazon_url => $result->{url} ); - } - - my @products; - my @reviews; - for my $details ( @{ $amazon_details->{Details} } ) { - - next unless $details->{SimilarProducts}; - for my $product ( @{ $details->{SimilarProducts}->{Product} } ) { - if (C4::Context->preference("AmazonSimilarItems") ) { - my @xisbns; - if (C4::Context->preference("XISBNAmazonSimilarItems") ) { - @xisbns = @{get_xisbns($product)}; - } - else { - push @xisbns, get_biblio_from_xisbn($product); - } - push @products, +{ product => \@xisbns }; - } - } - next unless $details->{Reviews}; - for my $product ( @{ $details->{Reviews}->{AvgCustomerRating} } ) { - $template->param( rating => $product * 20 ); - } - for my $reviews ( @{ $details->{Reviews}->{CustomerReview} } ) { - push @reviews, - +{ - summary => $reviews->{Summary}, - comment => $reviews->{Comment}, - }; +if ( C4::Context->preference("AmazonEnabled") == 1 ) { + my $similar_products_exist; + my $amazon_details = &get_amazon_details( $isbn, $record, $marcflavour ); + my $item_attributes = \%{$amazon_details->{Items}->{Item}->{ItemAttributes}}; + my $customer_reviews = \@{$amazon_details->{Items}->{Item}->{CustomerReviews}->{Review}}; + my @similar_products; + for my $similar_product (@{$amazon_details->{Items}->{Item}->{SimilarProducts}->{SimilarProduct}}) { + # 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 + 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} }; } } - $template->param( SIMILAR_PRODUCTS => \@products ); - $template->param( AMAZONREVIEWS => \@reviews ); + my $editorial_reviews = \@{$amazon_details->{Items}->{Item}->{EditorialReviews}->{EditorialReview}}; + my $average_rating = $amazon_details->{Items}->{Item}->{CustomerReviews}->{AverageRating}; + $template->param( AmazonSimilarItems => $similar_products_exist ); + $template->param( amazon_average_rating => $average_rating * 20); + $template->param( AMAZON_CUSTOMER_REVIEWS => $customer_reviews ); + $template->param( AMAZON_SIMILAR_PRODUCTS => \@similar_products ); + $template->param( AMAZON_EDITORIAL_REVIEWS => $editorial_reviews ); } - output_html_with_http_headers $query, $cookie, $template->output;