check git-show-private *after* we have unfiltered_targets
This commit is contained in:
parent
b4b610ec18
commit
18b27c256e
1 changed files with 4 additions and 4 deletions
|
@ -85,10 +85,6 @@ class Module(ModuleManager.BaseModule):
|
||||||
channel = server.channels.get(channel_name)
|
channel = server.channels.get(channel_name)
|
||||||
hooks = channel.get_setting("git-webhooks", {})
|
hooks = channel.get_setting("git-webhooks", {})
|
||||||
|
|
||||||
if is_private and not channel.get_setting(
|
|
||||||
PRIVATE_SETTING_NAME, False):
|
|
||||||
continue
|
|
||||||
|
|
||||||
if hooks:
|
if hooks:
|
||||||
found_hook = self._find_hook(
|
found_hook = self._find_hook(
|
||||||
full_name_lower, repo_username_lower,
|
full_name_lower, repo_username_lower,
|
||||||
|
@ -104,6 +100,10 @@ class Module(ModuleManager.BaseModule):
|
||||||
repo_hooked = bool(unfiltered_targets)
|
repo_hooked = bool(unfiltered_targets)
|
||||||
targets = []
|
targets = []
|
||||||
for server, channel, hook in unfiltered_targets:
|
for server, channel, hook in unfiltered_targets:
|
||||||
|
if is_private and not channel.get_setting(
|
||||||
|
PRIVATE_SETTING_NAME, False):
|
||||||
|
continue
|
||||||
|
|
||||||
if (branch and
|
if (branch and
|
||||||
hook["branches"] and
|
hook["branches"] and
|
||||||
not branch in hook["branches"]):
|
not branch in hook["branches"]):
|
||||||
|
|
Loading…
Reference in a new issue