From 77c6ce89c6ffafe81af11343b2459ff17c9765a9 Mon Sep 17 00:00:00 2001
From: Jo Rhett <geek@jorhett.com>
Date: Sat, 7 Sep 2024 13:32:06 -0700
Subject: [PATCH] Bugfix cannot reassign variable fr_service

Bugfix cannot reassign variable fr_service
Bugfix unknown variable in ldap module
---
 manifests/module/ldap.pp | 2 +-
 manifests/params.pp      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/manifests/module/ldap.pp b/manifests/module/ldap.pp
index 279e3771..dc547bbb 100644
--- 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
index 94071f25..87e3fd17 100644
--- 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,