use C4::Reserves;
use C4::Accounts;
use C4::Biblio;
+use C4::SQLHelper qw(InsertInTable UpdateInTable SearchInTable);
+use C4::Members::Attributes qw(SearchIdMatchingAttribute);
our ($VERSION,@ISA,@EXPORT,@EXPORT_OK,$debug);
@ISA = qw(Exporter);
#Get data
push @EXPORT, qw(
+ &Search
&SearchMember
&GetMemberDetails
&GetMember
&PutPatronImage
&RmPatronImage
+ &IsMemberBlocked
&GetMemberAccountRecords
&GetBorNotifyAcctRecord
&GetBorrowersWithIssuesHistoryOlderThan
&GetExpiryDate
+
+ &AddMessage
+ &DeleteMessage
+ &GetMessages
+ &GetMessagesCount
);
#Modify data
&AddMember
&add_member_orgs
&MoveMemberToDeleted
- &ExtendMemberSubscriptionTo
+ &ExtendMemberSubscriptionTo
);
#Check data
#'
#used by member enquiries from the intranet
-#called by member.pl and circ/circulation.pl
sub SearchMember {
my ($searchstring, $orderby, $type,$category_type,$filter,$showallbranches ) = @_;
my $dbh = C4::Context->dbh;
if (@$data){
return ( scalar(@$data), $data );
}
- $sth->finish;
if ( $type eq "simple" ) # simple search for one letter only
{
}
$query = $query . ") OR cardnumber LIKE ? ";
push( @bind, $searchstring );
- if (C4::Context->preference('ExtendedPatronAttributes')) {
- $query .= "OR borrowernumber IN (
-SELECT borrowernumber
-FROM borrower_attributes
-JOIN borrower_attribute_types USING (code)
-WHERE staff_searchable = 1
-AND attribute like ?
-)";
- push (@bind, $searchstring);
- }
$query .= "order by $orderby";
# FIXME - .= <<EOT;
my @results;
$data = $sth->fetchall_arrayref({});
- $sth->finish;
return ( scalar(@$data), $data );
}
+sub Search {
+ my ($filter,$orderby, $limit, $columns_out, $search_on_fields,$searchtype) = @_;
+ my @filters;
+ if (ref($filter) eq "ARRAY"){
+ push @filters,@$filter;
+ }
+ else {
+ push @filters,$filter;
+ }
+ if (C4::Context->preference('ExtendedPatronAttributes')) {
+ my $matching_records = C4::Members::Attributes::SearchIdMatchingAttribute($filter);
+ push @filters,@$matching_records;
+ }
+ $searchtype||="start_with";
+ my $data=SearchInTable("borrowers",\@filters,$orderby,$limit,$columns_out,$search_on_fields,$searchtype);
+
+ return ( $data );
+}
+
=head2 GetMemberDetails
($borrower) = &GetMemberDetails($borrowernumber, $cardnumber);
# Patron's card was reported lost
}
-Each flag has a C<message> key, giving a human-readable explanation of
-the flag. If the state of a flag means that the patron should not be
+If the state of a flag means that the patron should not be
allowed to borrow any more books, then it will have a C<noissues> key
with a true value.
-The possible flags are:
-
-=head3 CHARGES
-
-=over 4
-
-=item Shows the patron's credit or debt, if any.
-
-=back
-
-=head3 GNA
-
-=over 4
-
-=item (Gone, no address.) Set if the patron has left without giving a
-forwarding address.
-
-=back
-
-=head3 LOST
-
-=over 4
-
-=item Set if the patron's card has been reported as lost.
-
-=back
-
-=head3 DBARRED
-
-=over 4
-
-=item Set if the patron has been debarred.
-
-=back
-
-=head3 NOTES
-
-=over 4
-
-=item Any additional notes about the patron.
-
-=back
-
-=head3 ODUES
-
-=over 4
-
-=item Set if the patron has overdue items. This flag has several keys:
-
-C<$flags-E<gt>{ODUES}{itemlist}> is a reference-to-array listing the
-overdue items. Its elements are references-to-hash, each describing an
-overdue item. The keys are selected fields from the issues, biblio,
-biblioitems, and items tables of the Koha database.
-
-C<$flags-E<gt>{ODUES}{itemlist}> is a string giving a text listing of
-the overdue items, one per line.
-
-=back
-
-=head3 WAITING
-
-=over 4
-
-=item Set if any items that the patron has reserved are available.
-
-C<$flags-E<gt>{WAITING}{itemlist}> is a reference-to-array listing the
-available items. Each element is a reference-to-hash whose keys are
-fields from the reserves table of the Koha database.
-
-=back
+See patronflags for more details.
C<$borrower-E<gt>{authflags}> is a hash giving more detailed information
about the top-level permissions flags set for the borrower. For example,
$accessflagshash->{$flag} = 1;
}
}
- $sth->finish;
$borrower->{'flags'} = $flags;
$borrower->{'authflags'} = $accessflagshash;
=head2 patronflags
- Not exported
+ $flags = &patronflags($patron);
- NOTE!: If you change this function, be sure to update the POD for
- &GetMemberDetails.
+ This function is not exported.
- $flags = &patronflags($patron);
+ The following will be set where applicable:
+ $flags->{CHARGES}->{amount} Amount of debt
+ $flags->{CHARGES}->{noissues} Set if debt amount >$5.00 (or syspref noissuescharge)
+ $flags->{CHARGES}->{message} Message -- deprecated
+
+ $flags->{CREDITS}->{amount} Amount of credit
+ $flags->{CREDITS}->{message} Message -- deprecated
+
+ $flags->{ GNA } Patron has no valid address
+ $flags->{ GNA }->{noissues} Set for each GNA
+ $flags->{ GNA }->{message} "Borrower has no valid address" -- deprecated
+
+ $flags->{ LOST } Patron's card reported lost
+ $flags->{ LOST }->{noissues} Set for each LOST
+ $flags->{ LOST }->{message} Message -- deprecated
+
+ $flags->{DBARRED} Set if patron debarred, no access
+ $flags->{DBARRED}->{noissues} Set for each DBARRED
+ $flags->{DBARRED}->{message} Message -- deprecated
+
+ $flags->{ NOTES }
+ $flags->{ NOTES }->{message} The note itself. NOT deprecated
- $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
+ $flags->{ ODUES } Set if patron has overdue books.
+ $flags->{ ODUES }->{message} "Yes" -- deprecated
+ $flags->{ ODUES }->{itemlist} ref-to-array: list of overdue books
+ $flags->{ ODUES }->{itemlisttext} Text list of overdue items -- deprecated
+
+ $flags->{WAITING} Set if any of patron's reserves are available
+ $flags->{WAITING}->{message} Message -- deprecated
+ $flags->{WAITING}->{itemlist} ref-to-array: list of available items
+
+=over 4
+
+C<$flags-E<gt>{ODUES}-E<gt>{itemlist}> is a reference-to-array listing the
+overdue items. Its elements are references-to-hash, each describing an
+overdue item. The keys are selected fields from the issues, biblio,
+biblioitems, and items tables of the Koha database.
+
+C<$flags-E<gt>{ODUES}-E<gt>{itemlisttext}> is a string giving a text listing of
+the overdue items, one per line. Deprecated.
+
+C<$flags-E<gt>{WAITING}-E<gt>{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
+
+All the "message" fields that include language generated in this function are deprecated,
+because such strings belong properly in the display layer.
+
+The "message" field that comes from the DB is OK.
=cut
+
+# TODO: use {anonymous => hashes} instead of a dozen %flaginfo
# FIXME rename this function.
sub patronflags {
my %flags;
my ($amount) = GetMemberAccountRecords( $patroninformation->{'borrowernumber'});
if ( $amount > 0 ) {
my %flaginfo;
- my $noissuescharge = C4::Context->preference("noissuescharge");
+ my $noissuescharge = C4::Context->preference("noissuescharge") || 5;
$flaginfo{'message'} = sprintf "Patron owes \$%.02f", $amount;
- $flaginfo{'amount'} = sprintf "%.02f",$amount;
+ $flaginfo{'amount'} = sprintf "%.02f", $amount;
if ( $amount > $noissuescharge ) {
$flaginfo{'noissues'} = 1;
}
elsif ( $amount < 0 ) {
my %flaginfo;
$flaginfo{'message'} = sprintf "Patron has credit of \$%.02f", -$amount;
+ $flaginfo{'amount'} = sprintf "%.02f", $amount;
$flags{'CREDITS'} = \%flaginfo;
}
if ( $patroninformation->{'gonenoaddress'}
&& $patroninformation->{'borrowernotes'} )
{
my %flaginfo;
- $flaginfo{'message'} = "$patroninformation->{'borrowernotes'}";
+ $flaginfo{'message'} = $patroninformation->{'borrowernotes'};
$flags{'NOTES'} = \%flaginfo;
}
- my ( $odues, $itemsoverdue ) =
- checkoverdues( $patroninformation->{'borrowernumber'}, $dbh );
+ my ( $odues, $itemsoverdue ) = checkoverdues($patroninformation->{'borrowernumber'});
if ( $odues > 0 ) {
my %flaginfo;
$flaginfo{'message'} = "Yes";
@$itemsoverdue )
{
$flaginfo{'itemlisttext'} .=
- "$_->{'date_due'} $_->{'barcode'} $_->{'title'} \n";
+ "$_->{'date_due'} $_->{'barcode'} $_->{'title'} \n"; # newline is display layer
}
$flags{'ODUES'} = \%flaginfo;
}
=head2 GetMember
- $borrower = &GetMember($information, $type);
+ $borrower = &GetMember(%information);
Looks up information about a patron (borrower) by either card number
,firstname, or borrower number, depending on $type value.
#'
sub GetMember {
- my ( $information, $type ) = @_;
+ my ( %information ) = @_;
my $dbh = C4::Context->dbh;
my $sth;
my $select = "
FROM borrowers
LEFT JOIN categories on borrowers.categorycode=categories.categorycode
";
- if (defined($type) and ( $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;
- ($data) and return ($data);
+ $select.=" WHERE ".join(" AND ",map {"$_ = ?"}keys %information);
+ $select=~s/AND $//;
+ $debug && warn $select, " ",values %information;
+ $sth = $dbh->prepare("$select");
+ $sth->execute(map{$information{$_}} keys %information);
+ my $data = $sth->fetchall_arrayref({});
+ return undef if (scalar(@$data)==0);
+ if (scalar(@$data)==1) {return $$data[0];}
+ ($data) and return $data;
+}
- if (defined($type) and ($type eq 'cardnumber' || $type eq 'firstname')) { # otherwise, try with firstname
- $sth = $dbh->prepare("$select WHERE firstname like ?");
- $sth->execute($information);
- $data = $sth->fetchrow_hashref;
- ($data) and return ($data);
- }
- return undef;
+
+=head2 IsMemberBlocked
+
+=over 4
+
+my $blocked = IsMemberBlocked( $borrowernumber );
+
+return the status, and the number of day or documents, depends his punishment
+
+return :
+-1 if the user have overdue returns
+1 if the user is punished X days
+0 if the user is authorised to loan
+
+=back
+
+=cut
+
+sub IsMemberBlocked {
+ my $borrowernumber = shift;
+ my $dbh = C4::Context->dbh;
+ # if he have late issues
+ my $sth = $dbh->prepare(
+ "SELECT COUNT(*) as latedocs
+ FROM issues
+ WHERE borrowernumber = ?
+ AND date_due < now()"
+ );
+ $sth->execute($borrowernumber);
+ my $latedocs = $sth->fetchrow_hashref->{'latedocs'};
+
+ return (-1, $latedocs) if $latedocs > 0;
+
+ my $strsth=qq{
+ SELECT
+ ADDDATE(returndate, finedays * DATEDIFF(returndate,date_due) ) AS blockingdate,
+ DATEDIFF(ADDDATE(returndate, finedays * DATEDIFF(returndate,date_due)),NOW()) AS blockedcount
+ FROM old_issues
+ };
+ # or if he must wait to loan
+ if(C4::Context->preference("item-level_itypes")){
+ $strsth.=
+ qq{ LEFT JOIN items ON (items.itemnumber=old_issues.itemnumber)
+ LEFT JOIN issuingrules ON (issuingrules.itemtype=items.itype)}
+ }else{
+ $strsth .=
+ qq{ LEFT JOIN items ON (items.itemnumber=old_issues.itemnumber)
+ LEFT JOIN biblioitems ON (biblioitems.biblioitemnumber=items.biblioitemnumber)
+ LEFT JOIN issuingrules ON (issuingrules.itemtype=biblioitems.itemtype) };
+ }
+ $strsth.=
+ qq{ WHERE finedays IS NOT NULL
+ AND date_due < returndate
+ AND borrowernumber = ?
+ ORDER BY blockingdate DESC, blockedcount DESC
+ LIMIT 1};
+ $sth=$dbh->prepare($strsth);
+ $sth->execute($borrowernumber);
+ my $row = $sth->fetchrow_hashref;
+ my $blockeddate = $row->{'blockeddate'};
+ my $blockedcount = $row->{'blockedcount'};
+
+ return (1, $blockedcount) if $blockedcount > 0;
+
+ return 0
}
=head2 GetMemberIssuesAndFines
my $sth = $dbh->prepare($query);
$sth->execute($borrowernumber);
my $issue_count = $sth->fetchrow_arrayref->[0];
- $sth->finish;
$sth = $dbh->prepare(
"SELECT COUNT(*) FROM issues
);
$sth->execute($borrowernumber);
my $overdue_count = $sth->fetchrow_arrayref->[0];
- $sth->finish;
$sth = $dbh->prepare("SELECT SUM(amountoutstanding) FROM accountlines WHERE borrowernumber = ?");
$sth->execute($borrowernumber);
my $total_fines = $sth->fetchrow_arrayref->[0];
- $sth->finish;
return ($overdue_count, $issue_count, $total_fines);
}
=back
=cut
-
sub ModMember {
my (%data) = @_;
- my $dbh = C4::Context->dbh;
- my $iso_re = C4::Dates->new()->regexp('iso');
- foreach (qw(dateofbirth dateexpiry dateenrolled)) {
- if (my $tempdate = $data{$_}) { # assignment, not comparison
- ($tempdate =~ /$iso_re/) and next; # Congatulations, you sent a valid ISO date.
- warn "ModMember given $_ not in ISO format ($tempdate)";
- my $tempdate2 = format_date_in_iso($tempdate);
- if (!$tempdate2 or $tempdate2 eq '0000-00-00') {
- warn "ModMember cannot convert '$tempdate' (from syspref to ISO)";
- next;
- }
- $data{$_} = $tempdate2;
- }
- }
- if (!$data{'dateofbirth'}){
- delete $data{'dateofbirth'};
- }
- my @columns = &columns;
- my %hashborrowerfields = (map {$_=>1} @columns);
- my $query = "UPDATE borrowers SET \n";
- my $sth;
- my @parameters;
-
# test to know if you must update or not the borrower password
if (exists $data{password}) {
if ($data{password} eq '****' or $data{password} eq '') {
$data{password} = md5_base64($data{password});
}
}
- my @badkeys;
- foreach (keys %data) {
- next if ($_ eq 'borrowernumber' or $_ eq 'flags');
- if ($hashborrowerfields{$_}){
- $query .= " $_=?, ";
- push @parameters,$data{$_};
- } else {
- push @badkeys, $_;
- delete $data{$_};
- }
- }
- (@badkeys) and warn scalar(@badkeys) . " Illegal key(s) passed to ModMember: " . join(',',@badkeys);
- $query =~ s/, $//;
- $query .= " WHERE borrowernumber=?";
- push @parameters, $data{'borrowernumber'};
- $debug and print STDERR "$query (executed w/ arg: $data{'borrowernumber'})";
- $sth = $dbh->prepare($query);
- my $execute_success = $sth->execute(@parameters);
- $sth->finish;
-
+ my $execute_success=UpdateInTable("borrowers",\%data);
# 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
# is adult check guarantees;
UpdateGuarantees(%data);
}
- logaction("MEMBERS", "MODIFY", $data{'borrowernumber'}, "$query (executed w/ arg: $data{'borrowernumber'})")
+ logaction("MEMBERS", "MODIFY", $data{'borrowernumber'}, "UPDATE (executed w/ arg: $data{'borrowernumber'})")
if C4::Context->preference("BorrowersLog");
return $execute_success;
my $dbh = C4::Context->dbh;
$data{'userid'} = '' unless $data{'password'};
$data{'password'} = md5_base64( $data{'password'} ) if $data{'password'};
-
- # WE SHOULD NEVER PASS THIS SUBROUTINE ANYTHING OTHER THAN ISO DATES
- # IF YOU UNCOMMENT THESE LINES YOU BETTER HAVE A DARN COMPELLING REASON
-# $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.
- if (!$data{'dateofbirth'}){
- undef ($data{'dateofbirth'});
- }
- 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'} )
- . ",altcontactsurname=" . $dbh->quote( $data{'altcontactsurname'} )
- . ",altcontactfirstname=" . $dbh->quote( $data{'altcontactfirstname'} )
- . ",altcontactaddress1=" . $dbh->quote( $data{'altcontactaddress1'} )
- . ",altcontactaddress2=" . $dbh->quote( $data{'altcontactaddress2'} )
- . ",altcontactaddress3=" . $dbh->quote( $data{'altcontactaddress3'} )
- . ",altcontactzipcode=" . $dbh->quote( $data{'altcontactzipcode'} )
- . ",altcontactphone=" . $dbh->quote( $data{'altcontactphone'} ) ;
- $debug and print STDERR "AddMember SQL: ($query)\n";
- my $sth = $dbh->prepare($query);
- # print "Executing SQL: $query\n";
- $sth->execute();
- $sth->finish;
- $data{'borrowernumber'} = $dbh->{'mysql_insertid'}; # unneeded w/ autoincrement ?
+ $data{'borrowernumber'}=InsertInTable("borrowers",\%data);
# mysql_insertid is probably bad. not necessarily accurate and mysql-specific at best.
-
logaction("MEMBERS", "CREATE", $data{'borrowernumber'}, "") if C4::Context->preference("BorrowersLog");
# check for enrollment fee & add it if needed
- $sth = $dbh->prepare("SELECT enrolmentfee FROM categories WHERE categorycode=?");
+ my $sth = $dbh->prepare("SELECT enrolmentfee FROM categories WHERE categorycode=?");
$sth->execute($data{'categorycode'});
my ($enrolmentfee) = $sth->fetchrow;
if ($enrolmentfee && $enrolmentfee > 0) {
return $data{'borrowernumber'};
}
+
sub Check_Userid {
my ($uid,$member) = @_;
my $dbh = C4::Context->dbh;
my @dat;
my $data = $sth->fetchall_arrayref({});
- $sth->finish;
return ( scalar(@$data), $data );
}
|;
my $sth3 = $dbh->prepare($guaquery);
$sth3->execute;
- $sth3->finish;
}
}
=head2 GetPendingIssues
$result[$i] = $data2;
$i++;
}
- $sth2->finish;
}
- $sth->finish;
return ( $i, \@result );
}
$data->{title} = $biblio->{title};
$acctlines[$numlines] = $data;
$numlines++;
- $total += int(100 * $data->{'amountoutstanding'}); # convert float to integer to avoid round-off errors
+ $total += int(1000 * $data->{'amountoutstanding'}); # convert float to integer to avoid round-off errors
}
- $total /= 100;
- $sth->finish;
+ $total /= 1000;
return ( $total, \@acctlines,$numlines);
}
$total += int(100 * $data->{'amountoutstanding'});
}
$total /= 100;
- $sth->finish;
return ( $total, \@acctlines, $numlines );
}
$sth->execute( uc($surname), ucfirst($firstname));
}
my @data = $sth->fetchrow;
- $sth->finish;
( $data[0] ) and return $data[0], $data[1];
return 0;
}
else {
return 0;
}
- $sth->finish();
}
push @codes, $data->{'categorycode'};
$labels{ $data->{'categorycode'} } = $data->{'description'};
}
- $sth->finish;
return ( \@codes, \%labels );
}
$sth->execute($catcode);
my $data =
$sth->fetchrow_hashref;
- $sth->finish();
return $data;
}
return;
$sth->execute;
my $data =
$sth->fetchall_arrayref({});
- $sth->finish();
return $data;
} # sub getborrowercategory
push @codes, $data->{'code'};
$labels{ $data->{'code'} } = $data->{'name'};
}
- $sth->finish;
return ( \@codes, \%labels );
}
my $sth = $dbh->prepare("Select name from ethnicity where code = ?");
$sth->execute($ethnicity);
my $data = $sth->fetchrow_hashref;
- $sth->finish;
return $data->{'name'};
} # sub fixEthnicity
while ( my $data = $sth->fetchrow_hashref() ) {
$orgs{ $data->{'borrowernumber'} } = $data;
}
- $sth->finish();
return ( \%orgs );
} # sub get_institutions
foreach my $otherborrowernumber (@$otherborrowers) {
$sth->execute( $borrowernumber, $otherborrowernumber );
}
- $sth->finish();
} # sub add_member_orgs
#test to know if the table contain some records if no the function return nothing
my $id = @id;
- $sth->finish;
if ( $id == 1 ) {
# all we have is the one blank row
return ();
return ($sortvalue) unless ($lib);
}
-=head2 DeleteBorrower
+=head2 MoveMemberToDeleted
- () = &DeleteBorrower($member);
+ $result = &MoveMemberToDeleted($borrowernumber);
-delete all data fo borrowers and add record to deletedborrowers table
-C<&$member>this is the borrowernumber
+Copy the record from borrowers to deletedborrowers table.
=cut
+# FIXME: should do it in one SQL statement w/ subquery
+# Otherwise, we should return the @data on success
+
sub MoveMemberToDeleted {
- my ($member) = @_;
+ my ($member) = shift or return;
my $dbh = C4::Context->dbh;
- my $query;
- $query = qq|SELECT *
+ my $query = qq|SELECT *
FROM borrowers
WHERE borrowernumber=?|;
my $sth = $dbh->prepare($query);
$sth->execute($member);
my @data = $sth->fetchrow_array;
- $sth->finish;
+ (@data) or return; # if we got a bad borrowernumber, there's nothing to insert
$sth =
$dbh->prepare( "INSERT INTO deletedborrowers VALUES ("
. ( "?," x ( scalar(@data) - 1 ) )
. "?)" );
$sth->execute(@data);
- $sth->finish;
}
=head2 DelMember
WHERE borrowernumber=?|;
my $sth = $dbh->prepare($query);
$sth->execute($borrowernumber);
- $sth->finish;
$query = "
DELETE
FROM borrowers
";
$sth = $dbh->prepare($query);
$sth->execute($borrowernumber);
- $sth->finish;
logaction("MEMBERS", "DELETE", $borrowernumber, "") if C4::Context->preference("BorrowersLog");
return $sth->rows;
}
sub ExtendMemberSubscriptionTo {
my ( $borrowerid,$date) = @_;
my $dbh = C4::Context->dbh;
- my $borrower = GetMember($borrowerid,'borrowernumber');
+ my $borrower = GetMember('borrowernumber'=>$borrowerid);
unless ($date){
$date=POSIX::strftime("%Y-%m-%d",localtime());
- my $borrower = GetMember($borrowerid,'borrowernumber');
$date = GetExpiryDate( $borrower->{'categorycode'}, $date );
}
my $sth = $dbh->do(<<EOF);
#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 ();
}
my $sth = $dbh->prepare($query);
$sth->execute($cardnumber);
my $imagedata = $sth->fetchrow_hashref;
- my $dberror = $sth->errstr;
warn "Database error!" if $sth->errstr;
- $sth->finish;
- return $imagedata, $dberror;
+ return $imagedata, $sth->errstr;
}
=head2 PutPatronImage
my $sth = $dbh->prepare($query);
$sth->execute($cardnumber,$mimetype,$imgfile,$imgfile);
warn "Error returned inserting $cardnumber.$mimetype." if $sth->errstr;
- my $dberror = $sth->errstr;
- $sth->finish;
- return $dberror;
+ return $sth->errstr;
}
=head2 RmPatronImage
$sth->execute($cardnumber);
my $dberror = $sth->errstr;
warn "Database error!" if $sth->errstr;
- $sth->finish;
return $dberror;
}
}
+=head2 AddMessage
+
+=over 4
+
+AddMessage( $borrowernumber, $message_type, $message, $branchcode );
+
+Adds a message to the messages table for the given borrower.
+
+Returns:
+ True on success
+ False on failure
+
+=back
+
+=cut
+
+sub AddMessage {
+ my ( $borrowernumber, $message_type, $message, $branchcode ) = @_;
+
+ my $dbh = C4::Context->dbh;
+
+ if ( ! ( $borrowernumber && $message_type && $message && $branchcode ) ) {
+ return;
+ }
+
+ my $query = "INSERT INTO messages ( borrowernumber, branchcode, message_type, message ) VALUES ( ?, ?, ?, ? )";
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $branchcode, $message_type, $message );
+
+ return 1;
+}
+
+=head2 GetMessages
+
+=over 4
+
+GetMessages( $borrowernumber, $type );
+
+$type is message type, B for borrower, or L for Librarian.
+Empty type returns all messages of any type.
+
+Returns all messages for the given borrowernumber
+
+=back
+
+=cut
+
+sub GetMessages {
+ my ( $borrowernumber, $type, $branchcode ) = @_;
+
+ if ( ! $type ) {
+ $type = '%';
+ }
+
+ my $dbh = C4::Context->dbh;
+
+ my $query = "SELECT
+ branches.branchname,
+ messages.*,
+ DATE_FORMAT( message_date, '%m/%d/%Y' ) AS message_date_formatted,
+ messages.branchcode LIKE '$branchcode' AS can_delete
+ FROM messages, branches
+ WHERE borrowernumber = ?
+ AND message_type LIKE ?
+ AND messages.branchcode = branches.branchcode
+ ORDER BY message_date DESC";
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $type ) ;
+ my @results;
+
+ while ( my $data = $sth->fetchrow_hashref ) {
+ push @results, $data;
+ }
+ return \@results;
+
+}
+
+=head2 GetMessages
+
+=over 4
+
+GetMessagesCount( $borrowernumber, $type );
+
+$type is message type, B for borrower, or L for Librarian.
+Empty type returns all messages of any type.
+
+Returns the number of messages for the given borrowernumber
+
+=back
+
+=cut
+
+sub GetMessagesCount {
+ my ( $borrowernumber, $type, $branchcode ) = @_;
+
+ if ( ! $type ) {
+ $type = '%';
+ }
+
+ my $dbh = C4::Context->dbh;
+
+ my $query = "SELECT COUNT(*) as MsgCount FROM messages WHERE borrowernumber = ? AND message_type LIKE ?";
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $borrowernumber, $type ) ;
+ my @results;
+
+ my $data = $sth->fetchrow_hashref;
+ my $count = $data->{'MsgCount'};
+
+ return $count;
+}
+
+
+
+=head2 DeleteMessage
+
+=over 4
+
+DeleteMessage( $message_id );
+
+=back
+
+=cut
+
+sub DeleteMessage {
+ my ( $message_id ) = @_;
+
+ my $dbh = C4::Context->dbh;
+
+ my $query = "DELETE FROM messages WHERE message_id = ?";
+ my $sth = $dbh->prepare($query);
+ $sth->execute( $message_id );
+
+}
+
END { } # module clean-up code here (global destructor)
1;