Refactor autofs template

Author: silug

manifests/service/autofs.pp

@@ -37,15 +37,50 @@
       },
     )
   } else {
+    # Build configuration content for the AutoFS service
+    $_base_content = [
+      '# sssd::service::autofs',
+      '[autofs]',
+    ]
+
+    # Add conditional parameters if defined
+    if $description {
+      $_description_entries = ["description = ${description}"]
+    } else {
+      $_description_entries = []
+    }
+
+    if $debug_level {
+      $_debug_level_entries = ["debug_level = ${debug_level}"]
+    } else {
+      $_debug_level_entries = []
+    }
+
+    $_debug_timestamps_entries = $debug_timestamps ? {
+      true  => ['debug_timestamps = true'],
+      false => ['debug_timestamps = false'],
+    }
+
+    $_debug_microseconds_entries = $debug_microseconds ? {
+      true  => ['debug_microseconds = true'],
+      false => ['debug_microseconds = false'],
+    }
+
+    if $autofs_negative_timeout {
+      $_autofs_negative_timeout_entries = ["autofs_negative_timeout = ${autofs_negative_timeout}"]
+    } else {
+      $_autofs_negative_timeout_entries = []
+    }
+
+    # Combine all configuration entries in the expected order
+    $_all_entries = $_base_content + $_description_entries + $_debug_level_entries + $_debug_timestamps_entries + $_debug_microseconds_entries + $_autofs_negative_timeout_entries
+
+    $_final_content = $_all_entries.join("\n")
+
     $_content = epp(
       "${module_name}/service/autofs.epp",
       {
-        'description'              => $description,
-        'debug_level'              => $debug_level,
-        'debug_timestamps'         => $debug_timestamps,
-        'debug_microseconds'       => $debug_microseconds,
-        'autofs_negative_timeout'  => $autofs_negative_timeout,
-        'custom_options'           => $custom_options,
+        'content' => $_final_content,
       },
     )
   }

templates/service/autofs.epp

@@ -1,22 +1,4 @@
-<% |
-  Optional[String]            $description,
-  Optional[Sssd::DebugLevel]  $debug_level,
-  Boolean                     $debug_timestamps,
-  Boolean                     $debug_microseconds,
-  Optional[Integer]           $autofs_negative_timeout,
-  Optional[Hash]              $custom_options,
+<%- |
+  String $content,
 | -%>
-
-# sssd::service::autofs
-[autofs]
-<% if $description { -%>
-description = <%= $description %>
-<% } -%>
-<% if $debug_level { -%>
-debug_level = <%= $debug_level %>
-<% } -%>
-debug_timestamps = <%= $debug_timestamps %>
-debug_microseconds = <%= $debug_microseconds %>
-<% if $autofs_negative_timeout { -%>
-autofs_negative_timeout = <%= $autofs_negative_timeout %>
-<% } -%>
+<%= $content %>