bugfix : misusing prepare & execute => now using prepare(?) and execute($var)
[koha.git] / C4 / Biblio.pm
index 55a18a1..eafaaa4 100644 (file)
@@ -1,74 +1,4 @@
-package C4::Biblio; 
-# $Id$
-# $Log$
-# Revision 1.9  2002/09/20 12:57:46  tipaul
-# long is the road to 1.4.0
-# * MARCadditem and MARCmoditem now wroks
-# * various bugfixes in MARC management
-# !!! 1.3.0 should be released very soon now. Be careful !!!
-#
-# Revision 1.8  2002/09/10 13:53:52  tipaul
-# MARC API continued...
-# * some bugfixes
-# * multiple item management : MARCadditem and MARCmoditem have been added. They suppose that ALL the MARC field linked to koha-item are in the same MARC tag (on the same line of MARC file)
-#
-# Note : it should not be hard for marcimport and marcexport to re-link fields from internal tag/subfield to "legal" tag/subfield.
-#
-# Revision 1.7  2002/08/14 18:12:51  tonnesen
-# Added copyright statement to all .pl and .pm files
-#
-# Revision 1.6  2002/07/25 13:40:31  tipaul
-# pod documenting the API.
-#
-# Revision 1.5  2002/07/24 16:11:37  tipaul
-# Now, the API...
-# Database.pm and Output.pm are almost not modified (var test...)
-#
-# Biblio.pm is almost completly rewritten.
-#
-# WHAT DOES IT ??? ==> END of Hitchcock suspens
-#
-# 1st, it does... nothing...
-# Every old API should be there. So if MARC-stuff is not done, the behaviour is EXACTLY the same (if there is no added bug, of course). So, if you use normal acquisition, you won't find anything new neither on screen or old-DB tables ...
-#
-# All old-API functions have been cloned. for example, the "newbiblio" sub, now has become :
-# * a "newbiblio" sub, with the same parameters. It just call a sub named OLDnewbiblio
-# * a "OLDnewbiblio" sub, which is a copy/paste of the previous newbiblio sub. Then, when you want to add the MARC-DB stuff, you can modify the newbiblio sub without modifying the OLDnewbiblio one. If we correct a bug in 1.2 in newbiblio, we can do the same in main branch by correcting OLDnewbiblio.
-# * The MARC stuff is usually done through a sub named MARCxxx where xxx is the same as OLDxxx. For example, newbiblio calls MARCnewbiblio. the MARCxxx subs use a MARC::Record as parameter.
-# The last thing to solve was to manage biblios through real MARC import : they must populate the old-db, but must populate the MARC-DB too, without loosing information (if we go from MARC::Record to old-data then back to MARC::Record, we loose A LOT OF ROWS). To do this, there are subs beginning by "ALLxxx" : they manage datas with MARC::Record datas. they call OLDxxx sub too (to populate old-DB), but MARCxxx subs too, with a complete MARC::Record ;-)
-#
-# In Biblio.pm, there are some subs that permits to build a old-style record from a MARC::Record, and the opposite. There is also a sub finding a MARC-bibid from a old-biblionumber and the opposite too.
-# Note we have decided with steve that a old-biblio <=> a MARC-Biblio.
-#
-
-
-# move from 1.2 to 1.4 version : 
-# 1.2 and previous version uses a specific API to manage biblios. This API uses old-DB style parameters.
-# In the 1.4 version, we want to do 2 differents things :
-#  - keep populating the old-DB, that has a LOT less datas than MARC
-#  - populate the MARC-DB
-# To populate the DBs we have 2 differents sources :
-#  - the standard acquisition system (through book sellers), that does'nt use MARC data
-#  - the MARC acquisition system, that uses MARC data.
-#
-# thus, we have 2 differents cases :
-#   - with the standard acquisition system, we have non MARC data and want to populate old-DB and MARC-DB, knowing it's an incomplete MARC-record
-#   - with the MARC acquisition system, we have MARC datas, and want to loose nothing in MARC-DB. So, we can't store datas in old-DB, then copy in MARC-DB.
-#       we MUST have an API for true MARC data, that populate MARC-DB then old-DB
-#
-# That's why we need 4 subs :
-# all subs beginning by MARC manage only MARC tables. They manage MARC-DB with MARC::Record parameters
-# all subs beginning by OLD manage only OLD-DB tables. They manage old-DB with old-DB parameters
-# all subs beginning by ALL manage both OLD-DB and MARC tables. They use MARC::Record as parameters. it's the API that MUST be used in MARC acquisition system
-# all subs beginning by seomething else are the old-style API. They use old-DB as parameter, then call internally the OLD and MARC subs.
-#
-# only ALL and old-style API should be used in koha. MARC and OLD is used internally only
-#
-# Thus, we assume a nice translation to future versions : if we want in a 1.6 release completly forget old-DB, we can do it easily.
-# in 1.4 version, the translations will be nicer, as we have NOTHING to do in code. Everything has to be done in Biblio.pm ;-)
-
-
-
+package C4::Biblio;
 # Copyright 2000-2002 Katipo Communications
 #
 # This file is part of Koha.
@@ -88,184 +18,238 @@ package C4::Biblio;
 
 use strict;
 require Exporter;
+use C4::Context;
 use C4::Database;
 use MARC::Record;
 
-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
+use vars qw($VERSION @ISA @EXPORT);
 
 # set the version for version checking
 $VERSION = 0.01;
 
 @ISA = qw(Exporter);
 #
-# don't forget MARCxxx subs are here only for testing purposes. Should not be used
-# as the old-style API and the ALL one are the only public functions.
+# don't forget MARCxxx subs are exported only for testing purposes. Should not be used
+# as the old-style API and the NEW one are the only public functions.
 #
 @EXPORT = qw(
-            &updateBiblio &updateBiblioItem &updateItem 
-            &itemcount &newbiblio &newbiblioitem 
+            &updateBiblio &updateBiblioItem &updateItem
+            &itemcount &newbiblio &newbiblioitem
             &modnote &newsubject &newsubtitle
             &modbiblio &checkitems
             &newitems &modbibitem
-            &modsubtitle &modsubject &modaddauthor &moditem &countitems 
-            &delitem &deletebiblioitem &delbiblio  
+            &modsubtitle &modsubject &modaddauthor &moditem &countitems
+            &delitem &deletebiblioitem &delbiblio
             &getitemtypes &getbiblio
             &getbiblioitembybiblionumber
-            &getbiblioitem &getitemsbybiblioitem &isbnsearch
+            &getbiblioitem &getitemsbybiblioitem
             &skip
             &newcompletebiblioitem
 
-            &ALLnewbiblio &ALLnewitem
-
+            &MARCfind_oldbiblionumber_from_MARCbibid
+            &MARCfind_MARCbibid_from_oldbiblionumber
+               &MARCfind_marc_from_kohafield
+            &MARCfindsubfield
             &MARCgettagslib
+
+               &NEWnewbiblio &NEWnewitem
+               &NEWmodbiblio &NEWmoditem
+               &NEWdelbiblio &NEWdelitem
+
             &MARCaddbiblio &MARCadditem
-            &MARCmodsubfield &MARCaddsubfield 
+            &MARCmodsubfield &MARCaddsubfield
             &MARCmodbiblio &MARCmoditem
-            &MARCfindsubfield 
-            &MARCkoha2marcBiblio &MARCmarc2koha &MARCkoha2marcItem
+            &MARCkoha2marcBiblio &MARCmarc2koha
+               &MARCkoha2marcItem &MARChtml2marc
             &MARCgetbiblio &MARCgetitem
             &MARCaddword &MARCdelword
+               &char_decode
  );
-%EXPORT_TAGS = ( );     # eg: TAG => [ qw!name1 name2! ],
 
-# your exported package globals go here,
-# as well as any optionally exported functions
+#
+#
+# MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC
+#
+#
+# all the following subs takes a MARC::Record as parameter and manage
+# the MARC-DB. They are called by the 1.0/1.2 xxx subs, and by the
+# NEWxxx subs (xxx deals with old-DB parameters, the NEWxxx deals with MARC-DB parameter)
 
-@EXPORT_OK   = qw($Var1 %Hashit);
+=head1 NAME
 
+C4::Biblio - acquisition, catalog  management functions
 
-# non-exported package globals go here
-use vars qw(@more $stuff);
+=head1 SYNOPSIS
 
-# initalize package globals, first exported ones
+move from 1.2 to 1.4 version :
+1.2 and previous version uses a specific API to manage biblios. This API uses old-DB style parameters.
+In the 1.4 version, we want to do 2 differents things :
+ - keep populating the old-DB, that has a LOT less datas than MARC
+ - populate the MARC-DB
+To populate the DBs we have 2 differents sources :
+ - the standard acquisition system (through book sellers), that does'nt use MARC data
+ - the MARC acquisition system, that uses MARC data.
 
-my $Var1   = '';
-my %Hashit = ();
+Thus, we have 2 differents cases :
+- with the standard acquisition system, we have non MARC data and want to populate old-DB and MARC-DB, knowing it's an incomplete MARC-record
+- with the MARC acquisition system, we have MARC datas, and want to loose nothing in MARC-DB. So, we can't store datas in old-DB, then copy in MARC-DB. we MUST have an API for true MARC data, that populate MARC-DB then old-DB
 
+That's why we need 4 subs :
+all I<subs beginning by MARC> manage only MARC tables. They manage MARC-DB with MARC::Record parameters
+all I<subs beginning by OLD> manage only OLD-DB tables. They manage old-DB with old-DB parameters
+all I<subs beginning by NEW> manage both OLD-DB and MARC tables. They use MARC::Record as parameters. it's the API that MUST be used in MARC acquisition system
+all I<subs beginning by seomething else> are the old-style API. They use old-DB as parameter, then call internally the OLD and MARC subs.
 
-# then the others (which are still accessible as $Some::Module::stuff)
-my $stuff  = '';
-my @more   = ();
+- NEW and old-style API should be used in koha to manage biblio
+- MARCsubs are divided in 2 parts :
+* some of them manage MARC parameters. They are heavily used in koha.
+* some of them manage MARC biblio : they are mostly used by NEW and old-style subs.
+- OLD are used internally only
 
-# all file-scoped lexicals must be created before
-# the functions below that use them.
+all subs requires/use $dbh as 1st parameter.
 
-# file-private lexicals go here
-my $priv_var    = '';
-my %secret_hash = ();
+I<NEWxxx related subs>
 
-# here's a file-private function as a closure,
-# callable as &$priv_func;  it cannot be prototyped.
-my $priv_func = sub {
-  # stuff goes here.
-  };
-  
-# make all your functions, whether exported or not;
+all subs requires/use $dbh as 1st parameter.
+those subs are used by the MARC-compliant version of koha : marc import, or marc management.
 
-#
-#
-# MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC
-#
-#
-# all the following subs takes a MARC::Record as parameter and manage
-# the MARC-DB. They are called by the 1.0/1.2 xxx subs, and by the 
-# ALLxxx subs (xxx deals with old-DB parameters, the ALLxxx deals with MARC-DB parameter)
+I<OLDxxx related subs>
 
-=head1 SYNOPSIS
-  MARCxxx related subs
-  all subs requires/use $dbh as 1st parameter.
-  NOTE : all those subs are private and must be used only inside Biblio.pm (called by a old API sub, or the ALLsub)
+all subs requires/use $dbh as 1st parameter.
+those subs are used by the MARC-compliant version of koha : marc import, or marc management.
+
+They all are the exact copy of 1.0/1.2 version of the sub without the OLD.
+The OLDxxx is called by the original xxx sub.
+the 1.4 xxx sub also builds MARC::Record an calls the MARCxxx
+
+WARNING : there is 1 difference between initialxxx and OLDxxx :
+the db header $dbh is always passed as parameter to avoid over-DB connexion
 
 =head1 DESCRIPTION
 
-=head2 @tagslib = &MARCgettagslib($dbh,1|0);
-      last param is 1 for liblibrarian and 0 for libopac
-      returns a hash with tag/subfield meaning
+=over 4
 
-=head2 ($tagfield,$tagsubfield) = &MARCfindmarc_from_kohafield($dbh,$kohafield);
-      finds MARC tag and subfield for a given kohafield
-      kohafield is "table.field" where table= biblio|biblioitems|items, and field a field of the previous table
+=item @tagslib = &MARCgettagslib($dbh,1|0);
 
-=head2 $biblionumber = &MARCfind_oldbiblionumber_from_MARCbibid($dbh,$MARCbibi);
-      finds a old-db biblio number for a given MARCbibid number
+last param is 1 for liblibrarian and 0 for libopac
+returns a hash with tag/subfield meaning
+=item ($tagfield,$tagsubfield) = &MARCfind_marc_from_kohafield($dbh,$kohafield);
 
-=head2 $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$oldbiblionumber);
-      finds a MARC bibid from a old-db biblionumber
+finds MARC tag and subfield for a given kohafield
+kohafield is "table.field" where table= biblio|biblioitems|items, and field a field of the previous table
 
-=head2 &MARCaddbiblio($dbh,$MARC::Record,$biblionumber);
-      creates a biblio (in the MARC tables only). $biblionumber is the old-db biblionumber of the biblio
+=item $biblionumber = &MARCfind_oldbiblionumber_from_MARCbibid($dbh,$MARCbibi);
 
-=head2 &MARCaddsubfield($dbh,$bibid,$tagid,$indicator,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
-      adds a subfield in a biblio (in the MARC tables only).
-     
-=head2 $MARCRecord = &MARCgetbiblio($dbh,$bibid);
-      Returns a MARC::Record for the biblio $bibid.
+finds a old-db biblio number for a given MARCbibid number
 
-=head2 &MARCmodbiblio($dbh,$bibid,$delete,$record);
-      MARCmodbiblio changes a biblio for a biblio,MARC::Record passed as parameter
-      if $delete == 1, every field/subfield not found is deleted in the biblio
-      otherwise, only data passed to MARCmodbiblio is managed.
-      thus, you can change only a small part of a biblio (like an item, or a subtitle, or a additionalauthor...)
+=item $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$oldbiblionumber);
 
-=head2 ($subfieldid,$subfieldvalue) = &MARCmodsubfield($dbh,$subfieldid,$subfieldvalue);
-      MARCmodsubfield changes the value of a given subfield
+finds a MARC bibid from a old-db biblionumber
 
-=head2 $subfieldid = &MARCfindsubfield($dbh,$bibid,$tag,$subfieldcode,$subfieldorder,$subfieldvalue);
-      MARCfindsubfield returns a subfield number given a bibid/tag/subfieldvalue values.
-      Returns -1 if more than 1 answer
+=item $MARCRecord = &MARCkoha2marcBiblio($dbh,$biblionumber,biblioitemnumber);
 
-=head2 $subfieldid = &MARCfindsubfieldid($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder);
-      MARCfindsubfieldid find a subfieldid for a bibid/tag/tagorder/subfield/subfieldorder
+MARCkoha2marcBiblio is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB biblio/biblioitem
 
-=head2 &MARCdelsubfield($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder);
-      MARCdelsubfield delete a subfield for a bibid/tag/tagorder/subfield/subfieldorder
+=item $MARCRecord = &MARCkoha2marcItem($dbh,$biblionumber,itemnumber);
 
-=head2 &MARCdelbiblio($dbh,$bibid);
-      MARCdelbiblio delete biblio $bibid
+MARCkoha2marcItem is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB item
 
-=head2 $MARCRecord = &MARCkoha2marcBiblio($dbh,$biblionumber,biblioitemnumber);
-      MARCkoha2marcBiblio is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB biblio/biblioitem
+=item $MARCRecord = &MARCkoha2marcSubtitle($dbh,$biblionumber,$subtitle);
 
-=head2 $MARCRecord = &MARCkoha2marcItem($dbh,$biblionumber,itemnumber);
-      MARCkoha2marcItem is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB item
+MARCkoha2marcSubtitle is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB subtitle
 
-=head2 $MARCRecord = &MARCkoha2marcSubtitle($dbh,$biblionumber,$subtitle);
-      MARCkoha2marcSubtitle is a wrapper between old-DB and MARC-DB. It returns a MARC::Record builded with old-DB subtitle
+=item $olddb = &MARCmarc2koha($dbh,$MARCRecord);
 
-=head2 &MARCkoha2marcOnefield => used by MARCkoha2marc and should not be useful elsewhere
+builds a hash with old-db datas from a MARC::Record
 
-=head2 $olddb = &MARCmarc2koha($dbh,$MARCRecord);
-      builds a hash with old-db datas from a MARC::Record
+=item &MARCaddbiblio($dbh,$MARC::Record,$biblionumber);
 
-=head2 &MARCmarc2kohaOnefield => used by MARCmarc2koha and should not be useful elsewhere
+creates a biblio (in the MARC tables only). $biblionumber is the old-db biblionumber of the biblio
 
