opendev/gerrit
Code Issues Proposed changes

Merge changes Id1afb706,I6170270c

Browse Source 
* changes:
  Users who starred a change should receive all the emails about a change.
  Add bccStarredBy to ReplacePatchSetSender class
This commit is contained in:
Shawn Pearce
2012-03-14 18:18:43 -07:00
committed by gerrit code review
parent 471e29f6d9 a8c4bd1158
commit 3fb96fe171
2 changed files with 2 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( for (StarredChange w : args.db.get().starredChanges().byChange(
change.getId())) { change.getId())) {
add(RecipientType.BCC, w.getAccountId()); super.add(RecipientType.BCC, w.getAccountId());
} }
} catch (OrmException err) { } catch (OrmException err) {
// Just don't BCC everyone. Better to send a partial message to those // Just don't BCC everyone. Better to send a partial message to those

1
gerrit-server/src/main/java/com/google/gerrit/server/mail/ReplacePatchSetSender.java
View File

@@ -67,6 +67,7 @@ public class ReplacePatchSetSender extends ReplyToChangeSender {
add(RecipientType.TO, reviewers); add(RecipientType.TO, reviewers);
add(RecipientType.CC, extraCC); add(RecipientType.CC, extraCC);
rcptToAuthors(RecipientType.CC); rcptToAuthors(RecipientType.CC);
bccStarredBy();
} }
@Override @Override