Removing $dbh from GetMarcFromKohaField (dbh is not used in this function.)
[koha.git] / C4 / Biblio.pm
index 8822a4a..1c777ba 100644 (file)
@@ -1,5 +1,5 @@
-package C4::Biblio;
-# New subs added by tgarip@neu.edu.tr 05/11/05
+package C4::Biblio;
+
 # Copyright 2000-2002 Katipo Communications
 #
 # This file is part of Koha.
 # You should have received a copy of the GNU General Public License along with
 # Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
 # Suite 330, Boston, MA  02111-1307 USA
+
 use strict;
+
 require Exporter;
 use C4::Context;
 use MARC::Record;
 use MARC::File::USMARC;
 use MARC::File::XML;
-use XML::Simple;
-use Encode;
+use ZOOM;
+use C4::Koha;
+use C4::Date;
+use utf8;
+use C4::Log; # logaction
 
 use vars qw($VERSION @ISA @EXPORT);
 
 # set the version for version checking
-$VERSION = 2.01;
+$VERSION = do { my @v = '$Revision$' =~ /\d+/g; shift(@v).".".join( "_", map { sprintf "%03d", $_ } @v ); };
 
-@ISA = qw(Exporter);
+@ISA = qw( Exporter );
 
-# &itemcount removed, now  resides in Search.pm
-#
-@EXPORT = qw(
-
-&getitemtypes
-&getkohafields
-&getshelves
-
-&NEWnewbiblio 
-&NEWnewitem
-&NEWmodbiblio 
-&NEWmoditem
-&NEWdelbiblio 
-&NEWdelitem
-&NEWmodbiblioframework
-
-&MARCgetallitems 
-&MARCfind_marc_from_kohafield
-&MARCfind_frameworkcode
-&MARCfind_itemtype
-&MARCgettagslib
-&MARCitemsgettagslib
-&MARCmoditemonefield
-&MARCkoha2marc
-&MARCmarc2koha 
-&MARCkoha2marcOnefield 
-&MARCfind_attr_from_kohafield
-&MARChtml2marc 
-&MARChtml2xml 
-&MARChtml2marcxml
-&MARCgetbiblio 
-&MARCgetitem 
-
-&XMLgetbiblio 
-&XMLgetitem 
-&XMLgetallitems 
-&XML_xml2hash 
-&XML_hash2xml 
-&XMLmarc2koha
-&XML_readline
-&XML_writeline
-
-&ZEBRAgetrecord   
-&ZEBRAgetallitems 
-&ZEBRAop &ZEBRAopserver 
-&ZEBRA_readyXML 
-&ZEBRA_readyXML_noheader
-
-&newbiblio
-&modbiblio
-&DisplayISBN
+# EXPORTED FUNCTIONS.
 
+# to add biblios or items
+push @EXPORT, qw( &AddBiblio &AddItem );
+
+# to get something
+push @EXPORT, qw(
+  &GetBiblio
+  &GetBiblioData
+  &GetBiblioItemData
+  &GetBiblioItemInfosOf
+  &GetBiblioItemByBiblioNumber
+  &GetBiblioFromItemNumber
+  
+  &GetMarcItem
+  &GetItem
+  &GetItemInfosOf
+  &GetItemStatus
+  &GetItemLocation
+  &GetLostItems
+  &GetItemsForInventory
+
+  &GetMarcNotes
+  &GetMarcSubjects
+  &GetMarcBiblio
+  &GetMarcAuthors
+  &GetMarcSeries
+
+  &GetItemsInfo
+  &GetItemnumberFromBarcode
+  &get_itemnumbers_of
+  &GetXmlBiblio
+
+  &GetAuthorisedValueDesc
+  &GetMarcStructure
+  &GetMarcFromKohaField
+  &GetFrameworkCode
+  &TransformKohaToMarc
 );
 
-#################### XML XML  XML  XML ###################
-### XML Read- Write functions
-
-
-sub XML_readline{
-my ($xml,$kohafield,$recordtype)=@_;
-#$xml represents one record node hashed of holdings or a complete xml koharecord
-### $recordtype is needed for reading the child records( like holdings records) .Otherwise main  record is assumed ( like biblio)
-## holding records are parsed and sent here one by one
-my ($tag,$subf)=MARCfind_marc_from_kohafield($kohafield,$recordtype);
-my @itemresults;
-if ($tag){
-if ($recordtype eq "holdings"){
-       my $item=$xml->{'datafield'};
-       my $hcontrolfield=$xml->{'controlfield'};
-     if ($tag>9){
-       foreach my $data (@$item){
-           if ($data->{'tag'} eq $tag){
-               foreach my $subfield ( $data->{'subfield'}){
-                   foreach my $code ( @$subfield){
-                       if ($code->{'code'} eq $subf){
-                       return Encode::decode("UTF-8",$code->{content});
-                       }
-                  }
-               }
-          }
-       }
-      }else{
-       foreach my $control (@$hcontrolfield){
-               if ($control->{'tag'} eq $tag){
-               return  Encode::decode("UTF-8",$control->{'content'});
-               }
-       }
-      }##tag
-
-}else{ ##Not a holding read biblio
-my $biblio=$xml->{'record'}->[0]->{'datafield'};
-my $controlfields=$xml->{'record'}->[0]->{'controlfield'};
- if ($tag>9){
-       foreach my $data (@$biblio){
-           if ($data->{'tag'} eq $tag){
-               foreach my $subfield ( $data->{'subfield'}){
-                   foreach my $code ( @$subfield){
-                       if ($code->{'code'} eq $subf){
-                       return Encode::decode("UTF-8",$code->{'content'});
-                       }
-                  }
-               }
-          }
-       }
-  }else{
-       
-       foreach my $control (@$controlfields){
-               if ($control->{'tag'} eq $tag){
-               return  Encode::decode("UTF-8",$control->{'content'}) if $control->{'content'};
-               }
-       }
-   }##tag
-}## Holding or not
-}## if tag is mapped
-return "";
-}
+# To modify something
+push @EXPORT, qw(
+  &ModBiblio
+  &ModItem
+  &ModItemTransfer
+  &ModBiblioframework
+  &ModZebra
+  &ModItemInMarc
+  &ModItemInMarconefield
+  &ModDateLastSeen
+);
 
-sub XML_writeline{
-## This routine modifies one line of marcxml record mainly useful for updating circulation data
-my ($xml,$kohafield,$newvalue,$recordtype)=@_;
-my $biblio=$xml->{'record'}->[0]->{'datafield'};
-my $controlfield=$xml->{'record'}->[0]->{'controlfield'};
-my ($tag,$subf)=MARCfind_kohafield($kohafield,$recordtype);
-my $updated=0;
-    if ($tag>9){
-       foreach my $data (@$biblio){
-                       if ($data->{'tag'} eq $tag){
-                       my @subfields=$data->{'subfield'};
-                       foreach my $subfield ( @subfields){
-                             foreach my $code ( @$subfield){
-                               if ($code->{'code'} eq $subf){  
-                               $code->{content}=$newvalue;
-                               $updated=1;
-                               }
-                             }
-                       }
-                    if (!$updated){    
-                        push @subfields,{code=>$subf,content=>$newvalue};
-                       $data->{subfield}= \@subfields;
-                       
-                    }  
-               }
-                }
-               ## Tag did not exist
-                 if (!$updated){
-                       push @$biblio,{datafield=>[{
-                                                                               'ind1' => ' ',
-                                                                               'ind2' => ' ',
-                                                                               'subfield' => [
-                                                                                               {
-                                                                                                 'content' => $newvalue,
-                                                                                                 'code' => $subf
-                                                                                               }
-                                                                                             ],
-                                                                               'tag' => $tag
-                                                                             }]
-                               };
-                 }## created now
-    }else{
-       foreach my $control(@$controlfield){
-               if ($control->{'tag'} eq $tag){
-                       $control->{'content'}=$newvalue;
-                       $updated=1;
-               }
-            }
-        if (!$updated){
-          push @$controlfield,{tag=>$tag,content=>$newvalue};     
-       }
-   }
-return $xml;
-}
+# To delete something
+push @EXPORT, qw(
+  &DelBiblio
+  &DelItem
+);
 
-sub XML_xml2hash{
-##make a perl hash from xml file
-my ($xml)=@_;
-  my $hashed = XMLin( $xml ,KeyAttr =>['leader','controlfield','datafield'],ForceArray => ['leader','controlfield','datafield','subfield','holdings','record'],KeepRoot=>0);
-return $hashed;
-}
+# Internal functions
+# those functions are exported but should not be used
+# they are usefull is few circumstances, so are exported.
+# but don't use them unless you're a core developer ;-)
+push @EXPORT, qw(
+  &ModBiblioMarc
+  &AddItemInMarc
+  &calculatelc
+  &itemcalculator
+);
 
-sub XML_hash2xml{
-## turn a hash back to xml
-my ($hashed,$root)=@_;
-$root="record" unless $root;
-my $xml= XMLout($hashed,KeyAttr=>['collection','record','leader','controlfıeld','datafield'],NoSort => 1,AttrIndent => 0,KeepRoot=>0,SuppressEmpty => 1,RootName=>$root);
-return $xml;
-}
+# Others functions
+push @EXPORT, qw(
+  &TransformMarcToKoha
+  &TransformHtmlToMarc
+  &TransformHtmlToXml
+  &PrepareItemrecordDisplay
+  &char_decode
+);
 
+=head1 NAME
 
-sub XMLgetbiblio {
-    # Returns MARC::XML of the biblionumber passed in parameter.
-    my ( $dbh, $biblionumber ) = @_;
-    my $sth =      $dbh->prepare("select marcxml from biblio where biblionumber=? "  );
-    $sth->execute( $biblionumber);
-   my ($marcxml)=$sth->fetchrow;
- return ($marcxml);
-}
+C4::Biblio - cataloging management functions
 
-sub XMLgetitem {
-   # Returns MARC::XML   of the item passed in parameter uses either itemnumber or barcode
-    my ( $dbh, $itemnumber,$barcode ) = @_;
-my $sth;
-if ($itemnumber){
-   $sth = $dbh->prepare("select marcxml from items  where itemnumber=?"  ); 
-    $sth->execute($itemnumber);
-}else{
- $sth = $dbh->prepare("select marcxml from items where barcode=?"  ); 
-    $sth->execute($barcode);
-}
- my ($marcxml)=$sth->fetchrow;
-    return ($marcxml);
-}
+=head1 DESCRIPTION
 
-sub XMLgetallitems {
-# warn "XMLgetallitems";
-    # Returns an array of MARC:XML   of the items passed in parameter as biblionumber
-    my ( $dbh, $biblionumber ) = @_;
-my @results;
-my   $sth = $dbh->prepare("select marcxml from items where biblionumber =?"  ); 
-    $sth->execute($biblionumber);
+Biblio.pm contains functions for managing storage and editing of bibliographic data within Koha. Most of the functions in this module are used for cataloging records: adding, editing, or removing biblios, biblioitems, or items. Koha's stores bibliographic information in three places:
 
- while(my ($marcxml)=$sth->fetchrow_array){
-    push @results,$marcxml;
-}
-return @results;
-}
+=over 4
 
-sub XMLmarc2koha {
-# warn "XMLmarc2koha";
-##Returns two hashes from KOHA_XML record
-## A biblio hash and and array of item hashes
-       my ($dbh,$xml,$related_record,@fields) = @_;
-       my ($result,@items);
-       
-## if @fields is given do not bother about the rest of fields just parse those
-
-if ($related_record eq "biblios" || $related_record eq "" || !$related_record){
-       if (@fields){
-               foreach my $field(@fields){
-               my $val=&XML_readline($xml,$field,'biblios');
-                       $result->{$field}=$val if $val;
-                       
-               }
-       }else{
-       my $sth2=$dbh->prepare("SELECT  marctokoha from koha_attr where  recordtype like 'biblios' and tagfield is not null" );
-       $sth2->execute();
-       my $field;
-               while ($field=$sth2->fetchrow) {
-               $result->{$field}=&XML_readline($xml,$field,'biblios');
-               }
-       }
-
-## we only need the following for biblio data
-       
-# 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;
-       }
-}
-if ($related_record eq "holdings" || $related_record eq ""  || !$related_record){
-my $holdings=$xml->{holdings}->[0]->{record};
-
-
-       if (@fields){
-           foreach my $holding (@$holdings){   
-my $itemresult;
-               foreach my $field(@fields){
-               my $val=&XML_readline($holding,$field,'holdings');
-               $itemresult->{$field}=$val if $val;     
-               }
-           push @items, $itemresult;
-          }
-       }else{
-       my $sth2=$dbh->prepare("SELECT  marctokoha from koha_attr where recordtype like 'holdings' and tagfield is not null" );
-          foreach my $holding (@$holdings){    
-          $sth2->execute();
-           my $field;
-my $itemresult;
-               while ($field=$sth2->fetchrow) {
-               $itemresult->{$field}=&XML_readline($xml,$field,'holdings');
-               }
-        push @items, $itemresult;
-          }
-       }
+=item 1. in the biblio,biblioitems,items, etc tables, which are limited to a one-to-one mapping to underlying MARC data
 
-}
+=item 2. as raw MARC in the Zebra index and storage engine
 
-       return ($result,@items);
-}
+=item 3. as raw MARC the biblioitems.marc and biblioitems.marcxml
 
-#
-#
-# MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC MARC
-#
-## Script to deal with MARC read write operations
+=back
 
+In the 3.0 version of Koha, the authoritative record-level information is in biblioitems.marcxml
 
-##Sub to match kohafield to Z3950 -attributes
+Because the data isn't completely normalized there's a chance for information to get out of sync. The design choice to go with a un-normalized schema was driven by performance and stability concerns. However, if this occur, it can be considered as a bug : The API is (or should be) complete & the only entry point for all biblio/items managements.
 
