Comment 9 for bug 1635737

Revision history for this message
Dan Wells (dbw2) wrote :

I think the current direction is workable, but I feel like the original direction still had legs. Basically, go straight from the interval to the DateTime::Duration, no? They are almost the same already, so it seems (from here) to be an easy win. Doing so would also allow for interval_to_seconds to implicitly (or optionally) become interval_to_seconds(...from now, or given date) and in many cases do the right thing in a backwards compatible way.

I am going to give it a go and see how it shakes out. I'm not convinced it will be a better overall approach if it even works, but I feel like there will be some benefit to seeing it play out before making that judgment.