Arreglado vision y funcion de zonas a evitar

RutasDBUS/Components/Pages/PlanificadorRutas.Itinerarios.cs

@@ -1400,7 +1400,7 @@ public partial class PlanificadorRutas : ComponentBase
         private bool _elevPopupMinimizado = false;
     
         
-        private const string _versionApp = "(v-20260909a)";
+        private const string _versionApp = "(v-20260910a)";
     
     
         private double? _paradaElevIdee;

RutasDBUS/Components/Pages/PlanificadorRutas.RestriccionesCaminata.cs

@@ -69,6 +69,7 @@ public partial class PlanificadorRutas
     private async Task CambiarZonaActivaAsync(ZonaExclusionCaminata zona, ChangeEventArgs args)
     {
         zona.Activa = args.Value is true;
+        await GuardarZonasAsync();
         await ActualizarEditorExclusionCaminataAsync();
     }
     private ZonaExclusionCaminata? ZonaExclusionSeleccionada =>
@@ -127,6 +128,12 @@ public partial class PlanificadorRutas
         await ActualizarEditorExclusionCaminataAsync();
     }
 
+    private void SeleccionarTipoExclusionCaminata(bool esPunto)
+    {
+        if (_recalculandoZonas || _modoDibujoExclusionCaminata) return;
+        _nuevaExclusionEsPunto = esPunto;
+    }
+
     private async Task SeleccionarZonaExclusionAsync(string id)
     {
         if (_recalculandoZonas) return;

RutasDBUS/Components/Pages/PlanificadorRutas.razor

@@ -1515,12 +1515,16 @@
                 }
                 else
                 {
+                    <div class="zones-panel__type" role="group" aria-label="Tipo de exclusión">
+                        <button type="button" class="@(!_nuevaExclusionEsPunto ? "is-active" : null)"
+                                aria-pressed="@(!_nuevaExclusionEsPunto)" disabled="@_recalculandoZonas"
+                                @onclick="() => SeleccionarTipoExclusionCaminata(false)">Polígono</button>
+                        <button type="button" class="@(_nuevaExclusionEsPunto ? "is-active" : null)"
+                                aria-pressed="@_nuevaExclusionEsPunto" disabled="@_recalculandoZonas"
+                                @onclick="() => SeleccionarTipoExclusionCaminata(true)">Punto</button>
+                    </div>
                     <button type="button" class="btn-ghost" disabled="@(_recalculandoZonas || _zonasExclusionCaminata.Count >= 10)"
                             @onclick="IniciarDibujoExclusionCaminataAsync">@(_nuevaExclusionEsPunto ? "Añadir punto" : "Añadir zona")</button>
-                    <select class="zones-panel__type" aria-label="Tipo de exclusión" @bind="_nuevaExclusionEsPunto" disabled="@_recalculandoZonas">
-                        <option value="false">Polígono</option>
-                        <option value="true">Punto</option>
-                    </select>
                 }
                 <button type="button" class="btn-ghost" disabled="@(_recalculandoZonas || _zonasExclusionCaminata.Count == 0)"
                         @onclick="LimpiarZonasExclusionAsync">Borrar todas</button>

RutasDBUS/Servicios/Planificacion/PlanificadorRutasServicio.cs

@@ -3885,16 +3885,6 @@ public sealed class PlanificadorRutasServicio : IPlanificadorRutas
     {
         if (excludePolygons is { Count: > 0 } || excludeLocations is { Count: > 0 })
         {
-            // Solo reutilizar una geometria conocida y alejada de todas las zonas.
-            var claveNormal = BuildWalkKey("valhalla", latitudOrigen, longitudOrigen, latitudDestino, longitudDestino);
-            // Un punto se asocia a una arista, que puede extenderse mas alla del punto.
-            if (excludeLocations is not { Count: > 0 } && excludePolygons is not null &&
-                _cacheCaminatas.TryGetValue(claveNormal, out var normal) && normal.linea is not null &&
-                !ExclusionPuedeAfectarTrayecto(normal.linea, excludePolygons))
-            {
-                return normal;
-            }
-
             var claveExclusion = BuildWalkKey("valhalla-exclusion", latitudOrigen, longitudOrigen, latitudDestino, longitudDestino)
                 + "|" + JsonSerializer.Serialize(excludePolygons) + "|" + JsonSerializer.Serialize(excludeLocations);
             if (_cacheCaminatasConExclusion.TryGetValue(claveExclusion, out var cacheadoConExclusion))
@@ -3950,38 +3940,6 @@ public sealed class PlanificadorRutasServicio : IPlanificadorRutas
         }
     }
 
