X-Git-Url: http://koha-dev.rot13.org:8081/gitweb/?a=blobdiff_plain;f=catalogue%2Fdetail.pl;h=03271b47d7f020f4c5e9eb79de8152cefd0bc64c;hb=716442708b199f07a85df75434b1fa99d9ccaf0f;hp=7dd96dd3770fa6a78dd853cf2eaa1f4580a921e1;hpb=5d01574d431a6b04214009b93a29c9b16122b617;p=koha_gimpoz diff --git a/catalogue/detail.pl b/catalogue/detail.pl index 7dd96dd377..03271b47d7 100755 --- a/catalogue/detail.pl +++ b/catalogue/detail.pl @@ -17,7 +17,8 @@ use strict; -require Exporter; +use warnings; + use CGI; use C4::Auth; use C4::Dates qw/format_date/; @@ -29,14 +30,17 @@ use C4::Items; use C4::Circulation; use C4::Branch; use C4::Reserves; -use C4::Members; +use C4::Members; # to use GetMember use C4::Serials; -use C4::XISBN qw(get_xisbns get_biblionumber_from_isbn get_biblio_from_xisbn); -use C4::Amazon; +use C4::XISBN qw(get_xisbns get_biblionumber_from_isbn); +use C4::External::Amazon; +use C4::Search; # enabled_staff_search_views +use C4::VirtualShelves; +use C4::XSLT; # use Smart::Comments; -my $query = new CGI; +my $query = CGI->new(); my ( $template, $borrowernumber, $cookie ) = get_template_and_user( { template_name => "catalogue/detail.tmpl", @@ -48,85 +52,151 @@ my ( $template, $borrowernumber, $cookie ) = get_template_and_user( ); my $biblionumber = $query->param('biblionumber'); -my $fw = GetFrameworkCode($biblionumber); +my $record = GetMarcBiblio($biblionumber); + +if ( not defined $record ) { + # biblionumber invalid -> report and exit + $template->param( unknownbiblionumber => 1, + biblionumber => $biblionumber ); + output_html_with_http_headers $query, $cookie, $template->output; + exit; +} -## get notes and subjects from MARC record -my $marcflavour = C4::Context->preference("marcflavour"); -my $record = GetMarcBiblio($biblionumber); +if($query->cookie("holdfor")){ + my $holdfor_patron = GetMember('borrowernumber' => $query->cookie("holdfor")); + $template->param( + holdfor => $query->cookie("holdfor"), + holdfor_surname => $holdfor_patron->{'surname'}, + holdfor_firstname => $holdfor_patron->{'firstname'}, + holdfor_cardnumber => $holdfor_patron->{'cardnumber'}, + ); +} + +my $fw = GetFrameworkCode($biblionumber); +my $showallitems = $query->param('showallitems'); +my $marcflavour = C4::Context->preference("marcflavour"); -unless (defined($record)) { - print $query->redirect("/cgi-bin/koha/errors/404.pl"); - exit; +# XSLT processing of some stuff +if (C4::Context->preference("XSLTDetailsDisplay") ) { + $template->param('XSLTDetailsDisplay' =>'1', + 'XSLTBloc' => XSLTParse4Display($biblionumber, $record, 'Detail','intranet') ); } +$template->param( 'SpineLabelShowPrintOnBibDetails' => C4::Context->preference("SpineLabelShowPrintOnBibDetails") ); + +# 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, +); + my $marcnotesarray = GetMarcNotes( $record, $marcflavour ); +my $marcisbnsarray = GetMarcISBN( $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, ''); +my $subtitle = GetRecordValue('subtitle', $record, $fw); # Get Branches, Itemtypes and Locations my $branches = GetBranches(); my $itemtypes = GetItemTypes(); - -# FIXME: move this to a pm, check waiting status for holds my $dbh = C4::Context->dbh; -# change back when ive fixed request.pl -my @items = &GetItemsInfo( $biblionumber, 'intra' ); +# 'intra' param included, even though it's not respected in GetItemsInfo currently +my @all_items= GetItemsInfo($biblionumber, 'intra'); +my @items; +for my $itm (@all_items) { + push @items, $itm unless ( $itm->{itemlost} && GetHideLostItemsPreference($borrowernumber) && !$showallitems); +} my $dat = &GetBiblioData($biblionumber); +# get count of holds +my ( $holdcount, $holds ) = GetReservesFromBiblionumber($biblionumber,1); + #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}; + $cell{branchcode} = $subscription->{branchcode}; + $cell{branchname} = GetBranchName($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'} = scalar @items; + +if ( defined $dat->{'itemtype'} ) { + $dat->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }{imageurl} ); +} + +$dat->{'count'} = scalar @all_items; +$dat->{'showncount'} = scalar @items; +$dat->{'hiddencount'} = scalar @all_items - 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) { + $item->{homebranch} = GetBranchName($item->{homebranch}); + # 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}; + 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($shelflocations) && exists($shelflocations->{$shelfcode})); - my $ccode= $item->{'ccode'}; - $item->{'ccode'} = $collections->{$ccode} if(defined($collections) && exists($collections->{$ccode})); - foreach (qw(ccode enumchron copynumber)) { - $itemfields{$_} = 1 if($item->{$_}); - } + 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 itemnotes uri)) { + $itemfields{$_} = 1 if ( $item->{$_} ); + } # checking for holds my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($item->{itemnumber}); my $ItemBorrowerReserveInfo = GetMemberDetails( $reservedfor, 0); + + if (C4::Context->preference('HidePatronName')){ + $item->{'hidepatronname'} = 1; + } if ( defined $reservedate ) { $item->{backgroundcolor} = 'reserved'; @@ -134,12 +204,13 @@ foreach my $item (@items) { $item->{ReservedForBorrowernumber} = $reservedfor; $item->{ReservedForSurname} = $ItemBorrowerReserveInfo->{'surname'}; $item->{ReservedForFirstname} = $ItemBorrowerReserveInfo->{'firstname'}; - $item->{ExpectedAtLibrary} = $branches->{$expectedAt}{branchname}; + $item->{ExpectedAtLibrary} = $branches->{$expectedAt}{branchname}; + $item->{Reservedcardnumber} = $ItemBorrowerReserveInfo->{'cardnumber'}; } # 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}; @@ -163,18 +234,54 @@ $template->param( MARCAUTHORS => $marcauthorsarray, MARCSERIES => $marcseriesarray, MARCURLS => $marcurlsarray, + MARCISBNS => $marcisbnsarray, subtitle => $subtitle, itemdata_ccode => $itemfields{ccode}, itemdata_enumchron => $itemfields{enumchron}, + itemdata_uri => $itemfields{uri}, itemdata_copynumber => $itemfields{copynumber}, - volinfo => $itemfields{enumchron} || $dat->{'serial'} , + volinfo => $itemfields{enumchron}, + itemdata_itemnotes => $itemfields{itemnotes}, + z3950_search_params => C4::Search::z3950_search_args($dat), + holdcount => $holdcount, + C4::Search::enabled_staff_search_views, ); +if (C4::Context->preference("AlternateHoldingsField") && scalar @items == 0) { + my $fieldspec = C4::Context->preference("AlternateHoldingsField"); + my $subfields = substr $fieldspec, 3; + my $holdingsep = C4::Context->preference("AlternateHoldingsSeparator") || ' '; + my @alternateholdingsinfo = (); + my @holdingsfields = $record->field(substr $fieldspec, 0, 3); + + for my $field (@holdingsfields) { + my %holding = ( holding => '' ); + my $havesubfield = 0; + for my $subfield ($field->subfields()) { + if ((index $subfields, $$subfield[0]) >= 0) { + $holding{'holding'} .= $holdingsep if (length $holding{'holding'} > 0); + $holding{'holding'} .= $$subfield[1]; + $havesubfield++; + } + } + if ($havesubfield) { + push(@alternateholdingsinfo, \%holding); + } + } + + $template->param( + ALTERNATEHOLDINGS => \@alternateholdingsinfo, + ); +} + my @results = ( $dat, ); foreach ( keys %{$dat} ) { - $template->param( "$_" => $dat->{$_} . "" ); + $template->param( "$_" => defined $dat->{$_} ? $dat->{$_} : '' ); } +# does not work: my %views_enabled = map { $_ => 1 } $template->query(loop => 'EnableViews'); +# method query not found?!?! + $template->param( itemloop => \@itemloop, biblionumber => $biblionumber, @@ -186,40 +293,64 @@ $template->param( # $debug and $template->param(debug_display => 1); +# Lists + +if (C4::Context->preference("virtualshelves") ) { + $template->param( 'GetShelves' => GetBibliosShelves( $biblionumber ) ); +} + # 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 $similar_products_exist; - my $amazon_details = &get_amazon_details( $xisbn ); - 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} }; +if ( C4::Context->preference("AmazonEnabled") == 1 ) { + $template->param( AmazonTld => get_amazon_tld() ); + my $amazon_reviews = C4::Context->preference("AmazonReviews"); + my $amazon_similars = C4::Context->preference("AmazonSimilarItems"); + my @services; + if ( $amazon_reviews ) { + $template->param( AmazonReviews => 1 ); + push( @services, 'EditorialReview' ); + } + if ( $amazon_similars ) { + $template->param( AmazonSimilarItems => 1 ); + push( @services, 'Similarities' ); + } + my $amazon_details = &get_amazon_details( $isbn, $record, $marcflavour, \@services ); + if ( $amazon_similars ) { + my $similar_products_exist; + my @similar_products; + for my $similar_product (@{$amazon_details->{Items}->{Item}->[0]->{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( AmazonSimilarItems => $similar_products_exist ); + $template->param( AMAZON_SIMILAR_PRODUCTS => \@similar_products ); + } + if ( $amazon_reviews ) { + my $item = $amazon_details->{Items}->{Item}->[0]; + my $editorial_reviews = \@{ $item->{EditorialReviews}->{EditorialReview} }; + #my $customer_reviews = \@{$amazon_details->{Items}->{Item}->[0]->{CustomerReviews}->{Review}}; + #my $average_rating = $amazon_details->{Items}->{Item}->[0]->{CustomerReviews}->{AverageRating} || 0; + #$template->param( amazon_average_rating => $average_rating * 20 ); + #$template->param( AMAZON_CUSTOMER_REVIEWS => $customer_reviews ); + $template->param( AMAZON_EDITORIAL_REVIEWS => $editorial_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 ); } + +# Get OPAC URL +if (C4::Context->preference('OPACBaseURL')){ + $template->param( OpacUrl => C4::Context->preference('OPACBaseURL') ); +} + output_html_with_http_headers $query, $cookie, $template->output;