From ac68d2fbbd15f26acace0f013972bf337b985e6c Mon Sep 17 00:00:00 2001 From: "Schonecker, Brian" Date: Fri, 20 Nov 2015 12:25:08 -0500 Subject: [PATCH] Removed \'default\' variables in favor of a more readable case statement. --- manifests/params.pp | 16 ++++++---------- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/manifests/params.pp b/manifests/params.pp index 6a179e5..73a7f66 100644 --- a/manifests/params.pp +++ b/manifests/params.pp @@ -19,7 +19,6 @@ class nslcd::params { $ldap_tls_reqcert = 'allow' $ldap_tls_cacertfile = undef $ldap_tls_cacertder = undef - $default_config = '/etc/nslcd.conf' $bind_timelimit = 10 $timelimit = 0 $idle_timelimit = 0 @@ -31,10 +30,9 @@ class nslcd::params { case $::osfamily { 'Debian': { - $default_package_name = 'nslcd' - $config = $default_config - $package_name = $default_package_name - $service_name = $default_service_name + $config = '/etc/nslcd.conf' + $package_name = 'nslcd' + $service_name = 'nslcd' $uid = 'nslcd' $gid = 'nslcd' $config_user = 'root' @@ -42,10 +40,9 @@ class nslcd::params { $config_mode = '0640' } 'RedHat': { - $default_package_name = 'nss-pam-ldapd' - $config = $default_config - $package_name = $default_package_name - $service_name = $default_service_name + $config = '/etc/nslcd.conf' + $package_name = 'nss-pam-ldapd' + $service_name = 'nslcd' $uid = 'nslcd' $gid = 'root' $config_user = 'root' @@ -56,7 +53,6 @@ class nslcd::params { fail("The ${module_name} module is not supported on an ${::osfamily} based system.") } } - $default_service_name = 'nslcd' }