diff --git a/api/release_hold.php b/api/release_hold.php index 6871354..d464175 100644 --- a/api/release_hold.php +++ b/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'; diff --git a/conversations.php b/conversations.php index 0f30cf3..2987671 100644 --- a/conversations.php +++ b/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 { diff --git a/services/BotService.php b/services/BotService.php index 0874d64..af2444f 100644 --- a/services/BotService.php +++ b/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()); }