opendev/gerrit
Code Issues Proposed changes

Merge "Fix: Make PatchSetInserter set patch set id"

Browse Source
This commit is contained in:
Dave Borowitz
2016-01-12 15:20:18 +00:00
committed by Gerrit Code Review
parent cc9ce2ebc1 340e895d27
commit 2b4d158e0f
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java
View File

@@ -44,6 +44,7 @@ import com.google.gerrit.server.git.GroupCollector;
import com.google.gerrit.server.git.validators.CommitValidationException;
import com.google.gerrit.server.git.validators.CommitValidators;
import com.google.gerrit.server.mail.ReplacePatchSetSender;
import com.google.gerrit.server.notedb.ChangeUpdate;
import com.google.gerrit.server.notedb.ReviewerStateInternal;
import com.google.gerrit.server.patch.PatchSetInfoFactory;
import com.google.gerrit.server.project.ChangeControl;
@@ -210,6 +211,8 @@ public class PatchSetInserter extends BatchUpdate.Op {
ChangeControl ctl = ctx.getChangeControl();
change = ctx.getChange();
ChangeUpdate update = ctx.getChangeUpdate();
Change.Id id = change.getId();
final PatchSet.Id currentPatchSetId = change.currentPatchSetId();
if (!change.getStatus().isOpen() && !allowClosed) {
@@ -223,6 +226,8 @@ public class PatchSetInserter extends BatchUpdate.Op {
patchSet.setRevision(new RevId(commit.name()));
patchSet.setDraft(draft);
update.setPatchSetId(patchSet.getId());
if (groups != null) {
patchSet.setGroups(groups);
} else {