Add start of 2.3 installer. Remove obsolete? files conflicting with 2.3 installer
[koha.git] / C4 / Biblio.pm
index f320c5e..8f75db9 100644 (file)
@@ -1,141 +1,4 @@
 package C4::Biblio;
-# $Id$
-# $Log$
-# 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.
-#
-
 
 # Copyright 2000-2002 Katipo Communications
 #
@@ -166,41 +29,50 @@ use vars qw($VERSION @ISA @EXPORT);
 $VERSION = 0.01;
 
 @ISA = qw(Exporter);
+
 #
-# don't forget MARCxxx subs are here only for testing purposes. Should not be used
+# 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
-            &modnote &newsubject &newsubtitle
-            &modbiblio &checkitems
-            &newitems &modbibitem
-            &modsubtitle &modsubject &modaddauthor &moditem &countitems
-            &delitem &deletebiblioitem &delbiblio
-            &getitemtypes &getbiblio
-            &getbiblioitembybiblionumber
-            &getbiblioitem &getitemsbybiblioitem &isbnsearch
-            &skip
-            &newcompletebiblioitem
-
-            &MARCfind_oldbiblionumber_from_MARCbibid
-            &MARCfind_MARCbibid_from_oldbiblionumber
-               &MARCfind_marc_from_kohafield
-            &MARCfindsubfield
-            &MARCgettagslib
-
-               &NEWnewbiblio &NEWnewitem
-               &NEWmodbiblio &NEWmoditem
-
-            &MARCaddbiblio &MARCadditem
-            &MARCmodsubfield &MARCaddsubfield
-            &MARCmodbiblio &MARCmoditem
-            &MARCkoha2marcBiblio &MARCmarc2koha
-               &MARCkoha2marcItem &MARChtml2marc
-            &MARCgetbiblio &MARCgetitem
-            &MARCaddword &MARCdelword
- );
+  &updateBiblio &updateBiblioItem &updateItem
+  &itemcount &newbiblio &newbiblioitem
+  &modnote &newsubject &newsubtitle
+  &modbiblio &checkitems
+  &newitems &modbibitem
+  &modsubtitle &modsubject &modaddauthor &moditem &countitems
+  &delitem &deletebiblioitem &delbiblio
+  &getbiblio
+  &getbiblioitembybiblionumber
+  &getbiblioitem &getitemsbybiblioitem
+  &skip &getitemtypes
+  &newcompletebiblioitem
+
+  &MARCfind_oldbiblionumber_from_MARCbibid
+  &MARCfind_MARCbibid_from_oldbiblionumber
+  &MARCfind_marc_from_kohafield
+  &MARCfindsubfield
+  &MARCfind_frameworkcode
+  &MARCgettagslib
+
+  &NEWnewbiblio &NEWnewitem
+  &NEWmodbiblio &NEWmoditem
+  &NEWdelbiblio &NEWdelitem
+  &NEWmodbiblioframework
+
+  &MARCaddbiblio &MARCadditem
+  &MARCmodsubfield &MARCaddsubfield
+  &MARCmodbiblio &MARCmoditem
+  &MARCkoha2marcBiblio &MARCmarc2koha
+  &MARCkoha2marcItem &MARChtml2marc
+  &MARCgetbiblio &MARCgetitem
+  &MARCaddword &MARCdelword
+  &MARCdelsubfield
+  &char_decode
+  
+  &FindDuplicate
+  &DisplayISBN
+);
 
 #
 #
@@ -265,9 +137,10 @@ the db header $dbh is always passed as parameter to avoid over-DB connexion
 
 =over 4
 
-=item @tagslib = &MARCgettagslib($dbh,1|0);
+=item @tagslib = &MARCgettagslib($dbh,1|0,$itemtype);
 
 last param is 1 for liblibrarian and 0 for libopac
+$itemtype contains the itemtype framework reference. If empty or does not exist, the default one is used
 returns a hash with tag/subfield meaning
 =item ($tagfield,$tagsubfield) = &MARCfind_marc_from_kohafield($dbh,$kohafield);
 
@@ -310,13 +183,11 @@ adds a subfield in a biblio (in the MARC tables only).
 
 Returns a MARC::Record for the biblio $bibid.
 
-=item &MARCmodbiblio($dbh,$bibid,$delete,$record);
+=item &MARCmodbiblio($dbh,$bibid,$record,$frameworkcode,$delete);
 
 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...)
-
+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
@@ -333,6 +204,7 @@ MARCfindsubfieldid find a subfieldid for a bibid/tag/tagorder/subfield/subfieldo
 =item &MARCdelsubfield($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder);
 
 MARCdelsubfield delete a subfield for a bibid/tag/tagorder/subfield/subfieldorder
+If $subfieldorder is not set, delete all the $tag$subfield subfields 
 
 =item &MARCdelbiblio($dbh,$bibid);
 
@@ -357,104 +229,153 @@ 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,liblibrarian as lib from marc_tag_structure order by tagfield");
-       } else {
-               $sth=$dbh->prepare("select tagfield,libopac as lib from marc_tag_structure order by tagfield");
-       }
-       $sth->execute;
-       my ($lib,$tag,$res,$tab,$mandatory,$repeatable);
-       while ( ($tag,$lib,$tab) = $sth->fetchrow) {
-               $res->{$tag}->{lib}=$lib;
-               $res->{$tab}->{tab}="";
-       }
+    my ( $dbh, $forlibrarian, $frameworkcode ) = @_;
+    $frameworkcode = "" unless $frameworkcode;
+    my $sth;
+    my $libfield = ( $forlibrarian eq 1 ) ? 'liblibrarian' : 'libopac';
+
+    # check that framework exists
+    $sth =
+      $dbh->prepare(
+        "select count(*) from marc_tag_structure where frameworkcode=?");
+    $sth->execute($frameworkcode);
+    my ($total) = $sth->fetchrow;
+    $frameworkcode = "" unless ( $total > 0 );
+    $sth =
+      $dbh->prepare(
+"select tagfield,liblibrarian,libopac,mandatory,repeatable from marc_tag_structure where frameworkcode=? order by tagfield"
+    );
+    $sth->execute($frameworkcode);
+    my ( $liblibrarian, $libopac, $tag, $res, $tab, $mandatory, $repeatable );
+
+    while ( ( $tag, $liblibrarian, $libopac, $mandatory, $repeatable ) = $sth->fetchrow ) {
+        $res->{$tag}->{lib}        = ($forlibrarian or !$libopac)?$liblibrarian:$libopac;
+        $res->{$tab}->{tab}        = "";            # XXX
+        $res->{$tag}->{mandatory}  = $mandatory;
+        $res->{$tag}->{repeatable} = $repeatable;
+    }
 
-       if ($forlibrarian eq 1) {
-               $sth=$dbh->prepare("select tagfield,tagsubfield,liblibrarian as lib,tab, mandatory, repeatable,authorised_value,thesaurus_category,value_builder from marc_subfield_structure order by tagfield,tagsubfield");
-       } else {
-               $sth=$dbh->prepare("select tagfield,tagsubfield,libopac as lib,tab, mandatory, repeatable,authorised_value,thesaurus_category,value_builder from marc_subfield_structure order by tagfield,tagsubfield");
-       }
-       $sth->execute;
-
-       my $subfield;
-       my $authorised_value;
-       my $thesaurus_category;
-       my $value_builder;
-       while ( ($tag, $subfield, $lib, $tab, $mandatory, $repeatable,$authorised_value,$thesaurus_category,$value_builder) = $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;
-       }
-       return $res;
+    $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield,liblibrarian,libopac,tab, mandatory, repeatable,authorised_value,authtypecode,value_builder,kohafield,seealso,hidden,isurl,link from marc_subfield_structure where frameworkcode=? order by tagfield,tagsubfield"
+    );
+    $sth->execute($frameworkcode);
+
+    my $subfield;
+    my $authorised_value;
+    my $authtypecode;
+    my $value_builder;
+    my $kohafield;
+    my $seealso;
+    my $hidden;
+    my $isurl;
+       my $link;
+
+    while (
+        ( $tag,         $subfield,   $liblibrarian,   , $libopac,      $tab,
+        $mandatory,     $repeatable, $authorised_value, $authtypecode,
+        $value_builder, $kohafield,  $seealso,          $hidden,
+        $isurl,                        $link )
+        = $sth->fetchrow
+      )
+    {
+        $res->{$tag}->{$subfield}->{lib}              = ($forlibrarian or !$libopac)?$liblibrarian:$libopac;
+        $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}->{authtypecode}     = $authtypecode;
+        $res->{$tag}->{$subfield}->{value_builder}    = $value_builder;
+        $res->{$tag}->{$subfield}->{kohafield}        = $kohafield;
+        $res->{$tag}->{$subfield}->{seealso}          = $seealso;
+        $res->{$tag}->{$subfield}->{hidden}           = $hidden;
+        $res->{$tag}->{$subfield}->{isurl}            = $isurl;
+        $res->{$tag}->{$subfield}->{link}            = $link;
+    }
+    return $res;
 }
 
 sub MARCfind_marc_from_kohafield {
-    my ($dbh,$kohafield) = @_;
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
-    $sth->execute($kohafield);
-    my ($tagfield,$tagsubfield) = $sth->fetchrow;
-    return ($tagfield,$tagsubfield);
+    my ( $dbh, $kohafield,$frameworkcode ) = @_;
+    return 0, 0 unless $kohafield;
+       my $relations = C4::Context->marcfromkohafield;
+       return ($relations->{$frameworkcode}->{$kohafield}->[0],$relations->{$frameworkcode}->{$kohafield}->[1]);
 }
 
 sub MARCfind_oldbiblionumber_from_MARCbibid {
-    my ($dbh,$MARCbibid) = @_;
-    my $sth=$dbh->prepare("select biblionumber from marc_biblio where bibid=?");
+    my ( $dbh, $MARCbibid ) = @_;
+    my $sth =
+      $dbh->prepare("select biblionumber from marc_biblio where bibid=?");
     $sth->execute($MARCbibid);
     my ($biblionumber) = $sth->fetchrow;
     return $biblionumber;
 }
 
 sub MARCfind_MARCbibid_from_oldbiblionumber {
-    my ($dbh,$oldbiblionumber) = @_;
-    my $sth=$dbh->prepare("select bibid from marc_biblio where biblionumber=?");
+    my ( $dbh, $oldbiblionumber ) = @_;
+    my $sth =
+      $dbh->prepare("select bibid from marc_biblio where biblionumber=?");
     $sth->execute($oldbiblionumber);
     my ($bibid) = $sth->fetchrow;
     return $bibid;
 }
 
 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;
