bulkmarcimport.pl Bug Fix matching biblios enhanced
[koha.git] / misc / migration_tools / bulkmarcimport.pl
index 85e6b25..2e583c8 100755 (executable)
@@ -1,38 +1,37 @@
 #!/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;
 
@@ -40,155 +39,119 @@ GetOptions(
     '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";
 }
@@ -196,114 +159,307 @@ if ($test_parameter) {
 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";
+}