-    private static bool ExclusionPuedeAfectarTrayecto(double[,] linea, List<List<double[]>> poligonos)
-    {
-        if (linea.GetLength(0) < 2) return true;
-        var minLat = double.PositiveInfinity;
-        var maxLat = double.NegativeInfinity;
-        var minLon = double.PositiveInfinity;
-        var maxLon = double.NegativeInfinity;
-        for (var i = 0; i < linea.GetLength(0); i++)
-        {
-            if (!double.IsFinite(linea[i, 0]) || !double.IsFinite(linea[i, 1])) return true;
-            minLat = Math.Min(minLat, linea[i, 0]);
-            maxLat = Math.Max(maxLat, linea[i, 0]);
-            minLon = Math.Min(minLon, linea[i, 1]);
-            maxLon = Math.Max(maxLon, linea[i, 1]);
-        }
-        // Margen conservador para las vias y sus extremos, no solo los vertices dibujados.
-        const double margenLat = 2500.0 / 111320.0;
-        var margenLon = margenLat / Math.Max(0.01, Math.Cos((minLat + maxLat) * Math.PI / 360.0));
-        foreach (var poligono in poligonos)
-        {
-            if (poligono.Count < 3 || poligono.Any(p => p.Length < 2 || !double.IsFinite(p[0]) || !double.IsFinite(p[1])))
-                return true;
-            // Los poligonos usan [lon, lat]; sus limites incluyen lados y area interior.
-            if (poligono.Max(p => p[1]) >= minLat - margenLat &&
-                poligono.Min(p => p[1]) <= maxLat + margenLat &&
-                poligono.Max(p => p[0]) >= minLon - margenLon &&
-                poligono.Min(p => p[0]) <= maxLon + margenLon)
-                return true;
-        }
-        return false;
-    }
-
     /// <summary>
     /// Construye una clave estable de cache para caminatas.
     /// </summary>

RutasDBUS/wwwroot/app.css