+       my ($dbh,$record,$biblionumber,$frameworkcode,$bibid) = @_;
+       my @fields=$record->fields();
+# 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,frameworkcode) values (now(),?,?)"
+        );
+        $sth->execute( $biblionumber, $frameworkcode );
+        $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]
-                                );
-       }
+        $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);
@@ -462,556 +383,802 @@ sub MARCadditem {
     my $sth = $dbh->prepare("select max(tagorder) from marc_subfield_table where bibid=?");
     $sth->execute($bibid);
     my ($fieldcount) = $sth->fetchrow;
+
     # 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]
-                                );
-                                warn "ADDING :$bibid,".
-                                $field->tag().
-                                $field->indicator(1).$field->indicator(2).",
-                                $fieldcount,
-                                $subfields[$subfieldcount][0],
-                                $subfieldcount+1,
-                                $subfields[$subfieldcount][1]";
-       }
+        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]
+            );
+        }
     }
     $dbh->do("unlock tables");
     return $bibid;
 }
 
 sub MARCaddsubfield {
-# Add a new subfield to a tag into the DB.
-       my ($dbh,$bibid,$tagid,$tag_indicator,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue) = @_;
-       # if not value, end of job, we do nothing
-       if (length($subfieldvalue) ==0) {
-               return;
-       }
-    if (not($subfieldcode)) {
-       $subfieldcode=' ';
+
+    # Add a new subfield to a tag into the DB.
+    my (
+        $dbh,      $bibid,        $tagid,         $tag_indicator,
+        $tagorder, $subfieldcode, $subfieldorder, $subfieldvalues
+      )
+      = @_;
+         return unless $subfieldvalues;
+# warn "$tagid / $subfieldcode / $subfieldvalues";
+    # if not value, end of job, we do nothing
+#     if ( length($subfieldvalues) == 0 ) {
+#         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,tag_indicator,subfieldcode,subfieldorder,valuebloblink) values (?,?,?,?,?,?,?)");
-               if ($tagid<100) {
-               $sth->execute($bibid,'0'.$tagid,$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$res);
-               } else {
-               $sth->execute($bibid,$tagid,$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$res);
-               }
-               if ($sth->errstr) {
-               print STDERR "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,$tagid,$tagorder,$tag_indicator,$subfieldcode,$subfieldorder,$subfieldvalue);
-               if ($sth->errstr) {
-               print STDERR "ERROR ==> insert into marc_subfield_table (bibid,tag,tagorder,tag_indicator,subfieldcode,subfieldorder,subfieldvalue) values ($bibid,$tagid,$tagorder,$tag_indicator,$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)=@_;
+
+    # 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
+#      warn "". $bidid;
+
+    #---- TODO : the leader is missing
+    $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=?");
+                        "
+    );
+    my $sth2 =
+      $dbh->prepare(
+        "select subfieldvalue from marc_blob_subfield where blobidlink=?");
     $sth->execute($bibid);
-    my $prevtagorder=1;
-    my $prevtag='  ';
+    my $prevtagorder = 1;
+    my $prevtag      = 'XXX';
     my $previndicator;
-    my %subfieldlist={};
-    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'};
-               }
-#              warn "$row->{bibid} = $row->{tag} - $row->{subfieldcode}";
-               if ($row->{tagorder} ne $prevtagorder) {
-                   if (length($prevtag) <3) {
-                               $prevtag = "0".$prevtag;
-                       }
-                       $previndicator.="  ";
-#                      warn "NEW : subfieldcode : $prevtag";
-                       my $field = MARC::Field->new( $prevtag, substr($previndicator,0,1), substr($previndicator,1,1), %subfieldlist);
-#                      warn $field->as_formatted();
-                       $record->add_fields($field);
-                       $prevtagorder=$row->{tagorder};
-                       $prevtag = $row->{tag};
-                       $previndicator=$row->{tag_indicator};
-                       %subfieldlist={};
-                       %subfieldlist->{$row->{'subfieldcode'}} = $row->{'subfieldvalue'};
-               } else {
-                       warn "subfieldcode : $row->{'subfieldcode'} / value : $row->{'subfieldvalue'}, tag : $row->{tag}";
-                       if (%subfieldlist->{$row->{'subfieldcode'}}) {
-                               %subfieldlist->{$row->{'subfieldcode'}}.='|';
-                       }
-                       %subfieldlist->{$row->{'subfieldcode'}} .= $row->{'subfieldvalue'};
-                       $prevtag= $row->{tag};
-                       $previndicator=$row->{tag_indicator};
-               }
-       }
-       # the last has not been included inside the loop... do it now !
-       my $field = MARC::Field->new( $prevtag, "", "", %subfieldlist);
-       $record->add_fields($field);
-       return $record;
+    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};
+        }
+    }
+
+    # 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;
 }
+
 sub MARCgetitem {
-# Returns MARC::Record of the biblio passed in parameter.
-    my ($dbh,$bibid,$itemnumber)=@_;
+
+    # Returns MARC::Record of the biblio passed in parameter.
+    my ( $dbh, $bibid, $itemnumber ) = @_;
     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);
+
+    # 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
+
+    #---- 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 subfieldcode,subfieldorder
-                        ");
-       $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'};
-       }
-       if ($record->field($row->{'tag'})) {
-           my $field;
+                        "
+    );
+    $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'};
+        }
+        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);
-       }
+            #--- 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);
+        }
 
     }
     return $record;
 }
 
 sub MARCmodbiblio {
-    my ($dbh,$record,$bibid,$itemnumber,$delete)=@_;
-    my $oldrecord=&MARCgetbiblio($dbh,$bibid);
-    warn "OLD : ".$oldrecord->as_formatted();
-    warn "----------------------------------\nNEW : ".$record->as_formatted();
-    warn "\n";
-# if nothing to change, don't waste time...
-    if ($oldrecord eq $record) {
-#    warn "NOTHING TO CHANGE";
-       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 the 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 {
-# FIXME ???
-               }
-           }
+       my ($dbh,$bibid,$record,$frameworkcode,$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,$frameworkcode,$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 not (tagsubfield like \"$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) {
-#              warn "nothing to change";
-               return;
-       }
-       warn "MARCmoditem : ".$record->as_formatted;
-       warn "OLD : ".$oldrecord->as_formatted;
-
-       # 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]);
-                               } else {
-#FIXME ???
-                                       warn "nothing to change : ".$oldfield->subfield(@$subfield[0]);
-                               }
-                       }
-               }
-       }
-       warn "-----------------------";
+       my $biblionumber = MARCfind_oldbiblionumber_from_MARCbibid($dbh,$bibid);
+       &MARCdelitem($dbh,$bibid,$itemnumber);
+       &MARCadditem($dbh,$record,$biblionumber);
 }
 
-
 sub MARCmodsubfield {
-# Subroutine changes a subfield value given a subfieldid.
-    my ($dbh, $subfieldid, $subfieldvalue )=@_;
+
+    # Subroutine changes a subfield value given a subfieldid.
+    my ( $dbh, $subfieldid, $subfieldvalue ) = @_;
     $dbh->do("lock tables marc_blob_subfield WRITE,marc_subfield_table WRITE");
-    my $sth1=$dbh->prepare("select valuebloblink from marc_subfield_table where subfieldid=?");
+    my $sth1 =
+      $dbh->prepare(
+        "select valuebloblink from marc_subfield_table where subfieldid=?");
     $sth1->execute($subfieldid);
-    my ($oldvaluebloblink)=$sth1->fetchrow;
+    my ($oldvaluebloblink) = $sth1->fetchrow;
     $sth1->finish;
     my $sth;
+
     # if too long, use a bloblink
-    if (length($subfieldvalue)>255 ) {
-       # if already a bloblink, update it, otherwise, insert a new one.
-       if ($oldvaluebloblink) {
-           $sth=$dbh->prepare("update marc_blob_subfield set subfieldvalue=? where blobidlink=?");
-           $sth->execute($subfieldvalue,$oldvaluebloblink);
-       } else {
-           $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("update marc_subfield_table set subfieldvalue=null, valuebloblink=$res where subfieldid=?");
-           $sth->execute($subfieldid);
-       }
-    } else {
-       # note this can leave orphan bloblink. Not a big problem, but we should build somewhere a orphan deleting script...
-       $sth=$dbh->prepare("update marc_subfield_table set subfieldvalue=?,valuebloblink=null where subfieldid=?");
-       $sth->execute($subfieldvalue, $subfieldid);
+    if ( length($subfieldvalue) > 255 ) {
+
+        # if already a bloblink, update it, otherwise, insert a new one.
+        if ($oldvaluebloblink) {
+            $sth =
+              $dbh->prepare(
+"update marc_blob_subfield set subfieldvalue=? where blobidlink=?"
+            );
+            $sth->execute( $subfieldvalue, $oldvaluebloblink );
+        }
+        else {
+            $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(
+"update marc_subfield_table set subfieldvalue=null, valuebloblink=? where subfieldid=?"
+            );
+            $sth->execute( $res, $subfieldid );
+        }
+    }
+    else {
+
+# note this can leave orphan bloblink. Not a big problem, but we should build somewhere a orphan deleting script...
+        $sth =
+          $dbh->prepare(
+"update marc_subfield_table set subfieldvalue=?,valuebloblink=null where subfieldid=?"
+        );
+        $sth->execute( $subfieldvalue, $subfieldid );
     }
     $dbh->do("unlock tables");
     $sth->finish;
-    $sth=$dbh->prepare("select bibid,tag,tagorder,subfieldcode,subfieldid,subfieldorder from marc_subfield_table where subfieldid=?");
+    $sth =
+      $dbh->prepare(
+"select bibid,tag,tagorder,subfieldcode,subfieldid,subfieldorder from marc_subfield_table where subfieldid=?"
+    );
     $sth->execute($subfieldid);
-    my ($bibid,$tagid,$tagorder,$subfieldcode,$x,$subfieldorder) = $sth->fetchrow;
-    $subfieldid=$x;
-    &MARCdelword($dbh,$bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder);
-    &MARCaddword($dbh,$bibid,$tagid,$tagorder,$subfieldcode,$subfieldorder,$subfieldvalue);
-    return($subfieldid, $subfieldvalue);
+    my ( $bibid, $tagid, $tagorder, $subfieldcode, $x, $subfieldorder ) =
+      $sth->fetchrow;
+    $subfieldid = $x;
+    &MARCdelword( $dbh, $bibid, $tagid, $tagorder, $subfieldcode,
+        $subfieldorder );
+    &MARCaddword(
+        $dbh,          $bibid,         $tagid,       $tagorder,
+        $subfieldcode, $subfieldorder, $subfieldvalue
+    );
+    return ( $subfieldid, $subfieldvalue );
 }
 
 sub MARCfindsubfield {
-    my ($dbh,$bibid,$tag,$subfieldcode,$subfieldorder,$subfieldvalue) = @_;
-    my $resultcounter=0;
+    my ( $dbh, $bibid, $tag, $subfieldcode, $subfieldorder, $subfieldvalue ) =
+      @_;
+    my $resultcounter = 0;
     my $subfieldid;
     my $lastsubfieldid;
-    my $query="select subfieldid from marc_subfield_table where bibid=? and tag=? and subfieldcode=?";
+    my $query =
+"select subfieldid from marc_subfield_table where bibid=? and tag=? and subfieldcode=?";
+    my @bind_values = ( $bibid, $tag, $subfieldcode );
     if ($subfieldvalue) {
-       $query .= " and subfieldvalue=".$dbh->quote($subfieldvalue);
-    } else {
-       if ($subfieldorder<1) {
-           $subfieldorder=1;
-       }
-       $query .= " and subfieldorder=$subfieldorder";
+        $query .= " and subfieldvalue=?";
+        push ( @bind_values, $subfieldvalue );
+    }
+    else {
+        if ( $subfieldorder < 1 ) {
+            $subfieldorder = 1;
+        }
+        $query .= " and subfieldorder=?";
+        push ( @bind_values, $subfieldorder );
     }
-    my $sti=$dbh->prepare($query);
-    $sti->execute($bibid,$tag, $subfieldcode);
-    while (($subfieldid) = $sti->fetchrow) {
-       $resultcounter++;
-       $lastsubfieldid=$subfieldid;
+    my $sti = $dbh->prepare($query);
+    $sti->execute(@bind_values);
+    while ( ($subfieldid) = $sti->fetchrow ) {
+        $resultcounter++;
+        $lastsubfieldid = $subfieldid;
     }
-    if ($resultcounter>1) {
-       # Error condition.  Values given did not resolve into a unique record.  Don't know what to edit
-       # should rarely occur (only if we use subfieldvalue with a value that exists twice, which is strange)
-       return -1;
-    } else {
-       return $lastsubfieldid;
+    if ( $resultcounter > 1 ) {
+
+# Error condition.  Values given did not resolve into a unique record.  Don't know what to edit
+# should rarely occur (only if we use subfieldvalue with a value that exists twice, which is strange)
+        return -1;
+    }
+    else {
+        return $lastsubfieldid;
     }
 }
 
 sub MARCfindsubfieldid {
-       my ($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder) = @_;
-       my $sth=$dbh->prepare("select subfieldid from marc_subfield_table
+    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
+                                       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;
-       }
+                                       and subfieldcode=?"
+        );
+        $sth->execute( $bibid, $tag, $tagorder, $subfield );
+        ($res) = $sth->fetchrow;
+    }
     return $res;
 }
 
-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'
-                       and subfieldcode='$subfield' and subfieldorder='$subfieldorder
-                       ");
+sub MARCfind_frameworkcode {
+    my ( $dbh, $bibid ) = @_;
+    my $sth =
+      $dbh->prepare("select frameworkcode from marc_biblio where bibid=?");
+    $sth->execute($bibid);
+    my ($frameworkcode) = $sth->fetchrow;
+    return $frameworkcode;
 }
 
-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 MARCdelsubfield {
+
+    # delete a subfield for $bibid / tag / tagorder / subfield / subfieldorder
+    my ( $dbh, $bibid, $tag, $tagorder, $subfield, $subfieldorder ) = @_;
+       if ($subfieldorder) {
+               $dbh->do( "delete from marc_subfield_table where bibid='$bibid' and
+                               tag='$tag' and tagorder='$tagorder'
+                               and subfieldcode='$subfield' and subfieldorder='$subfieldorder'
+                               "
+               );
+               $dbh->do( "delete from marc_word where bibid='$bibid' and
+                               tagsubfield='$tag$subfield' and tagorder='$tagorder'
+                               and subfieldorder='$subfieldorder'
+                               "
+               );
+       } else {
+               $dbh->do( "delete from marc_subfield_table where bibid='$bibid' and
+                               tag='$tag' and tagorder='$tagorder'
+                               and subfieldcode='$subfield'"
+               );
+               $dbh->do( "delete from marc_word where bibid='$bibid' and
+                               tagsubfield='$tag$subfield' and tagorder='$tagorder'"
+               );
+       }
 }
 
 sub MARCkoha2marcBiblio {
-# this function builds partial MARC::Record from the old koha-DB fields
-    my ($dbh,$biblionumber,$biblioitemnumber) = @_;
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+
+    # this function builds partial MARC::Record from the old koha-DB fields
+    my ( $dbh, $biblionumber, $biblioitemnumber ) = @_;
+    my $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+    );
     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=?");
-       $sth2->execute($biblionumber);
-       my $row=$sth2->fetchrow_hashref;
-       my $code;
-       foreach $code (keys %$row) {
-           if ($row->{$code}) {
-               &MARCkoha2marcOnefield($sth,$record,"biblio.".$code,$row->{$code});
-           }
-       }
+
+    #--- 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=?"
+        );
+        $sth2->execute($biblionumber);
+        my $row = $sth2->fetchrow_hashref;
+        my $code;
+        foreach $code ( keys %$row ) {
+            if ( $row->{$code} ) {
+                &MARCkoha2marcOnefield( $sth, $record, "biblio." . $code,
+                    $row->{$code}, '');
+            }
+        }
     }
