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>
This commit is contained in:
parent
cd13475cf9
commit
caebb20cbb
|
|
@ -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