HomePhabricator

Don't notify without notifiable attendees

Description

Don't notify without notifiable attendees

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.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D18835

Details

Provenance
0Authored on Dec 21 2017, 8:46 PM
epriestleyCommitted on Dec 21 2017, 8:46 PM
epriestleyPushed on Dec 21 2017, 8:46 PM
Reviewer
Blessed Reviewers
Differential Revision
D18835: Don't notify without notifiable attendees
Parents
rPe411d75964e5: Fix an issue where blame could fatal for unrecognized authors
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18968
Build 25581: Run Core Tests