Removed \'default\' variables in favor of a more readable case statement.

This commit is contained in:
Schonecker, Brian 2015-11-20 12:25:08 -05:00
parent c73fbc11e7
commit ac68d2fbbd

View File

@ -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'
}