From eb081dcfbff29e53925131f0bc84564c62b18984 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Tue, 18 Sep 2012 21:44:18 +0900 Subject: [PATCH] Add `--is-draft` argument on patchset-created hook The `--is-draft` argument will be passed with either `true` if the patchset is a draft, or `false` otherwise. This can be used by hooks that need to behave differently if the change is a draft. Change-Id: Idd00efe799bb05b0fd00e917faaccd5c772b1e1f --- Documentation/config-hooks.txt | 2 +- .../main/java/com/google/gerrit/common/ChangeHookRunner.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/Documentation/config-hooks.txt b/Documentation/config-hooks.txt index ef1c826add..dfdba523d6 100644 --- a/Documentation/config-hooks.txt +++ b/Documentation/config-hooks.txt @@ -27,7 +27,7 @@ This is called whenever a patchset is created (this includes new changes and drafts). ==== - patchset-created --change --change-url --project --branch --topic --uploader --commit --patchset + patchset-created --change --is-draft --change-url --project --branch --topic --uploader --commit --patchset ==== draft-published diff --git a/gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java b/gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java index dcc670147a..37be293eec 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java +++ b/gerrit-server/src/main/java/com/google/gerrit/common/ChangeHookRunner.java @@ -232,6 +232,7 @@ public class ChangeHookRunner implements ChangeHooks { final List args = new ArrayList(); addArg(args, "--change", event.change.id); + addArg(args, "--is-draft", patchSet.isDraft() ? "true" : "false"); addArg(args, "--change-url", event.change.url); addArg(args, "--project", event.change.project); addArg(args, "--branch", event.change.branch);