);
use C4::Serials qw( CountSubscriptionFromBiblionumber SearchSubscriptions GetLatestSerials );
use C4::Output qw( output_html_with_http_headers );
-use C4::Biblio qw( GetBiblioData GetFrameworkCode GetMarcBiblio );
-use C4::Items qw( GetAnalyticsCount GetHostItemsInfo GetItemsInfo );
-use C4::Circulation qw( GetTransfers );
+use C4::Biblio qw( GetBiblioData GetFrameworkCode );
+use C4::Items qw( GetAnalyticsCount );
use C4::Reserves;
use C4::Serials qw( CountSubscriptionFromBiblionumber SearchSubscriptions GetLatestSerials );
use C4::XISBN qw( get_xisbns );
use C4::CourseReserves qw( GetItemCourseReservesInfo );
use Koha::AuthorisedValues;
use Koha::Biblios;
+use Koha::Biblio::ItemGroup::Items;
+use Koha::Biblio::ItemGroups;
use Koha::CoverImages;
+use Koha::DateUtils;
use Koha::Illrequests;
use Koha::Items;
use Koha::ItemTypes;
use Koha::Recalls;
use Koha::SearchEngine::Search;
use Koha::SearchEngine::QueryBuilder;
+use Koha::Serial::Items;
my $query = CGI->new();
my $biblionumber = $query->param('biblionumber');
$biblionumber = HTML::Entities::encode($biblionumber);
-my $record = GetMarcBiblio({ biblionumber => $biblionumber });
my $biblio = Koha::Biblios->find( $biblionumber );
$template->param( 'biblio', $biblio );
-if ( not defined $record ) {
+unless ( $biblio ) {
# biblionumber invalid -> report and exit
$template->param( unknownbiblionumber => 1,
biblionumber => $biblionumber );
}
my $marc_record = eval { $biblio->metadata->record };
-$template->param( decoding_error => $@ );
+my $invalid_marc_record = $@ || !$marc_record;
+if ($invalid_marc_record) {
+ $template->param( decoding_error => $@ );
+ my $marc_xml = C4::Charset::StripNonXmlChars( $biblio->metadata->metadata );
+
+ $marc_record = eval {
+ MARC::Record::new_from_xml( $marc_xml, 'UTF-8',
+ C4::Context->preference('marcflavour') );
+ };
+}
+
+my $op = $query->param('op') || q{};
+if ( $op eq 'set_item_group' ) {
+ my $item_group_id = $query->param('item_group_id');
+ my @itemnumbers = $query->multi_param('itemnumber');
+
+ foreach my $item_id (@itemnumbers) {
+ my $item_group_item = Koha::Biblio::ItemGroup::Items->find( { item_id => $item_id } );
+
+ if ($item_group_item) {
+ $item_group_item->item_group_id($item_group_id);
+ }
+ else {
+ $item_group_item = Koha::Biblio::ItemGroup::Item->new(
+ {
+ item_id => $item_id,
+ item_group_id => $item_group_id,
+ }
+ );
+ }
+
+ $item_group_item->store();
+ }
+}
+elsif ( $op eq 'unset_item_group' ) {
+ my $item_group_id = $query->param('item_group_id');
+ my @itemnumbers = $query->multi_param('itemnumber');
+
+ foreach my $item_id (@itemnumbers) {
+ my $item_group_item = Koha::Biblio::ItemGroup::Items->find( { item_id => $item_id } );
+ $item_group_item->delete() if $item_group_item;
+ }
+}
if($query->cookie("holdfor")){
my $holdfor_patron = Koha::Patrons->find( $query->cookie("holdfor") );
if ( $holdfor_patron ) {
$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 => $query->cookie("holdfor"),
+ holdfor_patron => $holdfor_patron,
);
}
}
$template->param( 'SpineLabelShowPrintOnBibDetails' => C4::Context->preference("SpineLabelShowPrintOnBibDetails") );
-# 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 );
+$template->param( ocoins => !$invalid_marc_record ? $biblio->get_coins : undef );
# 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);
+my $upc = GetNormalizedUPC($marc_record,$marcflavour);
+my $ean = GetNormalizedEAN($marc_record,$marcflavour);
+my $oclc = GetNormalizedOCLCNumber($marc_record,$marcflavour);
+my $isbn = GetNormalizedISBN(undef,$marc_record,$marcflavour);
my $content_identifier_exists;
if ( $isbn or $ean or $oclc or $upc ) {
$content_identifier_exists = 1;
content_identifier_exists => $content_identifier_exists,
);
-my $itemtypes = { map { $_->{itemtype} => $_ } @{ Koha::ItemTypes->search->unblessed } };
-
-my $dbh = C4::Context->dbh;
-
-my @all_items = GetItemsInfo( $biblionumber );
+my $itemtypes = { map { $_->itemtype => $_ } @{ Koha::ItemTypes->search_with_localization->as_list } };
+my $all_items = $biblio->items->search_ordered;
my @items;
my $patron = Koha::Patrons->find( $borrowernumber );
-for my $itm (@all_items) {
- push @items, $itm unless ( $itm->{itemlost} && $patron->category->hidelostitems && !$showallitems);
+while ( my $item = $all_items->next ) {
+ push @items, $item
+ unless $item->itemlost
+ && $patron->category->hidelostitems
+ && !$showallitems;
}
# flag indicating existence of at least one item linked via a host record
my $hostrecords;
# adding items linked via host biblios
-my @hostitems = GetHostItemsInfo($record);
-if (@hostitems){
- $hostrecords =1;
- push (@items,@hostitems);
+my $hostitems = $biblio->host_items;
+if ( $hostitems->count ) {
+ $hostrecords = 1;
+ push @items, $hostitems->as_list;
}
my $dat = &GetBiblioData($biblionumber);
+#is biblio a collection and are bundles enabled
+my $leader = $marc_record->leader();
+$dat->{bundlesEnabled} = ( ( substr( $leader, 7, 1 ) eq 'c' )
+ && C4::Context->preference('BundleNotLoanValue') ) ? 1 : 0;
+
#coping with subscriptions
my $subscriptionsnumber = CountSubscriptionFromBiblionumber($biblionumber);
my @subscriptions = SearchSubscriptions({ biblionumber => $biblionumber, orderby => 'title' });
my $showcomp = C4::Context->preference('ShowComponentRecords');
my $show_analytics;
if ( $showcomp eq 'both' || $showcomp eq 'staff' ) {
- if ( my $components = $marc_record ? $biblio->get_marc_components(C4::Context->preference('MaxComponentRecords')) : undef ) {
+ if ( my $components = !$invalid_marc_record ? $biblio->get_marc_components(C4::Context->preference('MaxComponentRecords')) : undef ) {
$show_analytics = 1 if @{$components}; # just show link when having results
$template->param( analytics_error => 1 ) if grep { $_->message eq 'component_search' } @{$biblio->object_messages};
my $parts;
);
}
$template->param( ComponentParts => $parts );
- $template->param( ComponentPartsQuery => $biblio->get_components_query );
+ my ( $comp_query, $comp_query_str, $comp_sort ) = $biblio->get_components_query;
+ my $cpq = $comp_query_str . "&sort_by=" . $comp_sort;
+ $template->param( ComponentPartsQuery => $cpq );
}
} else { # check if we should show analytics anyway
- $show_analytics = 1 if $marc_record && @{$biblio->get_marc_components(1)}; # count matters here, results does not
+ $show_analytics = 1 if !$invalid_marc_record && @{$biblio->get_marc_components(1)}; # count matters here, results does not
$template->param( analytics_error => 1 ) if grep { $_->message eq 'component_search' } @{$biblio->object_messages};
}
XSLTDetailsDisplay => '1',
XSLTBloc => XSLTParse4Display({
biblionumber => $biblionumber,
- record => $record,
+ record => $marc_record,
xsl_syspref => "XSLTDetailsDisplay",
fix_amps => 1,
xslt_variables => $xslt_variables,
}
if ( defined $dat->{'itemtype'} ) {
- $dat->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }{imageurl} );
+ $dat->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtypes->{ $dat->{itemtype} }->imageurl );
}
-$dat->{'count'} = scalar @all_items + @hostitems;
-$dat->{'showncount'} = scalar @items + @hostitems;
-$dat->{'hiddencount'} = scalar @all_items + @hostitems - scalar @items;
+$dat->{'count'} = $all_items->count + $hostitems->count;
+$dat->{'showncount'} = scalar @items + $hostitems->count;
+$dat->{'hiddencount'} = $all_items->count + $hostitems->count - scalar @items;
my $shelflocations =
{ map { $_->{authorised_value} => $_->{lib} } Koha::AuthorisedValues->get_descriptions_by_koha_field( { frameworkcode => $fw, kohafield => 'items.location' } ) };
}
my $separatebranch = C4::Context->preference('SeparateHoldingsBranch') || 'homebranch';
my ( $itemloop_has_images, $otheritemloop_has_images );
-foreach my $item (@items) {
- my $itembranchcode = $item->{$separatebranch};
- $item->{imageurl} = defined $item->{itype} ? getitemtypeimagelocation('intranet', $itemtypes->{ $item->{itype} }{imageurl})
- : '';
+foreach my $item (@items) {
+ my $itembranchcode = $item->$separatebranch;
- $item->{datedue} = format_sqldatetime($item->{datedue});
+ my $item_info = $item->unblessed;
+ $item_info->{itemtype} = $itemtypes->{$item->effective_itemtype};
#get shelf location and collection code description if they are authorised value.
# same thing for copy number
- 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} ) );
- 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 publisheddate)) { # Warning when removing GetItemsInfo - publisheddate (at least) is not part of the items table
- $itemfields{$_} = 1 if ( $item->{$_} );
+ my $shelfcode = $item->location;
+ $item_info->{'location'} = $shelflocations->{$shelfcode} if ( defined( $shelfcode ) && defined($shelflocations) && exists( $shelflocations->{$shelfcode} ) );
+ my $ccode = $item->ccode;
+ $item_info->{'ccode'} = $collections->{$ccode} if ( defined( $ccode ) && defined($collections) && exists( $collections->{$ccode} ) );
+ my $copynumber = $item->copynumber;
+ $item_info->{'copynumber'} = $copynumbers->{$copynumber} if ( defined($copynumber) && defined($copynumbers) && exists( $copynumbers->{$copynumber} ) );
+ foreach (qw(ccode enumchron copynumber stocknumber itemnotes itemnotes_nonpublic uri )) {
+ $itemfields{$_} = 1 if $item->$_;
+ }
+
+ # FIXME The following must be Koha::Item->serial
+ my $serial_item = Koha::Serial::Items->find($item->itemnumber);
+ if ( $serial_item ) {
+ $item_info->{serial} = $serial_item;
+ $itemfields{publisheddate} = 1;
}
+ $item_info->{object} = $item;
+
# checking for holds
- my $item_object = Koha::Items->find( $item->{itemnumber} );
- my $holds = $item_object->current_holds;
+ my $holds = $item->current_holds;
if ( my $first_hold = $holds->next ) {
- $item->{first_hold} = $first_hold;
+ $item_info->{first_hold} = $first_hold;
}
- if ( my $checkout = $item_object->checkout ) {
- $item->{CheckedOutFor} = $checkout->patron;
- }
+ $item_info->{checkout} = $item->checkout;
# Check the transit status
- my ( $transfertwhen, $transfertfrom, $transfertto ) = GetTransfers($item->{itemnumber});
- if ( defined( $transfertwhen ) && ( $transfertwhen ne '' ) ) {
- $item->{transfertwhen} = $transfertwhen;
- $item->{transfertfrom} = $transfertfrom;
- $item->{transfertto} = $transfertto;
- $item->{nocancel} = 1;
+ my $transfer = $item->get_transfer;
+ if ( $transfer ) {
+ $item_info->{transfer} = $transfer;
}
foreach my $f (qw( itemnotes )) {
- if ($item->{$f}) {
- $item->{$f} =~ s|\n|<br />|g;
+ if ($item_info->{$f}) {
+ $item_info->{$f} =~ s|\n|<br />|g;
$itemfields{$f} = 1;
}
}
#item has a host number if its biblio number does not match the current bib
- if ($item->{biblionumber} ne $biblionumber){
- $item->{hostbiblionumber} = $item->{biblionumber};
- $item->{hosttitle} = GetBiblioData($item->{biblionumber})->{title};
+ if ($item->biblionumber ne $biblionumber){
+ $item_info->{hostbiblionumber} = $item->biblionumber;
+ $item_info->{hosttitle} = $item->biblio->title;
}
-
+
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;
+ my $countanalytics = GetAnalyticsCount( $item->itemnumber );
if ($countanalytics > 0){
$analytics_flag=1;
- $item->{countanalytics} = $countanalytics;
+ $item_info->{countanalytics} = $countanalytics;
}
}
- if (defined($item->{'materials'}) && $item->{'materials'} =~ /\S/){
+ if (defined($item->materials) && $item->materials =~ /\S/){
$materials_flag = 1;
- if (defined $materials_map{ $item->{materials} }) {
- $item->{materials} = $materials_map{ $item->{materials} };
+ if (defined $materials_map{ $item->materials }) {
+ $item_info->{materials} = $materials_map{ $item->materials };
}
}
if ( C4::Context->preference('UseCourseReserves') ) {
- $item->{'course_reserves'} = GetItemCourseReservesInfo( itemnumber => $item->{'itemnumber'} );
+ $item_info->{'course_reserves'} = GetItemCourseReservesInfo( itemnumber => $item->itemnumber );
+ }
+
+ $item_info->{can_be_edited} = $patron->can_edit_items_from( $item->homebranch );
+
+ if ( C4::Context->preference("LocalCoverImages") == 1 ) {
+ $item_info->{cover_images} = $item->cover_images;
+ }
+
+ if ( C4::Context->preference('UseRecalls') ) {
+ $item_info->{recall} = $item->recall;
}
if ( C4::Context->preference('IndependentBranches') ) {
my $userenv = C4::Context->userenv();
if ( not C4::Context->IsSuperLibrarian()
- and $userenv->{branch} ne $item->{homebranch} ) {
- $item->{cannot_be_edited} = 1;
+ and $userenv->{branch} ne $item->homebranch ) {
+ $item_info->{cannot_be_edited} = 1;
+ $item_info->{not_same_branch} = 1;
}
}
- if ( C4::Context->preference("LocalCoverImages") == 1 ) {
- $item->{cover_images} = $item_object->cover_images;
+ if ( $item->is_bundle ) {
+ $item_info->{bundled} =
+ $item->bundle_items->search( { itemlost => { '!=' => 0 } } )
+ ->count;
+ $item_info->{bundled_lost} =
+ $item->bundle_items->search( { itemlost => 0 } )->count;
+ $item_info->{is_bundle} = 1;
}
- if ( C4::Context->preference('UseRecalls') ) {
- my $recall = Koha::Recalls->find({ item_id => $item->{itemnumber}, completed => 0 });
- if ( defined $recall ) {
- $item->{recalled} = 1;
- $item->{recall} = $recall;
- }
+ if ($item->in_bundle) {
+ $item_info->{bundle_host} = $item->bundle_host;
}
if ($currentbranch and C4::Context->preference('SeparateHoldings')) {
if ($itembranchcode and $itembranchcode eq $currentbranch) {
- push @itemloop, $item;
- $itemloop_has_images++ if $item_object->cover_images->count;
+ push @itemloop, $item_info;
+ $itemloop_has_images++ if $item->cover_images->count;
} else {
- push @otheritemloop, $item;
- $otheritemloop_has_images++ if $item_object->cover_images->count;
+ push @otheritemloop, $item_info;
+ $otheritemloop_has_images++ if $item->cover_images->count;
}
} else {
- push @itemloop, $item;
- $itemloop_has_images++ if $item_object->cover_images->count;
+ push @itemloop, $item_info;
+ $itemloop_has_images++ if $item->cover_images->count;
}
}
);
$template->param(
- MARCNOTES => $marc_record ? $biblio->get_marc_notes({ marcflavour => $marcflavour }) : undef,
- 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),
- hostrecords => $hostrecords,
- analytics_flag => $analytics_flag,
+ MARCNOTES => !$invalid_marc_record ? $biblio->get_marc_notes() : undef,
+ 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),
+ hostrecords => $hostrecords,
+ analytics_flag => $analytics_flag,
C4::Search::enabled_staff_search_views,
- materials => $materials_flag,
+ materials => $materials_flag,
);
if (C4::Context->preference("AlternateHoldingsField") && scalar @items == 0) {
my $subfields = substr $fieldspec, 3;
my $holdingsep = C4::Context->preference("AlternateHoldingsSeparator") || ' ';
my @alternateholdingsinfo = ();
- my @holdingsfields = $record->field(substr $fieldspec, 0, 3);
+ my @holdingsfields = $marc_record->field(substr $fieldspec, 0, 3);
for my $field (@holdingsfields) {
my %holding = ( holding => '' );
if ( C4::Context->preference("LocalCoverImages") == 1 ) {
my $images = $biblio->cover_images;
- $template->param( localimages => $biblio->cover_images );
+ $template->param(
+ localimages => $biblio->cover_images->search(
+ {}, { order_by => [ \"COALESCE(itemnumber, 0, 1)", 'timestamp' ] }
+ ),
+ );
}
# HTML5 Media
if ( (C4::Context->preference("HTML5MediaEnabled") eq 'both') or (C4::Context->preference("HTML5MediaEnabled") eq 'staff') ) {
- $template->param( C4::HTML5Media->gethtml5media($record));
+ $template->param( C4::HTML5Media->gethtml5media($marc_record));
}
# Displaying tags
$template->param( course_reserves => $course_reserves );
}
-$template->param(found1 => $query->param('found1') );
+$template->param(found1 => scalar $query->param('found1') );
$template->param(biblio => $biblio);
projects / srvgit / blobdiff