From 671fccd6b2f9a4b5ba727063e95b8256779ef8bb Mon Sep 17 00:00:00 2001
From: Lizandro Guarnizo <77708265+lizandrogd@users.noreply.github.com>
Date: Sat, 1 Aug 2026 09:42:39 -0500
Subject: [PATCH] =?UTF-8?q?fix:=20mantenimiento=20PDF=20vacio=20=E2=80=94?=
 =?UTF-8?q?=20conservar=20grupo=5Fmant=20y=20detectar=20placeholders?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

preservingReset borraba grupo_mant despues de cada informe, dejando
{grupo_id} sin resolver en la siguiente llamada del mismo submenu.
La API recibia grupo=0 y devolvia cero filas sin error visible.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 services/NormalBot.php | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/services/NormalBot.php b/services/NormalBot.php
index ce97803..acd2787 100644
--- a/services/NormalBot.php
+++ b/services/NormalBot.php
@@ -1152,7 +1152,9 @@ class NormalBot
     private static function preservingReset(int $ctxId, array $meta): void
     {
         $keep = [];
-        foreach (['finca'] as $group) {
+        // grupo_mant se conserva: el submenú de mantenimiento ofrece varios informes
+        // sobre el mismo grupo, y perderlo dejaba {grupo_id} sin resolver.
+        foreach (['finca', 'grupo_mant'] as $group) {
             if (isset($meta[$group])) $keep[$group] = $meta[$group];
         }
         ConversationContext::reset($ctxId);
@@ -1287,6 +1289,14 @@ class NormalBot
             self::substituteUrlVars($ep['url'], array_merge($meta, ['__ep_vars_combined' => $meta['__ep_vars_combined'] ?? []])),
             $company
         );
+        // Placeholder sin resolver → la API recibiría basura y devolvería un informe
+        // vacío sin explicación. Mejor pedir el dato que falta.
+        if (preg_match('/\{(\w+)\}/', $url, $ph)) {
+            self::log("API report: falta '{$ph[1]}' para [{$endpointKey}] — url: {$url}");
+            self::preservingReset($ctxId, ConversationContext::getMetadata($ctxId));
+            return self::sendText('Falta seleccionar un dato para generar este informe.' . $nav, $context['from'], $company);
+        }
+
         $method = strtoupper($ep['method'] ?? 'GET');
         $apiKey = $company['api_key'] ?? '';