opendev/gerrit
Code Issues Proposed changes

Add change topic in hook arguments

Browse Source 
It is not possible for hook scripts to include topic-specific
behaviour because the topic name is not included in the
arguments.

Add the topic as an argument in all relevant hooks.

Change-Id: Ib8b4703c851e97c32c08195ca27f3269742ec9e5
This commit is contained in:
David Pursehouse
2012-09-18 10:07:31 +09:00
committed by Edwin Kempin
parent b15a0eb726
commit 29d9ee1974
2 changed files with 12 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java
View File

@@ -235,6 +235,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--uploader", getDisplayName(uploader.getAccount()));
addArg(args, "--commit", event.patchSet.revision);
addArg(args, "--patchset", event.patchSet.number);
@@ -257,6 +258,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--uploader", getDisplayName(uploader.getAccount()));
addArg(args, "--commit", event.patchSet.revision);
addArg(args, "--patchset", event.patchSet.number);
@@ -289,6 +291,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--author", getDisplayName(account));
addArg(args, "--commit", event.patchSet.revision);
addArg(args, "--comment", comment == null ? "" : comment);
@@ -313,6 +316,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--submitter", getDisplayName(account));
addArg(args, "--commit", event.patchSet.revision);
@@ -333,6 +337,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--abandoner", getDisplayName(account));
addArg(args, "--reason", reason == null ? "" : reason);
@@ -353,6 +358,7 @@ public class ChangeHookRunner implements ChangeHooks {
addArg(args, "--change-url", event.change.url);
addArg(args, "--project", event.change.project);
addArg(args, "--branch", event.change.branch);
addArg(args, "--topic", event.change.topic);
addArg(args, "--restorer", getDisplayName(account));
addArg(args, "--reason", reason == null ? "" : reason);