Skip to content
Snippets Groups Projects
Commit 2ec65e91 authored by Rafael Guterres Jeffman's avatar Rafael Guterres Jeffman
Browse files

user: Use IPAAnsibleModule method to validate arguments.

Use the IPAAnsibleModule.params_fail_if_used method to validate
arguments provided by user.
parent 5a67aa77
Branches
Tags
No related merge requests found
...@@ -597,6 +597,7 @@ def check_parameters( # pylint: disable=unused-argument ...@@ -597,6 +597,7 @@ def check_parameters( # pylint: disable=unused-argument
userauthtype, userclass, radius, radiususer, departmentnumber, userauthtype, userclass, radius, radiususer, departmentnumber,
employeenumber, employeetype, preferredlanguage, certificate, employeenumber, employeetype, preferredlanguage, certificate,
certmapdata, noprivate, nomembers, preserve, update_password): certmapdata, noprivate, nomembers, preserve, update_password):
invalid = []
if state == "present": if state == "present":
if action == "member": if action == "member":
invalid = ["first", "last", "fullname", "displayname", "initials", invalid = ["first", "last", "fullname", "displayname", "initials",
...@@ -608,11 +609,6 @@ def check_parameters( # pylint: disable=unused-argument ...@@ -608,11 +609,6 @@ def check_parameters( # pylint: disable=unused-argument
"departmentnumber", "employeenumber", "employeetype", "departmentnumber", "employeenumber", "employeetype",
"preferredlanguage", "noprivate", "nomembers", "preferredlanguage", "noprivate", "nomembers",
"preserve", "update_password"] "preserve", "update_password"]
for x in invalid:
if vars()[x] is not None:
module.fail_json(
msg="Argument '%s' can not be used with action "
"'%s'" % (x, action))
else: else:
invalid = ["first", "last", "fullname", "displayname", "initials", invalid = ["first", "last", "fullname", "displayname", "initials",
...@@ -628,16 +624,13 @@ def check_parameters( # pylint: disable=unused-argument ...@@ -628,16 +624,13 @@ def check_parameters( # pylint: disable=unused-argument
invalid.extend(["principal", "manager", invalid.extend(["principal", "manager",
"certificate", "certmapdata", "certificate", "certmapdata",
]) ])
for x in invalid:
if vars()[x] is not None:
module.fail_json(
msg="Argument '%s' can not be used with state '%s'" %
(x, state))
if state != "absent" and preserve is not None: if state != "absent" and preserve is not None:
module.fail_json( module.fail_json(
msg="Preserve is only possible for state=absent") msg="Preserve is only possible for state=absent")
module.params_fail_used_invalid(invalid, state, action)
if certmapdata is not None: if certmapdata is not None:
for x in certmapdata: for x in certmapdata:
certificate = x.get("certificate") certificate = x.get("certificate")
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment