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
F15490659: D17150.diff
Fri, Apr 11, 7:30 PM
F15462560: D17150.id41242.diff
Tue, Apr 1, 4:00 PM
F15446755: D17150.id.diff
Thu, Mar 27, 8:03 PM
F15442542: D17150.id41243.diff
Wed, Mar 26, 11:47 PM
F15442541: D17150.id41242.diff
Wed, Mar 26, 11:47 PM
F15442540: D17150.id.diff
Wed, Mar 26, 11:47 PM
F15441572: D17150.diff
Wed, Mar 26, 6:50 PM
F15385641: D17150.diff
Mar 14 2025, 11:02 PM
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