From e5f7a0f4be18e3cd1e32b1defea194fce04fc797 Mon Sep 17 00:00:00 2001
From: vjrj <vjrj@comunes.org>
Date: Tue, 28 Jul 2026 07:46:19 +0200
Subject: [PATCH] fix(maps): entregar el mapa solo cuando tiene tamano real
 (MapReady)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

MapReady entregaba el mapa en el useEffect de montaje, cuando el contenedor
puede no tener tamano todavia. En FiresMap eso hacia que getBounds() lanzara;
el catch solo avisaba ("Failed to set map bounds and scale") y mapSize nunca
se seteaba, con lo que la suscripcion de fuegos por viewport NI SE CREABA:
loading eterno y tile-pane vacio (0 capas) pese a haber 7470 fuegos.

Evidencia: en Meteor.connection._subscriptions solo aparecian activefirestotal,
activefiresuniontotal, settings y userData — ninguna por localizacion.

Ahora se entrega via map.whenReady() + invalidateSize(), solo cuando getSize()
no es 0, con reintento en el evento resize y entrega unica (flag delivered).

Toca el componente compartido por todos los mapas: al desplegar hay que
re-verificar /fires, /zones, /subscriptions, home y detalle de fuego.
SIN DESPLEGAR todavia (la imagen viva es la de 01:33, sin este cambio).
---
 imports/ui/components/Maps/MapBridge.js | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/imports/ui/components/Maps/MapBridge.js b/imports/ui/components/Maps/MapBridge.js
index ea44ea2..bcfe22e 100644
--- a/imports/ui/components/Maps/MapBridge.js
+++ b/imports/ui/components/Maps/MapBridge.js
@@ -8,8 +8,28 @@ import { useMap, useMapEvents } from 'react-leaflet';
 export const MapReady = ({ onReady }) => {
   const map = useMap();
   useEffect(() => {
-    if (onReady) onReady(map);
-    // run once per map instance
+    if (!onReady) return undefined;
+    let cancelled = false;
+    let delivered = false;
+    // El efecto de montaje puede correr antes de que el contenedor tenga tamaño.
+    // Si entregamos el mapa entonces, un `getBounds()` del padre lanza y se queda
+    // sin bounds: en /fires eso dejaba la suscripcion por viewport sin crear y el
+    // mapa vacio (0 capas) pese a haber 7470 fuegos. Esperamos a que el mapa este
+    // listo Y tenga tamaño real; entregamos una sola vez.
+    const deliver = () => {
+      if (cancelled || delivered) return;
+      map.invalidateSize();
+      const size = map.getSize();
+      if (!size || size.x === 0 || size.y === 0) return;
+      delivered = true;
+      onReady(map);
+    };
+    map.whenReady(deliver);
+    map.on('resize', deliver);
+    return () => {
+      cancelled = true;
+      map.off('resize', deliver);
+    };
   }, [map]); // eslint-disable-line react-hooks/exhaustive-deps
   return null;
 };