@EXPORT_OK = qw(
AddBiblio
GetBiblioData
- GetMarcBiblio
GetISBDView
GetMarcControlnumber
GetMarcISBN
CountItemsIssued
ModBiblio
ModZebra
- EmbedItemsInMarcBiblio
UpdateTotalIssues
RemoveAllNsb
DelBiblio
TransformHtmlToMarc
TransformHtmlToXml
prepare_host_field
- TransformMarcToKohaOneField
);
# Internal functions
nsb_clean
SetMarcUnicodeFlag
SetUTF8Flag
- StripNonXmlChars
);
+use C4::Languages;
use C4::Linker;
use C4::OAI::Sets;
-use C4::Items qw( GetHiddenItemnumbers GetMarcItem );
+use C4::Items qw( GetMarcItem );
use Koha::Logger;
use Koha::Caches;
+use Koha::ClassSources;
use Koha::Authority::Types;
use Koha::Acquisition::Currencies;
+use Koha::BackgroundJob::BatchUpdateBiblioHoldsQueue;
use Koha::Biblio::Metadatas;
use Koha::Holds;
use Koha::ItemTypes;
bib to add, while the second argument is the desired MARC
framework code.
-This function also accepts a third, optional argument: a hashref
-to additional options. The only defined option is C<defer_marc_save>,
-which if present and mapped to a true value, causes C<AddBiblio>
-to omit the call to save the MARC in C<biblio_metadata.metadata>
-This option is provided B<only>
-for the use of scripts such as C<bulkmarcimport.pl> that may need
-to do some manipulation of the MARC record for item parsing before
-saving it and which cannot afford the performance hit of saving
-the MARC record twice. Consequently, do not use that option
-unless you can guarantee that C<ModBiblioMarc> will be called.
+The C<$options> argument is a hashref with additional parameters:
+
+=over 4
+
+=item B<defer_marc_save>: used when ModBiblioMarc is handled by the caller
+
+=item B<skip_record_index>: used when the indexing schedulling will be handled by the caller
+
+=back
=cut
sub AddBiblio {
- my $record = shift;
- my $frameworkcode = shift;
- my $options = @_ ? shift : undef;
- my $defer_marc_save = 0;
+ my ( $record, $frameworkcode, $options ) = @_;
+
+ $options //= {};
+ my $skip_record_index = $options->{skip_record_index} || 0;
+ my $defer_marc_save = $options->{defer_marc_save} || 0;
+
if (!$record) {
carp('AddBiblio called with undefined record');
return;
}
- if ( defined $options and exists $options->{'defer_marc_save'} and $options->{'defer_marc_save'} ) {
- $defer_marc_save = 1;
- }
my $schema = Koha::Database->schema;
my ( $biblionumber, $biblioitemnumber );
# transform the data into koha-table style data
SetUTF8Flag($record);
- my $olddata = TransformMarcToKoha( $record, $frameworkcode );
+ my $olddata = TransformMarcToKoha({ record => $record, limit_table => 'no_items' });
my $biblio = Koha::Biblio->new(
{
# update MARC subfield that stores biblioitems.cn_sort
_koha_marc_update_biblioitem_cn_sort( $record, $olddata, $frameworkcode );
- if (C4::Context->preference('BiblioAddsAuthorities')) {
+ if (C4::Context->preference('AutoLinkBiblios')) {
BiblioAutoLink( $record, $frameworkcode );
}
# now add the record
- ModBiblioMarc( $record, $biblionumber ) unless $defer_marc_save;
+ ModBiblioMarc( $record, $biblionumber, { skip_record_index => $skip_record_index } ) unless $defer_marc_save;
# update OAI-PMH sets
if(C4::Context->preference("OAI-PMH:AutoUpdateSets")) {
to authorities based on settings in the system preferences. This flag allows
us to not relink records when the authority linker is saving modifications.
+=item C<skip_holds_queue>
+
+Unless C<skip_holds_queue> is passed, ModBiblio will trigger the BatchUpdateBiblioHoldsQueue
+task to rebuild the holds queue for the biblio if I<RealTimeHoldsQueue> is enabled.
+
=back
Returns 1 on success 0 on failure
sub ModBiblio {
my ( $record, $biblionumber, $frameworkcode, $options ) = @_;
+
$options //= {};
+ my $skip_record_index = $options->{skip_record_index} || 0;
if (!$record) {
carp 'No record passed to ModBiblio';
}
if ( C4::Context->preference("CataloguingLog") ) {
- my $newrecord = GetMarcBiblio({ biblionumber => $biblionumber });
- logaction( "CATALOGUING", "MODIFY", $biblionumber, "biblio BEFORE=>" . $newrecord->as_formatted );
+ my $biblio = Koha::Biblios->find($biblionumber);
+ logaction( "CATALOGUING", "MODIFY", $biblionumber, "biblio BEFORE=>" . $biblio->metadata->record->as_formatted );
}
- if ( !$options->{disable_autolink} && C4::Context->preference('BiblioAddsAuthorities') ) {
+ if ( !$options->{disable_autolink} && C4::Context->preference('AutoLinkBiblios') ) {
BiblioAutoLink( $record, $frameworkcode );
}
_koha_marc_update_bib_ids( $record, $frameworkcode, $biblionumber, $biblioitemnumber );
# load the koha-table data object
- my $oldbiblio = TransformMarcToKoha( $record, $frameworkcode );
+ my $oldbiblio = TransformMarcToKoha({ record => $record });
# update MARC subfield that stores biblioitems.cn_sort
_koha_marc_update_biblioitem_cn_sort( $record, $oldbiblio, $frameworkcode );
# update the MARC record (that now contains biblio and items) with the new record data
- ModBiblioMarc( $record, $biblionumber );
+ ModBiblioMarc( $record, $biblionumber, { skip_record_index => $skip_record_index } );
# modify the other koha tables
_koha_modify_biblio( $dbh, $oldbiblio, $frameworkcode );
C4::OAI::Sets::UpdateOAISetsBiblio($biblionumber, $record);
}
+ Koha::BackgroundJob::BatchUpdateBiblioHoldsQueue->new->enqueue(
+ {
+ biblio_ids => [ $biblionumber ]
+ }
+ ) unless $options->{skip_holds_queue} or !C4::Context->preference('RealTimeHoldsQueue');
+
return 1;
}
=head2 DelBiblio
- my $error = &DelBiblio($biblionumber);
+ my $error = &DelBiblio($biblionumber, $params);
Exported function (core API) for deleting a biblio in koha.
Deletes biblio record from Zebra and Koha tables (biblio & biblioitems)
return:
C<$error> : undef unless an error occurs
+I<$params> is a hashref containing extra parameters. Valid keys are:
+
+=over 4
+
+=item B<skip_holds_queue>: used when the holds queue update will be handled by the caller
+
+=item B<skip_record_index>: used when the indexing schedulling will be handled by the caller
+
+=back
=cut
sub DelBiblio {
# We delete any existing holds
my $holds = $biblio->holds;
while ( my $hold = $holds->next ) {
- $hold->cancel;
+ # no need to update the holds queue on each step, we'll do it at the end
+ $hold->cancel({ skip_holds_queue => 1 });
}
unless ( $params->{skip_record_index} ){
logaction( "CATALOGUING", "DELETE", $biblionumber, "biblio" ) if C4::Context->preference("CataloguingLog");
+ Koha::BackgroundJob::BatchUpdateBiblioHoldsQueue->new->enqueue(
+ {
+ biblio_ids => [ $biblionumber ]
+ }
+ ) unless $params->{skip_holds_queue} or !C4::Context->preference('RealTimeHoldsQueue');
+
return;
}
$marcrecordauth->insert_fields_ordered(
MARC::Field->new(
'667', '', '',
- 'a' => "Machine generated authority record."
+ 'a' => C4::Context->preference('GenerateAuthorityField667')
)
);
my $cite =
$cite =~ s/^[\s\,]*//;
$cite =~ s/[\s\,]*$//;
$cite =
- "Work cat.: ("
+ C4::Context->preference('GenerateAuthorityField670') . ": ("
. ( $library ? $library->get_effective_marcorgcode : C4::Context->preference('MARCOrgCode') ) . ")"
. $bib->subfield( '999', 'c' ) . ": "
. $cite;
return wantarray ? @{$mss->{$kohafield}} : $mss->{$kohafield}->[0];
}
-=head2 GetMarcBiblio
-
- my $record = GetMarcBiblio({
- biblionumber => $biblionumber,
- embed_items => $embeditems,
- opac => $opac,
- borcat => $patron_category });
-
-Returns MARC::Record representing a biblio record, or C<undef> if the
-biblionumber doesn't exist.
-
-Both embed_items and opac are optional.
-If embed_items is passed and is 1, items are embedded.
-If opac is passed and is 1, the record is filtered as needed.
-
-=over 4
-
-=item C<$biblionumber>
-
-the biblionumber
-
-=item C<$embeditems>
-
-set to true to include item information.
-
-=item C<$opac>
-
-set to true to make the result suited for OPAC view. This causes things like
-OpacHiddenItems to be applied.
-
-=item C<$borcat>
-
-If the OpacHiddenItemsExceptions system preference is set, this patron category
-can be used to make visible OPAC items which would be normally hidden.
-It only makes sense in combination both embed_items and opac values true.
-
-=back
-
-=cut
-
-sub GetMarcBiblio {
- my ($params) = @_;
-
- if (not defined $params) {
- carp 'GetMarcBiblio called without parameters';
- return;
- }
-
- my $biblionumber = $params->{biblionumber};
- my $embeditems = $params->{embed_items} || 0;
- my $opac = $params->{opac} || 0;
- my $borcat = $params->{borcat} // q{};
-
- if (not defined $biblionumber) {
- carp 'GetMarcBiblio called with undefined biblionumber';
- return;
- }
-
- my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare("SELECT biblioitemnumber FROM biblioitems WHERE biblionumber=? ");
- $sth->execute($biblionumber);
- my $row = $sth->fetchrow_hashref;
- my $biblioitemnumber = $row->{'biblioitemnumber'};
- my $marcxml = GetXmlBiblio( $biblionumber );
- $marcxml = StripNonXmlChars( $marcxml );
- my $frameworkcode = GetFrameworkCode($biblionumber);
- MARC::File::XML->default_record_format( C4::Context->preference('marcflavour') );
- my $record = MARC::Record->new();
-
- if ($marcxml) {
- $record = eval {
- MARC::Record::new_from_xml( $marcxml, "UTF-8",
- C4::Context->preference('marcflavour') );
- };
- if ($@) { warn " problem with :$biblionumber : $@ \n$marcxml"; }
- return unless $record;
-
- C4::Biblio::_koha_marc_update_bib_ids( $record, $frameworkcode, $biblionumber,
- $biblioitemnumber );
- C4::Biblio::EmbedItemsInMarcBiblio({
- marc_record => $record,
- biblionumber => $biblionumber,
- opac => $opac,
- borcat => $borcat })
- if ($embeditems);
-
- return $record;
- }
- else {
- return;
- }
-}
-
=head2 GetXmlBiblio
my $marcxml = GetXmlBiblio($biblionumber);
sub GetAuthorisedValueDesc {
my ( $tag, $subfield, $value, $framework, $tagslib, $category, $opac ) = @_;
+ return q{} unless defined($value);
+
+ my $cache = Koha::Caches->get_instance();
+ my $cache_key;
if ( !$category ) {
return $value unless defined $tagslib->{$tag}->{$subfield}->{'authorised_value'};
#---- branch
if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "branches" ) {
- my $branch = Koha::Libraries->find($value);
- return $branch? $branch->branchname: q{};
+ $cache_key = "LibraryNames";
+ my $libraries = $cache->get_from_cache( $cache_key, { unsafe => 1 } );
+ if ( !$libraries ) {
+ $libraries = {
+ map { $_->branchcode => $_->branchname }
+ Koha::Libraries->search( {},
+ { columns => [ 'branchcode', 'branchname' ] } )
+ ->as_list
+ };
+ $cache->set_in_cache($cache_key, $libraries);
+ }
+ return $libraries->{$value};
}
#---- itemtypes
if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "itemtypes" ) {
- my $itemtype = Koha::ItemTypes->find( $value );
- return $itemtype ? $itemtype->translated_description : q||;
+ my $lang = C4::Languages::getlanguage;
+ $lang //= 'en';
+ $cache_key = $lang . 'ItemTypeDescriptions';
+ my $itypes = $cache->get_from_cache( $cache_key, { unsafe => 1 } );
+ if ( !$itypes ) {
+ $itypes =
+ { map { $_->itemtype => $_->translated_description }
+ Koha::ItemTypes->search()->as_list };
+ $cache->set_in_cache( $cache_key, $itypes );
+ }
+ return $itypes->{$value};
}
if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "cn_source" ) {
- my $source = GetClassSource($value);
- return $source ? $source->{description} : q||;
+ $cache_key = "cn_sources:description";
+ my $cn_sources = $cache->get_from_cache( $cache_key, { unsafe => 1 } );
+ if ( !$cn_sources ) {
+ $cn_sources = {
+ map { $_->cn_source => $_->description }
+ Koha::ClassSources->search( {},
+ { columns => [ 'cn_source', 'description' ] } )
+ ->as_list
+ };
+ $cache->set_in_cache($cache_key, $cn_sources);
+ }
+ return $cn_sources->{$value};
}
#---- "true" authorized value
my $dbh = C4::Context->dbh;
if ( $category ne "" ) {
- my $sth = $dbh->prepare( "SELECT lib, lib_opac FROM authorised_values WHERE category = ? AND authorised_value = ?" );
- $sth->execute( $category, $value );
- my $data = $sth->fetchrow_hashref;
- return ( $opac && $data->{'lib_opac'} ) ? $data->{'lib_opac'} : $data->{'lib'};
+ $cache_key = "AVDescriptions-" . $category;
+ my $av_descriptions = $cache->get_from_cache( $cache_key, { unsafe => 1 } );
+ if ( !$av_descriptions ) {
+ $av_descriptions = {
+ map {
+ $_->authorised_value =>
+ { lib => $_->lib, lib_opac => $_->lib_opac }
+ } Koha::AuthorisedValues->search(
+ { category => $category },
+ {
+ columns => [ 'authorised_value', 'lib_opac', 'lib' ]
+ }
+ )->as_list
+ };
+ $cache->set_in_cache($cache_key, $av_descriptions);
+ }
+ return ( $opac && $av_descriptions->{$value}->{'lib_opac'} )
+ ? $av_descriptions->{$value}->{'lib_opac'}
+ : $av_descriptions->{$value}->{'lib'};
} else {
return $value; # if nothing is found return the original value
}
push @link_loop, {
limit => $subject_limit,
'link' => $linkvalue,
- operator => (scalar @link_loop) ? ' and ' : undef
+ operator => (scalar @link_loop) ? ' AND ' : undef
};
}
my @this_link_loop = @link_loop;
push @link_loop, {
'link' => $linkvalue,
- operator => (scalar @link_loop) ? ' and ' : undef
+ operator => (scalar @link_loop) ? ' AND ' : undef
};
if ($volume_number) {
sub PrepHostMarcField {
my ($hostbiblionumber,$hostitemnumber, $marcflavour) = @_;
$marcflavour ||="MARC21";
-
- my $hostrecord = GetMarcBiblio({ biblionumber => $hostbiblionumber });
+
+ my $biblio = Koha::Biblios->find($hostbiblionumber);
+ my $hostrecord = $biblio->metadata->record;
my $item = Koha::Items->find($hostitemnumber);
my $hostmarcfield;
=head2 TransformMarcToKoha
- $result = TransformMarcToKoha( $record, undef, $limit )
+ $result = TransformMarcToKoha({ record => $record, limit_table => $limit })
Extract data from a MARC bib record into a hashref representing
Koha biblio, biblioitems, and items fields.
=cut
sub TransformMarcToKoha {
- my ( $record, $frameworkcode, $limit_table ) = @_;
- # FIXME Parameter $frameworkcode is obsolete and will be removed
- $limit_table //= q{};
+ my ( $params ) = @_;
+
+ my $record = $params->{record};
+ my $limit_table = $params->{limit_table} // q{};
+ my $kohafields = $params->{kohafields};
my $result = {};
if (!defined $record) {
# The next call acknowledges Default as the authoritative framework
# for Koha to MARC mappings.
my $mss = GetMarcSubfieldStructure( '', { unsafe => 1 } ); # Do not change framework
- foreach my $kohafield ( keys %{ $mss } ) {
+ @{$kohafields} = keys %{ $mss } unless $kohafields;
+ foreach my $kohafield ( @{$kohafields} ) {
my ( $table, $column ) = split /[.]/, $kohafield, 2;
next unless $tables{$table};
- my $val = TransformMarcToKohaOneField( $kohafield, $record );
- next if !defined $val;
+ my ( $value, @values );
+ foreach my $fldhash ( @{$mss->{$kohafield}} ) {
+ my $tag = $fldhash->{tagfield};
+ my $sub = $fldhash->{tagsubfield};
+ foreach my $fld ( $record->field($tag) ) {
+ if( $sub eq '@' || $fld->is_control_field ) {
+ push @values, $fld->data if $fld->data;
+ } else {
+ push @values, grep { $_ } $fld->subfield($sub);
+ }
+ }
+ }
+ if ( @values ){
+ $value = join ' | ', uniq(@values);
+
+ # Additional polishing for individual kohafields
+ if( $kohafield =~ /copyrightdate|publicationyear/ ) {
+ $value = _adjust_pubyear( $value );
+ }
+ }
+
+ next if !defined $value;
my $key = _disambiguate( $table, $column );
- $result->{$key} = $val;
+ $result->{$key} = $value;
}
return $result;
}
}
-=head2 TransformMarcToKohaOneField
-
- $val = TransformMarcToKohaOneField( 'biblio.title', $marc );
-
- Note: The authoritative Default framework is used implicitly.
-
-=cut
-
-sub TransformMarcToKohaOneField {
- my ( $kohafield, $marc ) = @_;
-
- my ( @rv, $retval );
- my @mss = GetMarcSubfieldStructureFromKohaField($kohafield);
- foreach my $fldhash ( @mss ) {
- my $tag = $fldhash->{tagfield};
- my $sub = $fldhash->{tagsubfield};
- foreach my $fld ( $marc->field($tag) ) {
- if( $sub eq '@' || $fld->is_control_field ) {
- push @rv, $fld->data if $fld->data;
- } else {
- push @rv, grep { $_ } $fld->subfield($sub);
- }
- }
- }
- return unless @rv;
- $retval = join ' | ', uniq(@rv);
-
- # Additional polishing for individual kohafields
- if( $kohafield =~ /copyrightdate|publicationyear/ ) {
- $retval = _adjust_pubyear( $retval );
- }
-
- return $retval;
-}
-
=head2 _adjust_pubyear
- Helper routine for TransformMarcToKohaOneField
+ Helper routine for TransformMarcToKoha
=cut
}
}
-=head2 EmbedItemsInMarcBiblio
-
- EmbedItemsInMarcBiblio({
- marc_record => $marc,
- biblionumber => $biblionumber,
- item_numbers => $itemnumbers,
- opac => $opac });
-
-Given a MARC::Record object containing a bib record,
-modify it to include the items attached to it as 9XX
-per the bib's MARC framework.
-if $itemnumbers is defined, only specified itemnumbers are embedded.
-
-If $opac is true, then opac-relevant suppressions are included.
-
-If opac filtering will be done, borcat should be passed to properly
-override if necessary.
-
-=cut
-
-sub EmbedItemsInMarcBiblio {
- my ($params) = @_;
- my ($marc, $biblionumber, $itemnumbers, $opac, $borcat);
- $marc = $params->{marc_record};
- if ( !$marc ) {
- carp 'EmbedItemsInMarcBiblio: No MARC record passed';
- return;
- }
- $biblionumber = $params->{biblionumber};
- $itemnumbers = $params->{item_numbers};
- $opac = $params->{opac};
- $borcat = $params->{borcat} // q{};
-
- $itemnumbers = [] unless defined $itemnumbers;
-
- my $frameworkcode = GetFrameworkCode($biblionumber);
- _strip_item_fields($marc, $frameworkcode);
-
- # ... and embed the current items
- my $dbh = C4::Context->dbh;
- my $sth = $dbh->prepare("SELECT itemnumber FROM items WHERE biblionumber = ?");
- $sth->execute($biblionumber);
- my ( $itemtag, $itemsubfield ) = GetMarcFromKohaField( "items.itemnumber" );
-
- my @item_fields; # Array holding the actual MARC data for items to be included.
- my @items; # Array holding items which are both in the list (sitenumbers)
- # and on this biblionumber
-
- # Flag indicating if there is potential hiding.
- my $opachiddenitems = $opac
- && ( C4::Context->preference('OpacHiddenItems') !~ /^\s*$/ );
-
- while ( my ($itemnumber) = $sth->fetchrow_array ) {
- next if @$itemnumbers and not grep { $_ == $itemnumber } @$itemnumbers;
- my $item;
- if ( $opachiddenitems ) {
- $item = Koha::Items->find($itemnumber);
- $item = $item ? $item->unblessed : undef;
- }
- push @items, { itemnumber => $itemnumber, item => $item };
- }
- my @items2pass = map { $_->{item} } @items;
- my @hiddenitems =
- $opachiddenitems
- ? C4::Items::GetHiddenItemnumbers({
- items => \@items2pass,
- borcat => $borcat })
- : ();
- # Convert to a hash for quick searching
- my %hiddenitems = map { $_ => 1 } @hiddenitems;
- foreach my $itemnumber ( map { $_->{itemnumber} } @items ) {
- next if $hiddenitems{$itemnumber};
- my $item_marc = C4::Items::GetMarcItem( $biblionumber, $itemnumber );
- push @item_fields, $item_marc->field($itemtag);
- }
- $marc->append_fields(@item_fields);
-}
-
=head1 INTERNAL FUNCTIONS
=head2 _koha_marc_update_bib_ids
} else {
C4::Biblio::UpsertMarcSubfield($record, $biblioitem_tag, $biblioitem_subfield, $biblioitemnumber);
}
+
+ # update the control number (001) in MARC
+ if(C4::Context->preference('autoControlNumber') eq 'biblionumber'){
+ unless($record->field('001')){
+ $record->insert_fields_ordered(MARC::Field->new('001', $biblionumber));
+ }
+ }
}
=head2 _koha_marc_update_biblioitem_cn_sort
sub ModBiblioMarc {
# pass the MARC::Record to this function, and it will create the records in
# the marcxml field
- my ( $record, $biblionumber ) = @_;
+ my ( $record, $biblionumber, $params ) = @_;
if ( !$record ) {
carp 'ModBiblioMarc passed an undefined record';
return;
}
+ my $skip_record_index = $params->{skip_record_index} || 0;
+
# Clone record as it gets modified
$record = $record->clone();
my $dbh = C4::Context->dbh;
$m_rs->metadata( $record->as_xml_record($encoding) );
$m_rs->store;
- my $indexer = Koha::SearchEngine::Indexer->new({ index => $Koha::SearchEngine::BIBLIOS_INDEX });
- $indexer->index_records( $biblionumber, "specialUpdate", "biblioserver" );
+ unless ( $skip_record_index ) {
+ my $indexer = Koha::SearchEngine::Indexer->new({ index => $Koha::SearchEngine::BIBLIOS_INDEX });
+ $indexer->index_records( $biblionumber, "specialUpdate", "biblioserver" );
+ }
return $biblionumber;
}
sub prepare_host_field {
my ( $hostbiblio, $marcflavour ) = @_;
$marcflavour ||= C4::Context->preference('marcflavour');
- my $host = GetMarcBiblio({ biblionumber => $hostbiblio });
+
+ my $biblio = Koha::Biblios->find($hostbiblio);
+ my $host = $biblio->metadata->record;
# unfortunately as_string does not 'do the right thing'
# if field returns undef
my %sfd;
=cut
sub UpdateTotalIssues {
- my ($biblionumber, $increase, $value) = @_;
+ my ($biblionumber, $increase, $value, $skip_holds_queue) = @_;
my $totalissues;
- my $record = GetMarcBiblio({ biblionumber => $biblionumber });
- unless ($record) {
- carp "UpdateTotalIssues could not get biblio record";
+ my $biblio = Koha::Biblios->find($biblionumber);
+ unless ($biblio) {
+ carp "UpdateTotalIssues could not get biblio";
return;
}
- my $biblio = Koha::Biblios->find( $biblionumber );
- unless ($biblio) {
- carp "UpdateTotalIssues could not get datas of biblio";
+
+ my $record = $biblio->metadata->record;
+ unless ($record) {
+ carp "UpdateTotalIssues could not get biblio record";
return;
}
my $biblioitem = $biblio->biblioitem;
$record->insert_grouped_field($field);
}
- return ModBiblio($record, $biblionumber, $biblio->frameworkcode);
+ return ModBiblio($record, $biblionumber, $biblio->frameworkcode, { skip_holds_queue => $skip_holds_queue });
}
=head2 RemoveAllNsb
carp 'ApplyMarcOverlayRules called on undefined record';
return;
}
- my $old_record = GetMarcBiblio({ biblionumber => $biblionumber });
+ my $biblio = Koha::Biblios->find($biblionumber);
+ my $old_record = $biblio->metadata->record;
# Skip overlay rules if called with no context
if ($old_record && defined $params->{overlay_context}) {
projects / koha-ffzg.git / blobdiff