Closed Bug 487310 Opened 16 years ago Closed 16 years ago

local-dateTime() doesn't handle daylight savings time

Categories

(Core Graveyard :: XForms, defect)

1.9.0 Branch
x86
Windows XP
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aaronr, Assigned: aaronr)

Details

Attachments

(2 files)

Found by a user and reported via the newsgroup. Looks like local-dateTime() doesn't handle daylight savings time correctly. Talked to Merle and he pointed me toward the fix. Just need to add time.tm_params.tp_dst_offset into the mix, (add it to time.tm_params.tp_gmt_offset and THEN figure out whether the gmt offset should be '+' or '-').
Attached patch fixSplinter Review
Attachment #371533 - Flags: review?(msterlin)
r = me. I am not authorized to edit the bug and add the r+.
Comment on attachment 371533 [details] [diff] [review] fix adding r+ for msterlin.
Attachment #371533 - Flags: review?(msterlin)
Attachment #371533 - Flags: review?(Olli.Pettay)
Attachment #371533 - Flags: review+
Attachment #371533 - Flags: review?(Olli.Pettay) → review+
Doh! Forgot to check in! Checked into cvs trunk (FF3). Needs to be checked into hg. Alex?
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: