Merge "change-merged: set correct newRev on submit by push" into stable-2.16

java/com/google/gerrit/server/git/MergedByPushOp.java

@@ -54,7 +54,10 @@ public class MergedByPushOp implements BatchUpdateOp {
 
   public interface Factory {
     MergedByPushOp create(
-        RequestScopePropagator requestScopePropagator, PatchSet.Id psId, String refName);
+        RequestScopePropagator requestScopePropagator,
+        PatchSet.Id psId,
+        @Assisted("refName") String refName,
+        @Assisted("mergeResultRevId") String mergeResultRevId);
   }
 
   private final RequestScopePropagator requestScopePropagator;
@@ -67,6 +70,7 @@ public class MergedByPushOp implements BatchUpdateOp {
 
   private final PatchSet.Id psId;
   private final String refName;
+  private final String mergeResultRevId;
 
   private Change change;
   private boolean correctBranch;
@@ -84,7 +88,8 @@ public class MergedByPushOp implements BatchUpdateOp {
       ChangeMerged changeMerged,
       @Assisted RequestScopePropagator requestScopePropagator,
       @Assisted PatchSet.Id psId,
-      @Assisted String refName) {
+      @Assisted("refName") String refName,
+      @Assisted("mergeResultRevId") String mergeResultRevId) {
     this.patchSetInfoFactory = patchSetInfoFactory;
     this.cmUtil = cmUtil;
     this.mergedSenderFactory = mergedSenderFactory;
@@ -94,6 +99,7 @@ public class MergedByPushOp implements BatchUpdateOp {
     this.requestScopePropagator = requestScopePropagator;
     this.psId = psId;
     this.refName = refName;
+    this.mergeResultRevId = mergeResultRevId;
   }
 
   public String getMergedIntoRef() {
@@ -196,8 +202,7 @@ public class MergedByPushOp implements BatchUpdateOp {
                   }
                 }));
 
-    changeMerged.fire(
-        change, patchSet, ctx.getAccount(), patchSet.getRevision().get(), ctx.getWhen());
+    changeMerged.fire(change, patchSet, ctx.getAccount(), mergeResultRevId, ctx.getWhen());
   }
 
   private PatchSetInfo getPatchSetInfo(ChangeContext ctx) throws IOException, OrmException {

java/com/google/gerrit/server/git/receive/ReceiveCommits.java

@@ -2818,6 +2818,7 @@ class ReceiveCommits {
                   projectState,
                   notes.getChange().getDest(),
                   checkMergedInto,
+                  checkMergedInto ? inputCommand.getNewId().name() : null,
                   priorPatchSet,
                   priorCommit,
                   psId,
@@ -3120,7 +3121,8 @@ class ReceiveCommits {
                     existingPatchSets++;
                     bu.addOp(
                         psId.getParentKey(),
-                        mergedByPushOpFactory.create(requestScopePropagator, psId, refName));
+                        mergedByPushOpFactory.create(
+                            requestScopePropagator, psId, refName, newTip.getId().getName()));
                     continue COMMIT;
                   }
                 }
@@ -3153,7 +3155,7 @@ class ReceiveCommits {
                 bu.addOp(
                     id,
                     mergedByPushOpFactory
-                        .create(requestScopePropagator, req.psId, refName)
+                        .create(requestScopePropagator, req.psId, refName, newTip.getId().getName())
                         .setPatchSetProvider(req.replaceOp::getPatchSet));
                 bu.addOp(id, new ChangeProgressOp(progress));
                 ids.add(id);

java/com/google/gerrit/server/git/receive/ReplaceOp.java

@@ -103,6 +103,7 @@ public class ReplaceOp implements BatchUpdateOp {
         ProjectState projectState,
         Branch.NameKey dest,
         boolean checkMergedInto,
+        @Nullable String mergeResultRevId,
         @Assisted("priorPatchSetId") PatchSet.Id priorPatchSetId,
         @Assisted("priorCommitId") ObjectId priorCommit,
         @Assisted("patchSetId") PatchSet.Id patchSetId,
@@ -136,6 +137,7 @@ public class ReplaceOp implements BatchUpdateOp {
   private final ProjectState projectState;
   private final Branch.NameKey dest;
   private final boolean checkMergedInto;
+  private final String mergeResultRevId;
   private final PatchSet.Id priorPatchSetId;
   private final ObjectId priorCommitId;
   private final PatchSet.Id patchSetId;
@@ -181,6 +183,7 @@ public class ReplaceOp implements BatchUpdateOp {
       @Assisted ProjectState projectState,
       @Assisted Branch.NameKey dest,
       @Assisted boolean checkMergedInto,
+      @Assisted @Nullable String mergeResultRevId,
       @Assisted("priorPatchSetId") PatchSet.Id priorPatchSetId,
       @Assisted("priorCommitId") ObjectId priorCommitId,
       @Assisted("patchSetId") PatchSet.Id patchSetId,
@@ -210,6 +213,7 @@ public class ReplaceOp implements BatchUpdateOp {
     this.projectState = projectState;
     this.dest = dest;
     this.checkMergedInto = checkMergedInto;
+    this.mergeResultRevId = mergeResultRevId;
     this.priorPatchSetId = priorPatchSetId;
     this.priorCommitId = priorCommitId.copy();
     this.patchSetId = patchSetId;
@@ -236,7 +240,8 @@ public class ReplaceOp implements BatchUpdateOp {
       String mergedInto = findMergedInto(ctx, dest.get(), commit);
       if (mergedInto != null) {
         mergedByPushOp =
-            mergedByPushOpFactory.create(requestScopePropagator, patchSetId, mergedInto);
+            mergedByPushOpFactory.create(
+                requestScopePropagator, patchSetId, mergedInto, mergeResultRevId);
       }
     }