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
Unknown Object (File)
Thu, Jul 28, 3:43 AM
Unknown Object (File)
Mon, Jul 25, 2:47 AM
Unknown Object (File)
Jul 15 2022, 4:01 AM
Unknown Object (File)
Jun 27 2022, 5:54 AM
Unknown Object (File)
Jun 27 2022, 5:52 AM
Unknown Object (File)
Jun 27 2022, 5:33 AM
Unknown Object (File)
Jun 25 2022, 3:25 PM
Unknown Object (File)
Jun 20 2022, 6: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
Branch
cal1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15126
Build 19862: Run Core Tests
Build 19861: arc lint + arc unit