-#--- if biblioitem, then retrieve old-style koha data
-    if ($biblioitemnumber>0) {
-       my $sth2=$dbh->prepare(" SELECT biblioitemnumber,biblionumber,volume,number,classification,
+
+    #--- if biblioitem, then retrieve old-style koha data
+    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);
-       my $row=$sth2->fetchrow_hashref;
-       my $code;
-       foreach $code (keys %$row) {
-           if ($row->{$code}) {
-               &MARCkoha2marcOnefield($sth,$record,"biblioitems.".$code,$row->{$code});
-           }
-       }
+                                       WHERE biblioitemnumber=?
+                                       "
+        );
+        $sth2->execute($biblioitemnumber);
+        my $row = $sth2->fetchrow_hashref;
+        my $code;
+        foreach $code ( keys %$row ) {
+            if ( $row->{$code} ) {
+                &MARCkoha2marcOnefield( $sth, $record, "biblioitems." . $code,
+                    $row->{$code},'' );
+            }
+        }
     }
-    return $record;
-# TODO : retrieve notes, additionalauthors
-}
+
+    # 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'},'' );
+    }
+    $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'},'' );
+    }
+    $sth2 =
+      $dbh->prepare(
+        " SELECT subtitle FROM bibliosubtitle WHERE biblionumber=?");
+    $sth2->execute($biblionumber);
+    while ( my $row = $sth2->fetchrow_hashref ) {
+        &MARCkoha2marcOnefield( $sth, $record, "bibliosubtitle.subtitle",
+            $row->{'subtitle'},'' );
+    }
+    return $record;
+}
 
 sub MARCkoha2marcItem {
-# this function builds partial MARC::Record from the old koha-DB fields
-    my ($dbh,$biblionumber,$itemnumber) = @_;
-#    my $dbh=&C4Connect;
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+
+    # this function builds partial MARC::Record from the old koha-DB fields
+    my ( $dbh, $biblionumber, $itemnumber ) = @_;
+
+    #    my $dbh=&C4Connect;
+    my $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+    );
     my $record = MARC::Record->new();
-#--- if item, then retrieve old-style koha data
-    if ($itemnumber>0) {
-#      print STDERR "prepare $biblionumber,$itemnumber\n";
-       my $sth2=$dbh->prepare("SELECT itemnumber,biblionumber,multivolumepart,biblioitemnumber,barcode,dateaccessioned,
+
+    #--- if item, then retrieve old-style koha data
+    if ( $itemnumber > 0 ) {
+
+        #      print STDERR "prepare $biblionumber,$itemnumber\n";
+        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,
+                                               datelastseen,multivolume,stack,notforloan,itemlost,wthdrawn,itemcallnumber,issues,renewals,
                                        reserves,restricted,binding,itemnotes,holdingbranch,timestamp
                                        FROM items
-                                       WHERE itemnumber=?");
-       $sth2->execute($itemnumber);
-       my $row=$sth2->fetchrow_hashref;
-       my $code;
-       foreach $code (keys %$row) {
-           if ($row->{$code}) {
-               &MARCkoha2marcOnefield($sth,$record,"items.".$code,$row->{$code});
-           }
-       }
+                                       WHERE itemnumber=?"
+        );
+        $sth2->execute($itemnumber);
+        my $row = $sth2->fetchrow_hashref;
+        my $code;
+        foreach $code ( keys %$row ) {
+            if ( $row->{$code} ) {
+                &MARCkoha2marcOnefield( $sth, $record, "items." . $code,
+                    $row->{$code},'' );
+            }
+        }
     }
     return $record;
-# TODO : retrieve notes, additionalauthors
 }
 
 sub MARCkoha2marcSubtitle {
-# this function builds partial MARC::Record from the old koha-DB fields
-    my ($dbh,$bibnum,$subtitle) = @_;
-    my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+
+    # this function builds partial MARC::Record from the old koha-DB fields
+    my ( $dbh, $bibnum, $subtitle ) = @_;
+    my $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+    );
     my $record = MARC::Record->new();
-    &MARCkoha2marcOnefield($sth,$record,"bibliosubtitle.subtitle",$subtitle);
+    &MARCkoha2marcOnefield( $sth, $record, "bibliosubtitle.subtitle",
+        $subtitle,'' );
     return $record;
 }
 
 sub MARCkoha2marcOnefield {
-    my ($sth,$record,$kohafieldname,$value)=@_;
+    my ( $sth, $record, $kohafieldname, $value,$frameworkcode ) = @_;
     my $tagfield;
     my $tagsubfield;
-    $sth->execute($kohafieldname);
-    if (($tagfield,$tagsubfield)=$sth->fetchrow) {
-       if ($record->field($tagfield)) {
-           my $tag =$record->field($tagfield);
-           if ($tag) {
-               $tag->add_subfields($tagsubfield,$value);
-               $record->delete_field($tag);
-               $record->add_fields($tag);
-           }
-       } else {
-           $record->add_fields($tagfield," "," ",$tagsubfield => $value);
-       }
+    $sth->execute($frameworkcode,$kohafieldname);
+    if ( ( $tagfield, $tagsubfield ) = $sth->fetchrow ) {
+        if ( $record->field($tagfield) ) {
+            my $tag = $record->field($tagfield);
+            if ($tag) {
+                $tag->add_subfields( $tagsubfield, $value );
+                $record->delete_field($tag);
+                $record->add_fields($tag);
+            }
+        }
+        else {
+            $record->add_fields( $tagfield, " ", " ", $tagsubfield => $value );
+        }
     }
     return $record;
 }
 
 sub MARChtml2marc {
        my ($dbh,$rtags,$rsubfields,$rvalues,%indicators) = @_;
-       my $prevtag = @$rtags[0];
+       my $prevtag = -1;
        my $record = MARC::Record->new();
-       my %subfieldlist={};
+#      my %subfieldlist=();
+       my $prevvalue; # if tag <10
+       my $field; # if tag >=10
        for (my $i=0; $i< @$rtags; $i++) {
                # rebuild MARC::Record
+#                      warn "0=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ";
                if (@$rtags[$i] ne $prevtag) {
-                       if ($prevtag<10) {
-                               $prevtag='0'.10;
+                       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];
+                               undef $field;
+                       } else {
+                               undef $prevvalue;
+                               $field = MARC::Field->new( (sprintf "%03s",@$rtags[$i]), substr($indicators{@$rtags[$i]},0,1),substr($indicators{@$rtags[$i]},1,1), @$rsubfields[$i] => @$rvalues[$i]);
+#                      warn "1=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ".$field->as_formatted;
                        }
-                       $indicators{$prevtag}.='  ';
-                       my $field = MARC::Field->new( $prevtag, substr($indicators{$prevtag},0,1),substr($indicators{$prevtag},1,1), %subfieldlist);
-                       $record->add_fields($field);
                        $prevtag = @$rtags[$i];
-                       %subfieldlist={};
-                       %subfieldlist->{@$rsubfields[$i]} = @$rvalues[$i];
-                       warn " ==>@$rsubfields[$i]} = @$rvalues[$i];";
                } else {
-#                      if (%subfieldlist->{@$rsubfields[$i]}) {
-#                              %subfieldlist->{@$rsubfields[$i]} .= '|';
-#                      }
-                       %subfieldlist->{@$rsubfields[$i]} .=@$rvalues[$i];
+                       if (@$rtags[$i] <10) {
+                               $prevvalue=@$rvalues[$i];
+                       } else {
+                               if (length(@$rvalues[$i])>0) {
+                                       $field->add_subfields(@$rsubfields[$i] => @$rvalues[$i]);
+#                      warn "2=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ".$field->as_formatted;
+                               }
+                       }
                        $prevtag= @$rtags[$i];
-                       warn " ==>@$rsubfields[$i]} ,= @$rvalues[$i];";
                }
        }
        # the last has not been included inside the loop... do it now !
-       my $field = MARC::Field->new( $prevtag, "", "", %subfieldlist);
-       $record->add_fields($field);
+       $record->add_fields($field) if $field;
+#      warn "HTML2MARC=".$record->as_formatted;
        return $record;
 }
 
 sub MARCmarc2koha {
-       my ($dbh,$record) = @_;
-       my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where kohafield=?");
+       my ($dbh,$record,$frameworkcode) = @_;
+       my $sth=$dbh->prepare("select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and 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);
+               $result=&MARCmarc2kohaOneField($sth,"biblio",$field,$record,$result,$frameworkcode);
        }
        $sth2=$dbh->prepare("SHOW COLUMNS from biblioitems");
        $sth2->execute;
        while (($field)=$sth2->fetchrow) {
-               $result=&MARCmarc2kohaOneField($sth,"biblioitems",$field,$record,$result);
+               if ($field eq 'notes') { $field = 'bnotes'; }
+               $result=&MARCmarc2kohaOneField($sth,"biblioitems",$field,$record,$result,$frameworkcode);
        }
        $sth2=$dbh->prepare("SHOW COLUMNS from items");
        $sth2->execute;
        while (($field)=$sth2->fetchrow) {
-               $result = &MARCmarc2kohaOneField($sth,"items",$field,$record,$result);
+               $result=&MARCmarc2kohaOneField($sth,"items",$field,$record,$result,$frameworkcode);
        }
        # additional authors : specific
-       $result = &MARCmarc2kohaOneField($sth,"additionalauthors","additionalauthors",$record,$result);
+       $result = &MARCmarc2kohaOneField($sth,"bibliosubtitle","subtitle",$record,$result,$frameworkcode);
+       $result = &MARCmarc2kohaOneField($sth,"additionalauthors","additionalauthors",$record,$result,$frameworkcode);
+# 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
+       $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 {
+
 # 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 ( $sth, $kohatable, $kohafield, $record, $result,$frameworkcode ) = @_;
+    #    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);
-           }
-       }
+    ( $tagfield, $subfield ) = MARCfind_marc_from_kohafield("",$kohatable.".".$kohafield,$frameworkcode);
+    foreach my $field ( $record->field($tagfield) ) {
+        if ( $field->subfields ) {
+            my @subfields = $field->subfields();
+            foreach my $subfieldcount ( 0 .. $#subfields ) {
+                               if ($subfields[$subfieldcount][0] eq $subfield) {
+                                       if ( $result->{$kohafield} ) {
+                                               $result->{$kohafield} .= " | " . $subfields[$subfieldcount][1];
+                                       }
+                                       else {
+                                               $result->{$kohafield} = $subfields[$subfieldcount][1];
+                                       }
+                               }
+                       }
+        }
     }
+#      warn "OneField for $kohatable.$kohafield and $frameworkcode=> $tagfield, $subfield";
     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;
