Bug 30042: Remove Date::Calc dependency in batch_anonymize.pl
This patch makes the script use the Koha::DateUtils tools instead.
To test:
1. Run:
$ kshell
k$ perl misc/cronjobs/batch_anonymise.pl --verbose --days 7
=> SUCCESS: You see (the date may vary):
Checkouts and holds before 2022-02-15 will be anonymised.
2. Apply this patch
3. Repeat 1
=> SUCCESS: Same output
4. Sign off :-D
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@iki.fi>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit caebb20cbb)
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
191aad389e
commit
0bbf91bf33
|
|
@ -25,10 +25,10 @@ use C4::Context;
|
|||
use C4::Log qw( cronlogaction );
|
||||
|
||||
use Koha::Database;
|
||||
use Koha::DateUtils qw(dt_from_string output_pref);
|
||||
use Koha::Old::Checkouts;
|
||||
use Koha::Old::Holds;
|
||||
|
||||
use Date::Calc qw( Add_Delta_Days Today );
|
||||
use Getopt::Long qw( GetOptions );
|
||||
|
||||
sub usage {
|
||||
|
|
@ -62,10 +62,10 @@ if ( !$days ) {
|
|||
|
||||
cronlogaction();
|
||||
|
||||
my ($year,$month,$day) = Today();
|
||||
my ($newyear,$newmonth,$newday) = Add_Delta_Days ($year,$month,$day,(-1)*$days);
|
||||
my $formatdate = sprintf "%4d-%02d-%02d",$newyear,$newmonth,$newday;
|
||||
$verbose and print "Checkouts and holds before $formatdate will be anonymised.\n";
|
||||
my $date = dt_from_string->subtract( days => $days );
|
||||
|
||||
print "Checkouts and holds before " . output_pref( { dt => $date, dateformat => 'iso', dateonly => 1 } ) . " will be anonymised.\n"
|
||||
if $verbose;
|
||||
|
||||
my $rows = Koha::Old::Checkouts
|
||||
->filter_by_anonymizable
|
||||
|
|
|
|||
Loading…
Reference in New Issue