diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java index e10077afd6..c1fdb39205 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/PostReview.java @@ -455,10 +455,9 @@ public class PostReview implements RestModifyView if (ent.getValue() == null || ent.getValue() == 0) { // User requested delete of this label. if (c != null) { - if (c.getValue() != 0) { - addLabelDelta(normName, (short) 0); - } del.add(c); + addLabelDelta(normName, (short) 0); + categories.put(normName, (short) 0); update.putApproval(ent.getKey(), (short) 0); } } else if (c != null && c.getValue() != ent.getValue()) { @@ -470,6 +469,9 @@ public class PostReview implements RestModifyView update.putApproval(ent.getKey(), ent.getValue()); } else if (c != null && c.getValue() == ent.getValue()) { current.put(normName, c); + addLabelDelta(normName, c.getValue()); + categories.put(normName, c.getValue()); + update.putApproval(ent.getKey(), ent.getValue()); } else if (c == null) { c = new PatchSetApproval(new PatchSetApproval.Key( rsrc.getPatchSet().getId(),