up

api/release_hold.php

@@ -20,6 +20,14 @@ try {
         exit;
     }
 
+    // Only release hold if there's an active hold/request; otherwise return success quietly
+    $userFlags = $db->fetch('SELECT on_hold, advisor_requested FROM users WHERE id = :id', ['id' => $userId]);
+    if (empty($userFlags) || (empty($userFlags['on_hold']) && empty($userFlags['advisor_requested']))) {
+        // Nothing to release
+        echo json_encode(['success' => true, 'info' => 'no_hold_active']);
+        exit;
+    }
+
     // Use BotService to release hold (sends notification and clears flags)
     try {
         require_once __DIR__ . '/../services/BotService.php';

conversations.php

@@ -2006,6 +2006,9 @@
                                         this._userStateCache = this._userStateCache || {};
                                         this._userStateCache[userId] = { state: { in_service: false, advisor_requested: false, on_hold: false }, ts: Date.now() };
 
+                                        // ocultar botón Liberar espera si existe
+                                        if (releaseBtn) releaseBtn.style.display = 'none';
+
                                         await this.loadConversations();
                                         await this.loadMessages(userId, true);
                                     } else {

services/BotService.php

@@ -69,10 +69,11 @@ class BotService {
                     $this->requestAdvisor($phoneNumber, 3);
 
                     // Registrar mensaje de sistema para trazabilidad (no altera flujo de notificación)
+                    $now = date('Y-m-d H:i:s');
                     $sys = [
                         'system' => 'attention',
                         'type' => 'user_sent_documents',
-                        'text' => 'El usuario indicó que ha enviado documentos o información; requiere atención.'
+                        'text' => "📄 *Documentos recibidos*\n\nEl usuario confirmó que ha enviado documentación (mensaje: *ENVIADA*).\n\n• Usuario: {$phoneNumber} (ID: {$user['id']})\n• Hora: {$now}\n\nPor favor, revise los archivos en la conversación y atiéndalo lo antes posible."
                     ];
 
                     $this->db->insert('conversations', [
@@ -976,13 +977,18 @@ class BotService {
         }
     }
 
-    public function releaseHold($phoneNumber) {
+    public function releaseHold($phoneNumber, $notify = true) {
         try {
             // Clear hold/advisor flags and also ensure in_service is cleared for consistency
             // Clear hold and reactivate bot
             $this->db->update('users', ['on_hold' => 0, 'bot_paused_until' => null, 'advisor_requested' => 0, 'in_service' => 0, 'in_service_by' => null, 'in_service_at' => null, 'bot_enabled' => 1], 'phone_number = :phone', ['phone' => $phoneNumber]);
-            $this->whatsappService->sendTextMessage($phoneNumber, "🔔 Su conversación ha sido retomada por el equipo. Puede usar *MENU* para continuar.");
+
-            if (function_exists('writeLog')) writeLog('INFO', "Advisor released hold for $phoneNumber");
+            // Send notification only when requested (avoid duplicate messages after finish)
+            if ($notify) {
+                $this->whatsappService->sendTextMessage($phoneNumber, "🔔 Su conversación ha sido retomada por el equipo. Puede usar *MENU* para continuar.");
+            }
+
+            if (function_exists('writeLog')) writeLog('INFO', "Advisor released hold for $phoneNumber notify=" . ($notify ? '1' : '0'));
         } catch (Exception $e) {
             error_log('releaseHold failed: ' . $e->getMessage());
         }
@@ -1074,7 +1080,8 @@ class BotService {
 
             // Ensure consistent behavior: release hold / clear flags when finishing attend
             try {
-                $this->releaseHold($phoneNumber);
+                // Release hold silently to avoid duplicate notification to the user
+                $this->releaseHold($phoneNumber, false);
             } catch (Exception $e) {
                 error_log('finishAttendConversation: releaseHold failed: ' . $e->getMessage());
             }