From 35d765883479fd950f4357cf54d46ef448055885 Mon Sep 17 00:00:00 2001
From: Sergio Oliveira Campos <seocam@redhat.com>
Date: Wed, 25 Mar 2020 19:00:47 -0300
Subject: [PATCH] Added alias module arguments in dnszone module

---
 README-dnszone.md             |  3 ++-
 plugins/modules/ipadnszone.py | 25 +++++++++++++++++++++----
 2 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/README-dnszone.md b/README-dnszone.md
index 60cc989d..766efe53 100644
--- a/README-dnszone.md
+++ b/README-dnszone.md
@@ -163,6 +163,7 @@ Variable | Description | Required
 -------- | ----------- | --------
 `ipaadmin_principal` | The admin principal is a string and defaults to `admin` | no
 `ipaadmin_password` | The admin password is a string and is required if there is no admin ticket available on the node | no
+`name` \| `zone_name` | The zone name string. | yes
 `forwarders` | The list of forwarders dicts. Each `forwarders` dict entry has:| no
 &nbsp; | `ip_address` - The IPv4 or IPv6 address of the DNS server. | yes
 &nbsp; | `port` - The custom port that should be used on this server. | no
@@ -172,7 +173,7 @@ Variable | Description | Required
 `name_server`| Authoritative nameserver domain name | no
 `admin_email`| Administrator e-mail address | no
 `update_policy`| BIND update policy | no
-`dynamic_update`| Allow dynamic updates | no
+`dynamic_update` \| `dynamicupdate` | Allow dynamic updates | no
 `dnssec`| Allow inline DNSSEC signing of records in the zone | no
 `allow_transfer`| List of IP addresses or networks which are allowed to transfer the zone | no
 `allow_query`| List of IP addresses or networks which are allowed to issue queries | no
diff --git a/plugins/modules/ipadnszone.py b/plugins/modules/ipadnszone.py
index 52a66c27..717978e2 100644
--- a/plugins/modules/ipadnszone.py
+++ b/plugins/modules/ipadnszone.py
@@ -38,6 +38,11 @@ options:
     description: The admin password
     required: false
 
+  name:
+    description: The zone name string.
+    required: true
+    type: str
+    alises: ["zone_name"]
   forwarders:
     description: The list of global DNS forwarders.
     required: false
@@ -71,10 +76,15 @@ options:
     description: Administrator e-mail address
     required: false
     type: str
-  update_policy: BIND update policy
-    description: Allow dynamic updates
+  update_policy:
+    description: BIND update policy
     required: false
     type: str
+  dynamic_update:
+    description: Allow dynamic updates
+    required: false
+    type: bool
+    alises: ["dynamicupdate"]
   dnssec:
     description: Allow inline DNSSEC signing of records in the zone
     required: false
@@ -415,7 +425,9 @@ def get_argument_spec():
         ),
         ipaadmin_principal=dict(type="str", default="admin"),
         ipaadmin_password=dict(type="str", required=False, no_log=True),
-        name=dict(type="str", default=None, required=True),
+        name=dict(
+            type="str", default=None, required=True, aliases=["zone_name"]
+        ),
         forwarders=dict(
             type="list",
             default=None,
@@ -432,7 +444,12 @@ def get_argument_spec():
         admin_email=dict(type="str", required=False, default=None),
         allow_sync_ptr=dict(type="bool", required=False, default=None),
         update_policy=dict(type="str", required=False, default=None),
-        dynamic_update=dict(type="bool", required=False, default=None),
+        dynamic_update=dict(
+            type="bool",
+            required=False,
+            default=None,
+            aliases=["dynamicupdate"],
+        ),
         dnssec=dict(type="bool", required=False, default=None),
         allow_transfer=dict(type="list", required=False, default=None),
         allow_query=dict(type="list", required=False, default=None),
-- 
GitLab