From 63edfbe3e89d73bab50f6f090ce8812bf93d64dd Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 20 Jan 2015 15:32:45 +0100 Subject: [PATCH] Bug 13601: get rid of DateTime::Format::DateParse This module is used in C4::Members::GetPendingIssues too, but we can use dt_from_string. Test plan: 1/ Verify that prove t/db_dependent/Members/GetPendingIssues.t returns green 2/ On the patron pending issue list, verify that the issue and the due dates are correctly displayed. --- C4/Installer/PerlDependencies.pm | 5 ----- C4/Members.pm | 6 ++---- 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/C4/Installer/PerlDependencies.pm b/C4/Installer/PerlDependencies.pm index 34cd8b5..67ec808 100644 --- a/C4/Installer/PerlDependencies.pm +++ b/C4/Installer/PerlDependencies.pm @@ -222,11 +222,6 @@ our $PERL_DEPS = { 'required' => '1', 'min_ver' => '1.20' }, - 'DateTime::Format::DateParse' => { - 'usage' => 'Core', - 'required' => '1', - 'min_ver' => '0.04' - }, 'DateTime::Format::MySQL' => { 'usage' => 'Core', 'required' => '1', diff --git a/C4/Members.pm b/C4/Members.pm index 4fb845d..a98d16f 100644 --- a/C4/Members.pm +++ b/C4/Members.pm @@ -36,7 +36,6 @@ use C4::SQLHelper qw(InsertInTable UpdateInTable SearchInTable); use C4::Members::Attributes qw(SearchIdMatchingAttribute UpdateBorrowerAttribute); use C4::NewsChannels; #get slip news use DateTime; -use DateTime::Format::DateParse; use Koha::Database; use Koha::DateUtils; use Koha::Borrower::Debarments qw(IsDebarred); @@ -1195,14 +1194,13 @@ sub GetPendingIssues { my $sth = C4::Context->dbh->prepare($query); $sth->execute(@borrowernumbers); my $data = $sth->fetchall_arrayref({}); - my $tz = C4::Context->tz(); - my $today = DateTime->now( time_zone => $tz); + my $today = dt_from_string; foreach (@{$data}) { if ($_->{issuedate}) { $_->{issuedate} = dt_from_string($_->{issuedate}, 'sql'); } $_->{date_due} or next; - $_->{date_due} = DateTime::Format::DateParse->parse_datetime($_->{date_due}, $tz->name()); + $_->{date_due} = dt_from_string($_->{date_due}, 'sql'); if ( DateTime->compare($_->{date_due}, $today) == -1 ) { $_->{overdue} = 1; } -- 2.1.0