diff --git a/plugins/modules/ipasudocmdgroup.py b/plugins/modules/ipasudocmdgroup.py index e260b699a055b3443ed580790f27f735fdf186f8..42f29fb87035a4f4c494235cbe6659157121a02a 100644 --- a/plugins/modules/ipasudocmdgroup.py +++ b/plugins/modules/ipasudocmdgroup.py @@ -168,6 +168,7 @@ def main(): state = ansible_module.params_get("state") # Check parameters + invalid = [] if state == "present": if len(names) != 1: @@ -175,11 +176,6 @@ def main(): msg="Only one sudocmdgroup can be added at a time.") if action == "member": invalid = ["description", "nomembers"] - for x in invalid: - if vars()[x] is not None: - ansible_module.fail_json( - msg="Argument '%s' can not be used with action " - "'%s'" % (x, action)) if state == "absent": if len(names) < 1: @@ -188,11 +184,8 @@ def main(): invalid = ["description", "nomembers"] if action == "sudocmdgroup": invalid.extend(["sudocmd"]) - for x in invalid: - if vars()[x] is not None: - ansible_module.fail_json( - msg="Argument '%s' can not be used with state '%s'" % - (x, state)) + + ansible_module.params_fail_used_invalid(invalid, state, action) # Init