Fixes two bugs affecting Web installer when used with Unimarc/French
[koha.git] / C4 / Koha.pm
index f08794f..6c6b6e0 100644 (file)
@@ -17,28 +17,27 @@ package C4::Koha;
 # 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 C4::Output;
+our ($VERSION,@ISA,@EXPORT);
 
-use vars qw($VERSION @ISA @EXPORT);
-
-$VERSION = 0.01;
+$VERSION = 3.00;
 
 =head1 NAME
 
-C4::Koha - Perl Module containing convenience functions for Koha scripts
+    C4::Koha - Perl Module containing convenience functions for Koha scripts
 
 =head1 SYNOPSIS
 
   use C4::Koha;
 
 
-  $date = slashifyDate("01-01-2002")
-
 =head1 DESCRIPTION
 
-Koha.pm provides many functions for Koha scripts.
+    Koha.pm provides many functions for Koha scripts.
 
 =head1 FUNCTIONS
 
@@ -46,235 +45,168 @@ Koha.pm provides many functions for Koha scripts.
 
 =cut
 
-@ISA = qw(Exporter);
+@ISA    = qw(Exporter);
 @EXPORT = qw(
-                       &subfield_is_koha_internal_p
-                       &getbranches &getbranch &getbranchdetail
-                       &getprinters &getprinter
-                       &getitemtypes &getitemtypeinfo
-                        get_itemtypeinfos_of
-                       &getframeworks &getframeworkinfo
-                       &getauthtypes &getauthtype
-                       &getallthemes &getalllanguages
-                       &getallbranches &getletters
-                       &getbranchname
-                        getnbpages
-                        getitemtypeimagedir
-                        getitemtypeimagesrc
-                        getitemtypeimagesrcfromurl
-                       &getcities
-                       &getroadtypes
-                        get_branchinfos_of
-                        get_notforloan_label_of
-                        get_infos_of
-                       $DEBUG);
-
-use vars qw();
+  &slashifyDate
+  &DisplayISBN
+  &subfield_is_koha_internal_p
+  &GetPrinters &GetPrinter
+  &GetItemTypes &getitemtypeinfo
+  &GetCcodes
+  &get_itemtypeinfos_of
+  &getframeworks &getframeworkinfo
+  &getauthtypes &getauthtype
+  &getallthemes
+  &getFacets
+  &displayServers
+  &getnbpages
+  &getitemtypeimagesrcfromurl
+  &get_infos_of
+  &get_notforloan_label_of
+  &getitemtypeimagedir
+  &getitemtypeimagesrc
+  &GetAuthorisedValues
+  &FixEncoding
+  &GetKohaAuthorisedValues
+  &GetAuthValCode
+  &GetManagedTagSubfields
+
+  $DEBUG
+  );
 
 my $DEBUG = 0;
 
-# FIXME.. this should be moved to a MARC-specific module
-sub subfield_is_koha_internal_p ($) {
-    my($subfield) = @_;
-
-    # We could match on 'lib' and 'tab' (and 'mandatory', & more to come!)
-    # But real MARC subfields are always single-character
-    # so it really is safer just to check the length
-
-    return length $subfield != 1;
-}
-
-=head2 getbranches
-
-  $branches = &getbranches();
-  returns informations about branches.
-  Create a branch selector with the following code
-  Is branchIndependant sensitive
-   When IndependantBranches is set AND user is not superlibrarian, displays only user's branch
-  
-=head3 in PERL SCRIPT
-
-my $branches = getbranches;
-my @branchloop;
-foreach my $thisbranch (sort keys %$branches) {
-       my $selected = 1 if $thisbranch eq $branch;
-       my %row =(value => $thisbranch,
-                               selected => $selected,
-                               branchname => $branches->{$thisbranch}->{'branchname'},
-                       );
-       push @branchloop, \%row;
-}
+=head2 slashifyDate
 
+  $slash_date = &slashifyDate($dash_date);
 
-=head3 in TEMPLATE  
-                       <select name="branch">
-                               <option value="">Default</option>
-                       <!-- TMPL_LOOP name="branchloop" -->
-                               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="branchname" --></option>
-                       <!-- /TMPL_LOOP -->
-                       </select>
+    Takes a string of the form "DD-MM-YYYY" (or anything separated by
+    dashes), converts it to the form "YYYY/MM/DD", and returns the result.
 
 =cut
 
