opendev/gerrit
Code Issues Proposed changes

Do not include project watchers on new draft changes

Browse Source 
If a draft change is created by pushing to refs/drafts/master only
the reviewers explicitly named on the command line (which may be
empty) should be notified of the change. Users watching the project
should not be notified, as the change has not yet been published.

Change-Id: I703ea779106c025c8002e79d39c060208b2e119d
This commit is contained in:
Shawn Pearce
2014-12-10 15:54:52 -08:00
parent c22888d3fc
commit 8690a108f6
1 changed files with 22 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -45,6 +45,7 @@ public class CreateChangeSender extends NewChangeSender {
protected void init() throws EmailException {
super.init();
if (change.getStatus() == Change.Status.NEW) {
try {
// Try to mark interested owners with TO and CC or BCC line.
Watchers matching = getWatchers(NotifyType.NEW_CHANGES);
@@ -70,6 +71,7 @@ public class CreateChangeSender extends NewChangeSender {
includeWatchers(NotifyType.NEW_PATCHSETS);
}
}
private boolean isOwnerOfProjectOrBranch(Account.Id user) {
return projectState != null