Bug #77066 for DateTime: delta_ms also captures hours; not reflected in docs

Tue May 08 12:01:23 2012 BBYRD [...] cpan.org - Ticket created

Subject:

delta_ms also captures hours; not reflected in docs

According to the docs, delta_ms only gives out the minutes and seconds of a DT delta. This isn't true, as the code is clearly passing an hours key/value. This should be fixed in the docs to reflect that it also gives the hours.

Tue May 08 12:09:06 2012 autarch [...] urth.org - Correspondence added

Subject:	Re: [rt.cpan.org #77066] delta_ms also captures hours; not reflected in docs
Date:	Tue, 8 May 2012 11:08:52 -0500 (CDT)
To:	Brendan Byrd via RT <bug-DateTime [...] rt.cpan.org>
From:	Dave Rolsky <autarch [...] urth.org>

On Tue, 8 May 2012, Brendan Byrd via RT wrote: Show quoted text

> According to the docs, delta_ms only gives out the minutes and seconds > of a DT delta. This isn't true, as the code is clearly passing an hours > key/value. This should be fixed in the docs to reflect that it also > gives the hours.

Hours are simply a count of minutes/60. In the same way, delta_days also can give you weeks. I'm not sure how to phrase this in the docs. The duration api can be a bit confusing, I understand. -dave /*============================================================ http://VegGuide.org http://blog.urth.org Your guide to all that's veg House Absolute(ly Pointless) ============================================================*/

Tue May 08 12:09:07 2012 The RT System itself - Status changed from 'new' to 'open'

Thu May 10 01:23:34 2012 BBYRD [...] cpan.org - Correspondence added

Oh, sorry, I was looking at the delta_ms code and saw the hours, but didn't go further into the DateTime::Duration new method, where it further converts the hours into minutes.

Sat May 26 11:46:32 2012 DROLSKY [...] cpan.org - Status changed from 'open' to 'resolved'

Bug #77066 for DateTime: delta_ms also captures hours; not reflected in docs

Preferred bug tracker