opendev/gerrit
Code Issues Proposed changes

Users who starred a change should receive all the emails about a change.

Browse Source 
The "emailOnlyAuthors" feature silences emails about actions performed by
users in some groups. The emails for any actions in these groups are sent
only to change authors. With this commit, if a user has starred a change,
it would override this "silencing" and send them an email even if they are
not an author of that change. This provides users with an easy way to opt
in again on specific change emails.

Change-Id: Id1afb7068fece58a9336cddfbc46f9c4bbad98f2
Bug: issue 904
This commit is contained in:
Raviteja Sunkara
2012-02-22 20:25:39 +05:30
parent 2f0cb8ead9
commit a8c4bd1158
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/mail/ChangeEmail.java
View File

@@ -296,7 +296,7 @@ public abstract class ChangeEmail extends OutgoingEmail {
//
for (StarredChange w : args.db.get().starredChanges().byChange(
change.getId())) {
add(RecipientType.BCC, w.getAccountId());
super.add(RecipientType.BCC, w.getAccountId());
}
} catch (OrmException err) {
// Just don't BCC everyone. Better to send a partial message to those