From 53defc9eec4b39fff7f9295edecd7d7466a16acb Mon Sep 17 00:00:00 2001
From: Rafael Guterres Jeffman <rjeffman@redhat.com>
Date: Thu, 30 Sep 2021 21:17:15 -0300
Subject: [PATCH] permission: Use IPAAnsibleModule method to validate
arguments.
Use the IPAAnsibleModule.params_fail_if_used method to validate
arguments provided by user.
---
plugins/modules/ipapermission.py | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/plugins/modules/ipapermission.py b/plugins/modules/ipapermission.py
index 657d934f..b1016133 100644
--- a/plugins/modules/ipapermission.py
+++ b/plugins/modules/ipapermission.py
@@ -304,11 +304,7 @@ def main():
invalid += ["right", "attrs", "memberof",
"extra_target_filter", "rawfilter"]
- 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 bindtype == "self" and ansible_module.ipa_check_version("<", "4.8.7"):
ansible_module.fail_json(
--
GitLab