Skip Menu |

This queue is for tickets about the Date-Manip CPAN distribution.

Report information
The Basics
Id: 86958
Status: resolved
Priority: 0/
Queue: Date-Manip

People
Owner: Nobody in particular
Requestors: vksameer [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: "0:20:58 AM 1/01/10" Invalid Date Date::Manip 6.32 [asia/kolkata]
Date: Tue, 16 Jul 2013 12:28:33 +0530
To: bug-Date-Manip [...] rt.cpan.org
From: VK Sameer <vksameer [...] gmail.com>

Message body is not shown because it is too large.

The string you give IS an invalid date. There is no valid time: "00:20:58 AM". AM/PM times have hours in the 1-12 range (not 0). The "problem" you're encountering is that Date::Manip 5.xx was far too flexible and would generally find a date, even if it wasn't completely correct, so 5.xx did parse this. Date::Manip 6.xx actually is much more rigid about only accepting valid dates, so it reports this as invalid (which is correct). Of course, if you remove the "AM" and just parse the time as "00:20:58", it'll find a valid time there.
Subject: Re: [rt.cpan.org #86958] "0:20:58 AM 1/01/10" Invalid Date Date::Manip 6.32 [asia/kolkata]
Date: Wed, 17 Jul 2013 08:18:15 +0530
To: bug-Date-Manip [...] rt.cpan.org
From: VK Sameer <vksameer [...] gmail.com>
On Tue, Jul 16, 2013 at 5:30 PM, Sullivan Beck via RT <bug-Date-Manip@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=86958 > > > The string you give IS an invalid date. There is no valid time: "00:20:58 AM". AM/PM times have hours in the 1-12 range (not 0).
Ah, my bad. Of course, it should be 12 AM and not 0 AM! Sorry to have troubled you. I was in the throes of a lot of data correction and couldn't believe it when data that had earlier been parsed successfully (or so I thought) threw up an error. Regards, Sameer