mirror of
https://github.com/freeipa/ansible-freeipa.git
synced 2026-05-06 21:33:14 +00:00
delegation: Use IPAAnsibleModule method to validate arguments.
Use the IPAAnsibleModule.params_fail_if_used method to validate arguments provided by user.
This commit is contained in:
@@ -176,17 +176,14 @@ def main():
|
||||
|
||||
# Check parameters
|
||||
|
||||
invalid = []
|
||||
|
||||
if state == "present":
|
||||
if len(names) != 1:
|
||||
ansible_module.fail_json(
|
||||
msg="Only one delegation be added at a time.")
|
||||
if action == "member":
|
||||
invalid = ["permission", "membergroup", "group"]
|
||||
for x in invalid:
|
||||
if vars()[x] is not None:
|
||||
ansible_module.fail_json(
|
||||
msg="Argument '%s' can not be used with action "
|
||||
"'%s' and state '%s'" % (x, action, state))
|
||||
|
||||
if state == "absent":
|
||||
if len(names) < 1:
|
||||
@@ -194,11 +191,8 @@ def main():
|
||||
invalid = ["permission", "membergroup", "group"]
|
||||
if action == "delegation":
|
||||
invalid.append("attribute")
|
||||
for x in invalid:
|
||||
if vars()[x] is not None:
|
||||
ansible_module.fail_json(
|
||||
msg="Argument '%s' can not be used with action "
|
||||
"'%s' and state '%s'" % (x, action, state))
|
||||
|
||||
ansible_module.params_fail_used_invalid(invalid, state, action)
|
||||
|
||||
if permission is not None:
|
||||
perm = [p for p in permission if p not in ("read", "write")]
|
||||
|
||||
Reference in New Issue
Block a user