From ddc9a68a4000b4b322a9ee7aa44c2a81baf5d724 Mon Sep 17 00:00:00 2001
From: lizandrogd <77708265+lizandrogd@users.noreply.github.com>
Date: Wed, 21 Jan 2026 15:12:15 -0500
Subject: [PATCH] up

---
 api/release_hold.php    | 14 +++++++++++---
 conversations.php       | 25 +++++++++++++++++++++++--
 index.php               |  2 +-
 services/BotService.php | 16 ++++++++++++++++
 4 files changed, 51 insertions(+), 6 deletions(-)

diff --git a/api/release_hold.php b/api/release_hold.php
index be7a4c1..6871354 100644
--- a/api/release_hold.php
+++ b/api/release_hold.php
@@ -20,9 +20,17 @@ try {
         exit;
     }
 
-    $db->update('users', ['on_hold' => 0], 'id = :id', ['id' => $userId]);
-
-    echo json_encode(['success' => true]);
+    // Use BotService to release hold (sends notification and clears flags)
+    try {
+        require_once __DIR__ . '/../services/BotService.php';
+        $bot = new BotService();
+        $bot->releaseHold($user['phone_number']);
+        echo json_encode(['success' => true]);
+    } catch (Exception $e) {
+        // Fallback to direct DB update if BotService fails
+        $db->update('users', ['on_hold' => 0, 'advisor_requested' => 0, 'bot_paused_until' => null], 'id = :id', ['id' => $userId]);
+        echo json_encode(['success' => true, 'warning' => 'BotService failed, flags cleared directly']);
+    }
 } catch (Exception $e) {
     http_response_code(500);
     echo json_encode(['success' => false, 'error' => $e->getMessage()]);
diff --git a/conversations.php b/conversations.php
index 6a23b94..4db4910 100644
--- a/conversations.php
+++ b/conversations.php
@@ -843,11 +843,23 @@
                     const releaseBtn = document.getElementById('release-hold-btn');
 
                     if (holdIndicator) {
-                        holdIndicator.style.display = conv.on_hold ? 'inline' : 'none';
+                        // Show different labels depending on state
+                        if (conv.on_hold) {
+                            holdIndicator.textContent = 'EN ESPERA';
+                            holdIndicator.style.color = '#b85';
+                            holdIndicator.style.display = 'inline';
+                        } else if (conv.advisor_requested) {
+                            holdIndicator.textContent = 'SOLICITUD PENDIENTE';
+                            holdIndicator.style.color = '#f39c12';
+                            holdIndicator.style.display = 'inline';
+                        } else {
+                            holdIndicator.style.display = 'none';
+                        }
                     }
 
                     if (releaseBtn) {
-                        releaseBtn.style.display = conv.on_hold ? 'inline-block' : 'none';
+                        // show button when on_hold or advisor_requested
+                        releaseBtn.style.display = (conv.on_hold || conv.advisor_requested) ? 'inline-block' : 'none';
                         releaseBtn.onclick = async () => {
                             try {
                                 const resp = await fetch('api/release_hold.php', {
@@ -855,9 +867,14 @@
                                     headers: { 'Content-Type': 'application/json' },
                                     body: JSON.stringify({ user_id: userId })
                                 });
+                                if (resp.status === 401) {
+                                    alert('Sesión expirada. Por favor inicie sesión de nuevo.');
+                                    return;
+                                }
                                 const json = await resp.json();
                                 if (json && json.success) {
                                     conv.on_hold = false;
+                                    conv.advisor_requested = 0;
                                     holdIndicator.style.display = 'none';
                                     releaseBtn.style.display = 'none';
                                     await this.loadConversations();
@@ -866,6 +883,10 @@
                                 }
                             } catch (e) {
                                 console.error('Error releasing hold', e);
+                                alert('Error liberando espera');
+                            }
+                        };
+                    }
                             }
                         };
                     }
diff --git a/index.php b/index.php
index 3b82e2f..cb3c0d7 100644
--- a/index.php
+++ b/index.php
@@ -653,7 +653,7 @@ try {
     <!-- Scripts -->
     <script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/js/bootstrap.bundle.min.js"></script>
     <script src="https://cdn.jsdelivr.net/npm/chart.js"></script>
-    <script src="./assets/js/app_simple.js?v=4"></script>
+    <script src="./assets/js/app_simple.js?v=5"></script>
     <script>
         // Función de diagnóstico mejorada
         function runDiagnostic() {
diff --git a/services/BotService.php b/services/BotService.php
index 206a608..abebd50 100644
--- a/services/BotService.php
+++ b/services/BotService.php
@@ -24,6 +24,14 @@ class BotService {
         
         $messageText = trim($messageText);
         $phoneNumber = $user['phone_number'];
+
+        // DEBUG: log basic context
+        try {
+            $userId = $user['id'] ?? 'unknown';
+            error_log("[BotService] processMessage start - user_id={$userId} phone={$phoneNumber} type={$messageType} message='" . substr($messageText,0,200) . "' current_menu_id=" . ($user['current_menu_id'] ?? 'null') . " on_hold=" . ($user['on_hold'] ?? '0') . " advisor_requested=" . ($user['advisor_requested'] ?? '0') . " bot_paused_until=" . ($user['bot_paused_until'] ?? 'null'));
+        } catch (Throwable $t) {
+            // ignore logging errors
+        }
         
         // Verificar si es un nuevo usuario (enviar mensaje de bienvenida)
         if ($this->isNewUser($user['id'])) {
@@ -50,6 +58,7 @@ class BotService {
                 } catch (Exception $e) {
                     // ignore
                 }
+                error_log("[BotService] processMessage - returning early because on_hold active for user {$user['id']}");
                 return;
             } else {
                 // El hold expiró, liberar y continuar
@@ -67,6 +76,7 @@ class BotService {
                 } catch (Exception $e) {
                     // ignore
                 }
+                error_log("[BotService] processMessage - returning early because advisor_requested active for user {$user['id']}");
                 return;
             } else {
                 // expiró la espera: limpiar flag
@@ -244,12 +254,16 @@ class BotService {
         $phoneNumber = $user['phone_number'];
         $currentMenuId = $user['current_menu_id'];
         
+        // DEBUG
+        try { error_log("[BotService] processMenuSelection - user_id={$user['id']} menu_id={$currentMenuId} received='{$messageText}'"); } catch (Throwable $t) {}
+
         // Verificar si es un número
         if (!is_numeric($messageText)) {
             $this->whatsappService->sendTextMessage(
                 $phoneNumber, 
                 "❌ Por favor, responde solo con el número de la opción deseada."
             );
+            error_log("[BotService] processMenuSelection - non-numeric response from user {$user['id']}: '{$messageText}'");
             return;
         }
         
@@ -266,9 +280,11 @@ class BotService {
                 $phoneNumber, 
                 "❌ Opción inválida. Por favor, selecciona una opción válida del menú."
             );
+            error_log("[BotService] processMenuSelection - no option found for user {$user['id']} menu={$currentMenuId} option={$optionNumber}");
             return;
         }
         
+        error_log("[BotService] processMenuSelection - option found for user {$user['id']} menu={$currentMenuId} option={$optionNumber} action={$option['action_type']}");
         // Procesar acción de la opción
         $this->processMenuAction($phoneNumber, $option);
     }