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
F18931427: D17150.id41242.diff
Mon, Nov 10, 3:15 AM
F18918283: D17150.id.diff
Sun, Nov 9, 10:39 AM
F18902094: D17150.id41243.diff
Fri, Nov 7, 11:49 PM
F18863864: D17150.id41242.diff
Mon, Nov 3, 1:36 AM
F18819775: D17150.id.diff
Wed, Oct 22, 9:26 AM
F18812906: D17150.diff
Mon, Oct 20, 12:58 PM
F18763536: D17150.id.diff
Oct 7 2025, 12:43 AM
F18705550: D17150.id41242.diff
Sep 28 2025, 11:52 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