Bugfix cannot reassign variable fr_service

Bugfix cannot reassign variable fr_service Bugfix unknown variable in ldap module
djjudas21 · Sep 8, 2024 · 77c6ce8 · 77c6ce8
1 parent 738232d
commit 77c6ce8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/manifests/module/ldap.pp b/manifests/module/ldap.pp
@@ -74,7 +74,7 @@
   # FR3.1 format server = 'ldap1.example.com'
   #              server = 'ldap2.example.com'
   #              server = 'ldap3.example.com'
-  $serverconcatarray = $::freeradius_version ? {
+  $serverconcatarray = $facts['freeradius_version'] ? {
     /^3\.0\./ => any2array(join($server, ',')),
     default   => $server,
   }

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -75,7 +75,7 @@
   }
 
   # Whether the FreeRADIUS init.d startup script has a status setting or not
-  $fr_service = $facts['os']['family'] ? {
+  $fr_service_has_status = $facts['os']['family'] ? {
     'RedHat' => true,
     'Debian' => true,
     default  => false,