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 6f4c5c6 commit 7365933
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions kronolith/lib/Kronolith.php
Expand Up @@ -2090,7 +2090,7 @@ public static 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 @@ -2104,7 +2104,7 @@ public static 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 7365933

Please sign in to comment.