diff options
author | Paul Sokolovsky <paul.sokolovsky@linaro.org> | 2014-05-12 15:54:24 +0300 |
---|---|---|
committer | Paul Sokolovsky <paul.sokolovsky@linaro.org> | 2014-05-12 16:40:39 +0300 |
commit | 8bb72b9395de308f3e7d4797a868c38a6fad4f0e (patch) | |
tree | bdcc9f5661084770b0246df5b0529f7c1b760456 | |
parent | 644e92f6c77c54eeea58be4a42adbc3123c1ff6d (diff) |
cron-disable.mangle: Revamp to make actually work.
Change-Id: I117df06a7ac908a382b54a074fb60693cca8d078
-rw-r--r-- | mangle-jobs/cron-disable.mangle | 12 | ||||
-rw-r--r-- | mangle-jobs/mangle_helper.py | 2 |
2 files changed, 10 insertions, 4 deletions
diff --git a/mangle-jobs/cron-disable.mangle b/mangle-jobs/cron-disable.mangle index c5f2c3d..b1ca6d5 100644 --- a/mangle-jobs/cron-disable.mangle +++ b/mangle-jobs/cron-disable.mangle @@ -2,8 +2,12 @@ # This is useful for sandboxes to avoid unexpected cron jobs runs # See also triggers-disable.mangle +from mangle_helper import * + + def mangle(tree): - tag = tree.xpath('//hudson.triggers.TimerTrigger')[0] - for t in tag: - tag.remove(t) - tag.text = None + nodes = tree.xpath('//hudson.triggers.TimerTrigger') + if not nodes: + return False + for n in nodes: + remove_node(tree, n) diff --git a/mangle-jobs/mangle_helper.py b/mangle-jobs/mangle_helper.py index 363d2c2..209b310 100644 --- a/mangle-jobs/mangle_helper.py +++ b/mangle-jobs/mangle_helper.py @@ -30,6 +30,8 @@ def add_or_replace_node(tree, node_xpath, node_text): parent = tree.xpath(parent_xpath)[0] parent.append(new_node) +def remove_node(tree, node): + node.getparent().remove(node) def add_child(tree, node_xpath, node_text): "Add new node (node_text) as last child of node_xpath." |