-    my @words = split / /,$sentence;
-    my $stopwords= C4::Context->stopwords;
-    my $sth=$dbh->prepare("insert into marc_word (bibid, tag, tagorder, subfieldid, subfieldorder, word, sndx_word)
-                       values (?,?,?,?,?,?,soundex(?))");
+
+    # split a subfield string and adds it into the word table.
+    # removes stopwords
+    my (
+        $dbh,        $bibid,         $tag,    $tagorder,
+        $subfieldid, $subfieldorder, $sentence
+      )
+      = @_;
+    $sentence =~ s/(\.|\?|\:|\!|\'|,|\-|\"|\(|\)|\[|\]|\{|\}|\/)/ /g;
+    my @words = split / /, $sentence;
+    my $stopwords = C4::Context->stopwords;
+    my $sth       =
+      $dbh->prepare(
+"insert into marc_word (bibid, tagsubfield, tagorder, subfieldorder, word, sndx_word)
+                       values (?,concat(?,?),?,?,?,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)})) {
-           $sth->execute($bibid,$tag,$tagorder,$subfieldid,$subfieldorder,$word,$word);
+# we record only words one char long and not in stopwords hash
+       if (length($word)>=1 and !($stopwords->{uc($word)})) {
+           $sth->execute($bibid,$tag,$subfieldid,$tagorder,$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, tagsubfield, tagorder, subfieldorder, word, sndx_word) values ($bibid,concat($tag,$subfieldid),$tagorder,$subfieldorder,$word,soundex($word))\n";
            }
        }
     }
 }
 
 sub MARCdelword {
+
 # delete words. this sub deletes all the words from a sentence. a subfield modif is done by a delete then a add
-    my ($dbh,$bibid,$tag,$tagorder,$subfield,$subfieldorder) = @_;
-    my $sth=$dbh->prepare("delete from marc_word where bibid=? and tag=? and tagorder=? and subfieldid=? and subfieldorder=?");
-    $sth->execute($bibid,$tag,$tagorder,$subfield,$subfieldorder);
+    my ( $dbh, $bibid, $tag, $tagorder, $subfield, $subfieldorder ) = @_;
+    my $sth =
+      $dbh->prepare(
+"delete from marc_word where bibid=? and tagsubfield=concat(?,?) and tagorder=? and subfieldorder=?"
+    );
+    $sth->execute( $bibid, $tag, $subfield, $tagorder, $subfieldorder );
 }
 
 #
@@ -1023,98 +1190,196 @@ sub MARCdelword {
 # it's used with marcimport, and marc management tools
 #
 
-
-=item (oldbibnum,$oldbibitemnum) = NEWnewbibilio($dbh,$MARCRecord,$oldbiblio,$oldbiblioitem);
+=item ($bibid,$oldbibnum,$oldbibitemnum) = NEWnewbibilio($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 NEWnewitem($dbh,$olditem);
+=item NEWnewitem($dbh, $record,$bibid);
 
-adds an item in the db. $olditem is a old-db hash.
+adds an item in the db.
 
 =cut
 
 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 ( $dbh, $record, $frameworkcode ) = @_;
     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);
-       $olddata->{'biblionumber'} = $oldbibnum;
-       $oldbibitemnum = OLDnewbiblioitem($dbh,$olddata);
+    my $olddata = MARCmarc2koha( $dbh, $record,$frameworkcode );
+    $oldbibnum = OLDnewbiblio( $dbh, $olddata );
+    $olddata->{'biblionumber'} = $oldbibnum;
+    $oldbibitemnum = OLDnewbiblioitem( $dbh, $olddata );
+
+    # search subtiles, addiauthors and subjects
+    my ( $tagfield, $tagsubfield ) =
+      MARCfind_marc_from_kohafield( $dbh, "additionalauthors.author",$frameworkcode );
+    my @addiauthfields = $record->field($tagfield);
+    foreach my $addiauthfield (@addiauthfields) {
+        my @addiauthsubfields = $addiauthfield->subfield($tagsubfield);
+        foreach my $subfieldcount ( 0 .. $#addiauthsubfields ) {
+            OLDmodaddauthor( $dbh, $oldbibnum,
+                $addiauthsubfields[$subfieldcount] );
+        }
+    }
+    ( $tagfield, $tagsubfield ) =
+      MARCfind_marc_from_kohafield( $dbh, "bibliosubtitle.subtitle",$frameworkcode );
+    my @subtitlefields = $record->field($tagfield);
+    foreach my $subtitlefield (@subtitlefields) {
+        my @subtitlesubfields = $subtitlefield->subfield($tagsubfield);
+        foreach my $subfieldcount ( 0 .. $#subtitlesubfields ) {
+            OLDnewsubtitle( $dbh, $oldbibnum,
+                $subtitlesubfields[$subfieldcount] );
+        }
     }
-# 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.
+    ( $tagfield, $tagsubfield ) =
+      MARCfind_marc_from_kohafield( $dbh, "bibliosubject.subject",$frameworkcode );
+    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=?");
+    # (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;
+    ( my $tagfield1, my $tagsubfield1 ) = $sth->fetchrow;
     $sth->execute("biblioitems.biblioitemnumber");
-    (my $tagfield2, my $tagsubfield2) = $sth->fetchrow;
-    if ($tagfield1 != $tagfield2) {
-       print STDERR "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 $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 $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 );
+    my $bibid = MARCaddbiblio( $dbh, $record, $oldbibnum, $frameworkcode );
+    return ( $bibid, $oldbibnum, $oldbibitemnum );
 }
 
+sub NEWmodbiblioframework {
+       my ($dbh,$bibid,$frameworkcode) =@_;
+       my $sth = $dbh->prepare("Update marc_biblio SET frameworkcode=? WHERE bibid=$bibid");
+       $sth->execute($frameworkcode);
+       return 1;
+}
 sub NEWmodbiblio {
-my ($dbh,$record,$bibid) =@_;
-&MARCmodbiblio($dbh,$record,$bibid);
-my $oldbiblio = MARCmarc2koha($dbh,$record);
-my $oldbiblionumber = OLDmodbiblio($dbh,$oldbiblio);
-OLDmodbibitem($dbh,$oldbiblio);
-return 1;
+       my ($dbh,$record,$bibid,$frameworkcode) =@_;
+       $frameworkcode="" unless $frameworkcode;
+       &MARCmodbiblio($dbh,$bibid,$record,$frameworkcode,0);
+       my $oldbiblio = MARCmarc2koha($dbh,$record,$frameworkcode);
+       my $oldbiblionumber = OLDmodbiblio($dbh,$oldbiblio);
+       OLDmodbibitem($dbh,$oldbiblio);
+       # now, modify addi authors, subject, addititles.
+       my ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"additionalauthors.author",$frameworkcode);
+       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",$frameworkcode);
+       my @subtitlefields = $record->field($tagfield);
+       foreach my $subtitlefield (@subtitlefields) {
+               my @subtitlesubfields = $subtitlefield->subfield($tagsubfield);
+               # delete & create subtitle again because OLDmodsubtitle can't handle new subtitles
+               # between 2 modifs
+               $dbh->do("delete from bibliosubtitle where biblionumber=$oldbiblionumber");
+               foreach my $subfieldcount (0..$#subtitlesubfields) {
+                       foreach my $subtit(split /\||#/,$subtitlesubfields[$subfieldcount]) {
+                               OLDnewsubtitle($dbh,$oldbiblionumber,$subtit);
+                       }
+               }
+       }
+       ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield($dbh,"bibliosubject.subject",$frameworkcode);
+       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'});
+    my ( $dbh, $record, $bibid ) = @_;
+
+    # add item in old-DB
+       my $frameworkcode=MARCfind_frameworkcode($dbh,$bibid);
+    my $item = &MARCmarc2koha( $dbh, $record,$frameworkcode );
+    # 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.
+    $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+    );
+    &MARCkoha2marcOnefield( $sth, $record, "items.itemnumber", $itemnumber,$frameworkcode );
+
+    # 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);
+    my ( $dbh, $record, $bibid, $itemnumber, $delete ) = @_;
+    
+       &MARCmoditem( $dbh, $record, $bibid, $itemnumber, $delete );
+       my $frameworkcode=MARCfind_frameworkcode($dbh,$bibid);
+    my $olditem = MARCmarc2koha( $dbh, $record,$frameworkcode );
+    OLDmoditem( $dbh, $olditem );
+}
+
+sub NEWdelitem {
+    my ( $dbh, $bibid, $itemnumber ) = @_;
+    my $biblio = &MARCfind_oldbiblionumber_from_MARCbibid( $dbh, $bibid );
+    &OLDdelitem( $dbh, $itemnumber );
+    &MARCdelitem( $dbh, $bibid, $itemnumber );
 }
 
 #
