From 75c0817c7e24cc735abe04f5fa6ee3a7a2d19f61 Mon Sep 17 00:00:00 2001
From: cschantz <admin@server.local>
Date: Fri, 12 Dec 2025 23:15:12 -0500
Subject: [PATCH] Fix backup function to pass domain parameter

Bug fix in lib/php-config-manager.sh:
- Line 124: find_fpm_pool_config() requires both username AND domain
- Was only passing username, causing backup to fail
- Fixed: find_fpm_pool_config "$username" "$domain"

Impact:
- Backup functionality now works correctly
- Successfully backs up PHP-FPM pool configs
- Tested with pickledperil.com - backup created successfully

Verification:
- Syntax validated
- Backup test: passed
- Pool config found and backed up to /root/server-toolkit/backups/php/
---
 lib/php-config-manager.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/php-config-manager.sh b/lib/php-config-manager.sh
index 25bef61..cb1c95d 100644
--- a/lib/php-config-manager.sh
+++ b/lib/php-config-manager.sh
@@ -121,7 +121,7 @@ EOF
 
     # Backup FPM pool config
     local pool_config
-    pool_config=$(find_fpm_pool_config "$username")
+    pool_config=$(find_fpm_pool_config "$username" "$domain")
 
     if [ -n "$pool_config" ] && [ -f "$pool_config" ]; then
         local backup_path