Page MenuHomePhabricator

Fix an issue where setting a recurrence end date on a Calendar event without one could fatal
ClosedPublic

Authored by epriestley on Jan 6 2017, 9:31 PM.
Tags
None
Attached Files
Unknown Object (File)
Apr 10 2017, 7:41 PM
Unknown Object (File)
Apr 10 2017, 4:19 AM
Unknown Object (File)
Feb 25 2017, 4:25 PM
Unknown Object (File)
Feb 16 2017, 10:06 AM
Unknown Object (File)
Jan 31 2017, 2:30 AM
Unknown Object (File)
Jan 31 2017, 2:28 AM
Unknown Object (File)
Jan 8 2017, 12:13 AM
Unknown Object (File)
Jan 7 2017, 2:02 AM
Subscribers
None

Details

Summary

Ref T11816. The underlying format of recurrence end dates swapped around a bit and we now try to compare null to a valid date if you're setting it for the first time.

Test Plan
  • On a new event, set a recurrence end date.
  • Then, removed a recurrence end date.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable