#
# Written by Antoine Farnault antoine@koha-fr.org on Nov. 2006.
-# $Id$
=head1 cleanborrowers.pl
=cut
use strict;
+#use warnings; FIXME - Bug 2505
use CGI;
use C4::Auth;
use C4::Output;
-
-
-use C4::Members; # GetBorrowersWhoHavexxxBorrowed.
+use C4::Dates qw/format_date format_date_in_iso/;
+use C4::Members; # GetBorrowersWhoHavexxxBorrowed.
use C4::Circulation; # AnonymiseIssueHistory.
-use Date::Calc qw/Date_to_Days Today/;
+use Date::Calc qw/Today Add_Delta_YM/;
my $cgi = new CGI;
query => $cgi,
type => "intranet",
authnotrequired => 0,
- flagsrequired => { tools => 1, catalogue => 1 },
+ flagsrequired => { tools => 'delete_anonymize_patrons', catalogue => 1 },
}
);
if ( $params->{'step2'} ) {
- $filterdate1 = $params->{'filterdate1'};
- $filterdate2 = $params->{'filterdate2'};
- my $checkbox = $params->{'checkbox'};
+ $filterdate1 = format_date_in_iso($params->{'filterdate1'});
+ $filterdate2 = format_date_in_iso($params->{'filterdate2'});
+ my %checkboxes = map { $_ => 1 } split /\0/, $params->{'checkbox'};
my $totalDel;
- if ($checkbox eq "borrower") {
- $filterdate1 = $params->{'filterdate1'};
- my $membersToDelete = GetBorrowersWhoHaveNotBorrowedSince($filterdate1);
+ my $membersToDelete;
+ if ($checkboxes{borrower}) {
+ $membersToDelete = GetBorrowersWhoHaveNotBorrowedSince($filterdate1, 1);
$totalDel = scalar @$membersToDelete;
+
}
-
my $totalAno;
- if ($checkbox eq "issue") {
- $filterdate2 = $params->{'filterdate2'};
- my $membersToAnonymize =
+ my $membersToAnonymize;
+ if ($checkboxes{issue}) {
+ $membersToAnonymize =
GetBorrowersWithIssuesHistoryOlderThan($filterdate2);
$totalAno = scalar @$membersToAnonymize;
}
step2 => 1,
totalToDelete => $totalDel,
totalToAnonymize => $totalAno,
- filterdate1 => $filterdate1,
- filterdate2 => $filterdate2
+ memberstodelete_list => $membersToDelete,
+ memberstoanonymize_list => $membersToAnonymize,
+ filterdate1 => format_date($filterdate1),
+ filterdate2 => format_date($filterdate2),
);
-
+### TODO : Use GetBorrowersNamesAndLatestIssue function in order to get the borrowers to delete or anonymize.
+### Now, we are only using total, which is not enough imlo
#writing the template
output_html_with_http_headers $cgi, $cookie, $template->output;
exit;
}
if ( $params->{'step3'} ) {
- $filterdate1 = $params->{'filterdate1'};
- $filterdate2 = $params->{'filterdate2'};
+ $filterdate1 = format_date_in_iso($params->{'filterdate1'});
+ $filterdate2 = format_date_in_iso($params->{'filterdate2'});
my $do_delete = $params->{'do_delete'};
my $do_anonym = $params->{'do_anonym'};
# delete members
if ($do_delete) {
- my $membersToDelete = GetBorrowersWhoHaveNotBorrowedSince($filterdate1);
+ my $membersToDelete = GetBorrowersWhoHaveNotBorrowedSince($filterdate1, 1);
$totalDel = scalar(@$membersToDelete);
$radio = $params->{'radio'};
if ( $radio eq 'trash' ) {
#default value set to the template are the 'CNIL' value.
my ( $year, $month, $day ) = &Today();
-my $tmpyear = $year - 1;
-my $tmpmonth = $month - 3;
-$filterdate1 = $tmpyear . "-" . $month . "-" . $day;
-$filterdate2 = $year . "-" . $tmpmonth . "-" . $day;
+$filterdate1 = format_date(sprintf("%-04.4d-%-02.2d-%02.2d", Add_Delta_YM($year, $month, $day, -1, 0)));
+$filterdate2 = format_date(sprintf("%-04.4d-%-02.2d-%02.2d", Add_Delta_YM($year, $month, $day, 0, -3)));
$template->param(
step1 => '1',
filterdate1 => $filterdate1,
- filterdate2 => $filterdate2
+ filterdate2 => $filterdate2,
+ DHTMLcalendar_dateformat => C4::Dates->DHTMLcalendar(),
);
#writing the template