diff options
author | Tomas Ljunggren <tomas.ljunggren@tieto.com> | 2011-11-11 16:03:07 +0100 |
---|---|---|
committer | Tomas Ljunggren <tomas.ljunggren@tieto.com> | 2011-11-11 16:32:44 +0100 |
commit | 7d038cc1cb04359fae8ef2264d1a92343ae73ac7 (patch) | |
tree | 6acd527cc3f0550f7e250ff11bf7c2cf7fb4e444 | |
parent | 92f93c09940cd6a9c3ca3adb8fc167719761327d (diff) | |
parent | a68783d40c9296dc67bde4816ac8b54d65f3ff77 (diff) |
RC-V2.2.1-INT-002
Consider sanity review column
Fix typo in project config's topic review checkbox label
Fixed automatic update of reviewer list
Fixed gerrit email verification link requires insecure login
Fixed late removal of review approvals
Fixed review database update from StagingApprove
Change-Id: I0b20af2aa4d0b9b871eba7c081d5298dea610611
-rw-r--r-- | gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java | 19 | ||||
-rw-r--r-- | gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/ApprovalCategory.java | 3 |
2 files changed, 15 insertions, 7 deletions
diff --git a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java index a96c3c9384..300d1ae0d9 100644 --- a/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java +++ b/gerrit-httpd/src/main/java/com/google/gerrit/httpd/rpc/patch/PatchDetailServiceImpl.java @@ -185,19 +185,24 @@ class PatchDetailServiceImpl extends BaseServiceImplementation implements functionStateFactory.create(change, ps_id, psas.values()); for (final PatchSetApproval ca : db.patchSetApprovals() - .byPatchSetUser(ps_id, aid)) { + .byPatchSet(ps_id)) { final ApprovalCategory.Id category = ca.getCategoryId(); if (ApprovalCategory.SUBMIT.equals(category) || ApprovalCategory.STAGING.equals(category)) { continue; } - if (change.getStatus().isOpen()) { - fs.normalize(approvalTypes.byId(category), ca); - } - if (ca.getValue() == 0) { - continue; + // Only process approval for account or if approval category is sanity review + if (ca.getAccountId().equals(aid) + || ApprovalCategory.SANITY_REVIEW.equals(category)) { + if (change.getStatus().isOpen()) { + fs.normalize(approvalTypes.byId(category), ca); + } + if (ca.getValue() == 0) { + continue; + } + psas.put(category, ca); } - psas.put(category, ca); + } approvals.put(id, new ApprovalSummary(psas.values())); diff --git a/gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/ApprovalCategory.java b/gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/ApprovalCategory.java index ad170362df..dd6d02d935 100644 --- a/gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/ApprovalCategory.java +++ b/gerrit-reviewdb/src/main/java/com/google/gerrit/reviewdb/ApprovalCategory.java @@ -27,6 +27,9 @@ public final class ApprovalCategory { /** Id of the custom "Staging" category. */ public static final ApprovalCategory.Id STAGING = new ApprovalCategory.Id("STGN"); + /** Id of the sanity review category */ + public static final ApprovalCategory.Id SANITY_REVIEW = new ApprovalCategory.Id("SRVW"); + public static class Id extends StringKey<Key<?>> { private static final long serialVersionUID = 1L; |