Bug #18922: Time at 24:00:00 UTC is not normalized

• Status: Closed
• Assignee: -
• Target version: -
• Backport: 2.7: REQUIRED, 3.0: REQUIRED, 3.1: DONE

[nobu (Nobuyoshi Nakada)]

Reported by @mame (Yusuke Endoh).

$ ruby3.0 -e 'p Time.new(2000, 1, 1, 24, 0, 0, "-00:00").to_a[0, 6]'
[0, 0, 0, 2, 1, 2000]

$ ruby3.1 -e 'p Time.new(2000, 1, 1, 24, 0, 0, "-00:00").to_a[0, 6]'
[0, 0, 24, 1, 1, 2000]

$ ruby3.2 -e 'p Time.new(2000, 1, 1, 24, 0, 0, "-00:00").to_a[0, 6]'
[0, 0, 24, 1, 1, 2000]

[nobu (Nobuyoshi Nakada)]

• Subject changed from Time at 24:00:00 UTC is not normalized since 3.1 to Time at 24:00:00 UTC is not normalized
• Backport changed from 2.7: DONTNEED, 3.0: DONTNEED, 3.1: REQUIRED to 2.7: REQUIRED, 3.0: DONTNEED, 3.1: REQUIRED

Time.new(2000, 1, 1, 24, 0, 0, "Z").to_a[0, 6] returns [0, 0, 24, 1, 1, 2000] since 2.7.
It seems to be a much older bug, just that "Z" was not supported 2.6 and earlier and didn’t appear.

[nobu (Nobuyoshi Nakada)]

• Backport changed from 2.7: REQUIRED, 3.0: DONTNEED, 3.1: REQUIRED to 2.7: REQUIRED, 3.0: REQUIRED, 3.1: REQUIRED

[nobu (Nobuyoshi Nakada)]

• Status changed from Open to Closed

Applied in changeset git|68903df6f6fc548f3bf68fb09ee8b2495dcd28f0.

---

[Bug #18922] Normalize time at 24:00:00 UTC

[mame (Yusuke Endoh)]

As a record: the behavior I found was about Time#inspect, which was clearly wrong.

$ ruby -ve 'p Time.new(2000, 1, 1, 24, 0, 0, "Z")'
ruby 3.1.2p20 (2022-04-12 revision 4491bb740a) [x86_64-linux]
2000-01-01 23:00:00 UTC

After nobu's patch, it works correctly.

$ ./miniruby -ve 'p Time.new(2000, 1, 1, 24, 0, 0, "Z")'
ruby 3.2.0dev (2022-07-18T10:07:55Z master a74634de10) [x86_64-linux]
2000-01-02 00:00:00 UTC

[nagachika (Tomoyuki Chikanaga)]

• Backport changed from 2.7: REQUIRED, 3.0: REQUIRED, 3.1: REQUIRED to 2.7: REQUIRED, 3.0: REQUIRED, 3.1: DONE

ruby_3_1 13ee4b2c35bad0f893d5ed5a6fdca62da406f958 merged revision(s) 68903df6f6fc548f3bf68fb09ee8b2495dcd28f0.

[nobu (Nobuyoshi Nakada)]

• Related to Bug #19106: Time - Hour 24 with tzinfo to_s bug added