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
Referenced Files
F13092240: D17150.diff
Thu, Apr 25, 3:32 AM
F13088529: D17150.diff
Thu, Apr 25, 1:28 AM
Unknown Object (File)
Tue, Apr 23, 3:18 PM
Unknown Object (File)
Sat, Apr 20, 2:43 PM
Unknown Object (File)
Sat, Apr 20, 1:55 AM
Unknown Object (File)
Fri, Apr 19, 6:02 PM
Unknown Object (File)
Wed, Apr 17, 2:46 PM
Unknown Object (File)
Mon, Apr 1, 7:26 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