@@ -1167,7 +1167,31 @@ html, body {
 .zones-panel__item > input { flex: 0 0 auto; align-self: center; }
 .zones-panel__actions { display: flex; flex-wrap: wrap; align-items: center; gap: .4rem; margin-top: .5rem; }
 .zones-panel__actions .btn-ghost { white-space: nowrap; }
-.zones-panel__type { background: #161d2a; color: #e5e7eb; border: 1px solid #64748b; border-radius: 6px; min-height: 30px; max-width: 110px; font: inherit; }
+.zones-panel__type {
+    display: inline-grid;
+    grid-template-columns: 1fr 1fr;
+    padding: 2px;
+    border: 1px solid rgba(148, 163, 184, .6);
+    border-radius: 999px;
+    background: #0b1220;
+}
+.zones-panel__type button {
+    min-height: 28px;
+    padding: .2rem .7rem;
+    border: 0;
+    border-radius: 999px;
+    background: transparent;
+    color: #94a3b8;
+    font: inherit;
+    font-size: .78rem;
+    cursor: pointer;
+}
+.zones-panel__type button.is-active {
+    background: #334155;
+    color: #fff;
+    box-shadow: 0 1px 3px rgba(0, 0, 0, .35);
+}
+.zones-panel__type button:disabled { cursor: not-allowed; opacity: .55; }
 .zones-panel__icon { padding-inline: .6rem; }
 .btn-ghost.zones-panel__primary { background: #1d4ed8; color: #fff; }
 .btn-ghost.zones-panel__primary:disabled { background: #111827; color: #94a3b8; }

RutasDBUS/wwwroot/app.js

@@ -36,7 +36,7 @@ window.initZonasShortcut = function (dotNetHelper) {
     window.rt._zonasShortcutInicializado = true;
 
     document.addEventListener("keydown", function (e) {
-        if (e.repeat || e.code !== "KeyZ" || !e.ctrlKey || !e.shiftKey || e.altKey || e.metaKey) return;
+        if (e.repeat || e.code !== "KeyE" || !e.ctrlKey || !e.shiftKey || e.altKey || e.metaKey) return;
 
         e.preventDefault();
         e.stopImmediatePropagation();
@@ -798,19 +798,8 @@ window.rt.updateExclusionPolygonEditor = function (zones, selectedId, busy) {
         });
     }
 
-    if (drawing) {
+    // El clic llega por el evento normal del mapa. No se registra otro manejador
-        const container = map.getContainer();
+    // aquí para evitar duplicar vértices, especialmente en las exclusiones puntuales.
-        const handler = event => {
-            if (event.target.closest(".rt-exclusion-vertex, .leaflet-control, .leaflet-popup")) return;
-            event.preventDefault();
-            event.stopImmediatePropagation();
-            const pos = map.mouseEventToLatLng(event);
-            window.rt._dotNetHelper?.invokeMethodAsync("OnExclusionPolygonMapClick", pos.lat, pos.lng);
-        };
-        container.addEventListener("click", handler, true);
-        window.rt._exclusionMapContainer = container;
-        window.rt._exclusionClickHandler = handler;
-    }
 };
 
 window.rt.isExclusionPolygonValid = function (points) {

tests/RutasDBUS.RegressionTests/Program.cs

@@ -58,23 +58,27 @@ List<double[]> Ring(double lon, double lat, double size) =>
 await Walk();
 var far = new List<List<double[]>> { Ring(-2.3, 43.0, .01) };
 await Walk(far);
-Check(fake.Calls == 1, "Zona lejana reutiliza la geometria conocida");
+Check(fake.Calls == 2 && fake.Zones == 1, "Toda zona activa se envia a V2 aunque este lejos de la ruta previa");
 var near = new List<List<double[]>> { Ring(-1.985, 43.315, .001), Ring(-1.982, 43.317, .001) };
 await Walk(near);
 Check(fake.Zones == 2, "Se envian todos los poligonos");
 await Walk(near);
-Check(fake.Calls == 2, "Mismas zonas reutilizan cache restringida");
+Check(fake.Calls == 3, "Mismas zonas reutilizan cache restringida");
 near[0][0][0] += .00001;
 await Walk(near);
-Check(fake.Calls == 3, "Mover un vertice invalida esa entrada de cache");
+Check(fake.Calls == 4, "Mover un vertice invalida esa entrada de cache");
 await Walk(new() { Ring(-2.2, 43.1, .5) });
-Check(fake.Calls == 4, "Una zona que rodea la ruta no se ignora aunque sus vertices esten lejos");
+Check(fake.Calls == 5, "Una zona que rodea la ruta no se ignora aunque sus vertices esten lejos");
 fake.Fail = true;
 var retry = new List<List<double[]>> { Ring(-1.984, 43.312, .0002) };
 await Walk(retry);
 fake.Fail = false;
 await Walk(retry);
-Check(fake.Calls == 6, "Un fallo temporal no queda guardado en cache de zonas");
+Check(fake.Calls == 7, "Un fallo temporal no queda guardado en cache de zonas");
+var puntos = new List<double[]> { new[] { 43.314, -1.984 }, new[] { 43.315, -1.983 } };
+await (Task<(double[,]?, double, double)>)walk.Invoke(planner,
+    new object?[] { 43.31, -1.99, 43.32, -1.98, near, puntos })!;
+Check(fake.Zones == 2 && fake.Points == 2, "Se envian juntos todos los poligonos y puntos activos");
 var page = new PlanificadorRutas();
 var flags = BindingFlags.Instance | BindingFlags.NonPublic;
 var options = (List<AlternativaRuta>)typeof(PlanificadorRutas).GetField("_opcionesRuta", flags)!.GetValue(page)!;
@@ -102,6 +106,7 @@ sealed class FakeValhalla : IClienteValhalla
 
     public int Calls { get; private set; }
     public int Zones { get; private set; }
+    public int Points { get; private set; }
     public bool Fail { get; set; }
     public Task<(double[,]? Linea, double DistanciaMetros, double DuracionSegundos)> ObtenerRutaAsync(
         double latitudDesde, double longitudDesde, double latitudHasta, double longitudHasta,
@@ -110,6 +115,7 @@ sealed class FakeValhalla : IClienteValhalla
     {
         Calls++;
         Zones = excludePolygons?.Count ?? 0;
+        Points = excludeLocations?.Count ?? 0;
         return Task.FromResult<(double[,]?, double, double)>(Fail ? (null, 0, 0) :
             (new[,] { { latitudDesde, longitudDesde }, { latitudHasta, longitudHasta } }, 100, 100));
     }

tests/verify-zones.cjs

@@ -14,9 +14,10 @@ const path = require("node:path");
         await page.goto("http://127.0.0.1:5199", { waitUntil: "domcontentloaded" });
         await page.waitForFunction(() => !!window.rt?._dotNetHelper && !!window.__rtLeafletMap);
         const initialErrors = errors.slice();
-        await page.keyboard.press("Control+Shift+KeyZ");
+        await page.keyboard.press("Control+Shift+KeyE");
         await page.locator(".zones-panel").waitFor();
         await page.waitForTimeout(1000);
+        await page.getByRole("button", { name: "Polígono", exact: true }).click();
         await page.getByRole("button", { name: "Añadir zona", exact: true }).click();
         for (const [x, y] of [[600, 420], [850, 430], [760, 640]]) {
             await page.mouse.click(x, y);
@@ -52,7 +53,7 @@ const path = require("node:path");
         await page.locator(".zones-panel").waitFor({ state: "detached" });
         assert.equal(await page.locator(".rt-exclusion-vertex").count(), 0);
         assert.equal(await page.locator('path[stroke="#e879f9"], path[stroke="#f97316"]').count(), 0);
-        await page.keyboard.press("Control+Shift+KeyZ");
+        await page.keyboard.press("Control+Shift+KeyE");
         await page.locator(".zones-panel").waitFor();
         assert.equal(await page.locator(".zones-panel__item").count(), 2, "Reabrir conserva zonas editables");
         await page.getByRole("checkbox", { name: "Activar zona", exact: true }).first().uncheck();
@@ -60,7 +61,7 @@ const path = require("node:path");
         await page.reload();
         await page.waitForFunction(() => !!window.rt?._dotNetHelper && !!window.__rtLeafletMap);
         await page.waitForTimeout(1000);
-        await page.keyboard.press("Control+Shift+KeyZ");
+        await page.keyboard.press("Control+Shift+KeyE");
         await page.locator(".zones-panel__item").first().waitFor();
         assert.equal(await page.locator(".zones-panel__item").count(), 2, "F5 conserva zonas");
         assert.equal(await page.getByRole("checkbox", { name: "Activar zona", exact: true }).first().isChecked(), false);
@@ -81,6 +82,16 @@ const path = require("node:path");
         await page.getByRole("button", { name: "Eliminar zona", exact: true }).first().click();
         await page.waitForFunction(() => document.querySelectorAll(".zones-panel__item").length === 1);
         assert.equal(await page.locator(".zones-panel__item").count(), 1);
+        await page.getByRole("button", { name: "Punto", exact: true }).click();
+        await page.getByRole("button", { name: "Añadir punto", exact: true }).click();
+        await page.mouse.click(1050, 720);
+        await page.waitForFunction(() => document.querySelectorAll(".zones-panel__item").length === 2);
+        assert.match(await page.locator(".zones-panel__item").last().innerText(), /Punto/);
+        assert.equal(await page.locator(".zones-panel__item").last().innerText().then(text => text.includes("abierta")), false,
+            "Un clic cierra la exclusion puntual");
+        const puntoGuardado = await page.evaluate(() => JSON.parse(localStorage.getItem("rutasdbus.zonas.v1")).at(-1));
+        assert.equal(puntoGuardado.EsPunto, true);
+        assert.equal(puntoGuardado.Vertices.length, 1, "El punto se guarda con un solo vertice");
         await page.getByRole("button", { name: "Borrar todas", exact: true }).click();
         await page.waitForFunction(() => document.querySelectorAll(".zones-panel__item").length === 0);
         assert.equal(await page.locator(".zones-panel__item").count(), 0);
@@ -88,6 +99,7 @@ const path = require("node:path");
         await page.screenshot({ path: path.join(artifacts, "mobile.png") });
         const box = await page.locator(".zones-panel").boundingBox();
         assert.ok(box.x >= 0 && box.x + box.width <= 390 && box.y + box.height <= 844, "Panel dentro del movil");
+        await page.getByRole("button", { name: "Polígono", exact: true }).click();
         await page.getByRole("button", { name: "Añadir zona", exact: true }).click();
         await page.getByRole("button", { name: "Cerrar herramienta", exact: true }).click();
         await page.setViewportSize({ width: 1440, height: 1000 });
@@ -99,7 +111,7 @@ const path = require("node:path");
         const newErrors = errors.filter(error => !initialErrors.includes(error) && !externalErrors.includes(error));
         assert.equal(newErrors.length, 0, newErrors.join("\n"));
         if (externalErrors.length) console.log("Aviso del complemento externo de geocodificacion: " + externalErrors[0]);
-        console.log("OK multiples zonas, arrastre en vivo, cierre, reapertura, borrado, alturas de botones y movil");
+        console.log("OK poligonos y puntos, arrastre en vivo, cierre, reapertura, borrado, alturas de botones y movil");
     } finally {
         await browser.close();
     }