renaming currenttransfers to transferstoreceive
[koha.git] / C4 / Members.pm
index a2ca438..f628e48 100644 (file)
@@ -1,5 +1,3 @@
-# -*- tab-width: 8 -*-
-
 package C4::Members;
 
 # Copyright 2000-2003 Katipo Communications
@@ -24,10 +22,12 @@ package C4::Members;
 use strict;
 require Exporter;
 use C4::Context;
-use Date::Manip;
 use C4::Date;
 use Digest::MD5 qw(md5_base64);
-use Date::Calc qw/Today/;
+use Date::Calc qw/Today Add_Delta_YM/;
+use C4::Log; # logaction
+use C4::Overdues;
+use C4::Reserves;
 
 use vars qw($VERSION @ISA @EXPORT @EXPORT_OK);
 
@@ -51,31 +51,79 @@ This module contains routines for adding, modifying and deleting members/patrons
 
 =cut
 
-#'
+@ISA = qw(Exporter);
+
+#Get data
+push @EXPORT, qw(
+  &SearchBorrower 
+  &GetMemberDetails
+  &GetMember
+  
+  &GetGuarantees 
+  &findguarantor 
+  &GuarantornameSearch
+  
+  &GetBorrowerIssuesAndFines
+  &GetPendingIssues
+  &GetAllIssues
+  
+  &get_institutions 
+  &getzipnamecity 
+  &getidcity
+   
+  &GetAge 
+  &GetCities 
+  &GetRoadTypes 
+  &GetRoadTypeDetails 
+  
+  &GetBorrowerAcctRecord
+  
+  &GetborCatFromCatType 
+  &GetBorrowercategory
+  
+  &GetBorNotifyAcctRecord
+  &GetMembeReregistration
+  &GetSortDetails
+  
+  &GetBorrowersTitles  
+  &GetBorrowersWhoHaveNotBorrowedSince
+  &GetBorrowersWhoHaveNeverBorrowed
+  &GetBorrowersWithIssuesHistoryOlderThan
+  &GetBorrowersFromSurname 
+  
+  &GetExpiryDate
+);
 
-@ISA    = qw(Exporter);
-@EXPORT = qw();
+#Modify data
+push @EXPORT, qw(
+  &ModMember
+  &fixup_cardnumber
+  &changepassword
+);
+  
+#Delete data
+push @EXPORT, qw(
+  &DelMember
+);
 
