diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java index ff833f4e63..df5c254dda 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/change/ChangeInserter.java @@ -31,6 +31,7 @@ import com.google.gerrit.reviewdb.server.ReviewDb; import com.google.gerrit.server.ApprovalsUtil; import com.google.gerrit.server.ChangeMessagesUtil; import com.google.gerrit.server.ChangeUtil; +import com.google.gerrit.server.IdentifiedUser; import com.google.gerrit.server.account.AccountCache; import com.google.gerrit.server.extensions.events.GitReferenceUpdated; import com.google.gerrit.server.git.WorkQueue; @@ -278,6 +279,12 @@ public class ChangeInserter { accountCache.get(change.getOwner()).getAccount(), hashtags, null, hashtags, db); } + + if (approvals != null && !approvals.isEmpty()) { + hooks.doCommentAddedHook(change, + ((IdentifiedUser) ctl.getCurrentUser()).getAccount(), patchSet, + null, approvals, db); + } } return change; diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommits.java b/gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommits.java index 1184a94536..a950eabdc2 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommits.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommits.java @@ -2306,6 +2306,11 @@ public class ReceiveCommits { change, currentUser.getAccount(), newPatchSet, db, newCommit.getName()); } + if (!approvals.isEmpty()) { + hooks.doCommentAddedHook(change, currentUser.getAccount(), newPatchSet, + null, approvals, db); + } + if (magicBranch != null && magicBranch.submit) { submit(changeCtl, newPatchSet); }