Skip to content

Commit

Permalink
Fix building up notification list.
Browse files Browse the repository at this point in the history
We want to add recipients if ANY of the preference conditions match.
  • Loading branch information
yunosh committed Apr 20, 2015
1 parent fb08528 commit 8180a96
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions kronolith/lib/Kronolith.php
Expand Up @@ -2085,7 +2085,7 @@ static public function sendNotification($event, $action)
->create($event->creator ?: $owner);

foreach ($share->listUsers(Horde_Perms::READ) as $user) {
if (!isset($recipients[$user])) {
if (empty($recipients[$user])) {
$recipients[$user] = self::_notificationPref($user, 'read', $calendar);
}
}
Expand All @@ -2099,7 +2099,7 @@ static public function sendNotification($event, $action)
}

foreach ($group_users as $user) {
if (!isset($recipients[$user])) {
if (empty($recipients[$user])) {
$recipients[$user] = self::_notificationPref($user, 'read', $calendar);
}
}
Expand Down

0 comments on commit 8180a96

Please sign in to comment.