@@ -1188,196 +1453,185 @@ delete a biblio
 =cut
 
 sub OLDnewbiblio {
-  my ($dbh,$biblio) = @_;
-#  my $dbh    = &C4Connect;
-  my $query  = "Select max(biblionumber) from biblio";
-  my $sth    = $dbh->prepare($query);
-  $sth->execute;
-  my $data   = $sth->fetchrow_arrayref;
-  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'}";
-
-  $sth = $dbh->prepare($query);
-  $sth->execute;
-
-  $sth->finish;
-#  $dbh->disconnect;
-  return($bibnum);
+    my ( $dbh, $biblio ) = @_;
+
+    #  my $dbh    = &C4Connect;
+    my $sth = $dbh->prepare("Select max(biblionumber) from biblio");
+    $sth->execute;
+    my $data   = $sth->fetchrow_arrayref;
+    my $bibnum = $$data[0] + 1;
+    my $series = 0;
+
+    if ( $biblio->{'seriestitle'} ) { $series = 1 }
+    $sth->finish;
+    $sth =
+      $dbh->prepare(
+"insert into biblio set biblionumber  = ?, title = ?, author = ?, copyrightdate = ?, serial = ?, seriestitle = ?, notes = ?, abstract = ?, unititle = ?"
+    );
+    $sth->execute(
+        $bibnum,             $biblio->{'title'},
+        $biblio->{'author'}, $biblio->{'copyrightdate'},
+        $biblio->{'serial'},             $biblio->{'seriestitle'},
+        $biblio->{'notes'},  $biblio->{'abstract'},
+               $biblio->{'unititle'},
+    );
+
+    $sth->finish;
+
+    #  $dbh->disconnect;
+    return ($bibnum);
 }
 
 sub OLDmodbiblio {
-    my ($dbh,$biblio) = @_;
-#  my $dbh   = C4Connect;
+    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;
+    $query = "";
+    $sth   =
+      $dbh->prepare(
+"Update biblio set title = ?, author = ?, abstract = ?, copyrightdate = ?, seriestitle = ?, serial = ?, unititle = ?, notes = ? where biblionumber = ?"
+    );
+    $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
+    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;
-} # sub modsubtitle
-
+    my ( $dbh, $bibnum, $subtitle ) = @_;
+    my $sth =
+      $dbh->prepare(
+        "update bibliosubtitle set subtitle = ? where biblionumber = ?");
+    $sth->execute( $subtitle, $bibnum );
+    $sth->finish;
+}    # sub modsubtitle
 
 sub OLDmodaddauthor {
-    my ($dbh,$bibnum, $author) = @_;
-#    my $dbh   = C4Connect;
-    my $query = "Delete from additionalauthors where biblionumber = $bibnum";
-    my $sth = $dbh->prepare($query);
-
-    $sth->execute;
-    $sth->finish;
+    my ( $dbh, $bibnum, @authors ) = @_;
 
-    if ($author ne '') {
-        $query = "Insert into additionalauthors set
-                        author       = '$author',
-                        biblionumber = '$bibnum'";
-        $sth   = $dbh->prepare($query);
+    #    my $dbh   = C4Connect;
+    my $sth =
+      $dbh->prepare("Delete from additionalauthors where biblionumber = ?");
 
-        $sth->execute;
+    $sth->execute($bibnum);
+    $sth->finish;
+    foreach my $author (@authors) {
+        if ( $author ne '' ) {
+            $sth =
+              $dbh->prepare(
+                "Insert into additionalauthors set author = ?, biblionumber = ?"
+            );
 
-        $sth->finish;
-    } # if
-} # sub modaddauthor
+            $sth->execute( $author, $bibnum );
 
+            $sth->finish;
+        }    # if
+    }
+}    # sub modaddauthor
 
 sub OLDmodsubject {
-    my ($dbh,$bibnum, $force, @subject) = @_;
-#  my $dbh   = C4Connect;
+    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 .= "<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
+    for ( my $i = 0 ; $i < $count ; $i++ ) {
+        $subject[$i] =~ s/^ //g;
+        $subject[$i] =~ s/ $//g;
+        my $sth =
+          $dbh->prepare(
+"select * from catalogueentry where entrytype = 's' and catalogueentry = ?"
+        );
+        $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
+                my $sth2 =
+                  $dbh->prepare(
+"Insert into catalogueentry (entrytype,catalogueentry) values ('s',?)"
+                );
+
+                $sth2->execute( $subject[$i] ) if ( $subject[$i] );
+                $sth2->finish;
+            }
+            else {
+                $error =
+                  "$subject[$i]\n does not exist in the subject authority file";
+                my $sth2 =
+                  $dbh->prepare(
+"Select * from catalogueentry where entrytype = 's' and (catalogueentry like ? or catalogueentry like ? or catalogueentry like ?)"
+                );
+                $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 $sth =
+          $dbh->prepare("Delete from bibliosubject where biblionumber = ?");
+        $sth->execute($bibnum);
+        $sth->finish;
+        $sth =
+          $dbh->prepare(
+            "Insert into bibliosubject (subject,biblionumber) values (?,?)");
+        my $query;
+        foreach $query (@subject) {
+            $sth->execute( $query, $bibnum ) if ( $query && $bibnum );
+        }    # foreach
+        $sth->finish;
+    }    # if
 
-#  $dbh->disconnect;
-  return($error);
-} # sub modsubject
+    #  $dbh->disconnect;
+    return ($error);
+}    # sub modsubject
 
 sub OLDmodbibitem {
-    my ($dbh,$biblioitem) = @_;
-#    my $dbh   = C4Connect;
+    my ( $dbh, $biblioitem ) = @_;
     my $query;
 
-    $biblioitem->{'itemtype'}        = $dbh->quote($biblioitem->{'itemtype'});
-    $biblioitem->{'url'}             = $dbh->quote($biblioitem->{'url'});
-    $biblioitem->{'isbn'}            = $dbh->quote($biblioitem->{'isbn'});
-    $biblioitem->{'publishercode'}   = $dbh->quote($biblioitem->{'publishercode'});
-    $biblioitem->{'publicationyear'} = $dbh->quote($biblioitem->{'publicationyear'});
-    $biblioitem->{'classification'}  = $dbh->quote($biblioitem->{'classification'});
-    $biblioitem->{'dewey'}          = $dbh->quote($biblioitem->{'dewey'});
-    $biblioitem->{'subclass'}       = $dbh->quote($biblioitem->{'subclass'});
-    $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->{'size'}            = $dbh->quote($biblioitem->{'size'});
-    $biblioitem->{'place'}           = $dbh->quote($biblioitem->{'place'});
+    $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->{'publishercode'} =
+      $dbh->quote( $biblioitem->{'publishercode'} );
+    $biblioitem->{'publicationyear'} =
+      $dbh->quote( $biblioitem->{'publicationyear'} );
+    $biblioitem->{'classification'} =
+      $dbh->quote( $biblioitem->{'classification'} );
+    $biblioitem->{'dewey'}       = $dbh->quote( $biblioitem->{'dewey'} );
+    $biblioitem->{'subclass'}    = $dbh->quote( $biblioitem->{'subclass'} );
+    $biblioitem->{'illus'}       = $dbh->quote( $biblioitem->{'illus'} );
+    $biblioitem->{'pages'}       = $dbh->quote( $biblioitem->{'pages'} );
+    $biblioitem->{'volumeddesc'} = $dbh->quote( $biblioitem->{'volumeddesc'} );
+    $biblioitem->{'bnotes'}      = $dbh->quote( $biblioitem->{'bnotes'} );
+    $biblioitem->{'size'}        = $dbh->quote( $biblioitem->{'size'} );
+    $biblioitem->{'place'}       = $dbh->quote( $biblioitem->{'place'} );
 
     $query = "Update biblioitems set
 itemtype        = $biblioitem->{'itemtype'},
 url             = $biblioitem->{'url'},
 isbn            = $biblioitem->{'isbn'},
+issn            = $biblioitem->{'issn'},
 publishercode   = $biblioitem->{'publishercode'},
 publicationyear = $biblioitem->{'publicationyear'},
 classification  = $biblioitem->{'classification'},
@@ -1386,42 +1640,45 @@ 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->disconnect;
-} # sub modbibitem
+    if ( $dbh->errstr ) {
+        warn "$query";
+    }
+}    # sub modbibitem
 
 sub OLDmodnote {
-  my ($dbh,$bibitemnum,$note)=@_;
-#  my $dbh=C4Connect;
-  my $query="update biblioitems set notes='$note' where
+    my ( $dbh, $bibitemnum, $note ) = @_;
+
+    #  my $dbh=C4Connect;
+    my $query = "update biblioitems set notes='$note' where
   biblioitemnumber='$bibitemnum'";
-  my $sth=$dbh->prepare($query);
-  $sth->execute;
-  $sth->finish;
-#  $dbh->disconnect;
+    my $sth = $dbh->prepare($query);
+    $sth->execute;
+    $sth->finish;
+
+    #  $dbh->disconnect;
 }
 
 sub OLDnewbiblioitem {
-       my ($dbh,$biblioitem) = @_;
-       #  my $dbh   = C4Connect;
-       my $query = "Select max(biblioitemnumber) from biblioitems";
-       my $sth   = $dbh->prepare($query);
-       my $data;
-       my $bibitemnum;
+    my ( $dbh, $biblioitem ) = @_;
+
+    #  my $dbh   = C4Connect;
+    my $sth = $dbh->prepare("Select max(biblioitemnumber) from biblioitems");
+    my $data;
+    my $bibitemnum;
 
-       $sth->execute;
-       $data       = $sth->fetchrow_arrayref;
-       $bibitemnum = $$data[0] + 1;
+    $sth->execute;
+    $data       = $sth->fetchrow_arrayref;
+    $bibitemnum = $$data[0] + 1;
 
-       $sth->finish;
+    $sth->finish;
 
-       $sth = $dbh->prepare("insert into biblioitems set
+    $sth = $dbh->prepare( "insert into biblioitems set
                                                                        biblioitemnumber = ?,           biblionumber     = ?,
                                                                        volume           = ?,                   number           = ?,
                                                                        classification  = ?,                    itemtype         = ?,
@@ -1432,212 +1689,277 @@ sub OLDnewbiblioitem {
                                                                        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->{'notes'},
-                                               $biblioitem->{'size'},                          $biblioitem->{'lccn'},
-                                               $biblioitem->{'marc'},                          $biblioitem->{'place'});
-       $sth->finish;
-       #    $dbh->disconnect;
-       return($bibitemnum);
+                                                                       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 $sth=$dbh->prepare($query);
-#  print $query;
-  $sth->execute;
-  $sth->finish;
-#  $dbh->disconnect;
+    my ( $dbh, $bibnum ) = @_;
+    my $sth =
+      $dbh->prepare("insert into bibliosubject (biblionumber) values (?)");
+    $sth->execute($bibnum);
+    $sth->finish;
 }
 
 sub OLDnewsubtitle {
-    my ($dbh,$bibnum, $subtitle) = @_;
-#  my $dbh   = C4Connect;
-    $subtitle = $dbh->quote($subtitle);
-    my $query = "insert into bibliosubtitle set
-                            biblionumber = $bibnum,
-                            subtitle = $subtitle";
-    my $sth   = $dbh->prepare($query);
+    my ( $dbh, $bibnum, $subtitle ) = @_;
+    my $sth =
+      $dbh->prepare(
+        "insert into bibliosubtitle set biblionumber = ?, subtitle = ?");
+    $sth->execute( $bibnum, $subtitle ) if $subtitle;
+    $sth->finish;
+}
 
-    $sth->execute;
+sub OLDnewitems {
+    my ( $dbh, $item, $barcode ) = @_;
 
+    #  my $dbh   = C4Connect;
+    my $sth = $dbh->prepare("Select max(itemnumber) from items");
+    my $data;
+    my $itemnumber;
+    my $error = "";
+
+    $sth->execute;
+    $data       = $sth->fetchrow_hashref;
+    $itemnumber = $data->{'max(itemnumber)'} + 1;
     $sth->finish;
-#  $dbh->disconnect;
-}
 
+# FIXME the "notforloan" field seems to be named "loan" in some places. workaround bugfix.
+    if ( $item->{'loan'} ) {
+        $item->{'notforloan'} = $item->{'loan'};
+    }
 
-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;
-
-       $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->{'homebranch'},
-                                                       $item->{'price'},$item->{'replacementprice'},
-                                                       $item->{'itemnotes'},$item->{'loan'});
-
-       $sth->execute;
-       if (defined $sth->errstr) {
-               $error .= $sth->errstr;
-       }
-       $sth->finish;
-       #  $itemnumber++;
-       #  $dbh->disconnect;
-       return($itemnumber,$error);
+    # 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            = ?,
+                                                       itemcallnumber  =?,                                                     notforloan = ?,
+                                                       location = ?
+                                                       "
+        );
+        $sth->execute(
+            $itemnumber,                 $item->{'biblionumber'},
+            $item->{'biblioitemnumber'}, $barcode,
+            $item->{'booksellerid'},     $item->{'dateaccessioned'},
+            $item->{'homebranch'},       $item->{'holdingbranch'},
+            $item->{'price'},            $item->{'replacementprice'},
+            $item->{'itemnotes'},        $item->{'itemcallnumber'},
+            $item->{'notforloan'},             $item->{'location'}
+        );
+    }
+    else {
+        $sth = $dbh->prepare( "Insert into items set
+                                                       itemnumber           = ?,                               biblionumber         = ?,
+                                                       biblioitemnumber     = ?,                               barcode              = ?,
+                                                       booksellerid         = ?,                                       dateaccessioned      = NOW(),
+                                                       homebranch           = ?,                               holdingbranch        = ?,
+                                                       price                = ?,                                               replacementprice     = ?,
+                                                       replacementpricedate = NOW(),   itemnotes            = ?,
+                                                       itemcallnumber = ? , notforloan = ?,
+                                                       location = ?
+                                                       "
+        );
+        $sth->execute(
+            $itemnumber,                 $item->{'biblionumber'},
+            $item->{'biblioitemnumber'}, $barcode,
+            $item->{'booksellerid'},     $item->{'homebranch'},
+            $item->{'holdingbranch'},    $item->{'price'},
+            $item->{'replacementprice'}, $item->{'itemnotes'},
+            $item->{'itemcallnumber'},   $item->{'notforloan'},
+                       $item->{'location'}
+        );
+    }
+    if ( defined $sth->errstr ) {
+        $error .= $sth->errstr;
+    }
+    $sth->finish;
+    return ( $itemnumber, $error );
 }
 
 sub OLDmoditem {
-    my ($dbh,$item) = @_;
+    my ( $dbh, $item ) = @_;
+
 #  my ($dbh,$loan,$itemnum,$bibitemnum,$barcode,$notes,$homebranch,$lost,$wthdrawn,$replacement)=@_;
-#  my $dbh=C4Connect;
-$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 notforloan=$item->{'loan'} where itemnumber=$item->{'itemnum'}";
-  }
-  if ($item->{'lost'} ne ''){
-    $query="update items set biblioitemnumber=$item->{'bibitemnum'},
-                             barcode='$item->{'barcode'}',
-                             itemnotes='$item->{'notes'}',
-                             homebranch='$item->{'homebranch'}',
-                             itemlost='$item->{'lost'}',
-                             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;
-#  $dbh->disconnect;
+    #  my $dbh=C4Connect;
+    $item->{'itemnum'} = $item->{'itemnumber'} unless $item->{'itemnum'};
+    my $query = "update items set  barcode=?,itemnotes=?,itemcallnumber=?,notforloan=?,location=? where itemnumber=?";
+    my @bind = (
+        $item->{'barcode'},        $item->{'notes'},
+        $item->{'itemcallnumber'}, $item->{'notforloan'},
+        $item->{'location'},      $item->{'itemnum'}
+    );
+    if ( $item->{'lost'} ne '' ) {
+        $query = "update items set biblioitemnumber=?,
+                             barcode=?,
+                             itemnotes=?,
+                             homebranch=?,
+                             itemlost=?,
+                             wthdrawn=?,
+                            itemcallnumber=?,
+                            notforloan=?,
+                                location=?";
+        @bind = (
+            $item->{'bibitemnum'},     $item->{'barcode'},
+            $item->{'notes'},          $item->{'homebranch'},
+            $item->{'lost'},           $item->{'wthdrawn'},
+            $item->{'itemcallnumber'}, $item->{'notforloan'},
+            $item->{'location'},               $item->{'itemnum'}
+        );
+               if ($item->{homebranch}) {
+                       $query.=",homebranch=?";
+                       push @bind, $item->{homebranch};
+               }
+               if ($item->{holdingbranch}) {
+                       $query.=",holdingbranch=?";
+                       push @bind, $item->{holdingbranch};
+               }
+               $query.=" where itemnumber=?";
+    }
+    if ( $item->{'replacement'} ne '' ) {
+        $query =~ s/ where/,replacementprice='$item->{'replacement'}' where/;
+    }
+    my $sth = $dbh->prepare($query);
+    $sth->execute(@bind);
+    $sth->finish;
+
+    #  $dbh->disconnect;
 }
 
