# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
# Suite 330, Boston, MA 02111-1307 USA
-# $Id$
use strict;
require Exporter;
use Digest::MD5 qw(md5_base64);
use Date::Calc qw/Today Add_Delta_YM/;
use C4::Log; # logaction
-use C4::Accounts;
use C4::Overdues;
-use C4::Reserves2;
+use C4::Reserves;
-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK);
+our ($VERSION,@ISA,@EXPORT,@EXPORT_OK,$debug);
-$VERSION = do { my @v = '$Revision$' =~ /\d+/g; shift(@v) . "." . join( "_", map { sprintf "%03d", $_ } @v ); };
+BEGIN {
+ $VERSION = 3.01;
+ $debug = $ENV{DEBUG} || 0;
+}
=head1 NAME
@ISA = qw(Exporter);
-@EXPORT = qw(
- &BornameSearch &GetMember &GetMemberDetails
- &borrdata &borrdata2
- &fixup_cardnumber &findguarantees &findguarantor &GuarantornameSearch
- &modmember &newmember &changepassword &borrissues &allissues
- &checkuniquemember &getzipnamecity &getidcity &getguarantordata &getcategorytype
- &DeleteBorrower
- &calcexpirydate &checkuserpassword
- &getboracctrecord
- &GetborCatFromCatType &getborrowercategory
- &fixEthnicity
- ðnicitycategories &get_institutions add_member_orgs
- &get_age &GetBorrowersFromSurname &GetBranchCodeFromBorrowers
- &GetFlagsAndBranchFromBorrower
- &GetCities &GetRoadTypes &GetRoadTypeDetails &GetBorNotifyAcctRecord
- &GetMembeReregistration
+#Get data
+push @EXPORT, qw(
+ &SearchMember
+ &GetMemberDetails
+ &GetMember
+
+ &GetGuarantees
+
+ &GetMemberIssuesAndFines
+ &GetPendingIssues
+ &GetAllIssues
+
+ &get_institutions
+ &getzipnamecity
+ &getidcity
+
+ &GetAge
+ &GetCities
+ &GetRoadTypes
+ &GetRoadTypeDetails
&GetSortDetails
- &GetBorrowersTitles
+ &GetTitles
+
+ &GetMemberAccountRecords
+ &GetBorNotifyAcctRecord
+
+ &GetborCatFromCatType
+ &GetBorrowercategory
+
+
&GetBorrowersWhoHaveNotBorrowedSince
&GetBorrowersWhoHaveNeverBorrowed
&GetBorrowersWithIssuesHistoryOlderThan
+
+ &GetExpiryDate
+);
+
+#Modify data
+push @EXPORT, qw(
+ &ModMember
+ &changepassword
+);
+
+#Delete data
+push @EXPORT, qw(
+ &DelMember
);
-=item BornameSearch
+#Insert data
+push @EXPORT, qw(
+ &AddMember
+ &add_member_orgs
+ &MoveMemberToDeleted
+ &ExtendMemberSubscriptionTo
+);
- ($count, $borrowers) = &BornameSearch($env, $searchstring, $type);
+#Check data
+push @EXPORT, qw(
+ &checkuniquemember
+ &checkuserpassword
+ &Check_Userid
+ &fixEthnicity
+ ðnicitycategories
+ &fixup_cardnumber
+ &checkcardnumber
+);
-Looks up patrons (borrowers) by name.
+=item SearchMember
-C<$env> is ignored.
+ ($count, $borrowers) = &SearchMember($searchstring, $type,$category_type,$filter,$showallbranches);
+
+Looks up patrons (borrowers) by name.
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.
+$category_type is used to get a specified type of user.
+(mainly adults when creating a child.)
+
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<$filter> is assumed to be a list of elements to filter results on
+
+C<$showallbranches> is used in IndependantBranches Context to display all branches results.
+
+C<&SearchMember> 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>.
#'
#used by member enquiries from the intranet
-#called by member.pl
-sub BornameSearch {
- my ( $env, $searchstring, $orderby, $type ) = @_;
- my $dbh = C4::Context->dbh;
- my $query = "";
- my $count;
- my @data;
- my @bind = ();
+#called by member.pl and circ/circulation.pl
+sub SearchMember {
+ my ($searchstring, $orderby, $type,$category_type,$filter,$showallbranches ) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = "";
+ my $count;
+ my @data;
+ my @bind = ();
+
+ # this is used by circulation everytime a new borrowers cardnumber is scanned
+ # so we can check an exact match first, if that works return, otherwise do the rest
+ $query = "SELECT * FROM borrowers
+ LEFT JOIN categories ON borrowers.categorycode=categories.categorycode
+ ";
+ my $sth = $dbh->prepare("$query WHERE cardnumber = ?");
+ $sth->execute($searchstring);
+ my $data = $sth->fetchall_arrayref({});
+ if (@$data){
+ return ( scalar(@$data), $data );
+ }
+ $sth->finish;
if ( $type eq "simple" ) # simple search for one letter only
{
- $query =
- "SELECT * FROM borrowers
- LEFT JOIN categories ON borrowers.categorycode=categories.categorycode
- WHERE surname LIKE ? ORDER BY $orderby";
- @bind = ("$searchstring%");
+ $query .= ($category_type ? " AND category_type = ".$dbh->quote($category_type) : "");
+ $query .= " WHERE (surname LIKE ? OR cardnumber like ?) ";
+ if (C4::Context->preference("IndependantBranches") && !$showallbranches){
+ if (C4::Context->userenv && C4::Context->userenv->{flags}!=1 && C4::Context->userenv->{'branch'}){
+ $query.=" AND borrowers.branchcode =".$dbh->quote(C4::Context->userenv->{'branch'}) unless (C4::Context->userenv->{'branch'} eq "insecure");
+ }
+ }
+ $query.=" ORDER BY $orderby";
+ @bind = ("$searchstring%","$searchstring");
}
else # advanced search looking in surname, firstname and othernames
{
@data = split( ' ', $searchstring );
$count = @data;
- $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 ?)
- ";
- @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 ?
+ $query .= " WHERE ";
+ if (C4::Context->preference("IndependantBranches") && !$showallbranches){
+ if (C4::Context->userenv && C4::Context->userenv->{flags}!=1 && C4::Context->userenv->{'branch'}){
+ $query.=" borrowers.branchcode =".$dbh->quote(C4::Context->userenv->{'branch'})." AND " unless (C4::Context->userenv->{'branch'} eq "insecure");
+ }
+ }
+ $query.="((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 ?)";
+ 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 );
+ push( @bind, $searchstring );
- # FIXME - .= <<EOT;
+ # FIXME - .= <<EOT;
}
- my $sth = $dbh->prepare($query);
+ $sth = $dbh->prepare($query);
- # warn "Q $orderby : $query";
+ $debug and print STDERR "Q $orderby : $query\n";
$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 );
-}
-
-=head3 GetFlagsAndBranchFromBorrower
-
-=over 4
-
-($flags, $homebranch) = GetFlagsAndBranchFromBorrower($loggedinuser);
-
-this function read on the database to get flags and homebranch for a user
-given on input arg.
-
-return :
-it returns the $flags & the homebranch in scalar context.
-
-=back
-
-=cut
-
-sub GetFlagsAndBranchFromBorrower {
- my $loggedinuser = @_;
- my $dbh = C4::Context->dbh;
- my $query = "
- SELECT flags, branchcode
- FROM borrowers
- WHERE borrowernumber = ?
- ";
- my $sth = $dbh->prepare($query);
- $sth->execute($loggedinuser);
-
- return $sth->fetchrow;
-}
-
-=item GetMember
-
- $borrower = &GetMember($cardnumber, $borrowernumber);
+ $data = $sth->fetchall_arrayref({});
-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.
-
-C<&GetMember> returns a reference-to-hash whose keys are the fields of
-the C<borrowers> table in the Koha database.
-
-=cut
-
-sub GetMember {
- my ( $cardnumber, $borrowernumber ) = @_;
- $cardnumber = uc $cardnumber;
- my $dbh = C4::Context->dbh;
- my $sth;
- if ( $borrowernumber eq '' ) {
- $sth = $dbh->prepare("Select * from borrowers where cardnumber=?");
- $sth->execute($cardnumber);
- }
- else {
- $sth = $dbh->prepare("Select * from borrowers where borrowernumber=?");
- $sth->execute($borrowernumber);
- }
- my $data = $sth->fetchrow_hashref;
$sth->finish;
- if ($data) {
- return ($data);
- }
- 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);
- }
- }
- return undef;
+ return ( scalar(@$data), $data );
}
=head2 GetMemberDetails
up the borrower by number; otherwise, it looks up the borrower by card
number.
-C<$env> is effectively ignored, but should be a reference-to-hash.
-
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
return undef;
}
my $borrower = $sth->fetchrow_hashref;
- my $amount = C4::Accounts::checkaccount( $borrowernumber, $dbh );
+ my ($amount) = GetMemberAccountRecords( $borrowernumber);
$borrower->{'amountoutstanding'} = $amount;
- my $flags = patronflags( $borrower, $dbh );
+ # FIXME - patronflags calls GetMemberAccountRecords... just have patronflags return $amount
+ my $flags = patronflags( $borrower);
my $accessflagshash;
$sth = $dbh->prepare("select bit,flag from userflags");
NOTE!: If you change this function, be sure to update the POD for
&GetMemberDetails.
- $flags = &patronflags($env, $patron, $dbh);
+ $flags = &patronflags($patron);
$flags->{CHARGES}
{message} Message showing patron's credit or debt
{itemlist} ref-to-array: list of available items
=cut
-
+# FIXME rename this function.
sub patronflags {
my %flags;
- my ( $patroninformation, $dbh ) = @_;
- my $amount =
- C4::Accounts::checkaccount( $patroninformation->{'borrowernumber'}, $dbh );
+ my ( $patroninformation) = @_;
+ my $dbh=C4::Context->dbh;
+ my ($amount) = GetMemberAccountRecords( $patroninformation->{'borrowernumber'});
if ( $amount > 0 ) {
my %flaginfo;
my $noissuescharge = C4::Context->preference("noissuescharge");
$flaginfo{'message'} = sprintf "Patron owes \$%.02f", $amount;
+ $flaginfo{'amount'} = sprintf "%.02f",$amount;
if ( $amount > $noissuescharge ) {
$flaginfo{'noissues'} = 1;
}
}
$flags{'ODUES'} = \%flaginfo;
}
- my $itemswaiting =
- C4::Reserves2::GetWaitingReserves( $patroninformation->{'borrowernumber'} );
- my $nowaiting = scalar @$itemswaiting;
+ my @itemswaiting = C4::Reserves::GetReservesFromBorrowernumber( $patroninformation->{'borrowernumber'},'W' );
+ my $nowaiting = scalar @itemswaiting;
if ( $nowaiting > 0 ) {
my %flaginfo;
$flaginfo{'message'} = "Reserved items available";
- $flaginfo{'itemlist'} = $itemswaiting;
+ $flaginfo{'itemlist'} = \@itemswaiting;
$flags{'WAITING'} = \%flaginfo;
}
return ( \%flags );
}
-=item borrdata
+=item GetMember
- $borrower = &borrdata($cardnumber, $borrowernumber);
+ $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, $borrowernumber ) = @_;
- $cardnumber = uc $cardnumber;
+sub GetMember {
+ my ( $information, $type ) = @_;
my $dbh = C4::Context->dbh;
my $sth;
- if ( $borrowernumber eq '' ) {
- $sth =
- $dbh->prepare(
-"Select borrowers.*,categories.category_type from borrowers left join categories on borrowers.categorycode=categories.categorycode where cardnumber=?"
- );
- $sth->execute($cardnumber);
- }
- else {
- $sth =
- $dbh->prepare(
-"Select borrowers.*,categories.category_type from borrowers left join categories on borrowers.categorycode=categories.categorycode where borrowernumber=?"
- );
- $sth->execute($borrowernumber);
- }
+ my $select = "
+SELECT borrowers.*, categories.category_type, categories.description
+FROM borrowers
+LEFT JOIN categories on borrowers.categorycode=categories.categorycode
+";
+ if ($type eq 'cardnumber' || $type eq 'firstname'|| $type eq 'userid'|| $type eq 'borrowernumber'){
+ $information = uc $information;
+ $sth = $dbh->prepare("$select WHERE $type=?");
+ } else {
+ $sth = $dbh->prepare("$select WHERE borrowernumber=?");
+ }
+ $sth->execute($information);
my $data = $sth->fetchrow_hashref;
-
$sth->finish;
- if ($data) {
- return ($data);
- }
- elsif ($cardnumber) { # 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);
- my $data = $sth->fetchrow_hashref;
- $sth->finish;
- return ($data);
- }
- else {
- return undef;
+ ($data) and return ($data);
+
+ if ($type eq 'cardnumber' || $type eq 'firstname') { # otherwise, try with firstname
+ $sth = $dbh->prepare("$select WHERE firstname like ?");
+ $sth->execute($information);
+ $data = $sth->fetchrow_hashref;
+ $sth->finish;
+ return ($data);
}
+ return undef;
}
-=item borrdata2
+=item GetMemberIssuesAndFines
- ($borrowed, $due, $fine) = &borrdata2($env, $borrowernumber);
+ ($borrowed, $due, $fine) = &GetMemberIssuesAndFines($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<&GetMemberIssuesAndFines> 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.
=cut
#'
-sub borrdata2 {
- my ( $env, $borrowernumber ) = @_;
+sub GetMemberIssuesAndFines {
+ my ( $borrowernumber ) = @_;
my $dbh = C4::Context->dbh;
my $query =
"Select count(*) from issues where borrowernumber='$borrowernumber' and
returndate is NULL";
- # print $query;
+ $debug and print $query, "\n";
my $sth = $dbh->prepare($query);
$sth->execute;
my $data = $sth->fetchrow_hashref;
$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'} );
+=head2
- # warn "num user".$data{'borrowernumber'};
- my $query;
- my $sth;
- $data{'userid'} = '' if ( $data{'password'} eq '' );
+=item ModMember
- # test to know if u must update or not the borrower password
- if ( $data{'password'} eq '****' ) {
+ &ModMember($borrowernumber);
- $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 = ?,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{'userid'}, $data{'opacnote'},
- $data{'contactnote'}, $data{'sort1'},
- $data{'sort2'}
- );
- }
- else {
+Modify borrower's data
- ( $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 = ?,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{'userid'},
- $data{'opacnote'}, $data{'contactnote'},
- $data{'sort1'}, $data{'sort2'}
- );
- }
- $sth->finish;
+=cut
+
+#'
+sub ModMember {
+ my (%data) = @_;
+ my $dbh = C4::Context->dbh;
+ $data{'dateofbirth'} = format_date_in_iso( $data{'dateofbirth' } ) if ($data{'dateofbirth' } );
+ $data{'dateexpiry'} = format_date_in_iso( $data{ 'dateexpiry' } ) if ($data{ 'dateexpiry' } );
+ $data{'dateenrolled'} = format_date_in_iso( $data{'dateenrolled'} ) if ($data{'dateenrolled'} );
+ my $qborrower=$dbh->prepare("SHOW columns from borrowers");
+ $qborrower->execute;
+ my %hashborrowerfields;
+ while (my ($field)=$qborrower->fetchrow){
+ $hashborrowerfields{$field}=1;
+ }
+ my $query = "UPDATE borrowers SET \n";
+ my $sth;
+ my @parameters;
+
+ # test to know if you must update or not the borrower password
+ if ( $data{'password'} eq '****' ) {
+ delete $data{'password'};
+ } else {
+ $data{'password'} = md5_base64( $data{'password'} ) if ($data{'password'} ne "");
+ delete $data{'password'} if ($data{password} eq "");
+ }
+ foreach (keys %data)
+ { push @parameters,"$_ = ".$dbh->quote($data{$_}) if ($_ ne 'borrowernumber' and $_ ne 'flags' and $hashborrowerfields{$_}); }
+ $query .= join (',',@parameters) . "\n WHERE borrowernumber=? \n";
+ $debug and print STDERR "$query (executed w/ arg: $data{'borrowernumber'})";
+ $sth = $dbh->prepare($query);
+ $sth->execute($data{'borrowernumber'});
+ $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' ) {
-
+ my $borrowercategory= GetBorrowercategory( $data{'category_type'} );
+ if ( $borrowercategory->{'category_type'} eq ('A' || 'S') ) {
# is adult check guarantees;
- updateguarantees(%data);
-
+ UpdateGuarantees(%data);
}
&logaction(C4::Context->userenv->{'number'},"MEMBERS","MODIFY",$data{'borrowernumber'},"")
if C4::Context->preference("BorrowersLog");
}
-sub newmember {
+
+=head2
+
+=item AddMember
+
+ $borrowernumber = &AddMember(%borrower);
+
+insert new borrower into table
+Returns the borrowernumber
+
+=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{'dateofbirth'} = format_date_in_iso( $data{'dateofbirth'} );
+ $data{'dateenrolled'} = format_date_in_iso( $data{'dateenrolled'});
+ $data{'dateexpiry'} = format_date_in_iso( $data{'dateexpiry'} );
+ # This query should be rewritten to use "?" at execute.
my $query =
- "insert into borrowers set cardnumber="
- . $dbh->quote( $data{'cardnumber'} )
- . ",surname="
- . $dbh->quote( $data{'surname'} )
- . ",firstname="
- . $dbh->quote( $data{'firstname'} )
- . ",title="
- . $dbh->quote( $data{'title'} )
- . ",othernames="
- . $dbh->quote( $data{'othernames'} )
- . ",initials="
- . $dbh->quote( $data{'initials'} )
- . ",streetnumber="
- . $dbh->quote( $data{'streetnumber'} )
- . ",streettype="
- . $dbh->quote( $data{'streettype'} )
- . ",address="
- . $dbh->quote( $data{'address'} )
- . ",address2="
- . $dbh->quote( $data{'address2'} )
- . ",zipcode="
- . $dbh->quote( $data{'zipcode'} )
- . ",city="
- . $dbh->quote( $data{'city'} )
- . ",phone="
- . $dbh->quote( $data{'phone'} )
- . ",email="
- . $dbh->quote( $data{'email'} )
- . ",mobile="
- . $dbh->quote( $data{'mobile'} )
- . ",phonepro="
- . $dbh->quote( $data{'phonepro'} )
- . ",opacnote="
- . $dbh->quote( $data{'opacnote'} )
- . ",guarantorid="
- . $dbh->quote( $data{'guarantorid'} )
- . ",dateofbirth="
- . $dbh->quote( $data{'dateofbirth'} )
- . ",branchcode="
- . $dbh->quote( $data{'branchcode'} )
- . ",categorycode="
- . $dbh->quote( $data{'categorycode'} )
- . ",dateenrolled="
- . $dbh->quote( $data{'dateenrolled'} )
- . ",contactname="
- . $dbh->quote( $data{'contactname'} )
- . ",borrowernotes="
- . $dbh->quote( $data{'borrowernotes'} )
- . ",dateexpiry="
- . $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'}, )
- . ",password="
- . $dbh->quote( $data{'password'} )
- . ",userid="
- . $dbh->quote( $data{'userid'} )
- . ",sort1="
- . $dbh->quote( $data{'sort1'} )
- . ",sort2="
- . $dbh->quote( $data{'sort2'} )
- . ",contacttitle="
- . $dbh->quote( $data{'contacttitle'} )
- . ",emailpro="
- . $dbh->quote( $data{'emailpro'} )
- . ",contactfirstname="
- . $dbh->quote( $data{'contactfirstname'} ) . ",sex="
- . $dbh->quote( $data{'sex'} ) . ",fax="
- . $dbh->quote( $data{'fax'} )
- . ",relationship="
- . $dbh->quote( $data{'relationship'} )
- . ",B_streetnumber="
- . $dbh->quote( $data{'B_streetnumber'} )
- . ",B_streettype="
- . $dbh->quote( $data{'B_streettype'} )
- . ",gonenoaddress="
- . $dbh->quote( $data{'gonenoaddress'} )
- . ",lost="
- . $dbh->quote( $data{'lost'} )
- . ",debarred="
- . $dbh->quote( $data{'debarred'} )
- . ",ethnicity="
- . $dbh->quote( $data{'ethnicity'} )
- . ",ethnotes="
- . $dbh->quote( $data{'ethnotes'} );
-
+ "insert into borrowers set cardnumber=" . $dbh->quote( $data{'cardnumber'} )
+ . ",surname=" . $dbh->quote( $data{'surname'} )
+ . ",firstname=" . $dbh->quote( $data{'firstname'} )
+ . ",title=" . $dbh->quote( $data{'title'} )
+ . ",othernames=" . $dbh->quote( $data{'othernames'} )
+ . ",initials=" . $dbh->quote( $data{'initials'} )
+ . ",streetnumber=". $dbh->quote( $data{'streetnumber'} )
+ . ",streettype=" . $dbh->quote( $data{'streettype'} )
+ . ",address=" . $dbh->quote( $data{'address'} )
+ . ",address2=" . $dbh->quote( $data{'address2'} )
+ . ",zipcode=" . $dbh->quote( $data{'zipcode'} )
+ . ",city=" . $dbh->quote( $data{'city'} )
+ . ",phone=" . $dbh->quote( $data{'phone'} )
+ . ",email=" . $dbh->quote( $data{'email'} )
+ . ",mobile=" . $dbh->quote( $data{'mobile'} )
+ . ",phonepro=" . $dbh->quote( $data{'phonepro'} )
+ . ",opacnote=" . $dbh->quote( $data{'opacnote'} )
+ . ",guarantorid=" . $dbh->quote( $data{'guarantorid'} )
+ . ",dateofbirth=" . $dbh->quote( $data{'dateofbirth'} )
+ . ",branchcode=" . $dbh->quote( $data{'branchcode'} )
+ . ",categorycode=" . $dbh->quote( $data{'categorycode'} )
+ . ",dateenrolled=" . $dbh->quote( $data{'dateenrolled'} )
+ . ",contactname=" . $dbh->quote( $data{'contactname'} )
+ . ",borrowernotes=" . $dbh->quote( $data{'borrowernotes'} )
+ . ",dateexpiry=" . $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'} )
+ . ",password=" . $dbh->quote( $data{'password'} )
+ . ",userid=" . $dbh->quote( $data{'userid'} )
+ . ",sort1=" . $dbh->quote( $data{'sort1'} )
+ . ",sort2=" . $dbh->quote( $data{'sort2'} )
+ . ",contacttitle=" . $dbh->quote( $data{'contacttitle'} )
+ . ",emailpro=" . $dbh->quote( $data{'emailpro'} )
+ . ",contactfirstname=" . $dbh->quote( $data{'contactfirstname'} )
+ . ",sex=" . $dbh->quote( $data{'sex'} )
+ . ",fax=" . $dbh->quote( $data{'fax'} )
+ . ",relationship=" . $dbh->quote( $data{'relationship'} )
+ . ",B_streetnumber=" . $dbh->quote( $data{'B_streetnumber'} )
+ . ",B_streettype=" . $dbh->quote( $data{'B_streettype'} )
+ . ",gonenoaddress=" . $dbh->quote( $data{'gonenoaddress'} )
+ . ",lost=" . $dbh->quote( $data{'lost'} )
+ . ",debarred=" . $dbh->quote( $data{'debarred'} )
+ . ",ethnicity=" . $dbh->quote( $data{'ethnicity'} )
+ . ",ethnotes=" . $dbh->quote( $data{'ethnotes'} );
my $sth = $dbh->prepare($query);
+ print "Executing SQL: $query\n";
$sth->execute;
$sth->finish;
$data{'borrowernumber'} = $dbh->{'mysql_insertid'};
return $data{'borrowernumber'};
}
+sub Check_Userid {
+ my ($uid,$member) = @_;
+ my $dbh = C4::Context->dbh;
+ # Make sure the userid chosen is unique and not theirs if non-empty. If it is not,
+ # Then we need to tell the user and have them create a new one.
+ my $sth =
+ $dbh->prepare(
+ "SELECT * FROM borrowers WHERE userid=? AND borrowernumber != ?");
+ $sth->execute( $uid, $member );
+ if ( ( $uid ne '' ) && ( my $row = $sth->fetchrow_hashref ) ) {
+ return 0;
+ }
+ else {
+ return 1;
+ }
+}
+
+
sub changepassword {
my ( $uid, $member, $digest ) = @_;
my $dbh = C4::Context->dbh;
#Then we need to tell the user and have them create a new one.
my $sth =
$dbh->prepare(
- "select * from borrowers where userid=? and borrowernumber != ?");
+ "SELECT * FROM borrowers WHERE userid=? AND borrowernumber != ?");
$sth->execute( $uid, $member );
- if ( ( $uid ne '' ) && ( $sth->fetchrow ) ) {
+ if ( ( $uid ne '' ) && ( my $row = $sth->fetchrow_hashref ) ) {
return 0;
}
else {
-
#Everything is good so we can update the information.
$sth =
$dbh->prepare(
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 {
- my (%data) = @_;
- my $dbh = C4::Context->dbh;
- my ( $count, $guarantees ) = findguarantees( $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 = 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
return $cardnumber;
}
-=head2 findguarantees
+=head2 GetGuarantees
- ($num_children, $children_arrayref) = &findguarantees($parent_borrno);
+ ($num_children, $children_arrayref) = &GetGuarantees($parent_borrno);
$child0_cardno = $children_arrayref->[0]{"cardnumber"};
$child0_borrno = $children_arrayref->[0]{"borrowernumber"};
-C<&findguarantees> takes a borrower number (e.g., that of a patron
+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<&findguarantees> returns two values: an integer giving the number of
+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 findguarantees {
+sub GetGuarantees {
my ($borrowernumber) = @_;
my $dbh = C4::Context->dbh;
my $sth =
$sth->execute($borrowernumber);
my @dat;
- while ( my $data = $sth->fetchrow_hashref ) {
- push @dat, $data;
- }
+ my $data = $sth->fetchall_arrayref({});
$sth->finish;
- return ( scalar(@dat), \@dat );
+ return ( scalar(@$data), $data );
}
-=head2 findguarantor
+=head2 UpdateGuarantees
- $guarantor = &findguarantor($borrower_no);
- $guarantor_cardno = $guarantor->{"cardnumber"};
- $guarantor_surname = $guarantor->{"surname"};
- ...
+ &UpdateGuarantees($parent_borrno);
+
-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<&findguarantor> returns a reference-to-hash. Its keys are the fields
-from the C<borrowers> database table;
+C<&UpdateGuarantees> borrower data for an adulte and updates all the guarantees
+with the modified information
=cut
#'
-sub findguarantor {
- my ($borrowernumber) = @_;
- my $dbh = C4::Context->dbh;
- my $sth =
- $dbh->prepare("select guarantorid from borrowers where borrowernumber=?");
- $sth->execute($borrowernumber);
- my $data = $sth->fetchrow_hashref;
- $sth->finish;
- $sth = $dbh->prepare("Select * from borrowers where borrowernumber=?");
- $sth->execute( $data->{'guarantorid'} );
- $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;
- }
+sub UpdateGuarantees {
+ my (%data) = @_;
+ my $dbh = C4::Context->dbh;
+ my ( $count, $guarantees ) = GetGuarantees( $data{'borrowernumber'} );
+ for ( my $i = 0 ; $i < $count ; $i++ ) {
- my $sth = $dbh->prepare($query);
- $sth->execute(@bind);
- my @results;
- my $cnt = $sth->rows;
- while ( my $data = $sth->fetchrow_hashref ) {
- push( @results, $data );
+ # 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 = 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;
}
-
- # $sth->execute;
- $sth->finish;
- return ( $cnt, \@results );
}
+=head2 GetPendingIssues
-=head2 borrissues
-
- ($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
=cut
#'
-sub borrissues {
+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.biblionumber=biblio.biblionumber
- and issues.returndate is NULL order by date_due"
+ "SELECT * FROM issues
+ LEFT JOIN items ON issues.itemnumber=items.itemnumber
+ LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ LEFT JOIN biblioitems ON items.biblioitemnumber=biblioitems.biblioitemnumber
+ WHERE
+ borrowernumber=?
+ AND returndate IS NULL
+ ORDER BY issues.issuedate"
);
$sth->execute($borrowernumber);
- my @result;
- while ( my $data = $sth->fetchrow_hashref ) {
- push @result, $data;
+ my $data = $sth->fetchall_arrayref({});
+ my $today = POSIX::strftime("%Y%m%d", localtime);
+ foreach( @$data ) {
+ my $datedue = $_->{'date_due'};
+ $datedue =~ s/-//g;
+ if ( $datedue < $today ) {
+ $_->{'overdue'} = 1;
+ }
}
$sth->finish;
- return ( scalar(@result), \@result );
+ 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.
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
=cut
#'
-sub allissues {
+sub GetAllIssues {
my ( $borrowernumber, $order, $limit ) = @_;
#FIXME: sanity-check order and limit
my $dbh = C4::Context->dbh;
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
- items.biblionumber=biblio.biblionumber order by $order";
+ "Select *,items.timestamp AS itemstimestamp from
+ issues
+ LEFT JOIN items on items.itemnumber=issues.itemnumber
+ LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ LEFT JOIN biblioitems ON items.biblioitemnumber=biblioitems.biblioitemnumber
+ where borrowernumber=?
+ order by $order";
if ( $limit != 0 ) {
$query .= " limit $limit";
}
# 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
+ my $query2 = "SELECT * FROM oldissues
+ LEFT JOIN items ON items.itemnumber=oldissues.itemnumber
+ LEFT JOIN biblio ON items.biblionumber=biblio.biblionumber
+ LEFT JOIN biblioitems ON items.biblioitemnumber=biblioitems.biblioitemnumber
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;
return ( $i, \@result );
}
-=head2 getboracctrecord
- ($count, $acctlines, $total) = &getboracctrecord($env, $borrowernumber);
-
-Looks up accounting data for the patron with the given borrowernumber.
+=head2 GetMemberAccountRecords
-C<$env> is ignored.
+ ($total, $acctlines, $count) = &GetMemberAccountRecords($borrowernumber);
-(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<&GetMemberAccountRecords> 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
=cut
#'
-sub getboracctrecord {
- my ( $env, $params ) = @_;
+sub GetMemberAccountRecords {
+ 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"
- );
-
- $sth->execute( $params->{'borrowernumber'} );
+ my $strsth = qq(
+ SELECT *
+ FROM accountlines
+ WHERE borrowernumber=?);
+ my @bind = ($borrowernumber);
+ if ($date && $date ne ''){
+ $strsth.=" AND date < ? ";
+ push(@bind,$date);
+ }
+ $strsth.=" ORDER BY date desc,timestamp DESC";
+ my $sth= $dbh->prepare( $strsth );
+ $sth->execute( @bind );
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 ( $numlines, \@acctlines, $total );
+ return ( $total, \@acctlines,$numlines);
}
=head2 GetBorNotifyAcctRecord
- ($count, $acctlines, $total) = &GetBorNotifyAcctRecord($env, $params,$notifyid);
+ ($count, $acctlines, $total) = &GetBorNotifyAcctRecord($params,$notifyid);
Looks up accounting data for the patron with the given borrowernumber per file number.
-C<$env> is ignored.
-
(FIXME - I'm not at all sure what this is about.)
C<&GetBorNotifyAcctRecord> returns a three-element array. C<$acctlines> is a
=cut
sub GetBorNotifyAcctRecord {
- my ( $env, $params, $notifyid ) = @_;
+ my ( $borrowernumber, $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 $sth = $dbh->prepare(
+ "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
+ ");
+ $sth->execute( $borrowernumber, $notifyid );
my $total = 0;
while ( my $data = $sth->fetchrow_hashref ) {
$acctlines[$numlines] = $data;
$total += $data->{'amountoutstanding'};
}
$sth->finish;
- return ( $numlines, \@acctlines, $total );
+ return ( $total, \@acctlines, $numlines );
}
=head2 checkuniquemember (OUEST-PROVENCE)
}
}
+sub checkcardnumber {
+ my ($cardnumber,$borrowernumber) = @_;
+ my $dbh = C4::Context->dbh;
+ my $query = "SELECT * FROM borrowers WHERE cardnumber=?";
+ $query .= " AND borrowernumber <> ?" if ($borrowernumber);
+ my $sth = $dbh->prepare($query);
+ if ($borrowernumber) {
+ $sth->execute($cardnumber,$borrowernumber);
+ } else {
+ $sth->execute($cardnumber);
+ }
+ if (my $data= $sth->fetchrow_hashref()){
+ return 1;
+ }
+ else {
+ return 0;
+ }
+ $sth->finish();
+}
+
+
=head2 getzipnamecity (OUEST-PROVENCE)
take all info from table city for the fields city and zip
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,fax from borrowers where borrowernumber =? "
- );
- $sth->execute($borrowerid);
- my $guarantor_data = $sth->fetchrow_hashref;
- $sth->finish;
- return $guarantor_data;
-}
=head2 getdcity (OUEST-PROVENCE)
return $data;
}
-=head2 getcategorytype (OUEST-PROVENCE)
-
-check for the category_type with categorycode
-and return the category_type
-=cut
+=head2 GetExpiryDate
-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 =
$sth->execute($categorycode);
my ($enrolmentperiod) = $sth->fetchrow;
$enrolmentperiod = 12 unless ($enrolmentperiod);
-# warn "Avant format_date_in_iso :".$dateenrolled;
-# $dateenrolled=format_date_in_iso($dateenrolled);
-# warn "Apres format_date_in_iso :".$dateenrolled;
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]);
return ( \@codes, \%labels );
}
-=head2 getborrowercategory
+=head2 GetBorrowercategory
- $description,$dateofbirthrequired,$upperagelimit,$category_type = &getborrowercategory($categorycode);
+ $hashref = &GetBorrowercategory($categorycode);
Given the borrower's category code, the function returns the corresponding
-description , dateofbirthrequired , upperagelimit and category type 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,dateofbirthrequired,upperagelimit,category_type FROM categories WHERE categorycode = ?"
+"SELECT description,dateofbirthrequired,upperagelimit,category_type
+ FROM categories
+ WHERE categorycode = ?"
);
$sth->execute($catcode);
- my ( $description, $dateofbirthrequired, $upperagelimit, $category_type ) =
- $sth->fetchrow();
+ my $data =
+ $sth->fetchrow_hashref;
$sth->finish();
- return ( $description, $dateofbirthrequired, $upperagelimit,
- $category_type );
+ return $data;
} # sub getborrowercategory
=head2 ethnicitycategories
return $data->{'name'};
} # sub fixEthnicity
-=head2 get_age
+=head2 GetAge
- $dateofbirth,$date = &get_age($date);
+ $dateofbirth,$date = &GetAge($date);
this function return the borrowers age with the value of dateofbirth
=cut
#'
-sub get_age {
+sub GetAge{
my ( $date, $date_ref ) = @_;
if ( not defined $date_ref ) {
} # sub add_member_orgs
-=head2 GetBorrowersFromSurname
-
-=over 4
+=head2 GetCities (OUEST-PROVENCE)
-\@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;
-
- while ( my $data = $sth->fetchrow_hashref ) {
- push( @results, $data );
- $count++;
- }
- $sth->finish;
- return ( $count, \@results );
-}
-
-=head2 citycaracteristiques (OUEST-PROVENCE)
-
- ($id_cityarrayref, $city_hashref) = &citycaracteristic();
+ ($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
=cut
-sub DeleteBorrower {
+sub MoveMemberToDeleted {
my ($member) = @_;
my $dbh = C4::Context->dbh;
my $query;
my @data = $sth->fetchrow_array;
$sth->finish;
$sth =
- $dbh->prepare( "Insert into deletedborrowers values ("
+ $dbh->prepare( "INSERT INTO deletedborrowers VALUES ("
. ( "?," x ( scalar(@data) - 1 ) )
. "?)" );
$sth->execute(@data);
$sth->finish;
- $query = qq|DELETE
- FROM borrowers
- WHERE borrowernumber=?|;
- $sth = $dbh->prepare($query);
- $sth->execute($member);
- $sth->finish;
- $query = qq|DELETE
- FROM reserves
- WHERE borrowernumber=?|;
- $sth = $dbh->prepare($query);
- $sth->execute($member);
- $sth->finish;
-
- # logging to action_log
- &logaction(C4::Context->userenv->{'number'},"MEMBERS","DELETE",$member,"")
- if C4::Context->preference("BorrowersLog");
}
-=head2 DelBorrowerCompletly
+=head2 DelMember
-DelBorrowerCompletly($borrowernumber);
+DelMember($borrowernumber);
This function remove directly a borrower whitout writing it on deleteborrower.
++ Deletes reserves for the borrower
=cut
-sub DelBorrowerCompletly {
+sub DelMember {
my $dbh = C4::Context->dbh;
my $borrowernumber = shift;
- return unless $borrowernumber; # date is mandatory.
- my $query = "
- DELETE *
+ warn "in delmember with $borrowernumber";
+ 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);
+ $sth = $dbh->prepare($query);
$sth->execute($borrowernumber);
+ $sth->finish;
+ &logaction(C4::Context->userenv->{'number'},"MEMBERS","DELETE",$borrowernumber,"")
+ if C4::Context->preference("BorrowersLog");
return $sth->rows;
}
-=head2 member_reregistration (OUEST-PROVENCE)
-
-automatic reregistration in borrowers table
-with dateexpiry .
+=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 GetMembeReregistration {
- my ( $categorycode, $borrowerid ) = @_;
+sub ExtendMemberSubscriptionTo {
+ my ( $borrowerid,$date) = @_;
my $dbh = C4::Context->dbh;
- my ( $sec, $min, $hour, $mday, $mon, $year, $wday, $yday, $isdst ) =
- localtime(time);
- $mon++;
- $year = $year + 1900;
- if ( $mon < '10' ) {
- $mon = "0" . $mon;
- }
- if ( $mday < '10' ) {
- $mday = "0" . $mday;
- }
- my $today = sprintf("%04d-%02d-%02d",$year,$mon,$mday);
- my $dateexpiry = calcexpirydate( $categorycode, $today );
- my $query = qq| UPDATE borrowers
- SET dateexpiry='$dateexpiry'
- WHERE borrowernumber='$borrowerid'|;
- my $sth = $dbh->prepare($query);
- $sth->execute;
- $sth->finish;
- return $dateexpiry;
+ unless ($date){
+ $date=POSIX::strftime("%Y-%m-%d",localtime(time));
+ my $borrower = GetMember($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)
sub GetRoadTypes {
my $dbh = C4::Context->dbh;
- my $query = qq|SELECT roadtypeid,road_type
- FROM roadtype
- ORDER BY road_type|;
+ my $query = qq|
+SELECT roadtypeid,road_type
+FROM roadtype
+ORDER BY road_type|;
my $sth = $dbh->prepare($query);
$sth->execute();
my %roadtype;
-=head2 GetBorrowersTitles (OUEST-PROVENCE)
+=head2 GetTitles (OUEST-PROVENCE)
- ($borrowertitle)= &GetBorrowersTitles();
+ ($borrowertitle)= &GetTitles();
Looks up the different title . Returns array with all borrowers title
=cut
-sub GetBorrowersTitles {
+sub GetTitles {
my @borrowerTitle = split /,|\|/,C4::Context->preference('BorrowersTitles');
unshift( @borrowerTitle, "" );
return ( \@borrowerTitle);
sub GetRoadTypeDetails {
my ($roadtypeid) = @_;
my $dbh = C4::Context->dbh;
- my $query = qq|SELECT road_type
- FROM roadtype
- WHERE roadtypeid=?|;
+ my $query = qq|
+SELECT road_type
+FROM roadtype
+WHERE roadtypeid=?|;
my $sth = $dbh->prepare($query);
$sth->execute($roadtypeid);
my $roadtype = $sth->fetchrow;