diff options
author | Jan Opacki <jan.opacki@gmail.com> | 2013-04-14 23:44:26 +0200 |
---|---|---|
committer | Jan Opacki <jan.opacki@gmail.com> | 2013-04-16 19:07:15 +0200 |
commit | ca421aee6944295f15328adf738917a982f9ad1e (patch) | |
tree | 4e6a14ae10d824facb6c555fbb56cbfbabb4dfb4 /gerrit-httpd | |
parent | 72b714137b7a4734773591f007b8f38003566698 (diff) |
Highlight patch sets that have drafts
On the change screen, if a patch set has a draft comment, its header will
be highlighted.
Bug: Issue 667
Change-Id: I537db90a940c9df7c4b7c28974adac5b29c8abf4
Diffstat (limited to 'gerrit-httpd')
-rw-r--r-- | gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java index 120b9af847..56f481ac3b 100644 --- a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java +++ b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/changedetail/ChangeDetailFactory.java @@ -178,6 +178,12 @@ public class ChangeDetailFactory extends Handler<ChangeDetail> { ResultSet<PatchSet> source = db.patchSets().byChange(changeId); List<PatchSet> patches = new ArrayList<PatchSet>(); for (PatchSet ps : source) { + final CurrentUser user = control.getCurrentUser(); + if (user instanceof IdentifiedUser) { + final Account.Id me = ((IdentifiedUser) user).getAccountId(); + ps.setHasDraftComments(db.patchComments() + .draftByPatchSetAuthor(ps.getId(), me).toList().size() > 0); + } if (control.isPatchVisible(ps, db)) { patches.add(ps); } |