From 58275ecd132d6f56529592baf02b207ee89f73dc Mon Sep 17 00:00:00 2001
From: Adam Warner <github@adamwarner.co.uk>
Date: Wed, 22 Mar 2023 21:52:39 +0000
Subject: [PATCH] Revert "Ignore commented lines when reading PRIVACYLEVEL from
 config file"

---
 advanced/Scripts/utils.sh          | 22 ++--------------------
 automated install/basic-install.sh |  3 +--
 test/test_any_utils.py             | 16 ----------------
 3 files changed, 3 insertions(+), 38 deletions(-)

diff --git a/advanced/Scripts/utils.sh b/advanced/Scripts/utils.sh
index 2d3c7fb1..f655e56c 100755
--- a/advanced/Scripts/utils.sh
+++ b/advanced/Scripts/utils.sh
@@ -44,7 +44,7 @@ addOrEditKeyValPair() {
 }
 
 #######################
-# Takes two arguments: file and key.
+# Takes two arguments: file, and key.
 # Adds a key to target file
 #
 # Example usage:
@@ -68,7 +68,7 @@ addKey(){
 }
 
 #######################
-# Takes two arguments: file and key.
+# Takes two arguments: file, and key.
 # Deletes a key or key/value pair from target file
 #
 # Example usage:
@@ -80,24 +80,6 @@ removeKey() {
   sed -i "/^${key}/d" "${file}"
 }
 
-#######################
-# Takes two arguments: file and key.
-# Returns the value of a given key from target file
-# - ignores all commented lines
-# - only returns the first value if multiple identical keys exist
-#
-#
-# Example usage:
-# getVal "/etc/pihole/setupVars.conf" "PIHOLE_DNS_1"
-#######################
-getVal() {
-  local file="${1}"
-  local key="${2}"
-  local value
-  value=$(sed -e '/^[[:blank:]]*#/d' "${file}" | grep "${key}" | awk -F "=" 'NR==1{printf$2}')
-  printf "%s" "$value"
-}
-
 
 #######################
 # returns FTL's current telnet API port based on the setting in /etc/pihole-FTL.conf
diff --git a/automated install/basic-install.sh b/automated install/basic-install.sh
index a781f8c6..ccb5eac7 100755
--- a/automated install/basic-install.sh	
+++ b/automated install/basic-install.sh	
@@ -2612,8 +2612,7 @@ main() {
 
         # Get the privacy level if it exists (default is 0)
         if [[ -f "${FTL_CONFIG_FILE}" ]]; then
-            # use getVal from utils.sh to get PRIVACYLEVEL
-            PRIVACY_LEVEL=$(getVal "${FTL_CONFIG_FILE}" "PRIVACYLEVEL")
+            PRIVACY_LEVEL=$(sed -ne 's/PRIVACYLEVEL=\(.*\)/\1/p' "${FTL_CONFIG_FILE}")
 
             # If no setting was found, default to 0
             PRIVACY_LEVEL="${PRIVACY_LEVEL:-0}"
diff --git a/test/test_any_utils.py b/test/test_any_utils.py
index cb5ddeaa..b3fabe6c 100644
--- a/test/test_any_utils.py
+++ b/test/test_any_utils.py
@@ -82,22 +82,6 @@ def test_key_removal_works(host):
     assert expected_stdout == output.stdout
 
 
-def test_get_value_works(host):
-    """Confirms getVal returns the correct value for a given key"""
-    output = host.run(
-        """
-    source /opt/pihole/utils.sh
-    echo "Somekey=xxx" >> /tmp/testfile
-    echo "#Testkey=1234" >> /tmp/testfile
-    echo "Testkey=5678" >> /tmp/testfile
-    echo "Testkey=abcd" >> /tmp/testfile
-    getVal "/tmp/testfile" "Testkey"
-    """
-    )
-    expected_stdout = "5678"
-    assert expected_stdout == output.stdout
-
-
 def test_getFTLAPIPort_default(host):
     """Confirms getFTLAPIPort returns the default API port"""
     output = host.run(