use CGI qw ( -utf8 );
use HTML::Entities;
-use C4::Acquisition qw( GetHistory );
+use Try::Tiny;
use C4::Auth;
+use C4::Context;
use C4::Koha;
use C4::Serials; #uses getsubscriptionfrom biblionumber
use C4::Output;
use C4::Items;
use C4::Circulation;
use C4::Reserves;
-use C4::Members; # to use GetMember
use C4::Serials;
-use C4::XISBN qw(get_xisbns get_biblionumber_from_isbn);
+use C4::XISBN qw(get_xisbns);
use C4::External::Amazon;
-use C4::Search; # enabled_staff_search_views
+use C4::Search; # enabled_staff_search_views
use C4::Tags qw(get_tags);
use C4::XSLT;
use C4::Images;
use C4::HTML5Media;
use C4::CourseReserves qw(GetItemCourseReservesInfo);
use C4::Acquisition qw(GetOrdersByBiblionumber);
+use Koha::AuthorisedValues;
+use Koha::Biblios;
+use Koha::Items;
+use Koha::ItemTypes;
+use Koha::Patrons;
use Koha::Virtualshelves;
+use Koha::Plugins;
my $query = CGI->new();
}
);
+# Determine if we should be offering any enhancement plugin buttons
+if ( C4::Context->config('enable_plugins') ) {
+ # Only pass plugins that can offer a toolbar button
+ my @plugins = Koha::Plugins->new()->GetPlugins({
+ method => 'intranet_catalog_biblio_enhancements_toolbar_button'
+ });
+
+ my @tab_plugins = Koha::Plugins->new()->GetPlugins({
+ method => 'intranet_catalog_biblio_tab',
+ });
+ my @tabs;
+ foreach my $tab_plugin (@tab_plugins) {
+ my @biblio_tabs;
+
+ try {
+ @biblio_tabs = $tab_plugin->intranet_catalog_biblio_tab();
+ foreach my $tab (@biblio_tabs) {
+ my $tab_id = 'tab-' . $tab->title;
+ $tab_id =~ s/[^0-9A-Za-z]+/-/g;
+ $tab->id( $tab_id );
+ push @tabs, $tab,
+ }
+ }
+ catch {
+ warn "Error calling 'intranet_catalog_biblio_tab' on the " . $tab_plugin->{class} . "plugin ($_)";
+ };
+ }
+
+ $template->param(
+ plugins => \@plugins,
+ tabs => \@tabs,
+ );
+}
+
my $biblionumber = $query->param('biblionumber');
$biblionumber = HTML::Entities::encode($biblionumber);
-my $record = GetMarcBiblio($biblionumber);
+my $record = GetMarcBiblio({ biblionumber => $biblionumber });
+my $biblio = Koha::Biblios->find( $biblionumber );
+$template->param( 'biblio', $biblio );
if ( not defined $record ) {
# biblionumber invalid -> report and exit
exit;
}
+eval { $biblio->metadata->record };
+$template->param( decoding_error => $@ );
+
if($query->cookie("holdfor")){
- my $holdfor_patron = GetMember('borrowernumber' => $query->cookie("holdfor"));
+ my $holdfor_patron = Koha::Patrons->find( $query->cookie("holdfor") );
$template->param(
+ # FIXME Should pass the patron object
holdfor => $query->cookie("holdfor"),
- holdfor_surname => $holdfor_patron->{'surname'},
- holdfor_firstname => $holdfor_patron->{'firstname'},
- holdfor_cardnumber => $holdfor_patron->{'cardnumber'},
+ holdfor_surname => $holdfor_patron->surname,
+ holdfor_firstname => $holdfor_patron->firstname,
+ holdfor_cardnumber => $holdfor_patron->cardnumber,
+ );
+}
+
+if($query->cookie("searchToOrder")){
+ my ( $basketno, $vendorid ) = split( /\//, $query->cookie("searchToOrder") );
+ $template->param(
+ searchtoorder_basketno => $basketno,
+ searchtoorder_vendorid => $vendorid
);
}
my $marcflavour = C4::Context->preference("marcflavour");
# XSLT processing of some stuff
-my $xslfile = C4::Context->preference('XSLTDetailsDisplay');
+my $xslfile = C4::Context->preference('XSLTDetailsDisplay') || "default";
my $lang = $xslfile ? C4::Languages::getlanguage() : undef;
my $sysxml = $xslfile ? C4::XSLT::get_xslt_sysprefs() : undef;
}
$template->param( 'SpineLabelShowPrintOnBibDetails' => C4::Context->preference("SpineLabelShowPrintOnBibDetails") );
-$template->param( ocoins => GetCOinSBiblio($record) );
+
+# Catch the exception as Koha::Biblio::Metadata->record can explode if the MARCXML is invalid
+# Do not propagate it as we already deal with it previously in this script
+my $coins = eval { $biblio->get_coins };
+$template->param( ocoins => $coins );
# some useful variables for enhanced content;
# in each case, we're grabbing the first value we find in
my $marcsubjctsarray = GetMarcSubjects( $record, $marcflavour );
my $marcseriesarray = GetMarcSeries($record,$marcflavour);
my $marcurlsarray = GetMarcUrls ($record,$marcflavour);
-my $marchostsarray = GetMarcHosts($record,$marcflavour);
-my $subtitle = GetRecordValue('subtitle', $record, $fw);
+my $marchostsarray = GetMarcHosts($record,$marcflavour);
+
+my $itemtypes = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search->unblessed } };
-# Get Branches, Itemtypes and Locations
-my $itemtypes = GetItemTypes();
my $dbh = C4::Context->dbh;
my @all_items = GetItemsInfo( $biblionumber );
my @items;
+my $patron = Koha::Patrons->find( $borrowernumber );
for my $itm (@all_items) {
- push @items, $itm unless ( $itm->{itemlost} && GetHideLostItemsPreference($borrowernumber) && !$showallitems);
+ push @items, $itm unless ( $itm->{itemlost} && $patron->category->hidelostitems && !$showallitems);
}
# flag indicating existence of at least one item linked via a host record
# adding items linked via host biblios
my @hostitems = GetHostItemsInfo($record);
if (@hostitems){
- $hostrecords =1;
- push (@items,@hostitems);
+ $hostrecords =1;
+ push (@items,@hostitems);
}
my $dat = &GetBiblioData($biblionumber);
foreach my $subscription (@subscriptions) {
my %cell;
- my $serials_to_display;
+ my $serials_to_display;
$cell{subscriptionid} = $subscription->{subscriptionid};
$cell{subscriptionnotes} = $subscription->{internalnotes};
$cell{missinglist} = $subscription->{missinglist};
$cell{branchcode} = $subscription->{branchcode};
$cell{hasalert} = $subscription->{hasalert};
$cell{callnumber} = $subscription->{callnumber};
+ $cell{location} = $subscription->{location};
$cell{closed} = $subscription->{closed};
#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;
+ $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}, $serials_to_display );
push @subs, \%cell;
# Get acquisition details
if ( C4::Context->preference('AcquisitionDetails') ) {
- my $orders = C4::Acquisition::GetHistory( biblionumber => $biblionumber, get_canceled_order => 1 );
+ my $orders = Koha::Acquisition::Orders->search(
+ { biblionumber => $biblionumber },
+ {
+ join => 'basketno',
+ order_by => 'basketno.booksellerid'
+ }
+ ); # GetHistory sorted by aqbooksellerid, but does it make sense?
+
$template->param(
orders => $orders,
);
}
+if ( C4::Context->preference('suggestion') ) {
+ my $suggestions = Koha::Suggestions->search(
+ {
+ biblionumber => $biblionumber,
+ archived => 0,
+ },
+ {
+ order_by => { -desc => 'suggesteddate' }
+ }
+ );
+ my $nb_archived_suggestions = Koha::Suggestions->search({ biblionumber => $biblionumber, archived => 1 })->count;
+ $template->param( suggestions => $suggestions, nb_archived_suggestions => $nb_archived_suggestions );
+}
+
if ( defined $dat->{'itemtype'} ) {
$dat->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }{imageurl} );
}
$dat->{'showncount'} = scalar @items + @hostitems;
$dat->{'hiddencount'} = scalar @all_items + @hostitems - scalar @items;
-my $shelflocations = GetKohaAuthorisedValues('items.location', $fw);
-my $collections = GetKohaAuthorisedValues('items.ccode' , $fw);
-my $copynumbers = GetKohaAuthorisedValues('items.copynumber', $fw);
+my $shelflocations =
+ { map { $_->{authorised_value} => $_->{lib} } Koha::AuthorisedValues->get_descriptions_by_koha_field( { frameworkcode => $fw, kohafield => 'items.location' } ) };
+my $collections =
+ { map { $_->{authorised_value} => $_->{lib} } Koha::AuthorisedValues->get_descriptions_by_koha_field( { frameworkcode => $fw, kohafield => 'items.ccode' } ) };
+my $copynumbers =
+ { map { $_->{authorised_value} => $_->{lib} } Koha::AuthorisedValues->get_descriptions_by_koha_field( { frameworkcode => $fw, kohafield => 'items.copynumber' } ) };
my (@itemloop, @otheritemloop, %itemfields);
my $norequests = 1;
-if ( my $lost_av = GetAuthValCode('items.itemlost', $fw) ) {
- $template->param( itemlostloop => GetAuthorisedValues( $lost_av ) );
+my $mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.itemlost', authorised_value => [ -and => {'!=' => undef }, {'!=' => ''}] });
+if ( $mss->count ) {
+ $template->param( itemlostloop => GetAuthorisedValues( $mss->next->authorised_value ) );
}
-if ( my $damaged_av = GetAuthValCode('items.damaged', $fw) ) {
- $template->param( itemdamagedloop => GetAuthorisedValues( $damaged_av ) );
+$mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.damaged', authorised_value => [ -and => {'!=' => undef }, {'!=' => ''}] });
+if ( $mss->count ) {
+ $template->param( itemdamagedloop => GetAuthorisedValues( $mss->next->authorised_value ) );
+}
+$mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.withdrawn', authorised_value => { not => undef } });
+if ( $mss->count ) {
+ $template->param( itemwithdrawnloop => GetAuthorisedValues( $mss->next->authorised_value) );
}
-my $materials_authvalcode = GetAuthValCode('items.materials', $fw);
+$mss = Koha::MarcSubfieldStructures->search({ frameworkcode => $fw, kohafield => 'items.materials', authorised_value => [ -and => {'!=' => undef }, {'!=' => ''}] });
my %materials_map;
-if ($materials_authvalcode) {
- my $materials_authvals = GetAuthorisedValues($materials_authvalcode);
+if ($mss->count) {
+ my $materials_authvals = GetAuthorisedValues($mss->next->authorised_value);
if ($materials_authvals) {
foreach my $value (@$materials_authvals) {
$materials_map{$value->{authorised_value}} = $value->{lib};
$item->{'ccode'} = $collections->{$ccode} if ( defined( $ccode ) && defined($collections) && exists( $collections->{$ccode} ) );
my $copynumber = $item->{'copynumber'};
$item->{'copynumber'} = $copynumbers->{$copynumber} if ( defined($copynumber) && defined($copynumbers) && exists( $copynumbers->{$copynumber} ) );
- foreach (qw(ccode enumchron copynumber stocknumber itemnotes itemnotes_nonpublic uri)) {
+ foreach (qw(ccode enumchron copynumber stocknumber itemnotes itemnotes_nonpublic uri publisheddate)) { # Warning when removing GetItemsInfo - publisheddate (at least) is not part of the items table
$itemfields{$_} = 1 if ( $item->{$_} );
}
# checking for holds
- my ($reservedate,$reservedfor,$expectedAt,undef,$wait) = GetReservesFromItemnumber($item->{itemnumber});
- my $ItemBorrowerReserveInfo = C4::Members::GetMember( borrowernumber => $reservedfor);
-
- if (C4::Context->preference('HidePatronName')){
- $item->{'hidepatronname'} = 1;
+ my $item_object = Koha::Items->find( $item->{itemnumber} );
+ my $holds = $item_object->current_holds;
+ if ( my $first_hold = $holds->next ) {
+ $item->{first_hold} = $first_hold;
}
- if ( defined $reservedate ) {
- $item->{backgroundcolor} = 'reserved';
- $item->{reservedate} = $reservedate;
- $item->{ReservedForBorrowernumber} = $reservedfor;
- $item->{ReservedForSurname} = $ItemBorrowerReserveInfo->{'surname'};
- $item->{ReservedForFirstname} = $ItemBorrowerReserveInfo->{'firstname'};
- $item->{ExpectedAtLibrary} = $expectedAt;
- $item->{Reservedcardnumber} = $ItemBorrowerReserveInfo->{'cardnumber'};
- # Check waiting status
- $item->{waitingdate} = $wait;
+ if ( my $checkout = $item_object->checkout ) {
+ $item->{CheckedOutFor} = $checkout->patron;
}
-
- # Check the transit status
+ # Check the transit status
my ( $transfertwhen, $transfertfrom, $transfertto ) = GetTransfers($item->{itemnumber});
if ( defined( $transfertwhen ) && ( $transfertwhen ne '' ) ) {
$item->{transfertwhen} = $transfertwhen;
if ($item->{biblionumber} ne $biblionumber){
$item->{hostbiblionumber} = $item->{biblionumber};
- $item->{hosttitle} = GetBiblioData($item->{biblionumber})->{title};
+ $item->{hosttitle} = GetBiblioData($item->{biblionumber})->{title};
}
- #count if item is used in analytical bibliorecords
- my $countanalytics= GetAnalyticsCount($item->{itemnumber});
- if ($countanalytics > 0){
- $analytics_flag=1;
- $item->{countanalytics} = $countanalytics;
+
+ if ( $analyze ) {
+ # count if item is used in analytical bibliorecords
+ # The 'countanalytics' flag is only used in the templates if analyze is set
+ my $countanalytics = C4::Context->preference('EasyAnalyticalRecords') ? GetAnalyticsCount($item->{itemnumber}) : 0;
+ if ($countanalytics > 0){
+ $analytics_flag=1;
+ $item->{countanalytics} = $countanalytics;
+ }
}
if (defined($item->{'materials'}) && $item->{'materials'} =~ /\S/){
}
}
- if ($currentbranch and $currentbranch ne "NO_LIBRARY_SET"
- and C4::Context->preference('SeparateHoldings')) {
+ if ($currentbranch and C4::Context->preference('SeparateHoldings')) {
if ($itembranchcode and $itembranchcode eq $currentbranch) {
push @itemloop, $item;
} else {
$template->param( norequests => $norequests );
$template->param(
- MARCNOTES => $marcnotesarray,
- MARCSUBJCTS => $marcsubjctsarray,
- MARCAUTHORS => $marcauthorsarray,
- MARCSERIES => $marcseriesarray,
- MARCURLS => $marcurlsarray,
+ MARCNOTES => $marcnotesarray,
+ MARCSUBJCTS => $marcsubjctsarray,
+ MARCAUTHORS => $marcauthorsarray,
+ MARCSERIES => $marcseriesarray,
+ MARCURLS => $marcurlsarray,
MARCISBNS => $marcisbnsarray,
- MARCHOSTS => $marchostsarray,
- subtitle => $subtitle,
- itemdata_ccode => $itemfields{ccode},
- itemdata_enumchron => $itemfields{enumchron},
- itemdata_uri => $itemfields{uri},
- itemdata_copynumber => $itemfields{copynumber},
- itemdata_stocknumber => $itemfields{stocknumber},
- volinfo => $itemfields{enumchron},
+ MARCHOSTS => $marchostsarray,
+ itemdata_ccode => $itemfields{ccode},
+ itemdata_enumchron => $itemfields{enumchron},
+ itemdata_uri => $itemfields{uri},
+ itemdata_copynumber => $itemfields{copynumber},
+ itemdata_stocknumber => $itemfields{stocknumber},
+ itemdata_publisheddate => $itemfields{publisheddate},
+ volinfo => $itemfields{enumchron},
itemdata_itemnotes => $itemfields{itemnotes},
itemdata_nonpublicnotes => $itemfields{itemnotes_nonpublic},
- z3950_search_params => C4::Search::z3950_search_args($dat),
+ z3950_search_params => C4::Search::z3950_search_args($dat),
hostrecords => $hostrecords,
- analytics_flag => $analytics_flag,
- C4::Search::enabled_staff_search_views,
+ analytics_flag => $analytics_flag,
+ C4::Search::enabled_staff_search_views,
materials => $materials_flag,
);
if (C4::Context->preference("FRBRizeEditions")==1) {
eval {
$template->param(
- XISBNS => get_xisbns($isbn)
+ XISBNS => scalar get_xisbns($isbn, $biblionumber)
);
};
if ($@) { warn "XISBN Failed $@"; }
}
#we only need to pass the number of holds to the template
-my $holds = C4::Reserves::GetReservesFromBiblionumber({ biblionumber => $biblionumber, all_dates => 1 });
-$template->param( holdcount => scalar ( @$holds ) );
+my $holds = $biblio->holds;
+$template->param( holdcount => $holds->count );
my $StaffDetailItemSelection = C4::Context->preference('StaffDetailItemSelection');
if ($StaffDetailItemSelection) {
my $basket = $myorder->{'basketno'};
if ((defined $myorder->{'datecancellationprinted'}) and ($myorder->{'datecancellationprinted'} ne '0000-00-00') ){
push @deletedorders_using_biblio, $myorder;
- unless (grep(/^$basket$/, @baskets_deletedorders)){
+ unless (grep{ $_ eq $basket } @baskets_deletedorders){
push @baskets_deletedorders,$myorder->{'basketno'};
}
}
else {
push @orders_using_biblio, $myorder;
- unless (grep(/^$basket$/, @baskets_orders)){
+ unless (grep{ $_ eq $basket } @baskets_orders){
push @baskets_orders,$myorder->{'basketno'};
}
}
}
+$template->param(biblio => $biblio);
+
my $count_orders_using_biblio = scalar @orders_using_biblio ;
$template->param (countorders => $count_orders_using_biblio);
projects / srvgit / blobdiff