-sub OLDdelitem{
-  my ($dbh,$itemnum)=@_;
-#  my $dbh=C4Connect;
-  my $query="select * from items where itemnumber=$itemnum";
-  my $sth=$dbh->prepare($query);
-  $sth->execute;
-  my @data=$sth->fetchrow_array;
-  $sth->finish;
-  $query="Insert into deleteditems values (";
-  foreach my $temp (@data){
-    $query .= "'$temp',";
-  }
-  $query=~ s/\,$/\)/;
-#  print $query;
-  $sth=$dbh->prepare($query);
-  $sth->execute;
-  $sth->finish;
-  $query = "Delete from items where itemnumber=$itemnum";
-  $sth=$dbh->prepare($query);
-  $sth->execute;
-  $sth->finish;
-#  $dbh->disconnect;
+sub OLDdelitem {
+    my ( $dbh, $itemnum ) = @_;
+
+    #  my $dbh=C4Connect;
+    my $sth = $dbh->prepare("select * from items where itemnumber=?");
+    $sth->execute($itemnum);
+    my $data = $sth->fetchrow_hashref;
+    $sth->finish;
+    my $query = "Insert into deleteditems set ";
+    my @bind  = ();
+    foreach my $temp ( keys %$data ) {
+        $query .= "$temp = ?,";
+        push ( @bind, $data->{$temp} );
+    }
+    $query =~ s/\,$//;
+
+    #  print $query;
+    $sth = $dbh->prepare($query);
+    $sth->execute(@bind);
+    $sth->finish;
+    $sth = $dbh->prepare("Delete from items where itemnumber=?");
+    $sth->execute($itemnum);
+    $sth->finish;
+
+    #  $dbh->disconnect;
 }
 
 sub OLDdeletebiblioitem {
-    my ($dbh,$biblioitemnumber) = @_;
-#    my $dbh   = C4Connect;
-    my $query = "Select * from biblioitems
-where biblioitemnumber = $biblioitemnumber";
-    my $sth   = $dbh->prepare($query);
-    my @results;
-
-    $sth->execute;
+    my ( $dbh, $biblioitemnumber ) = @_;
 
-    if (@results = $sth->fetchrow_array) {
-        $query = "Insert into deletedbiblioitems values (";
-        foreach my $value (@results) {
-            $value  = $dbh->quote($value);
-            $query .= "$value,";
-        } # foreach
+    #    my $dbh   = C4Connect;
+    my $sth = $dbh->prepare( "Select * from biblioitems
+where biblioitemnumber = ?"
+    );
+    my $results;
 
-        $query =~ s/\,$/\)/;
-        $dbh->do($query);
+    $sth->execute($biblioitemnumber);
 
-        $query = "Delete from biblioitems
-                        where biblioitemnumber = $biblioitemnumber";
-        $dbh->do($query);
-    } # if
-    $sth->finish;
-# Now delete all the items attached to the biblioitem
-    $query = "Select * from items where biblioitemnumber = $biblioitemnumber";
-    $sth   = $dbh->prepare($query);
-    $sth->execute;
-    while (@results = $sth->fetchrow_array) {
-       $query = "Insert into deleteditems values (";
-       foreach my $value (@results) {
-           $value  = $dbh->quote($value);
-           $query .= "$value,";
-       } # foreach
-       $query =~ s/\,$/\)/;
-       $dbh->do($query);
-    } # while
+    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}
+        );
+        my $sth2 =
+          $dbh->prepare("Delete from biblioitems where biblioitemnumber = ?");
+        $sth2->execute($biblioitemnumber);
+        $sth2->finish();
+    }    # if
     $sth->finish;
-    $query = "Delete from items where biblioitemnumber = $biblioitemnumber";
-    $dbh->do($query);
-#    $dbh->disconnect;
-} # sub deletebiblioitem
-
-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){
+
+    # Now delete all the items attached to the biblioitem
+    $sth = $dbh->prepare("Select * from items where biblioitemnumber = ?");
+    $sth->execute($biblioitemnumber);
+    my @results;
+    while ( my $data = $sth->fetchrow_hashref ) {
+        my $query = "Insert into deleteditems set ";
+        my @bind  = ();
+        foreach my $temp ( keys %$data ) {
+            $query .= "$temp = ?,";
+            push ( @bind, $data->{$temp} );
+        }
+        $query =~ s/\,$//;
+        my $sth2 = $dbh->prepare($query);
+        $sth2->execute(@bind);
+    }    # while
     $sth->finish;
-    $query="Insert into deletedbiblio values (";
-    foreach my $temp (@data){
-      $temp=~ s/\'/\\\'/g;
-      $query .= "'$temp',";
+    $sth = $dbh->prepare("Delete from items where biblioitemnumber = ?");
+    $sth->execute($biblioitemnumber);
+    $sth->finish();
+
+    #    $dbh->disconnect;
+}    # sub deletebiblioitem
+
+sub OLDdelbiblio {
+    my ( $dbh, $biblio ) = @_;
+    my $sth = $dbh->prepare("select * from biblio where biblionumber=?");
+    $sth->execute($biblio);
+    if ( my $data = $sth->fetchrow_hashref ) {
+        $sth->finish;
+        my $query = "Insert into deletedbiblio set ";
+        my @bind  = ();
+        foreach my $temp ( keys %$data ) {
+            $query .= "$temp = ?,";
+            push ( @bind, $data->{$temp} );
+        }
+
+        #replacing the last , by ",?)"
+        $query =~ s/\,$//;
+        $sth = $dbh->prepare($query);
+        $sth->execute(@bind);
+        $sth->finish;
+        $sth = $dbh->prepare("Delete from biblio where biblionumber=?");
+        $sth->execute($biblio);
+        $sth->finish;
     }
-    $query=~ s/\,$/\)/;
-#   print $query;
-    $sth=$dbh->prepare($query);
-    $sth->execute;
-    $sth->finish;
-    $query = "Delete from biblio where biblionumber=$biblio";
-    $sth=$dbh->prepare($query);
-    $sth->execute;
     $sth->finish;
-  }
-  $sth->finish;
-#  $dbh->disconnect;
 }
 
 #
