X-Git-Url: http://git.rot13.org/?a=blobdiff_plain;f=C4%2FMembers.pm;h=508618d45bce1397847dad0124c943af06c6daea;hb=9a32fe85c27196de96770eea8f7a34bd24d706a6;hp=e03d1d456995635c816e60eba151be20ef08f6c5;hpb=58249a980712f71758fe6c431478f34eb9ad9e0f;p=koha.git diff --git a/C4/Members.pm b/C4/Members.pm index e03d1d4569..508618d45b 100644 --- a/C4/Members.pm +++ b/C4/Members.pm @@ -1,5 +1,3 @@ -# -*- tab-width: 8 -*- - package C4::Members; # Copyright 2000-2003 Katipo Communications @@ -24,12 +22,14 @@ 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); +our ($VERSION,@ISA,@EXPORT,@EXPORT_OK); $VERSION = do { my @v = '$Revision$' =~ /\d+/g; shift(@v) . "." . join( "_", map { sprintf "%03d", $_ } @v ); }; @@ -51,40 +51,92 @@ This module contains routines for adding, modifying and deleting members/patrons =cut -#' +@ISA = qw(Exporter); + +#Get data +push @EXPORT, qw( + &SearchMember + &GetMemberDetails + &GetMember + + &GetGuarantees + + &GetMemberIssuesAndFines + &GetPendingIssues + &GetAllIssues + + &get_institutions + &getzipnamecity + &getidcity + + &GetAge + &GetCities + &GetRoadTypes + &GetRoadTypeDetails + &GetSortDetails + &GetTitles + + &GetMemberAccountRecords + &GetBorNotifyAcctRecord + + &GetborCatFromCatType + &GetBorrowercategory + + + &GetBorrowersWhoHaveNotBorrowedSince + &GetBorrowersWhoHaveNeverBorrowed + &GetBorrowersWithIssuesHistoryOlderThan + + &GetExpiryDate +); + +#Modify data +push @EXPORT, qw( + &ModMember + &changepassword +); + +#Delete data +push @EXPORT, qw( + &DelMember +); -@ISA = qw(Exporter); -@EXPORT = qw(); +#Insert data +push @EXPORT, qw( + &AddMember + &add_member_orgs + &MoveMemberToDeleted + &ExtendMemberSubscriptionTo +); -@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 +#Check data +push @EXPORT, qw( + &checkuniquemember + &checkuserpassword &fixEthnicity - ðnicitycategories get_institutions add_member_orgs - &get_age + ðnicitycategories + &fixup_cardnumber + &checkcardnumber ); +=item SearchMember -=item BornameSearch - - ($count, $borrowers) = &BornameSearch($env, $searchstring, $type); + ($count, $borrowers) = &SearchMember($searchstring, $type,$category_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. +$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<&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 table in the Koha database. C<$count> is the number of elements in C<$borrowers>. @@ -94,8 +146,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 SearchMember { + my ($searchstring, $orderby, $type,$category_type ) = @_; my $dbh = C4::Context->dbh; my $query = ""; my $count; @@ -105,33 +157,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 - .= <execute(@bind); my @results; - my $cnt = $sth->rows; - while ( my $data = $sth->fetchrow_hashref ) { - push( @results, $data ); - } + my $data = $sth->fetchall_arrayref({}); - # $sth->execute; $sth->finish; - return ( $cnt, \@results ); + return ( scalar(@$data), $data ); } -=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 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{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 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 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{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{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{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) = GetMemberAccountRecords( $borrowernumber); + $borrower->{'amountoutstanding'} = $amount; + my $flags = patronflags( $borrower); + 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); + + $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) = GetMemberAccountRecords( $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; + } + my @itemswaiting = GetReservesFromBorrowernumber( $patroninformation->{'borrowernumber'},'W' ); + my $nowaiting = scalar @itemswaiting; + if ( $nowaiting > 0 ) { + my %flaginfo; + $flaginfo{'message'} = "Reserved items available"; + $flaginfo{'itemlist'} = \@itemswaiting; + $flags{'WAITING'} = \%flaginfo; } - return undef; + 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 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 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. @@ -269,10 +524,11 @@ the total fine currently due by the borrower. =cut #' -sub borrdata2 { - my ( $env, $bornum ) = @_; +sub GetMemberIssuesAndFines { + 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 +538,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 +555,145 @@ 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 "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 +=head2 + +=item ModMember + + &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 = &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{'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 { @@ -533,7 +815,6 @@ sub changepassword { return 0; } else { - #Everything is good so we can update the information. $sth = $dbh->prepare( @@ -541,37 +822,12 @@ 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 { - 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 = -"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 $sth3 = $dbh->prepare($guaquery); - $sth3->execute; - $sth3->finish; - } -} -################################################################################ =item fixup_cardnumber @@ -657,165 +913,76 @@ 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 - - $guarantor = &findguarantor($borrower_no); - $guarantor_cardno = $guarantor->{"cardnumber"}; - $guarantor_surname = $guarantor->{"surname"}; - ... - -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 database table; - -=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. +=head2 GetGuarantees -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. + ($num_children, $children_arrayref) = &GetGuarantees($parent_borrno); + $child0_cardno = $children_arrayref->[0]{"cardnumber"}; + $child0_borrno = $children_arrayref->[0]{"borrowernumber"}; -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 table in the Koha database. -C<$count> is the number of elements in C<$borrowers>. +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). -return all info from guarantor =>only category_type A +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 #' -#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 - .= <prepare($query); - $sth->execute(@bind); - my @results; - my $cnt = $sth->rows; - while ( my $data = $sth->fetchrow_hashref ) { - push( @results, $data ); - } +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); - # $sth->execute; + my @dat; + my $data = $sth->fetchall_arrayref({}); $sth->finish; - return ( $cnt, \@results ); + return ( scalar(@$data), $data ); } -=item NewBorrowerNumber +=head2 UpdateGuarantees - $num = &NewBorrowerNumber(); + &UpdateGuarantees($parent_borrno); + -Allocates a new, unused borrower number, and returns it. +C<&UpdateGuarantees> borrower data for an adulte and updates all the guarantees +with the modified information =cut #' -# 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 { +sub UpdateGuarantees { + my (%data) = @_; my $dbh = C4::Context->dbh; - my $sth = $dbh->prepare("Select max(borrowernumber) from borrowers"); - $sth->execute; - my $data = $sth->fetchrow_hashref; - $sth->finish; - $data->{'max(borrowernumber)'}++; - return ( $data->{'max(borrowernumber)'} ); -} + my ( $count, $guarantees ) = GetGuarantees( $data{'borrowernumber'} ); + for ( my $i = 0 ; $i < $count ; $i++ ) { -=head2 borrissues + # 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; + } +} +=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, C, and C tables in the Koha database. C<$count> is the number of elements in @@ -824,27 +991,37 @@ C<$issues>. =cut #' -sub borrissues { - my ($bornum) = @_; - 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" +sub GetPendingIssues { + my ($borrowernumber) = @_; + my $dbh = C4::Context->dbh; + + my $sth = $dbh->prepare( + "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.date_due" ); - $sth->execute($bornum); - my @result; - while ( my $data = $sth->fetchrow_hashref ) { - push @result, $data; + $sth->execute($borrowernumber); + 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. @@ -855,7 +1032,7 @@ C, or C 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, C, C, and C tables of the Koha database. C<$count> is the number of @@ -864,13 +1041,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 $count=0; - 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 @@ -881,55 +1059,98 @@ 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++; + $count++; } - # get all issued items for bornum from oldissues table + # 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 + 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($bornum); - - while (my $data2=$sth2->fetchrow_hashref){ - $result[$i]=$data2; - $i++; - } - $sth2->finish; + 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 GetMemberAccountRecords + + ($total, $acctlines, $count) = &GetMemberAccountRecords($borrowernumber); Looks up accounting data for the patron with the given borrowernumber. -C<$env> is ignored. +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 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 GetMemberAccountRecords { + my ($borrowernumber,$date) = @_; + my $dbh = C4::Context->dbh; + my @acctlines; + my $numlines = 0; + 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 ) { + $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<&getboracctrecord> returns a three-element array. C<$acctlines> is a +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 table in the Koha database. C<$count> is the number of elements in C<$acctlines>. C<$total> is the @@ -937,38 +1158,30 @@ total amount outstanding for all of the account lines. =cut -#' -sub getboracctrecord { - my ( $env, $params ) = @_; +sub GetBorNotifyAcctRecord { + my ( $borrowernumber, $notifyid ) = @_; 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 $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); - # print $query; - $sth->execute( $params->{'borrowernumber'} ); + $sth->execute( $borrowernumber, $notifyid ); 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 checkuniquemember (OUEST-PROVENCE) @@ -1022,6 +1235,22 @@ sub checkuniquemember { } } +sub checkcardnumber { + my ($cardnumber) = @_; + my $dbh = C4::Context->dbh; + my $query = "SELECT * FROM borrowers WHERE cardnumber=?"; + my $sth = $dbh->prepare($query); + $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 @@ -1040,30 +1269,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 { @@ -1075,26 +1285,14 @@ sub getidcity { 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 = @@ -1103,8 +1301,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) @@ -1129,9 +1328,9 @@ sub checkuserpassword { } -=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 @@ -1141,14 +1340,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; @@ -1160,25 +1366,29 @@ 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 @@ -1220,9 +1430,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); @@ -1231,40 +1441,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 #' @@ -1299,10 +1507,349 @@ 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 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; + 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 = ? + "; + $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 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 = GetMember($borrowerid,'borrowernumber'); + $date = GetExpiryDate( $borrower->{'categorycode'}, $date ); + } + my $sth = $dbh->do(<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 GetTitles (OUEST-PROVENCE) + + ($borrowertitle)= &GetTitles(); + +Looks up the different title . Returns array with all borrowers title + +=cut + +sub GetTitles { + 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