use strict;
-require Exporter;
+use warnings;
+
use CGI;
use C4::Auth;
use C4::Dates qw/format_date/;
use C4::Reserves;
use C4::Members;
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 Smart::Comments;
-my $query = new CGI;
+my $query = CGI->new();
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
{
template_name => "catalogue/detail.tmpl",
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;
# 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
$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} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }{imageurl} );
+
+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} = getitemtypeimagelocation( 'intranet', $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)) {
+ $itemfields{$_} = 1 if ( $item->{$_} );
+ }
# checking for holds
my ($reservedate,$reservedfor,$expectedAt) = GetReservesFromItemnumber($item->{itemnumber});
# 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};
itemdata_enumchron => $itemfields{enumchron},
itemdata_copynumber => $itemfields{copynumber},
volinfo => $itemfields{enumchron} || $dat->{'serial'} ,
+ z3950_search_params => C4::Search::z3950_search_args($dat),
+ C4::Search::enabled_staff_search_views,
);
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,
# $debug and $template->param(debug_display => 1);
# XISBN Stuff
-my $xisbn=$dat->{'isbn'};
-$xisbn =~ /(\d*[X]*)/;
-$template->param(amazonisbn => $1); # FIXME: so it is OK if the ISBN = 'XXXXX' ?
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 );
}
output_html_with_http_headers $query, $cookie, $template->output;