@@ -1646,86 +1968,27 @@ sub OLDdelbiblio{
 #
 #
 
-sub itemcount{
-  my ($biblio)=@_;
-  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;
-  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 = 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'});
-#  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.
+sub itemcount {
+    my ($biblio) = @_;
+    my $dbh = C4::Context->dbh;
 
-=cut
-#'
-# FIXME - This is effectively identical to
-# &C4::Catalogue::getsingleorder.
-# Pick one and stick with it.
-sub getsingleorder {
-  my ($ordnum)=@_;
-  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";
-  my $sth=$dbh->prepare($query);
-  $sth->execute($ordnum);
-  my $data=$sth->fetchrow_hashref;
-  $sth->finish;
-  return($data);
+    #  print $query;
+    my $sth = $dbh->prepare("Select count(*) from items where biblionumber=?");
+    $sth->execute($biblio);
+    my $data = $sth->fetchrow_hashref;
+    $sth->finish;
+    return ( $data->{'count(*)'} );
 }
 
 sub newbiblio {
-  my ($biblio) = @_;
-  my $dbh    = C4::Context->dbh;
-  my $bibnum=OLDnewbiblio($dbh,$biblio);
-# FIXME : MARC add
-  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
@@ -1747,11 +2010,14 @@ successful or not.
 =cut
 
 sub modbiblio {
-  my ($biblio) = @_;
-  my $dbh  = C4::Context->dbh;
-  my $biblionumber=OLDmodbiblio($dbh,$biblio);
-  return($biblionumber);
-# FIXME : MARC mod
+       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
@@ -1767,10 +2033,10 @@ C<$subtitle> is the new subtitle.
 =cut
 
 sub modsubtitle {
-  my ($bibnum, $subtitle) = @_;
-  my $dbh   = C4::Context->dbh;
-  &OLDmodsubtitle($dbh,$bibnum,$subtitle);
-} # sub modsubtitle
+    my ( $bibnum, $subtitle ) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDmodsubtitle( $dbh, $bibnum, $subtitle );
+}    # sub modsubtitle
 
 =item modaddauthor
 
@@ -1783,10 +2049,10 @@ C<&modaddauthor> deletes all additional authors.
 =cut
 
 sub modaddauthor {
-    my ($bibnum, $author) = @_;
-    my $dbh   = C4::Context->dbh;
-    &OLDmodaddauthor($dbh,$bibnum,$author);
-} # sub modaddauthor
+    my ( $bibnum, @authors ) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDmodaddauthor( $dbh, $bibnum, @authors );
+}    # sub modaddauthor
 
 =item modsubject
 
@@ -1799,364 +2065,609 @@ $error - Error message, or undef if successful.
 =cut
 
 sub modsubject {
-  my ($bibnum, $force, @subject) = @_;
-  my $dbh   = C4::Context->dbh;
-  my $error= &OLDmodsubject($dbh,$bibnum,$force, @subject);
-  return($error);
-} # sub modsubject
+    my ( $bibnum, $force, @subject ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $error = &OLDmodsubject( $dbh, $bibnum, $force, @subject );
+        if ($error eq ''){
+    # When MARC is off, ensures that the MARC biblio table gets updated with new
+    # subjects, of course, it deletes the biblio in marc, and then recreates.
+    # This check is to ensure that no MARC data exists to lose.
+
+        if (C4::Context->preference("MARC") eq '0'){
+            my $MARCRecord = &MARCkoha2marcBiblio($dbh,$bibnum);
+            my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$bibnum);
+            &MARCmodbiblio($dbh,$bibid, $MARCRecord);
+        }
+
+    }
+    return ($error);
+}    # sub modsubject
 
 sub modbibitem {
     my ($biblioitem) = @_;
-    my $dbh   = C4::Context->dbh;
-    &OLDmodbibitem($dbh,$biblioitem);
-    my $MARCbibitem = MARCkoha2marcBiblio($dbh,$biblioitem);
-    &MARCmodbiblio($dbh,$biblioitem->{biblionumber},0,$MARCbibitem);
-} # sub modbibitem
+    my $dbh = C4::Context->dbh;
+    &OLDmodbibitem( $dbh, $biblioitem );
+}    # sub modbibitem
 
 sub modnote {
-  my ($bibitemnum,$note)=@_;
-  my $dbh = C4::Context->dbh;
-  &OLDmodnote($dbh,$bibitemnum,$note);
+    my ( $bibitemnum, $note ) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDmodnote( $dbh, $bibitemnum, $note );
 }
 
 sub newbiblioitem {
-  my ($biblioitem) = @_;
-  my $dbh   = C4::Context->dbh;
-  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 = C4::Context->dbh;
-  &OLDnewsubject($dbh,$bibnum);
+    my ($bibnum) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDnewsubject( $dbh, $bibnum );
 }
 
 sub newsubtitle {
-    my ($bibnum, $subtitle) = @_;
-    my $dbh   = C4::Context->dbh;
-    &OLDnewsubtitle($dbh,$bibnum,$subtitle);
+    my ( $bibnum, $subtitle ) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDnewsubtitle( $dbh, $bibnum, $subtitle );
 }
 
 sub newitems {
-  my ($item, @barcodes) = @_;
-  my $dbh   = C4::Context->dbh;
-  my $errors;
-  my $itemnumber;
-  my $error;
-  foreach my $barcode (@barcodes) {
-      ($itemnumber,$error)=&OLDnewitems($dbh,$item,uc($barcode));
-      $errors .=$error;
-      my $MARCitem = &MARCkoha2marcItem($dbh,$item->{biblionumber},$itemnumber);
-      &MARCadditem($dbh,$MARCitem,$item->{biblionumber});
-  }
-  return($errors);
+    my ( $item, @barcodes ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $errors;
+    my $itemnumber;
+    my $error;
+    foreach my $barcode (@barcodes) {
+        ( $itemnumber, $error ) = &OLDnewitems( $dbh, $item, uc($barcode) );
+        $errors .= $error;
+        my $MARCitem =
+          &MARCkoha2marcItem( $dbh, $item->{biblionumber}, $itemnumber );
+        &MARCadditem( $dbh, $MARCitem, $item->{biblionumber} );
+    }
+    return ($errors);
 }
 
 sub moditem {
     my ($item) = @_;
     my $dbh = C4::Context->dbh;
-    &OLDmoditem($dbh,$item);
-    my $MARCitem = &MARCkoha2marcItem($dbh,$item->{'biblionumber'},$item->{'itemnum'});
-    my $bibid = &MARCfind_MARCbibid_from_oldbiblionumber($dbh,$item->{biblionumber});
-    &MARCmoditem($dbh,$MARCitem,$bibid,$item->{itemnum},0);
+    &OLDmoditem( $dbh, $item );
+    my $MARCitem =
+      &MARCkoha2marcItem( $dbh, $item->{'biblionumber'}, $item->{'itemnum'} );
+    my $bibid =
+      &MARCfind_MARCbibid_from_oldbiblionumber( $dbh, $item->{biblionumber} );
+    &MARCmoditem( $dbh, $MARCitem, $bibid, $item->{itemnum}, 0 );
 }
 
-sub checkitems{
-  my ($count,@barcodes)=@_;
-  my $dbh = C4::Context->dbh;
-  my $error;
-  for (my $i=0;$i<$count;$i++){
-    $barcodes[$i]=uc $barcodes[$i];
-    my $query="Select * from items where barcode='$barcodes[$i]'";
-    my $sth=$dbh->prepare($query);
-    $sth->execute;
-    if (my $data=$sth->fetchrow_hashref){
-      $error.=" Duplicate Barcode: $barcodes[$i]";
+sub checkitems {
+    my ( $count, @barcodes ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $error;
+    my $sth = $dbh->prepare("Select * from items where barcode=?");
+    for ( my $i = 0 ; $i < $count ; $i++ ) {
+        $barcodes[$i] = uc $barcodes[$i];
+        $sth->execute( $barcodes[$i] );
+        if ( my $data = $sth->fetchrow_hashref ) {
+            $error .= " Duplicate Barcode: $barcodes[$i]";
+        }
     }
     $sth->finish;
-  }
-  return($error);
+    return ($error);
 }
 
-sub countitems{
-  my ($bibitemnum)=@_;
-  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;
-  return($data->{'count(*)'});
+sub countitems {
+    my ($bibitemnum) = @_;
+    my $dbh   = C4::Context->dbh;
+    my $query = "";
+    my $sth   =
+      $dbh->prepare("Select count(*) from items where biblioitemnumber=?");
+    $sth->execute($bibitemnum);
+    my $data = $sth->fetchrow_hashref;
+    $sth->finish;
+    return ( $data->{'count(*)'} );
 }
 
-sub delitem{
-  my ($itemnum)=@_;
-  my $dbh = C4::Context->dbh;
-  &OLDdelitem($dbh,$itemnum);
+sub delitem {
+    my ($itemnum) = @_;
+    my $dbh = C4::Context->dbh;
+    &OLDdelitem( $dbh, $itemnum );
 }
 
 sub deletebiblioitem {
     my ($biblioitemnumber) = @_;
-    my $dbh   = C4::Context->dbh;
-    &OLDdeletebiblioitem($dbh,$biblioitemnumber);
-} # sub deletebiblioitem
-
+    my $dbh = C4::Context->dbh;
+    &OLDdeletebiblioitem( $dbh, $biblioitemnumber );
+}    # sub deletebiblioitem
 
 sub delbiblio {
-  my ($biblio)=@_;
-  my $dbh = C4::Context->dbh;
-  &OLDdelbiblio($dbh,$biblio);
+    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   = C4::Context->dbh;
-  my $query = "select * from itemtypes";
-  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;
-  return($count, @results);
-} # sub getitemtypes
-
 sub getbiblio {
     my ($biblionumber) = @_;
-    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 $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("Select * from biblio where biblionumber = ?");
+
+    # || 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->execute($biblionumber);
+
+    # || die "Cannot execute $query\n" . $sth->errstr;
+    while ( my $data = $sth->fetchrow_hashref ) {
+        $results[$count] = $data;
+        $count++;
+    }    # while
 
     $sth->finish;
-    return($count, @results);
-} # sub getbiblio
+    return ( $count, @results );
+}    # sub getbiblio
 
 sub getbiblioitem {
     my ($biblioitemnum) = @_;
-    my $dbh   = C4::Context->dbh;
-    my $query = "Select * from biblioitems where
-biblioitemnumber = $biblioitemnum";
-    my $sth   = $dbh->prepare($query);
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare( "Select * from biblioitems where
+biblioitemnumber = ?"
+    );
     my $count = 0;
     my @results;
 
-    $sth->execute;
+    $sth->execute($biblioitemnum);
 
-    while (my $data = $sth->fetchrow_hashref) {
+    while ( my $data = $sth->fetchrow_hashref ) {
         $results[$count] = $data;
-       $count++;
-    } # while
+        $count++;
+    }    # while
 
     $sth->finish;
-    return($count, @results);
-} # sub getbiblioitem
+    return ( $count, @results );
+}    # sub getbiblioitem
 
 sub getbiblioitembybiblionumber {
     my ($biblionumber) = @_;
-    my $dbh   = C4::Context->dbh;
-    my $query = "Select * from biblioitems where biblionumber =
-$biblionumber";
-    my $sth   = $dbh->prepare($query);
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("Select * from biblioitems where biblionumber = ?");
     my $count = 0;
     my @results;
 
-    $sth->execute;
+    $sth->execute($biblionumber);
 
-    while (my $data = $sth->fetchrow_hashref) {
+    while ( my $data = $sth->fetchrow_hashref ) {
         $results[$count] = $data;
-       $count++;
-    } # while
+        $count++;
+    }    # while
 
     $sth->finish;
-    return($count, @results);
-} # sub
+    return ( $count, @results );
+}    # sub
 
-sub getitemsbybiblioitem {
-    my ($biblioitemnum) = @_;
+sub getitemtypes {
     my $dbh   = C4::Context->dbh;
-    my $query = "Select * from items, biblio where
-biblio.biblionumber = items.biblionumber and biblioitemnumber
-= $biblioitemnum";
+    my $query = "select * from itemtypes order by description";
     my $sth   = $dbh->prepare($query);
-      # || die "Cannot prepare $query\n" . $dbh->errstr;
+
+    # || 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
+
+    # || die "Cannot execute $query\n" . $sth->errstr;
+    while ( my $data = $sth->fetchrow_hashref ) {
+        $results[$count] = $data;
+        $count++;
+    }    # while
 
     $sth->finish;
-    return($count, @results);
-} # sub getitemsbybiblioitem
+    return ( $count, @results );
+}    # sub getitemtypes
 
-sub isbnsearch {
-    my ($isbn) = @_;
-    my $dbh   = C4::Context->dbh;
+sub getitemsbybiblioitem {
+    my ($biblioitemnum) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare( "Select * from items, biblio where
+biblio.biblionumber = items.biblionumber and biblioitemnumber
+= ?"
+    );
+
+    # || die "Cannot prepare $query\n" . $dbh->errstr;
     my $count = 0;
-    my $query;
-    my $sth;
     my @results;
 
-    $isbn  = $dbh->quote($isbn);
-    $query = "Select distinct biblio.* from biblio, biblioitems where
-biblio.biblionumber = biblioitems.biblionumber
-and isbn = $isbn";
-    $sth   = $dbh->prepare($query);
+    $sth->execute($biblioitemnum);
 
-    $sth->execute;
-    while (my $data = $sth->fetchrow_hashref) {
+    # || die "Cannot execute $query\n" . $sth->errstr;
+    while ( my $data = $sth->fetchrow_hashref ) {
         $results[$count] = $data;
-       $count++;
-    } # while
+        $count++;
+    }    # while
 
     $sth->finish;
-    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;
-#}
+    return ( $count, @results );
+}    # sub getitemsbybiblioitem
 
 sub logchange {
-# Subroutine to log changes to databases
+
+    # Subroutine to log changes to databases
 # Eventually, this subroutine will be used to create a log of all changes made,
-# with the possibility of "undo"ing some changes
-    my $database=shift;
-    if ($database eq 'kohadb') {
-       my $type=shift;
-       my $section=shift;
-       my $item=shift;
-       my $original=shift;
-       my $new=shift;
-#      print STDERR "KOHA: $type $section $item $original $new\n";
-    } elsif ($database eq 'marc') {
-       my $type=shift;
-       my $Record_ID=shift;
-       my $tag=shift;
-       my $mark=shift;
-       my $subfield_ID=shift;
-       my $original=shift;
-       my $new=shift;
+    # with the possibility of "undo"ing some changes
+    my $database = shift;
+    if ( $database eq 'kohadb' ) {
+        my $type     = shift;
+        my $section  = shift;
+        my $item     = shift;
+        my $original = shift;
+        my $new      = shift;
+
+        #      print STDERR "KOHA: $type $section $item $original $new\n";
+    }
+    elsif ( $database eq 'marc' ) {
+        my $type        = shift;
+        my $Record_ID   = shift;
+        my $tag         = shift;
+        my $mark        = shift;
+        my $subfield_ID = shift;
+        my $original    = shift;
+        my $new         = shift;
+
 #      print STDERR "MARC: $type $Record_ID $tag $mark $subfield_ID $original $new\n";
     }
 }
 
 #------------------------------------------------
 
-
 #---------------------------------------
 # Find a biblio entry, or create a new one if it doesn't exist.
 #  If a "subtitle" entry is in hash, add it to subtitle table
 sub getoraddbiblio {
-       # input params
-       my (
-         $dbh,         # db handle
-                       # FIXME - Unused argument
-         $biblio,      # hash ref to fields
-       )=@_;
 
-       # return
-       my $biblionumber;
+    # input params
+    my (
+        $dbh,       # db handle
+                    # FIXME - Unused argument
+        $biblio,    # hash ref to fields
+    ) = @_;
 
-       my $debug=0;
-       my $sth;
-       my $error;
+    # return
+    my $biblionumber;
+
+    my $debug = 0;
+    my $sth;
+    my $error;
 
-       #-----
-       $dbh = C4::Context->dbh;
+    #-----
+    $dbh = C4::Context->dbh;
 
-       print "<PRE>Looking for biblio </PRE>\n" if $debug;
-       $sth=$dbh->prepare("select biblionumber
+    print "<PRE>Looking for biblio </PRE>\n" if $debug;
+    $sth = $dbh->prepare( "select biblionumber
                from biblio
                where title=? and author=?
-                 and copyrightdate=? and seriestitle=?");
-       $sth->execute(
-               $biblio->{title}, $biblio->{author},
-               $biblio->{copyright}, $biblio->{seriestitle} );
-       if ($sth->rows) {
-           ($biblionumber) = $sth->fetchrow;
-           print "<PRE>Biblio exists with number $biblionumber</PRE>\n" if $debug;
-       } else {
-           # Doesn't exist.  Add new one.
-           print "<PRE>Adding biblio</PRE>\n" if $debug;
-           ($biblionumber,$error)=&newbiblio($biblio);
-           if ( $biblionumber ) {
-             print "<PRE>Added with biblio number=$biblionumber</PRE>\n" if $debug;
-             if ( $biblio->{subtitle} ) {
-               &newsubtitle($biblionumber,$biblio->{subtitle} );
-             } # if subtitle
-           } else {
-               print "<PRE>Couldn't add biblio: $error</PRE>\n" if $debug;
-           } # if added
-       }
+                 and copyrightdate=? and seriestitle=?"
+    );
+    $sth->execute(
+        $biblio->{title},     $biblio->{author},
+        $biblio->{copyright}, $biblio->{seriestitle}
+    );
+    if ( $sth->rows ) {
+        ($biblionumber) = $sth->fetchrow;
+        print "<PRE>Biblio exists with number $biblionumber</PRE>\n" if $debug;
+    }
+    else {
+
+        # Doesn't exist.  Add new one.
+        print "<PRE>Adding biblio</PRE>\n" if $debug;
+        ( $biblionumber, $error ) = &newbiblio($biblio);
+        if ($biblionumber) {
+            print "<PRE>Added with biblio number=$biblionumber</PRE>\n"
+              if $debug;
+            if ( $biblio->{subtitle} ) {
+                &newsubtitle( $biblionumber, $biblio->{subtitle} );
+            }    # if subtitle
+        }
+        else {
+            print "<PRE>Couldn't add biblio: $error</PRE>\n" if $debug;
+        }    # if added
+    }
 
-       return $biblionumber,$error;
+    return $biblionumber, $error;
+
+}    # sub getoraddbiblio
+
+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($_);
+        }
+    }
+    return ($string);
+}
 
-} # sub getoraddbiblio
+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);
+}
 
