#!/usr/bin/perl
-# small script that import an iso2709 file into koha 2.0
+# Import an iso2709 file into Koha 3
use strict;
-# use warnings;
+use warnings;
+#use diagnostics;
+BEGIN {
+ # find Koha's Perl modules
+ # test carefully before changing this
+ use FindBin;
+ eval { require "$FindBin::Bin/../kohalib.pl" };
+}
# Koha modules used
use MARC::File::USMARC;
-# Uncomment the line below and use MARC::File::XML again when it works better.
-# -- thd
-# use MARC::File::XML;
+use MARC::File::XML;
use MARC::Record;
use MARC::Batch;
use MARC::Charset;
-# According to kados, an undocumented feature of setting MARC::Charset to
-# ignore_errors(1) is that errors are not ignored. Instead of deleting the
-# whole subfield when a character does not translate properly from MARC8 into
-# UTF-8, just the problem characters are deleted. This should solve at least
-# some of the fixme problems for fMARC8ToUTF8().
-#
-# Problems remain if there are MARC 21 records where 000/09 is set incorrectly.
-# -- thd.
-# MARC::Charset->ignore_errors(1);
-
use C4::Context;
use C4::Biblio;
+use C4::Koha;
+use C4::Charset;
+use C4::Items;
+use Unicode::Normalize;
use Time::HiRes qw(gettimeofday);
use Getopt::Long;
-binmode(STDOUT, ":utf8");
-
-use Getopt::Long;
+use IO::File;
-my ( $input_marc_file, $number) = ('',0);
-my ($version, $delete, $test_parameter,$char_encoding, $verbose, $commit,$fk_off);
+binmode(STDOUT, ":utf8");
+my ( $input_marc_file, $number, $offset) = ('',0,0);
+my ($version, $delete, $test_parameter, $skip_marc8_conversion, $char_encoding, $verbose, $commit, $fk_off,$format,$biblios,$authorities,$keepids,$match, $isbn_check, $logfile);
+my ($sourcetag,$sourcesubfield,$idmapfl);
$|=1;
'commit:f' => \$commit,
'file:s' => \$input_marc_file,
'n:f' => \$number,
+ 'o|offset:f' => \$offset,
'h' => \$version,
'd' => \$delete,
't' => \$test_parameter,
+ 's' => \$skip_marc8_conversion,
'c:s' => \$char_encoding,
'v:s' => \$verbose,
'fk' => \$fk_off,
+ 'm:s' => \$format,
+ 'l:s' => \$logfile,
+ 'k|keepids:s' => \$keepids,
+ 'b|biblios' => \$biblios,
+ 'a|authorities' => \$authorities,
+ 'match=s@' => \$match,
+ 'i|isbn' => \$isbn_check,
+ 'x:s' => \$sourcetag,
+ 'y:s' => \$sourcesubfield,
+ 'idmap:s' => \$idmapfl,
);
-
-# FIXME: Management of error conditions needed for record parsing problems
-# and MARC8 character sets with mappings to Unicode not yet included in
-# MARC::Charset. The real world rarity of these problems is not fully tested.
-# Unmapped character sets will throw a warning currently and processing will
-# continue with the error condition. A fairly trivial correction should
-# address some record parsing and unmapped character set problems but I need
-# time to implement a test and correction for undef subfields and revert to
-# MARC8 if mappings are missing. -- thd
-sub fMARC8ToUTF8($$) {
- my ($record) = shift;
- my ($verbose) = shift;
- if ($verbose) {
- if ($verbose >= 2) {
- my $leader = $record->leader();
- $leader =~ s/ /#/g;
- print "\n000 " . $leader;
- }
- }
- foreach my $field ($record->fields()) {
- if ($field->is_control_field()) {
- if ($verbose) {
- if ($verbose >= 2) {
- my $fieldName = $field->tag();
- my $fieldValue = $field->data();
- $fieldValue =~ s/ /#/g;
- print "\n" . $fieldName;
- print ' ' . $fieldValue;
- }
- }
- } else {
- my @subfieldsArray;
- my $fieldName = $field->tag();
- my $indicator1Value = $field->indicator(1);
- my $indicator2Value = $field->indicator(2);
- if ($verbose) {
- if ($verbose >= 2) {
- $indicator1Value =~ s/ /#/;
- $indicator2Value =~ s/ /#/;
- print "\n" . $fieldName . ' ' .
- $indicator1Value .
- $indicator2Value;
- }
- }
- foreach my $subfield ($field->subfields()) {
- my $subfieldName = $subfield->[0];
- my $subfieldValue = $subfield->[1];
- $subfieldValue = MARC::Charset::marc8_to_utf8($subfieldValue);
-
- # Alas, MARC::Field::update() does not work correctly.
- ## push (@subfieldsArray, $subfieldName, $subfieldValue);
-
- push @subfieldsArray, [$subfieldName, $subfieldValue];
- if ($verbose) {
- if ($verbose >= 2) {
- print " \$" . $subfieldName . ' ' . $subfieldValue;
- }
- }
- }
-
- # Alas, MARC::Field::update() does not work correctly.
- #
- # The first instance in the field of a of a repeated subfield
- # overwrites the content from later instances with the content
- # from the first instance.
- ## $field->update(@subfieldsArray);
-
- foreach my $subfieldRow(@subfieldsArray) {
- my $subfieldName = $subfieldRow->[0];
- $field->delete_subfields($subfieldName);
- }
- foreach my $subfieldRow(@subfieldsArray) {
- $field->add_subfields(@$subfieldRow);
- }
-
- if ($verbose) {
- if ($verbose >= 2) {
- # Reading the indicator values again is not necessary.
- # They were not converted.
- # $indicator1Value = $field->indicator(1);
- # $indicator2Value = $field->indicator(2);
- # $indicator1Value =~ s/ /#/;
- # $indicator2Value =~ s/ /#/;
- print "\nCONVERTED TO UTF-8:\n" . $fieldName . ' ' .
- $indicator1Value .
- $indicator2Value;
- foreach my $subfield ($field->subfields()) {
- my $subfieldName = $subfield->[0];
- my $subfieldValue = $subfield->[1];
- print " \$" . $subfieldName . ' ' . $subfieldValue;
- }
- }
- }
- if ($verbose) {
- if ($verbose >= 2) {
- print "\n" if $verbose;
- }
- }
- }
- }
- $record->encoding('UTF-8');
- return $record;
-}
-
+$biblios=!$authorities||$biblios;
if ($version || ($input_marc_file eq '')) {
print <<EOF
-small script to import an iso2709 file into Koha.
-parameters :
-\th : this version/help screen
-\tfile /path/to/file/to/dump : the file to import
-\tv : verbose mode. 1 means "some infos", 2 means "MARC dumping"
-\tfk : Turn off foreign key checks during import.
-\tn : the number of records to import. If missing, all the file is imported
-\tcommit : the number of records to wait before performing a 'commit' operation
-\tt : test mode : parses the file, saying what he would do, but doing nothing.
-\tc : the characteristic MARC flavour. At the moment, only MARC21 and UNIMARC
-\tsupported. MARC21 by default.
-\td : delete EVERYTHING related to biblio in koha-DB before import :tables :
-\t\tbiblio, \tbiblioitems,\titems
-IMPORTANT : don't use this script before you've entered and checked your MARC parameters tables twice (or more!).
-Otherwise, the import won't work correctly and you will get invalid data.
-
-SAMPLE :
-\t\$ export KOHA_CONF=/etc/koha.conf
-\t\$ perl misc/migration_tools/bulkmarcimport.pl -d -commit 1000 -file /home/jmf/koha.mrc -n 3000
+Small script to import bibliographic records into Koha.
+
+Parameters:
+ h this version/help screen
+ file /path/to/file/to/dump: the file to import
+ v verbose mode. 1 means "some infos", 2 means "MARC dumping"
+ fk Turn off foreign key checks during import.
+ n the number of records to import. If missing, all the file is imported
+ o file offset before importing, ie number of records to skip.
+ commit the number of records to wait before performing a 'commit' operation
+ l file logs actions done for each record and their status into file
+ t test mode: parses the file, saying what he would do, but doing nothing.
+ s skip automatic conversion of MARC-8 to UTF-8. This option is
+ provided for debugging.
+ c the characteristic MARC flavour. At the moment, only MARC21 and
+ UNIMARC are supported. MARC21 by default.
+ d delete EVERYTHING related to biblio in koha-DB before import. Tables:
+ biblio, biblioitems, titems
+ m format, MARCXML or ISO2709 (defaults to ISO2709)
+ keepids field store ids in field (usefull for authorities, where 001 contains the authid for Koha, that can contain a very valuable info for authorities coming from LOC or BNF. useless for biblios probably)
+ b|biblios type of import : bibliographic records
+ a|authorities type of import : authority records
+ match matchindex,fieldtomatch matchpoint to use to deduplicate
+ fieldtomatch can be either 001 to 999
+ or field and list of subfields as such 100abcde
+ i|isbn if set, a search will be done on isbn, and, if the same isbn is found, the biblio is not added. It's another
+ method to deduplicate.
+ match & i can be both set.
+ x source bib tag for reporting the source bib number
+ y source subfield for reporting the source bib number
+ idmap file for the koha bib and source id
+ keepids store ids in 009 (usefull for authorities, where 001 contains the authid for Koha, that can contain a very valuable info for authorities coming from LOC or BNF. useless for biblios probably)
+ b|biblios type of import : bibliographic records
+ a|authorities type of import : authority records
+ match matchindex,fieldtomatch matchpoint to use to deduplicate
+ fieldtomatch can be either 001 to 999
+ or field and list of subfields as such 100abcde
+ i|isbn if set, a search will be done on isbn, and, if the same isbn is found, the biblio is not added. It's another
+ method to deduplicate.
+ match & i can be both set.
+IMPORTANT: don't use this script before you've entered and checked your MARC
+ parameters tables twice (or more!). Otherwise, the import won't work
+ correctly and you will get invalid data.
+
+SAMPLE:
+ \$ export KOHA_CONF=/etc/koha.conf
+ \$ perl misc/migration_tools/bulkmarcimport.pl -d -commit 1000 \\
+ -file /home/jmf/koha.mrc -n 3000
EOF
;#'
exit;
}
-my $dbh = C4::Context->dbh;
+if (defined $idmapfl) {
+ open(IDMAP,">$idmapfl") or die "cannot open $idmapfl \n";
+}
-if ($delete) {
- print "deleting biblios\n";
- $dbh->do("truncate biblio");
- $dbh->do("truncate biblioitems");
- $dbh->do("truncate items");
+if ((not defined $sourcesubfield) && (not defined $sourcetag)){
+ $sourcetag="910";
+ $sourcesubfield="a";
}
+
+my $dbh = C4::Context->dbh;
+
+# save the CataloguingLog property : we don't want to log a bulkmarcimport. It will slow the import &
+# will create problems in the action_logs table, that can't handle more than 1 entry per second per user.
+my $CataloguingLog = C4::Context->preference('CataloguingLog');
+$dbh->do("UPDATE systempreferences SET value=0 WHERE variable='CataloguingLog'");
+
if ($fk_off) {
$dbh->do("SET FOREIGN_KEY_CHECKS = 0");
}
+
+
+if ($delete) {
+ if ($biblios){
+ print "deleting biblios\n";
+ $dbh->do("truncate biblio");
+ $dbh->do("truncate biblioitems");
+ $dbh->do("truncate items");
+ }
+ else {
+ print "deleting authorities\n";
+ $dbh->do("truncate auth_header");
+ }
+ $dbh->do("truncate zebraqueue");
+}
+
+
+
if ($test_parameter) {
print "TESTING MODE ONLY\n DOING NOTHING\n===============\n";
}
my $marcFlavour = C4::Context->preference('marcflavour') || 'MARC21';
print "Characteristic MARC flavour: $marcFlavour\n" if $verbose;
-# die;
my $starttime = gettimeofday;
-my $batch = MARC::Batch->new( 'USMARC', $input_marc_file );
+my $batch;
+my $fh = IO::File->new($input_marc_file); # don't let MARC::Batch open the file, as it applies the ':utf8' IO layer
+if (defined $format && $format =~ /XML/i) {
+ # ugly hack follows -- MARC::File::XML, when used by MARC::Batch,
+ # appears to try to convert incoming XML records from MARC-8
+ # to UTF-8. Setting the BinaryEncoding key turns that off
+ # TODO: see what happens to ISO-8859-1 XML files.
+ # TODO: determine if MARC::Batch can be fixed to handle
+ # XML records properly -- it probably should be
+ # be using a proper push or pull XML parser to
+ # extract the records, not using regexes to look
+ # for <record>.*</record>.
+ $MARC::File::XML::_load_args{BinaryEncoding} = 'utf-8';
+ my $recordformat= ($marcFlavour eq "MARC21"?"USMARC":uc($marcFlavour));
+#UNIMARC Authorities have a different way to manage encoding than UNIMARC biblios.
+ $recordformat=$recordformat."AUTH" if ($authorities and $marcFlavour ne "MARC21");
+ $MARC::File::XML::_load_args{RecordFormat} = $recordformat;
+ $batch = MARC::Batch->new( 'XML', $fh );
+} else {
+ $batch = MARC::Batch->new( 'USMARC', $fh );
+}
$batch->warnings_off();
$batch->strict_off();
my $i=0;
-my $commitnum = 50;
+my $commitnum = $commit ? $commit : 50;
-if ($commit) {
-$commitnum = $commit;
+# Skip file offset
+if ( $offset ) {
+ print "Skipping file offset: $offset records\n";
+ $batch->next() while ($offset--);
+}
+my ($tagid,$subfieldid);
+if ($authorities){
+ $tagid='001';
+}
+else {
+ ( $tagid, $subfieldid ) =
+ GetMarcFromKohaField( "biblio.biblionumber", '' );
+ $tagid||="001";
}
-#1st of all, find item MARC tag.
-my ($tagfield,$tagsubfield) = &GetMarcFromKohaField("items.itemnumber",'');
-# $dbh->do("lock tables biblio write, biblioitems write, items write, marc_biblio write, marc_subfield_table write, marc_blob_subfield write, marc_word write, marc_subfield_structure write, stopwords write");
-while ( my $record = $batch->next() ) {
-# warn "=>".$record->as_formatted;
-# warn "I:".$i;
-# warn "NUM:".$number;
+# the SQL query to search on isbn
+my $sth_isbn = $dbh->prepare("SELECT biblionumber,biblioitemnumber FROM biblioitems WHERE isbn=?");
+
+$dbh->{AutoCommit} = 0;
+my $loghandle;
+if ($logfile){
+ $loghandle= IO::File->new($logfile,"w") ;
+ print $loghandle "id;operation;status\n";
+}
+RECORD: while ( ) {
+ my $record;
+ # get records
+ eval { $record = $batch->next() };
+ if ( $@ ) {
+ print "Bad MARC record: skipped\n";
+ # FIXME - because MARC::Batch->next() combines grabbing the next
+ # blob and parsing it into one operation, a correctable condition
+ # such as a MARC-8 record claiming that it's UTF-8 can't be recovered
+ # from because we don't have access to the original blob. Note
+ # that the staging import can deal with this condition (via
+ # C4::Charset::MarcToUTF8Record) because it doesn't use MARC::Batch.
+ next;
+ }
+ # skip if we get an empty record (that is MARC valid, but will result in AddBiblio failure
+ last unless ( $record );
$i++;
print ".";
print "\r$i" unless $i % 100;
-# if ($i==$number) {
-# z3950_extended_services('commit',set_service_options('commit'));
-# print "COMMIT OPERATION SUCCESSFUL\n";
-#
-# my $timeneeded = gettimeofday - $starttime;
-# die "$i MARC records imported in $timeneeded seconds\n";
-# }
-# # perform the commit operation ever so often
-# if ($i==$commit) {
-# z3950_extended_services('commit',set_service_options('commit'));
-# $commit+=$commitnum;
-# print "COMMIT OPERATION SUCCESSFUL\n";
-# }
- #now, parse the record, extract the item fields, and store them in somewhere else.
-
- ## create an empty record object to populate
- my $newRecord = MARC::Record->new();
- $newRecord->leader($record->leader());
-
- # go through each field in the existing record
- foreach my $oldField ( $record->fields() ) {
-
- # just reproduce tags < 010 in our new record
- #
- # Fields are not necessarily only numeric in the actual world of records
- # nor in what I would recommend for additonal safe non-interfering local
- # use fields. The following regular expression match is much safer than
- # a numeric evaluation. -- thd
- if ( $oldField->tag() =~ m/^00/ ) {
- $newRecord->append_fields( $oldField );
- next();
+
+ # transcode the record to UTF8 if needed & applicable.
+ if ($record->encoding() eq 'MARC-8' and not $skip_marc8_conversion) {
+ # FIXME update condition
+ my ($guessed_charset, $charset_errors);
+ ($record, $guessed_charset, $charset_errors) = MarcToUTF8Record($record, $marcFlavour.(($authorities and $marcFlavour ne "MARC21")?'AUTH':''));
+ if ($guessed_charset eq 'failed') {
+ warn "ERROR: failed to perform character conversion for record $i\n";
+ next RECORD;
+ }
}
-
- # store our new subfield data in this list
- my @newSubfields = ();
-
- # go through each subfield code/data pair
- foreach my $pair ( $oldField->subfields() ) {
- #$pair->[1] =~ s/\<//g;
- #$pair->[1] =~ s/\>//g;
- push( @newSubfields, $pair->[0], $pair->[1] ); #char_decode($pair->[1],$char_encoding) );
+ my $isbn;
+ # remove trailing - in isbn (only for biblios, of course)
+ if ($biblios) {
+ if ($marcFlavour eq 'UNIMARC') {
+ if (my $f010 = $record->field('010')) {
+ $isbn = $f010->subfield('a');
+ $isbn =~ s/-//g;
+ $f010->update('a' => $isbn);
+ }
+ } else {
+ if (my $f020 = $record->field('020')) {
+ $isbn = $f020->subfield('a');
+ $isbn =~ s/-//g;
+ $f020->update('a' => $isbn);
+ }
+ }
}
-
- # add the new field to our new record
- my $newField = MARC::Field->new(
- $oldField->tag(),
- $oldField->indicator(1),
- $oldField->indicator(2),
- @newSubfields
- );
-
- $newRecord->append_fields( $newField );
-
+ my $id;
+ # search for duplicates (based on Local-number)
+ if ($match){
+ require C4::Search;
+ my $query=build_query($match,$record);
+ my $server=($authorities?'authorityserver':'biblioserver');
+ my ($error, $results,$totalhits)=C4::Search::SimpleSearch( $query, 0, 3, [$server] );
+ die "unable to search the database for duplicates : $error" if (defined $error);
+ warn "$query $server : $totalhits";
+ if ($results && scalar(@$results)==1){
+ my $marcrecord = MARC::File::USMARC::decode($results->[0]);
+ $id=GetRecordId($marcrecord,$tagid,$subfieldid);
+ }
+ elsif ($results && scalar(@$results)>1){
+ warn "more than one match for $query";
+ }
+ else {
+ warn "nomatch for $query";
+ }
}
-
- warn "$i ==>".$newRecord->as_formatted() if $verbose eq 2;
- my @fields = $newRecord->field($tagfield);
- my @items;
- my $nbitems=0;
-
- foreach my $field (@fields) {
- my $item = MARC::Record->new();
- $item->append_fields($field);
- push @items,$item;
- $newRecord->delete_field($field);
- $nbitems++;
+ my $originalid;
+ if ($keepids){
+ $originalid=GetRecordId($record,$tagid,$subfieldid);
+ if ($originalid){
+ my $storeidfield;
+ if (length($keepids)==3){
+ $storeidfield=MARC::Field->new($keepids,$originalid);
+ }
+ else {
+ $storeidfield=MARC::Field->new(substr($keepids,0,3),"","",substr($keepids,3,1),$originalid);
+ }
+ $record->insert_fields_ordered($storeidfield);
+ $record->delete_field($record->field($tagid));
+ }
}
- print "$i : $nbitems items found\n" if $verbose;
- # now, create biblio and items with Addbiblio call.
unless ($test_parameter) {
- my ($bibid,$oldbibitemnum) = AddBiblio($newRecord,'');
- warn "ADDED biblio NB $bibid in DB\n" if $verbose;
- for (my $i=0;$i<=$#items;$i++) {
-# warn "here is the biblioitemnumber $oldbibitemnum";
- AddItem($items[$i],$bibid,$oldbibitemnum);
+ if ($authorities){
+ use C4::AuthoritiesMarc;
+ my $authtypecode=GuessAuthTypeCode($record);
+ my $authid= ($id?$id:GuessAuthId($record));
+ if ($authid && GetAuthority($authid)){
+ ## Authority has an id and is in database : Replace
+ eval { ( $authid ) = ModAuthority($authid,$record, $authtypecode) };
+ if ($@){
+ warn "Problem with authority $authid Cannot Modify";
+ printlog({id=>$originalid||$id||$authid, op=>"edit",status=>"ERROR"}) if ($logfile);
+ }
+ else{
+ printlog({id=>$originalid||$id||$authid, op=>"edit",status=>"ok"}) if ($logfile);
+ }
+ }
+ elsif (defined $authid) {
+ ## An authid is defined but no authority in database : add
+ eval { ( $authid ) = AddAuthority($record,$authid, $authtypecode) };
+ if ($@){
+ warn "Problem with authority $authid Cannot Add ".$@;
+ printlog({id=>$originalid||$id||$authid, op=>"insert",status=>"ERROR"}) if ($logfile);
+ }
+ else{
+ printlog({id=>$originalid||$id||$authid, op=>"insert",status=>"ok"}) if ($logfile);
+ }
+ }
+ else {
+ ## True insert in database
+ eval { ( $authid ) = AddAuthority($record,"", $authtypecode) };
+ if ($@){
+ warn "Problem with authority $authid Cannot Add".$@;
+ printlog({id=>$originalid||$id||$authid, op=>"insert",status=>"ERROR"}) if ($logfile);
+ }
+ else{
+ printlog({id=>$originalid||$id||$authid, op=>"insert",status=>"ok"}) if ($logfile);
+ }
+ }
}
+ else {
+ my ( $biblionumber, $biblioitemnumber, $itemnumbers_ref, $errors_ref );
+ $biblionumber = $id;
+ # check for duplicate, based on ISBN (skip it if we already have found a duplicate with match parameter
+ if (!$biblionumber && $isbn_check && $isbn) {
+ # warn "search ISBN : $isbn";
+ $sth_isbn->execute($isbn);
+ ($biblionumber,$biblioitemnumber) = $sth_isbn->fetchrow;
+ }
+ if (defined $idmapfl) {
+ if ($sourcetag < "010"){
+ if ($record->field($sourcetag)){
+ my $source = $record->field($sourcetag)->data();
+ printf(IDMAP "%s|%s\n",$source,$biblionumber);
+ }
+ } else {
+ my $source=$record->subfield($sourcetag,$sourcesubfield);
+ printf(IDMAP "%s|%s\n",$source,$biblionumber);
+ }
+ }
+ # create biblio, unless we already have it ( either match or isbn )
+ if ($biblionumber) {
+ eval{$biblioitemnumber=GetBiblioData($biblionumber)->{biblioitemnumber};}
+ }
+ else
+ {
+ eval { ( $biblionumber, $biblioitemnumber ) = AddBiblio($record, '', { defer_marc_save => 1 }) };
+ }
+ if ( $@ ) {
+ warn "ERROR: Adding biblio $biblionumber failed: $@\n";
+ printlog({id=>$id||$originalid||$biblionumber, op=>"insert",status=>"ERROR"}) if ($logfile);
+ next RECORD;
+ }
+ else{
+ printlog({id=>$id||$originalid||$biblionumber, op=>"insert",status=>"ok"}) if ($logfile);
+ }
+ eval { ( $itemnumbers_ref, $errors_ref ) = AddItemBatchFromMarc( $record, $biblionumber, $biblioitemnumber, '' ); };
+ if ( $@ ) {
+ warn "ERROR: Adding items to bib $biblionumber failed: $@\n";
+ printlog({id=>$id||$originalid||$biblionumber, op=>"insertitem",status=>"ERROR"}) if ($logfile);
+ # if we failed because of an exception, assume that
+ # the MARC columns in biblioitems were not set.
+ ModBiblioMarc( $record, $biblionumber, '' );
+ next RECORD;
+ }
+ else{
+ printlog({id=>$id||$originalid||$biblionumber, op=>"insert",status=>"ok"}) if ($logfile);
+ }
+ if ($#{ $errors_ref } > -1) {
+ report_item_errors($biblionumber, $errors_ref);
+ }
+ }
+ $dbh->commit() if (0 == $i % $commitnum);
}
+ last if $i == $number;
}
+$dbh->commit();
+
+
+
if ($fk_off) {
$dbh->do("SET FOREIGN_KEY_CHECKS = 1");
}
-# final commit of the changes
-#z3950_extended_services('commit',set_service_options('commit'));
-#print "COMMIT OPERATION SUCCESSFUL\n";
+
+# restore CataloguingLog
+$dbh->do("UPDATE systempreferences SET value=$CataloguingLog WHERE variable='CataloguingLog'");
my $timeneeded = gettimeofday - $starttime;
-print "$i MARC records done in $timeneeded seconds\n";
+print "\n$i MARC records done in $timeneeded seconds\n";
+if ($logfile){
+ print $loghandle "file : $input_marc_file\n";
+ print $loghandle "$i MARC records done in $timeneeded seconds\n";
+ $loghandle->close;
+}
+exit 0;
+
+sub GetRecordId{
+ my $marcrecord=shift;
+ my $tag=shift;
+ my $subfield=shift;
+ my $id;
+ if ($tag lt "010"){
+ return $marcrecord->field($tag)->data() if $marcrecord->field($tag);
+ }
+ elsif ($subfield){
+ if ($marcrecord->field($tag)){
+ return $marcrecord->subfield($tag,$subfield);
+ }
+ }
+ return $id;
+}
+sub build_query {
+ my $match = shift;
+ my $record=shift;
+ my @searchstrings;
+ foreach my $matchingpoint (@$match){
+ my $string = build_simplequery($matchingpoint,$record);
+ push @searchstrings,$string if (length($string)>0);
+ }
+ return join(" and ",@searchstrings);
+}
+sub build_simplequery {
+ my $element=shift;
+ my $record=shift;
+ my ($index,$recorddata)=split /,/,$element;
+ my ($tag,$subfields) =($1,$2) if ($recorddata=~/(\d{3})(.*)/);
+ my @searchstrings;
+ foreach my $field ($record->field($tag)){
+ if (length($field->as_string("$subfields"))>0){
+ push @searchstrings,"$index,wrdl=\"".$field->as_string("$subfields")."\"";
+ }
+ }
+ return join(" and ",@searchstrings);
+}
+sub report_item_errors {
+ my $biblionumber = shift;
+ my $errors_ref = shift;
+
+ foreach my $error (@{ $errors_ref }) {
+ my $msg = "Item not added (bib $biblionumber, item tag #$error->{'item_sequence'}, barcode $error->{'item_barcode'}): ";
+ my $error_code = $error->{'error_code'};
+ $error_code =~ s/_/ /g;
+ $msg .= "$error_code $error->{'error_information'}";
+ print $msg, "\n";
+ }
+}
+sub printlog{
+ my $logelements=shift;
+ print $loghandle join (";",@$logelements{qw<id op status>}),"\n";
+}