summaryrefslogtreecommitdiffstats
path: root/gerrit-httpd
diff options
context:
space:
mode:
authorJan Opacki <jan.opacki@gmail.com>2013-04-14 23:44:26 +0200
committerJan Opacki <jan.opacki@gmail.com>2013-04-16 19:07:15 +0200
commitca421aee6944295f15328adf738917a982f9ad1e (patch)
tree4e6a14ae10d824facb6c555fbb56cbfbabb4dfb4 /gerrit-httpd
parent72b714137b7a4734773591f007b8f38003566698 (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.java6
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);
}