Page MenuHomePhabricator

Don't notify without notifiable attendees
ClosedPublic

Authored by 0 on Dec 21 2017, 8:44 PM.
Tags
None
Referenced Files
F18847370: D18835.diff
Oct 30 2025, 12:17 AM
F18837166: D18835.id45191.diff
Oct 27 2025, 4:35 AM
F18836197: D18835.id.diff
Oct 26 2025, 11:07 PM
F18830073: D18835.diff
Oct 25 2025, 3:56 AM
F18813029: D18835.id45190.diff
Oct 20 2025, 1:34 PM
F18808247: D18835.diff
Oct 19 2025, 5:45 AM
F18652711: D18835.id45190.diff
Sep 21 2025, 9:10 AM
F18600385: D18835.diff
Sep 13 2025, 9:37 AM
Subscribers

Details

Summary

Events with no attendees (e.g. fresh instances of recurring events) would trigger an exception when sending notifications, because $attendee_map would still get populated.

Test Plan

Declined event, restarted daemons. Did not see exception. Accepted event, restarted daemons. Saw "[Calendar] [Reminder]" email.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable