Skip to content

Commit

Permalink
KNL-1419 Only send notifications to site members.
Browse files Browse the repository at this point in the history
If a user is a member of the global review realm and has the content.all.groups permission then they get all notification message even if they aren't actially a member of the site. This prevents non-site members from ever getting a notification.
  • Loading branch information
buckett committed Mar 16, 2016
1 parent 723c255 commit 75c9c58
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -131,12 +131,12 @@ protected List<User> getRecipients(Event event)
users.retainAll(users2);
}

//only use direct site members for the base list of users
refineToSiteMembers(users, site);

// add any other users
addSpecialRecipients(users, ref);

//only use direct site members for the base list of users
refineToSiteMembers(users, site);

return users;
}
catch (Exception any)
Expand Down

0 comments on commit 75c9c58

Please sign in to comment.