opendev/gerrit
Code Issues Proposed changes

Merge "Do not automatically add author/committer as reviewer to drafts"

Browse Source
This commit is contained in:
Edwin Kempin
2012-06-14 07:40:44 -07:00
committed by gerrit code review
parent de980a4ddd 59eae52e9d
commit abfffef5d4
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/ApprovalsUtil.java
View File

@@ -124,14 +124,14 @@ public class ApprovalsUtil {
Account.Id authorId = info.getAuthor() != null Account.Id authorId = info.getAuthor() != null
? info.getAuthor().getAccount() ? info.getAuthor().getAccount()
: null; : null;
if (authorId != null) { if (authorId != null && !ps.isDraft()) {
need.add(authorId); need.add(authorId);
} }
Account.Id committerId = info.getCommitter() != null Account.Id committerId = info.getCommitter() != null
? info.getCommitter().getAccount() ? info.getCommitter().getAccount()
: null; : null;
if (committerId != null) { if (committerId != null && !ps.isDraft()) {
need.add(committerId); need.add(committerId);
} }
need.remove(change.getOwner()); need.remove(change.getOwner());