Page MenuHomePhabricator

Fix an issue with date parsing when viewer timezone differs from server timezone
ClosedPublic

Authored by epriestley on Apr 11 2016, 2:37 PM.
Tags
None
Referenced Files
F14063432: D15680.diff
Mon, Nov 18, 6:33 PM
F14050339: D15680.diff
Thu, Nov 14, 5:58 PM
F14038218: D15680.diff
Sun, Nov 10, 10:12 PM
F14029025: D15680.id37781.diff
Fri, Nov 8, 6:53 PM
F14024301: D15680.diff
Thu, Nov 7, 8:03 AM
F14014648: D15680.id37781.diff
Sun, Nov 3, 6:31 AM
F14006135: D15680.id37781.diff
Mon, Oct 28, 1:51 AM
F13997120: D15680.diff
Thu, Oct 24, 2:21 AM
Subscribers
None

Details

Summary

The way DateTime works with epochs is weird, I goofed this by having my server/viewer timezone the same and not noticing.

Also fix an issue where you do ?epoch=... and then manually fiddle with the control: the control should win.

Test Plan
  • Set viewer and server timezone to different vlaues.
  • Created a countdown using ?epoch=....
  • Created a countdown using ?epoch=... and fiddling with date controls.
  • Created and edited a countdown using date/time control.
  • Poked around Calendar to make sure I didn't ruin anything this time (browsed, created event, edited event).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fix an issue with date parsing when viewer timezone differs from server timezone.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, lpriestley.
chad edited edge metadata.

Your viewer is gmt?

This revision is now accepted and ready to land.Apr 11 2016, 2:47 PM
This revision was automatically updated to reflect the committed changes.

For testing this, my viewer was EST and my server was PDT (I think?).

For the original bug on C144, the server (secure.phabricator.com) is UTC/GMT, and my viewer is PDT.