-@EXPORT = qw(
- &BornameSearch &getmember &borrdata &borrdata2 &fixup_cardnumber &findguarantees &findguarantor &GuarantornameSearch &NewBorrowerNumber   &modmember &newmember &changepassword &borrissues &allissues
-  &checkuniquemember &getzipnamecity &getidcity &getguarantordata &getcategorytype
-  &calcexpirydate &checkuserpassword
-  &getboracctrecord
-  &borrowercategories &getborrowercategory
+#Insert data
+push @EXPORT, qw(
+  &AddMember  
+  &checkuniquemember 
+  &checkuserpassword
   &fixEthnicity
-  &ethnicitycategories get_institutions add_member_orgs
-  &get_age
+  &ethnicitycategories 
+  &add_member_orgs
+  
+  &MoveMemberToDeleted
 );
 
+=item Searchborrower
 
-=item BornameSearch
-
-  ($count, $borrowers) = &BornameSearch($env, $searchstring, $type);
+  ($count, $borrowers) = &SearchBorrower($searchstring, $type);
 
 Looks up patrons (borrowers) by name.
 
-C<$env> is ignored.
-
 BUGFIX 499: C<$type> is now used to determine type of search.
 if $type is "simple", search is performed on the first letter of the
 surname only.
@@ -84,7 +132,7 @@ C<$searchstring> is a space-separated list of search terms. Each term
 must match the beginning a borrower's surname, first name, or other
 name.
 
-C<&BornameSearch> returns a two-element list. C<$borrowers> is a
+C<&SearchBorrower> returns a two-element list. C<$borrowers> is a
 reference-to-array; each element is a reference-to-hash, whose keys
 are the fields of the C<borrowers> table in the Koha database.
 C<$count> is the number of elements in C<$borrowers>.
@@ -94,8 +142,8 @@ C<$count> is the number of elements in C<$borrowers>.
 #'
 #used by member enquiries from the intranet
 #called by member.pl
-sub BornameSearch {
-    my ( $env, $searchstring, $orderby, $type ) = @_;
+sub SearchBorrower {
+    my ($searchstring, $orderby, $type,$category_type ) = @_;
     my $dbh   = C4::Context->dbh;
     my $query = "";
     my $count;
@@ -105,33 +153,38 @@ sub BornameSearch {
     if ( $type eq "simple" )    # simple search for one letter only
     {
         $query =
-          "Select * from borrowers where surname like ? order by $orderby";
+          "SELECT * FROM borrowers
+                  LEFT JOIN categories ON borrowers.categorycode=categories.categorycode ".
+                  ($category_type?" AND category_type = ".$dbh->quote($category_type):"").
+                  " WHERE surname LIKE ? ORDER BY $orderby";
         @bind = ("$searchstring%");
     }
     else    # advanced search looking in surname, firstname and othernames
     {
         @data  = split( ' ', $searchstring );
         $count = @data;
-        $query = "Select * from borrowers
-               where ((surname like ? or surname like ?
-               or firstname  like ? or firstname like ?
-               or othernames like ? or othernames like ?)
-               ";
+        $query = "SELECT * FROM borrowers
+                    LEFT JOIN categories ON borrowers.categorycode=categories.categorycode
+               WHERE ((surname LIKE ? OR surname LIKE ?
+               OR firstname  LIKE ? OR firstname LIKE ?
+               OR othernames LIKE ? OR othernames LIKE ?)
+               ".
+                  ($category_type?" AND category_type = ".$dbh->quote($category_type):"");
         @bind = (
             "$data[0]%", "% $data[0]%", "$data[0]%", "% $data[0]%",
             "$data[0]%", "% $data[0]%"
         );
         for ( my $i = 1 ; $i < $count ; $i++ ) {
-            $query = $query . " and (" . " surname like ? or surname like ?
-                        or firstname  like ? or firstname like ?
-                       or othernames like ? or othernames like ?)";
+            $query = $query . " AND (" . " surname LIKE ? OR surname LIKE ?
+                        OR firstname  LIKE ? OR firstname LIKE ?
+                       OR othernames LIKE ? OR othernames LIKE ?)";
             push( @bind,
                 "$data[$i]%",   "% $data[$i]%", "$data[$i]%",
                 "% $data[$i]%", "$data[$i]%",   "% $data[$i]%" );
 
             # FIXME - .= <<EOT;
         }
-        $query = $query . ") or cardnumber like ?
+        $query = $query . ") OR cardnumber LIKE ?
                order by $orderby";
         push( @bind, $searchstring );
 
@@ -153,115 +206,318 @@ sub BornameSearch {
     return ( $cnt, \@results );
 }
 
-=item getmember
+=head2 GetMemberDetails
 
-  $borrower = &getmember($cardnumber, $borrowernumber);
+($borrower, $flags) = &GetMemberDetails($borrowernumber, $cardnumber);
 
-Looks up information about a patron (borrower) by either card number
-or borrower number. If $borrowernumber is specified, C<&borrdata>
-searches by borrower number; otherwise, it searches by card number.
+Looks up a patron and returns information about him or her. If
+C<$borrowernumber> is true (nonzero), C<&GetMemberDetails> looks
+up the borrower by number; otherwise, it looks up the borrower by card
+number.
 
-C<&getmember> returns a reference-to-hash whose keys are the fields of
-the C<borrowers> table in the Koha database.
+C<$borrower> is a reference-to-hash whose keys are the fields of the
+borrowers table in the Koha database. In addition,
+C<$borrower-E<gt>{flags}> is a hash giving more detailed information
+about the patron. Its keys act as flags :
+
+    if $borrower->{flags}->{LOST} {
+        # Patron's card was reported lost
+    }
+
+Each flag has a C<message> key, giving a human-readable explanation of
+the flag. If the state of a flag means that the patron should not be
+allowed to borrow any more books, then it will have a C<noissues> key
+with a true value.
+
+The possible flags are:
+
+=head3 CHARGES
+
+=over 4
+
+=item Shows the patron's credit or debt, if any.
+
+=back
+
+=head3 GNA
+
+=over 4
+
+=item (Gone, no address.) Set if the patron has left without giving a
+forwarding address.
+
+=back
+
+=head3 LOST
+
+=over 4
+
+=item Set if the patron's card has been reported as lost.
+
+=back
+
+=head3 DBARRED
+
+=over 4
+
+=item Set if the patron has been debarred.
+
+=back
+
+=head3 NOTES
+
+=over 4
+
+=item Any additional notes about the patron.
+
+=back
+
+=head3 ODUES
+
+=over 4
+
+=item Set if the patron has overdue items. This flag has several keys:
+
+C<$flags-E<gt>{ODUES}{itemlist}> is a reference-to-array listing the
+overdue items. Its elements are references-to-hash, each describing an
+overdue item. The keys are selected fields from the issues, biblio,
+biblioitems, and items tables of the Koha database.
+
+C<$flags-E<gt>{ODUES}{itemlist}> is a string giving a text listing of
+the overdue items, one per line.
+
+=back
+
+=head3 WAITING
+
+=over 4
+
+=item Set if any items that the patron has reserved are available.
+
+C<$flags-E<gt>{WAITING}{itemlist}> is a reference-to-array listing the
+available items. Each element is a reference-to-hash whose keys are
+fields from the reserves table of the Koha database.
+
+=back
 
 =cut
 
-#'
-sub getmember {
-    my ( $cardnumber, $bornum ) = @_;
-    $cardnumber = uc $cardnumber;
+sub GetMemberDetails {
+    my ( $borrowernumber, $cardnumber ) = @_;
     my $dbh = C4::Context->dbh;
+    my $query;
     my $sth;
-    if ( $bornum eq '' ) {
-        $sth = $dbh->prepare("Select * from borrowers where cardnumber=?");
+    if ($borrowernumber) {
+        $sth = $dbh->prepare("select * from borrowers where borrowernumber=?");
+        $sth->execute($borrowernumber);
+    }
+    elsif ($cardnumber) {
+        $sth = $dbh->prepare("select * from borrowers where cardnumber=?");
         $sth->execute($cardnumber);
     }
     else {
-        $sth = $dbh->prepare("Select * from borrowers where borrowernumber=?");
-        $sth->execute($bornum);
+        return undef;
+    }
+    my $borrower = $sth->fetchrow_hashref;
+    my ($amount) = GetBorrowerAcctRecord( $borrowernumber);
+    $borrower->{'amountoutstanding'} = $amount;
+    my $flags = patronflags( $borrower, $dbh );
+    my $accessflagshash;
+
+    $sth = $dbh->prepare("select bit,flag from userflags");
+    $sth->execute;
+    while ( my ( $bit, $flag ) = $sth->fetchrow ) {
+        if ( $borrower->{'flags'} && $borrower->{'flags'} & 2**$bit ) {
+            $accessflagshash->{$flag} = 1;
+        }
     }
-    my $data = $sth->fetchrow_hashref;
     $sth->finish;
-    if ($data) {
-        return ($data);
+    $borrower->{'flags'}     = $flags;
+    $borrower->{'authflags'} = $accessflagshash;
+
+    # find out how long the membership lasts
+    $sth =
+      $dbh->prepare(
+        "select enrolmentperiod from categories where categorycode = ?");
+    $sth->execute( $borrower->{'categorycode'} );
+    my $enrolment = $sth->fetchrow;
+    $borrower->{'enrolmentperiod'} = $enrolment;
+    return ($borrower);    #, $flags, $accessflagshash);
+}
+
+=head2 patronflags
+
+ Not exported
+
+ NOTE!: If you change this function, be sure to update the POD for
+ &GetMemberDetails.
+
+ $flags = &patronflags($patron, $dbh);
+
+ $flags->{CHARGES}
+        {message}    Message showing patron's credit or debt
+       {noissues}    Set if patron owes >$5.00
+         {GNA}            Set if patron gone w/o address
+        {message}    "Borrower has no valid address"
+        {noissues}    Set.
+        {LOST}        Set if patron's card reported lost
+        {message}    Message to this effect
+        {noissues}    Set.
+        {DBARRED}        Set is patron is debarred
+        {message}    Message to this effect
+        {noissues}    Set.
+         {NOTES}        Set if patron has notes
+        {message}    Notes about patron
+         {ODUES}        Set if patron has overdue books
+        {message}    "Yes"
+        {itemlist}    ref-to-array: list of overdue books
+        {itemlisttext}    Text list of overdue items
+         {WAITING}        Set if there are items available that the
+                patron reserved
+        {message}    Message to this effect
+        {itemlist}    ref-to-array: list of available items
+
+=cut
+
+sub patronflags {
+    my %flags;
+    my ( $patroninformation) = @_;
+    my $dbh=C4::Context->dbh;
+    my ($amount) = GetBorrowerAcctRecord( $patroninformation->{'borrowernumber'});
+    if ( $amount > 0 ) {
+        my %flaginfo;
+        my $noissuescharge = C4::Context->preference("noissuescharge");
+        $flaginfo{'message'} = sprintf "Patron owes \$%.02f", $amount;
+        if ( $amount > $noissuescharge ) {
+            $flaginfo{'noissues'} = 1;
+        }
+        $flags{'CHARGES'} = \%flaginfo;
     }
-    else {    # try with firstname
-        if ($cardnumber) {
-            my $sth =
-              $dbh->prepare("select * from borrowers where firstname=?");
-            $sth->execute($cardnumber);
-            my $data = $sth->fetchrow_hashref;
-            $sth->finish;
-            return ($data);
+    elsif ( $amount < 0 ) {
+        my %flaginfo;
+        $flaginfo{'message'} = sprintf "Patron has credit of \$%.02f", -$amount;
+        $flags{'CHARGES'} = \%flaginfo;
+    }
+    if (   $patroninformation->{'gonenoaddress'}
+        && $patroninformation->{'gonenoaddress'} == 1 )
+    {
+        my %flaginfo;
+        $flaginfo{'message'}  = 'Borrower has no valid address.';
+        $flaginfo{'noissues'} = 1;
+        $flags{'GNA'}         = \%flaginfo;
+    }
+    if ( $patroninformation->{'lost'} && $patroninformation->{'lost'} == 1 ) {
+        my %flaginfo;
+        $flaginfo{'message'}  = 'Borrower\'s card reported lost.';
+        $flaginfo{'noissues'} = 1;
+        $flags{'LOST'}        = \%flaginfo;
+    }
+    if (   $patroninformation->{'debarred'}
+        && $patroninformation->{'debarred'} == 1 )
+    {
+        my %flaginfo;
+        $flaginfo{'message'}  = 'Borrower is Debarred.';
+        $flaginfo{'noissues'} = 1;
+        $flags{'DBARRED'}     = \%flaginfo;
+    }
+    if (   $patroninformation->{'borrowernotes'}
+        && $patroninformation->{'borrowernotes'} )
+    {
+        my %flaginfo;
+        $flaginfo{'message'} = "$patroninformation->{'borrowernotes'}";
+        $flags{'NOTES'}      = \%flaginfo;
+    }
+    my ( $odues, $itemsoverdue ) =
+      checkoverdues( $patroninformation->{'borrowernumber'}, $dbh );
+    if ( $odues > 0 ) {
+        my %flaginfo;
+        $flaginfo{'message'}  = "Yes";
+        $flaginfo{'itemlist'} = $itemsoverdue;
+        foreach ( sort { $a->{'date_due'} cmp $b->{'date_due'} }
+            @$itemsoverdue )
+        {
+            $flaginfo{'itemlisttext'} .=
+              "$_->{'date_due'} $_->{'barcode'} $_->{'title'} \n";
         }
+        $flags{'ODUES'} = \%flaginfo;
     }
-    return undef;
+    my $itemswaiting =
+      GetWaitingReserves( $patroninformation->{'borrowernumber'} );
+    my $nowaiting = scalar @$itemswaiting;
+    if ( $nowaiting > 0 ) {
+        my %flaginfo;
+        $flaginfo{'message'}  = "Reserved items available";
+        $flaginfo{'itemlist'} = $itemswaiting;
+        $flags{'WAITING'}     = \%flaginfo;
+    }
+    return ( \%flags );
 }
 
-=item borrdata
 
-  $borrower = &borrdata($cardnumber, $borrowernumber);
+=item GetMember
+
+  $borrower = &GetMember($information, $type);
 
 Looks up information about a patron (borrower) by either card number
-or borrower number. If $borrowernumber is specified, C<&borrdata>
-searches by borrower number; otherwise, it searches by card number.
+,firstname, or borrower number, depending on $type value.
+If C<$type> == 'cardnumber', C<&GetBorrower>
+searches by cardnumber then by firstname if not found in cardnumber; 
+otherwise, it searches by borrowernumber.
 
-C<&borrdata> returns a reference-to-hash whose keys are the fields of
+C<&GetBorrower> returns a reference-to-hash whose keys are the fields of
 the C<borrowers> table in the Koha database.
 
 =cut
 
 #'
-sub borrdata {
-    my ( $cardnumber, $bornum ) = @_;
-    $cardnumber = uc $cardnumber;
+sub GetMember {
+    my ( $information, $type ) = @_;
     my $dbh = C4::Context->dbh;
     my $sth;
-    if ( $bornum eq '' ) {
-        $sth =
+    if ($type eq 'cardnumber' || $type eq 'firstname'|| $type eq 'userid'|| $type eq 'borrowernumber'){
+      $information = uc $information;
+      $sth =
           $dbh->prepare(
-"Select borrowers.*,categories.category_type from borrowers left join categories on borrowers.categorycode=categories.categorycode where cardnumber=?"
+"Select borrowers.*,categories.category_type,categories.description  from borrowers left join categories on borrowers.categorycode=categories.categorycode where $type=?"
           );
-        $sth->execute($cardnumber);
+        $sth->execute($information);
     }
     else {
         $sth =
           $dbh->prepare(
-"Select borrowers.*,categories.category_type from borrowers left join categories on borrowers.categorycode=categories.categorycode where borrowernumber=?"
+"Select borrowers.*,categories.category_type, categories.description from borrowers left join categories on borrowers.categorycode=categories.categorycode where borrowernumber=?"
           );
-        $sth->execute($bornum);
+        $sth->execute($information);
     }
     my $data = $sth->fetchrow_hashref;
-#     warn "DATA" . $data->{category_type};
+
     $sth->finish;
     if ($data) {
         return ($data);
     }
-    else {    # try with firstname
-        if ($cardnumber) {
-            my $sth =
+    elsif ($type eq 'cardnumber' ||$type eq 'firstname') {    # try with firstname
+        my $sth =
               $dbh->prepare(
-"Select borrowers.*,categories.category_type from borrowers left join categories on borrowers.categorycode=categories.categorycode  where firstname=?"
-              );
-            $sth->execute($cardnumber);
+"Select borrowers.*,categories.category_type,categories.description from borrowers left join categories on borrowers.categorycode=categories.categorycode  where firstname like ?"
+            );
+            $sth->execute($information);
             my $data = $sth->fetchrow_hashref;
             $sth->finish;
             return ($data);
-        }
     }
-    return undef;
+    else {
+        return undef;        
+    }
 }
 
-=item borrdata2
+=item GetBorrowerIssuesAndFines
 
-  ($borrowed, $due, $fine) = &borrdata2($env, $borrowernumber);
+  ($borrowed, $due, $fine) = &GetBorrowerIssuesAndFines($borrowernumber);
 
 Returns aggregate data about items borrowed by the patron with the
 given borrowernumber.
 
-C<$env> is ignored.
-
-C<&borrdata2> returns a three-element array. C<$borrowed> is the
+C<&GetBorrowerIssuesAndFines> returns a three-element array. C<$borrowed> is the
 number of books the patron currently has borrowed. C<$due> is the
 number of overdue items the patron currently has borrowed. C<$fine> is
 the total fine currently due by the borrower.
@@ -269,10 +525,11 @@ the total fine currently due by the borrower.
 =cut
 
 #'
-sub borrdata2 {
-    my ( $env, $bornum ) = @_;
+sub GetBorrowerIssuesAndFines {
+    my ( $borrowernumber ) = @_;
     my $dbh   = C4::Context->dbh;
-    my $query = "Select count(*) from issues where borrowernumber='$bornum' and
+    my $query =
+      "Select count(*) from issues where borrowernumber='$borrowernumber' and
     returndate is NULL";
 
     # print $query;
@@ -282,14 +539,14 @@ sub borrdata2 {
     $sth->finish;
     $sth = $dbh->prepare(
         "Select count(*) from issues where
-    borrowernumber='$bornum' and date_due < now() and returndate is NULL"
+    borrowernumber='$borrowernumber' and date_due < now() and returndate is NULL"
     );
     $sth->execute;
     my $data2 = $sth->fetchrow_hashref;
     $sth->finish;
     $sth = $dbh->prepare(
         "Select sum(amountoutstanding) from accountlines where
-    borrowernumber='$bornum'"
+    borrowernumber='$borrowernumber'"
     );
     $sth->execute;
     my $data3 = $sth->fetchrow_hashref;
@@ -299,119 +556,144 @@ sub borrdata2 {
         $data3->{'sum(amountoutstanding)'} );
 }
 
-sub modmember {
-       my (%data) = @_;
-       my $dbh = C4::Context->dbh;
-       $data{'dateofbirth'}=format_date_in_iso($data{'dateofbirth'});
-       $data{'dateexpiry'}=format_date_in_iso($data{'dateexpiry'});
-       $data{'dateenrolled'}=format_date_in_iso($data{'dateenrolled'});
-       warn "la date:".$data{dateenrolled};
-       my $query;
-       my $sth;
-       $data{'userid'}='' if ($data{'password'}eq '');
-       # test to know if u must update or not the borrower password
-       if ($data{'password'} eq '****'){
-               
-               $query="UPDATE borrowers SET 
+=head2
+
+=item ModMember
+
+  ($borrowed, $due, $fine) = &ModMember($borrowernumber);
+
+Modify borrower's data
+
+=cut
+
+#'
+sub ModMember {
+    my (%data) = @_;
+    my $dbh = C4::Context->dbh;
+    $data{'dateofbirth'}  = format_date_in_iso( $data{'dateofbirth'} );
+    $data{'dateexpiry'}   = format_date_in_iso( $data{'dateexpiry'} );
+    $data{'dateenrolled'} = format_date_in_iso( $data{'dateenrolled'} );
+
+    #          warn "num user".$data{'borrowernumber'};
+    my $query;
+    my $sth;
+    $data{'userid'} = '' if ( $data{'password'} eq '' );
+
+    # test to know if u must update or not the borrower password
+    if ( $data{'password'} eq '****' ) {
+
+        $query = "UPDATE borrowers SET 
                cardnumber  = ?,surname = ?,firstname = ?,title = ?,othernames = ?,initials = ?,
                streetnumber = ?,streettype = ?,address = ?,address2 = ?,city = ?,zipcode = ?,
                email = ?,phone = ?,mobile = ?,fax = ?,emailpro = ?,phonepro = ?,B_streetnumber = ?,
                B_streettype = ?,B_address = ?,B_city = ?,B_zipcode = ?,B_email = ?,B_phone = ?,dateofbirth = ?,branchcode = ?,
                categorycode = ?,dateenrolled = ?,dateexpiry = ?,gonenoaddress = ?,lost = ?,debarred = ?,contactname = ?,
                contactfirstname = ?,contacttitle = ?,guarantorid = ?,borrowernotes = ?,relationship =  ?,ethnicity = ?,
-               ethnotes = ?,sex = ?,flags = ?,userid = ?,opacnote = ?,contactnote = ?,sort1 = ?,sort2 = ? 
+               ethnotes = ?,sex = ?,userid = ?,opacnote = ?,contactnote = ?,sort1 = ?,sort2 = ? 
                WHERE borrowernumber=$data{'borrowernumber'}";
-       $sth=$dbh->prepare($query);
-       $sth->execute(
-               $data{'cardnumber'},$data{'surname'},
-               $data{'firstname'},$data{'title'},
-               $data{'othernames'},$data{'initials'},
-               $data{'streetnumber'},$data{'streettype'},
-               $data{'address'},$data{'address2'},
-               $data{'city'},$data{'zipcode'},
-               $data{'email'},$data{'phone'},
-               $data{'mobile'},$data{'fax'},
-               $data{'emailpro'},$data{'phonepro'},
-               $data{'B_streetnumber'},$data{'B_streettype'},
-               $data{'B_address'},$data{'B_city'},
-               $data{'B_zipcode'},$data{'B_email'},$data{'B_phone'},
-               $data{'dateofbirth'},$data{'branchcode'},
-               $data{'categorycode'},$data{'dateenrolled'},
-               $data{'dateexpiry'},$data{'gonenoaddress'},
-               $data{'lost'},$data{'debarred'},
-               $data{'contactname'},$data{'contactfirstname'},
-               $data{'contacttitle'},$data{'guarantorid'},
-               $data{'borrowernotes'},$data{'relationship'},
-               $data{'ethnicity'},$data{'ethnotes'},
-               $data{'sex'},
-               $data{'flags'},$data{'userid'},
-               $data{'opacnote'},$data{'contactnote'},
-               $data{'sort1'},$data{'sort2'});
-       }
-       else{
-               
-               ($data{'password'}=md5_base64($data{'password'})) if ($data{'password'} ne '');
-               $query="UPDATE borrowers SET 
+        $sth = $dbh->prepare($query);
+        $sth->execute(
+            $data{'cardnumber'},       $data{'surname'},
+            $data{'firstname'},        $data{'title'},
+            $data{'othernames'},       $data{'initials'},
+            $data{'streetnumber'},     $data{'streettype'},
+            $data{'address'},          $data{'address2'},
+            $data{'city'},             $data{'zipcode'},
+            $data{'email'},            $data{'phone'},
+            $data{'mobile'},           $data{'fax'},
+            $data{'emailpro'},         $data{'phonepro'},
+            $data{'B_streetnumber'},   $data{'B_streettype'},
+            $data{'B_address'},        $data{'B_city'},
+            $data{'B_zipcode'},        $data{'B_email'},
+            $data{'B_phone'},          $data{'dateofbirth'},
+            $data{'branchcode'},       $data{'categorycode'},
+            $data{'dateenrolled'},     $data{'dateexpiry'},
+            $data{'gonenoaddress'},    $data{'lost'},
+            $data{'debarred'},         $data{'contactname'},
+            $data{'contactfirstname'}, $data{'contacttitle'},
+            $data{'guarantorid'},      $data{'borrowernotes'},
+            $data{'relationship'},     $data{'ethnicity'},
+            $data{'ethnotes'},         $data{'sex'},
+            $data{'userid'},           $data{'opacnote'},
+            $data{'contactnote'},      $data{'sort1'},
+            $data{'sort2'}
+        );
+    }
+    else {
+
+        ( $data{'password'} = md5_base64( $data{'password'} ) )
+          if ( $data{'password'} ne '' );
+        $query = "UPDATE borrowers SET 
                cardnumber  = ?,surname = ?,firstname = ?,title = ?,othernames = ?,initials = ?,
                streetnumber = ?,streettype = ?,address = ?,address2 = ?,city = ?,zipcode = ?,
                email = ?,phone = ?,mobile = ?,fax = ?,emailpro = ?,phonepro = ?,B_streetnumber = ?,
                B_streettype = ?,B_address = ?,B_city = ?,B_zipcode = ?,B_email = ?,B_phone = ?,dateofbirth = ?,branchcode = ?,
                categorycode = ?,dateenrolled = ?,dateexpiry = ?,gonenoaddress = ?,lost = ?,debarred = ?,contactname = ?,
                contactfirstname = ?,contacttitle = ?,guarantorid = ?,borrowernotes = ?,relationship =  ?,ethnicity = ?,
-               ethnotes = ?,sex = ?,password = ?,flags = ?,userid = ?,opacnote = ?,contactnote = ?,sort1 = ?,sort2 = ? 
+               ethnotes = ?,sex = ?,password = ?,userid = ?,opacnote = ?,contactnote = ?,sort1 = ?,sort2 = ? 
                WHERE borrowernumber=$data{'borrowernumber'}";
-       $sth=$dbh->prepare($query);
-       $sth->execute(
-               $data{'cardnumber'},$data{'surname'},
-               $data{'firstname'},$data{'title'},
-               $data{'othernames'},$data{'initials'},
-               $data{'streetnumber'},$data{'streettype'},
-               $data{'address'},$data{'address2'},
-               $data{'city'},$data{'zipcode'},
-               $data{'email'},$data{'phone'},
-               $data{'mobile'},$data{'fax'},
-               $data{'emailpro'},$data{'phonepro'},
-               $data{'B_streetnumber'},$data{'B_streettype'},
-               $data{'B_address'},$data{'B_city'},
-               $data{'B_zipcode'},$data{'B_email'},$data{'B_phone'},
-               $data{'dateofbirth'},$data{'branchcode'},
-               $data{'categorycode'},$data{'dateenrolled'},
-               $data{'dateexpiry'},$data{'gonenoaddress'},
-               $data{'lost'},$data{'debarred'},
-               $data{'contactname'},$data{'contactfirstname'},
-               $data{'contacttitle'},$data{'guarantorid'},
-               $data{'borrowernotes'},$data{'relationship'},
-               $data{'ethnicity'},$data{'ethnotes'},
-               $data{'sex'},$data{'password'},
-               $data{'flags'},$data{'userid'},
-               $data{'opacnote'},$data{'contactnote'},
-               $data{'sort1'},$data{'sort2'}
-               );
-       }
-       $sth->finish;
-       # ok if its an adult (type) it may have borrowers that depend on it as a guarantor
-       # so when we update information for an adult we should check for guarantees and update the relevant part
-       # of their records, ie addresses and phone numbers
-       my ($category_type,undef)=getcategorytype($data{'category_type'});
-       if ($category_type eq 'A' ){
-       
-       # is adult check guarantees;
-               updateguarantees(%data);
-       
-       }       
-
-       
+        $sth = $dbh->prepare($query);
+        $sth->execute(
+            $data{'cardnumber'},       $data{'surname'},
+            $data{'firstname'},        $data{'title'},
+            $data{'othernames'},       $data{'initials'},
+            $data{'streetnumber'},     $data{'streettype'},
+            $data{'address'},          $data{'address2'},
+            $data{'city'},             $data{'zipcode'},
+            $data{'email'},            $data{'phone'},
+            $data{'mobile'},           $data{'fax'},
+            $data{'emailpro'},         $data{'phonepro'},
+            $data{'B_streetnumber'},   $data{'B_streettype'},
+            $data{'B_address'},        $data{'B_city'},
+            $data{'B_zipcode'},        $data{'B_email'},
+            $data{'B_phone'},          $data{'dateofbirth'},
+            $data{'branchcode'},       $data{'categorycode'},
+            $data{'dateenrolled'},     $data{'dateexpiry'},
+            $data{'gonenoaddress'},    $data{'lost'},
+            $data{'debarred'},         $data{'contactname'},
+            $data{'contactfirstname'}, $data{'contacttitle'},
+            $data{'guarantorid'},      $data{'borrowernotes'},
+            $data{'relationship'},     $data{'ethnicity'},
+            $data{'ethnotes'},         $data{'sex'},
+            $data{'password'},         $data{'userid'},
+            $data{'opacnote'},         $data{'contactnote'},
+            $data{'sort1'},            $data{'sort2'}
+        );
+    }
+    $sth->finish;
 
+# ok if its an adult (type) it may have borrowers that depend on it as a guarantor
+# so when we update information for an adult we should check for guarantees and update the relevant part
+# of their records, ie addresses and phone numbers
+    my $borrowercategory= GetBorrowercategory( $data{'category_type'} );
+    if ( $borrowercategory->{'category_type'} eq 'A' ) {
+        # is adult check guarantees;
+        UpdateGuarantees(%data);
+
+    }
+    &logaction(C4::Context->userenv->{'number'},"MEMBERS","MODIFY",$data{'borrowernumber'},"") 
+        if C4::Context->preference("BorrowersLog");
 }
 
-sub newmember {
+=head2
+
+=item AddMember
+
+  $borrowernumber = &ModMember(%borrower);
+
+insert new borrower into table
+
+=cut
+
+#'
+sub AddMember {
     my (%data) = @_;
     my $dbh = C4::Context->dbh;
     $data{'userid'} = '' unless $data{'password'};
     $data{'password'} = md5_base64( $data{'password'} ) if $data{'password'};
     $data{'dateofbirth'} = format_date_in_iso( $data{'dateofbirth'} );
     $data{'dateenrolled'} = format_date_in_iso( $data{'dateenrolled'} );
-    $data{'dateexpiry'} = format_date_in_iso( $data{'dateexpiry'} );
+    $data{'dateexpiry'}   = format_date_in_iso( $data{'dateexpiry'} );
     my $query =
         "insert into borrowers set cardnumber="
       . $dbh->quote( $data{'cardnumber'} )
@@ -465,16 +747,16 @@ sub newmember {
       . $dbh->quote( $data{'dateexpiry'} )
       . ",contactnote="
       . $dbh->quote( $data{'contactnote'} )
-      . ",b_address="
-      . $dbh->quote( $data{'b_address'} )
-      . ",b_zipcode="
-      . $dbh->quote( $data{'b_zipcode'} )
-      . ",b_city="
-      . $dbh->quote( $data{'b_city'} )
-      . ",b_phone="
-      . $dbh->quote( $data{'b_phone'} )
-      . ",b_email="
-      . $dbh->quote( $data{'b_email'}, )
+      . ",B_address="
+      . $dbh->quote( $data{'B_address'} )
+      . ",B_zipcode="
+      . $dbh->quote( $data{'B_zipcode'} )
+      . ",B_city="
+      . $dbh->quote( $data{'B_city'} )
+      . ",B_phone="
+      . $dbh->quote( $data{'B_phone'} )
+      . ",B_email="
+      . $dbh->quote( $data{'B_email'}, )
       . ",password="
       . $dbh->quote( $data{'password'} )
       . ",userid="
@@ -488,35 +770,35 @@ sub newmember {
       . ",emailpro="
       . $dbh->quote( $data{'emailpro'} )
       . ",contactfirstname="
-      . $dbh->quote( $data{'contactfirstname'} ) 
-      . ",sex="
-      . $dbh->quote( $data{'sex'} ) 
-      . ",fax="
+      . $dbh->quote( $data{'contactfirstname'} ) . ",sex="
+      . $dbh->quote( $data{'sex'} ) . ",fax="
       . $dbh->quote( $data{'fax'} )
-      . ",flags="
-      . $dbh->quote( $data{'flags'} )
       . ",relationship="
       . $dbh->quote( $data{'relationship'} )
-      . ",b_streetnumber="
-      . $dbh->quote( $data{'b_streetnumber'}) 
-      . ",b_streettype="
-      . $dbh->quote( $data{'b_streettype'})
+      . ",B_streetnumber="
+      . $dbh->quote( $data{'B_streetnumber'} )
+      . ",B_streettype="
+      . $dbh->quote( $data{'B_streettype'} )
       . ",gonenoaddress="
-      . $dbh->quote( $data{'gonenoaddress'})   
+      . $dbh->quote( $data{'gonenoaddress'} )
       . ",lost="
-      . $dbh->quote( $data{'lost'})                    
+      . $dbh->quote( $data{'lost'} )
       . ",debarred="
-      . $dbh->quote( $data{'debarred'})
+      . $dbh->quote( $data{'debarred'} )
       . ",ethnicity="
-      . $dbh->quote( $data{'ethnicity'})
+      . $dbh->quote( $data{'ethnicity'} )
       . ",ethnotes="
-      . $dbh->quote( $data{'ethnotes'});
+      . $dbh->quote( $data{'ethnotes'} );
 
     my $sth = $dbh->prepare($query);
     $sth->execute;
     $sth->finish;
-    $data{'borrowerid'} = $dbh->{'mysql_insertid'};
-    return $data{'borrowerid'};
+    $data{'borrowernumber'} = $dbh->{'mysql_insertid'};
+    
+    &logaction(C4::Context->userenv->{'number'},"MEMBERS","CREATE",$data{'borrowernumber'},"") 
+        if C4::Context->preference("BorrowersLog");
+        
+    return $data{'borrowernumber'};
 }
 
 sub changepassword {
@@ -541,37 +823,32 @@ sub changepassword {
         $sth->execute( $uid, $digest, $member );
         return 1;
     }
+    
+    &logaction(C4::Context->userenv->{'number'},"MEMBERS","CHANGE PASS",$member,"") 
+        if C4::Context->preference("BorrowersLog");
 }
 
-sub getmemberfromuserid {
-    my ($userid) = @_;
-    my $dbh      = C4::Context->dbh;
-    my $sth      = $dbh->prepare("select * from borrowers where userid=?");
-    $sth->execute($userid);
-    return $sth->fetchrow_hashref;
-}
 
-sub updateguarantees {
+sub UpdateGuarantees {
     my (%data) = @_;
     my $dbh = C4::Context->dbh;
-    my ( $count, $guarantees ) = findguarantees( $data{'borrowernumber'} );
+    my ( $count, $guarantees ) = GetGuarantees( $data{'borrowernumber'} );
     for ( my $i = 0 ; $i < $count ; $i++ ) {
 
         # FIXME
         # It looks like the $i is only being returned to handle walking through
         # the array, which is probably better done as a foreach loop.
         #
-        my $guaquery =
-"update borrowers set streetaddress='$data{'address'}',faxnumber='$data{'faxnumber'}',
-               streetcity='$data{'streetcity'}',phoneday='$data{'phoneday'}',city='$data{'city'}',area='$data{'area'}',phone='$data{'phone'}'
-               ,streetaddress='$data{'address'}'
-               where borrowernumber='$guarantees->[$i]->{'borrowernumber'}'";
+        my $guaquery = qq|UPDATE borrowers 
+                         SET address='$data{'address'}',fax='$data{'fax'}',
+                             B_city='$data{'B_city'}',mobile='$data{'mobile'}',city='$data{'city'}',phone='$data{'phone'}'
+                         WHERE borrowernumber='$guarantees->[$i]->{'borrowernumber'}'
+               |;
         my $sth3 = $dbh->prepare($guaquery);
         $sth3->execute;
         $sth3->finish;
     }
 }
-################################################################################
 
 =item fixup_cardnumber
 
@@ -657,165 +934,45 @@ sub fixup_cardnumber ($) {
     return $cardnumber;
 }
 
-sub findguarantees {
-    my ($bornum) = @_;
-    my $dbh      = C4::Context->dbh;
-    my $sth      = $dbh->prepare(
-        "select cardnumber,borrowernumber from borrowers where
-  guarantorid=?"
-    );
-    $sth->execute($bornum);
-    my @dat;
-    my $i = 0;
-    while ( my $data = $sth->fetchrow_hashref ) {
-        $dat[$i] = $data;
-        $i++;
-    }
-    $sth->finish;
-    return ( $i, \@dat );
-}
-
-=item findguarantor
+=head2 GetGuarantees
 
-  $guarantor = &findguarantor($borrower_no);
-  $guarantor_cardno = $guarantor->{"cardnumber"};
-  $guarantor_surname = $guarantor->{"surname"};
-  ...
+  ($num_children, $children_arrayref) = &GetGuarantees($parent_borrno);
+  $child0_cardno = $children_arrayref->[0]{"cardnumber"};
+  $child0_borrno = $children_arrayref->[0]{"borrowernumber"};
 
-C<&findguarantor> takes a borrower number (presumably that of a child
-patron), finds the guarantor for C<$borrower_no> (the child's parent),
-and returns the record for the guarantor.
+C<&GetGuarantees> takes a borrower number (e.g., that of a patron
+with children) and looks up the borrowers who are guaranteed by that
+borrower (i.e., the patron's children).
 
-C<&findguarantor> returns a reference-to-hash. Its keys are the fields
-from the C<borrowers> database table;
+C<&GetGuarantees> returns two values: an integer giving the number of
+borrowers guaranteed by C<$parent_borrno>, and a reference to an array
+of references to hash, which gives the actual results.
 
 =cut
 
 #'
-sub findguarantor {
-    my ($bornum) = @_;
-    my $dbh = C4::Context->dbh;
-    my $sth = $dbh->prepare("Select * from borrowers where borrowernumber=?");
-    $sth->execute($bornum);
-    my $data = $sth->fetchrow_hashref;
-    $sth->finish;
-    return ($data);
-}
-
-=item GuarantornameSearch
-
-  ($count, $borrowers) = &GuarantornameSearch($env, $searchstring, $type);
-
-Looks up guarantor  by name.
-
-C<$env> is ignored.
-
-BUGFIX 499: C<$type> is now used to determine type of search.
-if $type is "simple", search is performed on the first letter of the
-surname only.
-
-C<$searchstring> is a space-separated list of search terms. Each term
-must match the beginning a borrower's surname, first name, or other
-name.
-
-C<&GuarantornameSearch> returns a two-element list. C<$borrowers> is a
-reference-to-array; each element is a reference-to-hash, whose keys
-are the fields of the C<borrowers> table in the Koha database.
-C<$count> is the number of elements in C<$borrowers>.
-
-return all info from guarantor =>only category_type A
-
-=cut
-
-#'
-#used by member enquiries from the intranet
-#called by guarantor_search.pl
-sub GuarantornameSearch {
-    my ( $env, $searchstring, $orderby, $type ) = @_;
-    my $dbh   = C4::Context->dbh;
-    my $query = "";
-    my $count;
-    my @data;
-    my @bind = ();
-
-    if ( $type eq "simple" )    # simple search for one letter only
-    {
-        $query =
-"Select * from borrowers,categories  where borrowers.categorycode=categories.categorycode and category_type='A'  and  surname like ? order by $orderby";
-        @bind = ("$searchstring%");
-    }
-    else    # advanced search looking in surname, firstname and othernames
-    {
-        @data  = split( ' ', $searchstring );
-        $count = @data;
-        $query = "Select * from borrowers,categories
-               where ((surname like ? or surname like ?
-               or firstname  like ? or firstname like ?
-               or othernames like ? or othernames like ?) and borrowers.categorycode=categories.categorycode and category_type='A' 
-               ";
-        @bind = (
-            "$data[0]%", "% $data[0]%", "$data[0]%", "% $data[0]%",
-            "$data[0]%", "% $data[0]%"
-        );
-        for ( my $i = 1 ; $i < $count ; $i++ ) {
-            $query = $query . " and (" . " surname like ? or surname like ?
-                        or firstname  like ? or firstname like ?
-                       or othernames like ? or othernames like ?)";
-            push( @bind,
-                "$data[$i]%",   "% $data[$i]%", "$data[$i]%",
-                "% $data[$i]%", "$data[$i]%",   "% $data[$i]%" );
-
-            # FIXME - .= <<EOT;
-        }
-        $query = $query . ") or cardnumber like ?
-               order by $orderby";
-        push( @bind, $searchstring );
-
-        # FIXME - .= <<EOT;
-    }
-
-    my $sth = $dbh->prepare($query);
-    $sth->execute(@bind);
-    my @results;
-    my $cnt = $sth->rows;
-    while ( my $data = $sth->fetchrow_hashref ) {
-        push( @results, $data );
-    }
-
-    #  $sth->execute;
-    $sth->finish;
-    return ( $cnt, \@results );
-}
-
-=item NewBorrowerNumber
-
-  $num = &NewBorrowerNumber();
-
-Allocates a new, unused borrower number, and returns it.
-
-=cut
+sub GetGuarantees {
+    my ($borrowernumber) = @_;
+    my $dbh              = C4::Context->dbh;
+    my $sth              =
+      $dbh->prepare(
+"select cardnumber,borrowernumber, firstname, surname from borrowers where guarantorid=?"
+      );
+    $sth->execute($borrowernumber);
 
-#'
-# FIXME - This is identical to C4::Circulation::Borrower::NewBorrowerNumber.
-# Pick one and stick with it. Preferably use the other one. This function
-# doesn't belong in C4::Search.
-sub NewBorrowerNumber {
-    my $dbh = C4::Context->dbh;
-    my $sth = $dbh->prepare("Select max(borrowernumber) from borrowers");
-    $sth->execute;
-    my $data = $sth->fetchrow_hashref;
+    my @dat;
+    my $data = $sth->fetchall_arrayref({}); 
     $sth->finish;
-    $data->{'max(borrowernumber)'}++;
-    return ( $data->{'max(borrowernumber)'} );
+    return ( scalar(@$data), $data );
 }
 
-=head2 borrissues
+=head2 GetPendingIssues
 
-  ($count, $issues) = &borrissues($borrowernumber);
+  ($count, $issues) = &GetPendingIssues($borrowernumber);
 
 Looks up what the patron with the given borrowernumber has borrowed.
 
-C<&borrissues> returns a two-element array. C<$issues> is a
+C<&GetPendingIssues> returns a two-element array. C<$issues> is a
 reference-to-array, where each element is a reference-to-hash; the
 keys are the fields from the C<issues>, C<biblio>, and C<items> tables
 in the Koha database. C<$count> is the number of elements in
@@ -824,27 +981,23 @@ C<$issues>.
 =cut
 
 #'
-sub borrissues {
-    my ($bornum) = @_;
-    my $dbh      = C4::Context->dbh;
-    my $sth      = $dbh->prepare(
+sub GetPendingIssues {
+    my ($borrowernumber) = @_;
+    my $dbh              = C4::Context->dbh;
+    my $sth              = $dbh->prepare(
         "Select * from issues,biblio,items where borrowernumber=?
-   and items.itemnumber=issues.itemnumber
+        and items.itemnumber=issues.itemnumber
        and items.biblionumber=biblio.biblionumber
        and issues.returndate is NULL order by date_due"
     );
-    $sth->execute($bornum);
-    my @result;
-    while ( my $data = $sth->fetchrow_hashref ) {
-        push @result, $data;
-    }
-    $sth->finish;
-    return ( scalar(@result), \@result );
+    $sth->execute($borrowernumber);
+    my $data = $sth->fetchall_arrayref({});
+    return ( scalar(@$data), $data );
 }
 
-=head2 allissues
+=head2 GetAllIssues
 
-  ($count, $issues) = &allissues($borrowernumber, $sortkey, $limit);
+  ($count, $issues) = &GetAllIssues($borrowernumber, $sortkey, $limit);
 
 Looks up what the patron with the given borrowernumber has borrowed,
 and sorts the results.
@@ -855,7 +1008,7 @@ C<biblioitems>, or C<items> table in the Koha database.
 
 C<$limit> is the maximum number of results to return.
 
-C<&allissues> returns a two-element array. C<$issues> is a
+C<&GetAllIssues> returns a two-element array. C<$issues> is a
 reference-to-array, where each element is a reference-to-hash; the
 keys are the fields from the C<issues>, C<biblio>, C<biblioitems>, and
 C<items> tables of the Koha database. C<$count> is the number of
@@ -864,12 +1017,14 @@ elements in C<$issues>
 =cut
 
 #'
-sub allissues {
-    my ( $bornum, $order, $limit ) = @_;
+sub GetAllIssues {
+    my ( $borrowernumber, $order, $limit ) = @_;
 
     #FIXME: sanity-check order and limit
     my $dbh   = C4::Context->dbh;
-    my $query = "Select * from issues,biblio,items,biblioitems
+    my $count = 0;
+    my $query =
+"Select *,items.timestamp AS itemstimestamp from issues,biblio,items,biblioitems
   where borrowernumber=? and
   items.biblioitemnumber=biblioitems.biblioitemnumber and
   items.itemnumber=issues.itemnumber and
@@ -880,28 +1035,52 @@ sub allissues {
 
     #print $query;
     my $sth = $dbh->prepare($query);
-    $sth->execute($bornum);
+    $sth->execute($borrowernumber);
     my @result;
     my $i = 0;
     while ( my $data = $sth->fetchrow_hashref ) {
         $result[$i] = $data;
         $i++;
+        $count++;
+    }
+
+    # get all issued items for borrowernumber from oldissues table
+    # large chunk of older issues data put into table oldissues
+    # to speed up db calls for issuing items
+    if ( C4::Context->preference("ReadingHistory") ) {
+        my $query2 = "SELECT * FROM oldissues,biblio,items,biblioitems
+                      WHERE borrowernumber=? 
+                      AND items.biblioitemnumber=biblioitems.biblioitemnumber
+                      AND items.itemnumber=oldissues.itemnumber
+                      AND items.biblionumber=biblio.biblionumber
+                      ORDER BY $order";
+        if ( $limit != 0 ) {
+            $limit = $limit - $count;
+            $query2 .= " limit $limit";
+        }
+
+        my $sth2 = $dbh->prepare($query2);
+        $sth2->execute($borrowernumber);
+
+        while ( my $data2 = $sth2->fetchrow_hashref ) {
+            $result[$i] = $data2;
+            $i++;
+        }
+        $sth2->finish;
     }
     $sth->finish;
+
     return ( $i, \@result );
 }
 
-=head2 getboracctrecord
 
-  ($count, $acctlines, $total) = &getboracctrecord($env, $borrowernumber);
+=head2 GetBorrowerAcctRecord
 
-Looks up accounting data for the patron with the given borrowernumber.
+  ($total, $acctlines, $count) = &GetBorrowerAcctRecord($borrowernumber);
 
-C<$env> is ignored.
-
-(FIXME - I'm not at all sure what this is about.)
+Looks up accounting data for the patron with the given borrowernumber.
 
-C<&getboracctrecord> returns a three-element array. C<$acctlines> is a
+C<&GetBorrowerAcctRecord> returns a three-element array. C<$acctlines> is a
 reference-to-array, where each element is a reference-to-hash; the
 keys are the fields of the C<accountlines> table in the Koha database.
 C<$count> is the number of elements in C<$acctlines>. C<$total> is the
@@ -910,31 +1089,68 @@ total amount outstanding for all of the account lines.
 =cut
 
 #'
-sub getboracctrecord {
-    my ( $env, $params ) = @_;
+sub GetBorrowerAcctRecord {
+    my ($borrowernumber,$date) = @_;
     my $dbh = C4::Context->dbh;
     my @acctlines;
     my $numlines = 0;
-    my $sth      = $dbh->prepare(
-        "Select * from accountlines where
-borrowernumber=? order by date desc,timestamp desc"
+    my $strsth      = qq(
+        Select * from accountlines where
+borrowernumber=? order by date desc,timestamp desc
     );
+    my @bind = ($borrowernumber);
+    if ($date && $date ne ''){
+    $strsth.=" AND date < ?";
+    push(@bind,$date);
+    }
 
-    #   print $query;
-    $sth->execute( $params->{'borrowernumber'} );
+    my $sth= $dbh->prepare( $strsth );
+    $sth->execute( $borrowernumber);
     my $total = 0;
     while ( my $data = $sth->fetchrow_hashref ) {
 
-        #FIXME before reinstating: insecure?
-        #      if ($data->{'itemnumber'} ne ''){
-        #        $query="Select * from items,biblio where items.itemnumber=
-        #      '$data->{'itemnumber'}' and biblio.biblionumber=items.biblionumber";
-        #      my $sth2=$dbh->prepare($query);
-        #      $sth2->execute;
-        #      my $data2=$sth2->fetchrow_hashref;
-        #      $sth2->finish;
-        #      $data=$data2;
-        #     }
+        $acctlines[$numlines] = $data;
+        $numlines++;
+        $total += $data->{'amountoutstanding'};
+    }
+    $sth->finish;
+    return ( $total, \@acctlines,$numlines);
+}
+
+=head2 GetBorNotifyAcctRecord
+
+  ($count, $acctlines, $total) = &GetBorNotifyAcctRecord($params,$notifyid);
+
+Looks up accounting data for the patron with the given borrowernumber per file number.
+
+(FIXME - I'm not at all sure what this is about.)
+
+C<&GetBorNotifyAcctRecord> returns a three-element array. C<$acctlines> is a
+reference-to-array, where each element is a reference-to-hash; the
+keys are the fields of the C<accountlines> table in the Koha database.
+C<$count> is the number of elements in C<$acctlines>. C<$total> is the
+total amount outstanding for all of the account lines.
+
+=cut
+
+sub GetBorNotifyAcctRecord {
+    my ( $params, $notifyid ) = @_;
+    my $dbh = C4::Context->dbh;
+    my @acctlines;
+    my $numlines = 0;
+    my $query    = qq| SELECT * 
+                       FROM accountlines 
+                       WHERE borrowernumber=? 
+                       AND notify_id=? 
+                       AND (accounttype='FU' OR accounttype='N' OR accounttype='M'OR accounttype='A'OR accounttype='F'OR accounttype='L' OR accounttype='IP' OR accounttype='CH' OR accounttype='RE' OR accounttype='RL')
+                       AND amountoutstanding != '0' 
+                       ORDER BY notify_id,accounttype
+               |;
+    my $sth = $dbh->prepare($query);
+
+    $sth->execute( $params->{'borrowernumber'}, $notifyid );
+    my $total = 0;
+    while ( my $data = $sth->fetchrow_hashref ) {
         $acctlines[$numlines] = $data;
         $numlines++;
         $total += $data->{'amountoutstanding'};
@@ -1012,30 +1228,11 @@ sub getzipnamecity {
     return $data[0], $data[1];
 }
 
-=head2 updatechildguarantor (OUEST-PROVENCE)
-
-check for title,firstname,surname,adress,zip code and city  from guarantor to 
-guarantorchild
-
-=cut
-
-#'
-
-sub getguarantordata {
-    my ($borrowerid) = @_;
-    my $dbh          = C4::Context->dbh;
-    my $sth          =
-      $dbh->prepare(
-"Select title,firstname,surname,streetnumber,address,streettype,address2,zipcode,city,phone,phonepro,mobile,email,emailpro  from borrowers where borrowernumber =? "
-      );
-    $sth->execute($borrowerid);
-    my $guarantor_data = $sth->fetchrow_hashref;
-    $sth->finish;
-    return $guarantor_data;
-}
 
 =head2 getdcity (OUEST-PROVENCE)
+
 recover cityid  with city_name condition
+
 =cut
 
 sub getidcity {
@@ -1047,26 +1244,14 @@ sub getidcity {
     return $data;
 }
 
-=head2 getcategorytype (OUEST-PROVENCE)
 
-check for the category_type with categorycode
-and return the category_type 
+=head2 GetExpiryDate 
 
-=cut
-
-sub getcategorytype {
-    my ($categorycode) = @_;
-    my $dbh            = C4::Context->dbh;
-    my $sth            =
-      $dbh->prepare(
-"Select category_type,description from categories where categorycode=?  "
-      );
-    $sth->execute($categorycode);
-    my ( $category_type, $description ) = $sth->fetchrow;
-    return $category_type, $description;
-}
+  $expirydate = GetExpiryDate($categorycode, $dateenrolled);
+process expiry date given a date and a categorycode
 
-sub calcexpirydate {
+=cut
+sub GetExpiryDate {
     my ( $categorycode, $dateenrolled ) = @_;
     my $dbh = C4::Context->dbh;
     my $sth =
@@ -1075,8 +1260,9 @@ sub calcexpirydate {
     $sth->execute($categorycode);
     my ($enrolmentperiod) = $sth->fetchrow;
     $enrolmentperiod = 12 unless ($enrolmentperiod);
-    return format_date_in_iso(
-        &DateCalc( $dateenrolled, "$enrolmentperiod months" ) );
+    my @date=split /-/,format_date_in_iso($dateenrolled);
+    @date=Add_Delta_YM($date[0],$date[1],$date[2],0,$enrolmentperiod);
+    return sprintf("%04d-%02d-%02d",$date[0],$date[1],$date[2]);
 }
 
 =head2 checkuserpassword (OUEST-PROVENCE)
@@ -1088,22 +1274,22 @@ return the number of record
 =cut
 
 sub checkuserpassword {
-    my ( $borrowerid, $userid, $password ) = @_;
+    my ( $borrowernumber, $userid, $password ) = @_;
     $password = md5_base64($password);
     my $dbh = C4::Context->dbh;
     my $sth =
       $dbh->prepare(
 "Select count(*) from borrowers where borrowernumber !=? and userid =? and password=? "
       );
-    $sth->execute( $borrowerid, $userid, $password );
+    $sth->execute( $borrowernumber, $userid, $password );
     my $number_rows = $sth->fetchrow;
     return $number_rows;
 
 }
 
-=head2 borrowercategories
+=head2 GetborCatFromCatType
 
-  ($codes_arrayref, $labels_hashref) = &borrowercategories();
+  ($codes_arrayref, $labels_hashref) = &GetborCatFromCatType();
 
 Looks up the different types of borrowers in the database. Returns two
 elements: a reference-to-array, which lists the borrower category
@@ -1113,14 +1299,21 @@ to category descriptions.
 =cut
 
 #'
-sub borrowercategories {
+sub GetborCatFromCatType {
     my ( $category_type, $action ) = @_;
-    my $dbh = C4::Context->dbh;
-    my $request;
-    $request =
-"Select categorycode,description from categories where category_type=? order by categorycode";
+    my $dbh     = C4::Context->dbh;
+    my $request = qq|  SELECT categorycode,description 
+                       FROM categories 
+                       $action
+                       ORDER BY categorycode|;
     my $sth = $dbh->prepare($request);
-    $sth->execute($category_type);
+    if ($action) {
+        $sth->execute($category_type);
+    }
+    else {
+        $sth->execute();
+    }
+
     my %labels;
     my @codes;
 
@@ -1132,25 +1325,27 @@ sub borrowercategories {
     return ( \@codes, \%labels );
 }
 
-=head2 getborrowercategory
+=head2 GetBorrowercategory
 
-  $description = &getborrowercategory($categorycode);
+  $hashref = &GetBorrowercategory($categorycode);
 
 Given the borrower's category code, the function returns the corresponding
-description for a comprehensive information display.
+data hashref for a comprehensive information display.
 
 =cut
 
-sub getborrowercategory {
+sub GetBorrowercategory {
     my ($catcode) = @_;
     my $dbh       = C4::Context->dbh;
     my $sth       =
       $dbh->prepare(
-        "SELECT description FROM categories WHERE categorycode = ?");
+"SELECT description,dateofbirthrequired,upperagelimit,category_type FROM categories WHERE categorycode = ?"
+      );
     $sth->execute($catcode);
-    my $description = $sth->fetchrow();
+    my $data =
+      $sth->fetchrow_hashref;
     $sth->finish();
-    return $description;
+    return $data;
 }    # sub getborrowercategory
 
 =head2 ethnicitycategories
@@ -1192,9 +1387,9 @@ Koha database ("European" or "Pacific Islander").
 
 #'
 
-sub fixEthnicity($) {
-
+sub fixEthnicity {
     my $ethnicity = shift;
+    return unless $ethnicity;
     my $dbh       = C4::Context->dbh;
     my $sth       = $dbh->prepare("Select name from ethnicity where code = ?");
     $sth->execute($ethnicity);
@@ -1203,40 +1398,38 @@ sub fixEthnicity($) {
     return $data->{'name'};
 }    # sub fixEthnicity
 
+=head2 GetAge
 
-
-=head2 get_age
-
-  $dateofbirth,$date = &get_age($date);
+  $dateofbirth,$date = &GetAge($date);
 
 this function return the borrowers age with the value of dateofbirth
 
 =cut
+
 #'
-sub get_age {
-    my ($date, $date_ref) = @_;
+sub GetAge{
+    my ( $date, $date_ref ) = @_;
 
-    if (not defined $date_ref) {
-        $date_ref = sprintf('%04d-%02d-%02d', Today());
+    if ( not defined $date_ref ) {
+        $date_ref = sprintf( '%04d-%02d-%02d', Today() );
     }
 
-    my ($year1, $month1, $day1) = split /-/, $date;
-    my ($year2, $month2, $day2) = split /-/, $date_ref;
+    my ( $year1, $month1, $day1 ) = split /-/, $date;
+    my ( $year2, $month2, $day2 ) = split /-/, $date_ref;
 
     my $age = $year2 - $year1;
-    if ($month1.$day1 > $month2.$day2) {
+    if ( $month1 . $day1 > $month2 . $day2 ) {
         $age--;
     }
 
     return $age;
-}# sub get_age
-
-
+}    # sub get_age
 
 =head2 get_institutions
   $insitutions = get_institutions();
 
 Just returns a list of all the borrowers of type I, borrownumber and name
+
 =cut
 
 #'
@@ -1271,10 +1464,379 @@ sub add_member_orgs {
     my $query =
       "INSERT INTO borrowers_to_borrowers (borrower1,borrower2) VALUES (?,?)";
     my $sth = $dbh->prepare($query);
-    foreach my $bornum (@$otherborrowers) {
-        $sth->execute( $borrowernumber, $bornum );
+    foreach my $otherborrowernumber (@$otherborrowers) {
+        $sth->execute( $borrowernumber, $otherborrowernumber );
     }
     $sth->finish();
 
 }    # sub add_member_orgs
+
+=head2 GetBorrowersFromSurname
+
+=over 4
+
+\@resutlts = GetBorrowersFromSurname($surname)
+this function get the list of borrower names like $surname.
+return :
+the table of results in @results
+
+=back
+
+=cut
+
+sub GetBorrowersFromSurname {
+    my ($searchstring) = @_;
+    my $dbh = C4::Context->dbh;
+    $searchstring =~ s/\'/\\\'/g;
+    my @data  = split( ' ', $searchstring );
+    my $count = @data;
+    my $query = qq|
+        SELECT   surname,firstname
+        FROM     borrowers
+        WHERE    (surname like ?)
+        ORDER BY surname
+    |;
+    my $sth = $dbh->prepare($query);
+    $sth->execute("$data[0]%");
+    my @results;
+    $count = 0;
+
+    my $data = $sth->fetchall_arrayref({});
+    $sth->finish;
+    return ( scalar(@$data), $data );
+}
+
+=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 $query = qq|SELECT cityid,city_name 
+               FROM cities 
+               ORDER BY city_name|;
+    my $sth = $dbh->prepare($query);
+
+    #$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'};
+    }
+
+#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 );
+    }
+}
+
+=head2 GetSortDetails (OUEST-PROVENCE)
+
+  ($lib) = &GetSortDetails($category,$sortvalue);
+
+Returns the authorized value  details
+C<&$lib>return value of authorized value details
+C<&$sortvalue>this is the value of authorized value 
+C<&$category>this is the value of authorized value category
+
+=cut
+
+sub GetSortDetails {
+    my ( $category, $sortvalue ) = @_;
+    my $dbh   = C4::Context->dbh;
+    my $query = qq|SELECT lib 
+               FROM authorised_values 
+               WHERE category=?
+               AND authorised_value=? |;
+    my $sth = $dbh->prepare($query);
+    $sth->execute( $category, $sortvalue );
+    my $lib = $sth->fetchrow;
+    return ($lib);
+}
+
+=head2 DeleteBorrower 
+
+  () = &DeleteBorrower($member);
+
+delete all data fo borrowers and add record to deletedborrowers table
+C<&$member>this is the borrowernumber
+
+=cut
+
+sub MoveMemberToDeleted {
+    my ($member) = @_;
+    my $dbh = C4::Context->dbh;
+    my $query;
+    $query = qq|SELECT * 
+                 FROM borrowers 
+                 WHERE borrowernumber=?|;
+    my $sth = $dbh->prepare($query);
+    $sth->execute($member);
+    my @data = $sth->fetchrow_array;
+    $sth->finish;
+    $sth =
+      $dbh->prepare( "Insert into deletedborrowers values ("
+          . ( "?," x ( scalar(@data) - 1 ) )
+          . "?)" );
+    $sth->execute(@data);
+    $sth->finish;
+}
+
+=head2 DelMember
+
+DelMember($borrowernumber);
+
+This function remove directly a borrower whitout writing it on deleteborrower.
++ Deletes reserves for the borrower
+
+=cut
+
+sub DelMember {
+    my $dbh            = C4::Context->dbh;
+    my $borrowernumber = shift;
+    return unless $borrowernumber;    # borrowernumber is mandatory.
+
+    my $query = qq|DELETE 
+                 FROM  reserves 
+                 WHERE borrowernumber=?|;
+    my $sth = $dbh->prepare($query);
+    $sth->execute($borrowernumber);
+    $sth->finish;
+    $query = "
+       DELETE
+       FROM borrowers
+       WHERE borrowernumber = ?
+   ";
+    my $sth = $dbh->prepare($query);
+    &logaction(C4::Context->userenv->{'number'},"MEMBERS","DELETE",$borrowernumber,"") 
+        if C4::Context->preference("BorrowersLog");
+    return $sth->rows;
+}
+
+=head2 ExtendMemberSubscriptionTo (OUEST-PROVENCE)
+
+$date= ExtendMemberSubscriptionTo($borrowerid, $date);
+Extending the subscription to a given date or to the expiry date calculated on local date.
+returns date 
+=cut
+
+sub ExtendMemberSubscriptionTo {
+    my ( $borrowerid,$date) = @_;
+    my $dbh = C4::Context->dbh;
+    unless ($date){
+      $date=POSIX::strftime("%Y-%m-%d",localtime(time));
+      my $borrower = GetBorrower($borrowerid,'borrowernumber');
+      $date = GetExpiryDate( $borrower->{'categorycode'}, $date );
+    }
+    my $sth = $dbh->do(<<EOF);
+UPDATE borrowers 
+SET  dateexpiry='$date' 
+WHERE borrowernumber='$borrowerid'
+EOF
+    return $date if ($sth);
+    return 0;
+}
+
+=head2 GetRoadTypes (OUEST-PROVENCE)
+
+  ($idroadtypearrayref, $roadttype_hashref) = &GetRoadTypes();
+
+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 .
+
+
+=cut
+
+sub GetRoadTypes {
+    my $dbh   = C4::Context->dbh;
+    my $query = qq|SELECT roadtypeid,road_type 
+               FROM roadtype 
+               ORDER BY road_type|;
+    my $sth = $dbh->prepare($query);
+    $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 GetBorrowersTitles (OUEST-PROVENCE)
+
+  ($borrowertitle)= &GetBorrowersTitles();
+
+Looks up the different title . Returns array  with all borrowers title
+
+=cut
+
+sub GetBorrowersTitles {
+    my @borrowerTitle = split /,|\|/,C4::Context->preference('BorrowersTitles');
+    unshift( @borrowerTitle, "" );
+    return ( \@borrowerTitle);
+    }
+
+
+
+=head2 GetRoadTypeDetails (OUEST-PROVENCE)
+
+  ($roadtype) = &GetRoadTypeDetails($roadtypeid);
+
+Returns the description of roadtype
+C<&$roadtype>return description of road type
+C<&$roadtypeid>this is the value of roadtype s
+
+=cut
+
+sub GetRoadTypeDetails {
+    my ($roadtypeid) = @_;
+    my $dbh          = C4::Context->dbh;
+    my $query        = qq|SELECT road_type 
+               FROM roadtype 
+               WHERE roadtypeid=?|;
+    my $sth = $dbh->prepare($query);
+    $sth->execute($roadtypeid);
+    my $roadtype = $sth->fetchrow;
+    return ($roadtype);
+}
+
+=head2 GetBorrowersWhoHaveNotBorrowedSince
+
+&GetBorrowersWhoHaveNotBorrowedSince($date)
+
+this function get all borrowers who haven't borrowed since the date given on input arg.
+
+=cut
+
+sub GetBorrowersWhoHaveNotBorrowedSince {
+    my $date = shift;
+    return unless $date;    # date is mandatory.
+    my $dbh   = C4::Context->dbh;
+    my $query = "
+        SELECT borrowers.borrowernumber,max(timestamp)
+        FROM   borrowers
+          LEFT JOIN issues ON borrowers.borrowernumber = issues.borrowernumber
+        WHERE issues.borrowernumber IS NOT NULL
+        GROUP BY borrowers.borrowernumber
+   ";
+    my $sth = $dbh->prepare($query);
+    $sth->execute;
+    my @results;
+
+    while ( my $data = $sth->fetchrow_hashref ) {
+        push @results, $data;
+    }
+    return \@results;
+}
+
+=head2 GetBorrowersWhoHaveNeverBorrowed
+
+$results = &GetBorrowersWhoHaveNeverBorrowed
+
+this function get all borrowers who have never borrowed.
+
+I<$result> is a ref to an array which all elements are a hasref.
+
+=cut
+
+sub GetBorrowersWhoHaveNeverBorrowed {
+    my $dbh   = C4::Context->dbh;
+    my $query = "
+        SELECT borrowers.borrowernumber,max(timestamp)
+        FROM   borrowers
+          LEFT JOIN issues ON borrowers.borrowernumber = issues.borrowernumber
+        WHERE issues.borrowernumber IS NULL
+   ";
+    my $sth = $dbh->prepare($query);
+    $sth->execute;
+    my @results;
+    while ( my $data = $sth->fetchrow_hashref ) {
+        push @results, $data;
+    }
+    return \@results;
+}
+
+=head2 GetBorrowersWithIssuesHistoryOlderThan
+
+$results = &GetBorrowersWithIssuesHistoryOlderThan($date)
+
+this function get all borrowers who has an issue history older than I<$date> given on input arg.
+
+I<$result> is a ref to an array which all elements are a hashref.
+This hashref is containt the number of time this borrowers has borrowed before I<$date> and the borrowernumber.
+
+=cut
+
+sub GetBorrowersWithIssuesHistoryOlderThan {
+    my $dbh  = C4::Context->dbh;
+    my $date = shift;
+    return unless $date;    # date is mandatory.
+    my $query = "
+       SELECT count(borrowernumber) as n,borrowernumber
+       FROM issues
+       WHERE returndate < ?
+         AND borrowernumber IS NOT NULL 
+       GROUP BY borrowernumber
+   ";
+    my $sth = $dbh->prepare($query);
+    $sth->execute($date);
+    my @results;
+
+    while ( my $data = $sth->fetchrow_hashref ) {
+        push @results, $data;
+    }
+    return \@results;
+}
+
+END { }    # module clean-up code here (global destructor)
+
 1;
+
+__END__
+
+=back
+
+=head1 AUTHOR
+
+Koha Team
+
+=cut