-sub getbranches {
-# returns a reference to a hash of references to branches...
-       my %branches;
-       my $dbh = C4::Context->dbh;
-       my $sth;
-       if (C4::Context->preference("IndependantBranches") && (C4::Context->userenv->{flags}!=1)){
-               my $strsth ="Select * from branches ";
-               $strsth.= " WHERE branchcode = ".$dbh->quote(C4::Context->userenv->{branch});
-               $strsth.= " order by branchname";
-               $sth=$dbh->prepare($strsth);
-       } else {
-       $sth = $dbh->prepare("Select * from branches order by branchname");
-       }
-       $sth->execute;
-       while (my $branch=$sth->fetchrow_hashref) {
-               my $nsth = $dbh->prepare("select categorycode from branchrelations where branchcode = ?");
-               $nsth->execute($branch->{'branchcode'});
-               while (my ($cat) = $nsth->fetchrow_array) {
-                       # FIXME - This seems wrong. It ought to be
-                       # $branch->{categorycodes}{$cat} = 1;
-                       # otherwise, there's a namespace collision if there's a
-                       # category with the same name as a field in the 'branches'
-                       # table (i.e., don't create a category called "issuing").
-                       # In addition, the current structure doesn't really allow
-                       # you to list the categories that a branch belongs to:
-                       # you'd have to list keys %$branch, and remove those keys
-                       # that aren't fields in the "branches" table.
-                       $branch->{$cat} = 1;
-                       }
-                       $branches{$branch->{'branchcode'}}=$branch;
-       }
-       return (\%branches);
-}
+sub slashifyDate {
 
-sub getbranchname {
-       my ($branchcode)=@_;
-       my $dbh = C4::Context->dbh;
-       my $sth;
-       $sth = $dbh->prepare("Select branchname from branches where branchcode=?");
-       $sth->execute($branchcode);
-       my $branchname = $sth->fetchrow_array;
-       $sth->finish;
-       
-       return($branchname);
+    # accepts a date of the form xx-xx-xx[xx] and returns it in the
+    # form xx/xx/xx[xx]
+    my @dateOut = split( '-', shift );
+    return ("$dateOut[2]/$dateOut[1]/$dateOut[0]");
 }
 
-=head2 getallbranches
 
-  $branches = &getallbranches();
-  returns informations about ALL branches.
-  Create a branch selector with the following code
-  IndependantBranches Insensitive...
-  
-=head3 in PERL SCRIPT
-
-my $branches = getallbranches;
-my @branchloop;
-foreach my $thisbranch (keys %$branches) {
-       my $selected = 1 if $thisbranch eq $branch;
-       my %row =(value => $thisbranch,
-                               selected => $selected,
-                               branchname => $branches->{$thisbranch}->{'branchname'},
-                       );
-       push @branchloop, \%row;
-}
+=head2 DisplayISBN
 
-
-=head3 in TEMPLATE  
-                       <select name="branch">
-                               <option value="">Default</option>
-                       <!-- TMPL_LOOP name="branchloop" -->
-                               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="branchname" --></option>
-                       <!-- /TMPL_LOOP -->
-                       </select>
+    my $string = DisplayISBN( $isbn );
 
 =cut
 
+sub DisplayISBN {
+    my ($isbn) = @_;
+    if (length ($isbn)<13){
+    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 ) {
 
-sub getallbranches {
-# returns a reference to a hash of references to ALL branches...
-       my %branches;
-       my $dbh = C4::Context->dbh;
-       my $sth;
-       $sth = $dbh->prepare("Select * from branches order by branchname");
-       $sth->execute;
-       while (my $branch=$sth->fetchrow_hashref) {
-               my $nsth = $dbh->prepare("select categorycode from branchrelations where branchcode = ?");
-               $nsth->execute($branch->{'branchcode'});
-               while (my ($cat) = $nsth->fetchrow_array) {
-                       # FIXME - This seems wrong. It ought to be
-                       # $branch->{categorycodes}{$cat} = 1;
-                       # otherwise, there's a namespace collision if there's a
-                       # category with the same name as a field in the 'branches'
-                       # table (i.e., don't create a category called "issuing").
-                       # In addition, the current structure doesn't really allow
-                       # you to list the categories that a branch belongs to:
-                       # you'd have to list keys %$branch, and remove those keys
-                       # that aren't fields in the "branches" table.
-                       $branch->{$cat} = 1;
-                       }
-                       $branches{$branch->{'branchcode'}}=$branch;
-       }
-       return (\%branches);
-}
-
-=head2 getletters
-
-  $letters = &getletters($category);
-  returns informations about letters.
-  if needed, $category filters for letters given category
-  Create a letter selector with the following code
+        # 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";
+    } else {
+      my $seg1;
+      $seg1 = substr( $isbn, 0, 3 );
+      my $seg2;
+      if ( substr( $isbn, 3, 1 ) <= 7 ) {
+          $seg2 = substr( $isbn, 3, 1 );
+      }
+      elsif ( substr( $isbn, 3, 2 ) <= 94 ) {
+          $seg2 = substr( $isbn, 3, 2 );
+      }
+      elsif ( substr( $isbn, 3, 3 ) <= 995 ) {
+          $seg2 = substr( $isbn, 3, 3 );
+      }
+      elsif ( substr( $isbn, 3, 4 ) <= 9989 ) {
+          $seg2 = substr( $isbn, 3, 4 );
+      }
+      else {
+          $seg2 = substr( $isbn, 3, 5 );
+      }
+      my $x = substr( $isbn, length($seg2) +3);
+      my $seg3;
+      if ( substr( $x, 0, 2 ) <= 19 ) {
   
-=head3 in PERL SCRIPT
-
-my $letters = getletters($cat);
-my @letterloop;
-foreach my $thisletter (keys %$letters) {
-       my $selected = 1 if $thisletter eq $letter;
-       my %row =(value => $thisletter,
-                               selected => $selected,
-                               lettername => $letters->{$thisletter},
-                       );
-       push @letterloop, \%row;
+          # if(sTmp2 < 10) sTmp2 = "0" sTmp2;
+          $seg3 = substr( $x, 0, 2 );
+      }
+      elsif ( substr( $x, 0, 3 ) <= 699 ) {
+          $seg3 = substr( $x, 0, 3 );
+      }
+      elsif ( substr( $x, 0, 4 ) <= 8399 ) {
+          $seg3 = substr( $x, 0, 4 );
+      }
+      elsif ( substr( $x, 0, 5 ) <= 89999 ) {
+          $seg3 = substr( $x, 0, 5 );
+      }
+      elsif ( substr( $x, 0, 6 ) <= 9499999 ) {
+          $seg3 = substr( $x, 0, 6 );
+      }
+      else {
+          $seg3 = substr( $x, 0, 7 );
+      }
+      my $seg4 = substr( $x, length($seg3) );
+      $seg4 = substr( $seg4, 0, length($seg4) - 1 );
+      my $seg5 = substr( $x, -1, 1 );
+      return "$seg1-$seg2-$seg3-$seg4-$seg5";       
+    }    
 }
 
+# FIXME.. this should be moved to a MARC-specific module
+sub subfield_is_koha_internal_p ($) {
+    my ($subfield) = @_;
 
-=head3 in TEMPLATE  
-                       <select name="letter">
-                               <option value="">Default</option>
-                       <!-- TMPL_LOOP name="letterloop" -->
-                               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="lettername" --></option>
-                       <!-- /TMPL_LOOP -->
-                       </select>
-
-=cut
+    # We could match on 'lib' and 'tab' (and 'mandatory', & more to come!)
+    # But real MARC subfields are always single-character
+    # so it really is safer just to check the length
 
-sub getletters {
-# returns a reference to a hash of references to ALL letters...
-       my $cat =@_;
-       my %letters;
-       my $dbh = C4::Context->dbh;
-       my $sth;
-       if ($cat ne ""){
-               $sth = $dbh->prepare("Select * from letter where module = \'".$cat."\' order by name");
-       } else {
-               $sth = $dbh->prepare("Select * from letter order by name");
-       }
-       $sth->execute;
-       my $count;
-       while (my $letter=$sth->fetchrow_hashref) {
-                       $letters{$letter->{'code'}}=$letter->{'name'};
-                       $count++;
-       }
-       return ($count,\%letters);
+    return length $subfield != 1;
 }
 
-=head2 getitemtypes
+=head2 GetItemTypes
 
-  $itemtypes = &getitemtypes();
+  $itemtypes = &GetItemTypes();
 
 Returns information about existing itemtypes.
 
@@ -282,59 +214,84 @@ build a HTML select with the following code :
 
 =head3 in PERL SCRIPT
 
-my $itemtypes = getitemtypes;
-my @itemtypesloop;
-foreach my $thisitemtype (sort keys %$itemtypes) {
-       my $selected = 1 if $thisitemtype eq $itemtype;
-       my %row =(value => $thisitemtype,
-                               selected => $selected,
-                               description => $itemtypes->{$thisitemtype}->{'description'},
-                       );
-       push @itemtypesloop, \%row;
-}
-$template->param(itemtypeloop => \@itemtypesloop);
+    my $itemtypes = GetItemTypes;
+    my @itemtypesloop;
+    foreach my $thisitemtype (sort keys %$itemtypes) {
+        my $selected = 1 if $thisitemtype eq $itemtype;
+        my %row =(value => $thisitemtype,
+                    selected => $selected,
+                    description => $itemtypes->{$thisitemtype}->{'description'},
+                );
+        push @itemtypesloop, \%row;
+    }
+    $template->param(itemtypeloop => \@itemtypesloop);
 
 =head3 in TEMPLATE
 
-<form action='<!-- TMPL_VAR name="script_name" -->' method=post>
-       <select name="itemtype">
-               <option value="">Default</option>
-       <!-- TMPL_LOOP name="itemtypeloop" -->
-               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="description" --></option>
-       <!-- /TMPL_LOOP -->
-       </select>
-       <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
-       <input type="submit" value="OK" class="button">
-</form>
-
+    <form action='<!-- TMPL_VAR name="script_name" -->' method=post>
+        <select name="itemtype">
+            <option value="">Default</option>
+        <!-- TMPL_LOOP name="itemtypeloop" -->
+            <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="description" --></option>
+        <!-- /TMPL_LOOP -->
+        </select>
+        <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
+        <input type="submit" value="OK" class="button">
+    </form>
 
 =cut
 
-sub getitemtypes {
-# returns a reference to a hash of references to branches...
-       my %itemtypes;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from itemtypes");
-       $sth->execute;
-       while (my $IT=$sth->fetchrow_hashref) {
-                       $itemtypes{$IT->{'itemtype'}}=$IT;
-       }
-       return (\%itemtypes);
+sub GetItemTypes {
+
+    # returns a reference to a hash of references to branches...
+    my %itemtypes;
+    my $dbh   = C4::Context->dbh;
+    my $query = qq|
+        SELECT *
+        FROM   itemtypes
+    |;
+    my $sth = $dbh->prepare($query);
+    $sth->execute;
+    while ( my $IT = $sth->fetchrow_hashref ) {
+        $itemtypes{ $IT->{'itemtype'} } = $IT;
+    }
+    return ( \%itemtypes );
 }
 
-# FIXME this function is better and should replace getitemtypes everywhere
 sub get_itemtypeinfos_of {
     my @itemtypes = @_;
 
     my $query = '
 SELECT itemtype,
        description,
+       imageurl,
        notforloan
   FROM itemtypes
-  WHERE itemtype IN ('.join(',', map({"'".$_."'"} @itemtypes)).')
+  WHERE itemtype IN (' . join( ',', map( { "'" . $_ . "'" } @itemtypes ) ) . ')
 ';
 
-    return get_infos_of($query, 'itemtype');
+    return get_infos_of( $query, 'itemtype' );
+}
+
+# this is temporary until we separate collection codes and item types
+sub GetCcodes {
+    my $count = 0;
+    my @results;
+    my $dbh = C4::Context->dbh;
+    my $sth =
+      $dbh->prepare(
+        "SELECT * FROM authorised_values ORDER BY authorised_value");
+    $sth->execute;
+    while ( my $data = $sth->fetchrow_hashref ) {
+        if ( $data->{category} eq "CCODE" ) {
+            $count++;
+            $results[$count] = $data;
+
+            #warn "data: $data";
+        }
+    }
+    $sth->finish;
+    return ( $count, @results );
 }
 
 =head2 getauthtypes
@@ -350,51 +307,53 @@ build a HTML select with the following code :
 my $authtypes = getauthtypes;
 my @authtypesloop;
 foreach my $thisauthtype (keys %$authtypes) {
-       my $selected = 1 if $thisauthtype eq $authtype;
-       my %row =(value => $thisauthtype,
-                               selected => $selected,
-                               authtypetext => $authtypes->{$thisauthtype}->{'authtypetext'},
-                       );
-       push @authtypesloop, \%row;
+    my $selected = 1 if $thisauthtype eq $authtype;
+    my %row =(value => $thisauthtype,
+                selected => $selected,
+                authtypetext => $authtypes->{$thisauthtype}->{'authtypetext'},
+            );
+    push @authtypesloop, \%row;
 }
 $template->param(itemtypeloop => \@itemtypesloop);
 
 =head3 in TEMPLATE
 
 <form action='<!-- TMPL_VAR name="script_name" -->' method=post>
-       <select name="authtype">
-       <!-- TMPL_LOOP name="authtypeloop" -->
-               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="authtypetext" --></option>
-       <!-- /TMPL_LOOP -->
-       </select>
-       <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
-       <input type="submit" value="OK" class="button">
+    <select name="authtype">
+    <!-- TMPL_LOOP name="authtypeloop" -->
+        <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="authtypetext" --></option>
+    <!-- /TMPL_LOOP -->
+    </select>
+    <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
+    <input type="submit" value="OK" class="button">
 </form>
 
 
 =cut
 
 sub getauthtypes {
-# returns a reference to a hash of references to authtypes...
-       my %authtypes;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from auth_types order by authtypetext");
-       $sth->execute;
-       while (my $IT=$sth->fetchrow_hashref) {
-                       $authtypes{$IT->{'authtypecode'}}=$IT;
-       }
-       return (\%authtypes);
+
+    # returns a reference to a hash of references to authtypes...
+    my %authtypes;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("select * from auth_types order by authtypetext");
+    $sth->execute;
+    while ( my $IT = $sth->fetchrow_hashref ) {
+        $authtypes{ $IT->{'authtypecode'} } = $IT;
+    }
+    return ( \%authtypes );
 }
 
 sub getauthtype {
-       my ($authtypecode) = @_;
-# returns a reference to a hash of references to authtypes...
-       my %authtypes;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from auth_types where authtypecode=?");
-       $sth->execute($authtypecode);
-       my $res=$sth->fetchrow_hashref;
-       return $res;
+    my ($authtypecode) = @_;
+
+    # returns a reference to a hash of references to authtypes...
+    my %authtypes;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("select * from auth_types where authtypecode=?");
+    $sth->execute($authtypecode);
+    my $res = $sth->fetchrow_hashref;
+    return $res;
 }
 
 =head2 getframework
@@ -410,42 +369,44 @@ build a HTML select with the following code :
 my $frameworks = frameworks();
 my @frameworkloop;
 foreach my $thisframework (keys %$frameworks) {
-       my $selected = 1 if $thisframework eq $frameworkcode;
-       my %row =(value => $thisframework,
-                               selected => $selected,
-                               description => $frameworks->{$thisframework}->{'frameworktext'},
-                       );
-       push @frameworksloop, \%row;
+    my $selected = 1 if $thisframework eq $frameworkcode;
+    my %row =(value => $thisframework,
+                selected => $selected,
+                description => $frameworks->{$thisframework}->{'frameworktext'},
+            );
+    push @frameworksloop, \%row;
 }
 $template->param(frameworkloop => \@frameworksloop);
 
 =head3 in TEMPLATE
 
 <form action='<!-- TMPL_VAR name="script_name" -->' method=post>
-       <select name="frameworkcode">
-               <option value="">Default</option>
-       <!-- TMPL_LOOP name="frameworkloop" -->
-               <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="frameworktext" --></option>
-       <!-- /TMPL_LOOP -->
-       </select>
-       <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
-       <input type="submit" value="OK" class="button">
+    <select name="frameworkcode">
+        <option value="">Default</option>
+    <!-- TMPL_LOOP name="frameworkloop" -->
+        <option value="<!-- TMPL_VAR name="value" -->" <!-- TMPL_IF name="selected" -->selected<!-- /TMPL_IF -->><!-- TMPL_VAR name="frameworktext" --></option>
+    <!-- /TMPL_LOOP -->
+    </select>
+    <input type=text name=searchfield value="<!-- TMPL_VAR name="searchfield" -->">
+    <input type="submit" value="OK" class="button">
 </form>
 
 
 =cut
 
 sub getframeworks {
-# returns a reference to a hash of references to branches...
-       my %itemtypes;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from biblio_framework");
-       $sth->execute;
-       while (my $IT=$sth->fetchrow_hashref) {
-                       $itemtypes{$IT->{'frameworkcode'}}=$IT;
-       }
-       return (\%itemtypes);
+
+    # returns a reference to a hash of references to branches...
+    my %itemtypes;
+    my $dbh = C4::Context->dbh;
+    my $sth = $dbh->prepare("select * from biblio_framework");
+    $sth->execute;
+    while ( my $IT = $sth->fetchrow_hashref ) {
+        $itemtypes{ $IT->{'frameworkcode'} } = $IT;
+    }
+    return ( \%itemtypes );
 }
+
 =head2 getframeworkinfo
 
   $frameworkinfo = &getframeworkinfo($frameworkcode);
@@ -455,15 +416,15 @@ Returns information about an frameworkcode.
 =cut
 
 sub getframeworkinfo {
-       my ($frameworkcode) = @_;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from biblio_framework where frameworkcode=?");
-       $sth->execute($frameworkcode);
-       my $res = $sth->fetchrow_hashref;
-       return $res;
+    my ($frameworkcode) = @_;
+    my $dbh             = C4::Context->dbh;
+    my $sth             =
+      $dbh->prepare("select * from biblio_framework where frameworkcode=?");
+    $sth->execute($frameworkcode);
+    my $res = $sth->fetchrow_hashref;
+    return $res;
 }
 
-
 =head2 getitemtypeinfo
 
   $itemtype = &getitemtype($itemtype);
@@ -473,217 +434,89 @@ Returns information about an itemtype.
 =cut
 
 sub getitemtypeinfo {
-       my ($itemtype) = @_;
-       my $dbh = C4::Context->dbh;
-       my $sth=$dbh->prepare("select * from itemtypes where itemtype=?");
-       $sth->execute($itemtype);
-       my $res = $sth->fetchrow_hashref;
+    my ($itemtype) = @_;
+    my $dbh        = C4::Context->dbh;
+    my $sth        = $dbh->prepare("select * from itemtypes where itemtype=?");
+    $sth->execute($itemtype);
+    my $res = $sth->fetchrow_hashref;
 
-        $res->{imageurl} = getitemtypeimagesrcfromurl($res->{imageurl});
+    $res->{imageurl} = getitemtypeimagesrcfromurl( $res->{imageurl} );
 
-       return $res;
+    return $res;
 }
 
 sub getitemtypeimagesrcfromurl {
     my ($imageurl) = @_;
 
-    if (defined $imageurl and $imageurl !~ m/^http/) {
-        $imageurl =
-            getitemtypeimagesrc()
-            .'/'.$imageurl
-            ;
+    if ( defined $imageurl and $imageurl !~ m/^http/ ) {
+        $imageurl = getitemtypeimagesrc() . '/' . $imageurl;
     }
 
     return $imageurl;
 }
 
 sub getitemtypeimagedir {
-    return
-        C4::Context->intrahtdocs
-        .'/'.C4::Context->preference('template')
-        .'/itemtypeimg'
-        ;
+    return C4::Context->opachtdocs . '/'
+      . C4::Context->preference('template')
+      . '/itemtypeimg';
 }
 
 sub getitemtypeimagesrc {
-    return
-        '/intranet-tmpl'
-        .'/'.C4::Context->preference('template')
-        .'/itemtypeimg'
-        ;
+    return '/opac-tmpl' . '/'
+      . C4::Context->preference('template')
+      . '/itemtypeimg';
 }
 
-=head2 getprinters
+=head2 GetPrinters
 
-  $printers = &getprinters($env);
+  $printers = &GetPrinters();
   @queues = keys %$printers;
 
 Returns information about existing printer queues.
 
-C<$env> is ignored.
-
 C<$printers> is a reference-to-hash whose keys are the print queues
 defined in the printers table of the Koha database. The values are
 references-to-hash, whose keys are the fields in the printers table.
 
 =cut
 
-sub getprinters {
-    my ($env) = @_;
+sub GetPrinters {
     my %printers;
     my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare("select * from printers");
+    my $sth = $dbh->prepare("select * from printers");
     $sth->execute;
-    while (my $printer=$sth->fetchrow_hashref) {
-       $printers{$printer->{'printqueue'}}=$printer;
+    while ( my $printer = $sth->fetchrow_hashref ) {
+        $printers{ $printer->{'printqueue'} } = $printer;
     }
-    return (\%printers);
-}
-
-sub getbranch ($$) {
-    my($query, $branches) = @_; # get branch for this query from branches
-    my $branch = $query->param('branch');
-    ($branch) || ($branch = $query->cookie('branch'));
-    ($branches->{$branch}) || ($branch=(keys %$branches)[0]);
-    return $branch;
+    return ( \%printers );
 }
 
-=item getbranchdetail
-
-  $branchname = &getbranchdetail($branchcode);
+=head2 GetPrinter
 
-Given the branch code, the function returns the corresponding
-branch name for a comprehensive information display
+$printer = GetPrinter( $query, $printers );
 
 =cut
 
-sub getbranchdetail
-{
-       my ($branchcode) = @_;
-       my $dbh = C4::Context->dbh;
-       my $sth = $dbh->prepare("SELECT * FROM branches WHERE branchcode = ?");
-       $sth->execute($branchcode);
-       my $branchname = $sth->fetchrow_hashref();
-       $sth->finish();
-       return $branchname;
-} # sub getbranchname
-
-
-sub getprinter ($$) {
-    my($query, $printers) = @_; # get printer for this query from printers
+sub GetPrinter ($$) {
+    my ( $query, $printers ) = @_;    # get printer for this query from printers
     my $printer = $query->param('printer');
-    ($printer) || ($printer = $query->cookie('printer')) || ($printer='');
-    ($printers->{$printer}) || ($printer = (keys %$printers)[0]);
+    my %cookie = $query->cookie('userenv');
+    ($printer) || ( $printer = $cookie{'printer'} ) || ( $printer = '' );
+    ( $printers->{$printer} ) || ( $printer = ( keys %$printers )[0] );
     return $printer;
 }
 
-=item getalllanguages
-
-  (@languages) = &getalllanguages($type);
-  (@languages) = &getalllanguages($type,$theme);
+=item getnbpages
 
-Returns an array of all available languages.
+Returns the number of pages to display in a pagination bar, given the number
+of items and the number of items per page.
 
 =cut
 
-sub getalllanguages {
-       my $type=shift;
-       my $theme=shift;
-       my $htdocs;
-       my @languages;
-       if ($type eq 'opac') {
-               $htdocs=C4::Context->config('opachtdocs');
-               if ($theme and -d "$htdocs/$theme") {
-                       opendir D, "$htdocs/$theme";
-                       foreach my $language (readdir D) {
-                               next if $language=~/^\./;
-                               next if $language eq 'all';
-                               next if $language=~ /png$/;
-                               next if $language=~ /css$/;
-                               next if $language=~ /CVS$/;
-                               next if $language=~ /itemtypeimg$/;
-                               push @languages, $language;
-                       }
-                       return sort @languages;
-               } else {
-                       my $lang;
-                       foreach my $theme (getallthemes('opac')) {
-                               opendir D, "$htdocs/$theme";
-                               foreach my $language (readdir D) {
-                                       next if $language=~/^\./;
-                                       next if $language eq 'all';
-                                       next if $language=~ /png$/;
-                                       next if $language=~ /css$/;
-                                       next if $language=~ /CVS$/;
-                                       next if $language=~ /itemtypeimg$/;
-                                       $lang->{$language}=1;
-                               }
-                       }
-                       @languages=keys %$lang;
-                       return sort @languages;
-               }
-       } elsif ($type eq 'intranet') {
-               $htdocs=C4::Context->config('intrahtdocs');
-               if ($theme and -d "$htdocs/$theme") {
-                       opendir D, "$htdocs/$theme";
-                       foreach my $language (readdir D) {
-                               next if $language=~/^\./;
-                               next if $language eq 'all';
-                               next if $language=~ /png$/;
-                               next if $language=~ /css$/;
-                               next if $language=~ /CVS$/;
-                               next if $language=~ /itemtypeimg$/;
-                               push @languages, $language;
-                       }
-                       return sort @languages;
-               } else {
-                       my $lang;
-                       foreach my $theme (getallthemes('opac')) {
-                               opendir D, "$htdocs/$theme";
-                               foreach my $language (readdir D) {
-                                       next if $language=~/^\./;
-                                       next if $language eq 'all';
-                                       next if $language=~ /png$/;
-                                       next if $language=~ /css$/;
-                                       next if $language=~ /CVS$/;
-                                       next if $language=~ /itemtypeimg$/;
-                                       $lang->{$language}=1;
-                               }
-                       }
-                       @languages=keys %$lang;
-                       return sort @languages;
-               }
-    } else {
-               my $lang;
-               my $htdocs=C4::Context->config('intrahtdocs');
-               foreach my $theme (getallthemes('intranet')) {
-                       opendir D, "$htdocs/$theme";
-                       foreach my $language (readdir D) {
-                               next if $language=~/^\./;
-                               next if $language eq 'all';
-                               next if $language=~ /png$/;
-                               next if $language=~ /css$/;
-                               next if $language=~ /CVS$/;
-                               next if $language=~ /itemtypeimg$/;
-                               $lang->{$language}=1;
-                       }
-               }
-               $htdocs=C4::Context->config('opachtdocs');
-               foreach my $theme (getallthemes('opac')) {
-               opendir D, "$htdocs/$theme";
-               foreach my $language (readdir D) {
-                       next if $language=~/^\./;
-                       next if $language eq 'all';
-                       next if $language=~ /png$/;
-                       next if $language=~ /css$/;
-                       next if $language=~ /CVS$/;
-                       next if $language=~ /itemtypeimg$/;
-                       $lang->{$language}=1;
-                       }
-               }
-               @languages=keys %$lang;
-               return sort @languages;
-    }
+sub getnbpages {
+    my ( $nb_items, $nb_items_per_page ) = @_;
+
+    return int( ( $nb_items - 1 ) / $nb_items_per_page ) + 1;
 }
 
 =item getallthemes
@@ -696,133 +529,161 @@ Returns an array of all available themes.
 =cut
 
 sub getallthemes {
-    my $type=shift;
+    my $type = shift;
     my $htdocs;
     my @themes;
-    if ($type eq 'intranet') {
-       $htdocs=C4::Context->config('intrahtdocs');
-    } else {
-       $htdocs=C4::Context->config('opachtdocs');
+    if ( $type eq 'intranet' ) {
+        $htdocs = C4::Context->config('intrahtdocs');
+    }
+    else {
+        $htdocs = C4::Context->config('opachtdocs');
     }
     opendir D, "$htdocs";
-    my @dirlist=readdir D;
+    my @dirlist = readdir D;
     foreach my $directory (@dirlist) {
-       -d "$htdocs/$directory/en" and push @themes, $directory;
+        -d "$htdocs/$directory/en" and push @themes, $directory;
     }
     return @themes;
 }
 
-=item getnbpages
-
-Returns the number of pages to display in a pagination bar, given the number
-of items and the number of items per page.
-
-=cut
-
-sub getnbpages {
-    my ($nb_items, $nb_items_per_page) = @_;
-
-    return int(($nb_items - 1) / $nb_items_per_page) + 1;
-}
-
-
-=head2 getcities (OUEST-PROVENCE)
-
-  ($id_cityarrayref, $city_hashref) = &getcities();
-
-Looks up the different city and zip in the database. Returns two
-elements: a reference-to-array, which lists the zip city
-codes, and a reference-to-hash, which maps the name of the city.
-WHERE =>OUEST PROVENCE OR EXTERIEUR
-
-=cut
-sub getcities {
-    #my ($type_city) = @_;
-    my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare("Select cityid,city_name from cities order by cityid  ");
-    #$sth->execute($type_city);
-    $sth->execute();    
-    my %city;
-    my @id;
-#    insert empty value to create a empty choice in cgi popup 
-        
-while (my $data=$sth->fetchrow_hashref){
-      
-       push @id,$data->{'cityid'};
-      $city{$data->{'cityid'}}=$data->{'city_name'};
+sub getFacets {
+    my $facets;
+    if ( C4::Context->preference("marcflavour") eq "UNIMARC" ) {
+        $facets = [
+            {
+                link_value  => 'su-to',
+                label_value => 'Topics',
+                tags        =>
+                  [ '600', '601', '602', '603', '604', '605', '606', '610' ],
+                subfield => 'a',
+            },
+            {
+                link_value  => 'su-geo',
+                label_value => 'Places',
+                tags        => ['651'],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'su-ut',
+                label_value => 'Titles',
+                tags        => [ '500', '501', '502', '503', '504', ],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'au',
+                label_value => 'Authors',
+                tags        => [ '700', '701', '702', ],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'se',
+                label_value => 'Series',
+                tags        => ['225'],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'branch',
+                label_value => 'Libraries',
+                tags        => [ '995', ],
+                subfield    => 'b',
+                expanded    => '1',
+            },
+        ];
     }
-       
-       #test to know if the table contain some records if no the function return nothing
-       my $id=@id;
-       $sth->finish;
-       if ($id eq 0)
-       {
-       return();
-       }
-       else{
-       unshift (@id ,"");
-       return(\@id,\%city);
-       }
+    else {
+        $facets = [
+            {
+                link_value  => 'su-to',
+                label_value => 'Topics',
+                tags        => ['650'],
+                subfield    => 'a',
+            },
+
+            #        {
+            #        link_value => 'su-na',
+            #        label_value => 'People and Organizations',
+            #        tags => ['600', '610', '611'],
+            #        subfield => 'a',
+            #        },
+            {
+                link_value  => 'su-geo',
+                label_value => 'Places',
+                tags        => ['651'],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'su-ut',
+                label_value => 'Titles',
+                tags        => ['630'],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'au',
+                label_value => 'Authors',
+                tags        => [ '100', '110', '700', ],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'se',
+                label_value => 'Series',
+                tags        => [ '440', '490', ],
+                subfield    => 'a',
+            },
+            {
+                link_value  => 'branch',
+                label_value => 'Libraries',
+                tags        => [ '952', ],
+                subfield    => 'b',
+                expanded    => '1',
+            },
+        ];
+    }
+    return $facets;
 }
 
+=head2 get_infos_of
 
-=head2 getroadtypes (OUEST-PROVENCE)
+Return a href where a key is associated to a href. You give a query, the
+name of the key among the fields returned by the query. If you also give as
+third argument the name of the value, the function returns a href of scalar.
 
-  ($idroadtypearrayref, $roadttype_hashref) = &getroadtypes();
+  my $query = '
+SELECT itemnumber,
+       notforloan,
+       barcode
+  FROM items
+';
 
-Looks up the different road type . Returns two
-elements: a reference-to-array, which lists the id_roadtype
-codes, and a reference-to-hash, which maps the road type of the road .
+  # generic href of any information on the item, href of href.
+  my $iteminfos_of = get_infos_of($query, 'itemnumber');
+  print $iteminfos_of->{$itemnumber}{barcode};
 
+  # specific information, href of scalar
+  my $barcode_of_item = get_infos_of($query, 'itemnumber', 'barcode');
+  print $barcode_of_item->{$itemnumber};
 
 =cut
-sub getroadtypes {
-    my $dbh = C4::Context->dbh;
-    my $sth=$dbh->prepare("Select roadtypeid,road_type from roadtype order by road_type  ");
-    $sth->execute();
-    my %roadtype;
-    my @id;
-#    insert empty value to create a empty choice in cgi popup 
-while (my $data=$sth->fetchrow_hashref){
-       push @id,$data->{'roadtypeid'};
-      $roadtype{$data->{'roadtypeid'}}=$data->{'road_type'};
-    }
-       #test to know if the table contain some records if no the function return nothing
-       my $id=@id;
-       $sth->finish;
-       if ($id eq 0)
-       {
-       return();
-       }
-       else{
-               unshift (@id ,"");
-               return(\@id,\%roadtype);
-       }
-}
-
-=head2 get_branchinfos_of
 
-  my $branchinfos_of = get_branchinfos_of(@branchcodes);
-
-Associates a list of branchcodes to the information of the branch, taken in
-branches table.
+sub get_infos_of {
+    my ( $query, $key_name, $value_name ) = @_;
 
-Returns a href where keys are branchcodes and values are href where keys are
-branch information key.
+    my $dbh = C4::Context->dbh;
 
-  print 'branchname is ', $branchinfos_of->{$code}->{branchname};
+    my $sth = $dbh->prepare($query);
+    $sth->execute();
 
-=cut
-sub get_branchinfos_of {
-    my @branchcodes = @_;
+    my %infos_of;
+    while ( my $row = $sth->fetchrow_hashref ) {
+        if ( defined $value_name ) {
+            $infos_of{ $row->{$key_name} } = $row->{$value_name};
+        }
+        else {
+            $infos_of{ $row->{$key_name} } = $row;
+        }
+    }
+    $sth->finish;
 
-    my $query = '
-SELECT branchcode,
-       branchname
-  FROM branches
-  WHERE branchcode IN ('.join(',', map({"'".$_."'"} @branchcodes)).')
-';
-    return get_infos_of($query, 'branchcode');
+    return \%infos_of;
 }
 
 =head2 get_notforloan_label_of
@@ -844,6 +705,9 @@ labels.
   }
 
 =cut
+
+# FIXME - why not use GetAuthorisedValues ??
+#
 sub get_notforloan_label_of {
     my $dbh = C4::Context->dbh;
 
@@ -866,7 +730,7 @@ SELECT lib,
     $sth = $dbh->prepare($query);
     $sth->execute($statuscode);
     my %notforloan_label_of;
-    while (my $row = $sth->fetchrow_hashref) {
+    while ( my $row = $sth->fetchrow_hashref ) {
         $notforloan_label_of{ $row->{authorised_value} } = $row->{lib};
     }
     $sth->finish;
@@ -874,55 +738,227 @@ SELECT lib,
     return \%notforloan_label_of;
 }
 
-=head2 get_infos_of
+sub displayServers {
+    my ( $position, $type ) = @_;
+    my $dbh    = C4::Context->dbh;
+    my $strsth = "SELECT * FROM z3950servers where 1";
+    $strsth .= " AND position=\"$position\"" if ($position);
+    $strsth .= " AND type=\"$type\""         if ($type);
+    my $rq = $dbh->prepare($strsth);
+    $rq->execute;
+    my @primaryserverloop;
+
+    while ( my $data = $rq->fetchrow_hashref ) {
+        my %cell;
+        $cell{label} = $data->{'description'};
+        $cell{id}    = $data->{'name'};
+        $cell{value} =
+            $data->{host}
+          . ( $data->{port} ? ":" . $data->{port} : "" ) . "/"
+          . $data->{database}
+          if ( $data->{host} );
+        $cell{checked} = $data->{checked};
+        push @primaryserverloop,
+          {
+            label => $data->{description},
+            id    => $data->{name},
+            name  => "server",
+            value => $data->{host} . ":"
+              . $data->{port} . "/"
+              . $data->{database},
+            checked    => "checked",
+            icon       => $data->{icon},
+            zed        => $data->{type} eq 'zed',
+            opensearch => $data->{type} eq 'opensearch'
+          };
+    }
+    return \@primaryserverloop;
+}
 
-Return a href where a key is associated to a href. You give a query, the
-name of the key among the fields returned by the query. If you also give as
-third argument the name of the value, the function returns a href of scalar.
+sub displaySecondaryServers {
 
-  my $query = '
-SELECT itemnumber,
-       notforloan,
-       barcode
-  FROM items
-';
+#      my $secondary_servers_loop = [
+#              { inner_sup_servers_loop => [
+#              {label => "Google", id=>"GOOG", value=>"google",icon => "google.ico",opensearch => "1"},
+#              {label => "Yahoo", id=>"YAH", value=>"yahoo", icon =>"yahoo.ico", zed => "1"},
+#              {label => "Worldcat", id=>"WCT", value=>"worldcat", icon => "worldcat.gif", zed => "1"},
+#              {label => "Library of Congress", id=>"LOC", name=> "server", value=>"z3950.loc.gov:7090/Voyager", icon =>"loc.ico", zed => "1"},
+#      ],
+#      },
+#      ];
+    return;    #$secondary_servers_loop;
+}
 
-  # generic href of any information on the item, href of href.
-  my $iteminfos_of = get_infos_of($query, 'itemnumber');
-  print $iteminfos_of->{$itemnumber}{barcode};
+=head2 GetAuthValCode
 
-  # specific information, href of scalar
-  my $barcode_of_item = get_infos_of($query, 'itemnumber', 'barcode');
-  print $barcode_of_item->{$itemnumber};
+$authvalcode = GetAuthValCode($kohafield,$frameworkcode);
 
 =cut
-sub get_infos_of {
-    my ($query, $key_name, $value_name) = @_;
 
-    my $dbh = C4::Context->dbh;
+sub GetAuthValCode {
+       my ($kohafield,$fwcode) = @_;
+       my $dbh = C4::Context->dbh;
+       $fwcode='' unless $fwcode;
+       my $sth = $dbh->prepare('select authorised_value from marc_subfield_structure where kohafield=? and frameworkcode=?');
+       $sth->execute($kohafield,$fwcode);
+       my ($authvalcode) = $sth->fetchrow_array;
+       return $authvalcode;
+}
+
+=head2 GetAuthorisedValues
+
+$authvalues = GetAuthorisedValues($category);
+
+this function get all authorised values from 'authosied_value' table into a reference to array which
+each value containt an hashref.
+
+Set C<$category> on input args if you want to limits your query to this one. This params is not mandatory.
+
+=cut
+
+sub GetAuthorisedValues {
+    my ($category,$selected) = @_;
+       my $count = 0;
+       my @results;
+    my $dbh      = C4::Context->dbh;
+    my $query    = "SELECT * FROM authorised_values";
+    $query .= " WHERE category = '" . $category . "'" if $category;
 
     my $sth = $dbh->prepare($query);
-    $sth->execute();
+    $sth->execute;
+       while (my $data=$sth->fetchrow_hashref) {
+               if ($selected eq $data->{'authorised_value'} ) {
+                       $data->{'selected'} = 1;
+               }
+               $results[$count] = $data;
+               $count++;
+       }
+    #my $data = $sth->fetchall_arrayref({});
+    return \@results; #$data;
+}
 
-    my %infos_of;
-    while (my $row = $sth->fetchrow_hashref) {
-        if (defined $value_name) {
-            $infos_of{ $row->{$key_name} } = $row->{$value_name};
+=item fixEncoding
+
+  $marcrecord = &fixEncoding($marcblob);
+
+Returns a well encoded marcrecord.
+
+=cut
+sub FixEncoding {
+  my $marc=shift;
+  my $record = MARC::Record->new_from_usmarc($marc);
+  if (C4::Context->preference("MARCFLAVOUR") eq "UNIMARC"){
+    use Encode::Guess;
+    my $targetcharset="utf8" if (C4::Context->preference("TemplateEncoding") eq "utf-8");
+    $targetcharset="latin1" if (C4::Context->preference("TemplateEncoding") eq "iso-8859-1");
+    my $decoder = guess_encoding($marc, qw/utf8 latin1/);
+#     die $decoder unless ref($decoder);
+    if (ref($decoder)) {
+        my $newRecord=MARC::Record->new();
+        foreach my $field ($record->fields()){
+        if ($field->tag()<'010'){
+            $newRecord->insert_grouped_field($field);
+        } else {
+            my $newField;
+            my $createdfield=0;
+            foreach my $subfield ($field->subfields()){
+            if ($createdfield){
+                if (($newField->tag eq '100')) {
+                    substr($subfield->[1],26,2,"0103") if ($targetcharset eq "latin1");
+                    substr($subfield->[1],26,4,"5050") if ($targetcharset eq "utf8");
+                }
+                map {C4::Biblio::char_decode($_,"UNIMARC")} @$subfield;
+                $newField->add_subfields($subfield->[0]=>$subfield->[1]);
+            } else {
+                map {C4::Biblio::char_decode($_,"UNIMARC")} @$subfield;
+                $newField=MARC::Field->new($field->tag(),$field->indicator(1),$field->indicator(2),$subfield->[0]=>$subfield->[1]);
+                $createdfield=1;
+            }
+            }
+            $newRecord->insert_grouped_field($newField);
         }
-        else {
-            $infos_of{ $row->{$key_name} } = $row;
         }
+    #     warn $newRecord->as_formatted(); 
+        return $newRecord;
+    } else {
+        return $record;
     }
-    $sth->finish;
+  } else {
+    return $record;
+  }
+}
 
-    return \%infos_of;
+=head2 GetKohaAuthorisedValues
+       
+       Takes $dbh , $kohafield as parameters.
+       returns hashref of authvalCode => liblibrarian
+       or undef if no authvals defined for kohafield.
+
+=cut
+
+sub GetKohaAuthorisedValues {
+  my ($kohafield,$fwcode) = @_;
+  $fwcode='' unless $fwcode;
+  my %values;
+  my $dbh = C4::Context->dbh;
+  my $avcode = GetAuthValCode($kohafield,$fwcode);
+  if ($avcode) {  
+    my $sth = $dbh->prepare("select authorised_value, lib from authorised_values where category=? ");
+    $sth->execute($avcode);
+       while ( my ($val, $lib) = $sth->fetchrow_array ) { 
+               $values{$val}= $lib;
+       }
+  }
+  return \%values;
 }
 
-1;
-__END__
+=head2 GetManagedTagSubfields
+
+=over 4
+
+$res = GetManagedTagSubfields();
 
 =back
 
+Returns a reference to a big hash of hash, with the Marc structure fro the given frameworkcode
+
+NOTE: This function is used only by the (incomplete) bulk editing feature.  Since
+that feature currently does not deal with items and biblioitems changes 
+correctly, those tags are specifically excluded from the list prepared
+by this function.
+
+For future reference, if a bulk item editing feature is implemented at some point, it
+needs some design thought -- for example, circulation status fields should not 
+be changed willy-nilly.
+
+=cut
+
+sub GetManagedTagSubfields{
+  my $dbh=C4::Context->dbh;
+  my $rq=$dbh->prepare(qq|
+SELECT 
+  DISTINCT CONCAT( marc_subfield_structure.tagfield, tagsubfield ) AS tagsubfield, 
+  marc_subfield_structure.liblibrarian as subfielddesc, 
+  marc_tag_structure.liblibrarian as tagdesc
+FROM marc_subfield_structure
+  LEFT JOIN marc_tag_structure 
+    ON marc_tag_structure.tagfield = marc_subfield_structure.tagfield
+    AND marc_tag_structure.frameworkcode = marc_subfield_structure.frameworkcode
+WHERE marc_subfield_structure.tab>=0
+AND marc_tag_structure.tagfield NOT IN (SELECT tagfield FROM marc_subfield_structure WHERE kohafield like 'items.%')
+AND marc_tag_structure.tagfield NOT IN (SELECT tagfield FROM marc_subfield_structure WHERE kohafield = 'biblioitems.itemtype')
+AND marc_subfield_structure.kohafield <> 'biblio.biblionumber'
+AND marc_subfield_structure.kohafield <>  'biblioitems.biblioitemnumber'
+ORDER BY marc_subfield_structure.tagfield, tagsubfield|);
+  $rq->execute;
+  my $data=$rq->fetchall_arrayref({});
+  return $data;
+}
+
+1;
+
+__END__
+
 =head1 AUTHOR
 
 Koha Team