summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorEdwin Kempin <edwin.kempin@gmail.com>2013-09-18 05:43:46 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2013-09-18 05:43:46 +0000
commit640b95cf1b26795e2cb6013499f0d59be4dc0767 (patch)
tree8fc1715a74b13b2d60105f1c81f3de1eed7347df
parent5ff6ebe2ed8322e2c94973d347c78d492db7ec24 (diff)
parent357b5be893088b4dcd69262d1dec7b6127cf567d (diff)
Merge "NPE when deleting draft patch set when previous draft already deleted" into stable-2.6
-rw-r--r--gerrit-server/src/main/java/com/google/gerrit/server/changedetail/DeleteDraftPatchSet.java4
1 files changed, 1 insertions, 3 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/changedetail/DeleteDraftPatchSet.java b/gerrit-server/src/main/java/com/google/gerrit/server/changedetail/DeleteDraftPatchSet.java
index adec292ac2..d20b0f63a6 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/changedetail/DeleteDraftPatchSet.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/changedetail/DeleteDraftPatchSet.java
@@ -112,9 +112,7 @@ public class DeleteDraftPatchSet implements Callable<ReviewResult> {
}
if (change.currentPatchSetId().equals(patchSetId)) {
try {
- PatchSet.Id id =
- new PatchSet.Id(patchSetId.getParentKey(), patchSetId.get() - 1);
- change.setCurrentPatchSet(patchSetInfoFactory.get(db, id));
+ change.setCurrentPatchSet(patchSetInfoFactory.get(db, highestId));
} catch (PatchSetInfoNotAvailableException e) {
throw new NoSuchChangeException(changeId);
}