-=head2 MARCaddword => used to manage MARC_word table and should not be useful elsewhere
+=item &MARCaddsubfield($dbh,$bibid,$tagid,$indicator,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
 
-=head2 MARCdelword => used to manage MARC_word table and should not be useful elsewhere
+adds a subfield in a biblio (in the MARC tables only).
 
-=head1 AUTHOR
+=item $MARCRecord = &MARCgetbiblio($dbh,$bibid);
 
-Paul POULAIN paul.poulain@free.fr
+Returns a MARC::Record for the biblio $bibid.
+
+=item &MARCmodbiblio($dbh,$bibid,$record,$delete);
+
+MARCmodbiblio changes a biblio for a biblio,MARC::Record passed as parameter
+It 1st delete the biblio, then recreates it.
+WARNING : the $delete parameter is not used anymore (too much unsolvable cases).
+=item ($subfieldid,$subfieldvalue) = &MARCmodsubfield($dbh,$subfieldid,$subfieldvalue);
+
+MARCmodsubfield changes the value of a given subfield
+
+=item $subfieldid = &MARCfindsubfield($dbh,$bibid,$tag,$subfieldcode,$subfieldorder,$subfieldvalue);
+
+MARCfindsubfield returns a subfield number given a bibid/tag/subfieldvalue values.
+Returns -1 if more than 1 answer
+
+=item $subfieldid = &MARCfindsubfieldid($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder);
+
+MARCfindsubfieldid find a subfieldid for a bibid/tag/tagorder/subfield/subfieldorder
+
+=item &MARCdelsubfield($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder);
+
+MARCdelsubfield delete a subfield for a bibid/tag/tagorder/subfield/subfieldorder
+
+=item &MARCdelbiblio($dbh,$bibid);
+
+MARCdelbiblio delete biblio $bibid
+
+=item &MARCkoha2marcOnefield
+
+used by MARCkoha2marc and should not be useful elsewhere
+
+=item &MARCmarc2kohaOnefield
+
+used by MARCmarc2koha and should not be useful elsewhere
+
+=item MARCaddword
+
+used to manage MARC_word table and should not be useful elsewhere
+
+=item MARCdelword
+
+used to manage MARC_word table and should not be useful elsewhere
 
 =cut
 
 sub MARCgettagslib {
-    my ($dbh,$forlibrarian)= @_;
-    my $sth;
-    if ($forlibrarian eq 1) {
-       $sth=$dbh->prepare("select tagfield,tagsubfield,liblibrarian as lib from marc_subfield_structure");
-    } else {
-       $sth=$dbh->prepare("select tagfield,tagsubfield,libopac as lib from marc_subfield_structure");
-    }
-    $sth->execute;
-    my $lib;
-    my $tag;
-    my $subfield;
-    my $res;
-    while ( ($tag,$subfield,$lib) = $sth->fetchrow) {
-       $res->{$tag}->{$subfield}=$lib;
-    }
-    return $res;
+       my ($dbh,$forlibrarian)= @_;
+       my $sth;
+       my $libfield = ($forlibrarian eq 1)? 'liblibrarian' : 'libopac';
+       $sth=$dbh->prepare("select tagfield,$libfield as lib,mandatory from marc_tag_structure order by tagfield");
+       $sth->execute;
+       my ($lib,$tag,$res,$tab,$mandatory,$repeatable);
+       while ( ($tag,$lib,$mandatory) = $sth->fetchrow) {
+               $res->{$tag}->{lib}=$lib;
+               $res->{$tab}->{tab}=""; # XXX
+               $res->{$tag}->{mandatory}=$mandatory;
+       }
+
+       $sth=$dbh->prepare("select tagfield,tagsubfield,$libfield as lib,tab, mandatory, repeatable,authorised_value,thesaurus_category,value_builder,kohafield from marc_subfield_structure order by tagfield,tagsubfield");
+       $sth->execute;
+
+       my $subfield;
+       my $authorised_value;
+       my $thesaurus_category;
+       my $value_builder;
+       my $kohafield;
+       while ( ($tag, $subfield, $lib, $tab, $mandatory, $repeatable,$authorised_value,$thesaurus_category,$value_builder,$kohafield) = $sth->fetchrow) {
+               $res->{$tag}->{$subfield}->{lib}=$lib;
+               $res->{$tag}->{$subfield}->{tab}=$tab;
+               $res->{$tag}->{$subfield}->{mandatory}=$mandatory;
+               $res->{$tag}->{$subfield}->{repeatable}=$repeatable;
+               $res->{$tag}->{$subfield}->{authorised_value}=$authorised_value;
+               $res->{$tag}->{$subfield}->{thesaurus_category}=$thesaurus_category;
+               $res->{$tag}->{$subfield}->{value_builder}=$value_builder;
+               $res->{$tag}->{$subfield}->{kohafield}=$kohafield;
+       }
+       return $res;
 }
 
 sub MARCfind_marc_from_kohafield {
@@ -294,40 +278,57 @@ sub MARCfind_MARCbibid_from_oldbiblionumber {
 
 sub MARCaddbiblio {
 # pass the MARC::Record to this function, and it will create the records in the marc tables
-    my ($dbh,$record,$biblionumber) = @_;
-    my @fields=$record->fields();
-    my $bibid;
-    # adding main table, and retrieving bibid
-    $dbh->do("lock tables marc_biblio WRITE,marc_subfield_table WRITE, marc_word WRITE, marc_blob_subfield WRITE, stopwords READ");
-    my $sth=$dbh->prepare("insert into marc_biblio (datecreated,biblionumber) values (now(),?)");
-    $sth->execute($biblionumber);
-    $sth=$dbh->prepare("select max(bibid) from marc_biblio");
-    $sth->execute;
-    ($bibid)=$sth->fetchrow;
-    $sth->finish;
-    my $fieldcount=0;
-    # now, add subfields...
-    foreach my $field (@fields) {
-       my @subfields=$field->subfields();
-       $fieldcount++;
-       foreach my $subfieldcount (0..$#subfields) {
-                   &MARCaddsubfield($dbh,$bibid,
-                                $field->tag(),
-                                $field->indicator(1).$field->indicator(2),
-                                $fieldcount,
-                                $subfields[$subfieldcount][0],
-                                $subfieldcount+1,
-                                $subfields[$subfieldcount][1]
-                                );
+       my ($dbh,$record,$biblionumber,$bibid) = @_;
+       my @fields=$record->fields();
+#      warn "IN MARCaddbiblio $bibid => ".$record->as_formatted;
+# my $bibid;
+# adding main table, and retrieving bibid
+# if bibid is sent, then it's not a true add, it's only a re-add, after a delete (ie, a mod)
+# if bibid empty => true add, find a new bibid number
+       unless ($bibid) {
+               $dbh->do("lock tables marc_biblio WRITE,marc_subfield_table WRITE, marc_word WRITE, marc_blob_subfield WRITE, stopwords READ");
+               my $sth=$dbh->prepare("insert into marc_biblio (datecreated,biblionumber) values (now(),?)");
+               $sth->execute($biblionumber);
+               $sth=$dbh->prepare("select max(bibid) from marc_biblio");
+               $sth->execute;
+               ($bibid)=$sth->fetchrow;
+               $sth->finish;
        }
-    }
-    $dbh->do("unlock tables");
-    return $bibid;
+       my $fieldcount=0;
+       # now, add subfields...
+       foreach my $field (@fields) {
+               $fieldcount++;
+               if ($field->tag() <10) {
+                               &MARCaddsubfield($dbh,$bibid,
+                                               $field->tag(),
+                                               '',
+                                               $fieldcount,
+                                               '',
+                                               1,
+                                               $field->data()
+                                               );
+               } else {
+                       my @subfields=$field->subfields();
+                       foreach my $subfieldcount (0..$#subfields) {
+                               &MARCaddsubfield($dbh,$bibid,
+                                               $field->tag(),
+                                               $field->indicator(1).$field->indicator(2),
+                                               $fieldcount,
+                                               $subfields[$subfieldcount][0],
+                                               $subfieldcount+1,
+                                               $subfields[$subfieldcount][1]
+                                               );
+                       }
+               }
+       }
+       $dbh->do("unlock tables");
+       return $bibid;
 }
 
 sub MARCadditem {
 # pass the MARC::Record to this function, and it will create the records in the marc tables
     my ($dbh,$record,$biblionumber) = @_;
+#    warn "adding : ".$record->as_formatted();
 # search for MARC biblionumber
     $dbh->do("lock tables marc_biblio WRITE,marc_subfield_table WRITE, marc_word WRITE, marc_blob_subfield WRITE, stopwords READ");
     my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$biblionumber);
@@ -356,106 +357,124 @@ sub MARCadditem {
 
 sub MARCaddsubfield {
 # Add a new subfield to a tag into the DB.
-    my ($dbh,$bibid,$tagid,$indicator,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue) = @_;
-    # if not value, end of job, we do nothing
-    if (not($subfieldvalue)) {
-       return;
-    }
-    if (not($subfieldcode)) {
-       $subfieldcode=' ';
-    }
-    if (length($subfieldvalue)>255) {
-#      $dbh->do("lock tables marc_blob_subfield WRITE, marc_subfield_table WRITE");
-       my $sth=$dbh->prepare("insert into marc_blob_subfield (subfieldvalue) values (?)");
-       $sth->execute($subfieldvalue);
-       $sth=$dbh->prepare("select max(blobidlink)from marc_blob_subfield");
-       $sth->execute;
-       my ($res)=$sth->fetchrow;
-       $sth=$dbh->prepare("insert into marc_subfield_table (bibid,tag,tagorder,subfieldcode,subfieldorder,valuebloblink) values (?,?,?,?,?,?)");
-       if ($tagid<100) {
-           $sth->execute($bibid,'0'.$tagid,$tagorder,$subfieldcode,$subfieldorder,$res);
-       } else {
-           $sth->execute($bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$res);
+       my ($dbh,$bibid,$tagid,$tag_indicator,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalues) = @_;
+       # if not value, end of job, we do nothing
+       if (length($subfieldvalues) ==0) {
+               return;
        }
-       if ($sth->errstr) {
-           print STDERR "ERROR ==> insert into marc_subfield_table (bibid,tag,tagorder,subfieldcode,subfieldorder,subfieldvalue) values ($bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue)\n";
+       if (not($subfieldcode)) {
+               $subfieldcode=' ';
        }
-#      $dbh->do("unlock tables");
-    } else {
-       my $sth=$dbh->prepare("insert into marc_subfield_table (bibid,tag,tagorder,subfieldcode,subfieldorder,subfieldvalue) values (?,?,?,?,?,?)");
-       $sth->execute($bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
-       if ($sth->errstr) {
-           print STDERR "ERROR ==> insert into marc_subfield_table (bibid,tag,tagorder,subfieldcode,subfieldorder,subfieldvalue) values ($bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue)\n";
+       my @subfieldvalues = split /\|/,$subfieldvalues;
+       foreach my $subfieldvalue (@subfieldvalues) {
+               if (length($subfieldvalue)>255) {
+                       $dbh->do("lock tables marc_blob_subfield WRITE, marc_subfield_table WRITE");
+                       my $sth=$dbh->prepare("insert into marc_blob_subfield (subfieldvalue) values (?)");
+                       $sth->execute($subfieldvalue);
+                       $sth=$dbh->prepare("select max(blobidlink)from marc_blob_subfield");
+                       $sth->execute;
+                       my ($res)=$sth->fetchrow;
+                       $sth=$dbh->prepare("insert into marc_subfield_table (bibid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,valuebloblink) values (?,?,?,?,?,?,?)");
+                       $sth->execute($bibid,(sprintf "%03s",$tagid),$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$res);
+                       if ($sth->errstr) {
+                               warn "ERROR ==> insert into marc_subfield_table (bibid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue) values ($bibid,$tagid,$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$subfieldvalue)\n";
+                       }
+               $dbh->do("unlock tables");
+               } else {
+                       my $sth=$dbh->prepare("insert into marc_subfield_table (bibid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue) values (?,?,?,?,?,?,?)");
+                       $sth->execute($bibid,(sprintf "%03s",$tagid),$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$subfieldvalue);
+                       if ($sth->errstr) {
+                       warn "ERROR ==> insert into marc_subfield_table (bibid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue) values ($bibid,$tagid,$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$subfieldvalue)\n";
+                       }
+               }
+               &MARCaddword($dbh,$bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
        }
-    }
-    &MARCaddword($dbh,$bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
 }
 
-
 sub MARCgetbiblio {
 # Returns MARC::Record of the biblio passed in parameter.
     my ($dbh,$bibid)=@_;
     my $record = MARC::Record->new();
 #---- TODO : the leader is missing
-    my $sth=$dbh->prepare("select bibid,subfieldid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue,valuebloblink 
-                                from marc_subfield_table 
-                                where bibid=? order by tagorder,subfieldorder
+       $record->leader('                        ');
+    my $sth=$dbh->prepare("select bibid,subfieldid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue,valuebloblink
+                                from marc_subfield_table
+                                where bibid=? order by tag,tagorder,subfieldcode
                         ");
-    my $sth2=$dbh->prepare("select subfieldvalue from marc_blob_subfield where blobidlink=?");
-    $sth->execute($bibid);
-    while (my $row=$sth->fetchrow_hashref) {
-       if ($row->{'valuebloblink'}) { #---- search blob if there is one
-           $sth2->execute($row->{'valuebloblink'});
-           my $row2=$sth2->fetchrow_hashref;
-           $sth2->finish;
-           $row->{'subfieldvalue'}=$row2->{'subfieldvalue'};
+       my $sth2=$dbh->prepare("select subfieldvalue from marc_blob_subfield where blobidlink=?");
+       $sth->execute($bibid);
+       my $prevtagorder=1;
+       my $prevtag='XXX';
+       my $previndicator;
+       my $field; # for >=10 tags
+       my $prevvalue; # for <10 tags
+       while (my $row=$sth->fetchrow_hashref) {
+               if ($row->{'valuebloblink'}) { #---- search blob if there is one
+                       $sth2->execute($row->{'valuebloblink'});
+                       my $row2=$sth2->fetchrow_hashref;
+                       $sth2->finish;
+                       $row->{'subfieldvalue'}=$row2->{'subfieldvalue'};
+               }
+               if ($row->{tagorder} ne $prevtagorder || $row->{tag} ne $prevtag) {
+                       $previndicator.="  ";
+                       if ($prevtag <10) {
+                       $record->add_fields((sprintf "%03s",$prevtag),$prevvalue) unless $prevtag eq "XXX"; # ignore the 1st loop
+                       } else {
+                               $record->add_fields($field) unless $prevtag eq "XXX";
+                       }
+                       undef $field;
+                       $prevtagorder=$row->{tagorder};
+                       $prevtag = $row->{tag};
+                       $previndicator=$row->{tag_indicator};
+                       if ($row->{tag}<10) {
+                               $prevvalue = $row->{subfieldvalue};
+                       } else {
+                               $field = MARC::Field->new((sprintf "%03s",$prevtag), substr($row->{tag_indicator}.'  ',0,1), substr($row->{tag_indicator}.'  ',1,1), $row->{'subfieldcode'}, $row->{'subfieldvalue'} );
+                       }
+               } else {
+                       if ($row->{tag} <10) {
+                               $record->add_fields((sprintf "%03s",$row->{tag}), $row->{'subfieldvalue'});
+                       } else {
+                               $field->add_subfields($row->{'subfieldcode'}, $row->{'subfieldvalue'} );
+                       }
+                       $prevtag= $row->{tag};
+                       $previndicator=$row->{tag_indicator};
+               }
        }
-       if ($record->field($row->{'tag'})) {
-           my $field;
-#--- this test must stay as this, because of strange behaviour of mySQL/Perl DBI with char var containing a number...
-#--- sometimes, eliminates 0 at beginning, sometimes no ;-\\\
-           if (length($row->{'tag'}) <3) {
-               $row->{'tag'} = "0".$row->{'tag'};
-           }
-           $field =$record->field($row->{'tag'});
-           if ($field) {
-               my $x = $field->add_subfields($row->{'subfieldcode'},$row->{'subfieldvalue'});
-               $record->delete_field($field);
-               $record->add_fields($field);
-           }
-       } else {
-           if (length($row->{'tag'}) < 3) {
-               $row->{'tag'} = "0".$row->{'tag'};
-           }
-           my $temp = MARC::Field->new($row->{'tag'}," "," ", $row->{'subfieldcode'} => $row->{'subfieldvalue'});
-           $record->add_fields($temp);
+       # the last has not been included inside the loop... do it now !
+       if ($prevtag ne "XXX") { # check that we have found something. Otherwise, prevtag is still XXX and we
+                                               # must return an empty record, not make MARC::Record fail because we try to
+                                               # create a record with XXX as field :-(
+               if ($prevtag <10) {
+                       $record->add_fields($prevtag,$prevvalue);
+               } else {
+       #               my $field = MARC::Field->new( $prevtag, "", "", %subfieldlist);
+                       $record->add_fields($field);
+               }
        }
-
-    }
-    return $record;
+       return $record;
 }
 sub MARCgetitem {
 # Returns MARC::Record of the biblio passed in parameter.
     my ($dbh,$bibid,$itemnumber)=@_;
-    warn "MARCgetitem :   $bibid, $itemnumber\n";
     my $record = MARC::Record->new();
 # search MARC tagorder
     my $sth2 = $dbh->prepare("select tagorder from marc_subfield_table,marc_subfield_structure where marc_subfield_table.tag=marc_subfield_structure.tagfield and marc_subfield_table.subfieldcode=marc_subfield_structure.tagsubfield and bibid=? and kohafield='items.itemnumber' and subfieldvalue=?");
     $sth2->execute($bibid,$itemnumber);
     my ($tagorder) = $sth2->fetchrow_array();
 #---- TODO : the leader is missing
-    my $sth=$dbh->prepare("select bibid,subfieldid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue,valuebloblink 
-                                from marc_subfield_table 
-                                where bibid=? and tagorder=? order by subfieldorder
+    my $sth=$dbh->prepare("select bibid,subfieldid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue,valuebloblink
+                                from marc_subfield_table
+                                where bibid=? and tagorder=? order by subfieldcode,subfieldorder
                         ");
-    my $sth2=$dbh->prepare("select subfieldvalue from marc_blob_subfield where blobidlink=?");
-    $sth->execute($bibid,$tagorder);
-    while (my $row=$sth->fetchrow_hashref) {
+       $sth2=$dbh->prepare("select subfieldvalue from marc_blob_subfield where blobidlink=?");
+       $sth->execute($bibid,$tagorder);
+       while (my $row=$sth->fetchrow_hashref) {
        if ($row->{'valuebloblink'}) { #---- search blob if there is one
-           $sth2->execute($row->{'valuebloblink'});
-           my $row2=$sth2->fetchrow_hashref;
-           $sth2->finish;
-           $row->{'subfieldvalue'}=$row2->{'subfieldvalue'};
+               $sth2->execute($row->{'valuebloblink'});
+               my $row2=$sth2->fetchrow_hashref;
+               $sth2->finish;
+               $row->{'subfieldvalue'}=$row2->{'subfieldvalue'};
        }
        if ($record->field($row->{'tag'})) {
            my $field;
@@ -483,73 +502,101 @@ sub MARCgetitem {
 }
 
 sub MARCmodbiblio {
-    my ($dbh,$record,$bibid,$itemnumber,$delete)=@_;
-    my $oldrecord=&MARCgetbiblio($dbh,$bibid);
-# if nothing to change, don't waste time...
-    if ($oldrecord eq $record) {
-       return;
-    }
-# otherwise, skip through each subfield...
-    my @fields = $record->fields();
-    my $tagorder=0;
-    foreach my $field (@fields) {
-       my $oldfield = $oldrecord->field($field->tag());
-       my @subfields=$field->subfields();
-       my $subfieldorder=0;
-       $tagorder++;
-       foreach my $subfield (@subfields) {
-           $subfieldorder++;
-           if ($oldfield eq 0 or (! $oldfield->subfield(@$subfield[0])) ) {
-# just adding datas...
-               &MARCaddsubfield($dbh,$bibid,$field->tag(),$field->indicator(1).$field->indicator(2),
-                                1,@$subfield[0],$subfieldorder,@$subfield[1]);
-           } else {
-# modify he subfield if it's a different string
-               if ($oldfield->subfield(@$subfield[0]) ne @$subfield[1] ) {
-                   my $subfieldid=&MARCfindsubfieldid($dbh,$bibid,$field->tag(),$tagorder,@$subfield[0],$subfieldorder);
-                   &MARCmodsubfield($dbh,$subfieldid,@$subfield[1]);
-               } else {
-               }
-           }
+       my ($dbh,$bibid,$record,$delete)=@_;
+       my $oldrecord=&MARCgetbiblio($dbh,$bibid);
+       if ($oldrecord eq $record) {
+               return;
        }
-    }
+# 1st delete the biblio,
+# 2nd recreate it
+       my $biblionumber = MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       &MARCdelbiblio($dbh,$bibid,1);
+       &MARCaddbiblio($dbh,$record,$biblionumber,$bibid);
+}
+
+sub MARCdelbiblio {
+       my ($dbh,$bibid,$keep_items) = @_;
+# if the keep_item is set to 1, then all items are preserved.
+# This flag is set when the delbiblio is called by modbiblio
+# due to a too complex structure of MARC (repeatable fields and subfields),
+# the best solution for a modif is to delete / recreate the record.
+
+# 1st of all, copy the MARC::Record to deletedbiblio table => if a true deletion, MARC data will be kept.
+# if deletion called before MARCmodbiblio => won't do anything, as the oldbiblionumber doesn't
+# exist in deletedbiblio table
+       my $record = MARCgetbiblio($dbh,$bibid);
+       my $oldbiblionumber = MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       my $copy2deleted=$dbh->prepare("update deletedbiblio set marc=? where biblionumber=?");
+       $copy2deleted->execute($record->as_usmarc(),$oldbiblionumber);
+# now, delete in MARC tables.
+       if ($keep_items eq 1) {
+       #search item field code
+               my $sth = $dbh->prepare("select tagfield from marc_subfield_structure where kohafield like 'items.%'");
+               $sth->execute;
+               my $itemtag = $sth->fetchrow_hashref->{tagfield};
+               $dbh->do("delete from marc_subfield_table where bibid=$bibid and tag<>$itemtag");
+               $dbh->do("delete from marc_word where bibid=$bibid and tag<>$itemtag");
+       } else {
+               $dbh->do("delete from marc_biblio where bibid=$bibid");
+               $dbh->do("delete from marc_subfield_table where bibid=$bibid");
+               $dbh->do("delete from marc_word where bibid=$bibid");
+       }
+}
+
+sub MARCdelitem {
+# delete the item passed in parameter in MARC tables.
+       my ($dbh,$bibid,$itemnumber)=@_;
+       #    my $record = MARC::Record->new();
+       # search MARC tagorder
+       my $record = MARCgetitem($dbh,$bibid,$itemnumber);
+       my $copy2deleted=$dbh->prepare("update deleteditems set marc=? where itemnumber=?");
+       $copy2deleted->execute($record->as_usmarc(),$itemnumber);
+
+       my $sth2 = $dbh->prepare("select tagorder from marc_subfield_table,marc_subfield_structure where marc_subfield_table.tag=marc_subfield_structure.tagfield and marc_subfield_table.subfieldcode=marc_subfield_structure.tagsubfield and bibid=? and kohafield='items.itemnumber' and subfieldvalue=?");
+       $sth2->execute($bibid,$itemnumber);
+       my ($tagorder) = $sth2->fetchrow_array();
+       my $sth=$dbh->prepare("delete from marc_subfield_table where bibid=? and tagorder=?");
+       $sth->execute($bibid,$tagorder);
 }
+
 sub MARCmoditem {
-    my ($dbh,$record,$bibid,$itemnumber,$delete)=@_;
-    my $oldrecord=&MARCgetitem($dbh,$bibid,$itemnumber);
-# if nothing to change, don't waste time...
-    if ($oldrecord eq $record) {
-       return;
-    }
-# otherwise, skip through each subfield...
-    my @fields = $record->fields();
-# search old MARC item 
-    my $sth2 = $dbh->prepare("select tagorder from marc_subfield_table,marc_subfield_structure where marc_subfield_table.tag=marc_subfield_structure.tagfield and marc_subfield_table.subfieldcode=marc_subfield_structure.tagsubfield and bibid=? and kohafield='items.itemnumber' and subfieldvalue=?");
-    $sth2->execute($bibid,$itemnumber);
-    my ($tagorder) = $sth2->fetchrow_array();
-    foreach my $field (@fields) {
-       my $oldfield = $oldrecord->field($field->tag());
-       my @subfields=$field->subfields();
-       my $subfieldorder=0;
-       foreach my $subfield (@subfields) {
-           $subfieldorder++;
-           if ($oldfield eq 0 or (! $oldfield->subfield(@$subfield[0])) ) {
-# just adding datas...
-warn "ADD = $bibid,".$field->tag().",".$field->indicator(1).".".$field->indicator(2).", $tagorder,".@$subfield[0].",$subfieldorder,@$subfield[1])\n";
-               &MARCaddsubfield($dbh,$bibid,$field->tag(),$field->indicator(1).$field->indicator(2),
-                                $tagorder,@$subfield[0],$subfieldorder,@$subfield[1]);
-           } else {
-# modify he subfield if it's a different string
-warn "MODIFY = $bibid,".$field->tag().",".$field->indicator(1).".".$field->indicator(2).", $tagorder,".@$subfield[0].",$subfieldorder,@$subfield[1])\n";
-               if ($oldfield->subfield(@$subfield[0]) ne @$subfield[1] ) {
-                   my $subfieldid=&MARCfindsubfieldid($dbh,$bibid,$field->tag(),$tagorder,@$subfield[0],$subfieldorder);
-warn "MODIFY2 = $bibid, $subfieldid, ".@$subfield[1]."\n";
-                   &MARCmodsubfield($dbh,$subfieldid,@$subfield[1]);
-               } else {
+       my ($dbh,$record,$bibid,$itemnumber,$delete)=@_;
+       my $oldrecord=&MARCgetitem($dbh,$bibid,$itemnumber);
+       # if nothing to change, don't waste time...
+       if ($oldrecord eq $record) {
+               return;
+       }
+
+       # otherwise, skip through each subfield...
+       my @fields = $record->fields();
+       # search old MARC item
+       my $sth2 = $dbh->prepare("select tagorder from marc_subfield_table,marc_subfield_structure where marc_subfield_table.tag=marc_subfield_structure.tagfield and marc_subfield_table.subfieldcode=marc_subfield_structure.tagsubfield and bibid=? and kohafield='items.itemnumber' and subfieldvalue=?");
+       $sth2->execute($bibid,$itemnumber);
+       my ($tagorder) = $sth2->fetchrow_array();
+       foreach my $field (@fields) {
+               my $oldfield = $oldrecord->field($field->tag());
+               my @subfields=$field->subfields();
+               my $subfieldorder=0;
+               foreach my $subfield (@subfields) {
+                       $subfieldorder++;
+#                      warn "compare : $oldfield".$oldfield->subfield(@$subfield[0]);
+                       if ($oldfield eq 0 or (length($oldfield->subfield(@$subfield[0])) ==0) ) {
+               # just adding datas...
+#              warn "addfield : / $subfieldorder / @$subfield[0] - @$subfield[1]";
+#                              warn "NEW subfield : $bibid,".$field->tag().",".$tagorder.",".@$subfield[0].",".$subfieldorder.",".@$subfield[1].")";
+                               &MARCaddsubfield($dbh,$bibid,$field->tag(),$field->indicator(1).$field->indicator(2),
+                                               $tagorder,@$subfield[0],$subfieldorder,@$subfield[1]);
+                       } else {
+#              warn "modfield : / $subfieldorder / @$subfield[0] - @$subfield[1]";
+               # modify he subfield if it's a different string
+                               if ($oldfield->subfield(@$subfield[0]) ne @$subfield[1] ) {
+                                       my $subfieldid=&MARCfindsubfieldid($dbh,$bibid,$field->tag(),$tagorder,@$subfield[0],$subfieldorder);
+#                                      warn "changing : $subfieldid, $bibid,".$field->tag(),",$tagorder,@$subfield[0],@$subfield[1],$subfieldorder";
+                                       &MARCmodsubfield($dbh,$subfieldid,@$subfield[1]);
+                               }
+                       }
                }
-           }
        }
-    }
 }
 
 
@@ -623,12 +670,19 @@ sub MARCfindsubfield {
 }
 
 sub MARCfindsubfieldid {
-    my ($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder) = @_;
-    my $sth=$dbh->prepare("select subfieldid from marc_subfield_table
-                       where bibid=? and tag=? and tagorder=? 
-                               and subfieldcode=? and subfieldorder=?");
-    $sth->execute($bibid,$tag,$tagorder,$subfield,$subfieldorder);
-    my ($res) = $sth->fetchrow;
+       my ($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder) = @_;
+       my $sth=$dbh->prepare("select subfieldid from marc_subfield_table
+                               where bibid=? and tag=? and tagorder=?
+                                       and subfieldcode=? and subfieldorder=?");
+       $sth->execute($bibid,$tag,$tagorder,$subfield,$subfieldorder);
+       my ($res) = $sth->fetchrow;
+       unless ($res) {
+               $sth=$dbh->prepare("select subfieldid from marc_subfield_table
+                               where bibid=? and tag=? and tagorder=?
+                                       and subfieldcode=?");
+               $sth->execute($bibid,$tag,$tagorder,$subfield);
+               ($res) = $sth->fetchrow;
+       }
     return $res;
 }
 
@@ -636,18 +690,11 @@ sub MARCdelsubfield {
 # delete a subfield for $bibid / tag / tagorder / subfield / subfieldorder
     my ($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder) = @_;
     $dbh->do("delete from marc_subfield_table where bibid='$bibid' and
-                       tag='$tag' and tagorder='$tagorder' 
+                       tag='$tag' and tagorder='$tagorder'
                        and subfieldcode='$subfield' and subfieldorder='$subfieldorder
                        ");
 }
 
-sub MARCdelbiblio {
-# delete a biblio for a $bibid
-    my ($dbh,$bibid) = @_;
-    $dbh->do("delete from marc_subfield_table where bibid='$bibid'");
-    $dbh->do("delete from marc_biblio where bibid='$bibid'");
-}
-
 sub MARCkoha2marcBiblio {
 # this function builds partial MARC::Record from the old koha-DB fields
     my ($dbh,$biblionumber,$biblioitemnumber) = @_;
@@ -655,8 +702,8 @@ sub MARCkoha2marcBiblio {
     my $record = MARC::Record->new();
 #--- if bibid, then retrieve old-style koha data
     if ($biblionumber>0) {
-       my $sth2=$dbh->prepare("select biblionumber,author,title,unititle,notes,abstract,serial,seriestitle,copyrightdate,timestamp 
-               from biblio where biblionumber=?");             
+       my $sth2=$dbh->prepare("select biblionumber,author,title,unititle,notes,abstract,serial,seriestitle,copyrightdate,timestamp
+               from biblio where biblionumber=?");
        $sth2->execute($biblionumber);
        my $row=$sth2->fetchrow_hashref;
        my $code;
@@ -670,11 +717,11 @@ sub MARCkoha2marcBiblio {
     if ($biblioitemnumber>0) {
        my $sth2=$dbh->prepare(" SELECT biblioitemnumber,biblionumber,volume,number,classification,
                                                itemtype,url,isbn,issn,dewey,subclass,publicationyear,publishercode,
-                                               volumedate,volumeddesc,timestamp,illus,pages,notes,size,place 
+                                               volumedate,volumeddesc,timestamp,illus,pages,notes AS bnotes,size,place
                                        FROM biblioitems
-                                       WHERE biblionumber=? and biblioitemnumber=?
-                                       ");             
-       $sth2->execute($biblionumber,$biblioitemnumber);
+                                       WHERE biblioitemnumber=?
+                                       ");
+       $sth2->execute($biblioitemnumber);
        my $row=$sth2->fetchrow_hashref;
        my $code;
        foreach $code (keys %$row) {
@@ -683,8 +730,23 @@ sub MARCkoha2marcBiblio {
            }
        }
     }
+       # other fields => additional authors, subjects, subtitles
+       my $sth2=$dbh->prepare(" SELECT author FROM additionalauthors WHERE biblionumber=?");
+       $sth2->execute($biblionumber);
+       while (my $row=$sth2->fetchrow_hashref) {
+                       &MARCkoha2marcOnefield($sth,$record,"additionalauthors.author",$row->{'author'});
+               }
+       my $sth2=$dbh->prepare(" SELECT subject FROM bibliosubject WHERE biblionumber=?");
+       $sth2->execute($biblionumber);
+       while (my $row=$sth2->fetchrow_hashref) {
+                       &MARCkoha2marcOnefield($sth,$record,"bibliosubject.subject",$row->{'subject'});
+               }
+       my $sth2=$dbh->prepare(" SELECT subtitle FROM bibliosubtitle WHERE biblionumber=?");
+       $sth2->execute($biblionumber);
+       while (my $row=$sth2->fetchrow_hashref) {
+                       &MARCkoha2marcOnefield($sth,$record,"bibliosubtitle.title",$row->{'subtitle'});
+               }
     return $record;
-# TODO : retrieve notes, additionalauthors
 }
 
 sub MARCkoha2marcItem {
@@ -699,7 +761,7 @@ sub MARCkoha2marcItem {
        my $sth2=$dbh->prepare("SELECT itemnumber,biblionumber,multivolumepart,biblioitemnumber,barcode,dateaccessioned,
                                                booksellerid,homebranch,price,replacementprice,replacementpricedate,datelastborrowed,
                                                datelastseen,multivolume,stack,notforloan,itemlost,wthdrawn,bulk,issues,renewals,
-                                       reserves,restricted,binding,itemnotes,holdingbranch,interim,timestamp 
+                                       reserves,restricted,binding,itemnotes,holdingbranch,timestamp
                                        FROM items
                                        WHERE itemnumber=?");
        $sth2->execute($itemnumber);
@@ -712,7 +774,6 @@ sub MARCkoha2marcItem {
        }
     }
     return $record;
-# TODO : retrieve notes, additionalauthors
 }
 
 sub MARCkoha2marcSubtitle {
@@ -744,76 +805,131 @@ sub MARCkoha2marcOnefield {
     return $record;
 }
 
+sub MARChtml2marc {
+       my ($dbh,$rtags,$rsubfields,$rvalues,%indicators) = @_;
+       my $prevtag = -1;
+       my $record = MARC::Record->new();
+#      my %subfieldlist=();
+       my $prevvalue; # if tag <10
+       my $field; # if tag >=10
+       for (my $i=0; $i< @$rtags; $i++) {
+               # rebuild MARC::Record
+               if (@$rtags[$i] ne $prevtag) {
+                       if ($prevtag < 10) {
+                               if ($prevvalue) {
+                                       $record->add_fields((sprintf "%03s",$prevtag),$prevvalue);
+                               }
+                       } else {
+                               if ($field) {
+                                       $record->add_fields($field);
+                               }
+                       }
+                       $indicators{@$rtags[$i]}.='  ';
+                       if (@$rtags[$i] <10) {
+                               $prevvalue= @$rvalues[$i];
+                       } else {
+                               $field = MARC::Field->new( (sprintf "%03s",@$rtags[$i]), substr($indicators{@$rtags[$i]},0,1),substr($indicators{@$rtags[$i]},1,1), @$rsubfields[$i] => @$rvalues[$i]);
+                       }
+                       $prevtag = @$rtags[$i];
+               } else {
+                       if (@$rtags[$i] <10) {
+                               $prevvalue=@$rvalues[$i];
+                       } else {
+                               if (@$rvalues[$i]) {
+                                       $field->add_subfields(@$rsubfields[$i] => @$rvalues[$i]);
+                               }
+                       }
+                       $prevtag= @$rtags[$i];
+               }
+       }
+       # the last has not been included inside the loop... do it now !
+       $record->add_fields($field);
+#      warn $record->as_formatted;
+       return $record;
+}
+
 sub MARCmarc2koha {
-    my ($dbh,$record) = @_;
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
-    my $result;
-    my $sth2=$dbh->prepare("SHOW COLUMNS from biblio");
-    $sth2->execute;
-    my $field;
-#    print STDERR $record->as_formatted;
-    while (($field)=$sth2->fetchrow) {
-       $result=&MARCmarc2kohaOneField($sth,"biblio",$field,$record,$result);
-    }
-    my $sth2=$dbh->prepare("SHOW COLUMNS from biblioitems");
-    $sth2->execute;
-    my $field;
-    while (($field)=$sth2->fetchrow) {
-       $result=&MARCmarc2kohaOneField($sth,"biblioitems",$field,$record,$result);
-    }
-    my $sth2=$dbh->prepare("SHOW COLUMNS from items");
-    $sth2->execute;
-    my $field;
-    while (($field)=$sth2->fetchrow) {
-       $result = &MARCmarc2kohaOneField($sth,"items",$field,$record,$result);
-    }
-# additional authors : specific 
-    $result = &MARCmarc2kohaOneField($sth,"additionalauthors","additionalauthors",$record,$result);
-    return $result;
+       my ($dbh,$record) = @_;
+       my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+       my $result;
+       my $sth2=$dbh->prepare("SHOW COLUMNS from biblio");
+       $sth2->execute;
+       my $field;
+       #    print STDERR $record->as_formatted;
+       while (($field)=$sth2->fetchrow) {
+               $result=&MARCmarc2kohaOneField($sth,"biblio",$field,$record,$result);
+       }
+       $sth2=$dbh->prepare("SHOW COLUMNS from biblioitems");
+       $sth2->execute;
+       while (($field)=$sth2->fetchrow) {
+               if ($field eq 'notes') { $field = 'bnotes'; }
+               $result=&MARCmarc2kohaOneField($sth,"biblioitems",$field,$record,$result);
+       }
+       $sth2=$dbh->prepare("SHOW COLUMNS from items");
+       $sth2->execute;
+       while (($field)=$sth2->fetchrow) {
+               $result = &MARCmarc2kohaOneField($sth,"items",$field,$record,$result);
+       }
+       # additional authors : specific
+       $result = &MARCmarc2kohaOneField($sth,"bibliosubtitle","subtitle",$record,$result);
+       $result = &MARCmarc2kohaOneField($sth,"additionalauthors","additionalauthors",$record,$result);
+# modify copyrightdate to keep only the 1st year found
+       my $temp = $result->{'copyrightdate'};
+       $temp =~ m/c(\d\d\d\d)/; # search cYYYY first
+       if ($1>0) {
+               $result->{'copyrightdate'} = $1;
+       } else { # if no cYYYY, get the 1st date.
+               $temp =~ m/(\d\d\d\d)/;
+               $result->{'copyrightdate'} = $1;
+       }
+# modify publicationyear to keep only the 1st year found
+       my $temp = $result->{'publicationyear'};
+       $temp =~ m/c(\d\d\d\d)/; # search cYYYY first
+       if ($1>0) {
+               $result->{'publicationyear'} = $1;
+       } else { # if no cYYYY, get the 1st date.
+               $temp =~ m/(\d\d\d\d)/;
+               $result->{'publicationyear'} = $1;
+       }
+       return $result;
 }
 
 sub MARCmarc2kohaOneField {
-# to check : if a field has a repeatable subfield that is used in old-db, only the 1st will be retrieved...
-    my ($sth,$kohatable,$kohafield,$record,$result)= @_;
-    my $res="";
-    my $tagfield;
-    my $subfield;
-    $sth->execute($kohatable.".".$kohafield);
-    ($tagfield,$subfield) = $sth->fetchrow;
-    foreach my $field ($record->field($tagfield)) {
-       if ($field->subfield($subfield)) {
-           if ($result->{$kohafield}) {
-               $result->{$kohafield} .= " | ".$field->subfield($subfield);
-           } else {
-               $result->{$kohafield}=$field->subfield($subfield);
-           }
+# FIXME ? if a field has a repeatable subfield that is used in old-db, only the 1st will be retrieved...
+       my ($sth,$kohatable,$kohafield,$record,$result)= @_;
+#    warn "kohatable / $kohafield / $result / ";
+       my $res="";
+       my $tagfield;
+       my $subfield;
+       $sth->execute($kohatable.".".$kohafield);
+       ($tagfield,$subfield) = $sth->fetchrow;
+       foreach my $field ($record->field($tagfield)) {
+               if ($field->subfield($subfield)) {
+               if ($result->{$kohafield}) {
+                       $result->{$kohafield} .= " | ".$field->subfield($subfield);
+               } else {
+                       $result->{$kohafield}=$field->subfield($subfield);
+               }
+               }
        }
-    }
-    return $result;
+       return $result;
 }
 
 sub MARCaddword {
 # split a subfield string and adds it into the word table.
 # removes stopwords
     my ($dbh,$bibid,$tag,$tagorder,$subfieldid,$subfieldorder,$sentence) =@_;
-    $sentence =~ s/(\.|\?|\:|\!|\'|,|\-)/ /g;
+    $sentence =~ s/(\.|\?|\:|\!|\'|,|\-|\"|\(|\)|\[|\]|\{|\})/ /g;
     my @words = split / /,$sentence;
-# build stopword list
-    my $sth2 =$dbh->prepare("select word from stopwords");
-    $sth2->execute;
-    my $stopwords;
-    my $stopword;
-    while(($stopword) = $sth2->fetchrow_array)  {
-       $stopwords->{$stopword} = $stopword;
-    }
+    my $stopwords= C4::Context->stopwords;
     my $sth=$dbh->prepare("insert into marc_word (bibid, tag, tagorder, subfieldid, subfieldorder, word, sndx_word)
                        values (?,?,?,?,?,?,soundex(?))");
     foreach my $word (@words) {
 # we record only words longer than 2 car and not in stopwords hash
-       if (length($word)>1 and !($stopwords->{uc($word)})) {
+       if (length($word)>2 and !($stopwords->{uc($word)})) {
            $sth->execute($bibid,$tag,$tagorder,$subfieldid,$subfieldorder,$word,$word);
            if ($sth->err()) {
-               print STDERR "ERROR ==> insert into marc_word (bibid, tag, tagorder, subfieldid, subfieldorder, word, sndx_word) values ($bibid,$tag,$tagorder,$subfieldid,$subfieldorder,$word,soundex($word))\n";
+               warn "ERROR ==> insert into marc_word (bibid, tag, tagorder, subfieldid, subfieldorder, word, sndx_word) values ($bibid,$tag,$tagorder,$subfieldid,$subfieldorder,$word,soundex($word))\n";
            }
        }
     }
@@ -828,190 +944,247 @@ sub MARCdelword {
 
 #
 #
-# ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL ALL 
+# NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW NEW
 #
 #
 # all the following subs are useful to manage MARC-DB with complete MARC records.
 # it's used with marcimport, and marc management tools
 #
 
-=head1 SYNOPSIS
-  ALLxxx related subs
-  all subs requires/use $dbh as 1st parameter.
-  those subs are used by the MARC-compliant version of koha : marc import, or marc management.
-
-=head1 DESCRIPTION
 
-=head2 (oldbibnum,$oldbibitemnum) = ALLnewbibilio($dbh,$MARCRecord,$oldbiblio,$oldbiblioitem);
-  creates a new biblio from a MARC::Record. The 3rd and 4th parameter are hashes and may be ignored. If only 2 params are passed to the sub, the old-db hashes
-  are builded from the MARC::Record. If they are passed, they are used.
+=item ($bibid,$oldbibnum,$oldbibitemnum) = NEWnewbibilio($dbh,$MARCRecord,$oldbiblio,$oldbiblioitem);
 
-=head2 ALLnewitem($dbh,$olditem);
-  adds an item in the db. $olditem is a old-db hash.
+creates a new biblio from a MARC::Record. The 3rd and 4th parameter are hashes and may be ignored. If only 2 params are passed to the sub, the old-db hashes
+are builded from the MARC::Record. If they are passed, they are used.
 
-=head1 AUTHOR
+=item NEWnewitem($dbh, $record,$bibid);
 
-Paul POULAIN paul.poulain@free.fr
+adds an item in the db.
 
 =cut
 
-sub ALLnewbiblio {
-    my ($dbh, $record, $oldbiblio, $oldbiblioitem) = @_;
-# note $oldbiblio and $oldbiblioitem are not mandatory.
-# if not present, they will be builded from $record with MARCmarc2koha function
-    if (($oldbiblio) and not($oldbiblioitem)) {
-       print STDERR "ALLnewbiblio : missing parameter\n";
-       print "ALLnewbiblio : missing parameter : contact koha development  team\n";
-       die;
-    }
-    my $oldbibnum;
-    my $oldbibitemnum;
-    if ($oldbiblio) {
-       $oldbibnum = OLDnewbiblio($dbh,$oldbiblio);
-       $oldbiblioitem->{'biblionumber'} = $oldbibnum;
-       $oldbibitemnum = OLDnewbiblioitem($dbh,$oldbiblioitem);
-    } else {
-       my $olddata = MARCmarc2koha($dbh,$record);
-       $oldbibnum = OLDnewbiblio($dbh,$olddata);
-       $oldbibitemnum = OLDnewbiblioitem($dbh,$olddata);
-    }
-# we must add bibnum and bibitemnum in MARC::Record...
-# we build the new field with biblionumber and biblioitemnumber
-# we drop the original field
-# we add the new builded field.
-# NOTE : Works only if the field is ONLY for biblionumber and biblioitemnumber
-# (steve and paul : thinks 090 is a good choice)
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
-    $sth->execute("biblio.biblionumber");
-    (my $tagfield1, my $tagsubfield1) = $sth->fetchrow;
-    $sth->execute("biblioitems.biblioitemnumber");
-    (my $tagfield2, my $tagsubfield2) = $sth->fetchrow;
-    print STDERR "tag1 : $tagfield1 / $tagsubfield1\n tag2 : $tagfield2 / $tagsubfield2\n";
-    if ($tagsubfield1 != $tagsubfield2) {
-       print STDERR "Error in ALLnewbiblio : biblio.biblionumber and biblioitems.biblioitemnumber MUST have the same field number";
-       print "Error in ALLnewbiblio : biblio.biblionumber and biblioitems.biblioitemnumber MUST have the same field number";
-       die;
-    }
-    my $newfield = MARC::Field->new( $tagfield1,'','', 
-                                    "$tagsubfield1" => $oldbibnum,
-                                    "$tagsubfield2" => $oldbibitemnum);
-# drop old field and create new one...
-    my $old_field = $record->field($tagfield1);
-    $record->delete_field($old_field);
-    $record->add_fields($newfield);
-    my $bibid = MARCaddbiblio($dbh,$record,$oldbibnum);
-    return ( $oldbibnum,$oldbibitemnum );
-}
-
-sub ALLnewitem {
-    my ($dbh, $item) = @_;
-    my $itemnumber;
-    my $error;
-    ($itemnumber,$error) = &OLDnewitems($dbh,$item,$item->{'barcode'});
-# search MARC biblionumber 
-    my $bibid=&MARCfind_MARCbibid_from_oldbiblionumber($dbh,$item->{'biblionumber'});
-# calculate tagorder
-    my $sth = $dbh->prepare("select max(tagorder) from marc_subfield_table where bibid=?");
-    $sth->execute($bibid);
-    my ($tagorder) = $sth->fetchrow;
-    $tagorder++;
-    my $subfieldorder=0;
-# for each field, find MARC tag and subfield, and call the proper MARC sub
-    foreach my $itemkey (keys %$item) {
-       my $tagfield;
-       my $tagsubfield;
-       print STDERR "=============> $itemkey : ".$item->{$itemkey}."\n";
-       if ($itemkey eq "biblionumber" || $itemkey eq "biblioitemnumber") {
-           ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"biblio.".$itemkey);
+sub NEWnewbiblio {
+       my ($dbh, $record, $oldbiblio, $oldbiblioitem) = @_;
+       # note $oldbiblio and $oldbiblioitem are not mandatory.
+       # if not present, they will be builded from $record with MARCmarc2koha function
+       if (($oldbiblio) and not($oldbiblioitem)) {
+               print STDERR "NEWnewbiblio : missing parameter\n";
+               print "NEWnewbiblio : missing parameter : contact koha development  team\n";
+               die;
+       }
+       my $oldbibnum;
+       my $oldbibitemnum;
+       if ($oldbiblio) {
+               $oldbibnum = OLDnewbiblio($dbh,$oldbiblio);
+               $oldbiblioitem->{'biblionumber'} = $oldbibnum;
+               $oldbibitemnum = OLDnewbiblioitem($dbh,$oldbiblioitem);
        } else {
-           ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"items.".$itemkey);
+               my $olddata = MARCmarc2koha($dbh,$record);
+               $oldbibnum = OLDnewbiblio($dbh,$olddata);
+               $olddata->{'biblionumber'} = $oldbibnum;
+               $oldbibitemnum = OLDnewbiblioitem($dbh,$olddata);
        }
-       if ($tagfield && $item->{$itemkey} ne 'NULL') {
-           $subfieldorder++;
-           &MARCaddsubfield($dbh,
-                            $bibid,
-                            $tagfield,
-                            "  ",
-                            $tagorder,
-                            $tagsubfield,
-                            $subfieldorder,
-                            $item->{$itemkey}
-                            );
+       # search subtiles, addiauthors and subjects
+       my ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"additionalauthors.author");
+       my @addiauthfields = $record->field($tagfield);
+       foreach my $addiauthfield (@addiauthfields) {
+               my @addiauthsubfields = $addiauthfield->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#addiauthsubfields) {
+                       OLDmodaddauthor($dbh,$oldbibnum,$addiauthsubfields[$subfieldcount]);
+               }
        }
-    }
-} # ALLnewitems
-
+       ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"bibliosubtitle.title");
+       my @subtitlefields = $record->field($tagfield);
+       foreach my $subtitlefield (@subtitlefields) {
+               my @subtitlesubfields = $subtitlefield->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#subtitlesubfields) {
+                       OLDnewsubtitle($dbh,$oldbibnum,$subtitlesubfields[$subfieldcount]);
+               }
+       }
+       ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"bibliosubject.subject");
+       my @subj = $record->field($tagfield);
+       my @subjects;
+       foreach my $subject (@subj) {
+               my @subjsubfield = $subject->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#subjsubfield) {
+                       push @subjects,$subjsubfield[$subfieldcount];
+               }
+       }
+       OLDmodsubject($dbh,$oldbibnum,1,@subjects);
+       # we must add bibnum and bibitemnum in MARC::Record...
+       # we build the new field with biblionumber and biblioitemnumber
+       # we drop the original field
+       # we add the new builded field.
+       # NOTE : Works only if the field is ONLY for biblionumber and biblioitemnumber
+       # (steve and paul : thinks 090 is a good choice)
+       my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+       $sth->execute("biblio.biblionumber");
+       (my $tagfield1, my $tagsubfield1) = $sth->fetchrow;
+       $sth->execute("biblioitems.biblioitemnumber");
+       (my $tagfield2, my $tagsubfield2) = $sth->fetchrow;
+       if ($tagfield1 != $tagfield2) {
+               warn "Error in NEWnewbiblio : biblio.biblionumber and biblioitems.biblioitemnumber MUST have the same field number";
+               print "Content-Type: text/html\n\nError in NEWnewbiblio : biblio.biblionumber and biblioitems.biblioitemnumber MUST have the same field number";
+               die;
+       }
+       my $newfield = MARC::Field->new( $tagfield1,'','',
+                                               "$tagsubfield1" => $oldbibnum,
+                                               "$tagsubfield2" => $oldbibitemnum);
+       # drop old field and create new one...
+       my $old_field = $record->field($tagfield1);
+       $record->delete_field($old_field);
+       $record->add_fields($newfield);
+       my $bibid = MARCaddbiblio($dbh,$record,$oldbibnum);
+       return ($bibid,$oldbibnum,$oldbibitemnum );
+}
+
+sub NEWmodbiblio {
+       my ($dbh,$record,$bibid) =@_;
+       &MARCmodbiblio($dbh,$bibid,$record,0);
+       my $oldbiblio = MARCmarc2koha($dbh,$record);
+       my $oldbiblionumber = OLDmodbiblio($dbh,$oldbiblio);
+       OLDmodbibitem($dbh,$oldbiblio);
+       # now, modify addi authors, subject, addititles.
+       my ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"additionalauthors.author");
+       my @addiauthfields = $record->field($tagfield);
+       foreach my $addiauthfield (@addiauthfields) {
+               my @addiauthsubfields = $addiauthfield->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#addiauthsubfields) {
+                       OLDmodaddauthor($dbh,$oldbiblionumber,$addiauthsubfields[$subfieldcount]);
+               }
+       }
+       ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"bibliosubtitle.subtitle");
+       my @subtitlefields = $record->field($tagfield);
+       foreach my $subtitlefield (@subtitlefields) {
+               my @subtitlesubfields = $subtitlefield->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#subtitlesubfields) {
+                       OLDmodsubtitle($dbh,$oldbiblionumber,$subtitlesubfields[$subfieldcount]);
+               }
+       }
+       ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"bibliosubject.subject");
+       my @subj = $record->field($tagfield);
+       my @subjects;
+       foreach my $subject (@subj) {
+               my @subjsubfield = $subject->subfield($tagsubfield);
+               foreach my $subfieldcount (0..$#subjsubfield) {
+                       push @subjects,$subjsubfield[$subfieldcount];
+               }
+       }
+       OLDmodsubject($dbh,$oldbiblionumber,1,@subjects);
+       return 1;
+}
+
+sub NEWdelbiblio {
+       my ($dbh,$bibid)=@_;
+       my $biblio = &MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       &OLDdelbiblio($dbh,$biblio);
+       my $sth = $dbh->prepare("select biblioitemnumber from biblioitems where biblionumber=?");
+       $sth->execute($biblio);
+       while(my ($biblioitemnumber) = $sth->fetchrow) {
+               OLDdeletebiblioitem($dbh,$biblioitemnumber);
+       }
+       &MARCdelbiblio($dbh,$bibid,0);
+}
+
+
+sub NEWnewitem {
+       my ($dbh, $record,$bibid) = @_;
+       # add item in old-DB
+       my $item = &MARCmarc2koha($dbh,$record);
+       # needs old biblionumber and biblioitemnumber
+       $item->{'biblionumber'} = MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       my $sth = $dbh->prepare("select biblioitemnumber from biblioitems where biblionumber=?");
+       $sth->execute($item->{'biblionumber'});
+       ($item->{'biblioitemnumber'}) = $sth->fetchrow;
+       my ($itemnumber,$error) = &OLDnewitems($dbh,$item,$item->{barcode});
+       # add itemnumber to MARC::Record before adding the item.
+       my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+       &MARCkoha2marcOnefield($sth,$record,"items.itemnumber",$itemnumber);
+       # add the item
+       my $bib = &MARCadditem($dbh,$record,$item->{'biblionumber'});
+}
+
+sub NEWmoditem {
+       my ($dbh,$record,$bibid,$itemnumber,$delete) = @_;
+       &MARCmoditem($dbh,$record,$bibid,$itemnumber,$delete);
+       my $olditem = MARCmarc2koha($dbh,$record);
+       OLDmoditem($dbh,$olditem);
+}
+
+sub NEWdelitem {
+       my ($dbh,$bibid,$itemnumber)=@_;
+       my $biblio = &MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       &OLDdelitem($dbh,$itemnumber);
+       &MARCdelitem($dbh,$bibid,$itemnumber);
+}
 
 #
 #
-# OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD
-#
-#
+# OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD
+#
+#
 
-=head1 SYNOPSIS
-  OLDxxx related subs
-  all subs requires/use $dbh as 1st parameter.
-  those subs are used by the MARC-compliant version of koha : marc import, or marc management.
-
-  They all are the exact copy of 1.0/1.2 version of the sub
-  without the OLD. The OLDxxx is called by the original xxx sub.
-  the 1.4 xxx sub also builds MARC::Record an calls the MARCxxx
-  WARNING : there is 1 difference between initialxxx and OLDxxx :
-  the db header $dbh is always passed as parameter
-  to avoid over-DB connexion
+=item $biblionumber = OLDnewbiblio($dbh,$biblio);
 
-=head1 DESCRIPTION
+adds a record in biblio table. Datas are in the hash $biblio.
 
-=head2 $biblionumber = OLDnewbiblio($dbh,$biblio);
-  adds a record in biblio table. Datas are in the hash $biblio.
+=item $biblionumber = OLDmodbiblio($dbh,$biblio);
 
-=head2 $biblionumber = OLDmodbiblio($dbh,$biblio);
-  modify a record in biblio table. Datas are in the hash $biblio.
+modify a record in biblio table. Datas are in the hash $biblio.
 
-=head2 OLDmodsubtitle($dbh,$bibnum,$subtitle);
-  modify subtitles in bibliosubtitle table.
+=item OLDmodsubtitle($dbh,$bibnum,$subtitle);
 
-=head2 OLDmodaddauthor($dbh,$bibnum,$author);
-  adds or modify additional authors
-  NOTE :  Strange sub : seems to delete MANY and add only ONE author... maybe buggy ?
+modify subtitles in bibliosubtitle table.
 
-=head2 $errors = OLDmodsubject($dbh,$bibnum, $force, @subject);
-  modify/adds subjects
+=item OLDmodaddauthor($dbh,$bibnum,$author);
 
-=head2 OLDmodbibitem($dbh, $biblioitem);
-  modify a biblioitem
+adds or modify additional authors
+NOTE :  Strange sub : seems to delete MANY and add only ONE author... maybe buggy ?
 
-=head2 OLDmodnote($dbh,$bibitemnum,$note
-  modify a note for a biblioitem
+=item $errors = OLDmodsubject($dbh,$bibnum, $force, @subject);
 
-=head2 OLDnewbiblioitem($dbh,$biblioitem);
-  adds a biblioitem ($biblioitem is a hash with the values)
+modify/adds subjects
 
-=head2 OLDnewsubject($dbh,$bibnum);
-  adds a subject
-=head2 OLDnewsubtitle($dbh,$bibnum,$subtitle);
-  create a new subtitle
+=item OLDmodbibitem($dbh, $biblioitem);
 
-=head2 ($itemnumber,$errors)= OLDnewitems($dbh,$item,$barcode);
-  create a item. $item is a hash and $barcode the barcode.
+modify a biblioitem
 
-=head2 OLDmoditem($dbh,$item);
-  modify item
+=item OLDmodnote($dbh,$bibitemnum,$note
 
-=head2 OLDdelitem($dbh,$itemnum);
-  delete item
+modify a note for a biblioitem
 
-=head2 OLDdeletebiblioitem($dbh,$biblioitemnumber);
-  deletes a biblioitem
-  NOTE : not standard sub name. Should be OLDdelbiblioitem()
-=head2 OLDdelbiblio($dbh,$biblio);
-  delete a biblio
+=item OLDnewbiblioitem($dbh,$biblioitem);
 
-=head1 AUTHOR
+adds a biblioitem ($biblioitem is a hash with the values)
 
-Paul POULAIN paul.poulain@free.fr
+=item OLDnewsubject($dbh,$bibnum);
+
+adds a subject
+
+=item OLDnewsubtitle($dbh,$bibnum,$subtitle);
+
+create a new subtitle
+
+=item ($itemnumber,$errors)= OLDnewitems($dbh,$item,$barcode);
+
+create a item. $item is a hash and $barcode the barcode.
+
+=item OLDmoditem($dbh,$item);
+
+modify item
+
+=item OLDdelitem($dbh,$itemnum);
+
+delete item
+
+=item OLDdeletebiblioitem($dbh,$biblioitemnumber);
+
+deletes a biblioitem
+NOTE : not standard sub name. Should be OLDdelbiblioitem()
+
+=item OLDdelbiblio($dbh,$biblio);
+
+delete a biblio
 
 =cut
 
@@ -1025,27 +1198,12 @@ sub OLDnewbiblio {
   my $bibnum = $$data[0] + 1;
   my $series = 0;
 
-  $biblio->{'title'}       = $dbh->quote($biblio->{'title'});
-  $biblio->{'author'}      = $dbh->quote($biblio->{'author'});
-  $biblio->{'copyright'}   = $dbh->quote($biblio->{'copyright'});
-  $biblio->{'seriestitle'} = $dbh->quote($biblio->{'seriestitle'});
-  $biblio->{'notes'}      = $dbh->quote($biblio->{'notes'});
-  $biblio->{'abstract'}    = $dbh->quote($biblio->{'abstract'});
   if ($biblio->{'seriestitle'}) { $series = 1 };
-
   $sth->finish;
-  $query = "insert into biblio set
-biblionumber  = $bibnum,
-title         = $biblio->{'title'},
-author        = $biblio->{'author'},
-copyrightdate = $biblio->{'copyright'},
-serial        = $series,
-seriestitle   = $biblio->{'seriestitle'},
-notes         = $biblio->{'notes'},
-abstract      = $biblio->{'abstract'}";
-
+  $query = "insert into biblio set biblionumber  = ?, title         = ?, author        = ?, copyrightdate = ?,
+                                                                       serial        = ?, seriestitle   = ?, notes         = ?, abstract      = ?";
   $sth = $dbh->prepare($query);
-  $sth->execute;
+  $sth->execute($bibnum,$biblio->{'title'},$biblio->{'author'},$biblio->{'copyrightdate'},$series,$biblio->{'seriestitle'},$biblio->{'notes'},$biblio->{'abstract'});
 
   $sth->finish;
 #  $dbh->disconnect;
@@ -1053,50 +1211,27 @@ abstract      = $biblio->{'abstract'}";
 }
 
 sub OLDmodbiblio {
-    my ($dbh,$biblio) = @_;
-#  my $dbh   = C4Connect;
-    my $query;
-    my $sth;
-    
-    $biblio->{'title'}         = $dbh->quote($biblio->{'title'});
-    $biblio->{'author'}        = $dbh->quote($biblio->{'author'});
-    $biblio->{'abstract'}      = $dbh->quote($biblio->{'abstract'});
-    $biblio->{'copyrightdate'} = $dbh->quote($biblio->{'copyrightdate'});
-    $biblio->{'seriestitle'}   = $dbh->quote($biblio->{'serirestitle'});
-    $biblio->{'serial'}        = $dbh->quote($biblio->{'serial'});
-    $biblio->{'unititle'}      = $dbh->quote($biblio->{'unititle'});
-    $biblio->{'notes'}         = $dbh->quote($biblio->{'notes'});
-    
-    $query = "Update biblio set
-title         = $biblio->{'title'},
-author        = $biblio->{'author'},
-abstract      = $biblio->{'abstract'},
-copyrightdate = $biblio->{'copyrightdate'},
-seriestitle   = $biblio->{'seriestitle'},
-serial        = $biblio->{'serial'},
-unititle      = $biblio->{'unititle'},
-notes         = $biblio->{'notes'}
-where biblionumber = $biblio->{'biblionumber'}";
-    $sth   = $dbh->prepare($query);
-    
-    $sth->execute;
-    
-    $sth->finish;
-    $dbh->disconnect;
-    return($biblio->{'biblionumber'});
+       my ($dbh,$biblio) = @_;
+       #  my $dbh   = C4Connect;
+       my $query;
+       my $sth;
+
+       $query = "Update biblio set title         = ?, author        = ?, abstract      = ?, copyrightdate = ?,
+                                       seriestitle   = ?, serial        = ?, unititle      = ?, notes         = ? where biblionumber = ?";
+       $sth   = $dbh->prepare($query);
+       $sth->execute($biblio->{'title'},$biblio->{'author'},$biblio->{'abstract'},$biblio->{'copyrightdate'},
+                                               $biblio->{'seriestitle'},$biblio->{'serial'},$biblio->{'unititle'},$biblio->{'notes'},$biblio->{'biblionumber'});
+
+       $sth->finish;
+       return($biblio->{'biblionumber'});
 } # sub modbiblio
 
 sub OLDmodsubtitle {
-  my ($dbh,$bibnum, $subtitle) = @_;
-#  my $dbh   = C4Connect;
-  my $query = "update bibliosubtitle set
-subtitle = '$subtitle'
-where biblionumber = $bibnum";
-  my $sth   = $dbh->prepare($query);
-
-  $sth->execute;
-  $sth->finish;
-#  $dbh->disconnect;
+       my ($dbh,$bibnum, $subtitle) = @_;
+       my $query = "update bibliosubtitle set subtitle = ? where biblionumber = ?";
+       my $sth   = $dbh->prepare($query);
+       $sth->execute($subtitle,$bibnum);
+       $sth->finish;
 } # sub modsubtitle
 
 
@@ -1111,79 +1246,67 @@ sub OLDmodaddauthor {
 
     if ($author ne '') {
         $query = "Insert into additionalauthors set
-                        author       = '$author',
-                        biblionumber = '$bibnum'";
+                        author       = ?,
+                        biblionumber = ?";
         $sth   = $dbh->prepare($query);
 
-        $sth->execute;
+        $sth->execute($author,$bibnum);
 
         $sth->finish;
     } # if
-
-  $dbh->disconnect;
 } # sub modaddauthor
 
 
 sub OLDmodsubject {
-    my ($dbh,$bibnum, $force, @subject) = @_;
-#  my $dbh   = C4Connect;
-    my $count = @subject;
-    my $error;
-    for (my $i = 0; $i < $count; $i++) {
-       $subject[$i] =~ s/^ //g;
-       $subject[$i] =~ s/ $//g;
-       my $query = "select * from catalogueentry
-                       where entrytype = 's'
-                               and catalogueentry = '$subject[$i]'";
-       my $sth   = $dbh->prepare($query);
-       $sth->execute;
-       
-       if (my $data = $sth->fetchrow_hashref) {
-       } else {
-           if ($force eq $subject[$i]) {
-               # subject not in aut, chosen to force anway
-               # so insert into cataloguentry so its in auth file
-               $query = "Insert into catalogueentry
-                               (entrytype,catalogueentry)
-                           values ('s','$subject[$i]')";
-        my $sth2 = $dbh->prepare($query);
-
-        $sth2->execute;
-        $sth2->finish;
-      } else {
-        $error = "$subject[$i]\n does not exist in the subject authority file";
-        $query = "Select * from catalogueentry
-                           where entrytype = 's'
-                           and (catalogueentry like '$subject[$i] %'
-                                or catalogueentry like '% $subject[$i] %'
-                                or catalogueentry like '% $subject[$i]')";
-        my $sth2 = $dbh->prepare($query);
-
-        $sth2->execute;
-        while (my $data = $sth2->fetchrow_hashref) {
-          $error = $error."<br>$data->{'catalogueentry'}";
-        } # while
-        $sth2->finish;
-      } # else
-    } # else
-    $sth->finish;
-  } # else
-  if ($error eq '') {
-    my $query = "Delete from bibliosubject where biblionumber = $bibnum";
-    my $sth   = $dbh->prepare($query);
-    $sth->execute;
-    $sth->finish;
-    for (my $i = 0; $i < $count; $i++) {
-      $sth = $dbh->prepare("Insert into bibliosubject
-                           values ('$subject[$i]', $bibnum)");
-
-      $sth->execute;
-      $sth->finish;
-    } # for
-  } # if
-
-#  $dbh->disconnect;
-  return($error);
+       my ($dbh,$bibnum, $force, @subject) = @_;
+       #  my $dbh   = C4Connect;
+       my $count = @subject;
+       my $error;
+       for (my $i = 0; $i < $count; $i++) {
+               $subject[$i] =~ s/^ //g;
+               $subject[$i] =~ s/ $//g;
+               my $query = "select * from catalogueentry where entrytype = 's' and catalogueentry = ?";
+               my $sth   = $dbh->prepare($query);
+               $sth->execute($subject[$i]);
+
+               if (my $data = $sth->fetchrow_hashref) {
+               } else {
+                       if ($force eq $subject[$i] || $force == 1) {
+                               # subject not in aut, chosen to force anway
+                               # so insert into cataloguentry so its in auth file
+                               $query = "Insert into catalogueentry (entrytype,catalogueentry) values ('s',?)";
+                               my $sth2 = $dbh->prepare($query);
+
+                               $sth2->execute($subject[$i]);
+                               $sth2->finish;
+                       } else {
+                               $error = "$subject[$i]\n does not exist in the subject authority file";
+                               $query = "Select * from catalogueentry where entrytype = 's' and (catalogueentry like ?
+                                                                       or catalogueentry like ? or catalogueentry like ?)";
+                               my $sth2 = $dbh->prepare($query);
+                               $sth2->execute("$subject[$i] %","% $subject[$i] %","% $subject[$i]");
+                               while (my $data = $sth2->fetchrow_hashref) {
+                                       $error .= "<br>$data->{'catalogueentry'}";
+                               } # while
+                               $sth2->finish;
+                       } # else
+               } # else
+               $sth->finish;
+       } # else
+       if ($error eq '') {
+               my $query = "Delete from bibliosubject where biblionumber = ?";
+               my $sth   = $dbh->prepare($query);
+               $sth->execute($bibnum);
+               $sth->finish;
+               $sth = $dbh->prepare("Insert into bibliosubject values (?,?)");
+               foreach $query (@subject) {
+                       $sth->execute($query,$bibnum);
+               } # foreach
+               $sth->finish;
+       } # if
+
+       #  $dbh->disconnect;
+       return($error);
 } # sub modsubject
 
 sub OLDmodbibitem {
@@ -1202,7 +1325,7 @@ sub OLDmodbibitem {
     $biblioitem->{'illus'}           = $dbh->quote($biblioitem->{'illus'});
     $biblioitem->{'pages'}           = $dbh->quote($biblioitem->{'pages'});
     $biblioitem->{'volumeddesc'}     = $dbh->quote($biblioitem->{'volumeddesc'});
-    $biblioitem->{'notes'}           = $dbh->quote($biblioitem->{'notes'});
+    $biblioitem->{'bnotes'}          = $dbh->quote($biblioitem->{'bnotes'});
     $biblioitem->{'size'}            = $dbh->quote($biblioitem->{'size'});
     $biblioitem->{'place'}           = $dbh->quote($biblioitem->{'place'});
 
@@ -1218,13 +1341,15 @@ subclass        = $biblioitem->{'subclass'},
 illus           = $biblioitem->{'illus'},
 pages           = $biblioitem->{'pages'},
 volumeddesc     = $biblioitem->{'volumeddesc'},
-notes          = $biblioitem->{'notes'},
+notes          = $biblioitem->{'bnotes'},
 size           = $biblioitem->{'size'},
 place          = $biblioitem->{'place'}
 where biblioitemnumber = $biblioitem->{'biblioitemnumber'}";
 
-    $dbh->do($query);
-
+$dbh->do($query);
+if ($dbh->errstr) {
+       warn "$query";
+}
 #    $dbh->disconnect;
 } # sub modbibitem
 
@@ -1240,159 +1365,132 @@ sub OLDmodnote {
 }
 
 sub OLDnewbiblioitem {
-    my ($dbh,$biblioitem) = @_;
-#  my $dbh   = C4Connect;
-    my $query = "Select max(biblioitemnumber) from biblioitems";
-    my $sth   = $dbh->prepare($query);
-    my $data;
-    my $bibitemnum;
-    
-    $biblioitem->{'volume'}          = $dbh->quote($biblioitem->{'volume'});
-    $biblioitem->{'number'}       = $dbh->quote($biblioitem->{'number'});
-    $biblioitem->{'classification'}  = $dbh->quote($biblioitem->{'classification'});
-    $biblioitem->{'itemtype'}        = $dbh->quote($biblioitem->{'itemtype'});
-    $biblioitem->{'url'}             = $dbh->quote($biblioitem->{'url'});
-    $biblioitem->{'isbn'}            = $dbh->quote($biblioitem->{'isbn'});
-    $biblioitem->{'issn'}            = $dbh->quote($biblioitem->{'issn'});
-    $biblioitem->{'dewey'}           = $dbh->quote($biblioitem->{'dewey'});
-    $biblioitem->{'subclass'}        = $dbh->quote($biblioitem->{'subclass'});
-    $biblioitem->{'publicationyear'} = $dbh->quote($biblioitem->{'publicationyear'});
-    $biblioitem->{'publishercode'}   = $dbh->quote($biblioitem->{'publishercode'});
-    $biblioitem->{'volumedate'}      = $dbh->quote($biblioitem->{'volumedate'});
-    $biblioitem->{'volumeddesc'}     = $dbh->quote($biblioitem->{'volumeddesc'});  $biblioitem->{'illus'}            = $dbh->quote($biblioitem->{'illus'});
-    $biblioitem->{'illus'}        = $dbh->quote($biblioitem->{'illus'});
-    $biblioitem->{'pages'}           = $dbh->quote($biblioitem->{'pages'});
-    $biblioitem->{'notes'}           = $dbh->quote($biblioitem->{'notes'});
-    $biblioitem->{'size'}            = $dbh->quote($biblioitem->{'size'});
-    $biblioitem->{'place'}           = $dbh->quote($biblioitem->{'place'});
-    $biblioitem->{'lccn'}            = $dbh->quote($biblioitem->{'lccn'});
-    $biblioitem->{'marc'}            = $dbh->quote($biblioitem->{'marc'});
-  
-    $sth->execute;
-    $data       = $sth->fetchrow_arrayref;
-    $bibitemnum = $$data[0] + 1;
-
-    $sth->finish;
+       my ($dbh,$biblioitem) = @_;
+       #  my $dbh   = C4Connect;
+       my $query = "Select max(biblioitemnumber) from biblioitems";
+       my $sth   = $dbh->prepare($query);
+       my $data;
+       my $bibitemnum;
 
-    $query = "insert into biblioitems set
-                        biblioitemnumber = $bibitemnum,
-                        biblionumber    = $biblioitem->{'biblionumber'},
-                        volume          = $biblioitem->{'volume'},
-                        number          = $biblioitem->{'number'},
-                        classification   = $biblioitem->{'classification'},
-                        itemtype         = $biblioitem->{'itemtype'},
-                        url              = $biblioitem->{'url'},
-                        isbn            = $biblioitem->{'isbn'},
-                        issn            = $biblioitem->{'issn'},
-                        dewey           = $biblioitem->{'dewey'},
-                        subclass        = $biblioitem->{'subclass'},
-                        publicationyear         = $biblioitem->{'publicationyear'},
-                        publishercode   = $biblioitem->{'publishercode'},
-                        volumedate      = $biblioitem->{'volumedate'},
-                        volumeddesc     = $biblioitem->{'volumeddesc'},
-                        illus           = $biblioitem->{'illus'},
-                        pages           = $biblioitem->{'pages'},
-                        notes           = $biblioitem->{'notes'},
-                        size            = $biblioitem->{'size'},
-                        lccn            = $biblioitem->{'lccn'},
-                        marc            = $biblioitem->{'marc'},
-                        place           = $biblioitem->{'place'}";
-
-    $sth = $dbh->prepare($query);
-    $sth->execute;
-    $sth->finish;
-#    $dbh->disconnect;
-    return($bibitemnum);
+       $sth->execute;
+       $data       = $sth->fetchrow_arrayref;
+       $bibitemnum = $$data[0] + 1;
+
+       $sth->finish;
+
+       $sth = $dbh->prepare("insert into biblioitems set
+                                                                       biblioitemnumber = ?,           biblionumber     = ?,
+                                                                       volume           = ?,                   number           = ?,
+                                                                       classification  = ?,                    itemtype         = ?,
+                                                                       url              = ?,                           isbn             = ?,
+                                                                       issn             = ?,                           dewey            = ?,
+                                                                       subclass         = ?,                           publicationyear  = ?,
+                                                                       publishercode    = ?,           volumedate       = ?,
+                                                                       volumeddesc      = ?,           illus            = ?,
+                                                                       pages            = ?,                           notes            = ?,
+                                                                       size             = ?,                           lccn             = ?,
+                                                                       marc             = ?,                           place            = ?");
+       $sth->execute($bibitemnum,                                                      $biblioitem->{'biblionumber'},
+                                               $biblioitem->{'volume'},                        $biblioitem->{'number'},
+                                               $biblioitem->{'classification'},                $biblioitem->{'itemtype'},
+                                               $biblioitem->{'url'},                                   $biblioitem->{'isbn'},
+                                               $biblioitem->{'issn'},                          $biblioitem->{'dewey'},
+                                               $biblioitem->{'subclass'},                      $biblioitem->{'publicationyear'},
+                                               $biblioitem->{'publishercode'}, $biblioitem->{'volumedate'},
+                                               $biblioitem->{'volumeddesc'},           $biblioitem->{'illus'},
+                                               $biblioitem->{'pages'},                         $biblioitem->{'bnotes'},
+                                               $biblioitem->{'size'},                          $biblioitem->{'lccn'},
+                                               $biblioitem->{'marc'},                          $biblioitem->{'place'});
+       $sth->finish;
+       #    $dbh->disconnect;
+       return($bibitemnum);
 }
 
 sub OLDnewsubject {
   my ($dbh,$bibnum)=@_;
-#  my $dbh=C4Connect;
-  my $query="insert into bibliosubject (biblionumber) values
-  ($bibnum)";
+  my $query="insert into bibliosubject (biblionumber) values ($bibnum)";
   my $sth=$dbh->prepare($query);
-#  print $query;
   $sth->execute;
   $sth->finish;
-#  $dbh->disconnect;
 }
 
 sub OLDnewsubtitle {
     my ($dbh,$bibnum, $subtitle) = @_;
-#  my $dbh   = C4Connect;
-    $subtitle = $dbh->quote($subtitle);
-    my $query = "insert into bibliosubtitle set
-                            biblionumber = $bibnum,
-                            subtitle = $subtitle";
+    my $query = "insert into bibliosubtitle set biblionumber = ?, subtitle = ?";
     my $sth   = $dbh->prepare($query);
-
-    $sth->execute;
-
+    $sth->execute($bibnum,$subtitle);
     $sth->finish;
-#  $dbh->disconnect;
 }
 
 
 sub OLDnewitems {
-  my ($dbh,$item, $barcode) = @_;
-#  my $dbh   = C4Connect;
-  my $query = "Select max(itemnumber) from items";
-  my $sth   = $dbh->prepare($query);
-  my $data;
-  my $itemnumber;
-  my $error = "";
-
-  $sth->execute;
-  $data       = $sth->fetchrow_hashref;
-  $itemnumber = $data->{'max(itemnumber)'} + 1;
-  $sth->finish;
-  
-  $item->{'booksellerid'}     = $dbh->quote($item->{'booksellerid'});
-  $item->{'homebranch'}       = $dbh->quote($item->{'homebranch'});
-  $item->{'price'}            = $dbh->quote($item->{'price'});
-  $item->{'replacementprice'} = $dbh->quote($item->{'replacementprice'});
-  $item->{'itemnotes'}        = $dbh->quote($item->{'itemnotes'});
-
-#  foreach my $barcode (@barcodes) {
-#    $barcode = uc($barcode);
-  $barcode = $dbh->quote($barcode);
-  $query   = "Insert into items set
-                            itemnumber           = $itemnumber,
-                            biblionumber         = $item->{'biblionumber'},
-                            biblioitemnumber     = $item->{'biblioitemnumber'},
-                            barcode              = $barcode,
-                            booksellerid         = $item->{'booksellerid'},
-                            dateaccessioned      = NOW(),
-                            homebranch           = $item->{'homebranch'},
-                            holdingbranch        = $item->{'homebranch'},
-                            price                = $item->{'price'},
-                            replacementprice     = $item->{'replacementprice'},
-                            replacementpricedate = NOW(),
-                            itemnotes            = $item->{'itemnotes'}";
-  if ($item->{'loan'}) {
-      $query .= ",notforloan           = $item->{'loan'}";
-  } # if
+       my ($dbh,$item, $barcode) = @_;
+       #  my $dbh   = C4Connect;
+       my $query = "Select max(itemnumber) from items";
+       my $sth   = $dbh->prepare($query);
+       my $data;
+       my $itemnumber;
+       my $error = "";
 
-  $sth = $dbh->prepare($query);
-  $sth->execute;
-  if (defined $sth->errstr) {
-      $error .= $sth->errstr;
-  }
-  $sth->finish;
-#  $itemnumber++;
-#  $dbh->disconnect;
-  return($itemnumber,$error);
+       $sth->execute;
+       $data       = $sth->fetchrow_hashref;
+       $itemnumber = $data->{'max(itemnumber)'} + 1;
+       $sth->finish;
+# FIXME the "notforloan" field seems to be named "loan" in some places. workaround bugfix.
+       if ($item->{'loan'}) {
+               $item->{'notforloan'} = $item->{'loan'};
+       }
+# if dateaccessioned is provided, use it. Otherwise, set to NOW()
+       if ($item->{'dateaccessioned'}) {
+               $sth=$dbh->prepare("Insert into items set
+                                                       itemnumber           = ?,                               biblionumber         = ?,
+                                                       biblioitemnumber     = ?,                               barcode              = ?,
+                                                       booksellerid         = ?,                                       dateaccessioned      = ?,
+                                                       homebranch           = ?,                               holdingbranch        = ?,
+                                                       price                = ?,                                               replacementprice     = ?,
+                                                       replacementpricedate = NOW(),   itemnotes            = ?,
+                                                       notforloan = ?
+                                                       ");
+               $sth->execute($itemnumber,      $item->{'biblionumber'},
+                                                               $item->{'biblioitemnumber'},$barcode,
+                                                               $item->{'booksellerid'},$item->{'dateaccessioned'},
+                                                               $item->{'homebranch'},$item->{'holdingbranch'},
+                                                               $item->{'price'},$item->{'replacementprice'},
+                                                               $item->{'itemnotes'},$item->{'notforloan'});
+       } else {
+               $sth=$dbh->prepare("Insert into items set
+                                                       itemnumber           = ?,                               biblionumber         = ?,
+                                                       biblioitemnumber     = ?,                               barcode              = ?,
+                                                       booksellerid         = ?,                                       dateaccessioned      = NOW(),
+                                                       homebranch           = ?,                               holdingbranch        = ?,
+                                                       price                = ?,                                               replacementprice     = ?,
+                                                       replacementpricedate = NOW(),   itemnotes            = ?,
+                                                       notforloan = ?
+                                                       ");
+               $sth->execute($itemnumber,      $item->{'biblionumber'},
+                                                               $item->{'biblioitemnumber'},$barcode,
+                                                               $item->{'booksellerid'},
+                                                               $item->{'homebranch'},$item->{'holdingbranch'},
+                                                               $item->{'price'},$item->{'replacementprice'},
+                                                               $item->{'itemnotes'},$item->{'notforloan'});
+       }
+       if (defined $sth->errstr) {
+               $error .= $sth->errstr;
+       }
+       $sth->finish;
+       return($itemnumber,$error);
 }
 
 sub OLDmoditem {
     my ($dbh,$item) = @_;
 #  my ($dbh,$loan,$itemnum,$bibitemnum,$barcode,$notes,$homebranch,$lost,$wthdrawn,$replacement)=@_;
 #  my $dbh=C4Connect;
-  my $query="update items set biblioitemnumber=$item->{'bibitemnum'},
-                              barcode='$item->{'barcode'}',itemnotes='$item->{'notes'}'
+$item->{'itemnum'}=$item->{'itemnumber'} unless $item->{'itemnum'};
+  my $query="update items set  barcode='$item->{'barcode'}',itemnotes='$item->{'notes'}'
                           where itemnumber=$item->{'itemnum'}";
   if ($item->{'barcode'} eq ''){
-    $query="update items set biblioitemnumber=$item->{'bibitemnum'},notforloan=$item->{'loan'} where itemnumber=$item->{'itemnum'}";
+       $item->{'notforloan'}=0 unless $item->{'notforloan'};
+    $query="update items set notforloan=$item->{'notforloan'} where itemnumber=$item->{'itemnum'}";
   }
   if ($item->{'lost'} ne ''){
     $query="update items set biblioitemnumber=$item->{'bibitemnum'},
@@ -1400,13 +1498,12 @@ sub OLDmoditem {
                              itemnotes='$item->{'notes'}',
                              homebranch='$item->{'homebranch'}',
                              itemlost='$item->{'lost'}',
-                             wthdrawn='$item->{'wthdrawn'}' 
+                             wthdrawn='$item->{'wthdrawn'}'
                           where itemnumber=$item->{'itemnum'}";
   }
   if ($item->{'replacement'} ne ''){
     $query=~ s/ where/,replacementprice='$item->{'replacement'}' where/;
   }
-
   my $sth=$dbh->prepare($query);
   $sth->execute;
   $sth->finish;
@@ -1419,13 +1516,13 @@ sub OLDdelitem{
   my $query="select * from items where itemnumber=$itemnum";
   my $sth=$dbh->prepare($query);
   $sth->execute;
-  my @data=$sth->fetchrow_array;
+  my $data=$sth->fetchrow_hashref;
   $sth->finish;
-  $query="Insert into deleteditems values (";
-  foreach my $temp (@data){
-    $query=$query."'$temp',";
+  $query="Insert into deleteditems set ";
+  foreach my $temp (keys %$data){
+    $query .= "$temp = ".$dbh->quote($data->{$temp}).",";
   }
-  $query=~ s/\,$/\)/;
+  $query=~ s/\,$//;
 #  print $query;
   $sth=$dbh->prepare($query);
   $sth->execute;
@@ -1443,20 +1540,19 @@ sub OLDdeletebiblioitem {
     my $query = "Select * from biblioitems
 where biblioitemnumber = $biblioitemnumber";
     my $sth   = $dbh->prepare($query);
-    my @results;
+    my $results;
 
     $sth->execute;
-  
-    if (@results = $sth->fetchrow_array) {
-        $query = "Insert into deletedbiblioitems values (";
-        foreach my $value (@results) {
-            $value  = $dbh->quote($value);
-            $query .= "$value,";
-        } # foreach
-
-        $query =~ s/\,$/\)/;
-        $dbh->do($query);
 
+    if ($results = $sth->fetchrow_hashref) {
+       $sth->finish;
+        $sth=$dbh->prepare("Insert into deletedbiblioitems (biblioitemnumber, biblionumber, volume, number, classification, itemtype,
+                                       isbn, issn ,dewey ,subclass ,publicationyear ,publishercode ,volumedate ,volumeddesc ,timestamp ,illus ,
+                                       pages ,notes ,size ,url ,lccn ) values(?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?)");
+
+        $sth->execute($results->{biblioitemnumber}, $results->{biblionumber}, $results->{volume}, $results->{number}, $results->{classification}, $results->{itemtype},
+                                       $results->{isbn}, $results->{issn} ,$results->{dewey} ,$results->{subclass} ,$results->{publicationyear} ,$results->{publishercode} ,$results->{volumedate} ,$results->{volumeddesc} ,$results->{timestamp} ,$results->{illus} ,
+                                       $results->{pages} ,$results->{notes} ,$results->{size} ,$results->{url} ,$results->{lccn} );
         $query = "Delete from biblioitems
                         where biblioitemnumber = $biblioitemnumber";
         $dbh->do($query);
@@ -1466,6 +1562,7 @@ where biblioitemnumber = $biblioitemnumber";
     $query = "Select * from items where biblioitemnumber = $biblioitemnumber";
     $sth   = $dbh->prepare($query);
     $sth->execute;
+    my @results;
     while (@results = $sth->fetchrow_array) {
        $query = "Insert into deleteditems values (";
        foreach my $value (@results) {
@@ -1483,19 +1580,19 @@ where biblioitemnumber = $biblioitemnumber";
 
 sub OLDdelbiblio{
   my ($dbh,$biblio)=@_;
-#  my $dbh=C4Connect;
   my $query="select * from biblio where biblionumber=$biblio";
   my $sth=$dbh->prepare($query);
   $sth->execute;
   if (my @data=$sth->fetchrow_array){
     $sth->finish;
+# FIXME => replace insert values by insert (field) values ($value)
     $query="Insert into deletedbiblio values (";
     foreach my $temp (@data){
       $temp=~ s/\'/\\\'/g;
-      $query=$query."'$temp',";
+      $query .= "'$temp',";
     }
-    $query=~ s/\,$/\)/;
-#   print $query;
+    #replacing the last , by ",?)"
+    $query=~ s/\,$/\,\?\)/;
     $sth=$dbh->prepare($query);
     $sth->execute;
     $sth->finish;
@@ -1505,7 +1602,6 @@ sub OLDdelbiblio{
     $sth->finish;
   }
   $sth->finish;
-#  $dbh->disconnect;
 }
 
 #
@@ -1516,38 +1612,68 @@ sub OLDdelbiblio{
 
 sub itemcount{
   my ($biblio)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select count(*) from items where biblionumber=$biblio";
 #  print $query;
   my $sth=$dbh->prepare($query);
   $sth->execute;
   my $data=$sth->fetchrow_hashref;
   $sth->finish;
-  $dbh->disconnect;
   return($data->{'count(*)'});
 }
 
+=item getorder
+
+  ($order, $ordernumber) = &getorder($biblioitemnumber, $biblionumber);
+
+Looks up the order with the given biblionumber and biblioitemnumber.
+
+Returns a two-element array. C<$ordernumber> is the order number.
+C<$order> is a reference-to-hash describing the order; its keys are
+fields from the biblio, biblioitems, aqorders, and aqorderbreakdown
+tables of the Koha database.
+
+=cut
+#'
+# FIXME - This is effectively identical to &C4::Catalogue::getorder.
+# Pick one and stick with it.
 sub getorder{
   my ($bi,$bib)=@_;
-  my $dbh=C4Connect;
-  my $query="Select ordernumber 
-       from aqorders 
+  my $dbh = C4::Context->dbh;
+  my $query="Select ordernumber
+       from aqorders
        where biblionumber=? and biblioitemnumber=?";
   my $sth=$dbh->prepare($query);
   $sth->execute($bib,$bi);
+  # FIXME - Use fetchrow_array(), since we're only interested in the one
+  # value.
   my $ordnum=$sth->fetchrow_hashref;
   $sth->finish;
   my $order=getsingleorder($ordnum->{'ordernumber'});
-  $dbh->disconnect;
 #  print $query;
   return ($order,$ordnum->{'ordernumber'});
 }
 
+=item getsingleorder
+
+  $order = &getsingleorder($ordernumber);
+
+Looks up an order by order number.
+
+Returns a reference-to-hash describing the order. The keys of
+C<$order> are fields from the biblio, biblioitems, aqorders, and
+aqorderbreakdown tables of the Koha database.
+
+=cut
+#'
+# FIXME - This is effectively identical to
+# &C4::Catalogue::getsingleorder.
+# Pick one and stick with it.
 sub getsingleorder {
   my ($ordnum)=@_;
-  my $dbh=C4Connect;
-  my $query="Select * from biblio,biblioitems,aqorders,aqorderbreakdown 
-  where aqorders.ordernumber=? 
+  my $dbh = C4::Context->dbh;
+  my $query="Select * from biblio,biblioitems,aqorders,aqorderbreakdown
+  where aqorders.ordernumber=?
   and biblio.biblionumber=aqorders.biblionumber and
   biblioitems.biblioitemnumber=aqorders.biblioitemnumber and
   aqorders.ordernumber=aqorderbreakdown.ordernumber";
@@ -1555,128 +1681,161 @@ sub getsingleorder {
   $sth->execute($ordnum);
   my $data=$sth->fetchrow_hashref;
   $sth->finish;
-  $dbh->disconnect;
   return($data);
 }
 
 sub newbiblio {
-  my ($biblio) = @_;
-  my $dbh    = &C4Connect;
-  my $bibnum=OLDnewbiblio($dbh,$biblio);
-# TODO : MARC add
-  $dbh->disconnect;
-  return($bibnum);
+       my ($biblio) = @_;
+       my $dbh    = C4::Context->dbh;
+       my $bibnum=OLDnewbiblio($dbh,$biblio);
+       # finds new (MARC bibid
+#      my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$bibnum);
+       my $record = &MARCkoha2marcBiblio($dbh,$bibnum);
+       MARCaddbiblio($dbh,$record,$bibnum);
+       return($bibnum);
 }
 
+=item modbiblio
+
+  $biblionumber = &modbiblio($biblio);
+
+Update a biblio record.
+
+C<$biblio> is a reference-to-hash whose keys are the fields in the
+biblio table in the Koha database. All fields must be present, not
+just the ones you wish to change.
+
+C<&modbiblio> updates the record defined by
+C<$biblio-E<gt>{biblionumber}> with the values in C<$biblio>.
+
+C<&modbiblio> returns C<$biblio-E<gt>{biblionumber}> whether it was
+successful or not.
+
+=cut
+
 sub modbiblio {
-  my ($biblio) = @_;
-  my $dbh  = C4Connect;
-  my $biblionumber=OLDmodbiblio($dbh,$biblio);
-  $dbh->disconnect;
-  return($biblionumber);
+       my ($biblio) = @_;
+       my $dbh  = C4::Context->dbh;
+       my $biblionumber=OLDmodbiblio($dbh,$biblio);
+       my $record = MARCkoha2marcBiblio($dbh,$biblionumber,$biblionumber);
+       # finds new (MARC bibid
+       my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$biblionumber);
+       MARCmodbiblio($dbh,$bibid,$record,0);
+       return($biblionumber);
 } # sub modbiblio
 
+=item modsubtitle
+
+  &modsubtitle($biblionumber, $subtitle);
+
+Sets the subtitle of a book.
+
+C<$biblionumber> is the biblionumber of the book to modify.
+
+C<$subtitle> is the new subtitle.
+
+=cut
+
 sub modsubtitle {
   my ($bibnum, $subtitle) = @_;
-  my $dbh   = C4Connect;
+  my $dbh   = C4::Context->dbh;
   &OLDmodsubtitle($dbh,$bibnum,$subtitle);
-  $dbh->disconnect;
 } # sub modsubtitle
 
+=item modaddauthor
+
+  &modaddauthor($biblionumber, $author);
+
+Replaces all additional authors for the book with biblio number
+C<$biblionumber> with C<$author>. If C<$author> is the empty string,
+C<&modaddauthor> deletes all additional authors.
+
+=cut
 
 sub modaddauthor {
     my ($bibnum, $author) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     &OLDmodaddauthor($dbh,$bibnum,$author);
-    $dbh->disconnect;
 } # sub modaddauthor
 
+=item modsubject
+
+  $error = &modsubject($biblionumber, $force, @subjects);
+
+$force - a subject to force
+
+$error - Error message, or undef if successful.
+
+=cut
 
 sub modsubject {
   my ($bibnum, $force, @subject) = @_;
-  my $dbh   = C4Connect;
+  my $dbh   = C4::Context->dbh;
   my $error= &OLDmodsubject($dbh,$bibnum,$force, @subject);
   return($error);
 } # sub modsubject
 
 sub modbibitem {
     my ($biblioitem) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     &OLDmodbibitem($dbh,$biblioitem);
-    my $MARCbibitem = MARCkoha2marcBiblio($dbh,$biblioitem);
-    &MARCmodbiblio($dbh,$biblioitem->{biblionumber},0,$MARCbibitem);
-    $dbh->disconnect;
 } # sub modbibitem
 
 sub modnote {
   my ($bibitemnum,$note)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   &OLDmodnote($dbh,$bibitemnum,$note);
-  $dbh->disconnect;
 }
 
 sub newbiblioitem {
-  my ($biblioitem) = @_;
-  my $dbh   = C4Connect;
-  my $bibitemnum = &OLDnewbiblioitem($dbh,$biblioitem);
-#  print STDERR "bibitemnum : $bibitemnum\n";
-  my $MARCbiblio= MARCkoha2marcBiblio($dbh,$biblioitem->{biblionumber},$bibitemnum);
-#  print STDERR $MARCbiblio->as_formatted();
-  &MARCaddbiblio($dbh,$MARCbiblio,$biblioitem->{biblionumber});
-  return($bibitemnum);
+       my ($biblioitem) = @_;
+       my $dbh   = C4::Context->dbh;
+       my $bibitemnum = &OLDnewbiblioitem($dbh,$biblioitem);
+       my $MARCbiblio= MARCkoha2marcBiblio($dbh,0,$bibitemnum); # the 0 means "do NOT retrieve biblio, only biblioitem, in the MARC record
+       my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$biblioitem->{biblionumber});
+       &MARCaddbiblio($dbh,$MARCbiblio,$biblioitem->{biblionumber},$bibid);
+       return($bibitemnum);
 }
 
 sub newsubject {
   my ($bibnum)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   &OLDnewsubject($dbh,$bibnum);
-  $dbh->disconnect;
 }
 
 sub newsubtitle {
     my ($bibnum, $subtitle) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     &OLDnewsubtitle($dbh,$bibnum,$subtitle);
-  $dbh->disconnect;
 }
 
 sub newitems {
   my ($item, @barcodes) = @_;
-  my $dbh   = C4Connect;
+  my $dbh   = C4::Context->dbh;
   my $errors;
   my $itemnumber;
   my $error;
   foreach my $barcode (@barcodes) {
       ($itemnumber,$error)=&OLDnewitems($dbh,$item,uc($barcode));
       $errors .=$error;
-#      print STDERR "biblionumber : $item->{biblionumber} / MARCbibid : $MARCbibid / itemnumber : $itemnumber\n";
       my $MARCitem = &MARCkoha2marcItem($dbh,$item->{biblionumber},$itemnumber);
-#      print STDERR "MARCitem ".$MARCitem->as_formatted()."\n";
       &MARCadditem($dbh,$MARCitem,$item->{biblionumber});
-#      print STDERR "MARCmodbiblio called\n";
   }
-  $dbh->disconnect;
   return($errors);
 }
 
 sub moditem {
     my ($item) = @_;
-#  my ($loan,$itemnum,$bibitemnum,$barcode,$notes,$homebranch,$lost,$wthdrawn,$replacement)=@_;
-    my $dbh=C4Connect;
+    my $dbh = C4::Context->dbh;
     &OLDmoditem($dbh,$item);
-    warn "biblionumber : $item->{'biblionumber'} / $item->{'itemnum'}\n";
     my $MARCitem = &MARCkoha2marcItem($dbh,$item->{'biblionumber'},$item->{'itemnum'});
-    warn "before MARCmoditem : $item->{biblionumber}, $item->{'itemnum'}\n";
-    warn $MARCitem->as_formatted();
-#      print STDERR "MARCitem ".$MARCitem->as_formatted()."\n";
     my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$item->{biblionumber});
     &MARCmoditem($dbh,$MARCitem,$bibid,$item->{itemnum},0);
-    $dbh->disconnect;
 }
 
 sub checkitems{
   my ($count,@barcodes)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $error;
   for (my $i=0;$i<$count;$i++){
     $barcodes[$i]=uc $barcodes[$i];
@@ -1688,88 +1847,83 @@ sub checkitems{
     }
     $sth->finish;
   }
-  $dbh->disconnect;
   return($error);
 }
 
 sub countitems{
   my ($bibitemnum)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   my $query="Select count(*) from items where biblioitemnumber='$bibitemnum'";
   my $sth=$dbh->prepare($query);
   $sth->execute;
   my $data=$sth->fetchrow_hashref;
   $sth->finish;
-  $dbh->disconnect;
   return($data->{'count(*)'});
 }
 
 sub delitem{
   my ($itemnum)=@_;
-  my $dbh=C4Connect;
+  my $dbh = C4::Context->dbh;
   &OLDdelitem($dbh,$itemnum);
-  $dbh->disconnect;
 }
 
 sub deletebiblioitem {
     my ($biblioitemnumber) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     &OLDdeletebiblioitem($dbh,$biblioitemnumber);
-    $dbh->disconnect;
 } # sub deletebiblioitem
 
 
 sub delbiblio {
-  my ($biblio)=@_;
-  my $dbh=C4Connect;
-  &OLDdelbiblio($dbh,$biblio);
-  $dbh->disconnect;
+       my ($biblio)=@_;
+       my $dbh = C4::Context->dbh;
+       &OLDdelbiblio($dbh,$biblio);
+       my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$biblio);
+       &MARCdelbiblio($dbh,$bibid,0);
 }
 
 sub getitemtypes {
-  my $dbh   = C4Connect;
-  my $query = "select * from itemtypes";
+  my $dbh   = C4::Context->dbh;
+  my $query = "select * from itemtypes order by description";
   my $sth   = $dbh->prepare($query);
     # || die "Cannot prepare $query" . $dbh->errstr;
   my $count = 0;
   my @results;
-  
+
   $sth->execute;
     # || die "Cannot execute $query\n" . $sth->errstr;
   while (my $data = $sth->fetchrow_hashref) {
     $results[$count] = $data;
     $count++;
   } # while
-  
+
   $sth->finish;
-  $dbh->disconnect;
   return($count, @results);
 } # sub getitemtypes
 
 sub getbiblio {
     my ($biblionumber) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     my $query = "Select * from biblio where biblionumber = $biblionumber";
     my $sth   = $dbh->prepare($query);
       # || die "Cannot prepare $query\n" . $dbh->errstr;
     my $count = 0;
     my @results;
-    
+
     $sth->execute;
       # || die "Cannot execute $query\n" . $sth->errstr;
     while (my $data = $sth->fetchrow_hashref) {
       $results[$count] = $data;
       $count++;
     } # while
-    
+
     $sth->finish;
-    $dbh->disconnect;
     return($count, @results);
 } # sub getbiblio
 
 sub getbiblioitem {
     my ($biblioitemnum) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     my $query = "Select * from biblioitems where
 biblioitemnumber = $biblioitemnum";
     my $sth   = $dbh->prepare($query);
@@ -1784,13 +1938,12 @@ biblioitemnumber = $biblioitemnum";
     } # while
 
     $sth->finish;
-    $dbh->disconnect;
     return($count, @results);
 } # sub getbiblioitem
 
 sub getbiblioitembybiblionumber {
     my ($biblionumber) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     my $query = "Select * from biblioitems where biblionumber =
 $biblionumber";
     my $sth   = $dbh->prepare($query);
@@ -1805,13 +1958,12 @@ $biblionumber";
     } # while
 
     $sth->finish;
-    $dbh->disconnect;
     return($count, @results);
 } # sub
 
 sub getitemsbybiblioitem {
     my ($biblioitemnum) = @_;
-    my $dbh   = C4Connect;
+    my $dbh   = C4::Context->dbh;
     my $query = "Select * from items, biblio where
 biblio.biblionumber = items.biblionumber and biblioitemnumber
 = $biblioitemnum";
@@ -1819,84 +1971,18 @@ biblio.biblionumber = items.biblionumber and biblioitemnumber
       # || die "Cannot prepare $query\n" . $dbh->errstr;
     my $count = 0;
     my @results;
-    
+
     $sth->execute;
       # || die "Cannot execute $query\n" . $sth->errstr;
     while (my $data = $sth->fetchrow_hashref) {
       $results[$count] = $data;
       $count++;
     } # while
-    
+
     $sth->finish;
-    $dbh->disconnect;
     return($count, @results);
 } # sub getitemsbybiblioitem
 
-sub isbnsearch {
-    my ($isbn) = @_;
-    my $dbh   = C4Connect;
-    my $count = 0;
-    my $query;
-    my $sth;
-    my @results;
-    
-    $isbn  = $dbh->quote($isbn);
-    $query = "Select biblio.* from biblio, biblioitems where
-biblio.biblionumber = biblioitems.biblionumber
-and isbn = $isbn";
-    $sth   = $dbh->prepare($query);
-    
-    $sth->execute;
-    while (my $data = $sth->fetchrow_hashref) {
-        $results[$count] = $data;
-       $count++;
-    } # while
-
-    $sth->finish;
-    $dbh->disconnect;
-    return($count, @results);
-} # sub isbnsearch
-
-#sub skip {
-# At the moment this is just a straight copy of the subject code.  Needs heavy
-# modification to work for additional authors, obviously.
-# Check for additional author changes
-    
-#    my $newadditionalauthor='';
-#    my $additionalauthors;
-#    foreach $newadditionalauthor (@{$biblio->{'additionalauthor'}}) {
-#      $additionalauthors->{$newadditionalauthor}=1;
-#      if ($origadditionalauthors->{$newadditionalauthor}) {
-#          $additionalauthors->{$newadditionalauthor}=2;
-#      } else {
-#          my $q_newadditionalauthor=$dbh->quote($newadditionalauthor);
-#          my $sth=$dbh->prepare("insert into biblioadditionalauthors (additionalauthor,biblionumber) values ($q_newadditionalauthor, $biblionumber)");
-#          $sth->execute;
-#          logchange('kohadb', 'add', 'biblio', 'additionalauthor', $newadditionalauthor);
-#          my $subfields;
-#          $subfields->{1}->{'Subfield_Mark'}='a';
-#          $subfields->{1}->{'Subfield_Value'}=$newadditionalauthor;
-#          my $tag='650';
-#          my $Record_ID;
-#          foreach $Record_ID (@marcrecords) {
-#              addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-#              logchange('marc', 'add', $Record_ID, '650', 'a', $newadditionalauthor);
-#          }
-#      }
-#    }
-#    my $origadditionalauthor;
-#    foreach $origadditionalauthor (keys %$origadditionalauthors) {
-#      if ($additionalauthors->{$origadditionalauthor} == 1) {
-#          my $q_origadditionalauthor=$dbh->quote($origadditionalauthor);
-#          logchange('kohadb', 'delete', 'biblio', '$biblionumber', 'additionalauthor', $origadditionalauthor);
-#          my $sth=$dbh->prepare("delete from biblioadditionalauthors where biblionumber=$biblionumber and additionalauthor=$q_origadditionalauthor");
-#          $sth->execute;
-#      }
-#    }
-#
-#}
-#    $dbh->disconnect;
-#}
 
 sub logchange {
 # Subroutine to log changes to databases
@@ -1909,7 +1995,7 @@ sub logchange {
        my $item=shift;
        my $original=shift;
        my $new=shift;
-       print STDERR "KOHA: $type $section $item $original $new\n";
+#      print STDERR "KOHA: $type $section $item $original $new\n";
     } elsif ($database eq 'marc') {
        my $type=shift;
        my $Record_ID=shift;
@@ -1918,7 +2004,7 @@ sub logchange {
        my $subfield_ID=shift;
        my $original=shift;
        my $new=shift;
-       print STDERR "MARC: $type $Record_ID $tag $mark $subfield_ID $original $new\n";
+#      print STDERR "MARC: $type $Record_ID $tag $mark $subfield_ID $original $new\n";
     }
 }
 
@@ -1932,6 +2018,7 @@ sub getoraddbiblio {
        # input params
        my (
          $dbh,         # db handle
+                       # FIXME - Unused argument
          $biblio,      # hash ref to fields
        )=@_;
 
@@ -1943,12 +2030,12 @@ sub getoraddbiblio {
        my $error;
 
        #-----
-       requireDBI($dbh,"getoraddbiblio");
+       $dbh = C4::Context->dbh;
 
        print "<PRE>Looking for biblio </PRE>\n" if $debug;
        $sth=$dbh->prepare("select biblionumber
                from biblio
-               where title=? and author=? 
+               where title=? and author=?
                  and copyrightdate=? and seriestitle=?");
        $sth->execute(
                $biblio->{title}, $biblio->{author},
@@ -1974,871 +2061,486 @@ sub getoraddbiblio {
 
 } # sub getoraddbiblio
 
-#
-#
-# UNUSEFUL SUBs. Could be deleted, kept only until beta test
-# maybe useful for some MARC tricks steve used.
-#
-
-sub OLD_MAYBE_DELETED_newBiblioItem {
-    my ($env, $biblioitem) = @_;
-    my $dbh=&C4Connect;  
-    my $biblionumber=$biblioitem->{'biblionumber'};
-    my $biblioitemnumber=$biblioitem->{'biblioitemnumber'};
-    my $volume=$biblioitem->{'volume'};
-    my $q_volume=$dbh->quote($volume);
-    my $number=$biblioitem->{'number'};
-    my $q_number=$dbh->quote($number);
-    my $classification=$biblioitem->{'classification'};
-    my $q_classification=$dbh->quote($classification);
-    my $itemtype=$biblioitem->{'itemtype'};
-    my $q_itemtype=$dbh->quote($itemtype);
-    my $isbn=$biblioitem->{'isbn'};
-    my $q_isbn=$dbh->quote($isbn);
-    my $issn=$biblioitem->{'issn'};
-    my $q_issn=$dbh->quote($issn);
-    my $dewey=$biblioitem->{'dewey'};
-    $dewey=~s/\.*0*$//;
-    ($dewey == 0) && ($dewey='');
-    my $subclass=$biblioitem->{'subclass'};
-    my $q_subclass=$dbh->quote($subclass);
-    my $publicationyear=$biblioitem->{'publicationyear'};
-    my $publishercode=$biblioitem->{'publishercode'};
-    my $q_publishercode=$dbh->quote($publishercode);
-    my $volumedate=$biblioitem->{'volumedate'};
-    my $q_volumedate=$dbh->quote($volumedate);
-    my $illus=$biblioitem->{'illus'};
-    my $q_illus=$dbh->quote($illus);
-    my $pages=$biblioitem->{'pages'};
-    my $q_pages=$dbh->quote($pages);
-    my $notes=$biblioitem->{'notes'};
-    my $q_notes=$dbh->quote($notes);
-    my $size=$biblioitem->{'size'};
-    my $q_size=$dbh->quote($size);
-    my $place=$biblioitem->{'place'};
-    my $q_place=$dbh->quote($place);
-    my $lccn=$biblioitem->{'lccn'};
-    my $q_lccn=$dbh->quote($lccn);
-
-
-# Unless the $env->{'marconly'} flag is set, update the biblioitems table with
-# the new data
-
-    unless ($env->{'marconly'}) {
-       #my $sth=$dbh->prepare("lock tables biblioitems write");
-       #$sth->execute;
-       my $sth=$dbh->prepare("select max(biblioitemnumber) from biblioitems");
-       $sth->execute;
-       my ($biblioitemnumber) =$sth->fetchrow;
-       $biblioitemnumber++;
-       $sth=$dbh->prepare("insert into biblioitems (biblionumber,biblioitemnumber,volume,number,classification,itemtype,isbn,issn,dewey,subclass,publicationyear,publishercode,volumedate,illus,pages,notes,size,place,lccn) values ($biblionumber, $biblioitemnumber, $q_volume, $q_number, $q_classification, $q_itemtype, $q_isbn, $q_issn, $dewey, $q_subclass, $publicationyear, $q_publishercode, $q_volumedate, $q_illus, $q_pages,$q_notes, $q_size, $q_place, $q_lccn)");
-       $sth->execute;
-       #my $sth=$dbh->prepare("unlock tables");
-       #$sth->execute;
-    }
-
-
-# Should we check if there is already a biblioitem/amrc with the
-# same isbn/lccn/issn?
-
-    my $sth=$dbh->prepare("select title,unititle,seriestitle,copyrightdate,notes,author from biblio where biblionumber=$biblionumber");
-    $sth->execute;
-    my ($title, $unititle,$seriestitle,$copyrightdate,$biblionotes,$author) = $sth->fetchrow;
-    $sth=$dbh->prepare("select subtitle from bibliosubtitle where biblionumber=$biblionumber");
-    $sth->execute;
-    my ($subtitle) = $sth->fetchrow;
-    $sth=$dbh->prepare("select author from additionalauthors where biblionumber=$biblionumber");
-    $sth->execute;
-    my @additionalauthors;
-    while (my ($additionalauthor) = $sth->fetchrow) {
-       push (@additionalauthors, $additionalauthor);
-    }
-    $sth=$dbh->prepare("select subject from bibliosubject where biblionumber=$biblionumber");
-    $sth->execute;
-    my @subjects;
-    while (my ($subject) = $sth->fetchrow) {
-       push (@subjects, $subject);
-    }
-
-# MARC SECTION
-
-    $sth=$dbh->prepare("insert into Resource_Table (Record_ID) values (0)");
-    $sth->execute;
-    my $Resource_ID=$dbh->{'mysql_insertid'};
-    my $Record_ID=$Resource_ID;
-    $sth=$dbh->prepare("update Resource_Table set Record_ID=$Record_ID where Resource_ID=$Resource_ID");
-    $sth->execute;
-
-# Title
-    {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$title;
-       if ($subtitle) {
-           $subfields->{2}->{'Subfield_Mark'}='b';
-           $subfields->{2}->{'Subfield_Value'}=$subtitle;
-       }
-       my $tag='245';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-
-# author
-    {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$author;
-       my $tag='100';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Series Title
-    if ($seriestitle) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$seriestitle;
-       my $tag='440';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Biblio Note
-    if ($biblionotes) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$biblionotes;
-       $subfields->{2}->{'Subfield_Mark'}='3';
-       $subfields->{2}->{'Subfield_Value'}='biblio';
-       my $tag='500';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Additional Authors
-    foreach (@additionalauthors) {
-       my $author=$_;
-       (next) unless ($author);
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$author;
-       $subfields->{2}->{'Subfield_Mark'}='e';
-       $subfields->{2}->{'Subfield_Value'}='author';
-       my $tag='700';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Illustrator
-    if ($illus) {
-       (next) unless ($illus);
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$illus;
-       $subfields->{2}->{'Subfield_Mark'}='e';
-       $subfields->{2}->{'Subfield_Value'}='illustrator';
-       my $tag='700';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Subjects
-    foreach (@subjects) {
-       my $subject=$_;
-       (next) unless ($subject);
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$subject;
-       my $tag='650';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-
-
-# ISBN
-    if ($isbn) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$isbn;
-       my $tag='020';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# LCCN
-    if ($lccn) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$lccn;
-       my $tag='010';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# ISSN
-    if ($issn) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$issn;
-       my $tag='022';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# DEWEY
-    if ($dewey) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$dewey;
-       my $tag='082';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# DEWEY subclass and itemtype
-    {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$itemtype;
-       $subfields->{2}->{'Subfield_Mark'}='b';
-       $subfields->{2}->{'Subfield_Value'}=$subclass;
-       $subfields->{3}->{'Subfield_Mark'}='c';
-       $subfields->{3}->{'Subfield_Value'}=$biblionumber;
-       $subfields->{4}->{'Subfield_Mark'}='d';
-       $subfields->{4}->{'Subfield_Value'}=$biblioitemnumber;
-       my $tag='090';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# PUBLISHER
-    {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$place;
-       $subfields->{2}->{'Subfield_Mark'}='b';
-       $subfields->{2}->{'Subfield_Value'}=$publishercode;
-       $subfields->{3}->{'Subfield_Mark'}='c';
-       $subfields->{3}->{'Subfield_Value'}=$publicationyear;
-       if ($copyrightdate) {
-           $subfields->{4}->{'Subfield_Mark'}='c';
-           $subfields->{4}->{'Subfield_Value'}="c$copyrightdate";
-       }
-       my $tag='260';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# PHYSICAL
-    if ($pages || $size) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$pages;
-       $subfields->{2}->{'Subfield_Mark'}='c';
-       $subfields->{2}->{'Subfield_Value'}=$size;
-       my $tag='300';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Volume/Number
-    if ($volume || $number) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='v';
-       $subfields->{1}->{'Subfield_Value'}=$volume;
-       $subfields->{2}->{'Subfield_Mark'}='n';
-       $subfields->{2}->{'Subfield_Value'}=$number;
-       my $tag='440';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-# Biblioitem Note
-    if ($notes) {
-       my $subfields;
-       $subfields->{1}->{'Subfield_Mark'}='a';
-       $subfields->{1}->{'Subfield_Value'}=$notes;
-       $subfields->{2}->{'Subfield_Mark'}='3';
-       $subfields->{2}->{'Subfield_Value'}='biblioitem';
-       my $tag='500';
-       addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    }
-    $sth->finish;
-    $dbh->disconnect;
-    return ($env, $Record_ID);
-}
-
-sub OLD_MAYBE_DELETED_newItem {
-    my ($env, $Record_ID, $item) = @_;
-    my $dbh=&C4Connect;  
-    my $barcode=$item->{'barcode'};
-    my $q_barcode=$dbh->quote($barcode);
-    my $biblionumber=$item->{'biblionumber'};
-    my $biblioitemnumber=$item->{'biblioitemnumber'};
-    my $dateaccessioned=$item->{'dateaccessioned'};
-    my $booksellerid=$item->{'booksellerid'};
-    my $q_booksellerid=$dbh->quote($booksellerid);
-    my $homebranch=$item->{'homebranch'};
-    my $q_homebranch=$dbh->quote($homebranch);
-    my $holdingbranch=$item->{'holdingbranch'};
-    my $price=$item->{'price'};
-    my $replacementprice=$item->{'replacementprice'};
-    my $replacementpricedate=$item->{'replacementpricedate'};
-    my $q_replacementpricedate=$dbh->quote($replacementpricedate);
-    my $notforloan=$item->{'notforloan'};
-    my $itemlost=$item->{'itemlost'};
-    my $wthdrawn=$item->{'wthdrawn'};
-    my $restricted=$item->{'restricted'};
-    my $itemnotes=$item->{'itemnotes'};
-    my $q_itemnotes=$dbh->quote($itemnotes);
-    my $itemtype=$item->{'itemtype'};
-    my $subclass=$item->{'subclass'};
-
-# KOHADB Section
-
-    unless ($env->{'marconly'}) {
-       my $sth=$dbh->prepare("select max(itemnumber) from items");
-       $sth->execute;
-       my ($itemnumber) =$sth->fetchrow;
-       $itemnumber++;
-       $sth=$dbh->prepare("insert into items (itemnumber,biblionumber,biblioitemnumber,barcode,dateaccessioned,booksellerid,homebranch,price,replacementprice,replacementpricedate,notforloan,itemlost,wthdrawn,restricted,itemnotes) values ($itemnumber,$biblionumber,$biblioitemnumber,$q_barcode,$dateaccessioned,$q_booksellerid,$q_homebranch,$price,$q_replacementpricedate,$notforloan,$itemlost,$wthdrawn,$restricted,$q_itemnotes)");
-       $sth->execute;
-    }
-
-
-# MARC SECTION
-    my $subfields;
-    $subfields->{1}->{'Subfield_Mark'}='p';
-    $subfields->{1}->{'Subfield_Value'}=$barcode;
-    $subfields->{2}->{'Subfield_Mark'}='d';
-    $subfields->{2}->{'Subfield_Value'}=$dateaccessioned;
-    $subfields->{3}->{'Subfield_Mark'}='e';
-    $subfields->{3}->{'Subfield_Value'}=$booksellerid;
-    $subfields->{4}->{'Subfield_Mark'}='b';
-    $subfields->{4}->{'Subfield_Value'}=$homebranch;
-    $subfields->{5}->{'Subfield_Mark'}='l';
-    $subfields->{5}->{'Subfield_Value'}=$holdingbranch;
-    $subfields->{6}->{'Subfield_Mark'}='c';
-    $subfields->{6}->{'Subfield_Value'}=$price;
-    $subfields->{7}->{'Subfield_Mark'}='c';
-    $subfields->{7}->{'Subfield_Value'}=$replacementprice;
-    $subfields->{8}->{'Subfield_Mark'}='d';
-    $subfields->{8}->{'Subfield_Value'}=$replacementpricedate;
-    if ($notforloan) {
-       $subfields->{9}->{'Subfield_Mark'}='h';
-       $subfields->{9}->{'Subfield_Value'}='Not for loan';
-    }
-    if ($notforloan) {
-       $subfields->{10}->{'Subfield_Mark'}='j';
-       $subfields->{10}->{'Subfield_Value'}='Item lost';
-    }
-    if ($notforloan) {
-       $subfields->{11}->{'Subfield_Mark'}='j';
-       $subfields->{11}->{'Subfield_Value'}='Item withdrawn';
-    }
-    if ($notforloan) {
-       $subfields->{12}->{'Subfield_Mark'}='z';
-       $subfields->{12}->{'Subfield_Value'}=$itemnotes;
-    }
-    my $tag='876';
-    my $Tag_ID;
-    $env->{'linkage'}=1;
-    ($env, $Tag_ID) = addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-    $env->{'linkage'}=0;
-    $env->{'linkid'}=$Tag_ID;
-    $tag='852';
-    my $subfields2;
-    $subfields2->{1}->{'Subfield_Mark'}='a';
-    $subfields2->{1}->{'Subfield_Value'}='Coast Mountains School District';
-    $subfields2->{1}->{'Subfield_Mark'}='b';
-    $subfields2->{1}->{'Subfield_Value'}=$homebranch;
-    $subfields2->{1}->{'Subfield_Mark'}='c';
-    $subfields2->{1}->{'Subfield_Value'}=$itemtype;
-    $subfields2->{2}->{'Subfield_Mark'}='m';
-    $subfields2->{2}->{'Subfield_Value'}=$subclass;
-    addTag($env, $Record_ID, $tag, ' ', ' ', $subfields2);
-    $env->{'linkid'}='';
-}
-
-sub OLD_MAYBE_DELETED_updateBiblio {
-# Update the biblio with biblionumber $biblio->{'biblionumber'}
-# I guess this routine should search through all marc records for a record that
-# has the same biblionumber stored in it, and modify the MARC record as well as
-# the biblio table.
-#
-# Also, this subroutine should search through the $biblio object and compare it
-# to the existing record and _LOG ALL CHANGES MADE_ in some way.  I'd like for
-# this logging feature to be usable to undo changes easily.
-
-    my ($env, $biblio) = @_;
-    my $Record_ID;
-    my $biblionumber=$biblio->{'biblionumber'};
-    my $dbh=&C4Connect;  
-    my $sth=$dbh->prepare("select * from biblio where biblionumber=$biblionumber");
-    $sth->execute;
-    my $origbiblio=$sth->fetchrow_hashref;
-    $sth=$dbh->prepare("select subtitle from bibliosubtitle where biblionumber=$biblionumber");
-    $sth->execute;
-    my ($subtitle)=$sth->fetchrow;
-    $origbiblio->{'subtitle'}=$subtitle;
-    $sth=$dbh->prepare("select author from additionalauthors where biblionumber=$biblionumber");
-    $sth->execute;
-    my $origadditionalauthors;
-    while (my ($author) = $sth->fetchrow) {
-       push (@{$origbiblio->{'additionalauthors'}}, $author);
-       $origadditionalauthors->{$author}=1;
-    }
-    $sth=$dbh->prepare("select subject from bibliosubject where biblionumber=$biblionumber");
-    $sth->execute;
-    my $origsubjects;
-    while (my ($subject) = $sth->fetchrow) {
-       push (@{$origbiblio->{'subjects'}}, $subject);
-       $origsubjects->{$subject}=1;
-    }
-
-    
-# Obtain a list of MARC Record_ID's that are tied to this biblio
-    $sth=$dbh->prepare("select bibid from marc_subfield_table where tag='090' and subfieldvalue=$biblionumber and subfieldcode='c'");
-    $sth->execute;
-    my @marcrecords;
-    while (my ($bibid) = $sth->fetchrow) {
-       push(@marcrecords, $bibid);
-    }
-
-    my $bibid='';
-    if ($biblio->{'author'} ne $origbiblio->{'author'}) {
-       my $q_author=$dbh->quote($biblio->{'author'});
-       logchange('kohadb', 'change', 'biblio', 'author', $origbiblio->{'author'}, $biblio->{'author'});
-       my $sti=$dbh->prepare("update biblio set author=$q_author where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $bibid (@marcrecords) {
-           logchange('marc', 'change', $bibid, '100', 'a', $origbiblio->{'author'}, $biblio->{'author'});
-           changeSubfield($bibid, '100', 'a', $origbiblio->{'author'}, $biblio->{'author'});
-       }
-    }
-    if ($biblio->{'title'} ne $origbiblio->{'title'}) {
-       my $q_title=$dbh->quote($biblio->{'title'});
-       logchange('kohadb', 'change', 'biblio', 'title', $origbiblio->{'title'}, $biblio->{'title'});
-       my $sti=$dbh->prepare("update biblio set title=$q_title where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $Record_ID (@marcrecords) {
-           logchange('marc', 'change', $Record_ID, '245', 'a', $origbiblio->{'title'}, $biblio->{'title'});
-           changeSubfield($Record_ID, '245', 'a', $origbiblio->{'title'}, $biblio->{'title'});
-       }
-    }
-    if ($biblio->{'subtitle'} ne $origbiblio->{'subtitle'}) {
-       my $q_subtitle=$dbh->quote($biblio->{'subtitle'});
-       logchange('kohadb', 'change', 'biblio', 'subtitle', $origbiblio->{'subtitle'}, $biblio->{'subtitle'});
-       my $sti=$dbh->prepare("update bibliosubtitle set subtitle=$q_subtitle where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $Record_ID (@marcrecords) {
-           logchange('marc', 'change', $Record_ID, '245', 'b', $origbiblio->{'subtitle'}, $biblio->{'subtitle'});
-           changeSubfield($Record_ID, '245', 'b', $origbiblio->{'subtitle'}, $biblio->{'subtitle'});
-       }
-    }
-    if ($biblio->{'unititle'} ne $origbiblio->{'unititle'}) {
-       my $q_unititle=$dbh->quote($biblio->{'unititle'});
-       logchange('kohadb', 'change', 'biblio', 'unititle', $origbiblio->{'unititle'}, $biblio->{'unititle'});
-       my $sti=$dbh->prepare("update biblio set unititle=$q_unititle where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-    }
-    if ($biblio->{'notes'} ne $origbiblio->{'notes'}) {
-       my $q_notes=$dbh->quote($biblio->{'notes'});
-       logchange('kohadb', 'change', 'biblio', 'notes', $origbiblio->{'notes'}, $biblio->{'notes'});
-       my $sti=$dbh->prepare("update biblio set notes=$q_notes where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $Record_ID (@marcrecords) {
-           logchange('marc', 'change', $Record_ID, '500', 'a', $origbiblio->{'notes'}, $biblio->{'notes'});
-           changeSubfield($Record_ID, '500', 'a', $origbiblio->{'notes'}, $biblio->{'notes'});
-       }
-    }
-    if ($biblio->{'serial'} ne $origbiblio->{'serial'}) {
-       my $q_serial=$dbh->quote($biblio->{'serial'});
-       logchange('kohadb', 'change', 'biblio', 'serial', $origbiblio->{'serial'}, $biblio->{'serial'});
-       my $sti=$dbh->prepare("update biblio set serial=$q_serial where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-    }
-    if ($biblio->{'seriestitle'} ne $origbiblio->{'seriestitle'}) {
-       my $q_seriestitle=$dbh->quote($biblio->{'seriestitle'});
-       logchange('kohadb', 'change', 'biblio', 'seriestitle', $origbiblio->{'seriestitle'}, $biblio->{'seriestitle'});
-       my $sti=$dbh->prepare("update biblio set seriestitle=$q_seriestitle where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $Record_ID (@marcrecords) {
-           logchange('marc', 'change', $Record_ID, '440', 'a', $origbiblio->{'seriestitle'}, $biblio->{'seriestitle'});
-           changeSubfield($Record_ID, '440', 'a', $origbiblio->{'seriestitle'}, $biblio->{'seriestitle'});
-       }
-    }
-    if ($biblio->{'copyrightdate'} ne $origbiblio->{'copyrightdate'}) {
-       my $q_copyrightdate=$dbh->quote($biblio->{'copyrightdate'});
-       logchange('kohadb', 'change', 'biblio', 'copyrightdate', $origbiblio->{'copyrightdate'}, $biblio->{'copyrightdate'});
-       my $sti=$dbh->prepare("update biblio set copyrightdate=$q_copyrightdate where biblionumber=$biblio->{'biblionumber'}");
-       $sti->execute;
-       foreach $Record_ID (@marcrecords) {
-           logchange('marc', 'change', $Record_ID, '260', 'c', "c$origbiblio->{'notes'}", "c$biblio->{'notes'}");
-           changeSubfield($Record_ID, '260', 'c', "c$origbiblio->{'notes'}", "c$biblio->{'notes'}");
-       }
-    }
-
-# Check for subject heading changes
-    
-    my $newsubject='';
-    my $subjects;
-    foreach $newsubject (@{$biblio->{'subject'}}) {
-       $subjects->{$newsubject}=1;
-       if ($origsubjects->{$newsubject}) {
-           $subjects->{$newsubject}=2;
-       } else {
-           my $q_newsubject=$dbh->quote($newsubject);
-           my $sth=$dbh->prepare("insert into bibliosubject (subject,biblionumber) values ($q_newsubject, $biblionumber)");
-           $sth->execute;
-           logchange('kohadb', 'add', 'biblio', 'subject', $newsubject);
-           my $subfields;
-           $subfields->{1}->{'Subfield_Mark'}='a';
-           $subfields->{1}->{'Subfield_Value'}=$newsubject;
-           my $tag='650';
-           my $Record_ID;
-           foreach $Record_ID (@marcrecords) {
-               addTag($env, $Record_ID, $tag, ' ', ' ', $subfields);
-               logchange('marc', 'add', $Record_ID, '650', 'a', $newsubject);
-           }
-       }
-    }
-    my $origsubject;
-    foreach $origsubject (keys %$origsubjects) {
-       if ($subjects->{$origsubject} == 1) {
-           my $q_origsubject=$dbh->quote($origsubject);
-           logchange('kohadb', 'delete', 'biblio', '$biblionumber', 'subject', $origsubject);
-           my $sth=$dbh->prepare("delete from bibliosubject where biblionumber=$biblionumber and subject=$q_origsubject");
-           $sth->execute;
+sub char_decode {
+       # converts ISO 5426 coded string to ISO 8859-1
+       # sloppy code : should be improved in next issue
+       my ($string,$encoding) = @_ ;
+       $_ = $string ;
+#      $encoding = C4::Context->preference("marcflavour") unless $encoding;
+       if ($encoding eq "UNIMARC") {
+               s/\xe1/Æ/gm ;
+               s/\xe2/Ð/gm ;
+               s/\xe9/Ø/gm ;
+               s/\xec/þ/gm ;
+               s/\xf1/æ/gm ;
+               s/\xf3/ð/gm ;
+               s/\xf9/ø/gm ;
+               s/\xfb/ß/gm ;
+               s/\xc1\x61/à/gm ;
+               s/\xc1\x65/è/gm ;
+               s/\xc1\x69/ì/gm ;
+               s/\xc1\x6f/ò/gm ;
+               s/\xc1\x75/ù/gm ;
+               s/\xc1\x41/À/gm ;
+               s/\xc1\x45/È/gm ;
+               s/\xc1\x49/Ì/gm ;
+               s/\xc1\x4f/Ò/gm ;
+               s/\xc1\x55/Ù/gm ;
+               s/\xc2\x41/Á/gm ;
+               s/\xc2\x45/É/gm ;
+               s/\xc2\x49/Í/gm ;
+               s/\xc2\x4f/Ó/gm ;
+               s/\xc2\x55/Ú/gm ;
+               s/\xc2\x59/Ý/gm ;
+               s/\xc2\x61/á/gm ;
+               s/\xc2\x65/é/gm ;
+               s/\xc2\x69/í/gm ;
+               s/\xc2\x6f/ó/gm ;
+               s/\xc2\x75/ú/gm ;
+               s/\xc2\x79/ý/gm ;
+               s/\xc3\x41/Â/gm ;
+               s/\xc3\x45/Ê/gm ;
+               s/\xc3\x49/Î/gm ;
+               s/\xc3\x4f/Ô/gm ;
+               s/\xc3\x55/Û/gm ;
+               s/\xc3\x61/â/gm ;
+               s/\xc3\x65/ê/gm ;
+               s/\xc3\x69/î/gm ;
+               s/\xc3\x6f/ô/gm ;
+               s/\xc3\x75/û/gm ;
+               s/\xc4\x41/Ã/gm ;
+               s/\xc4\x4e/Ñ/gm ;
+               s/\xc4\x4f/Õ/gm ;
+               s/\xc4\x61/ã/gm ;
+               s/\xc4\x6e/ñ/gm ;
+               s/\xc4\x6f/õ/gm ;
+               s/\xc8\x45/Ë/gm ;
+               s/\xc8\x49/Ï/gm ;
+               s/\xc8\x65/ë/gm ;
+               s/\xc8\x69/ï/gm ;
+               s/\xc8\x76/ÿ/gm ;
+               s/\xc9\x41/Ä/gm ;
+               s/\xc9\x4f/Ö/gm ;
+               s/\xc9\x55/Ü/gm ;
+               s/\xc9\x61/ä/gm ;
+               s/\xc9\x6f/ö/gm ;
+               s/\xc9\x75/ü/gm ;
+               s/\xca\x41/Å/gm ;
+               s/\xca\x61/å/gm ;
+               s/\xd0\x43/Ç/gm ;
+               s/\xd0\x63/ç/gm ;
+               # this handles non-sorting blocks (if implementation requires this)
+               $string = nsb_clean($_) ;
+       } elsif ($encoding eq "USMARC" || $encoding eq "MARC21") {
+               if(/[\xc1-\xff]/) {
+                       s/\xe1\x61/à/gm ;
+                       s/\xe1\x65/è/gm ;
+                       s/\xe1\x69/ì/gm ;
+                       s/\xe1\x6f/ò/gm ;
+                       s/\xe1\x75/ù/gm ;
+                       s/\xe1\x41/À/gm ;
+                       s/\xe1\x45/È/gm ;
+                       s/\xe1\x49/Ì/gm ;
+                       s/\xe1\x4f/Ò/gm ;
+                       s/\xe1\x55/Ù/gm ;
+                       s/\xe2\x41/Á/gm ;
+                       s/\xe2\x45/É/gm ;
+                       s/\xe2\x49/Í/gm ;
+                       s/\xe2\x4f/Ó/gm ;
+                       s/\xe2\x55/Ú/gm ;
+                       s/\xe2\x59/Ý/gm ;
+                       s/\xe2\x61/á/gm ;
+                       s/\xe2\x65/é/gm ;
+                       s/\xe2\x69/í/gm ;
+                       s/\xe2\x6f/ó/gm ;
+                       s/\xe2\x75/ú/gm ;
+                       s/\xe2\x79/ý/gm ;
+                       s/\xe3\x41/Â/gm ;
+                       s/\xe3\x45/Ê/gm ;
+                       s/\xe3\x49/Î/gm ;
+                       s/\xe3\x4f/Ô/gm ;
+                       s/\xe3\x55/Û/gm ;
+                       s/\xe3\x61/â/gm ;
+                       s/\xe3\x65/ê/gm ;
+                       s/\xe3\x69/î/gm ;
+                       s/\xe3\x6f/ô/gm ;
+                       s/\xe3\x75/û/gm ;
+                       s/\xe4\x41/Ã/gm ;
+                       s/\xe4\x4e/Ñ/gm ;
+                       s/\xe4\x4f/Õ/gm ;
+                       s/\xe4\x61/ã/gm ;
+                       s/\xe4\x6e/ñ/gm ;
+                       s/\xe4\x6f/õ/gm ;
+                       s/\xe8\x45/Ë/gm ;
+                       s/\xe8\x49/Ï/gm ;
+                       s/\xe8\x65/ë/gm ;
+                       s/\xe8\x69/ï/gm ;
+                       s/\xe8\x76/ÿ/gm ;
+                       s/\xe9\x41/Ä/gm ;
+                       s/\xe9\x4f/Ö/gm ;
+                       s/\xe9\x55/Ü/gm ;
+                       s/\xe9\x61/ä/gm ;
+                       s/\xe9\x6f/ö/gm ;
+                       s/\xe9\x75/ü/gm ;
+                       s/\xea\x41/Å/gm ;
+                       s/\xea\x61/å/gm ;
+                       # this handles non-sorting blocks (if implementation requires this)
+                       $string = nsb_clean($_) ;
+               }
        }
-    }
-}
-
-sub OLD_MAYBE_DELETED_updateBiblioItem {
-# Update the biblioitem with biblioitemnumber $biblioitem->{'biblioitemnumber'}
-#
-# This routine should also check to see which fields are actually being
-# modified, and log all changes.
-
-    my ($env, $biblioitem) = @_;
-    my $dbh=&C4Connect;  
-
-    my $biblioitemnumber=$biblioitem->{'biblioitemnumber'};
-    my $sth=$dbh->prepare("select * from biblioitems where biblioitemnumber=$biblioitemnumber");
-# obi = original biblioitem
-    my $obi=$sth->fetchrow_hashref;
-    $sth=$dbh->prepare("select B.Record_ID from Bib_Table B, 0XX_Tag_Table T, 0XX_Subfield_Table S where B.Tag_0XX_ID=T.Tag_ID and T.Subfield_ID=S.Subfield_ID and T.Tag='090' and S.Subfield_Mark='c' and S.Subfield_Value=$biblioitemnumber");
-    $sth->execute;
-    my ($Record_ID) = $sth->fetchrow;
-    if ($biblioitem->{'biblionumber'} ne $obi->{'biblionumber'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'biblionumber', $obi->{'biblionumber'}, $biblioitem->{'biblionumber'});
-       my $sth=$dbh->prepare("update biblioitems set biblionumber=$biblioitem->{'biblionumber'} where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '090', 'c', $obi->{'biblionumber'}, $biblioitem->{'biblionumber'});
-       changeSubfield($Record_ID, '090', 'c', $obi->{'biblionumber'}, $biblioitem->{'biblionumber'});
-    }
-    if ($biblioitem->{'volume'} ne $obi->{'volume'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'volume', $obi->{'volume'}, $biblioitem->{'volume'});
-       my $q_volume=$dbh->quote($biblioitem->{'volume'});
-       my $sth=$dbh->prepare("update biblioitems set volume=$q_volume where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '440', 'v', $obi->{'volume'}, $biblioitem->{'volume'});
-       changeSubfield($Record_ID, '440', 'v', $obi->{'volume'}, $biblioitem->{'volume'});
-    }
-    if ($biblioitem->{'number'} ne $obi->{'number'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'number', $obi->{'number'}, $biblioitem->{'number'});
-       my $q_number=$dbh->quote($biblioitem->{'number'});
-       my $sth=$dbh->prepare("update biblioitems set number=$q_number where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '440', 'v', $obi->{'number'}, $biblioitem->{'number'});
-       changeSubfield($Record_ID, '440', 'v', $obi->{'number'}, $biblioitem->{'number'});
-    }
-    if ($biblioitem->{'itemtype'} ne $obi->{'itemtype'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'itemtype', $obi->{'itemtype'}, $biblioitem->{'itemtype'});
-       my $q_itemtype=$dbh->quote($biblioitem->{'itemtype'});
-       my $sth=$dbh->prepare("update biblioitems set itemtype=$q_itemtype where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '090', 'a', $obi->{'itemtype'}, $biblioitem->{'itemtype'});
-       changeSubfield($Record_ID, '090', 'a', $obi->{'itemtype'}, $biblioitem->{'itemtype'});
-    }
-    if ($biblioitem->{'isbn'} ne $obi->{'isbn'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'isbn', $obi->{'isbn'}, $biblioitem->{'isbn'});
-       my $q_isbn=$dbh->quote($biblioitem->{'isbn'});
-       my $sth=$dbh->prepare("update biblioitems set isbn=$q_isbn where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '020', 'a', $obi->{'isbn'}, $biblioitem->{'isbn'});
-       changeSubfield($Record_ID, '020', 'a', $obi->{'isbn'}, $biblioitem->{'isbn'});
-    }
-    if ($biblioitem->{'issn'} ne $obi->{'issn'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'issn', $obi->{'issn'}, $biblioitem->{'issn'});
-       my $q_issn=$dbh->quote($biblioitem->{'issn'});
-       my $sth=$dbh->prepare("update biblioitems set issn=$q_issn where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '022', 'a', $obi->{'issn'}, $biblioitem->{'issn'});
-       changeSubfield($Record_ID, '022', 'a', $obi->{'issn'}, $biblioitem->{'issn'});
-    }
-    if ($biblioitem->{'dewey'} ne $obi->{'dewey'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'dewey', $obi->{'dewey'}, $biblioitem->{'dewey'});
-       my $sth=$dbh->prepare("update biblioitems set dewey=$biblioitem->{'dewey'} where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '082', 'a', $obi->{'dewey'}, $biblioitem->{'dewey'});
-       changeSubfield($Record_ID, '082', 'a', $obi->{'dewey'}, $biblioitem->{'dewey'});
-    }
-    if ($biblioitem->{'subclass'} ne $obi->{'subclass'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'subclass', $obi->{'subclass'}, $biblioitem->{'subclass'});
-       my $q_subclass=$dbh->quote($biblioitem->{'subclass'});
-       my $sth=$dbh->prepare("update biblioitems set subclass=$q_subclass where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '090', 'b', $obi->{'subclass'}, $biblioitem->{'subclass'});
-       changeSubfield($Record_ID, '090', 'b', $obi->{'subclass'}, $biblioitem->{'subclass'});
-    }
-    if ($biblioitem->{'place'} ne $obi->{'place'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'place', $obi->{'place'}, $biblioitem->{'place'});
-       my $q_place=$dbh->quote($biblioitem->{'place'});
-       my $sth=$dbh->prepare("update biblioitems set place=$q_place where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '260', 'a', $obi->{'place'}, $biblioitem->{'place'});
-       changeSubfield($Record_ID, '260', 'a', $obi->{'place'}, $biblioitem->{'place'});
-    }
-    if ($biblioitem->{'publishercode'} ne $obi->{'publishercode'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'publishercode', $obi->{'publishercode'}, $biblioitem->{'publishercode'});
-       my $q_publishercode=$dbh->quote($biblioitem->{'publishercode'});
-       my $sth=$dbh->prepare("update biblioitems set publishercode=$q_publishercode where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '260', 'b', $obi->{'publishercode'}, $biblioitem->{'publishercode'});
-       changeSubfield($Record_ID, '260', 'b', $obi->{'publishercode'}, $biblioitem->{'publishercode'});
-    }
-    if ($biblioitem->{'publicationyear'} ne $obi->{'publicationyear'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'publicationyear', $obi->{'publicationyear'}, $biblioitem->{'publicationyear'});
-       my $q_publicationyear=$dbh->quote($biblioitem->{'publicationyear'});
-       my $sth=$dbh->prepare("update biblioitems set publicationyear=$q_publicationyear where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '260', 'c', $obi->{'publicationyear'}, $biblioitem->{'publicationyear'});
-       changeSubfield($Record_ID, '260', 'c', $obi->{'publicationyear'}, $biblioitem->{'publicationyear'});
-    }
-    if ($biblioitem->{'illus'} ne $obi->{'illus'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'illus', $obi->{'illus'}, $biblioitem->{'illus'});
-       my $q_illus=$dbh->quote($biblioitem->{'illus'});
-       my $sth=$dbh->prepare("update biblioitems set illus=$q_illus where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '700', 'a', $obi->{'illus'}, $biblioitem->{'illus'});
-       changeSubfield($Record_ID, '700', 'a', $obi->{'illus'}, $biblioitem->{'illus'});
-    }
-    if ($biblioitem->{'pages'} ne $obi->{'pages'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'pages', $obi->{'pages'}, $biblioitem->{'pages'});
-       my $q_pages=$dbh->quote($biblioitem->{'pages'});
-       my $sth=$dbh->prepare("update biblioitems set pages=$q_pages where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '300', 'a', $obi->{'pages'}, $biblioitem->{'pages'});
-       changeSubfield($Record_ID, '300', 'a', $obi->{'pages'}, $biblioitem->{'pages'});
-    }
-    if ($biblioitem->{'size'} ne $obi->{'size'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'size', $obi->{'size'}, $biblioitem->{'size'});
-       my $q_size=$dbh->quote($biblioitem->{'size'});
-       my $sth=$dbh->prepare("update biblioitems set size=$q_size where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '300', 'c', $obi->{'size'}, $biblioitem->{'size'});
-       changeSubfield($Record_ID, '300', 'c', $obi->{'size'}, $biblioitem->{'size'});
-    }
-    if ($biblioitem->{'notes'} ne $obi->{'notes'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'notes', $obi->{'notes'}, $biblioitem->{'notes'});
-       my $q_notes=$dbh->quote($biblioitem->{'notes'});
-       my $sth=$dbh->prepare("update biblioitems set notes=$q_notes where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '500', 'a', $obi->{'notes'}, $biblioitem->{'notes'});
-       changeSubfield($Record_ID, '500', 'a', $obi->{'notes'}, $biblioitem->{'notes'});
-    }
-    if ($biblioitem->{'lccn'} ne $obi->{'lccn'}) {
-       logchange('kohadb', 'change', 'biblioitems', 'lccn', $obi->{'lccn'}, $biblioitem->{'lccn'});
-       my $q_lccn=$dbh->quote($biblioitem->{'lccn'});
-       my $sth=$dbh->prepare("update biblioitems set lccn=$q_lccn where biblioitemnumber=$biblioitemnumber");
-       logchange('marc', 'change', $Record_ID, '010', 'a', $obi->{'lccn'}, $biblioitem->{'lccn'});
-       changeSubfield($Record_ID, '010', 'a', $obi->{'lccn'}, $biblioitem->{'lccn'});
-    }
-    $sth->finish;
-    $dbh->disconnect;
-
+       return($string) ;
 }
 
-sub OLD_MAYBE_DELETED_updateItem {
-# Update the item with itemnumber $item->{'itemnumber'}
-# This routine should also modify the corresponding MARC record data. (852 and
-# 876 tags with 876p tag the same as $item->{'barcode'}
-#
-# This routine should also check to see which fields are actually being
-# modified, and log all changes.
-
-    my ($env, $item) = @_;
-    my $dbh=&C4Connect;  
-    my $itemnumber=$item->{'itemnumber'};
-    my $biblionumber=$item->{'biblionumber'};
-    my $biblioitemnumber=$item->{'biblioitemnumber'};
-    my $barcode=$item->{'barcode'};
-    my $dateaccessioned=$item->{'dateaccessioned'};
-    my $booksellerid=$item->{'booksellerid'};
-    my $homebranch=$item->{'homebranch'};
-    my $price=$item->{'price'};
-    my $replacementprice=$item->{'replacementprice'};
-    my $replacementpricedate=$item->{'replacementpricedate'};
-    my $multivolume=$item->{'multivolume'};
-    my $stack=$item->{'stack'};
-    my $notforloan=$item->{'notforloan'};
-    my $itemlost=$item->{'itemlost'};
-    my $wthdrawn=$item->{'wthdrawn'};
-    my $bulk=$item->{'bulk'};
-    my $restricted=$item->{'restricted'};
-    my $binding=$item->{'binding'};
-    my $itemnotes=$item->{'itemnotes'};
-    my $holdingbranch=$item->{'holdingbranch'};
-    my $interim=$item->{'interim'};
-    my $sth=$dbh->prepare("select * from items where itemnumber=$itemnumber");
-    $sth->execute;
-    my $olditem=$sth->fetchrow_hashref;
-    my $q_barcode=$dbh->quote($olditem->{'barcode'});
-    $sth=$dbh->prepare("select S.Subfield_ID, B.Record_ID from 8XX_Subfield_Table S, 8XX_Tag_Table T, Bib_Table B where B.Tag_8XX_ID=T.Tag_ID and T.Subfield_ID=S.Subfield_ID and Subfield_Mark='p' and Subfield_Value=$q_barcode");
-    $sth->execute;
-    my ($Subfield876_ID, $Record_ID) = $sth->fetchrow;
-    $sth=$dbh->prepare("select Subfield_Value from 8XX_Subfield_Table where Subfield_Mark=8 and Subfield_ID=$Subfield876_ID");
-    $sth->execute;
-    my ($link) = $sth->fetchrow;
-    $sth=$dbh->prepare("select Subfield_ID from 8XX_Subfield_Table where Subfield_Mark=8 and Subfield_Value=$link and !(Subfield_ID=$Subfield876_ID)");
-    $sth->execute;
-    my ($Subfield852_ID) = $sth->fetchrow;
-    
-    if ($item->{'barcode'} ne $olditem->{'barcode'}) {
-       logchange('kohadb', 'change', 'items', 'barcode', $olditem->{'barcode'}, $item->{'barcode'});
-       my $q_barcode=$dbh->quote($item->{'barcode'});
-       my $sth=$dbh->prepare("update items set barcode=$q_barcode where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'p', $olditem->{'barcode'}, $item->{'barcode'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'p', $Subfield_Key, $olditem->{'barcode'}, $item->{'barcode'});
-    }
-    if ($item->{'booksellerid'} ne $olditem->{'booksellerid'}) {
-       logchange('kohadb', 'change', 'items', 'booksellerid', $olditem->{'booksellerid'}, $item->{'booksellerid'});
-       my $q_booksellerid=$dbh->quote($item->{'booksellerid'});
-       my $sth=$dbh->prepare("update items set booksellerid=$q_booksellerid where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'e', $olditem->{'booksellerid'}, $item->{'booksellerid'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'e', $Subfield_Key, $olditem->{'booksellerid'}, $item->{'booksellerid'});
-    }
-    if ($item->{'dateaccessioned'} ne $olditem->{'dateaccessioned'}) {
-       logchange('kohadb', 'change', 'items', 'dateaccessioned', $olditem->{'dateaccessioned'}, $item->{'dateaccessioned'});
-       my $q_dateaccessioned=$dbh->quote($item->{'dateaccessioned'});
-       my $sth=$dbh->prepare("update items set dateaccessioned=$q_dateaccessioned where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'd', $olditem->{'dateaccessioned'}, $item->{'dateaccessioned'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'd', $Subfield_Key, $olditem->{'dateaccessioned'}, $item->{'dateaccessioned'});
-    }
-    if ($item->{'homebranch'} ne $olditem->{'homebranch'}) {
-       logchange('kohadb', 'change', 'items', 'homebranch', $olditem->{'homebranch'}, $item->{'homebranch'});
-       my $q_homebranch=$dbh->quote($item->{'homebranch'});
-       my $sth=$dbh->prepare("update items set homebranch=$q_homebranch where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'b', $olditem->{'homebranch'}, $item->{'homebranch'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'b', $Subfield_Key, $olditem->{'homebranch'}, $item->{'homebranch'});
-    }
-    if ($item->{'holdingbranch'} ne $olditem->{'holdingbranch'}) {
-       logchange('kohadb', 'change', 'items', 'holdingbranch', $olditem->{'holdingbranch'}, $item->{'holdingbranch'});
-       my $q_holdingbranch=$dbh->quote($item->{'holdingbranch'});
-       my $sth=$dbh->prepare("update items set holdingbranch=$q_holdingbranch where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'l', $olditem->{'holdingbranch'}, $item->{'holdingbranch'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'l', $Subfield_Key, $olditem->{'holdingbranch'}, $item->{'holdingbranch'});
-    }
-    if ($item->{'price'} ne $olditem->{'price'}) {
-       logchange('kohadb', 'change', 'items', 'price', $olditem->{'price'}, $item->{'price'});
-       my $q_price=$dbh->quote($item->{'price'});
-       my $sth=$dbh->prepare("update items set price=$q_price where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'c', $olditem->{'price'}, $item->{'price'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'c', $Subfield_Key, $olditem->{'price'}, $item->{'price'});
-    }
-    if ($item->{'itemnotes'} ne $olditem->{'itemnotes'}) {
-       logchange('kohadb', 'change', 'items', 'itemnotes', $olditem->{'itemnotes'}, $item->{'itemnotes'});
-       my $q_itemnotes=$dbh->quote($item->{'itemnotes'});
-       my $sth=$dbh->prepare("update items set itemnotes=$q_itemnotes where itemnumber=$itemnumber");
-       $sth->execute;
-       my ($Subfield_ID, $Subfield_Key) = changeSubfield($Record_ID, '876', 'c', $olditem->{'itemnotes'}, $item->{'itemnotes'}, $Subfield876_ID);
-       logchange('marc', 'change', $Record_ID, '876', 'c', $Subfield_Key, $olditem->{'itemnotes'}, $item->{'itemnotes'});
-    }
-    if ($item->{'notforloan'} ne $olditem->{'notforloan'}) {
-       logchange('kohadb', 'change', 'items', 'notforloan', $olditem->{'notforloan'}, $item->{'notforloan'});
-       my $sth=$dbh->prepare("update items set notforloan=$notforloan where itemnumber=$itemnumber");
-       $sth->execute;
-       if ($item->{'notforloan'}) {
-           my ($Subfield_ID, $Subfield_Key) = addSubfield($Record_ID, '876', 'h', 'Not for loan', $Subfield876_ID);
-           logchange('marc', 'add', $Record_ID, '876', 'h', $Subfield_Key, 'Not for loan');
-       } else {
-           my ($Subfield_ID, $Subfield_Key) = deleteSubfield($Record_ID, '876', 'h', 'Not for loan', $Subfield876_ID);
-           logchange('marc', 'delete', $Record_ID, '876', 'h', $Subfield_Key, 'Not for loan');
-       }
-    }
-    if ($item->{'itemlost'} ne $olditem->{'itemlost'}) {
-       logchange('kohadb', 'change', 'items', 'itemlost', $olditem->{'itemlost'}, $item->{'itemlost'});
-       my $sth=$dbh->prepare("update items set itemlost=$itemlost where itemnumber=$itemnumber");
-       $sth->execute;
-       if ($item->{'itemlost'}) {
-           my ($Subfield_ID, $Subfield_Key) = addSubfield($Record_ID, '876', 'h', 'Item lost', $Subfield876_ID);
-           logchange('marc', 'add', $Record_ID, '876', 'h', $Subfield_Key, 'Item lost');
-       } else {
-           my ($Subfield_ID, $Subfield_Key) = deleteSubfield($Record_ID, '876', 'h', 'Item lost', $Subfield876_ID);
-           logchange('marc', 'delete', $Record_ID, '876', 'h', $Subfield_Key, 'Item lost');
-       }
-    }
-    if ($item->{'wthdrawn'} ne $olditem->{'wthdrawn'}) {
-       logchange('kohadb', 'change', 'items', 'wthdrawn', $olditem->{'wthdrawn'}, $item->{'wthdrawn'});
-       my $sth=$dbh->prepare("update items set wthdrawn=$wthdrawn where itemnumber=$itemnumber");
-       $sth->execute;
-       if ($item->{'wthdrawn'}) {
-           my ($Subfield_ID, $Subfield_Key) = addSubfield($Record_ID, '876', 'h', 'Withdrawn', $Subfield876_ID);
-           logchange('marc', 'add', $Record_ID, '876', 'h', $Subfield_Key, 'Withdrawn');
-       } else {
-           my ($Subfield_ID, $Subfield_Key) = deleteSubfield($Record_ID, '876', 'h', 'Withdrawn', $Subfield876_ID);
-           logchange('marc', 'delete', $Record_ID, '876', 'h', $Subfield_Key, 'Withdrawn');
-       }
-    }
-    if ($item->{'restricted'} ne $olditem->{'restricted'}) {
-       logchange('kohadb', 'change', 'items', 'restricted', $olditem->{'restricted'}, $item->{'restricted'});
-       my $sth=$dbh->prepare("update items set restricted=$restricted where itemnumber=$itemnumber");
-       $sth->execute;
-       if ($item->{'restricted'}) {
-           my ($Subfield_ID, $Subfield_Key) = addSubfield($Record_ID, '876', 'h', 'Restricted', $Subfield876_ID);
-           logchange('marc', 'add', $Record_ID, '876', 'h', $Subfield_Key, 'Restricted');
-       } else {
-           my ($Subfield_ID, $Subfield_Key) = deleteSubfield($Record_ID, '876', 'h', 'Restricted', $Subfield876_ID);
-           logchange('marc', 'delete', $Record_ID, '876', 'h', $Subfield_Key, 'Restricted');
-       }
-    }
-    $sth->finish;
-    $dbh->disconnect;
+sub nsb_clean {
+       my $NSB = '\x88' ;              # NSB : begin Non Sorting Block
+       my $NSE = '\x89' ;              # NSE : Non Sorting Block end
+       # handles non sorting blocks
+       my ($string) = @_ ;
+       $_ = $string ;
+       s/$NSB/(/gm ;
+       s/[ ]{0,1}$NSE/) /gm ;
+       $string = $_ ;
+       return($string) ;
 }
 
-# Add a biblioitem and related data to Koha database
-sub OLD_MAY_BE_DELETED_newcompletebiblioitem {
-       use strict;
-
-       my (
-         $dbh,                 # DBI handle
-         $biblio,              # hash ref to biblio record
-         $biblioitem,          # hash ref to biblioitem record
-         $subjects,            # list ref of subjects
-         $addlauthors,         # list ref of additional authors
-       )=@_ ;
-
-       my ( $biblionumber, $biblioitemnumber, $error);         # return values
-
-       my $debug=0;
-       my $sth;
-       my $subjectheading;
-       my $additionalauthor;
-
-       #--------
-       requireDBI($dbh,"newcompletebiblioitem");
-
-       print "<PRE>Trying to add biblio item Title=$biblio->{title} " .
-               "ISBN=$biblioitem->{isbn} </PRE>\n" if $debug;
-
-       # Make sure master biblio entry exists
-       ($biblionumber,$error)=getoraddbiblio($dbh, $biblio);
-
-        if ( ! $error ) {
-
-         $biblioitem->{biblionumber}=$biblionumber;
-
-         # Add biblioitem
-         $biblioitemnumber=newbiblioitem($biblioitem);
-
-         # Add subjects
-         $sth=$dbh->prepare("insert into bibliosubject
-               (biblionumber,subject)
-               values (?, ? )" );
-         foreach $subjectheading (@{$subjects} ) {
-             $sth->execute($biblionumber, $subjectheading)
-                       or $error.=$sth->errstr ;
-
-         } # foreach subject
+END { }       # module clean-up code here (global destructor)
 
-         # Add additional authors
-         $sth=$dbh->prepare("insert into additionalauthors
-               (biblionumber,author)
-               values (?, ? )");
-         foreach $additionalauthor (@{$addlauthors} ) {
-           $sth->execute($biblionumber, $additionalauthor)
-                       or $error.=$sth->errstr ;
-         } # foreach author
+=back
 
-       } else {
-         # couldn't get biblio
-         $biblionumber='';
-         $biblioitemnumber='';
+=head1 AUTHOR
 
-       } # if no biblio error
+Koha Developement team <info@koha.org>
 
-       return ( $biblionumber, $biblioitemnumber, $error);
+Paul POULAIN paul.poulain@free.fr
 
-} # sub newcompletebiblioitem
+=cut
 
+# $Id$
+# $Log$
+# Revision 1.74  2003/11/28 09:48:33  tipaul
+# bugfix : misusing prepare & execute => now using prepare(?) and execute($var)
 #
+# Revision 1.73  2003/11/28 09:45:25  tipaul
+# bugfix for iso2709 file import in the "notforloan" field.
 #
-# END OF UNUSEFUL SUBs
+# But notforloan field called "loan" somewhere, so in case "loan" is used, copied to "notforloan" to avoid a bug.
 #
+# Revision 1.72  2003/11/24 17:40:14  tipaul
+# fix for #385
+#
+# Revision 1.71  2003/11/24 16:28:49  tipaul
+# biblio & item deletion now works fine in MARC editor.
+# Stores deleted biblio/item in the marc field of the deletedbiblio/deleteditem table.
+#
+# Revision 1.70  2003/11/24 13:29:55  tipaul
+# moving $id from beginning to end of file (70 commits... huge comments...)
+#
+# Revision 1.69  2003/11/24 13:27:17  tipaul
+# fix for #380 (bibliosubject)
+#
+# Revision 1.68  2003/11/06 17:18:30  tipaul
+# bugfix for #384
+#
+# 1st draft for MARC biblio deletion.
+# Still does not work well, but at least, Biblio.pm compiles & it should'nt break too many things
+# (Note the trash in the MARCdetail, but don't use it, please :-) )
+#
+# Revision 1.67  2003/10/25 08:46:27  tipaul
+# minor fixes for bilbio deletion (still buggy)
+#
+# Revision 1.66  2003/10/17 10:02:56  tipaul
+# Indexing only words longer than 2 letters. Was >=2 before, & 2 letters words usually means nothing.
+#
+# Revision 1.65  2003/10/14 09:45:29  tipaul
+# adding rebuildnonmarc.pl script : run this script when you change a link between marc and non MARC DB. It rebuilds the non-MARC DB (long operation)
+#
+# Revision 1.64  2003/10/06 15:20:51  tipaul
+# fix for 536 (subtitle error)
+#
+# Revision 1.63  2003/10/01 13:25:49  tipaul
+# seems a char encoding problem modified something in char_decode sub... changing back to something that works...
+#
+# Revision 1.62  2003/09/17 14:21:13  tipaul
+# fixing bug that makes a MARC biblio disappear when using full acquisition (order => recieve ==> MARC editor).
+# Before this 2 lines fix, the MARC biblio was deleted during recieve, and had to be entirely recreated :-(
+#
+# Revision 1.61  2003/09/17 10:24:39  tipaul
+# notforloan value in itemtype was overwritting notforloan value in a given item.
+# I changed this behaviour :
+# if notforloan is set for a given item, and NOT for all items from this itemtype, the notforloan is kept.
+# If notforloan is set for itemtype, it's used (and impossible to loan a specific item from this itemtype)
+#
+# Revision 1.60  2003/09/04 14:11:23  tipaul
+# fix for 593 (data duplication in MARC-DB)
+#
+# Revision 1.58  2003/08/06 12:54:52  tipaul
+# fix for publicationyear : extracting numeric value from MARC string, like for copyrightdate.
+# (note that copyrightdate still extracted to get numeric format)
+#
+# Revision 1.57  2003/07/15 23:09:18  slef
+# change show columns to use biblioitems bnotes too
+#
+# Revision 1.56  2003/07/15 11:34:52  slef
+# fixes from paul email
+#
+# Revision 1.55  2003/07/15 00:02:49  slef
+# Work on bug 515... can we do a single-side rename of notes to bnotes?
+#
+# Revision 1.54  2003/07/11 11:51:32  tipaul
+# *** empty log message ***
+#
+# Revision 1.52  2003/07/10 10:37:19  tipaul
+# fix for copyrightdate problem, #514
+#
+# Revision 1.51  2003/07/02 14:47:17  tipaul
+# fix for #519 : items.dateaccessioned imports incorrectly
+#
+# Revision 1.49  2003/06/17 11:21:13  tipaul
+# improvments/fixes for z3950 support.
+# * Works now even on ADD, not only on MODIFY
+# * able to search on ISBN, author, title
+#
+# Revision 1.48  2003/06/16 09:22:53  rangi
+# Just added an order clause to getitemtypes
+#
+# Revision 1.47  2003/05/20 16:22:44  tipaul
+# fixing typo in Biblio.pm POD
+#
+# Revision 1.46  2003/05/19 13:45:18  tipaul
+# support for subtitles, additional authors, subject.
+# This supports is only for MARC <-> OLD-DB link. It worked previously, but values entered as MARC were not reported to OLD-DB, neither values entered as OLD-DB were reported to MARC.
+# Note that some OLD-DB subs are strange (dummy ?) see OLDmodsubject, OLDmodsubtitle, OLDmodaddiauthor in C4/Biblio.pm
+# For example it seems impossible to have more that 1 addi author and 1 subtitle. In MARC it's not the case. So, if you enter more than one, I'm afraid only the LAST will be stored.
+#
+# Revision 1.45  2003/04/29 16:50:49  tipaul
+# really proud of this commit :-)
+# z3950 search and import seems to works fine.
+# Let me explain how :
+# * a "search z3950" button is added in the addbiblio template.
+# * when clicked, a popup appears and z3950/search.pl is called
+# * z3950/search.pl calls addz3950search in the DB
+# * the z3950 daemon retrieve the records and stores them in z3950results AND in marc_breeding table.
+# * as long as there as searches pending, the popup auto refresh every 2 seconds, and says how many searches are pending.
+# * when the user clicks on a z3950 result => the parent popup is called with the requested biblio, and auto-filled
+#
+# Note :
+# * character encoding support : (It's a nightmare...) In the z3950servers table, a "encoding" column has been added. You can put "UNIMARC" or "USMARC" in this column. Depending on this, the char_decode in C4::Biblio.pm replaces marc-char-encode by an iso 8859-1 encoding. Note that in the breeding import this value has been added too, for a better support.
+# * the marc_breeding and z3950* tables have been modified : they have an encoding column and the random z3950 number is stored too for convenience => it's the key I use to list only requested biblios in the popup.
+#
+# Revision 1.44  2003/04/28 13:07:14  tipaul
+# Those fixes solves the "internal server error" with MARC::Record 1.12.
+# It was due to an illegal contruction in Koha : we tried to retrive subfields from <10 tags.
+# That's not possible. MARC::Record accepted this in 0.93 version, but it was fixed after.
+# Now, the construct/retrieving is OK !
+#
+# Revision 1.43  2003/04/10 13:56:02  tipaul
+# Fix some bugs :
+# * worked in 1.9.0, but not in 1.9.1 :
+# - modif of a biblio didn't work
+# - empty fields where not shown when modifying a biblio. empty fields managed by the library (ie in tab 0->9 in MARC parameter table) MUST be entered, even if not presented.
+#
+# * did not work before :
+# - repeatable subfields now works correctly. Enter 2 subfields separated by | and they will be splitted during saving.
+# - dropped the last subfield of the MARC form :-(
+#
+# Internal changes :
+# - MARCmodbiblio now works by deleting and recreating the biblio. It's not perf optimized, but MARC is a "do_something_impossible_to_trace" standard, so, it's the best solution. not a problem for me, as biblio are rarely modified.
+# Note the MARCdelbiblio has been rewritted to enable deletion of a biblio WITHOUT deleting items.
+#
+# Revision 1.42  2003/04/04 08:41:11  tipaul
+# last commits before 1.9.1
+#
+# Revision 1.41  2003/04/01 12:26:43  tipaul
+# fixes
+#
+# Revision 1.40  2003/03/11 15:14:03  tipaul
+# pod updating
+#
+# Revision 1.39  2003/03/07 16:35:42  tipaul
+# * moving generic functions to Koha.pm
+# * improvement of SearchMarc.pm
+# * bugfixes
+# * code cleaning
+#
+# Revision 1.38  2003/02/27 16:51:59  tipaul
+# * moving prepare / execute to ? form.
+# * some # cleaning
+# * little bugfix.
+# * road to 1.9.2 => acquisition and cataloguing merging
+#
+# Revision 1.37  2003/02/12 11:03:03  tipaul
+# Support for 000 -> 010 fields.
+# Those fields doesn't have subfields.
+# In koha, we will use a specific "trick" : fields <10 will have a "virtual" subfield : "@".
+# Note it's only virtual : when rebuilding the MARC::Record, the koha API handle correctly "@" subfields => the resulting MARC record has a 00x field without subfield.
+#
+# Revision 1.36  2003/02/12 11:01:01  tipaul
+# Support for 000 -> 010 fields.
+# Those fields doesn't have subfields.
+# In koha, we will use a specific "trick" : fields <10 will have a "virtual" subfield : "@".
+# Note it's only virtual : when rebuilding the MARC::Record, the koha API handle correctly "@" subfields => the resulting MARC record has a 00x field without subfield.
+#
+# Revision 1.35  2003/02/03 18:46:00  acli
+# Minor factoring in C4/Biblio.pm, plus change to export the per-tag
+# 'mandatory' property to a per-subfield 'tag_mandatory' template parameter,
+# so that addbiblio.tmpl can distinguish between mandatory subfields in a
+# mandatory tag and mandatory subfields in an optional tag
+#
+# Not-minor factoring in acqui.simple/addbiblio.pl to make the if-else blocks
+# smaller, and to add some POD; need further testing for this
+#
+# Added function to check if a MARC subfield name is "koha-internal" (instead
+# of checking it for 'lib' and 'tag' everywhere); temporarily added to Koha.pm
+#
+# Use above function in acqui.simple/additem.pl and search.marc/search.pl
+#
+# Revision 1.34  2003/01/28 14:50:04  tipaul
+# fixing MARCmodbiblio API and reindenting code
+#
+# Revision 1.33  2003/01/23 12:22:37  tipaul
+# adding char_decode to decode MARC21 or UNIMARC extended chars
+#
+# Revision 1.32  2002/12/16 15:08:50  tipaul
+# small but important bugfix (fixes a problem in export)
+#
+# Revision 1.31  2002/12/13 16:22:04  tipaul
+# 1st draft of marc export
+#
+# Revision 1.30  2002/12/12 21:26:35  tipaul
+# YAB ! (Yet Another Bugfix) => related to biblio modif
+# (some warning cleaning too)
+#
+# Revision 1.29  2002/12/12 16:35:00  tipaul
+# adding authentification with Auth.pm and
+# MAJOR BUGFIX on marc biblio modification
+#
+# Revision 1.28  2002/12/10 13:30:03  tipaul
+# fugfixes from Dombes Abbey work
+#
+# Revision 1.27  2002/11/19 12:36:16  tipaul
+# road to 1.3.2
+# various bugfixes, improvments, and migration from acquisition.pm to biblio.pm
+#
+# Revision 1.26  2002/11/12 15:58:43  tipaul
+# road to 1.3.2 :
+# * many bugfixes
+# * adding value_builder : you can map a subfield in the marc_subfield_structure to a sub stored in "value_builder" directory. In this directory you can create screen used to build values with any method. In this commit is a 1st draft of the builder for 100$a unimarc french subfield, which is composed of 35 digits, with 12 differents values (only the 4th first are provided for instance)
+#
+# Revision 1.25  2002/10/25 10:58:26  tipaul
+# Road to 1.3.2
+# * bugfixes and improvements
+#
+# Revision 1.24  2002/10/24 12:09:01  arensb
+# Fixed "no title" warning when generating HTML documentation from POD.
+#
+# Revision 1.23  2002/10/16 12:43:08  arensb
+# Added some FIXME comments.
+#
+# Revision 1.22  2002/10/15 13:39:17  tipaul
+# removing Acquisition.pm
+# deleting unused code in biblio.pm, rewriting POD and answering most FIXME comments
+#
+# Revision 1.21  2002/10/13 11:34:14  arensb
+# Replaced expressions of the form "$x = $x <op> $y" with "$x <op>= $y".
+# Thus, $x = $x+2 becomes $x += 2, and so forth.
+#
+# Revision 1.20  2002/10/13 08:28:32  arensb
+# Deleted unused variables.
+# Removed trailing whitespace.
+#
+# Revision 1.19  2002/10/13 05:56:10  arensb
+# Added some FIXME comments.
+#
+# Revision 1.18  2002/10/11 12:34:53  arensb
+# Replaced &requireDBI with C4::Context->dbh
+#
+# Revision 1.17  2002/10/10 14:48:25  tipaul
+# bugfixes
+#
+# Revision 1.16  2002/10/07 14:04:26  tipaul
+# road to 1.3.1 : viewing MARC biblio
+#
+# Revision 1.15  2002/10/05 09:49:25  arensb
+# Merged with arensb-context branch: use C4::Context->dbh instead of
+# &C4Connect, and generally prefer C4::Context over C4::Database.
+#
+# Revision 1.14  2002/10/03 11:28:18  tipaul
+# Extending Context.pm to add stopword management and using it in MARC-API.
+# First benchmarks show a medium speed improvement, which  is nice as this part is heavily called.
+#
+# Revision 1.13  2002/10/02 16:26:44  tipaul
+# road to 1.3.1
+#
+# Revision 1.12.2.4  2002/10/05 07:09:31  arensb
+# Merged in changes from main branch.
+#
+# Revision 1.12.2.3  2002/10/05 06:12:10  arensb
+# Added a whole mess of FIXME comments.
+#
+# Revision 1.12.2.2  2002/10/05 04:03:14  arensb
+# Added some missing semicolons.
+#
+# Revision 1.12.2.1  2002/10/04 02:24:01  arensb
+# Use C4::Connect instead of C4::Database, C4::Connect->dbh instead
+# C4Connect.
+#
+# Revision 1.12.2.3  2002/10/05 06:12:10  arensb
+# Added a whole mess of FIXME comments.
+#
+# Revision 1.12.2.2  2002/10/05 04:03:14  arensb
+# Added some missing semicolons.
+#
+# Revision 1.12.2.1  2002/10/04 02:24:01  arensb
+# Use C4::Connect instead of C4::Database, C4::Connect->dbh instead
+# C4Connect.
+#
+# Revision 1.12  2002/10/01 11:48:51  arensb
+# Added some FIXME comments, mostly marking duplicate functions.
+#
+# Revision 1.11  2002/09/24 13:49:26  tipaul
+# long WAS the road to 1.3.0...
+# coming VERY SOON NOW...
+# modifying installer and buildrelease to update the DB
+#
+# Revision 1.10  2002/09/22 16:50:08  arensb
+# Added some FIXME comments.
+#
+# Revision 1.9  2002/09/20 12:57:46  tipaul
+# long is the road to 1.4.0
+# * MARCadditem and MARCmoditem now wroks
+# * various bugfixes in MARC management
+# !!! 1.3.0 should be released very soon now. Be careful !!!
+#
+# Revision 1.8  2002/09/10 13:53:52  tipaul
+# MARC API continued...
+# * some bugfixes
+# * multiple item management : MARCadditem and MARCmoditem have been added. They suppose that ALL the MARC field linked to koha-item are in the same MARC tag (on the same line of MARC file)
+#
+# Note : it should not be hard for marcimport and marcexport to re-link fields from internal tag/subfield to "legal" tag/subfield.
+#
+# Revision 1.7  2002/08/14 18:12:51  tonnesen
+# Added copyright statement to all .pl and .pm files
+#
+# Revision 1.6  2002/07/25 13:40:31  tipaul
+# pod documenting the API.
+#
+# Revision 1.5  2002/07/24 16:11:37  tipaul
+# Now, the API...
+# Database.pm and Output.pm are almost not modified (var test...)
+#
+# Biblio.pm is almost completly rewritten.
+#
+# WHAT DOES IT ??? ==> END of Hitchcock suspens
+#
+# 1st, it does... nothing...
+# Every old API should be there. So if MARC-stuff is not done, the behaviour is EXACTLY the same (if there is no added bug, of course). So, if you use normal acquisition, you won't find anything new neither on screen or old-DB tables ...
+#
+# All old-API functions have been cloned. for example, the "newbiblio" sub, now has become :
+# * a "newbiblio" sub, with the same parameters. It just call a sub named OLDnewbiblio
+# * a "OLDnewbiblio" sub, which is a copy/paste of the previous newbiblio sub. Then, when you want to add the MARC-DB stuff, you can modify the newbiblio sub without modifying the OLDnewbiblio one. If we correct a bug in 1.2 in newbiblio, we can do the same in main branch by correcting OLDnewbiblio.
+# * The MARC stuff is usually done through a sub named MARCxxx where xxx is the same as OLDxxx. For example, newbiblio calls MARCnewbiblio. the MARCxxx subs use a MARC::Record as parameter.
+# The last thing to solve was to manage biblios through real MARC import : they must populate the old-db, but must populate the MARC-DB too, without loosing information (if we go from MARC::Record to old-data then back to MARC::Record, we loose A LOT OF ROWS). To do this, there are subs beginning by "NEWxxx" : they manage datas with MARC::Record datas. they call OLDxxx sub too (to populate old-DB), but MARCxxx subs too, with a complete MARC::Record ;-)
+#
+# In Biblio.pm, there are some subs that permits to build a old-style record from a MARC::Record, and the opposite. There is also a sub finding a MARC-bibid from a old-biblionumber and the opposite too.
+# Note we have decided with steve that a old-biblio <=> a MARC-Biblio.
 #
-
-END { }       # module clean-up code here (global destructor)