From 168e8f5909e8ab5333663832aa71becac355d4fb Mon Sep 17 00:00:00 2001
From: Developer <dev@localhost>
Date: Mon, 20 Apr 2026 19:56:36 -0400
Subject: [PATCH] FIX: Remove all debug messages from php-analyzer functions

Removed echo statements to stderr that could interfere with function
return values if captured together with stdout:
- calculate_balanced_memory_allocation()
- calculate_balanced_memory_allocation_per_domain()
- Domain traffic analysis messages

These could cause similar 'integer expression expected' errors if
called with stderr capture (2>&1)
---
 lib/php-analyzer.sh | 7 -------
 1 file changed, 7 deletions(-)

diff --git a/lib/php-analyzer.sh b/lib/php-analyzer.sh
index 6545372..7ac49ce 100644
--- a/lib/php-analyzer.sh
+++ b/lib/php-analyzer.sh
@@ -1030,8 +1030,6 @@ calculate_server_memory_capacity() {
 # Usage: calculate_balanced_memory_allocation
 # Returns: recommendations for each user to fit within system limits
 calculate_balanced_memory_allocation() {
-    echo "Calculating balanced memory allocation..." >&2
-
     # Get total system memory
     local total_ram_mb
     total_ram_mb=$(free -m | awk '/^Mem:/ {print $2}')
@@ -1167,8 +1165,6 @@ calculate_balanced_memory_allocation() {
 # Usage: calculate_balanced_memory_allocation_per_domain
 # Returns: recommendations for each domain to fit within system limits
 calculate_balanced_memory_allocation_per_domain() {
-    echo "Calculating per-domain balanced memory allocation (cPanel)..." >&2
-
     # Verify this is cPanel
     if [ "$SYS_CONTROL_PANEL" != "cpanel" ]; then
         echo "ERROR|This function only supports cPanel. Use calculate_balanced_memory_allocation for other panels."
@@ -1268,8 +1264,6 @@ calculate_balanced_memory_allocation_per_domain() {
             domain_memory[$domain]=$((avg_kb / 1024))
 
             # Get advanced traffic stats for this domain (7-day, bot-filtered, 95th percentile)
-            echo "  Analyzing traffic for $domain..." >&2
-
             local traffic
             # Try fast method first (current process count)
             local current_processes
@@ -1278,7 +1272,6 @@ calculate_balanced_memory_allocation_per_domain() {
             if [ "$current_processes" -gt 0 ]; then
                 # Use current process count as baseline (fast, no log parsing)
                 traffic=$((current_processes * 2))  # Assume processes can handle ~2 req/min each
-                echo "  Using current process count: $current_processes processes" >&2
             else
                 # Fallback to traffic analysis only if no processes found
                 local traffic_stats