HomePhorge

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
Dmitri IouchtchenkoAuthored on Dec 21 2017, 12:46 PM
epriestleyCommitted on Dec 21 2017, 12:46 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPe411d75964e5: Fix an issue where blame could fatal for unrecognized authors
Branches
Unknown
Tags
Unknown

Event Timeline