From 9df3baa5f1fc7c14bc6a17b55c32042d0c7becc8 Mon Sep 17 00:00:00 2001
From: billz <billzimmerman@gmail.com>
Date: Sun, 20 Jul 2025 02:08:33 -0700
Subject: [PATCH] Minor: standardize comment header, declare strict_types=1

---
 src/RaspAP/Networking/Hotspot/DhcpcdManager.php | 17 ++++++++++++-----
 .../Networking/Hotspot/DnsmasqManager.php       |  7 ++++---
 .../Hotspot/Validators/HostapdValidator.php     | 11 +++++++----
 src/RaspAP/Networking/Hotspot/WiFiManager.php   | 10 ++++++----
 4 files changed, 29 insertions(+), 16 deletions(-)

diff --git a/src/RaspAP/Networking/Hotspot/DhcpcdManager.php b/src/RaspAP/Networking/Hotspot/DhcpcdManager.php
index 9e397112..50fc9433 100644
--- a/src/RaspAP/Networking/Hotspot/DhcpcdManager.php
+++ b/src/RaspAP/Networking/Hotspot/DhcpcdManager.php
@@ -1,12 +1,19 @@
 <?php
 
+/**
+ * A dhcpcd configuration class for RaspAP
+ *
+ * @description Handles building, saving and safe updating of dhcpcd configs
+ * @author      Bill Zimmerman <billzimmerman@gmail.com>
+ * @license     https://github.com/raspap/raspap-webgui/blob/master/LICENSE
+ */
+
+declare(strict_types=1);
+
 namespace RaspAP\Networking\Hotspot;
 
 use RaspAP\Messages\StatusMessage;
 
-/**
- * Handles dhcpcd.conf interface configuration.
- */
 class DhcpcdManager
 {
     private const CONF_DEFAULT = RASPI_DHCPCD_CONFIG;
@@ -99,6 +106,7 @@ class DhcpcdManager
             );
         }
         $dhcp_cfg = file_get_contents(SELF::CONF_DEFAULT);
+        $skip_dhcp = false;
 
         if (preg_match('/wlan[3-9]\d*|wlan[1-9]\d+/', $ap_iface)) {
             $skip_dhcp = true;
@@ -280,8 +288,7 @@ class DhcpcdManager
             $lines = [];
             exec('cat ' . escapeshellarg($dnsmasqFile), $lines);
             if (!function_exists('ParseConfig')) {
-                // ensure legacy parser available
-                require_once RASPI_CONFIG . '/functions.php';
+                require_once 'includes/functions.php';
             }
             $conf = ParseConfig($lines);
 
diff --git a/src/RaspAP/Networking/Hotspot/DnsmasqManager.php b/src/RaspAP/Networking/Hotspot/DnsmasqManager.php
index 94a45803..0b4cd81b 100644
--- a/src/RaspAP/Networking/Hotspot/DnsmasqManager.php
+++ b/src/RaspAP/Networking/Hotspot/DnsmasqManager.php
@@ -1,7 +1,5 @@
 <?php
 
-namespace RaspAP\Networking\Hotspot;
-
 /**
  * A dnsmasq configuration manager for RaspAP
  *
@@ -10,6 +8,10 @@ namespace RaspAP\Networking\Hotspot;
  * @license     https://github.com/raspap/raspap-webgui/blob/master/LICENSE
  */
 
+declare(strict_types=1);
+
+namespace RaspAP\Networking\Hotspot;
+
 class DnsmasqManager
 {
     private const CONF_SUFFIX = '.conf';
@@ -115,7 +117,6 @@ class DnsmasqManager
         $configFile = RASPI_DNSMASQ_PREFIX . $iface . self::CONF_SUFFIX;
         $tempFile = SELF::CONF_TMP; 
         $config = join(PHP_EOL, $config);
-        error_log('[DnsmasqManager::saveConfig] $config = ' . var_export($config, true));
 
         file_put_contents($tempFile, $config);
         $cmd = sprintf('sudo cp %s %s', escapeshellarg($tempFile), escapeshellarg($configFile));
diff --git a/src/RaspAP/Networking/Hotspot/Validators/HostapdValidator.php b/src/RaspAP/Networking/Hotspot/Validators/HostapdValidator.php
index 864ff371..2a8d70af 100644
--- a/src/RaspAP/Networking/Hotspot/Validators/HostapdValidator.php
+++ b/src/RaspAP/Networking/Hotspot/Validators/HostapdValidator.php
@@ -1,9 +1,5 @@
 <?php
 
-namespace RaspAP\Networking\Hotspot\Validators;
-
-use RaspAP\Messages\StatusMessage;
-
 /**
  * Hostapd validator class for RaspAP
  *  
@@ -11,6 +7,13 @@ use RaspAP\Messages\StatusMessage;
  * @author      Bill Zimmerman <billzimmerman@gmail.com>
  * @license     https://github.com/raspap/raspap-webgui/blob/master/LICENSE
  */
+
+declare(strict_types=1);
+
+namespace RaspAP\Networking\Hotspot\Validators;
+
+use RaspAP\Messages\StatusMessage;
+
 class HostapdValidator
 {
 
diff --git a/src/RaspAP/Networking/Hotspot/WiFiManager.php b/src/RaspAP/Networking/Hotspot/WiFiManager.php
index 4e2e4a9f..bc07fe1d 100644
--- a/src/RaspAP/Networking/Hotspot/WiFiManager.php
+++ b/src/RaspAP/Networking/Hotspot/WiFiManager.php
@@ -1,14 +1,16 @@
 <?php
 
-namespace RaspAP\Networking\Hotspot;
-
 /**
- * Wireless utility class
- * @description A collection of wireless utility methods for RaspAP
+ * A wireless utility class for RaspAP
+ * @description A collection of wireless utility methods
  * @author      Bill Zimmerman <billzimmerman@gmail.com>
  * @license     https://github.com/raspap/raspap-webgui/blob/master/LICENSE
  */
 
+declare(strict_types=1);
+
+namespace RaspAP\Networking\Hotspot;
+
 class WiFiManager
 {