-sub MARCfind_attr_from_kohafield {
-# warn "MARCfind_attr_from_kohafield";
-## returns attribute
-    my (  $kohafield ) = @_;
-    return 0, 0 unless $kohafield;
+=over 4
+
+=item 1. Compared with MySQL, Zebra is slow to update an index for small data changes -- especially for proc-intensive operations like circulation
+
+=item 2. Zebra's index has been known to crash and a backup of the data is necessary to rebuild it in such cases
+
+=back
+
+Because of this design choice, the process of managing storage and editing is a bit convoluted. Historically, Biblio.pm's grown to an unmanagable size and as a result we have several types of functions currently:
+
+=over 4
+
+=item 1. Add*/Mod*/Del*/ - high-level external functions suitable for being called from external scripts to manage the collection
+
+=item 2. _koha_* - low-level internal functions for managing the koha tables
+
+=item 3. Marc management function : as the MARC record is stored in biblioitems.marc(xml), some subs dedicated to it's management are in this package. They should be used only internally by Biblio.pm, the only official entry points being AddBiblio, AddItem, ModBiblio, ModItem.
+
+=item 4. Zebra functions used to update the Zebra index
+
+=item 5. internal helper functions such as char_decode, checkitems, etc. Some of these probably belong in Koha.pm
+
+=back
+
+The MARC record (in biblioitems.marcxml) contains the complete marc record, including items. It also contains the biblionumber. That is the reason why it is not stored directly by AddBiblio, with all other fields . To save a biblio, we need to :
+
+=over 4
+
+=item 1. save datas in biblio and biblioitems table, that gives us a biblionumber and a biblioitemnumber
+
+=item 2. add the biblionumber and biblioitemnumber into the MARC records
+
+=item 3. save the marc record
+
+=back
+
+When dealing with items, we must :
+
+=over 4
+
+=item 1. save the item in items table, that gives us an itemnumber
+
+=item 2. add the itemnumber to the item MARC field
+
+=item 3. overwrite the MARC record (with the added item) into biblioitems.marc(xml)
+
+When modifying a biblio or an item, the behaviour is quite similar.
+
+=back
+
+=head1 EXPORTED FUNCTIONS
+
+=head2 AddBiblio
 
-       my $relations = C4::Context->attrfromkohafield;
-       return ($relations->{$kohafield});
+=over 4
+
+($biblionumber,$biblioitemnumber) = AddBiblio($record,$frameworkcode);
+Exported function (core API) for adding a new biblio to koha.
+
+=back
+
+=cut
+
+sub AddBiblio {
+    my ( $record, $frameworkcode ) = @_;
+    my $biblionumber;
+    my $biblioitemnumber;
+    my $dbh = C4::Context->dbh;
+    # transform the data into koha-table style data
+    my $olddata = TransformMarcToKoha( $dbh, $record, $frameworkcode );
+    $biblionumber = _koha_add_biblio( $dbh, $olddata, $frameworkcode );
+    $olddata->{'biblionumber'} = $biblionumber;
+    $biblioitemnumber = _koha_add_biblioitem( $dbh, $olddata );
+
+    # we must add bibnum and bibitemnum in MARC::Record...
+    # we build the new field with biblionumber and biblioitemnumber
+    # we drop the original field
+    # we add the new builded field.
+    ( my $biblio_tag, my $biblio_subfield ) = GetMarcFromKohaField("biblio.biblionumber",$frameworkcode);
+    ( my $biblioitem_tag, my $biblioitem_subfield ) = GetMarcFromKohaField("biblioitems.biblioitemnumber",$frameworkcode);
+
+    my $newfield;
+
+    # biblionumber & biblioitemnumber are in different fields
+    if ( $biblio_tag != $biblioitem_tag ) {
+
+        # deal with biblionumber
+        if ( $biblio_tag < 10 ) {
+            $newfield = MARC::Field->new( $biblio_tag, $biblionumber );
+        }
+        else {
+            $newfield =
+              MARC::Field->new( $biblio_tag, '', '',
+                "$biblio_subfield" => $biblionumber );
+        }
+
+        # drop old field and create new one...
+        my $old_field = $record->field($biblio_tag);
+        $record->delete_field($old_field);
+        $record->append_fields($newfield);
+
+        # deal with biblioitemnumber
+        if ( $biblioitem_tag < 10 ) {
+            $newfield = MARC::Field->new( $biblioitem_tag, $biblioitemnumber, );
+        }
+        else {
+            $newfield =
+              MARC::Field->new( $biblioitem_tag, '', '',
+                "$biblioitem_subfield" => $biblioitemnumber, );
+        }
+        # drop old field and create new one...
+        $old_field = $record->field($biblioitem_tag);
+        $record->delete_field($old_field);
+        $record->insert_fields_ordered($newfield);
+
+# biblionumber & biblioitemnumber are in the same field (can't be <10 as fields <10 have only 1 value)
+    }
+    else {
+        my $newfield = MARC::Field->new(
+            $biblio_tag, '', '',
+            "$biblio_subfield" => $biblionumber,
+            "$biblioitem_subfield" => $biblioitemnumber
+        );
+
+        # drop old field and create new one...
+        my $old_field = $record->field($biblio_tag);
+        $record->delete_field($old_field);
+        $record->insert_fields_ordered($newfield);
+    }
+
+    # now add the record
+    my $biblionumber =
+      ModBiblioMarc( $record, $biblionumber, $frameworkcode );
+      
+    &logaction(C4::Context->userenv->{'number'},"CATALOGUING","ADD",$biblionumber,"biblio") 
+        if C4::Context->preference("CataloguingLog");
+      
+    return ( $biblionumber, $biblioitemnumber );
 }
 
+=head2 AddItem
 
-sub MARCgettagslib {
-# warn "MARCgettagslib";
-    my ( $dbh, $forlibrarian, $frameworkcode ) = @_;
-    $frameworkcode = "" unless $frameworkcode;
-    my $sth;
-    my $libfield = ( $forlibrarian eq 1 ) ? 'liblibrarian' : 'libopac';
+=over
 
-    # check that framework exists
-    $sth =
+$biblionumber = AddItem( $record, $biblionumber)
+Exported function (core API) for adding a new item to Koha
+
+=back
+
+=cut
+
+sub AddItem {
+    my ( $record, $biblionumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    
+    # add item in old-DB
+    my $frameworkcode = GetFrameworkCode( $biblionumber );
+    my $item = &TransformMarcToKoha( $dbh, $record, $frameworkcode );
+
+    # needs old biblionumber and biblioitemnumber
+    $item->{'biblionumber'} = $biblionumber;
+    my $sth =
       $dbh->prepare(
-        "select count(*) from biblios_tag_structure where frameworkcode=?");
-    $sth->execute($frameworkcode);
-    my ($total) = $sth->fetchrow;
-    $frameworkcode = "" unless ( $total > 0 );
+        "select biblioitemnumber,itemtype from biblioitems where biblionumber=?"
+      );
+    $sth->execute( $item->{'biblionumber'} );
+    my $itemtype;
+    ( $item->{'biblioitemnumber'}, $itemtype ) = $sth->fetchrow;
     $sth =
       $dbh->prepare(
-"select tagfield,liblibrarian,libopac,mandatory,repeatable from biblios_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;
+        "select notforloan from itemtypes where itemtype='$itemtype'");
+    $sth->execute();
+    my $notforloan = $sth->fetchrow;
+    ##Change the notforloan field if $notforloan found
+    if ( $notforloan > 0 ) {
+        $item->{'notforloan'} = $notforloan;
+        &MARCitemchange( $record, "items.notforloan", $notforloan );
+    }
+    if ( !$item->{'dateaccessioned'} || $item->{'dateaccessioned'} eq '' ) {
+
+        # find today's date
+        my ( $sec, $min, $hour, $mday, $mon, $year, $wday, $yday, $isdst ) =
+          localtime(time);
+        $year += 1900;
+        $mon  += 1;
+        my $date =
+          "$year-" . sprintf( "%0.2d", $mon ) . "-" . sprintf( "%0.2d", $mday );
+        $item->{'dateaccessioned'} = $date;
+        &MARCitemchange( $record, "items.dateaccessioned", $date );
     }
+    my ( $itemnumber, $error ) =
+      &_koha_new_items( $dbh, $item, $item->{barcode} );
 
+    # add itemnumber to MARC::Record before adding the item.
     $sth =
       $dbh->prepare(
-"select tagfield,tagsubfield,liblibrarian,libopac,tab, mandatory, repeatable,authorised_value,authtypecode,value_builder,seealso,hidden,isurl,link from biblios_subfield_structure where frameworkcode=? order by tagfield,tagsubfield"
-    );
-    $sth->execute($frameworkcode);
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+      );
+    &TransformKohaToMarcOneField( $sth, $record, "items.itemnumber", $itemnumber,
+        $frameworkcode );
 
-    my $subfield;
-    my $authorised_value;
-    my $authtypecode;
-    my $value_builder;
-   
-    my $seealso;
-    my $hidden;
-    my $isurl;
-       my $link;
+    # add the item
+    &AddItemInMarc( $record, $item->{'biblionumber'},$frameworkcode );
+    
+    &logaction(C4::Context->userenv->{'number'},"CATALOGUING","ADD",$itemnumber,"item") 
+        if C4::Context->preference("CataloguingLog");
+    
+    return ($item->{biblionumber}, $item->{biblioitemnumber},$itemnumber);
+}
 
-    while (
-        ( $tag,         $subfield,   $liblibrarian,   , $libopac,      $tab,
-        $mandatory,     $repeatable, $authorised_value, $authtypecode,
-        $value_builder,   $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}->{seealso}          = $seealso;
-        $res->{$tag}->{$subfield}->{hidden}           = $hidden;
-        $res->{$tag}->{$subfield}->{isurl}            = $isurl;
-        $res->{$tag}->{$subfield}->{link}            = $link;
+=head2 ModBiblio
+
+=over
+
+ModBiblio( $record,$biblionumber,$frameworkcode);
+Exported function (core API) to modify a biblio
+
+=back
+
+=cut
+
+sub ModBiblio {
+    my ( $record, $biblionumber, $frameworkcode ) = @_;
+    
+    if (C4::Context->preference("CataloguingLog")) {    
+        my $newrecord = GetMarcBiblio($biblionumber);
+        &logaction(C4::Context->userenv->{'number'},"CATALOGUING","MODIFY",$biblionumber,$newrecord->as_formatted) 
     }
-    return $res;
-}
-sub MARCitemsgettagslib {
-# warn "MARCitemsgettagslib";
-    my ( $dbh, $forlibrarian, $frameworkcode ) = @_;
+    
+    my $dbh = C4::Context->dbh;
+    
     $frameworkcode = "" unless $frameworkcode;
-    my $sth;
-    my $libfield = ( $forlibrarian eq 1 ) ? 'liblibrarian' : 'libopac';
 
-    # check that framework exists
-    $sth =
+    # update the MARC record with the new record data
+    &ModBiblioMarc($record, $biblionumber, $frameworkcode );
+
+    # load the koha-table data object
+    my $oldbiblio = TransformMarcToKoha( $dbh, $record, $frameworkcode );
+
+    # modify the other koha tables
+    my $biblionumber = _koha_modify_biblio( $dbh, $oldbiblio );
+    _koha_modify_biblioitem( $dbh, $oldbiblio );
+
+    return 1;
+}
+
+=head2 ModItem
+
+=over
+
+Exported function (core API) for modifying an item in Koha.
+
+=back
+
+=cut
+
+sub ModItem {
+    my ( $record, $biblionumber, $itemnumber, $delete, $new_item_hashref )
+      = @_;
+    
+    #logging
+    &logaction(C4::Context->userenv->{'number'},"CATALOGUING","MODIFY",$itemnumber,$record->as_formatted) 
+        if C4::Context->preference("CataloguingLog");
+      
+    my $dbh = C4::Context->dbh;
+    
+    # if we have a MARC record, we're coming from cataloging and so
+    # we do the whole routine: update the MARC and zebra, then update the koha
+    # tables
+    if ($record) {
+        my $frameworkcode = GetFrameworkCode( $biblionumber );
+        ModItemInMarc( $record, $biblionumber, $itemnumber, $frameworkcode );
+        my $olditem       = TransformMarcToKoha( $dbh, $record, $frameworkcode );
+        _koha_modify_item( $dbh, $olditem );
+        return $biblionumber;
+    }
+
+    # otherwise, we're just looking to modify something quickly
+    # (like a status) so we just update the koha tables
+    elsif ($new_item_hashref) {
+        _koha_modify_item( $dbh, $new_item_hashref );
+    }
+}
+
+sub ModItemTransfer {
+    my ( $itemnumber, $frombranch, $tobranch ) = @_;
+    
+    my $dbh = C4::Context->dbh;
+    
+    #new entry in branchtransfers....
+    my $sth = $dbh->prepare(
+        "INSERT INTO branchtransfers (itemnumber, frombranch, datesent, tobranch)
+        VALUES (?, ?, now(), $?)");
+    $sth->execute($itemnumber, $frombranch, $tobranch);
+    #update holdingbranch in items .....
+     $sth= $dbh->prepare(
+          "UPDATE items set holdingbranch = ? WHERE items.itemnumber = ?");
+    $sth->execute($tobranch,$itemnumber);
+    &ModDateLastSeen($itemnumber);
+    $sth = $dbh->prepare(
+        "SELECT biblionumber FROM items WHERE itemnumber=?"
+      );
+    $sth->execute($itemnumber);
+    while ( my ( $biblionumber ) = $sth->fetchrow ) {
+        &ModItemInMarconefield( $biblionumber, $itemnumber,
+            'items.holdingbranch', $tobranch );
+    }
+    return;
+}
+
+##New sub to dotransfer in marc tables as well. Not exported -TG 10/04/2006
+# sub domarctransfer {
+#     my ( $dbh, $itemnumber ) = @_;
+#     $itemnumber =~ s /\'//g;    ##itemnumber seems to come with quotes-TG
+#     my $sth =
+#       $dbh->prepare(
+#         "select biblionumber,holdingbranch from items where itemnumber=$itemnumber"
+#       );
+#     $sth->execute();
+#     while ( my ( $biblionumber, $holdingbranch ) = $sth->fetchrow ) {
+#         &ModItemInMarconefield( $biblionumber, $itemnumber,
+#             'items.holdingbranch', $holdingbranch );
+#     }
+#     return;
+# }
+=head2 ModBiblioframework
+
+=over
+
+ModBiblioframework($biblionumber,$frameworkcode);
+Exported function to modify a biblio framework
+
+=back
+
+=cut
+
+sub ModBiblioframework {
+    my ( $biblionumber, $frameworkcode ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth =
       $dbh->prepare(
-        "select count(*) from holdings_tag_structure where frameworkcode=?");
+        "UPDATE biblio SET frameworkcode=? WHERE biblionumber=$biblionumber");
     $sth->execute($frameworkcode);
-    my ($total) = $sth->fetchrow;
-    $frameworkcode = "" unless ( $total > 0 );
-    $sth =
+    return 1;
+}
+
+=head2 ModItemInMarconefield
+
+=over
+
+modify only 1 field in a MARC item (mainly used for holdingbranch, but could also be used for status modif - moving a book to "lost" on a long overdu for example)
+&ModItemInMarconefield( $biblionumber, $itemnumber, $itemfield, $newvalue )
+
+=back
+
+=cut
+
+sub ModItemInMarconefield {
+    my ( $biblionumber, $itemnumber, $itemfield, $newvalue ) = @_;
+    my $dbh = C4::Context->dbh;
+    if ( !defined $newvalue ) {
+        $newvalue = "";
+    }
+
+    my $record = GetMarcItem( $biblionumber, $itemnumber );
+    my ($tagfield, $tagsubfield) = GetMarcFromKohaField( $itemfield,'');
+    if ($tagfield && $tagsubfield) {
+        my $tag = $record->field($tagfield);
+        if ($tag) {
+#             my $tagsubs = $record->field($tagfield)->subfield($tagsubfield);
+            $tag->update( $tagsubfield => $newvalue );
+            $record->delete_field($tag);
+            $record->insert_fields_ordered($tag);
+            &ModItemInMarc( $record, $biblionumber, $itemnumber, 0 );
+        }
+    }
+}
+
+=head2 ModItemInMarc
+
+=over
+
+&ModItemInMarc( $record, $biblionumber, $itemnumber )
+
+=back
+
+=cut
+
+sub ModItemInMarc {
+    my ( $ItemRecord, $biblionumber, $itemnumber, $frameworkcode) = @_;
+    my $dbh = C4::Context->dbh;
+    
+    # get complete MARC record & replace the item field by the new one
+    my $completeRecord = GetMarcBiblio($biblionumber);
+    my ($itemtag,$itemsubfield) = GetMarcFromKohaField("items.itemnumber",$frameworkcode);
+    my $itemField = $ItemRecord->field($itemtag);
+    my @items = $completeRecord->field($itemtag);
+    foreach (@items) {
+        if ($_->subfield($itemsubfield) eq $itemnumber) {
+#             $completeRecord->delete_field($_);
+            $_->replace_with($itemField);
+        }
+    }
+    # save the record
+    my $sth = $dbh->prepare("update biblioitems set marc=?,marcxml=?  where biblionumber=?");
+    $sth->execute( $completeRecord->as_usmarc(), $completeRecord->as_xml_record(),$biblionumber );
+    $sth->finish;
+    ModZebra($biblionumber,"specialUpdate","biblioserver");
+}
+
+=head2 ModDateLastSeen
+
+&ModDateLastSeen($itemnum)
+Mark item as seen. Is called when an item is issued, returned or manually marked during inventory/stocktaking
+C<$itemnum> is the item number
+
+=cut
+
+sub ModDateLastSeen {
+    my ($itemnum) = @_;
+    my $dbh       = C4::Context->dbh;
+    my $sth       =
       $dbh->prepare(
-"select tagfield,liblibrarian,libopac,mandatory,repeatable from holdings_tag_structure where frameworkcode=? order by tagfield"
-    );
-    $sth->execute($frameworkcode);
-    my ( $liblibrarian, $libopac, $tag, $res, $tab, $mandatory, $repeatable );
+          "update items set itemlost=0, datelastseen  = now() where items.itemnumber = ?"
+      );
+    $sth->execute($itemnum);
+    return;
+}
+=head2 DelBiblio
 
-    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;
+=over
+
+my $error = &DelBiblio($dbh,$biblionumber);
+Exported function (core API) for deleting a biblio in koha.
+Deletes biblio record from Zebra and Koha tables (biblio,biblioitems,items)
+Also backs it up to deleted* tables
+Checks to make sure there are not issues on any of the items
+return:
+C<$error> : undef unless an error occurs
+
+=back
+
+=cut
+
+sub DelBiblio {
+    my ( $biblionumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $error;    # for error handling
+
+    # First make sure there are no items with issues are still attached
+    my $sth =
+      $dbh->prepare(
+        "SELECT biblioitemnumber FROM biblioitems WHERE biblionumber=?");
+    $sth->execute($biblionumber);
+    while ( my $biblioitemnumber = $sth->fetchrow ) {
+        my @issues = C4::Circulation::Circ2::itemissues($biblioitemnumber);
+        foreach my $issue (@issues) {
+            if (   ( $issue->{date_due} )
+                && ( $issue->{date_due} ne "Available" ) )
+            {
+
+#FIXME: we need a status system in Biblio like in Circ to return standard codes and messages
+# instead of hard-coded strings
+                $error .=
+"Item is checked out to a patron -- you must return it before deleting the Biblio";
+            }
+        }
     }
+    return $error if $error;
+
+    # Delete in Zebra
+    ModZebra($biblionumber,"delete_record","biblioserver");
 
+    # delete biblio from Koha tables and save in deletedbiblio
+    $error = &_koha_delete_biblio( $dbh, $biblionumber );
+
+    # delete biblioitems and items from Koha tables and save in deletedbiblioitems,deleteditems
     $sth =
       $dbh->prepare(
-"select tagfield,tagsubfield,liblibrarian,libopac,tab, mandatory, repeatable,authorised_value,authtypecode,value_builder,seealso,hidden,isurl,link from holdings_subfield_structure where frameworkcode=? order by tagfield,tagsubfield"
-    );
-    $sth->execute($frameworkcode);
+        "SELECT biblioitemnumber FROM biblioitems WHERE biblionumber=?");
+    $sth->execute($biblionumber);
+    while ( my $biblioitemnumber = $sth->fetchrow ) {
+
+        # delete this biblioitem
+        $error = &_koha_delete_biblioitems( $dbh, $biblioitemnumber );
+        return $error if $error;
+
+        # delete items
+        my $items_sth =
+          $dbh->prepare(
+            "SELECT itemnumber FROM items WHERE biblioitemnumber=?");
+        $items_sth->execute($biblioitemnumber);
+        while ( my $itemnumber = $items_sth->fetchrow ) {
+            $error = &_koha_delete_item( $dbh, $itemnumber );
+            return $error if $error;
+        }
+    }
+    &logaction(C4::Context->userenv->{'number'},"CATALOGUING","DELETE",$biblionumber,"") 
+        if C4::Context->preference("CataloguingLog");
+    return;
+}
 
-    my $subfield;
-    my $authorised_value;
-    my $authtypecode;
-    my $value_builder;
-   
-    my $seealso;
-    my $hidden;
-    my $isurl;
-       my $link;
+=head2 DelItem
 
-    while (
-        ( $tag,         $subfield,   $liblibrarian,   , $libopac,      $tab,
-        $mandatory,     $repeatable, $authorised_value, $authtypecode,
-        $value_builder, $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;
+=over
+
+DelItem( $biblionumber, $itemnumber );
+Exported function (core API) for deleting an item record in Koha.
+
+=back
+
+=cut
+
+sub DelItem {
+    my ( $biblionumber, $itemnumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    &_koha_delete_item( $dbh, $itemnumber );
+    # get the MARC record
+    my $record = GetMarcBiblio($biblionumber);
+    my $frameworkcode = GetFrameworkCode($biblionumber);
+
+    # backup the record
+    my $copy2deleted =
+      $dbh->prepare("UPDATE deleteditems SET marc=? WHERE itemnumber=?");
+    $copy2deleted->execute( $record->as_usmarc(), $itemnumber );
+
+    #search item field code
+    my ( $itemtag, $itemsubfield ) = GetMarcFromKohaField("items.itemnumber",$frameworkcode);
+    my @fields = $record->field($itemtag);
+    # delete the item specified
+    foreach my $field (@fields) {
+        if ( $field->subfield($itemsubfield) eq $itemnumber ) {
+            $record->delete_field($field);
+        }
+    }
+    &ModBiblioMarc( $record, $biblionumber, $frameworkcode );
+    &logaction(C4::Context->userenv->{'number'},"CATALOGUING","DELETE",$itemnumber,"item") 
+        if C4::Context->preference("CataloguingLog");
+}
+
+=head2 GetBiblioData
+
+=over 4
+
+$data = &GetBiblioData($biblionumber);
+Returns information about the book with the given biblionumber.
+C<&GetBiblioData> returns a reference-to-hash. The keys are the fields in
+the C<biblio> and C<biblioitems> tables in the
+Koha database.
+In addition, C<$data-E<gt>{subject}> is the list of the book's
+subjects, separated by C<" , "> (space, comma, space).
+If there are multiple biblioitems with the given biblionumber, only
+the first one is considered.
+
+=back
+
+=cut
+
+sub GetBiblioData {
+    my ( $bibnum ) = @_;
+    my $dbh = C4::Context->dbh;
+
+    my $query = "
+        SELECT * , biblioitems.notes AS bnotes, biblio.notes
+        FROM biblio
+            LEFT JOIN biblioitems ON biblio.biblionumber = biblioitems.biblionumber
+            LEFT JOIN itemtypes ON biblioitems.itemtype = itemtypes.itemtype
+        WHERE biblio.biblionumber = ?
+            AND biblioitems.biblionumber = biblio.biblionumber
+    ";
+    my $sth = $dbh->prepare($query);
+    $sth->execute($bibnum);
+    my $data;
+    $data = $sth->fetchrow_hashref;
+    $sth->finish;
+
+    return ($data);
+}    # sub GetBiblioData
+
+
+=head2 GetItemsInfo
+
+=over 4
+
+  @results = &GetItemsInfo($biblionumber, $type);
+
+Returns information about books with the given biblionumber.
+
+C<$type> may be either C<intra> or anything else. If it is not set to
+C<intra>, then the search will exclude lost, very overdue, and
+withdrawn items.
+
+C<&GetItemsInfo> returns a list of references-to-hash. Each element
+contains a number of keys. Most of them are table items from the
+C<biblio>, C<biblioitems>, C<items>, and C<itemtypes> tables in the
+Koha database. Other keys include:
+
+=over 4
+
+=item C<$data-E<gt>{branchname}>
+
+The name (not the code) of the branch to which the book belongs.
+
+=item C<$data-E<gt>{datelastseen}>
+
+This is simply C<items.datelastseen>, except that while the date is
+stored in YYYY-MM-DD format in the database, here it is converted to
+DD/MM/YYYY format. A NULL date is returned as C<//>.
+
+=item C<$data-E<gt>{datedue}>
+
+=item C<$data-E<gt>{class}>
+
+This is the concatenation of C<biblioitems.classification>, the book's
+Dewey code, and C<biblioitems.subclass>.
+
+=item C<$data-E<gt>{ocount}>
+
+I think this is the number of copies of the book available.
+
+=item C<$data-E<gt>{order}>
+
+If this is set, it is set to C<One Order>.
+
+=back
+
+=back
+
+=cut
+
+sub GetItemsInfo {
+    my ( $biblionumber, $type ) = @_;
+    my $dbh   = C4::Context->dbh;
+    my $query = "SELECT *,items.notforloan as itemnotforloan
+                 FROM items, biblio, biblioitems
+                 LEFT JOIN itemtypes on biblioitems.itemtype = itemtypes.itemtype
+                WHERE items.biblionumber = ?
+                    AND biblioitems.biblioitemnumber = items.biblioitemnumber
+                    AND biblio.biblionumber = items.biblionumber
+                ORDER BY items.dateaccessioned desc
+                 ";
+    my $sth = $dbh->prepare($query);
+    $sth->execute($biblionumber);
+    my $i = 0;
+    my @results;
+    my ( $date_due, $count_reserves );
+
+    while ( my $data = $sth->fetchrow_hashref ) {
+        my $datedue = '';
+        my $isth    = $dbh->prepare(
+            "SELECT issues.*,borrowers.cardnumber
+            FROM   issues, borrowers
+            WHERE  itemnumber = ?
+                AND returndate IS NULL
+                AND issues.borrowernumber=borrowers.borrowernumber"
+        );
+        $isth->execute( $data->{'itemnumber'} );
+        if ( my $idata = $isth->fetchrow_hashref ) {
+            $data->{borrowernumber} = $idata->{borrowernumber};
+            $data->{cardnumber}     = $idata->{cardnumber};
+            $datedue                = format_date( $idata->{'date_due'} );
+        }
+        if ( $datedue eq '' ) {
+            #$datedue="Available";
+            my ( $restype, $reserves ) =
+              C4::Reserves::CheckReserves( $data->{'itemnumber'} );
+            if ($restype) {
+
+                #$datedue=$restype;
+                $count_reserves = $restype;
+            }
+        }
+        $isth->finish;
+
+        #get branch information.....
+        my $bsth = $dbh->prepare(
+            "SELECT * FROM branches WHERE branchcode = ?
+        "
+        );
+        $bsth->execute( $data->{'holdingbranch'} );
+        if ( my $bdata = $bsth->fetchrow_hashref ) {
+            $data->{'branchname'} = $bdata->{'branchname'};
+        }
+        my $date = format_date( $data->{'datelastseen'} );
+        $data->{'datelastseen'}   = $date;
+        $data->{'datedue'}        = $datedue;
+        $data->{'count_reserves'} = $count_reserves;
+
+        # get notforloan complete status if applicable
+        my $sthnflstatus = $dbh->prepare(
+            'SELECT authorised_value
+            FROM   marc_subfield_structure
+            WHERE  kohafield="items.notforloan"
+        '
+        );
+
+        $sthnflstatus->execute;
+        my ($authorised_valuecode) = $sthnflstatus->fetchrow;
+        if ($authorised_valuecode) {
+            $sthnflstatus = $dbh->prepare(
+                "SELECT lib FROM authorised_values
+                 WHERE  category=?
+                 AND authorised_value=?"
+            );
+            $sthnflstatus->execute( $authorised_valuecode,
+                $data->{itemnotforloan} );
+            my ($lib) = $sthnflstatus->fetchrow;
+            $data->{notforloan} = $lib;
+        }
+
+        # my stack procedures
+        my $stackstatus = $dbh->prepare(
+            'SELECT authorised_value
+             FROM   marc_subfield_structure
+             WHERE  kohafield="items.stack"
+        '
+        );
+        $stackstatus->execute;
+
+        ($authorised_valuecode) = $stackstatus->fetchrow;
+        if ($authorised_valuecode) {
+            $stackstatus = $dbh->prepare(
+                "SELECT lib
+                 FROM   authorised_values
+                 WHERE  category=?
+                 AND    authorised_value=?
+            "
+            );
+            $stackstatus->execute( $authorised_valuecode, $data->{stack} );
+            my ($lib) = $stackstatus->fetchrow;
+            $data->{stack} = $lib;
+        }
+        $results[$i] = $data;
+        $i++;
+    }
+    $sth->finish;
+
+    return (@results);
+}
+
+=head2 getitemstatus
+
+=over 4
+
+$itemstatushash = &getitemstatus($fwkcode);
+returns information about status.
+Can be MARC dependant.
+fwkcode is optional.
+But basically could be can be loan or not
+Create a status selector with the following code
+
+=head3 in PERL SCRIPT
+
+my $itemstatushash = getitemstatus;
+my @itemstatusloop;
+foreach my $thisstatus (keys %$itemstatushash) {
+    my %row =(value => $thisstatus,
+                statusname => $itemstatushash->{$thisstatus}->{'statusname'},
+            );
+    push @itemstatusloop, \%row;
+}
+$template->param(statusloop=>\@itemstatusloop);
+
+
+=head3 in TEMPLATE
+
+            <select name="statusloop">
+                <option value="">Default</option>
+            <!-- TMPL_LOOP name="statusloop" -->
+                <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="statusname" --></option>
+            <!-- /TMPL_LOOP -->
+            </select>
+
+=cut
+
+sub GetItemStatus {
+
+    # returns a reference to a hash of references to status...
+    my ($fwk) = @_;
+    my %itemstatus;
+    my $dbh = C4::Context->dbh;
+    my $sth;
+    $fwk = '' unless ($fwk);
+    my ( $tag, $subfield ) =
+      GetMarcFromKohaField( "items.notforloan", $fwk );
+    if ( $tag and $subfield ) {
+        my $sth =
+          $dbh->prepare(
+"select authorised_value from marc_subfield_structure where tagfield=? and tagsubfield=? and frameworkcode=?"
+          );
+        $sth->execute( $tag, $subfield, $fwk );
+        if ( my ($authorisedvaluecat) = $sth->fetchrow ) {
+            my $authvalsth =
+              $dbh->prepare(
+"select authorised_value, lib from authorised_values where category=? order by lib"
+              );
+            $authvalsth->execute($authorisedvaluecat);
+            while ( my ( $authorisedvalue, $lib ) = $authvalsth->fetchrow ) {
+                $itemstatus{$authorisedvalue} = $lib;
+            }
+            $authvalsth->finish;
+            return \%itemstatus;
+            exit 1;
+        }
+        else {
+
+            #No authvalue list
+            # build default
+        }
+        $sth->finish;
+    }
+
+    #No authvalue list
+    #build default
+    $itemstatus{"1"} = "Not For Loan";
+    return \%itemstatus;
+}
+
+=head2 getitemlocation
+
+=over 4
+
+$itemlochash = &getitemlocation($fwk);
+returns informations about location.
+where fwk stands for an optional framework code.
+Create a location selector with the following code
+
+=head3 in PERL SCRIPT
+
+my $itemlochash = getitemlocation;
+my @itemlocloop;
+foreach my $thisloc (keys %$itemlochash) {
+    my $selected = 1 if $thisbranch eq $branch;
+    my %row =(locval => $thisloc,
+                selected => $selected,
+                locname => $itemlochash->{$thisloc},
+            );
+    push @itemlocloop, \%row;
+}
+$template->param(itemlocationloop => \@itemlocloop);
+
+=head3 in TEMPLATE
+
+<select name="location">
+    <option value="">Default</option>
+<!-- TMPL_LOOP name="itemlocationloop" -->
+    <option value="<!-- TMPL_VAR name="locval" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="locname" --></option>
+<!-- /TMPL_LOOP -->
+</select>
+
+=back
+
+=cut
+
+sub GetItemLocation {
+
+    # returns a reference to a hash of references to location...
+    my ($fwk) = @_;
+    my %itemlocation;
+    my $dbh = C4::Context->dbh;
+    my $sth;
+    $fwk = '' unless ($fwk);
+    my ( $tag, $subfield ) =
+      GetMarcFromKohaField( "items.location", $fwk );
+    if ( $tag and $subfield ) {
+        my $sth =
+          $dbh->prepare(
+"select authorised_value from marc_subfield_structure where tagfield=? and tagsubfield=? and frameworkcode=?"
+          );
+        $sth->execute( $tag, $subfield, $fwk );
+        if ( my ($authorisedvaluecat) = $sth->fetchrow ) {
+            my $authvalsth =
+              $dbh->prepare(
+"select authorised_value, lib from authorised_values where category=? order by lib"
+              );
+            $authvalsth->execute($authorisedvaluecat);
+            while ( my ( $authorisedvalue, $lib ) = $authvalsth->fetchrow ) {
+                $itemlocation{$authorisedvalue} = $lib;
+            }
+            $authvalsth->finish;
+            return \%itemlocation;
+            exit 1;
+        }
+        else {
+
+            #No authvalue list
+            # build default
+        }
+        $sth->finish;
+    }
+
+    #No authvalue list
+    #build default
+    $itemlocation{"1"} = "Not For Loan";
+    return \%itemlocation;
+}
+
+=head2 GetLostItems
+
+$items = GetLostItems($where,$orderby);
+
+This function get the items lost into C<$items>.
+
+=over 2
+
+=item input:
+C<$where> is a hashref. it containts a field of the items table as key
+and the value to match as value.
+C<$orderby> is a field of the items table.
+
+=item return:
+C<$items> is a reference to an array full of hasref which keys are items' table column.
+
+=item usage in the perl script:
+
+my %where;
+$where{barcode} = 0001548;
+my $items = GetLostItems( \%where, "homebranch" );
+$template->param(itemsloop => $items);
+
+=back
+
+=cut
+
+sub GetLostItems {
+    # Getting input args.
+    my $where   = shift;
+    my $orderby = shift;
+    my $dbh     = C4::Context->dbh;
+
+    my $query   = "
+        SELECT *
+        FROM   items
+        WHERE  itemlost IS NOT NULL
+          AND  itemlost <> 0
+    ";
+    foreach my $key (keys %$where) {
+        $query .= " AND " . $key . " LIKE '%" . $where->{$key} . "%'";
+    }
+    $query .= " ORDER BY ".$orderby if defined $orderby;
+
+    my $sth = $dbh->prepare($query);
+    $sth->execute;
+    my @items;
+    while ( my $row = $sth->fetchrow_hashref ){
+        push @items, $row;
+    }
+    return \@items;
+}
+
+=head2 GetItemsForInventory
+
+$itemlist = GetItemsForInventory($minlocation,$maxlocation,$datelastseen,$offset,$size)
+
+Retrieve a list of title/authors/barcode/callnumber, for biblio inventory.
+
+The sub returns a list of hashes, containing itemnumber, author, title, barcode & item callnumber.
+It is ordered by callnumber,title.
+
+The minlocation & maxlocation parameters are used to specify a range of item callnumbers
+the datelastseen can be used to specify that you want to see items not seen since a past date only.
+offset & size can be used to retrieve only a part of the whole listing (defaut behaviour)
+
+=cut
+
+sub GetItemsForInventory {
+    my ( $minlocation, $maxlocation, $datelastseen, $branch, $offset, $size ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth;
+    if ($datelastseen) {
+        my $query =
+                "SELECT itemnumber,barcode,itemcallnumber,title,author,datelastseen
+                 FROM items
+                   LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber 
+                 WHERE itemcallnumber>= ?
+                   AND itemcallnumber <=?
+                   AND (datelastseen< ? OR datelastseen IS NULL)";
+        $query.= " AND items.homebranch=".$dbh->quote($branch) if $branch;
+        $query .= " ORDER BY itemcallnumber,title";
+        $sth = $dbh->prepare($query);
+        $sth->execute( $minlocation, $maxlocation, $datelastseen );
+    }
+    else {
+        my $query ="
+                SELECT itemnumber,barcode,itemcallnumber,title,author,datelastseen
+                FROM items 
+                  LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber 
+                WHERE itemcallnumber>= ?
+                  AND itemcallnumber <=?";
+        $query.= " AND items.homebranch=".$dbh->quote($branch) if $branch;
+        $query .= " ORDER BY itemcallnumber,title";
+        $sth = $dbh->prepare($query);
+        $sth->execute( $minlocation, $maxlocation );
+    }
+    my @results;
+    while ( my $row = $sth->fetchrow_hashref ) {
+        $offset-- if ($offset);
+        if ( ( !$offset ) && $size ) {
+            push @results, $row;
+            $size--;
+        }
+    }
+    return \@results;
+}
+
+=head2 &GetBiblioItemData
+
+=over 4
+
+$itemdata = &GetBiblioItemData($biblioitemnumber);
+
+Looks up the biblioitem with the given biblioitemnumber. Returns a
+reference-to-hash. The keys are the fields from the C<biblio>,
+C<biblioitems>, and C<itemtypes> tables in the Koha database, except
+that C<biblioitems.notes> is given as C<$itemdata-E<gt>{bnotes}>.
+
+=back
+
+=cut
+
+#'
+sub GetBiblioItemData {
+    my ($bibitem) = @_;
+    my $dbh       = C4::Context->dbh;
+    my $sth       =
+      $dbh->prepare(
+"Select *,biblioitems.notes as bnotes from biblioitems, biblio,itemtypes where biblio.biblionumber = biblioitems.biblionumber and biblioitemnumber = ? and biblioitems.itemtype = itemtypes.itemtype"
+      );
+    my $data;
+
+    $sth->execute($bibitem);
+
+    $data = $sth->fetchrow_hashref;
+
+    $sth->finish;
+    return ($data);
+}    # sub &GetBiblioItemData
+
+=head2 GetItemnumberFromBarcode
+
+=over 4
+
+$result = GetItemnumberFromBarcode($barcode);
+
+=back
+
+=cut
+
+sub GetItemnumberFromBarcode {
+    my ($barcode) = @_;
+    my $dbh = C4::Context->dbh;
+
+    my $rq =
+      $dbh->prepare("SELECT itemnumber from items where items.barcode=?");
+    $rq->execute($barcode);
+    my ($result) = $rq->fetchrow;
+    return ($result);
+}
+
+=head2 GetBiblioItemByBiblioNumber
+
+=over 4
+
+NOTE : This function has been copy/paste from C4/Biblio.pm from head before zebra integration.
+
+=back
+
+=cut
+
+sub GetBiblioItemByBiblioNumber {
+    my ($biblionumber) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("Select * from biblioitems where biblionumber = ?");
+    my $count = 0;
+    my @results;
+
+    $sth->execute($biblionumber);
+
+    while ( my $data = $sth->fetchrow_hashref ) {
+        push @results, $data;
+    }
+
+    $sth->finish;
+    return @results;
+}
+
+=head2 GetBiblioFromItemNumber
+
+=over 4
+
+$item = &GetBiblioFromItemNumber($itemnumber);
+
+Looks up the item with the given itemnumber.
+
+C<&itemnodata> returns a reference-to-hash whose keys are the fields
+from the C<biblio>, C<biblioitems>, and C<items> tables in the Koha
+database.
+
+=back
+
+=cut
+
+#'
+sub GetBiblioFromItemNumber {
+    my ( $itemnumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare(
+        "SELECT * FROM biblio,items,biblioitems
+         WHERE items.itemnumber = ?
+           AND biblio.biblionumber = items.biblionumber
+           AND biblioitems.biblioitemnumber = items.biblioitemnumber"
+    );
+
+    $sth->execute($itemnumber);
+    my $data = $sth->fetchrow_hashref;
+    $sth->finish;
+    return ($data);
+}
+
+=head2 GetBiblio
+
+=over 4
+
+( $count, @results ) = &GetBiblio($biblionumber);
+
+=back
+
+=cut
+
+sub GetBiblio {
+    my ($biblionumber) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("Select * from biblio where biblionumber = ?");
+    my $count = 0;
+    my @results;
+    $sth->execute($biblionumber);
+    while ( my $data = $sth->fetchrow_hashref ) {
+        $results[$count] = $data;
+        $count++;
+    }    # while
+    $sth->finish;
+    return ( $count, @results );
+}    # sub GetBiblio
+
+=head2 GetItem
+
+=over 4
+
+$data = &GetItem($itemnumber,$barcode);
+
+return Item information, for a given itemnumber or barcode
+
+=back
+
+=cut
+
+sub GetItem {
+    my ($itemnumber,$barcode) = @_;
+    my $dbh = C4::Context->dbh;
+    if ($itemnumber) {
+        my $sth = $dbh->prepare("
+            SELECT * FROM items 
+            WHERE itemnumber = ?");
+        $sth->execute($itemnumber);
+        my $data = $sth->fetchrow_hashref;
+        return $data;
+    } else {
+        my $sth = $dbh->prepare("
+            SELECT * FROM items 
+            WHERE barcode = ?"
+            );
+        $sth->execute($barcode);
+        my $data = $sth->fetchrow_hashref;
+        return $data;
+    }
+}    # sub GetItem
+
+=head2 get_itemnumbers_of
+
+=over 4
+
+my @itemnumbers_of = get_itemnumbers_of(@biblionumbers);
+
+Given a list of biblionumbers, return the list of corresponding itemnumbers
+for each biblionumber.
+
+Return a reference on a hash where keys are biblionumbers and values are
+references on array of itemnumbers.
+
+=back
+
+=cut
+
+sub get_itemnumbers_of {
+    my @biblionumbers = @_;
+
+    my $dbh = C4::Context->dbh;
+
+    my $query = '
+        SELECT itemnumber,
+            biblionumber
+        FROM items
+        WHERE biblionumber IN (?' . ( ',?' x scalar @biblionumbers - 1 ) . ')
+    ';
+    my $sth = $dbh->prepare($query);
+    $sth->execute(@biblionumbers);
+
+    my %itemnumbers_of;
+
+    while ( my ( $itemnumber, $biblionumber ) = $sth->fetchrow_array ) {
+        push @{ $itemnumbers_of{$biblionumber} }, $itemnumber;
+    }
+
+    return \%itemnumbers_of;
+}
+
+=head2 GetItemInfosOf
+
+=over 4
+
+GetItemInfosOf(@itemnumbers);
+
+=back
+
+=cut
+
+sub GetItemInfosOf {
+    my @itemnumbers = @_;
+
+    my $query = '
+        SELECT *
+        FROM items
+        WHERE itemnumber IN (' . join( ',', @itemnumbers ) . ')
+    ';
+    return get_infos_of( $query, 'itemnumber' );
+}
+
+=head2 GetBiblioItemInfosOf
+
+=over 4
+
+GetBiblioItemInfosOf(@biblioitemnumbers);
+
+=back
+
+=cut
+
+sub GetBiblioItemInfosOf {
+    my @biblioitemnumbers = @_;
+
+    my $query = '
+        SELECT biblioitemnumber,
+            publicationyear,
+            itemtype
+        FROM biblioitems
+        WHERE biblioitemnumber IN (' . join( ',', @biblioitemnumbers ) . ')
+    ';
+    return get_infos_of( $query, 'biblioitemnumber' );
+}
+
+=head1 FUNCTIONS FOR HANDLING MARC MANAGEMENT
+
+=head2 GetMarcStructure
+
+=over 4
+
+$res = GetMarcStructure($dbh,$forlibrarian,$frameworkcode);
+
+Returns a reference to a big hash of hash, with the Marc structure fro the given frameworkcode
+$dbh : DB handler
+$forlibrarian  :if set to 1, the MARC descriptions are the librarians ones, otherwise it's the public (OPAC) ones
+$frameworkcode : the framework code to read
+
+=back
+
+=back
+
+=cut
+
+sub GetMarcStructure {
+    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;
+          # why the hell do we need to explicitly decode utf8 ? 
+          # that's a good question, but we must do it...
+          use utf8;
+          utf8::decode($res->{$tag}->{lib});
+#           warn "$liblibrarian";
+        $res->{$tab}->{tab}        = "";            # XXX
+        $res->{$tag}->{mandatory}  = $mandatory;
+        $res->{$tag}->{repeatable} = $repeatable;
+    }
+
+    $sth =
+      $dbh->prepare(
+"select tagfield,tagsubfield,liblibrarian,libopac,tab, mandatory, repeatable,authorised_value,authtypecode,value_builder,kohafield,seealso,hidden,isurl,link,defaultvalue 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;
+    my $defaultvalue;
+
+    while (
+        (
+            $tag,          $subfield,      $liblibrarian,
+            ,              $libopac,       $tab,
+            $mandatory,    $repeatable,    $authorised_value,
+            $authtypecode, $value_builder, $kohafield,
+            $seealso,      $hidden,        $isurl,
+            $link,$defaultvalue
+        )
+        = $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;
+        $res->{$tag}->{$subfield}->{link}             = $link;
+        $res->{$tag}->{$subfield}->{defaultvalue}     = $defaultvalue;
+    }
+    return $res;
+}
+
+=head2 GetMarcFromKohaField
+
+=over 4
+
+($MARCfield,$MARCsubfield)=GetMarcFromKohaField($kohafield,$frameworkcode);
+Returns the MARC fields & subfields mapped to the koha field 
+for the given frameworkcode
+
+=back
+
+=cut
+
+sub GetMarcFromKohaField {
+    my ( $kohafield, $frameworkcode ) = @_;
+    return 0, 0 unless $kohafield;
+    my $relations = C4::Context->marcfromkohafield;
+    return (
+        $relations->{$frameworkcode}->{$kohafield}->[0],
+        $relations->{$frameworkcode}->{$kohafield}->[1]
+    );
+}
+
+=head2 GetMarcBiblio
+
+=over 4
+
+Returns MARC::Record of the biblionumber passed in parameter.
+the marc record contains both biblio & item datas
+
+=back
+
+=cut
+
+sub GetMarcBiblio {
+    my $biblionumber = shift;
+    my $dbh          = C4::Context->dbh;
+    my $sth          =
+      $dbh->prepare("select marcxml from biblioitems where biblionumber=? ");
+    $sth->execute($biblionumber);
+    my ($marcxml) = $sth->fetchrow;
+    MARC::File::XML->default_record_format(C4::Context->preference('marcflavour'));
+    $marcxml =~ s/\x1e//g;
+    $marcxml =~ s/\x1f//g;
+    $marcxml =~ s/\x1d//g;
+    $marcxml =~ s/\x0f//g;
+    $marcxml =~ s/\x0c//g;
+    my $record = MARC::Record->new();
+    $record = MARC::Record::new_from_xml( $marcxml, "utf8",C4::Context->preference('marcflavour')) if $marcxml;
+    return $record;
+}
+
+=head2 GetXmlBiblio
+
+=over 4
+
+my $marcxml = GetXmlBiblio($biblionumber);
+
+Returns biblioitems.marcxml of the biblionumber passed in parameter.
+The XML contains both biblio & item datas
+
+=back
+
+=cut
+
+sub GetXmlBiblio {
+    my ( $biblionumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth =
+      $dbh->prepare("select marcxml from biblioitems where biblionumber=? ");
+    $sth->execute($biblionumber);
+    my ($marcxml) = $sth->fetchrow;
+    return $marcxml;
+}
+
+=head2 GetAuthorisedValueDesc
+
+=over 4
+
+my $subfieldvalue =get_authorised_value_desc(
+    $tag, $subf[$i][0],$subf[$i][1], '', $taglib);
+Retrieve the complete description for a given authorised value.
+
+=back
+
+=cut
+
+sub GetAuthorisedValueDesc {
+    my ( $tag, $subfield, $value, $framework, $tagslib ) = @_;
+    my $dbh = C4::Context->dbh;
+    
+    #---- branch
+    if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "branches" ) {
+        return C4::Branch::GetBranchName($value);
+    }
+
+    #---- itemtypes
+    if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq "itemtypes" ) {
+        return getitemtypeinfo($value);
+    }
+
+    #---- "true" authorized value
+    my $category = $tagslib->{$tag}->{$subfield}->{'authorised_value'};
+
+    if ( $category ne "" ) {
+        my $sth =
+          $dbh->prepare(
+            "select lib from authorised_values where category = ? and authorised_value = ?"
+          );
+        $sth->execute( $category, $value );
+        my $data = $sth->fetchrow_hashref;
+        return $data->{'lib'};
+    }
+    else {
+        return $value;    # if nothing is found return the original value
+    }
+}
+
+=head2 GetMarcItem
+
+=over 4
+
+Returns MARC::Record of the item passed in parameter.
+
+=back
+
+=cut
+
+sub GetMarcItem {
+    my ( $biblionumber, $itemnumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $newrecord = MARC::Record->new();
+    my $marcflavour = C4::Context->preference('marcflavour');
+    
+    my $marcxml = GetXmlBiblio($biblionumber);
+    my $record = MARC::Record->new();
+    $record = MARC::Record::new_from_xml( $marcxml, "utf8", $marcflavour );
+    # now, find where the itemnumber is stored & extract only the item
+    my ( $itemnumberfield, $itemnumbersubfield ) =
+      GetMarcFromKohaField( 'items.itemnumber', '' );
+    my @fields = $record->field($itemnumberfield);
+    foreach my $field (@fields) {
+        if ( $field->subfield($itemnumbersubfield) eq $itemnumber ) {
+            $newrecord->insert_fields_ordered($field);
+        }
+    }
+    return $newrecord;
+}
+
+
+
+=head2 GetMarcNotes
+
+=over 4
+
+$marcnotesarray = GetMarcNotes( $record, $marcflavour );
+Get all notes from the MARC record and returns them in an array.
+The note are stored in differents places depending on MARC flavour
+
+=back
+
+=cut
+
+sub GetMarcNotes {
+    my ( $record, $marcflavour ) = @_;
+    my $scope;
+    if ( $marcflavour eq "MARC21" ) {
+        $scope = '5..';
+    }
+    else {    # assume unimarc if not marc21
+        $scope = '3..';
+    }
+    my @marcnotes;
+    my $note = "";
+    my $tag  = "";
+    my $marcnote;
+    foreach my $field ( $record->field($scope) ) {
+        my $value = $field->as_string();
+        if ( $note ne "" ) {
+            $marcnote = { marcnote => $note, };
+            push @marcnotes, $marcnote;
+            $note = $value;
+        }
+        if ( $note ne $value ) {
+            $note = $note . " " . $value;
+        }
+    }
+
+    if ( $note ) {
+        $marcnote = { marcnote => $note };
+        push @marcnotes, $marcnote;    #load last tag into array
+    }
+    return \@marcnotes;
+}    # end GetMarcNotes
+
+=head2 GetMarcSubjects
+
+=over 4
+
+$marcsubjcts = GetMarcSubjects($record,$marcflavour);
+Get all subjects from the MARC record and returns them in an array.
+The subjects are stored in differents places depending on MARC flavour
+
+=back
+
+=cut
+
+sub GetMarcSubjects {
+    my ( $record, $marcflavour ) = @_;
+    my ( $mintag, $maxtag );
+    if ( $marcflavour eq "MARC21" ) {
+        $mintag = "600";
+        $maxtag = "699";
+    }
+    else {    # assume unimarc if not marc21
+        $mintag = "600";
+        $maxtag = "611";
+    }
+
+    my @marcsubjcts;
+
+    foreach my $field ( $record->fields ) {
+        next unless $field->tag() >= $mintag && $field->tag() <= $maxtag;
+        my @subfields = $field->subfields();
+        my $link;
+        my $label = "su:";
+        my $flag = 0;
+        for my $subject_subfield ( @subfields ) {
+            my $code = $subject_subfield->[0];
+            $label .= $subject_subfield->[1] . " and su-to:" unless ( $code == 9 );
+            if ( $code == 9 ) {
+                $link = "Koha-Auth-Number:".$subject_subfield->[1];
+                $flag = 1;
+            }
+            elsif ( ! $flag ) {
+                $link = $label;
+                $link =~ s/ and\ssu-to:$//;
+            }
+        }
+        $label =~ s/su/ /g;
+        $label =~ s/://g;
+        $label =~ s/-to//g;
+        $label =~ s/and//g;
+        push @marcsubjcts,
+          {
+            label => $label,
+            link  => $link
+          }
+    }
+    return \@marcsubjcts;
+}    #end GetMarcSubjects
+
+=head2 GetMarcAuthors
+
+=over 4
+
+authors = GetMarcAuthors($record,$marcflavour);
+Get all authors from the MARC record and returns them in an array.
+The authors are stored in differents places depending on MARC flavour
+
+=back
+
+=cut
+
+sub GetMarcAuthors {
+    my ( $record, $marcflavour ) = @_;
+    my ( $mintag, $maxtag );
+    if ( $marcflavour eq "MARC21" ) {
+        $mintag = "100";
+        $maxtag = "111"; 
+    }
+    else {    # assume unimarc if not marc21
+        $mintag = "701";
+        $maxtag = "712";
+    }
+
+    my @marcauthors;
+
+    foreach my $field ( $record->fields ) {
+        next unless $field->tag() >= $mintag && $field->tag() <= $maxtag;
+        my %hash;
+        my @subfields = $field->subfields();
+        my $count_auth = 0;
+        my $and ;
+        for my $authors_subfield (@subfields) {
+               if ($count_auth ne '0'){
+               $and = " and au:";
+               }
+            $count_auth++;
+            my $subfieldcode     = $authors_subfield->[0];
+            my $value            = $authors_subfield->[1];
+            $hash{'tag'}         = $field->tag;
+            $hash{value}        .= $value . " " if ($subfieldcode != 9) ;
+            $hash{link}        .= $value if ($subfieldcode eq 9);
+        }
+        push @marcauthors, \%hash;
+    }
+    return \@marcauthors;
+}
+
+=head2 GetMarcSeries
+
+=over 4
+
+$marcseriessarray = GetMarcSeries($record,$marcflavour);
+Get all series from the MARC record and returns them in an array.
+The series are stored in differents places depending on MARC flavour
+
+=back
+
+=cut
+
+sub GetMarcSeries {
+    my ($record, $marcflavour) = @_;
+    my ($mintag, $maxtag);
+    if ($marcflavour eq "MARC21") {
+        $mintag = "440";
+        $maxtag = "490";
+    } else {           # assume unimarc if not marc21
+        $mintag = "600";
+        $maxtag = "619";
+    }
+
+    my @marcseries;
+    my $subjct = "";
+    my $subfield = "";
+    my $marcsubjct;
+
+    foreach my $field ($record->field('440'), $record->field('490')) {
+        my @subfields_loop;
+        #my $value = $field->subfield('a');
+        #$marcsubjct = {MARCSUBJCT => $value,};
+        my @subfields = $field->subfields();
+        #warn "subfields:".join " ", @$subfields;
+        my $counter = 0;
+        my @link_loop;
+        for my $series_subfield (@subfields) {
+                       my $volume_number;
+                       undef $volume_number;
+                       # see if this is an instance of a volume
+                       if ($series_subfield->[0] eq 'v') {
+                               $volume_number=1;
+                       }
+
+            my $code = $series_subfield->[0];
+            my $value = $series_subfield->[1];
+            my $linkvalue = $value;
+            $linkvalue =~ s/(\(|\))//g;
+            my $operator = " and " unless $counter==0;
+            push @link_loop, {link => $linkvalue, operator => $operator };
+            my $separator = C4::Context->preference("authoritysep") unless $counter==0;
+                       if ($volume_number) {
+                       push @subfields_loop, {volumenum => $value};
+                       }
+                       else {
+            push @subfields_loop, {code => $code, value => $value, link_loop => \@link_loop, separator => $separator, volumenum => $volume_number};
+                       }
+            $counter++;
+        }
+        push @marcseries, { MARCSERIES_SUBFIELDS_LOOP => \@subfields_loop };
+        #$marcsubjct = {MARCSUBJCT => $field->as_string(),};
+        #push @marcsubjcts, $marcsubjct;
+        #$subjct = $value;
+
+    }
+    my $marcseriessarray=\@marcseries;
+    return $marcseriessarray;
+}  #end getMARCseriess
+
+=head2 GetFrameworkCode
+
+=over 4
+
+$frameworkcode = GetFrameworkCode( $biblionumber )
+
+=back
+
+=cut
+
+sub GetFrameworkCode {
+    my ( $biblionumber ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth =
+      $dbh->prepare("select frameworkcode from biblio where biblionumber=?");
+    $sth->execute($biblionumber);
+    my ($frameworkcode) = $sth->fetchrow;
+    return $frameworkcode;
+}
+
+=head2 TransformKohaToMarc
+
+=over 4
+
+$record = TransformKohaToMarc( $hash )
+This function builds partial MARC::Record from a hash
+Hash entries can be from biblio or biblioitems.
+This function is called in acquisition module, to create a basic catalogue entry from user entry
+
+=back
+
+=cut
+
+sub TransformKohaToMarc {
+
+    my ( $hash ) = @_;
+    my $dbh = C4::Context->dbh;
+    my $sth =
+    $dbh->prepare(
+        "select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+    );
+    my $record = MARC::Record->new();
+    foreach (keys %{$hash}) {
+        &TransformKohaToMarcOneField( $sth, $record, $_,
+            $hash->{$_}, '' );
+        }
+    return $record;
+}
+
+=head2 TransformKohaToMarcOneField
+
+=over 4
+
+$record = TransformKohaToMarcOneField( $sth, $record, $kohafieldname, $value, $frameworkcode );
+
+=back
+
+=cut
+
+sub TransformKohaToMarcOneField {
+    my ( $sth, $record, $kohafieldname, $value, $frameworkcode ) = @_;
+    $frameworkcode='' unless $frameworkcode;
+    my $tagfield;
+    my $tagsubfield;
+
+    if ( !defined $sth ) {
+        my $dbh = C4::Context->dbh;
+        $sth =
+          $dbh->prepare(
+"select tagfield,tagsubfield from marc_subfield_structure where frameworkcode=? and kohafield=?"
+          );
     }
-    return $res;
+    $sth->execute( $frameworkcode, $kohafieldname );
+    if ( ( $tagfield, $tagsubfield ) = $sth->fetchrow ) {
+        my $tag = $record->field($tagfield);
+        if ($tag) {
+            $tag->update( $tagsubfield => $value );
+            $record->delete_field($tag);
+            $record->insert_fields_ordered($tag);
+        }
+        else {
+            $record->add_fields( $tagfield, " ", " ", $tagsubfield => $value );
+        }
+    }
+    return $record;
 }
-sub MARCfind_marc_from_kohafield {
-# warn "MARCfind_marc_from_kohafield";
-    my (  $kohafield,$recordtype) = @_;
-    return 0, 0 unless $kohafield;
-$recordtype="biblios" unless $recordtype;
-       my $relations = C4::Context->marcfromkohafield;
-       return ($relations->{$recordtype}->{$kohafield}->[0],$relations->{$recordtype}->{$kohafield}->[1]);
+
+=head2 TransformHtmlToXml
+
+=over 4
+
+$xml = TransformHtmlToXml( $tags, $subfields, $values, $indicator, $ind_tag )
+
+=back
+
+=cut
+
+sub TransformHtmlToXml {
+    my ( $tags, $subfields, $values, $indicator, $ind_tag ) = @_;
+    my $xml = MARC::File::XML::header('UTF-8');
+    if ( C4::Context->preference('marcflavour') eq 'UNIMARC' ) {
+        MARC::File::XML->default_record_format('UNIMARC');
+        use POSIX qw(strftime);
+        my $string = strftime( "%Y%m%d", localtime(time) );
+        $string = sprintf( "%-*s", 35, $string );
+        substr( $string, 22, 6, "frey50" );
+        $xml .= "<datafield tag=\"100\" ind1=\"\" ind2=\"\">\n";
+        $xml .= "<subfield code=\"a\">$string</subfield>\n";
+        $xml .= "</datafield>\n";
+    }
+    my $prevvalue;
+    my $prevtag = -1;
+    my $first   = 1;
+    my $j       = -1;
+    for ( my $i = 0 ; $i <= @$tags ; $i++ ) {
+        @$values[$i] =~ s/&/&amp;/g;
+        @$values[$i] =~ s/</&lt;/g;
+        @$values[$i] =~ s/>/&gt;/g;
+        @$values[$i] =~ s/"/&quot;/g;
+        @$values[$i] =~ s/'/&apos;/g;
+        if ( !utf8::is_utf8( @$values[$i] ) ) {
+            utf8::decode( @$values[$i] );
+        }
+        if ( ( @$tags[$i] ne $prevtag ) ) {
+            $j++ unless ( @$tags[$i] eq "" );
+            if ( !$first ) {
+                $xml .= "</datafield>\n";
+                if (   ( @$tags[$i] && @$tags[$i] > 10 )
+                    && ( @$values[$i] ne "" ) )
+                {
+                    my $ind1 = substr( @$indicator[$j], 0, 1 );
+                    my $ind2;
+                    if ( @$indicator[$j] ) {
+                        $ind2 = substr( @$indicator[$j], 1, 1 );
+                    }
+                    else {
+                        warn "Indicator in @$tags[$i] is empty";
+                        $ind2 = " ";
+                    }
+                    $xml .=
+"<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
+                    $xml .=
+"<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
+                    $first = 0;
+                }
+                else {
+                    $first = 1;
+                }
+            }
+            else {
+                if ( @$values[$i] ne "" ) {
+
+                    # leader
+                    if ( @$tags[$i] eq "000" ) {
+                        $xml .= "<leader>@$values[$i]</leader>\n";
+                        $first = 1;
+
+                        # rest of the fixed fields
+                    }
+                    elsif ( @$tags[$i] < 10 ) {
+                        $xml .=
+"<controlfield tag=\"@$tags[$i]\">@$values[$i]</controlfield>\n";
+                        $first = 1;
+                    }
+                    else {
+                        my $ind1 = substr( @$indicator[$j], 0, 1 );
+                        my $ind2 = substr( @$indicator[$j], 1, 1 );
+                        $xml .=
+"<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
+                        $xml .=
+"<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
+                        $first = 0;
+                    }
+                }
+            }
+        }
+        else {    # @$tags[$i] eq $prevtag
+            if ( @$values[$i] eq "" ) {
+            }
+            else {
+                if ($first) {
+                    my $ind1 = substr( @$indicator[$j], 0, 1 );
+                    my $ind2 = substr( @$indicator[$j], 1, 1 );
+                    $xml .=
+"<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
+                    $first = 0;
+                }
+                $xml .=
+"<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
+            }
+        }
+        $prevtag = @$tags[$i];
+    }
+    $xml .= MARC::File::XML::footer();
+
+    return $xml;
+}
+
+=head2 TransformHtmlToMarc
+
+=over 4
+
+$record = TransformHtmlToMarc( $dbh, $rtags, $rsubfields, $rvalues, %indicators )
+
+=back
+
+=cut
+
+sub TransformHtmlToMarc {
+    my ( $dbh, $rtags, $rsubfields, $rvalues, %indicators ) = @_;
+    my $prevtag = -1;
+    my $record  = MARC::Record->new();
+
+    #     my %subfieldlist=();
+    my $prevvalue;    # if tag <10
+    my $field;        # if tag >=10
+    for ( my $i = 0 ; $i < @$rtags ; $i++ ) {
+        next unless @$rvalues[$i];
+
+ # rebuild MARC::Record
+ #             warn "0=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ";
+        if ( @$rtags[$i] ne $prevtag ) {
+            if ( $prevtag < 10 ) {
+                if ($prevvalue) {
+
+                    if ( $prevtag ne '000' ) {
+                        $record->insert_fields_ordered(
+                            ( sprintf "%03s", $prevtag ), $prevvalue );
+                    }
+                    else {
+
+                        $record->leader($prevvalue);
+
+                    }
+                }
+            }
+            else {
+                if ($field) {
+                    $record->insert_fields_ordered($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]
+                );
+            }
+            $prevtag = @$rtags[$i];
+        }
+        else {
+            if ( @$rtags[$i] < 10 ) {
+                $prevvalue = @$rvalues[$i];
+            }
+            else {
+                if ( length( @$rvalues[$i] ) > 0 ) {
+                    $field->add_subfields( @$rsubfields[$i] => @$rvalues[$i] );
+                }
+            }
+            $prevtag = @$rtags[$i];
+        }
+    }
+
+    # the last has not been included inside the loop... do it now !
+    $record->insert_fields_ordered($field) if $field;
+
+    $record->encoding('UTF-8');
+
+    #    $record->MARC::File::USMARC::update_leader();
+    return $record;
+}
+
+=head2 TransformMarcToKoha
+
+=over 4
+
+$result = TransformMarcToKoha( $dbh, $record, $frameworkcode )
+
+=back
+
+=cut
+
+sub TransformMarcToKoha {
+    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;
+    while ( ($field) = $sth2->fetchrow ) {
+        $result =
+          &TransformMarcToKohaOneField( "biblio", $field, $record, $result,
+            $frameworkcode );
+    }
+    $sth2 = $dbh->prepare("SHOW COLUMNS from biblioitems");
+    $sth2->execute;
+    while ( ($field) = $sth2->fetchrow ) {
+        if ( $field eq 'notes' ) { $field = 'bnotes'; }
+        $result =
+          &TransformMarcToKohaOneField( "biblioitems", $field, $record, $result,
+            $frameworkcode );
+    }
+    $sth2 = $dbh->prepare("SHOW COLUMNS from items");
+    $sth2->execute;
+    while ( ($field) = $sth2->fetchrow ) {
+        $result =
+          &TransformMarcToKohaOneField( "items", $field, $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;
+}
+
+=head2 TransformMarcToKohaOneField
+
+=over 4
+
+$result = TransformMarcToKohaOneField( $kohatable, $kohafield, $record, $result, $frameworkcode )
+
+=back
+
+=cut
+
+sub TransformMarcToKohaOneField {
+
+# FIXME ? if a field has a repeatable subfield that is used in old-db, only the 1st will be retrieved...
+    my ( $kohatable, $kohafield, $record, $result, $frameworkcode ) = @_;
+
+    my $res = "";
+    my ( $tagfield, $subfield ) =
+      GetMarcFromKohaField( $kohatable . "." . $kohafield,
+        $frameworkcode );
+    foreach my $field ( $record->field($tagfield) ) {
+        if ( $field->tag() < 10 ) {
+            if ( $result->{$kohafield} ) {
+                $result->{$kohafield} .= " | " . $field->data();
+            }
+            else {
+                $result->{$kohafield} = $field->data();
+            }
+        }
+        else {
+            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];
+                        }
+                    }
+                }
+            }
+        }
+    }
+    return $result;
+}
+=head1  OTHER FUNCTIONS
+
+=head2 char_decode
+
+=over 4
+
+my $string = char_decode( $string, $encoding );
+
+converts ISO 5426 coded string to UTF-8
+sloppy code : should be improved in next issue
+
+=back
+
+=cut
+
+sub char_decode {
+    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\x41/Ä/gm;
+        s/\xc8\x45/Ë/gm;
+        s/\xc8\x49/Ï/gm;
+        s/\xc8\x61/ä/gm;
+        s/\xc8\x65/ë/gm;
+        s/\xc8\x69/ï/gm;
+        s/\xc8\x6F/ö/gm;
+        s/\xc8\x75/ü/gm;
+        s/\xc8\x76/ÿ/gm;
+        s/\xc9\x41/Ä/gm;
+        s/\xc9\x45/Ë/gm;
+        s/\xc9\x49/Ï/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" ) {
+        ##MARC-8 to UTF-8
+
+        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/\xe6\x41/Ă/gm;
+        s/\xe6\x45/Ĕ/gm;
+        s/\xe6\x65/ĕ/gm;
+        s/\xe6\x61/ă/gm;
+        s/\xe8\x45/Ë/gm;
+        s/\xe8\x49/Ï/gm;
+        s/\xe8\x65/ë/gm;
+        s/\xe8\x69/ï/gm;
+        s/\xe8\x76/ÿ/gm;
+        s/\xe9\x41/A/gm;
+        s/\xe9\x4f/O/gm;
+        s/\xe9\x55/U/gm;
+        s/\xe9\x61/a/gm;
+        s/\xe9\x6f/o/gm;
+        s/\xe9\x75/u/gm;
+        s/\xea\x41/A/gm;
+        s/\xea\x61/a/gm;
+
+        #Additional Turkish characters
+        s/\x1b//gm;
+        s/\x1e//gm;
+        s/(\xf0)s/\xc5\x9f/gm;
+        s/(\xf0)S/\xc5\x9e/gm;
+        s/(\xf0)c/ç/gm;
+        s/(\xf0)C/Ç/gm;
+        s/\xe7\x49/\\xc4\xb0/gm;
+        s/(\xe6)G/\xc4\x9e/gm;
+        s/(\xe6)g/ğ\xc4\x9f/gm;
+        s/\xB8/ı/gm;
+        s/\xB9/£/gm;
+        s/(\xe8|\xc8)o/ö/gm;
+        s/(\xe8|\xc8)O/Ö/gm;
+        s/(\xe8|\xc8)u/ü/gm;
+        s/(\xe8|\xc8)U/Ü/gm;
+        s/\xc2\xb8/\xc4\xb1/gm;
+        s/¸/\xc4\xb1/gm;
+
+        # this handles non-sorting blocks (if implementation requires this)
+        $string = nsb_clean($_);
+    }
+    return ($string);
+}
+
+=head2 nsb_clean
+
+=over 4
+
+my $string = nsb_clean( $string, $encoding );
+
+=back
+
+=cut
+
+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);
+}
+
+=head2 PrepareItemrecordDisplay
+
+=over 4
+
+PrepareItemrecordDisplay($itemrecord,$bibnum,$itemumber);
+
+Returns a hash with all the fields for Display a given item data in a template
+
+=back
+
+=cut
+
+sub PrepareItemrecordDisplay {
+
+    my ( $bibnum, $itemnum ) = @_;
+
+    my $dbh = C4::Context->dbh;
+    my $frameworkcode = &GetFrameworkCode( $bibnum );
+    my ( $itemtagfield, $itemtagsubfield ) =
+      &GetMarcFromKohaField( "items.itemnumber", $frameworkcode );
+    my $tagslib = &GetMarcStructure( $dbh, 1, $frameworkcode );
+    my $itemrecord = GetMarcItem( $bibnum, $itemnum) if ($itemnum);
+    my @loop_data;
+    my $authorised_values_sth =
+      $dbh->prepare(
+"select authorised_value,lib from authorised_values where category=? order by lib"
+      );
+    foreach my $tag ( sort keys %{$tagslib} ) {
+        my $previous_tag = '';
+        if ( $tag ne '' ) {
+            # loop through each subfield
+            my $cntsubf;
+            foreach my $subfield ( sort keys %{ $tagslib->{$tag} } ) {
+                next if ( subfield_is_koha_internal_p($subfield) );
+                next if ( $tagslib->{$tag}->{$subfield}->{'tab'} ne "10" );
+                my %subfield_data;
+                $subfield_data{tag}           = $tag;
+                $subfield_data{subfield}      = $subfield;
+                $subfield_data{countsubfield} = $cntsubf++;
+                $subfield_data{kohafield}     =
+                  $tagslib->{$tag}->{$subfield}->{'kohafield'};
+
+         #        $subfield_data{marc_lib}=$tagslib->{$tag}->{$subfield}->{lib};
+                $subfield_data{marc_lib} =
+                    "<span id=\"error\" title=\""
+                  . $tagslib->{$tag}->{$subfield}->{lib} . "\">"
+                  . substr( $tagslib->{$tag}->{$subfield}->{lib}, 0, 12 )
+                  . "</span>";
+                $subfield_data{mandatory} =
+                  $tagslib->{$tag}->{$subfield}->{mandatory};
+                $subfield_data{repeatable} =
+                  $tagslib->{$tag}->{$subfield}->{repeatable};
+                $subfield_data{hidden} = "display:none"
+                  if $tagslib->{$tag}->{$subfield}->{hidden};
+                my ( $x, $value );
+                ( $x, $value ) = _find_value( $tag, $subfield, $itemrecord )
+                  if ($itemrecord);
+                $value =~ s/"/&quot;/g;
+
+                # search for itemcallnumber if applicable
+                if ( $tagslib->{$tag}->{$subfield}->{kohafield} eq
+                    'items.itemcallnumber'
+                    && C4::Context->preference('itemcallnumber') )
+                {
+                    my $CNtag =
+                      substr( C4::Context->preference('itemcallnumber'), 0, 3 );
+                    my $CNsubfield =
+                      substr( C4::Context->preference('itemcallnumber'), 3, 1 );
+                    my $temp = $itemrecord->field($CNtag) if ($itemrecord);
+                    if ($temp) {
+                        $value = $temp->subfield($CNsubfield);
+                    }
+                }
+                if ( $tagslib->{$tag}->{$subfield}->{authorised_value} ) {
+                    my @authorised_values;
+                    my %authorised_lib;
+
+                    # builds list, depending on authorised value...
+                    #---- branch
+                    if ( $tagslib->{$tag}->{$subfield}->{'authorised_value'} eq
+                        "branches" )
+                    {
+                        if ( ( C4::Context->preference("IndependantBranches") )
+                            && ( C4::Context->userenv->{flags} != 1 ) )
+                        {
+                            my $sth =
+                              $dbh->prepare(
+"select branchcode,branchname from branches where branchcode = ? order by branchname"
+                              );
+                            $sth->execute( C4::Context->userenv->{branch} );
+                            push @authorised_values, ""
+                              unless (
+                                $tagslib->{$tag}->{$subfield}->{mandatory} );
+                            while ( my ( $branchcode, $branchname ) =
+                                $sth->fetchrow_array )
+                            {
+                                push @authorised_values, $branchcode;
+                                $authorised_lib{$branchcode} = $branchname;
+                            }
+                        }
+                        else {
+                            my $sth =
+                              $dbh->prepare(
+"select branchcode,branchname from branches order by branchname"
+                              );
+                            $sth->execute;
+                            push @authorised_values, ""
+                              unless (
+                                $tagslib->{$tag}->{$subfield}->{mandatory} );
+                            while ( my ( $branchcode, $branchname ) =
+                                $sth->fetchrow_array )
+                            {
+                                push @authorised_values, $branchcode;
+                                $authorised_lib{$branchcode} = $branchname;
+                            }
+                        }
+
+                        #----- itemtypes
+                    }
+                    elsif ( $tagslib->{$tag}->{$subfield}->{authorised_value} eq
+                        "itemtypes" )
+                    {
+                        my $sth =
+                          $dbh->prepare(
+"select itemtype,description from itemtypes order by description"
+                          );
+                        $sth->execute;
+                        push @authorised_values, ""
+                          unless ( $tagslib->{$tag}->{$subfield}->{mandatory} );
+                        while ( my ( $itemtype, $description ) =
+                            $sth->fetchrow_array )
+                        {
+                            push @authorised_values, $itemtype;
+                            $authorised_lib{$itemtype} = $description;
+                        }
+
+                        #---- "true" authorised value
+                    }
+                    else {
+                        $authorised_values_sth->execute(
+                            $tagslib->{$tag}->{$subfield}->{authorised_value} );
+                        push @authorised_values, ""
+                          unless ( $tagslib->{$tag}->{$subfield}->{mandatory} );
+                        while ( my ( $value, $lib ) =
+                            $authorised_values_sth->fetchrow_array )
+                        {
+                            push @authorised_values, $value;
+                            $authorised_lib{$value} = $lib;
+                        }
+                    }
+                    $subfield_data{marc_value} = CGI::scrolling_list(
+                        -name     => 'field_value',
+                        -values   => \@authorised_values,
+                        -default  => "$value",
+                        -labels   => \%authorised_lib,
+                        -size     => 1,
+                        -tabindex => '',
+                        -multiple => 0,
+                    );
+                }
+                elsif ( $tagslib->{$tag}->{$subfield}->{thesaurus_category} ) {
+                    $subfield_data{marc_value} =
+"<input type=\"text\" name=\"field_value\"  size=47 maxlength=255> <a href=\"javascript:Dopop('cataloguing/thesaurus_popup.pl?category=$tagslib->{$tag}->{$subfield}->{thesaurus_category}&index=',)\">...</a>";
+
+#"
+# COMMENTED OUT because No $i is provided with this API.
+# And thus, no value_builder can be activated.
+# BUT could be thought over.
+#         } elsif ($tagslib->{$tag}->{$subfield}->{'value_builder'}) {
+#             my $plugin="value_builder/".$tagslib->{$tag}->{$subfield}->{'value_builder'};
+#             require $plugin;
+#             my $extended_param = plugin_parameters($dbh,$itemrecord,$tagslib,$i,0);
+#             my ($function_name,$javascript) = plugin_javascript($dbh,$record,$tagslib,$i,0);
+#             $subfield_data{marc_value}="<input type=\"text\" value=\"$value\" name=\"field_value\"  size=47 maxlength=255 DISABLE READONLY OnFocus=\"javascript:Focus$function_name()\" OnBlur=\"javascript:Blur$function_name()\"> <a href=\"javascript:Clic$function_name()\">...</a> $javascript";
+                }
+                else {
+                    $subfield_data{marc_value} =
+"<input type=\"text\" name=\"field_value\" value=\"$value\" size=50 maxlength=255>";
+                }
+                push( @loop_data, \%subfield_data );
+            }
+        }
+    }
+    my $itemnumber = $itemrecord->subfield( $itemtagfield, $itemtagsubfield )
+      if ( $itemrecord && $itemrecord->field($itemtagfield) );
+    return {
+        'itemtagfield'    => $itemtagfield,
+        'itemtagsubfield' => $itemtagsubfield,
+        'itemnumber'      => $itemnumber,
+        'iteminformation' => \@loop_data
+    };
+}
+#"
+
+#
+# true ModZebra commented until indexdata fixes zebraDB crashes (it seems they occur on multiple updates
+# at the same time
+# replaced by a zebraqueue table, that is filled with ModZebra to run.
+# the table is emptied by misc/cronjobs/zebraqueue_start.pl script
+# =head2 ModZebrafiles
+# 
+# &ModZebrafiles( $dbh, $biblionumber, $record, $folder, $server );
+# 
+# =cut
+# 
+# sub ModZebrafiles {
+# 
+#     my ( $dbh, $biblionumber, $record, $folder, $server ) = @_;
+# 
+#     my $op;
+#     my $zebradir =
+#       C4::Context->zebraconfig($server)->{directory} . "/" . $folder . "/";
+#     unless ( opendir( DIR, "$zebradir" ) ) {
+#         warn "$zebradir not found";
+#         return;
+#     }
+#     closedir DIR;
+#     my $filename = $zebradir . $biblionumber;
+# 
+#     if ($record) {
+#         open( OUTPUT, ">", $filename . ".xml" );
+#         print OUTPUT $record;
+#         close OUTPUT;
+#     }
+# }
+
+=head2 ModZebra
+
+=over 4
+
+ModZebra( $dbh, $biblionumber, $op, $server );
+
+=back
+
+=cut
+
+sub ModZebra {
+###Accepts a $server variable thus we can use it for biblios authorities or other zebra dbs
+    my ( $biblionumber, $op, $server ) = @_;
+    my $dbh=C4::Context->dbh;
+    #warn "SERVER:".$server;
+#
+# true ModZebra commented until indexdata fixes zebraDB crashes (it seems they occur on multiple updates
+# at the same time
+# replaced by a zebraqueue table, that is filled with ModZebra to run.
+# the table is emptied by misc/cronjobs/zebraqueue_start.pl script
+
+my $sth=$dbh->prepare("insert into zebraqueue  (biblio_auth_number ,server,operation) values(?,?,?)");
+$sth->execute($biblionumber,$server,$op);
+$sth->finish;
+
+#
+#     my @Zconnbiblio;
+#     my $tried     = 0;
+#     my $recon     = 0;
+#     my $reconnect = 0;
+#     my $record;
+#     my $shadow;
+# 
+#   reconnect:
+#     $Zconnbiblio[0] = C4::Context->Zconn( $server, 0, 1 );
+# 
+#     if ( $server eq "biblioserver" ) {
+# 
+#         # it's unclear to me whether this should be in xml or MARC format
+#         # but it is clear it should be nabbed from zebra rather than from
+#         # the koha tables
+#         $record = GetMarcBiblio($biblionumber);
+#         $record = $record->as_xml_record() if $record;
+# #            warn "RECORD $biblionumber => ".$record;
+#         $shadow="biblioservershadow";
+# 
+#         #           warn "RECORD $biblionumber => ".$record;
+#         $shadow = "biblioservershadow";
+# 
+#     }
+#     elsif ( $server eq "authorityserver" ) {
+#         $record = C4::AuthoritiesMarc::XMLgetauthority( $dbh, $biblionumber );
+#         $shadow = "authorityservershadow";
+#     }    ## Add other servers as necessary
+# 
+#     my $Zpackage = $Zconnbiblio[0]->package();
+#     $Zpackage->option( action => $op );
+#     $Zpackage->option( record => $record );
+# 
+#   retry:
+#     $Zpackage->send("update");
+#     my $i;
+#     my $event;
+# 
+#     while ( ( $i = ZOOM::event( \@Zconnbiblio ) ) != 0 ) {
+#         $event = $Zconnbiblio[0]->last_event();
+#         last if $event == ZOOM::Event::ZEND;
+#     }
+# 
+#     my ( $error, $errmsg, $addinfo, $diagset ) = $Zconnbiblio[0]->error_x();
+#     if ( $error == 10000 && $reconnect == 0 )
+#     {    ## This is serious ZEBRA server is not available -reconnect
+#         warn "problem with zebra server connection";
+#         $reconnect = 1;
+#         my $res = system('sc start "Z39.50 Server" >c:/zebraserver/error.log');
+# 
+#         #warn "Trying to restart ZEBRA Server";
+#         #goto "reconnect";
+#     }
+#     elsif ( $error == 10007 && $tried < 2 )
+#     {    ## timeout --another 30 looonng seconds for this update
+#         $tried = $tried + 1;
+#         warn "warn: timeout, trying again";
+#         goto "retry";
+#     }
+#     elsif ( $error == 10004 && $recon == 0 ) {    ##Lost connection -reconnect
+#         $recon = 1;
+#         warn "error: reconnecting to zebra";
+#         goto "reconnect";
+# 
+#    # as a last resort, we save the data to the filesystem to be indexed in batch
+#     }
+#     elsif ($error) {
+#         warn
+# "Error-$server   $op $biblionumber /errcode:, $error, /MSG:,$errmsg,$addinfo \n";
+#         $Zpackage->destroy();
+#         $Zconnbiblio[0]->destroy();
+#         ModZebrafiles( $dbh, $biblionumber, $record, $op, $server );
+#         return;
+#     }
+#     if ( C4::Context->$shadow ) {
+#         $Zpackage->send('commit');
+#         while ( ( $i = ZOOM::event( \@Zconnbiblio ) ) != 0 ) {
+# 
+#             #waiting zebra to finish;
+#          }
+#     }
+#     $Zpackage->destroy();
+}
+
+=head1 INTERNAL FUNCTIONS
+
+=head2 MARCitemchange
+
+=over 4
+
+&MARCitemchange( $record, $itemfield, $newvalue )
+
+Function to update a single value in an item field.
+Used twice, could probably be replaced by something else, but works well...
+
+=back
+
+=back
+
+=cut
+
+sub MARCitemchange {
+    my ( $record, $itemfield, $newvalue ) = @_;
+    my $dbh = C4::Context->dbh;
+    
+    my ( $tagfield, $tagsubfield ) =
+      GetMarcFromKohaField( $itemfield, "" );
+    if ( ($tagfield) && ($tagsubfield) ) {
+        my $tag = $record->field($tagfield);
+        if ($tag) {
+            $tag->update( $tagsubfield => $newvalue );
+            $record->delete_field($tag);
+            $record->insert_fields_ordered($tag);
+        }
+    }
+}
+
+=head2 _koha_add_biblio
+
+=over 4
+
+_koha_add_biblio($dbh,$biblioitem);
+
+Internal function to add a biblio ($biblio is a hash with the values)
+
+=back
+
+=cut
+
+sub _koha_add_biblio {
+    my ( $dbh, $biblio, $frameworkcode ) = @_;
+    my $sth = $dbh->prepare("Select max(biblionumber) from biblio");
+    $sth->execute;
+    my $data         = $sth->fetchrow_arrayref;
+    my $biblionumber = $$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 = ?, frameworkcode = ? "
+    );
+    $sth->execute(
+        $biblionumber,         $biblio->{'title'},
+        $biblio->{'author'},   $biblio->{'copyrightdate'},
+        $biblio->{'serial'},   $biblio->{'seriestitle'},
+        $biblio->{'notes'},    $biblio->{'abstract'},
+        $biblio->{'unititle'}, $frameworkcode
+    );
+
+    $sth->finish;
+    return ($biblionumber);
+}
+
+=head2 _find_value
+
+=over 4
+
+($indicators, $value) = _find_value($tag, $subfield, $record,$encoding);
+
+Find the given $subfield in the given $tag in the given
+MARC::Record $record.  If the subfield is found, returns
+the (indicators, value) pair; otherwise, (undef, undef) is
+returned.
+
+PROPOSITION :
+Such a function is used in addbiblio AND additem and serial-edit and maybe could be used in Authorities.
+I suggest we export it from this module.
+
+=back
+
+=cut
+
+sub _find_value {
+    my ( $tagfield, $insubfield, $record, $encoding ) = @_;
+    my @result;
+    my $indicator;
+    if ( $tagfield < 10 ) {
+        if ( $record->field($tagfield) ) {
+            push @result, $record->field($tagfield)->data();
+        }
+        else {
+            push @result, "";
+        }
+    }
+    else {
+        foreach my $field ( $record->field($tagfield) ) {
+            my @subfields = $field->subfields();
+            foreach my $subfield (@subfields) {
+                if ( @$subfield[0] eq $insubfield ) {
+                    push @result, @$subfield[1];
+                    $indicator = $field->indicator(1) . $field->indicator(2);
+                }
+            }
+        }
+    }
+    return ( $indicator, @result );
+}
+
+=head2 _koha_modify_biblio
+
+=over 4
+
+Internal function for updating the biblio table
+
+=back
+
+=cut
+
+sub _koha_modify_biblio {
+    my ( $dbh, $biblio ) = @_;
+
+# FIXME: this code could be made more portable by not hard-coding the values that are supposed to be in biblio table
+    my $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'} );
+}
+
+=head2 _koha_modify_biblioitem
+
+=over 4
+
+_koha_modify_biblioitem( $dbh, $biblioitem );
+
+=back
+
+=cut
+
+sub _koha_modify_biblioitem {
+    my ( $dbh, $biblioitem ) = @_;
+    my $query;
+##Recalculate LC in case it changed --TG
+
+    $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'} );
+    $biblioitem->{'ccode'}        = $dbh->quote( $biblioitem->{'ccode'} );
+    $biblioitem->{'biblionumber'} =
+      $dbh->quote( $biblioitem->{'biblionumber'} );
+
+    $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'},
+        dewey           = $biblioitem->{'dewey'},
+        subclass        = $biblioitem->{'subclass'},
+        illus           = $biblioitem->{'illus'},
+        pages           = $biblioitem->{'pages'},
+        volumeddesc     = $biblioitem->{'volumeddesc'},
+        notes           = $biblioitem->{'bnotes'},
+        size            = $biblioitem->{'size'},
+        place           = $biblioitem->{'place'},
+        ccode           = $biblioitem->{'ccode'}
+        where biblionumber = $biblioitem->{'biblionumber'}";
+
+    $dbh->do($query);
+    if ( $dbh->errstr ) {
+        warn "$query";
+    }
+}
+
+=head2 _koha_add_biblioitem
+
+=over 4
+
+_koha_add_biblioitem( $dbh, $biblioitem );
+
+Internal function to add a biblioitem
+
+=back
+
+=cut
+
+sub _koha_add_biblioitem {
+    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->finish;
+
+    $sth = $dbh->prepare(
+        "INSERT INTO biblioitems SET
+            biblioitemnumber = ?, biblionumber    = ?,
+            volume           = ?, number          = ?,
+            classification   = ?, itemtype        = ?,
+            url              = ?, isbn            = ?,
+            issn             = ?, dewey           = ?,
+            subclass         = ?, publicationyear = ?,
+            publishercode    = ?, volumedate      = ?,
+            volumeddesc      = ?, illus           = ?,
+            pages            = ?, notes           = ?,
+            size             = ?, lccn            = ?,
+            marc             = ?, lcsort          =?,
+            place            = ?, ccode           = ?
+          "
+    );
+    my ($lcsort) =
+      calculatelc( $biblioitem->{'classification'} )
+      . $biblioitem->{'subclass'};
+    $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'},
+        $lcsort,                         $biblioitem->{'ccode'}
+    );
+    $sth->finish;
+    return ($bibitemnum);
+}
+
+=head2 _koha_new_items
+
+=over 4
+
+_koha_new_items( $dbh, $item, $barcode );
+
+=back
+
+=cut
+
+sub _koha_new_items {
+    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;
+## Now calculate lccalnumber
+    my ($cutterextra) = itemcalculator(
+        $dbh,
+        $item->{'biblioitemnumber'},
+        $item->{'itemcallnumber'}
+    );
+
+# FIXME the "notforloan" field seems to be named "loan" in some places. workaround bugfix.
+    if ( $item->{'loan'} ) {
+        $item->{'notforloan'} = $item->{'loan'};
+    }
+
+    # if dateaccessioned is provided, use it. Otherwise, set to NOW()
+    if ( $item->{'dateaccessioned'} eq '' || !$item->{'dateaccessioned'} ) {
+
+        $sth = $dbh->prepare(
+            "Insert into items set
+            itemnumber           = ?,     biblionumber     = ?,
+            multivolumepart      = ?,
+            biblioitemnumber     = ?,     barcode          = ?,
+            booksellerid         = ?,     dateaccessioned  = NOW(),
+            homebranch           = ?,     holdingbranch    = ?,
+            price                = ?,     replacementprice = ?,
+            replacementpricedate = NOW(), datelastseen     = NOW(),
+            multivolume          = ?,     stack            = ?,
+            itemlost             = ?,     wthdrawn         = ?,
+            paidfor              = ?,     itemnotes        = ?,
+            itemcallnumber       =?,      notforloan       = ?,
+            location             = ?,     Cutterextra      = ?
+          "
+        );
+        $sth->execute(
+            $itemnumber,                $item->{'biblionumber'},
+            $item->{'multivolumepart'}, $item->{'biblioitemnumber'},
+            $barcode,                   $item->{'booksellerid'},
+            $item->{'homebranch'},      $item->{'holdingbranch'},
+            $item->{'price'},           $item->{'replacementprice'},
+            $item->{multivolume},       $item->{stack},
+            $item->{itemlost},          $item->{wthdrawn},
+            $item->{paidfor},           $item->{'itemnotes'},
+            $item->{'itemcallnumber'},  $item->{'notforloan'},
+            $item->{'location'},        $cutterextra
+        );
+    }
+    else {
+        $sth = $dbh->prepare(
+            "INSERT INTO items SET
+            itemnumber           = ?,     biblionumber     = ?,
+            multivolumepart      = ?,
+            biblioitemnumber     = ?,     barcode          = ?,
+            booksellerid         = ?,     dateaccessioned  = ?,
+            homebranch           = ?,     holdingbranch    = ?,
+            price                = ?,     replacementprice = ?,
+            replacementpricedate = NOW(), datelastseen     = NOW(),
+            multivolume          = ?,     stack            = ?,
+            itemlost             = ?,     wthdrawn         = ?,
+            paidfor              = ?,     itemnotes        = ?,
+            itemcallnumber       = ?,     notforloan       = ?,
+            location             = ?,
+            Cutterextra          = ?
+                            "
+        );
+        $sth->execute(
+            $itemnumber,                 $item->{'biblionumber'},
+            $item->{'multivolumepart'},  $item->{'biblioitemnumber'},
+            $barcode,                    $item->{'booksellerid'},
+            $item->{'dateaccessioned'},  $item->{'homebranch'},
+            $item->{'holdingbranch'},    $item->{'price'},
+            $item->{'replacementprice'}, $item->{multivolume},
+            $item->{stack},              $item->{itemlost},
+            $item->{wthdrawn},           $item->{paidfor},
+            $item->{'itemnotes'},        $item->{'itemcallnumber'},
+            $item->{'notforloan'},       $item->{'location'},
+            $cutterextra
+        );
+    }
+    if ( defined $sth->errstr ) {
+        $error .= $sth->errstr;
+    }
+    return ( $itemnumber, $error );
 }
 
+=head2 _koha_modify_item
 
+=over 4
 
-sub MARCgetbiblio {
-    # Returns MARC::Record of the biblio passed in parameter.
-    ### Takes a new parameter of $title_author =1 which parses the record obly on those fields and nothing else
-    ### Its useful when Koha requires only title&author for performance issues
-    my ( $dbh, $biblionumber, $title_author ) = @_;
-    my $sth =
-      $dbh->prepare("select marc from biblio where biblionumber=? "  );
-    $sth->execute( $biblionumber);
-   my ($marc)=$sth->fetchrow;
-my $record;
-       if ($title_author){
-       $record = MARC::File::USMARC::decode($marc,\&func_title_author);
-       }else{
-        $record = MARC::File::USMARC::decode($marc);
-       }
-$sth->finish;
- return $record;
+_koha_modify_item( $dbh, $item, $op );
+
+=back
+
+=cut
+
+sub _koha_modify_item {
+    my ( $dbh, $item, $op ) = @_;
+    $item->{'itemnum'} = $item->{'itemnumber'} unless $item->{'itemnum'};
+
+    # if all we're doing is setting statuses, just update those and get out
+    if ( $op eq "setstatus" ) {
+        my $query =
+          "UPDATE items SET itemlost=?,wthdrawn=?,binding=? WHERE itemnumber=?";
+        my @bind = (
+            $item->{'itemlost'}, $item->{'wthdrawn'},
+            $item->{'binding'},  $item->{'itemnumber'}
+        );
+        my $sth = $dbh->prepare($query);
+        $sth->execute(@bind);
+        $sth->finish;
+        return undef;
+    }
+## Now calculate lccalnumber
+    my ($cutterextra) =
+      itemcalculator( $dbh, $item->{'bibitemnum'}, $item->{'itemcallnumber'} );
+
+    my $query = "UPDATE items SET
+barcode=?,itemnotes=?,itemcallnumber=?,notforloan=?,location=?,multivolumepart=?,multivolume=?,stack=?,wthdrawn=?,holdingbranch=?,homebranch=?,cutterextra=?, onloan=?, binding=?";
+
+    my @bind = (
+        $item->{'barcode'},        $item->{'notes'},
+        $item->{'itemcallnumber'}, $item->{'notforloan'},
+        $item->{'location'},       $item->{multivolumepart},
+        $item->{multivolume},      $item->{stack},
+        $item->{wthdrawn},         $item->{holdingbranch},
+        $item->{homebranch},       $cutterextra,
+        $item->{onloan},           $item->{binding}
+    );
+    if ( $item->{'lost'} ne '' ) {
+        $query =
+"update items set biblioitemnumber=?,barcode=?,itemnotes=?,homebranch=?,
+                            itemlost=?,wthdrawn=?,itemcallnumber=?,notforloan=?,
+                             location=?,multivolumepart=?,multivolume=?,stack=?,wthdrawn=?,holdingbranch=?,cutterextra=?,onloan=?, binding=?";
+        @bind = (
+            $item->{'bibitemnum'},     $item->{'barcode'},
+            $item->{'notes'},          $item->{'homebranch'},
+            $item->{'lost'},           $item->{'wthdrawn'},
+            $item->{'itemcallnumber'}, $item->{'notforloan'},
+            $item->{'location'},       $item->{multivolumepart},
+            $item->{multivolume},      $item->{stack},
+            $item->{wthdrawn},         $item->{holdingbranch},
+            $cutterextra,              $item->{onloan},
+            $item->{binding}
+        );
+        if ( $item->{homebranch} ) {
+            $query .= ",homebranch=?";
+            push @bind, $item->{homebranch};
+        }
+        if ( $item->{holdingbranch} ) {
+            $query .= ",holdingbranch=?";
+            push @bind, $item->{holdingbranch};
+        }
+    }
+    $query .= " where itemnumber=?";
+    push @bind, $item->{'itemnum'};
+    if ( $item->{'replacement'} ne '' ) {
+        $query =~ s/ where/,replacementprice='$item->{'replacement'}' where/;
+    }
+    my $sth = $dbh->prepare($query);
+    $sth->execute(@bind);
+    $sth->finish;
 }
 
+=head2 _koha_delete_biblio
 
+=over 4
 
+$error = _koha_delete_biblio($dbh,$biblionumber);
 
+Internal sub for deleting from biblio table -- also saves to deletedbiblio
 
-sub MARCgetitem {
-# warn "MARCgetitem";
-    # Returns MARC::Record   of the item passed in parameter uses either itemnumber or barcode
-    my ( $dbh, $itemnumber,$barcode ) = @_;
-my $sth;
-if ($itemnumber){
-   $sth = $dbh->prepare("select i.marc from items i where i.itemnumber=?"  ); 
-    $sth->execute($itemnumber);
-}else{
- $sth = $dbh->prepare("select i.marc from  items i where i.barcode=?"  ); 
-    $sth->execute($barcode);
-}
- my ($marc)=$sth->fetchrow;
- my $record = MARC::File::USMARC::decode($marc);
-       
-    return ($record);
-}
+C<$dbh> - the database handle
+C<$biblionumber> - the biblionumber of the biblio to be deleted
+
+=back
+
+=cut
 
-sub MARCgetallitems {
-# warn "MARCgetallitems";
-    # Returns an array of MARC::Record   of the items passed in parameter as biblionumber
+# FIXME: add error handling
+
+sub _koha_delete_biblio {
     my ( $dbh, $biblionumber ) = @_;
-my @results;
-my   $sth = $dbh->prepare("select marc from items where biblionumber =?"  ); 
+
+    # get all the data for this biblio
+    my $sth = $dbh->prepare("SELECT * FROM biblio WHERE biblionumber=?");
     $sth->execute($biblionumber);
 
- while(my ($marc)=$sth->fetchrow_array){
- my $record = MARC::File::USMARC::decode($marc);
-    push @results,$record;
-}
-return @results;
-}
+    if ( my $data = $sth->fetchrow_hashref ) {
 
-sub MARCmoditemonefield{
-# This routine will be depraeciated as soon as mysql dependency on items is removed;
-## this function is different to MARCkoha2marcOnefield this one does not need the record but the itemnumber
-my ($dbh,$biblionumber,$itemnumber,$itemfield,$newvalue,$donotupdate)=@_;
-my ($record) = MARCgetitem($dbh,$itemnumber);
-   MARCkoha2marcOnefield( $record, $itemfield, $newvalue,"holdings" );
- if($donotupdate){
-       ## Prevent various update calls to zebra wait until all changes finish
-       ## Fix  to pass this record around to prevent Mysql update as well
-               my $sth=$dbh->prepare("update items set marc=? where itemnumber=?");
-               $sth->execute($record->as_usmarc,$itemnumber);
-               $sth->finish;
-       }else{
-               NEWmoditem($dbh,$record,$biblionumber,$itemnumber);
-}
+        # save the record in deletedbiblio
+        # find the fields to save
+        my $query = "INSERT INTO deletedbiblio SET ";
+        my @bind  = ();
+        foreach my $temp ( keys %$data ) {
+            $query .= "$temp = ?,";
+            push( @bind, $data->{$temp} );
+        }
 
+        # replace the last , by ",?)"
+        $query =~ s/\,$//;
+        my $bkup_sth = $dbh->prepare($query);
+        $bkup_sth->execute(@bind);
+        $bkup_sth->finish;
+
+        # delete the biblio
+        my $del_sth = $dbh->prepare("DELETE FROM biblio WHERE biblionumber=?");
+        $del_sth->execute($biblionumber);
+        $del_sth->finish;
+    }
+    $sth->finish;
+    return undef;
 }
 
+=head2 _koha_delete_biblioitems
 
+=over 4
 
+$error = _koha_delete_biblioitems($dbh,$biblioitemnumber);
 
-sub MARCfind_frameworkcode {
-# warn "MARCfind_frameworkcode";
-    my ( $dbh, $biblionumber ) = @_;
-    my $sth =
-      $dbh->prepare("select frameworkcode from biblio where biblionumber=?");
-    $sth->execute($biblionumber);
-    my ($frameworkcode) = $sth->fetchrow;
-    return $frameworkcode;
-}
-sub MARCfind_itemtype {
-# warn "MARCfind_itemtype";
-    my ( $dbh, $biblionumber ) = @_;
-    my $sth =
-      $dbh->prepare("select itemtype from biblio where biblionumber=?");
-    $sth->execute($biblionumber);
-    my ($itemtype) = $sth->fetchrow;
-    return $itemtype;
-}
-
+Internal sub for deleting from biblioitems table -- also saves to deletedbiblioitems
 
+C<$dbh> - the database handle
+C<$biblionumber> - the biblioitemnumber of the biblioitem to be deleted
 
-sub MARChtml2xml {
-# warn "MARChtml2xml ";
-       my ($tags,$subfields,$values,$indicator,$ind_tag) = @_;        
-#      use MARC::File::XML;
-       my $xml= marc_record_header('UTF-8'); #### we do not need a collection wrapper
+=back
 
-    my $prevvalue;
-    my $prevtag=-1;
-    my $first=1;
-       my $j = -1;
-    for (my $i=0;$i<=@$tags;$i++){
-               @$values[$i] =~ s/&/&amp;/g;
-               @$values[$i] =~ s/</&lt;/g;
-               @$values[$i] =~ s/>/&gt;/g;
-               @$values[$i] =~ s/"/&quot;/g;
-               @$values[$i] =~ s/'/&apos;/g;
-
-               if ((@$tags[$i] ne $prevtag)){
-                       $j++ unless (@$tags[$i] eq "");
-                       ## warn "IND:".substr(@$indicator[$j],0,1).substr(@$indicator[$j],1,1)." ".@$tags[$i];
-                       if (!$first){
-                       $xml.="</datafield>\n";
-                               if ((@$tags[$i] > 10) && (@$values[$i] ne "")){
-                                               my $ind1 = substr(@$indicator[$j],0,1);
-                        my $ind2 = substr(@$indicator[$j],1,1);
-                        $xml.="<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
-                        $xml.="<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
-                        $first=0;
-                               } else {
-                       $first=1;
-                               }
-            } else {
-                       if (@$values[$i] ne "") {
-                               # leader
-                               if (@$tags[$i] eq "000") {
-                                               $xml.="<leader>@$values[$i]</leader>\n";
-                                               $first=1;
-                                       # rest of the fixed fields
-                               } elsif (@$tags[$i] < 10) {
-                                               $xml.="<controlfield tag=\"@$tags[$i]\">@$values[$i]</controlfield>\n";
-                                               $first=1;
-                               } else {
-                                               my $ind1 = substr(@$indicator[$j],0,1);
-                                               my $ind2 = substr(@$indicator[$j],1,1);
-                                               $xml.="<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
-                                               $xml.="<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
-                                               $first=0;                       
-                               }
-                       }
-                       }
-               } else { # @$tags[$i] eq $prevtag
-                if (@$values[$i] eq "") {
-                }
-                else {
-                                       if ($first){
-                                               my $ind1 = substr(@$indicator[$j],0,1);                        
-                                               my $ind2 = substr(@$indicator[$j],1,1);
-                                               $xml.="<datafield tag=\"@$tags[$i]\" ind1=\"$ind1\" ind2=\"$ind2\">\n";
-                                               $first=0;
-                                       }
-                       $xml.="<subfield code=\"@$subfields[$i]\">@$values[$i]</subfield>\n";
-                               }
-               }
-               $prevtag = @$tags[$i];
-       }
-       $xml.="</record>";
-       # warn $xml;
-       return $xml;
-}
-sub marc_record_header {
-####  this one is for <record>
-    my $format = shift;
-    my $enc = shift || 'UTF-8';
-    return( <<MARC_XML_HEADER );
-<?xml version="1.0" encoding="$enc"?>
-<record
-  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-  xsi:schemaLocation="http://www.loc.gov/MARC21/slim http://www.loc.gov/standards/marcxml/schema/MARC21slim.xsd"
-  xmlns="http://www.loc.gov/MARC21/slim">
-MARC_XML_HEADER
-}
+=cut
 
+# FIXME: add error handling
 
-sub collection_header {
-####  this one is for koha collection 
-    my $format = shift;
-    my $enc = shift || 'UTF-8';
-    return( <<KOHA_XML_HEADER );
-<?xml version="1.0" encoding="$enc"?>
-<kohacollection xmlns:marc="http://loc.gov/MARC21/slim" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://library.neu.edu.tr/kohanamespace/koharecord.xsd">
-KOHA_XML_HEADER
-}
+sub _koha_delete_biblioitems {
+    my ( $dbh, $biblioitemnumber ) = @_;
 
-sub MARChtml2marc {
-# warn "MARChtml2marc";
-       my ($dbh,$rtags,$rsubfields,$rvalues,%indicators) = @_;
-       my $prevtag = -1;
-       my $record = MARC::Record->new();
-#      my %subfieldlist=();
-       my $prevvalue; # if tag <10
-       my $field; # if tag >=10
-       for (my $i=0; $i< @$rtags; $i++) {
-               next unless @$rvalues[$i];
-               # rebuild MARC::Record
-#                      # warn "0=>".@$rtags[$i].@$rsubfields[$i]." = ".@$rvalues[$i].": ";
-               if (@$rtags[$i] ne $prevtag) {
-                       if ($prevtag < 10) {
-                               if ($prevvalue) {
-
-                                       if ($prevtag ne '000') {
-                                               $record->insert_fields_ordered((sprintf "%03s",$prevtag),$prevvalue);
-                                       } else {
-
-                                               $record->leader($prevvalue);
-
-                                       }
-                               }
-                       } else {
-                               if ($field) {
-                                       $record->insert_fields_ordered($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;
-                       }
-                       $prevtag = @$rtags[$i];
-               } else {
-                       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];
-               }
-       }
-       # the last has not been included inside the loop... do it now !
-       $record->insert_fields_ordered($field) if $field;
-#      # warn "HTML2MARC=".$record->as_formatted;
-       $record->encoding( 'UTF-8' );
-#      $record->MARC::File::USMARC::update_leader();
-       return $record;
-}
+    # get all the data for this biblioitem
+    my $sth =
+      $dbh->prepare("SELECT * FROM biblioitems WHERE biblioitemnumber=?");
+    $sth->execute($biblioitemnumber);
 
-sub MARCkoha2marc {
-# warn "MARCkoha2marc";
-## This routine most probably will be depreaceated -- it is still used for acqui management
-##Returns a  MARC record from a hash
-       my ($dbh,$result,$recordtype) = @_;
-
-       my $record = MARC::Record->new();
-       my $sth2=$dbh->prepare("SELECT  marctokoha from koha_attr where tagfield is not null and recordtype=?");
-       $sth2->execute($recordtype);
-       my $field;
-       while (($field)=$sth2->fetchrow) {
-               $record=&MARCkoha2marcOnefield($record,$field,$result->{$field},$recordtype) if $result->{$field};
-       }
-return $record;
-}
-sub MARCmarc2koha {
-# warn "MARCmarc2koha";
-##Returns a hash from MARC record
-       my ($dbh,$record,$related_record) = @_;
-       my $result;
-if (!$related_record){$related_record="biblios";}
-       my $sth2=$dbh->prepare("SELECT  marctokoha from koha_attr where  recordtype like ? and tagfield is not null" );
-       $sth2->execute($related_record);
-       my $field;
-       while ($field=$sth2->fetchrow) {
-               $result=&MARCmarc2kohaOneField($field,$record,$result,$related_record);
-       }
-
-## we only need the following for biblio data
-if ($related_record eq "biblios"){     
-# 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;
-}
+    if ( my $data = $sth->fetchrow_hashref ) {
 
-sub MARCkoha2marcOnefield {
-##Updates or creates one field in MARC record
-    my ( $record, $kohafieldname, $value,$recordtype ) = @_;
-my ( $tagfield, $tagsubfield ) = MARCfind_marc_from_kohafield($kohafieldname,$recordtype);
-if ($tagfield){
-my $tag = $record->field($tagfield);
-    if  (  $tagfield>9) { 
-        if ($tag) {
-               if ($value){## We may be trying to delete a subfield value
-                        $tag->update( $tagsubfield=> $value );
-               }else{  
-               $tag->delete_subfield(code=>$tagsubfield);
-               }
-                $record->delete_field($tag);
-                $record->insert_fields_ordered($tag);         
-        }else {
-       my $newtag=MARC::Field->new( $tagfield, " ", " ", $tagsubfield => $value);
-            $record->insert_fields_ordered($newtag);   
-        }
-    }else {
-        if ($tag) {
-         if ($value){  
-                $tag->update( $value );
-                $record->delete_field($tag);
-                $record->insert_fields_ordered($tag);    
-         }else{
-         $record->delete_field($tag);  
-         }
-        }else {
-       my $newtag=MARC::Field->new( $tagfield => $value);
-            $record->insert_fields_ordered($newtag);   
+        # save the record in deletedbiblioitems
+        # find the fields to save
+        my $query = "INSERT INTO deletedbiblioitems SET ";
+        my @bind  = ();
+        foreach my $temp ( keys %$data ) {
+            $query .= "$temp = ?,";
+            push( @bind, $data->{$temp} );
         }
+
+        # replace the last , by ",?)"
+        $query =~ s/\,$//;
+        my $bkup_sth = $dbh->prepare($query);
+        $bkup_sth->execute(@bind);
+        $bkup_sth->finish;
+
+        # delete the biblioitem
+        my $del_sth =
+          $dbh->prepare("DELETE FROM biblioitems WHERE biblioitemnumber=?");
+        $del_sth->execute($biblioitemnumber);
+        $del_sth->finish;
     }
-}## $tagfield defined
-    return $record;
+    $sth->finish;
+    return undef;
 }
 
-sub MARCmarc2kohaOneField {
-    my (  $kohafield, $record, $result,$recordtype ) = @_;
-    #    # warn "kohatable / $kohafield / $result / ";
-    my $res = "";
+=head2 _koha_delete_item
 
-  my  ( $tagfield, $subfield ) = MARCfind_marc_from_kohafield($kohafield,$recordtype);
-if ($tagfield){
-    foreach my $field ( $record->field($tagfield) ) {
-               if ($field->tag()<10) {
-                       if ($result->{$kohafield}) {
-                               $result->{$kohafield} .= " | ".$field->data();
-                       } else {
-                               $result->{$kohafield} = $field->data();
-                       }
-               } else {
-                       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];
-                                               }
-                                       }
-                               }
-                       }
-               }
+=over 4
+
+_koha_delete_item( $dbh, $itemnum );
+
+Internal function to delete an item record from the koha tables
+
+=back
+
+=cut
+
+sub _koha_delete_item {
+    my ( $dbh, $itemnum ) = @_;
+
+    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} );
     }
-}
-    return $result;
+    $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;
 }
 
-sub MARCmodLCindex{
-# warn "MARCmodLCindex";
-my ($dbh,$record)=@_;
+=head1 UNEXPORTED FUNCTIONS
 
-my ($tagfield,$tagsubfield) = MARCfind_marc_from_kohafield("classification","biblios");
-my ($tagfield2,$tagsubfieldsub) = MARCfind_marc_from_kohafield("subclass","biblios");
-my $tag=$record->field($tagfield);
-if ($tag){
-my ($lcsort)=calculatelc($tag->subfield($tagsubfield)).$tag->subfield($tagsubfieldsub);
+=over 4
 
- &MARCkoha2marcOnefield( $record, "lcsort", $lcsort,"biblios");
-}
-return $record;
-}
+=head2 calculatelc
 
-##########################NEW NEW NEW#############################
-sub NEWnewbiblio {
-    my ( $dbh, $record, $frameworkcode) = @_;
-    my $biblionumber;
-$frameworkcode="" unless $frameworkcode;
-    my $olddata = MARCmarc2koha( $dbh, $record,"biblios" );
-## In case reimporting records with biblionumbers keep them
-if ($olddata->{'biblionumber'}){
-$biblionumber=NEWmodbiblio( $dbh, $olddata->{'biblionumber'},$record,$frameworkcode );
-}else{
-    $biblionumber = NEWaddbiblio( $dbh, $record,$frameworkcode );
-}
+$lc = calculatelc($classification);
 
-   return ( $biblionumber );
-}
+=back
+
+=cut
 
+sub calculatelc {
+    my ($classification) = @_;
+    $classification =~ s/^\s+|\s+$//g;
+    my $i = 0;
+    my $lc2;
+    my $lc1;
 
+    for ( $i = 0 ; $i < length($classification) ; $i++ ) {
+        my $c = ( substr( $classification, $i, 1 ) );
+        if ( $c ge '0' && $c le '9' ) {
 
+            $lc2 = substr( $classification, $i );
+            last;
+        }
+        else {
+            $lc1 .= substr( $classification, $i, 1 );
+
+        }
+    }    #while
 
+    my $other = length($lc1);
+    if ( !$lc1 ) {
+        $other = 0;
+    }
 
-sub NEWmodbiblioframework {
-       my ($dbh,$biblionumber,$frameworkcode) =@_;
-       my $sth = $dbh->prepare("Update biblio SET frameworkcode=? WHERE biblionumber=$biblionumber");
-       $sth->execute($frameworkcode);
-       return 1;
+    my $extras;
+    if ( $other < 4 ) {
+        for ( 1 .. ( 4 - $other ) ) {
+            $extras .= "0";
+        }
+    }
+    $lc1 .= $extras;
+    $lc2 =~ s/^ //g;
+
+    $lc2 =~ s/ //g;
+    $extras = "";
+    ##Find the decimal part of $lc2
+    my $pos = index( $lc2, "." );
+    if ( $pos < 0 ) { $pos = length($lc2); }
+    if ( $pos >= 0 && $pos < 5 ) {
+        ##Pad lc2 with zeros to create a 5digit decimal needed in marc record to sort as numeric
+
+        for ( 1 .. ( 5 - $pos ) ) {
+            $extras .= "0";
+        }
+    }
+    $lc2 = $extras . $lc2;
+    return ( $lc1 . $lc2 );
 }
 
+=head2 itemcalculator
 
-sub NEWdelbiblio {
+=over 4
 
-    my ( $dbh, $biblionumber ) = @_;
-my $sth=$dbh->prepare("SELECT itemnumber FROM items where biblionumber=?");
+$cutterextra = itemcalculator( $dbh, $biblioitem, $callnumber );
 
-$sth->execute($biblionumber);
-       while (my $itemnumber =$sth->fetchrow){
-       OLDdelitem($dbh,$itemnumber) ;
-       }
+=back
 
-       ZEBRAop($dbh,$biblionumber,"recordDelete","biblioserver");
-OLDdelbiblio($dbh,$biblionumber) ;
+=cut
 
-}
+sub itemcalculator {
+    my ( $dbh, $biblioitem, $callnumber ) = @_;
+    my $sth =
+      $dbh->prepare(
+"select classification, subclass from biblioitems where biblioitemnumber=?"
+      );
 
-sub NEWnewitem {
-    my ( $dbh, $record, $biblionumber ) = @_;
-       my $itemtype= MARCfind_itemtype($dbh,$biblionumber);
-    my $item = &MARCmarc2koha( $dbh, $record,"holdings" );
-## In case we are re-importing marc records from bulk import do not change itemnumbers
-if ($item->{itemnumber}){
-NEWmoditem ( $dbh, $record, $biblionumber, $item->{itemnumber});
-}else{
-    $item->{'biblionumber'} =$biblionumber;
-##Add biblionumber to $record
-    MARCkoha2marcOnefield($record,"biblionumber",$biblionumber,"holdings");
- my $sth=$dbh->prepare("select notforloan from itemtypes where itemtype='$itemtype'");
-$sth->execute();
-my $notforloan=$sth->fetchrow;
-##Change the notforloan field if $notforloan found
-       if ($notforloan >0){
-       $item->{'notforloan'}=$notforloan;
-       &MARCkoha2marcOnefield($record,"notforloan",$notforloan,"holdings");
-       }
-if(!$item->{'dateaccessioned'}||$item->{'dateaccessioned'} eq ''){
-# find today's date
-my ($sec,$min,$hour,$mday,$mon,$year,$wday,$yday,$isdst) =                                                           
-localtime(time); $year +=1900; $mon +=1;
-my $date = "$year-".sprintf ("%0.2d", $mon)."-".sprintf("%0.2d",$mday);
-$item->{'dateaccessioned'}=$date;
-&MARCkoha2marcOnefield($record,"dateaccessioned",$date,"holdings");
-}
-  
-## Now calculate itempart of cutter
-my ($cutterextra)=itemcalculator($dbh,$item->{'biblionumber'},$item->{'itemcallnumber'});
-&MARCkoha2marcOnefield($record,"cutterextra",$cutterextra,"holdings");
-
-##NEU specific add cataloguers cardnumber as well
-my ($tag,$cardtag)=MARCfind_marc_from_kohafield("circid","holdings");
-       if ($tag && $cardtag){  
-       my $me= C4::Context->userenv;
-       my $cataloguer=$me->{'cardnumber'} if ($me);
-       my $newtag= $record->field($tag);
-       $newtag->update($cardtag=>$cataloguer) if ($me);
-       $record->delete_field($newtag);
-       $record->insert_fields_ordered($newtag);        
-       }
-##Add item to SQL
-my  $itemnumber = &OLDnewitems( $dbh, $item->{barcode},$record );
-
-# add the item to zebra it will add the biblio as well!!!
-    ZEBRAop( $dbh, $biblionumber,"specialUpdate","biblioserver" );
-return $itemnumber;
-}## added new item
+    $sth->execute($biblioitem);
+    my ( $classification, $subclass ) = $sth->fetchrow;
+    my $all         = $classification . " " . $subclass;
+    my $total       = length($all);
+    my $cutterextra = substr( $callnumber, $total - 1 );
 
+    return $cutterextra;
 }
 
+=head2 ModBiblioMarc
 
+=over 4
 
-sub NEWmoditem{
-    my ( $dbh, $record, $biblionumber, $itemnumber ) = @_;
-##Get a hash of this record as well
-my $item=MARCmarc2koha($dbh,$record,"holdings");
-##Add itemnumber incase lost (old bug 090c was lost) --just incase
-my  (  $tagfield,  $tagsubfield )  =MARCfind_marc_from_kohafield("itemnumber","holdings");
-       my $newfield;
-my $old_field = $record->field($tagfield);
-if ($tagfield<10){
-        $newfield = MARC::Field->new($tagfield,  $itemnumber);
-}else{
-       if ($old_field){
-       $old_field->update($tagsubfield=>$biblionumber);
-       $newfield=$old_field->clone();
-       }else{  
-        $newfield = MARC::Field->new($tagfield, '', '', "$tagsubfield" => $itemnumber);
-       }
-}      
-               # drop old field and create new one...
-               
-               $record->delete_field($old_field);
-               $record->insert_fields_ordered($newfield);
-##Add biblionumber incase lost on html
-my  (  $tagfield,  $tagsubfield )  =MARCfind_marc_from_kohafield("biblionumber","holdings");
-       my $newfield;
-my $old_field = $record->field($tagfield);
-if ($tagfield<10){
-        $newfield = MARC::Field->new($tagfield,  $biblionumber);
-}else{
-       if ($old_field){
-       $old_field->update($tagsubfield=>$biblionumber);
-       $newfield=$old_field->clone();
-       }else{  
-        $newfield = MARC::Field->new($tagfield, '', '', "$tagsubfield" => $biblionumber);
-       }
-}      
-               # drop old field and create new one...
-               $record->delete_field($old_field);
-               $record->insert_fields_ordered($newfield);
-               
-###NEU specific add cataloguers cardnumber as well
-my ($tag,$cardtag)=MARCfind_marc_from_kohafield("circid","holdings");
-if ($tag && $cardtag){ 
-my $me= C4::Context->userenv;
-my $cataloger=$me->{'cardnumber'} if ($me);
-my $oldtag=$record->field($tag);
-       if (!$oldtag){
-       my $newtag=  MARC::Field->new($tag, '', '', $cardtag => $cataloger) if ($me);
-       $record->insert_fields_ordered($newtag);        
-       }else{
-       $oldtag->update($cardtag=>$cataloger) if ($me);
-       $record->delete_field($oldtag);
-       $record->insert_fields_ordered($oldtag);
-       }
-}
-## We must add the indexing fields for LC Cutter in MARC record in case it changed
-my ($cutterextra)=itemcalculator($dbh,$biblionumber,$item->{'itemcallnumber'});
-MARCkoha2marcOnefield($record,"cutterextra",$cutterextra,"holdings");
-    OLDmoditem( $dbh, $record,$biblionumber,$itemnumber,$item->{barcode} );
-    ZEBRAop($dbh,$biblionumber,"specialUpdate","biblioserver");
-}
+&ModBiblioMarc($newrec,$biblionumber,$frameworkcode);
 
-sub NEWdelitem {
-    my ( $dbh, $itemnumber ) = @_;
-       
-my $sth=$dbh->prepare("SELECT biblionumber from items where itemnumber=?");
-$sth->execute($itemnumber);
-my $biblionumber=$sth->fetchrow;
-OLDdelitem( $dbh, $itemnumber ) ;
-ZEBRAop($dbh,$biblionumber,"recordDelete","biblioserver");
+Add MARC data for a biblio to koha 
 
-}
+Function exported, but should NOT be used, unless you really know what you're doing
 
+=back
 
+=cut
 
+sub ModBiblioMarc {
 
-sub NEWaddbiblio {
-    my ( $dbh, $record,$frameworkcode ) = @_;
-     my $sth = $dbh->prepare("Select max(biblionumber) from biblio");
-    $sth->execute;
-    my $data   = $sth->fetchrow;
-    my $biblionumber = $data + 1;
+# pass the MARC::Record to this function, and it will create the records in the marc tables
+    my ( $record, $biblionumber, $frameworkcode ) = @_;
+    my $dbh = C4::Context->dbh;
+    my @fields = $record->fields();
+    if ( !$frameworkcode ) {
+        $frameworkcode = "";
+    }
+    my $sth =
+      $dbh->prepare("UPDATE biblio SET frameworkcode=? WHERE biblionumber=?");
+    $sth->execute( $frameworkcode, $biblionumber );
     $sth->finish;
-    # we must add biblionumber MARC::Record...
-  my  (  $tagfield,  $tagsubfield ) =MARCfind_marc_from_kohafield("biblionumber","biblios");
-       my $newfield;
-if ($tagfield<10){
-        $newfield = MARC::Field->new($tagfield,  $biblionumber);
-}else{
- $newfield = MARC::Field->new($tagfield, '', '', "$tagsubfield" => "$biblionumber");
-}
-               # drop old field and create new one..
-               $record->delete_field($newfield);
-               $record->insert_fields_ordered($newfield);
-
-###NEU specific add cataloguers cardnumber as well
-my ($tag,$cardtag)=MARCfind_marc_from_kohafield("indexedby","biblios");
-if ($tag && $cardtag){ 
-my $me= C4::Context->userenv;
-my $cataloger=$me->{'cardnumber'} if ($me);
-my $oldtag=$record->field($tag);
-       if (!$oldtag){
-       my $newtag=  MARC::Field->new($tag, '', '', $cardtag => $cataloger) if ($me);
-       $record->insert_fields_ordered($newtag);        
-       }else{
-       $oldtag->update($cardtag=>$cataloger) if ($me);
-       $record->delete_field($oldtag);
-       $record->insert_fields_ordered($oldtag);
-       }
-}
-## We must add the indexing fields for LC in MARC record--TG
-       &MARCmodLCindex($dbh,$record);
-
-##Find itemtype
- ($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("itemtype","biblios");
-my $itemtype=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-##Find ISBN
-($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("isbn","biblios") ;
-my $isbn=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-##Find ISSN
-($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("issn","biblios") ;
-my $issn=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-    $sth = $dbh->prepare("insert into biblio set biblionumber  = ?, marc = ?, frameworkcode=?, itemtype=?,marcxml=?,title=?,author=?,isbn=?,issn=?" );
-    $sth->execute( $biblionumber,  $record->as_usmarc,$frameworkcode, $itemtype,MARC::File::XML::record( $record ) ,$record->title(),$record->author,$isbn,$issn  );
-
+    my $encoding = C4::Context->preference("marcflavour");
+
+# deal with UNIMARC field 100 (encoding) : create it if needed & set encoding to unicode
+    if ( $encoding eq "UNIMARC" ) {
+        my $string;
+        if ( $record->subfield( 100, "a" ) ) {
+            $string = $record->subfield( 100, "a" );
+            my $f100 = $record->field(100);
+            $record->delete_field($f100);
+        }
+        else {
+            $string = POSIX::strftime( "%Y%m%d", localtime );
+            $string =~ s/\-//g;
+            $string = sprintf( "%-*s", 35, $string );
+        }
+        substr( $string, 22, 6, "frey50" );
+        unless ( $record->subfield( 100, "a" ) ) {
+            $record->insert_grouped_field(
+                MARC::Field->new( 100, "", "", "a" => $string ) );
+        }
+    }
+#     warn "biblionumber : ".$biblionumber;
+    $sth =
+      $dbh->prepare(
+        "update biblioitems set marc=?,marcxml=?  where biblionumber=?");
+    $sth->execute( $record->as_usmarc(), $record->as_xml_record(),
+        $biblionumber );
+#     warn $record->as_xml_record();
     $sth->finish;
-### Do not add biblio to ZEBRA unless there is an item with it -- depends on system preference defaults to NO
-if (C4::Context->preference('AddaloneBiblios')){
- ZEBRAop($dbh,$biblionumber,"specialUpdate","biblioserver");
-}
-    return ($biblionumber);
+    ModZebra($biblionumber,"specialUpdate","biblioserver");
+    return $biblionumber;
 }
 
-sub NEWmodbiblio {
-    my ( $dbh, $biblionumber,$record,$frameworkcode ) = @_;
-##Add biblionumber incase lost on html
-my  (  $tagfield,  $tagsubfield )  =MARCfind_marc_from_kohafield("biblionumber","biblios");
-       my $newfield;
-if ($tagfield<10){
-        $newfield = MARC::Field->new($tagfield,  $biblionumber);
-}else{
- $newfield = MARC::Field->new($tagfield, '', '', "$tagsubfield" => $biblionumber);
-}      
-               # drop old field and create new one...
-               my $old_field = $record->field($tagfield);
-               $record->delete_field($old_field);
-               $record->insert_fields_ordered($newfield);
-
-###NEU specific add cataloguers cardnumber as well
-my ($tag,$cardtag)=MARCfind_marc_from_kohafield("indexedby","biblios");
-if ($tag && $cardtag){ 
-my $me= C4::Context->userenv;
-my $cataloger=$me->{'cardnumber'} if ($me);
-my $oldtag=$record->field($tag);
-       if (!$oldtag){
-       my $newtag=  MARC::Field->new($tag, '', '', $cardtag => $cataloger) if ($me);
-       $record->insert_fields_ordered($newtag);        
-       }else{
-       $oldtag->update($cardtag=>$cataloger) if ($me);
-       $record->delete_field($oldtag);
-       $record->insert_fields_ordered($oldtag);
-       }
-}
-## We must add the indexing fields for LC in MARC record--TG
-   MARCmodLCindex($dbh,$record);
-    OLDmodbiblio ($dbh,$record,$biblionumber,$frameworkcode);
-    my $ok=ZEBRAop($dbh,$biblionumber,"specialUpdate","biblioserver");
-    return ($biblionumber);
-}
+=head2 AddItemInMarc
 
-#
-#
-# OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD OLD
-#
-#
+=over 4
 
-sub OLDnewitems {
+$newbiblionumber = AddItemInMarc( $record, $biblionumber, $frameworkcode );
 
-    my ( $dbh, $barcode,$record) = @_;
-    my $sth = $dbh->prepare("SELECT max(itemnumber) from items");
-    my $data;
-    my $itemnumber;
-    $sth->execute;
-    $data       = $sth->fetchrow_hashref;
-    $itemnumber = $data->{'max(itemnumber)'} + 1;
-    $sth->finish;
-      &MARCkoha2marcOnefield(  $record, "itemnumber", $itemnumber,"holdings" );
-    my ($biblionumbertag,$subf)=MARCfind_marc_from_kohafield( "biblionumber","holdings");
-
-my $biblionumber;
-  if ($biblionumbertag <10){
-  $biblionumber=$record->field($biblionumbertag)->data();
-  }else{
-   $biblionumber=$record->field($biblionumbertag)->subfield($subf);
-  }
-        $sth = $dbh->prepare( "Insert into items set itemnumber = ?,   biblionumber  = ?,barcode = ?,marc=?    ,marcxml=?"   );
-        $sth->execute($itemnumber,$biblionumber,$barcode,$record->as_usmarc(),MARC::File::XML::record( $record));
-    return $itemnumber;
-}
+Add an item in a MARC record and save the MARC record
 
-sub OLDmoditem {
-    my ( $dbh, $record,$biblionumber,$itemnumber,$barcode  ) = @_;
-    my $sth =$dbh->prepare("replace items set  biblionumber=?,marc=?,marcxml=?,barcode=? , itemnumber=?");
-    $sth->execute($biblionumber,$record->as_usmarc(),MARC::File::XML::record( $record),$barcode,$itemnumber);
-    $sth->finish;
-}
+Function exported, but should NOT be used, unless you really know what you're doing
 
-sub OLDdelitem {
-    my ( $dbh, $itemnumber ) = @_;
-my $sth = $dbh->prepare("select * from items where itemnumber=?");
-    $sth->execute($itemnumber);
-    if ( my $data = $sth->fetchrow_hashref ) {
-        $sth->finish;
-        my $query = "replace deleteditems set ";
-        my @bind  = ();
-        foreach my $temp ( keys %$data ) {
-            $query .= "$temp = ?,";
-            push ( @bind, $data->{$temp} );
-        }
+=back
 
-        #replacing the last , by ",?)"
-        $query =~ s/\,$//;
-        $sth = $dbh->prepare($query);
-        $sth->execute(@bind);
-        $sth->finish;
-   $sth = $dbh->prepare("Delete from items where itemnumber=?");
-    $sth->execute($itemnumber);
-    $sth->finish;
-  }
- $sth->finish;
-}
+=cut
 
-sub OLDmodbiblio {
-# modifies the biblio table
-my ($dbh,$record,$biblionumber,$frameworkcode) = @_;
-       if (!$frameworkcode){
-       $frameworkcode="";
-       }
-my ($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("itemtype","biblios");
-my $itemtype=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-my ($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("isbn","biblios");
-my $isbn=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-my ($tagfield,$tagsubfield)=MARCfind_marc_from_kohafield("issn","biblios");
-my $issn=$record->field($tagfield)->subfield($tagsubfield) if ($record->field($tagfield));
-$isbn=~ s/(\.|\?|\;|\=|\-|\/|\\|\||\:|\*|\!|\,|\(|\)|\[|\]|\{|\}|\/)//g;
-$issn=~ s/(\.|\?|\;|\=|\-|\/|\\|\||\:|\*|\!|\,|\(|\)|\[|\]|\{|\}|\/)//g;
-$isbn=~s/^\s+|\s+$//g;
-$isbn=substr($isbn,0,13);
-        my $sth = $dbh->prepare("REPLACE  biblio set biblionumber=?,marc=?,marcxml=?,frameworkcode=? ,itemtype=? , title=?,author=?,isbn=?,issn=?" );
-        $sth->execute( $biblionumber,$record->as_usmarc() ,MARC::File::XML::record( $record), $frameworkcode,$itemtype, $record->title(),$record->author(),$isbn,$issn);  
-        $sth->finish;
-    return $biblionumber;
-}
+sub AddItemInMarc {
 
-sub OLDdelbiblio {
-    my ( $dbh, $biblionumber ) = @_;
-    my $sth = $dbh->prepare("select * from biblio where biblionumber=?");
-    $sth->execute($biblionumber);
-    if ( my $data = $sth->fetchrow_hashref ) {
-        $sth->finish;
-        my $query = "replace deletedbiblio set ";
-        my @bind  = ();
-           foreach my $temp ( keys %$data ) {
-            $query .= "$temp = ?,";
-            push ( @bind, $data->{$temp} );
-           }
+# pass the MARC::Record to this function, and it will create the records in the marc tables
+    my ( $record, $biblionumber, $frameworkcode ) = @_;
+    my $newrec = &GetMarcBiblio($biblionumber);
 
-        #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($biblionumber);
-        $sth->finish;
+    # create it
+    my @fields = $record->fields();
+    foreach my $field (@fields) {
+        $newrec->append_fields($field);
     }
-    $sth->finish;
+
+    # FIXME: should we be making sure the biblionumbers are the same?
+    my $newbiblionumber =
+      &ModBiblioMarc( $newrec, $biblionumber, $frameworkcode );
+    return $newbiblionumber;
 }
 
+=head2 z3950_extended_services
 
-#
-#
-#
-#ZEBRA ZEBRA ZEBRA
-#
-#
+z3950_extended_services($serviceType,$serviceOptions,$record);
 
-sub ZEBRAopfiles{
-##Utility function to write an xml file to disk when the zebra server goes down
-my ($dbh,$biblionumber,$record,$folder,$server)=@_;
-#my $record = XMLgetbiblio($dbh,$biblionumber);
-my $op;
-my $zebradir = C4::Context->zebraconfig($server)->{directory}."/".$folder."/";
-my $zebraroot=C4::Context->zebraconfig($server)->{directory};
-my $serverbase=C4::Context->config($server);
-       unless (opendir(DIR, "$zebradir")) {
-# warn "$zebradir not found";
-                       return;
-       } 
-       closedir DIR;
-       my $filename = $zebradir.$biblionumber;
-if ($record){
-       open (OUTPUT,">", $filename.".xml");
-       print OUTPUT $record;
-       close OUTPUT;
-}
+    z3950_extended_services is used to handle all interactions with Zebra's extended serices package, which is employed to perform all management of the MARC data stored in Zebra.
 
-}
+C<$serviceType> one of: itemorder,create,drop,commit,update,xmlupdate
 
-sub ZEBRAop {
-### Puts the zebra update in queue writes in zebraserver table
-my ($dbh,$biblionumber,$op,$server)=@_;
-my ($record);
+C<$serviceOptions> a has of key/value pairs. For instance, if service_type is 'update', $service_options should contain:
 
-my $sth=$dbh->prepare("insert into zebraqueue  (biblio_auth_number ,server,operation) values(?,?,?)");
-$sth->execute($biblionumber,$server,$op);
-}
+    action => update action, one of specialUpdate, recordInsert, recordReplace, recordDelete, elementUpdate.
 
+and maybe
 
-sub ZEBRAopserver{
-
-###Accepts a $server variable thus we can use it to update  biblios, authorities or other zebra dbs
-my ($record,$op,$server)=@_;
-my @Zconnbiblio;
-my @port;
-my $Zpackage;
-my $tried=0;
-my $recon=0;
-my $reconnect=0;
-$record=Encode::encode("UTF-8",$record);
-my $shadow=$server."shadow";
-reconnect:
-
-$Zconnbiblio[0]=C4::Context->Zconnauth($server);
-if ($record){
-my $Zpackage = $Zconnbiblio[0]->package();
-$Zpackage->option(action => $op);
-       $Zpackage->option(record => $record);
-retry:
-               $Zpackage->send("update");
-my $i;
-my $event;
-
-while (($i = ZOOM::event(\@Zconnbiblio)) != 0) {
-    $event = $Zconnbiblio[0]->last_event();
-    last if $event == ZOOM::Event::ZEND;
-}
- my($error, $errmsg, $addinfo, $diagset) = $Zconnbiblio[0]->error_x();
-       if ($error==10007 && $tried<3) {## timeout --another 30 looonng seconds for this update
-               sleep 1;        ##  wait a sec!
-               $tried=$tried+1;
-               goto "retry";
-       }elsif ($error==2 && $tried<2) {## timeout --temporary zebra error !whatever that means
-               sleep 2;        ##  wait two seconds!
-               $tried=$tried+1;
-               goto "retry";
-       }elsif($error==10004 && $recon==0){##Lost connection -reconnect
-               sleep 1;        ##  wait a sec!
-               $recon=1;
-               $Zpackage->destroy();
-               $Zconnbiblio[0]->destroy();
-               goto "reconnect";
-       }elsif ($error){
-       #       warn "Error-$server   $op  /errcode:, $error, /MSG:,$errmsg,$addinfo \n";       
-               $Zpackage->destroy();
-               $Zconnbiblio[0]->destroy();
-       #       ZEBRAopfiles($dbh,$biblionumber,$record,$op,$server);
-               return 0;
-       }
-       ## System preference batchMode=1 means wea are bulk importing
-       ## DO NOT COMMIT while in batchMode for faster operation
-       my $batchmode=C4::Context->preference('batchMode');
-        if (C4::Context->$shadow >0 && !$batchmode){
-        $Zpackage->send('commit');
-               while (($i = ZOOM::event(\@Zconnbiblio)) != 0) {
-                $event = $Zconnbiblio[0]->last_event();
-               last if $event == ZOOM::Event::ZEND;
-               }
-            my($error, $errmsg, $addinfo, $diagset) = $Zconnbiblio[0]->error_x();
-            if ($error) { ## This is serious ZEBRA server is not updating      
-            $Zpackage->destroy();
-            $Zconnbiblio[0]->destroy();
-            return 0;
-           }
-        }##commit
-#
-$Zpackage->destroy();
-$Zconnbiblio[0]->destroy();
-return 1;
-}
-return 0;
-}
+    recordidOpaque => Opaque Record ID (user supplied) or recordidNumber => Record ID number (system number).
+    syntax => the record syntax (transfer syntax)
+    databaseName = Database from connection object
 
-sub ZEBRA_readyXML{
-my ($dbh,$biblionumber)=@_;
-my $biblioxml=XMLgetbiblio($dbh,$biblionumber);
-my @itemxml=XMLgetallitems($dbh,$biblionumber);
-my $zebraxml=collection_header();
-$zebraxml.="<koharecord>";
-$zebraxml.=$biblioxml;
-$zebraxml.="<holdings>";
-      foreach my $item(@itemxml){
-       $zebraxml.=$item if $item;
-     }
-$zebraxml.="</holdings>";
-$zebraxml.="</koharecord>";
-$zebraxml.="</kohacollection>";
-return $zebraxml;
-}
+    To set serviceOptions, call set_service_options($serviceType)
 
-sub ZEBRA_readyXML_noheader{
-my ($dbh,$biblionumber)=@_;
-my $biblioxml=XMLgetbiblio($dbh,$biblionumber);
-my @itemxml=XMLgetallitems($dbh,$biblionumber);
-my $zebraxml="<koharecord>";
-$zebraxml.=$biblioxml;
-$zebraxml.="<holdings>";
-      foreach my $item(@itemxml){
-       $zebraxml.=$item if $item;
-     }
-$zebraxml.="</holdings>";
-$zebraxml.="</koharecord>";
-return $zebraxml;
-}
+C<$record> the record, if one is needed for the service type
 
-#
-#
-# various utility subs and those not complying to new rules
-#
-#
+    A record should be in XML. You can convert it to XML from MARC by running it through marc2xml().
 
-sub newbiblio {
-## Used in acqui management -- creates the biblio from hash rather than marc-record
-    my ($biblio) = @_;
-    my $dbh    = C4::Context->dbh;
-my $record=MARCkoha2marc($dbh,$biblio,"biblios");
-$record->encoding('UTF-8');
-   my $biblionumber=NEWnewbiblio($dbh,$record);
-    return ($biblionumber);
-}
-sub modbiblio {
-## Used in acqui management -- modifies the biblio from hash rather than marc-record
-    my ($biblio) = @_;
-    my $dbh    = C4::Context->dbh;
-my $record=MARCkoha2marc($dbh,$biblio,"biblios");
-   my $biblionumber=NEWmodbiblio($dbh,$record,$biblio->{biblionumber});
-    return ($biblionumber);
-}
+=cut
 
-sub newitems {
-## Used in acqui management -- creates the item from hash rather than marc-record
-    my ( $item, @barcodes ) = @_;
-    my $dbh = C4::Context->dbh;
-    my $errors;
-    my $itemnumber;
-    my $error;
-    foreach my $barcode (@barcodes) {
-       $item->{barcode}=$barcode;
-my $record=MARCkoha2marc($dbh,$item,"holdings");       
-  my $itemnumber=     NEWnewitem($dbh,$record,$item->{biblionumber});
-    
-    }
-    return $itemnumber ;
-}
+sub z3950_extended_services {
+    my ( $server, $serviceType, $action, $serviceOptions ) = @_;
 
+    # get our connection object
+    my $Zconn = C4::Context->Zconn( $server, 0, 1 );
 
+    # create a new package object
+    my $Zpackage = $Zconn->package();
 
+    # set our options
+    $Zpackage->option( action => $action );
 
-sub getitemtypes {
-    my $dbh   = C4::Context->dbh;
-    my $query = "select * from itemtypes order by description";
-    my $sth   = $dbh->prepare($query);
+    if ( $serviceOptions->{'databaseName'} ) {
+        $Zpackage->option( databaseName => $serviceOptions->{'databaseName'} );
+    }
+    if ( $serviceOptions->{'recordIdNumber'} ) {
+        $Zpackage->option(
+            recordIdNumber => $serviceOptions->{'recordIdNumber'} );
+    }
+    if ( $serviceOptions->{'recordIdOpaque'} ) {
+        $Zpackage->option(
+            recordIdOpaque => $serviceOptions->{'recordIdOpaque'} );
+    }
 
-    # || 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
+ # this is an ILL request (Zebra doesn't support it, but Koha could eventually)
+ #if ($serviceType eq 'itemorder') {
+ #   $Zpackage->option('contact-name' => $serviceOptions->{'contact-name'});
+ #   $Zpackage->option('contact-phone' => $serviceOptions->{'contact-phone'});
+ #   $Zpackage->option('contact-email' => $serviceOptions->{'contact-email'});
+ #   $Zpackage->option('itemorder-item' => $serviceOptions->{'itemorder-item'});
+ #}
 
-    $sth->finish;
-    return ( $count, @results );
-}    # sub getitemtypes
-
-
-
-sub getkohafields{
-#returns MySQL like fieldnames to emulate searches on sql like fieldnames
-my $type=@_;
-## Either opac or intranet to select appropriate fields
-## Assumes intranet
-$type="intra" unless $type;
-if ($type eq "intranet"){ $type="intra";}
-my $dbh   = C4::Context->dbh;
-  my $i=0;
-my @results;
-$type=$type."show";
-my $sth=$dbh->prepare("SELECT  * FROM koha_attr  where $type=1 order by liblibrarian");
-$sth->execute();
-while (my $data=$sth->fetchrow_hashref){
-       $results[$i]=$data;
-       $i++;
-       }
-$sth->finish;
-return ($i,@results);
-}
+    if ( $serviceOptions->{record} ) {
+        $Zpackage->option( record => $serviceOptions->{record} );
+
+        # can be xml or marc
+        if ( $serviceOptions->{'syntax'} ) {
+            $Zpackage->option( syntax => $serviceOptions->{'syntax'} );
+        }
+    }
 
+    # send the request, handle any exception encountered
+    eval { $Zpackage->send($serviceType) };
+    if ( $@ && $@->isa("ZOOM::Exception") ) {
+        return "error:  " . $@->code() . " " . $@->message() . "\n";
+    }
 
+    # free up package resources
+    $Zpackage->destroy();
+}
 
+=head2 set_service_options
 
+my $serviceOptions = set_service_options($serviceType);
 
-sub DisplayISBN {
-## Old style ISBN handling should be modified to accept 13 digits
-       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";
-}
-sub calculatelc{
-## Function to create padded LC call number for sorting items with their LC code. Not exported
-my  ($classification)=@_;
-$classification=~s/^\s+|\s+$//g;
-my $i=0;
-my $lc2;
-my $lc1;
-for  ($i=0; $i<length($classification);$i++){
-my $c=(substr($classification,$i,1));
-       if ($c ge '0' && $c le '9'){
-       
-       $lc2=substr($classification,$i);
-       last;
-       }else{
-       $lc1.=substr($classification,$i,1);
-       
-       }
-}#while
-
-my $other=length($lc1);
-if(!$lc1){$other=0;}
-my $extras;
-if ($other<4){
-       for (1..(4-$other)){
-       $extras.="0";
-       }
-}
- $lc1.=$extras;
-$lc2=~ s/^ //g;
-
-$lc2=~ s/ //g;
-$extras="";
-##Find the decimal part of $lc2
-my $pos=index($lc2,".");
-if ($pos<0){$pos=length($lc2);}
-if ($pos>=0 && $pos<5){
-##Pad lc2 with zeros to create a 5digit decimal needed in marc record to sort as numeric
-
-       for (1..(5-$pos)){
-       $extras.="0";
-       }
-}
-$lc2=$extras.$lc2;
-return($lc1.$lc2);
-}
+C<$serviceType> itemorder,create,drop,commit,update,xmlupdate
 
-sub itemcalculator{
-## Sublimentary function to obtain sorted LC for items. Not exported
-my ($dbh,$biblionumber,$callnumber)=@_;
-my ($record,$frameworkcode)=MARCgetbiblio($dbh,$biblionumber);
-my $biblio=MARCmarc2koha($dbh,$record,$frameworkcode,"biblios");
+Currently, we only support 'create', 'commit', and 'update'. 'drop' support will be added as soon as Zebra supports it.
 
-my $all=$biblio->{classification}." ".$biblio->{subclass};
-my $total=length($all);
-my $cutterextra=substr($callnumber,$total);
+=cut
 
-return $cutterextra;
+sub set_service_options {
+    my ($serviceType) = @_;
+    my $serviceOptions;
 
-}
+# FIXME: This needs to be an OID ... if we ever need 'syntax' this sub will need to change
+#   $serviceOptions->{ 'syntax' } = ''; #zebra doesn't support syntaxes other than xml
 
+    if ( $serviceType eq 'commit' ) {
 
-#### This function allows decoding of only title and author out of a MARC record
-  sub func_title_author {
-        my ($tagno,$tagdata) = @_;
-  my ($titlef,$subf)=&MARCfind_marc_from_kohafield("title","biblios");
-  my ($authf,$subf)=&MARCfind_marc_from_kohafield("author","biblios");
-       return ($tagno == $titlef || $tagno == $authf);
+        # nothing to do
     }
+    if ( $serviceType eq 'create' ) {
 
-
+        # nothing to do
+    }
+    if ( $serviceType eq 'drop' ) {
+        die "ERROR: 'drop' not currently supported (by Zebra)";
+    }
+    return $serviceOptions;
+}
 
 END { }    # module clean-up code here (global destructor)
 
-=back
+1;
+
+__END__
 
 =head1 AUTHOR
 
 Koha Developement team <info@koha.org>
 
+Paul POULAIN paul.poulain@free.fr
 
+Joshua Ferraro jmf@liblime.com
 
+=cut
+
+# $Id$
+# $Log$
+# Revision 1.201  2007/04/27 14:00:49  hdl
+# Removing $dbh from GetMarcFromKohaField (dbh is not used in this function.)
+#
+# Revision 1.200  2007/04/25 16:26:42  tipaul
+# Koha 3.0 nozebra 1st commit : the script misc/migration_tools/rebuild_nozebra.pl build the nozebra table, and, if you set NoZebra to Yes, queries will be done through zebra. TODO :
+# - add nozebra table management on biblio editing
+# - the index table content is hardcoded. I still have to add some specific systempref to let the library update it
+# - manage pagination (next/previous)
+# - manage facets
+# WHAT works :
+# - NZgetRecords : has exactly the same API & returns as zebra getQuery, except that some parameters are unused
+# - search & sort works quite good
+# - CQL parser is better that what I thought I could do : title="harry and sally" and publicationyear>2000 not itemtype=LIVR should work fine
+#
+# Revision 1.199  2007/04/24 09:07:53  tipaul
+# moving dotransfer to Biblio.pm::ModItemTransfer + some CheckReserves fixes
+#
+# Revision 1.198  2007/04/23 15:21:17  tipaul
+# renaming currenttransfers to transferstoreceive
+#
+# Revision 1.197  2007/04/18 17:00:14  tipaul
+# removing all useless %env / $env
+#
+# Revision 1.196  2007/04/17 08:48:00  tipaul
+# circulation cleaning continued: bufixing
+#
+# Revision 1.195  2007/04/04 16:46:22  tipaul
+# HUGE COMMIT : code cleaning circulation.
+#
+# some stuff to do, i'll write a mail on koha-devel NOW !
+#
+# Revision 1.194  2007/03/30 12:00:42  tipaul
+# why the hell do we need to explicitly utf8 decode this string ? I really don't know, but it seems it's mandatory, otherwise, tag descriptions are not properly encoded...
+#
+# Revision 1.193  2007/03/29 16:45:53  tipaul
+# Code cleaning of Biblio.pm (continued)
+#
+# All subs have be cleaned :
+# - removed useless
+# - merged some
+# - reordering Biblio.pm completly
+# - using only naming conventions
+#
+# Seems to have broken nothing, but it still has to be heavily tested.
+# Note that Biblio.pm is now much more efficient than previously & probably more reliable as well.
+#
+# Revision 1.192  2007/03/29 13:30:31  tipaul
+# Code cleaning :
+# == Biblio.pm cleaning (useless) ==
+# * some sub declaration dropped
+# * removed modbiblio sub
+# * removed moditem sub
+# * removed newitems. It was used only in finishrecieve. Replaced by a TransformKohaToMarc+AddItem, that is better.
+# * removed MARCkoha2marcItem
+# * removed MARCdelsubfield declaration
+# * removed MARCkoha2marcBiblio
+#
+# == Biblio.pm cleaning (naming conventions) ==
+# * MARCgettagslib renamed to GetMarcStructure
+# * MARCgetitems renamed to GetMarcItem
+# * MARCfind_frameworkcode renamed to GetFrameworkCode
+# * MARCmarc2koha renamed to TransformMarcToKoha
+# * MARChtml2marc renamed to TransformHtmlToMarc
+# * MARChtml2xml renamed to TranformeHtmlToXml
+# * zebraop renamed to ModZebra
+#
+# == MARC=OFF ==
+# * removing MARC=OFF related scripts (in cataloguing directory)
+# * removed checkitems (function related to MARC=off feature, that is completly broken in head. If someone want to reintroduce it, hard work coming...)
+# * removed getitemsbybiblioitem (used only by MARC=OFF scripts, that is removed as well)
+#
+# Revision 1.191  2007/03/29 09:42:13  tipaul
+# adding default value new feature into cataloguing. The system (definition) part has already been added by toins
+#
+# Revision 1.190  2007/03/29 08:45:19  hdl
+# Deleting ignore_errors(1) pour MARC::Charset
+#
+# Revision 1.189  2007/03/28 10:39:16  hdl
+# removing $dbh as a parameter in AuthoritiesMarc functions
+# And reporting all differences into the scripts taht relies on those functions.