From 325b6c649274ddec79684fec441320f34bd9bd61 Mon Sep 17 00:00:00 2001 From: Edwin Kempin Date: Fri, 28 Jun 2013 00:17:39 +0200 Subject: [PATCH] Use updated change in PatchSetInserter When a new patch set is created with the PatchSetInserter the updating of the tracking IDs, the indexing and running the hooks isn't done on the updated change which contains the new patch set. Change-Id: If40e3e1aedacc8a338e9202eee6b5f6c418eea9b Signed-off-by: Edwin Kempin --- .../com/google/gerrit/server/change/PatchSetInserter.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java index 7156cc6580..8317859322 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/PatchSetInserter.java @@ -209,15 +209,15 @@ public class PatchSetInserter { } final List footerLines = commit.getFooterLines(); - ChangeUtil.updateTrackingIds(db, change, trackingFooters, footerLines); + ChangeUtil.updateTrackingIds(db, updatedChange, trackingFooters, footerLines); db.commit(); if (changeMessage != null) { db.changeMessages().insert(Collections.singleton(changeMessage)); } - indexer.index(change); - hooks.doPatchsetCreatedHook(change, patchSet, db); + indexer.index(updatedChange); + hooks.doPatchsetCreatedHook(updatedChange, patchSet, db); } finally { db.rollback(); }