-END { }       # module clean-up code here (global destructor)
+sub FindDuplicate {
+       my ($record)=@_;
+       my $dbh = C4::Context->dbh;
+       my $result = MARCmarc2koha($dbh,$record,'');
+       my $sth;
+       my ($biblionumber,$bibid,$title);
+       # search duplicate on ISBN, easy and fast...
+       if ($result->{isbn}) {
+               $sth = $dbh->prepare("select biblio.biblionumber,bibid,title from biblio,biblioitems,marc_biblio where biblio.biblionumber=biblioitems.biblionumber and marc_biblio.biblionumber=biblioitems.biblionumber and isbn=?");
+               $sth->execute($result->{'isbn'});
+               ($biblionumber,$bibid,$title) = $sth->fetchrow;
+               return $biblionumber,$bibid,$title if ($biblionumber);
+       }
+       # a more complex search : build a request for SearchMarc::catalogsearch()
+       my (@tags, @and_or, @excluding, @operator, @value, $offset,$length);
+       # search on biblio.title
+       my ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblio.title","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "contains";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for title, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+       # ... and on biblio.author
+       ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblio.author","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "contains";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for author, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+       # ... and on publicationyear.
+       ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblioitems.publicationyear","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "=";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for publicationyear, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+       # ... and on size.
+       ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblioitems.size","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "=";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for size, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+       # ... and on publisher.
+       ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblioitems.publishercode","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "=";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for publishercode, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+       # ... and on volume.
+       ($tag,$subfield) = MARCfind_marc_from_kohafield($dbh,"biblioitems.volume","");
+       if ($record->field($tag)) {
+               if ($record->field($tag)->subfields($subfield)) {
+                       push @tags, "'".$tag.$subfield."'";
+                       push @and_or, "and";
+                       push @excluding, "";
+                       push @operator, "=";
+                       push @value, $record->field($tag)->subfield($subfield);
+#                      warn "for volume, I add $tag / $subfield".$record->field($tag)->subfield($subfield);
+               }
+       }
+
+       my ($finalresult,$nbresult) = C4::SearchMarc::catalogsearch($dbh,\@tags,\@and_or,\@excluding,\@operator,\@value,0,10);
+       # there is at least 1 result => return the 1st one
+       if ($nbresult) {
+#              warn "$nbresult => ".@$finalresult[0]->{biblionumber},@$finalresult[0]->{bibid},@$finalresult[0]->{title};
+               return @$finalresult[0]->{biblionumber},@$finalresult[0]->{bibid},@$finalresult[0]->{title};
+       }
+       # no result, returns nothing
+       return;
+}
+sub DisplayISBN {
+       my ($isbn)=@_;
+       my $seg1;
+       if(substr($isbn, 0, 1) <=7) {
+               $seg1 = substr($isbn, 0, 1);
+       } elsif(substr($isbn, 0, 2) <= 94) {
+               $seg1 = substr($isbn, 0, 2);
+       } elsif(substr($isbn, 0, 3) <= 995) {
+               $seg1 = substr($isbn, 0, 3);
+       } elsif(substr($isbn, 0, 4) <= 9989) {
+               $seg1 = substr($isbn, 0, 4);
+       } else {
+               $seg1 = substr($isbn, 0, 5);
+       }
+       my $x = substr($isbn, length($seg1));
+       my $seg2;
+       if(substr($x, 0, 2) <= 19) {
+#              if(sTmp2 < 10) sTmp2 = "0" sTmp2;
+               $seg2 = substr($x, 0, 2);
+       } elsif(substr($x, 0, 3) <= 699) {
+               $seg2 = substr($x, 0, 3);
+       } elsif(substr($x, 0, 4) <= 8399) {
+               $seg2 = substr($x, 0, 4);
+       } elsif(substr($x, 0, 5) <= 89999) {
+               $seg2 = substr($x, 0, 5);
+       } elsif(substr($x, 0, 6) <= 9499999) {
+               $seg2 = substr($x, 0, 6);
+       } else {
+               $seg2 = substr($x, 0, 7);
+       }
+       my $seg3=substr($x,length($seg2));
+       $seg3=substr($seg3,0,length($seg3)-1) ;
+       my $seg4 = substr($x, -1, 1);
+       return "$seg1-$seg2-$seg3-$seg4";
+}
+END { }    # module clean-up code here (global destructor)
 
 =back
 
@@ -2168,3 +2679,539 @@ Paul POULAIN paul.poulain@free.fr
 
 =cut
 
+# $Id$
+# $Log$
+# Revision 1.121  2005/06/20 14:10:00  tipaul
+# synch'ing 2.2 and head
+#
+# Revision 1.120  2005/06/15 16:09:43  hdl
+# Displaying dashed isbn.
+#
+# Revision 1.119  2005/06/01 20:43:58  genjimoto
+# patch from Genji (Waylon R.) to update subjects in MARC tables when systempref has MARC=OFF
+#
+# Revision 1.118  2005/05/04 15:40:01  tipaul
+# synch'ing 2.2 and head
+#
+# Revision 1.115.2.9  2005/04/07 10:05:25  tipaul
+# adding / to the list of symbols that are replace by spaces for searches
+#
+# Revision 1.115.2.8  2005/03/25 16:23:49  tipaul
+# some improvements :
+# * return immediatly when a subfield is empty
+# * search duplicate on isbn must be done only when there is an isbn ;-)
+#
+# Revision 1.115.2.7  2005/03/10 15:52:28  tipaul
+# * adding glass to opac marc detail.
+# * changing glasses behaviour : It now appears only on subfields that have a "link" value. Avoid useless glasses and removes nothing. **** WARNING **** : if you don't change you MARC parameters, glasses DISAPPEAR, because no subfields have a link value. So you MUST "reactivate" them manually. If you want to enable the search glass on field 225$a (collection in UNIMARC), just put 225a to "link" field (Koha >> parameters >> framework >> 225 field >> subfield >> modify $a >> enter 225a in link input field (without quotes or anything else)
+# * fixing bug with libopac
+#
+# Revision 1.115.2.6  2005/03/09 15:56:01  tipaul
+# Changing MARCmoditem to be like MARCmodbiblio : a modif is a delete & create.
+# Longer, but solves problems with repeated subfields.
+#
+# The previous version was not buggy except under certain circumstances (a repeated subfield, that does not exist usually in items)
+#
+# Revision 1.115.2.5  2005/02/24 13:54:04  tipaul
+# exporting MARCdelsubfield sub. It's used in authority merging.
+# Modifying it too to enable deletion of all subfields from a given tag/subfield or just one.
+#
+# Revision 1.115.2.4  2005/02/17 12:44:25  tipaul
+# bug in acquisition : the title was also stored as subtitle.
+#
+# Revision 1.115.2.3  2005/02/10 13:14:36  tipaul
+# * multiple main authors are now correctly handled in simple (non-MARC) view
+#
+# Revision 1.115.2.2  2005/01/11 16:02:35  tipaul
+# in catalogue, modifs were not stored properly the non-MARC item DB. Affect only libraries without barcodes.
+#
+# Revision 1.115.2.1  2005/01/11 14:45:37  tipaul
+# bugfix : issn were not stored correctly in non-MARC DB on biblio modification
+#
+# Revision 1.115  2005/01/06 14:32:17  tipaul
+# improvement of speed for bulkmarcimport.
+# A sub had been forgotten to use the C4::Context->marcfromkohafield array, that caches DB datas.
+# this is only a little improvement for normal DB modif, but almost x2 the speed of bulkmarcimport... from 6records/seconds to more than 10.
+#
+# Revision 1.114  2005/01/03 10:48:33  tipaul
+# * bugfix for the search on a MARC detail, when you clic on the magnifying glass (caused an internal server error)
+# * partial support of the "linkage" MARC feature : if you enter a "link" on a MARC subfield, the magnifying glass won't search on the field, but on the linked field. I agree it's a partial support. Will be improved, but I need to investigate MARC21 & UNIMARC diffs on this topic.
+#
+# Revision 1.113  2004/12/10 16:27:53  tipaul
+# limiting the number of search term to 8. There was no limit before, but 8 words seems to be the upper limit mySQL can deal with (in less than a second. tested on a DB with 13 000 items)
+# In 2.4, a new DB structure will highly speed things and this limit will be removed.
+# FindDuplicate is activated again, the perf problems were due to this problem.
+#
+# Revision 1.112  2004/12/08 10:14:42  tipaul
+# * desactivate FindDuplicate
+# * fix from Genji
+#
+# Revision 1.111  2004/11/25 17:39:44  tipaul
+# removing useless &branches in package declaration
+#
+# Revision 1.110  2004/11/24 16:00:01  tipaul
+# removing sub branches (commited by chris for MARC=OFF bugfix, but sub branches is already in Acquisition.pm)
+#
+# Revision 1.109  2004/11/24 15:58:31  tipaul
+# * critical fix for acquisition (see RC3 release notes)
+# * critical fix for duplicate finder
+#
+# Revision 1.108  2004/11/19 19:41:22  rangi
+# Shifting branches() from deprecated C4::Catalogue to C4::Biblio
+# Allowing the non marc interface acquisitions to work.
+#
+# Revision 1.107  2004/11/05 10:15:27  tipaul
+# Improving FindDuplicate to find duplicate records on adding biblio
+#
+# Revision 1.106  2004/11/02 16:44:45  tipaul
+# new feature : checking for duplicate biblio.
+#
+# For instance, it's only done on ISBN only. Will be improved soon.
+#
+# When a duplicate is detected, the biblio is not saved, but the user is asked for a confirmations.
+#
+# Revision 1.105  2004/09/23 16:15:37  tipaul
+# indenting diff
+#
+# Revision 1.104  2004/09/16 15:06:46  tipaul
+# enabling # (| still possible too) for repeatable subfields
+#
+# Revision 1.103  2004/09/06 14:17:34  tipaul
+# some commented warning added + 1 major bugfix => drop empty fields, NOT fields containing 0
+#
+# Revision 1.102  2004/09/06 10:00:19  tipaul
+# adding a "location" field to the library.
+# This field is useful when the callnumber contains no information on the room where the item is stored.
+# With this field, we now have 3 levels of informations to find a book :
+# * the branch.
+# * the location.
+# * the callnumber.
+#
+# This should be versatile enough to solve any storing method.
+# This hack is quite simple, due to the nice Biblio.pm API. The MARC => koha db link is automatically managed. Just add the link in the parameters section.
+#
+# Revision 1.101  2004/08/18 16:01:37  tipaul
+# modifs to support frameworkcodes
+#
+# Revision 1.100  2004/08/13 16:37:25  tipaul
+# adding frameworkcode to API in some subs
+#
+# Revision 1.99  2004/07/30 13:54:50  doxulting
+# Beginning of serial commit
+#
+# Revision 1.98  2004/07/15 09:48:10  tipaul
+# * removing useless sub
+# * minor bugfix in moditem (managing homebranch & holdingbranch)
+#
+# Revision 1.97  2004/07/02 15:53:53  tipaul
+# bugfix (due to frameworkcode field)
+#
+# Revision 1.96  2004/06/29 16:07:10  tipaul
+# last sync for 2.1.0 release
+#
+# Revision 1.95  2004/06/26 23:19:59  rangi
+# Fixing modaddauthor, and adding getitemtypes.
+# Also tidying up formatting of code
+#
+# Revision 1.94  2004/06/17 08:16:32  tipaul
+# merging tag & subfield in marc_word for better perfs
+#
+# Revision 1.93  2004/06/11 15:38:06  joshferraro
+# Changes MARCaddword to index words >= 1 char ... needed for more accurate
+# searches using SearchMarc routines.
+#
+# Revision 1.92  2004/06/10 08:29:01  tipaul
+# MARC authority management (continued)
+#
+# Revision 1.91  2004/06/03 10:03:01  tipaul
+# * frameworks and itemtypes are independant
+# * in the MARC editor, showing the + to duplicate a tag only if the tag is repeatable
+#
+# Revision 1.90  2004/05/28 08:25:53  tipaul
+# hidding hidden & isurl constraints into MARC subfield structure
+#
+# Revision 1.89  2004/05/27 21:47:21  rangi
+# Fix for bug 787
+#
+# Revision 1.88  2004/05/18 15:23:49  tipaul
+# framework management : 1 MARC framework for each itemtype
+#
+# Revision 1.87  2004/05/18 11:54:07  tipaul
+# getitemtypes moved in Koha.pm
+#
+# Revision 1.86  2004/05/03 09:19:22  tipaul
+# some fixes for mysql prepare & execute
+#
+# Revision 1.85  2004/04/02 14:55:48  tipaul
+# renaming items.bulk field to items.itemcallnumber.
+# Will be used to store call number for libraries that don't use dewey classification.
+# Note it's related to ITEMS, not biblio.
+#
+# Revision 1.84  2004/03/24 17:18:30  joshferraro
+# Fixes bug 749 by removing the comma on line 1488.
+#
+# Revision 1.83  2004/03/15 14:31:50  tipaul
+# adding a minor check
+#
+# Revision 1.82  2004/03/07 05:47:31  acli
+# Various updates/fixes from rel_2_0
+# Fixes for bugs 721 (templating), 727, and 734
+#
+# Revision 1.81  2004/03/06 20:26:13  tipaul
+# adding seealso feature in MARC searches
+#
+# Revision 1.80  2004/02/12 13:40:56  tipaul
+# deleting subs duplicated by error
+#
+# Revision 1.79  2004/02/11 08:40:09  tipaul
+# synch'ing 2.0.0 branch and head
+#
+# Revision 1.78.2.3  2004/02/10 13:15:46  tipaul
+# removing 2 warnings
+#
+# Revision 1.78.2.2  2004/01/26 10:38:06  tipaul
+# dealing correctly "bulk" field
+#
+# Revision 1.78.2.1  2004/01/13 17:29:53  tipaul
+# * minor html fixes
+# * adding publisher in acquisition process (& ordering basket by publisher)
+#
+# Revision 1.78  2003/12/09 15:57:28  tipaul
+# rolling back to working char_decode sub
+#
+# Revision 1.77  2003/12/03 17:47:14  tipaul
+# bugfixes for biblio deletion
+#
+# Revision 1.76  2003/12/03 01:43:41  slef
+# conflict markers?
+#
+# Revision 1.75  2003/12/03 01:42:03  slef
+# bug 662 fixes securing DBI
+#
+# 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.
+#
+# 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.
+#