From 487995366e5b63ba550433b218ebe2216f5c2a89 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicol=C3=B2=20P=2E?= <nicolo.paraciani@cnr.it>
Date: Sun, 24 May 2026 12:28:50 +0200
Subject: [PATCH] Add markers action to search results

---
 webgis/index.html                          |  18 ++-
 webgis/js/controllers/form_controller.js   |  79 -------------
 webgis/js/controllers/menu_controller.js   |   4 +
 webgis/js/controllers/search_controller.js | 122 +++++++++++++++++++++
 webgis/js/index.js                         |  11 +-
 5 files changed, 141 insertions(+), 93 deletions(-)
 delete mode 100644 webgis/js/controllers/form_controller.js
 create mode 100644 webgis/js/controllers/search_controller.js

diff --git a/webgis/index.html b/webgis/index.html
index 3b27187..a47e1d3 100644
--- a/webgis/index.html
+++ b/webgis/index.html
@@ -83,9 +83,10 @@
             </div>
         </div>
         <div class="menu-overlay column is-hidden is-4 is-4-desktop is-5-mobile is-pulled-right is-overlay has-background-white-ter"
-            data-menu-target="search" data-controller="layer form">
+            data-menu-target="search" data-controller="layer search">
+            <button title="Chiudi ricerca" class="delete is-pulled-right" data-action="menu#closeSearch"></button>
             <h1 class="is-size-5">Ricerca</h1>
-            <form id="search-form" method="POST" data-form-target="search" data-action="submit->form#submitSearch">
+            <form id="search-form" method="POST" data-search-target="search" data-action="submit->search#submitSearch">
                 <div class="field">
                     <label class="label">Testo libero</label>
                     <div class="control is-full-width">
@@ -99,7 +100,7 @@
                                 <select name="category">
                                     <option default value="">-- Scegliere la categoria del sito --</option>
                                     <option value="site">Sito conservato</option>
-                                    <option value="not_conserved">Sito non conservato</option>
+                                    <!--<option value="not_conserved">Sito non conservato</option>-->
                                 </select>
                             </div>
                         </div>
@@ -130,7 +131,7 @@
                         </button>
                     </div>
                     <div class="control">
-                        <button class="button is-link is-light" type="reset">
+                        <button class="button is-link is-light" type="reset" data-action="search#clearSearch">
                             <span>Cancella filtri</span>
                             <span class="icon is-small">
                                 <i class="fa fa-times"></i>
@@ -139,7 +140,14 @@
                     </div>
                 </div>
             </form>
-            <output data-form-target="results"></output>
+            <div class="content pt-2 mt-3 is-hidden" data-search-target="container">
+                <table class="table is-fullwidth is-striped">
+                    <thead>
+                        <tr><td class="has-text-centered has-text-weight-bold is-size-5" colspan="2">Risultati</td></tr>
+                    </thead>
+                    <tbody data-search-target="results"></tbody>
+                </table>
+            </div>
         </div>
         <div class="menu-overlay column is-hidden is-4 is-4-desktop is-5-mobile is-pulled-right is-overlay has-background-white-ter"
             data-menu-target="menu" data-controller="layer">
diff --git a/webgis/js/controllers/form_controller.js b/webgis/js/controllers/form_controller.js
deleted file mode 100644
index 404429f..0000000
--- a/webgis/js/controllers/form_controller.js
+++ /dev/null
@@ -1,79 +0,0 @@
-import { Controller } from "@hotwired/stimulus";
-import { GisState } from "../state.js";
-import UI from "../ui.js";
-
-const html = String.raw;
-
-export default class extends Controller {
-    static values = {
-        'coords': String,
-        'group': String,
-        'id': String,
-    };
-
-    static targets = ['search', 'results'];
-    
-    /**
-     * 
-     * @param {Event} event 
-     */
-    async submitSearch(event) {
-        event.preventDefault();
-        const data = new FormData(event.target);
-        const body = {};
-
-        for (const entry of data.entries()) {
-            body[entry[0]] = entry[1];
-        }
-
-        const response = await fetch("https://caprigis-api.ddev.site:33001/search?" + new URLSearchParams(body));
-        const results = await response.json(); 
-
-        console.debug(results);
-
-        this.#injectResults(results);
-        this.#filterMap(results);
-    }
-
-    #injectResults(results) {
-        /**
-         * @type {HTMLOutputElement} output
-         */
-        const output = this.resultsTarget;
-        output.innerHTML = '';
-
-        for (const result of results) {
-            const item = html`
-                <p class="p-1">${result.label}</p>
-            `;
-
-            output.innerHTML += item;
-        }
-    }
-
-    /**
-     * 
-     * @param {Array<Object>} results 
-     */
-    #filterMap(results) {
-        const map = GisState.map;
-        const labels = [];
-        results.forEach(r => labels.push(r.label));
-
-        for (const key of Object.keys(GisState.layers)) {
-            map.removeLayer(GisState.layers[key]);
-        }
-
-        const sites = GisState.markers.sites;
-
-        for (let key of Object.keys(sites)) {
-            // If map has layers from previous search results...
-            map.removeLayer(sites[key]);
-            for (const label of labels) {
-                if (sites[key].options.data.label === label) {
-                    map.addLayer(sites[key]);
-                }
-            }
-        }
-    }
-}
diff --git a/webgis/js/controllers/menu_controller.js b/webgis/js/controllers/menu_controller.js
index f68f198..4fda139 100644
--- a/webgis/js/controllers/menu_controller.js
+++ b/webgis/js/controllers/menu_controller.js
@@ -128,6 +128,10 @@ export default class extends Controller {
         this.cartographyTarget.classList.add('is-hidden');
     }
 
+    closeSearch() {
+        this.searchTarget.classList.add('is-hidden');
+    }
+
     toggleList(id) {
         document.querySelector(`#${id}`).classList.toggle('is-hidden');
     }
diff --git a/webgis/js/controllers/search_controller.js b/webgis/js/controllers/search_controller.js
new file mode 100644
index 0000000..22b7fac
--- /dev/null
+++ b/webgis/js/controllers/search_controller.js
@@ -0,0 +1,122 @@
+import { Controller } from "@hotwired/stimulus";
+import { GisState } from "../state.js";
+import UI from "../ui.js";
+
+const html = String.raw;
+
+export default class extends Controller {
+    static targets = [
+        'search',
+        'results',
+        'clear',
+        'container',
+    ];
+    
+    /**
+     * 
+     * @param {Event} event 
+     */
+    async submitSearch(event) {
+        event.preventDefault();
+        const data = new FormData(event.target);
+        const body = {};
+
+        for (const entry of data.entries()) {
+            body[entry[0]] = entry[1];
+        }
+
+        const response = await fetch(`${GisState.apiUrl}/search?` + new URLSearchParams(body));
+        const results = await response.json(); 
+
+        this.containerTarget.classList.remove('is-hidden');
+        this.#injectResults(results);
+        if (results.length) this.#filterMap(results);
+    }
+
+    clearSearch() {
+        const map = GisState.map;
+
+        // Restore layer groups in map
+        for (const key of Object.keys(GisState.layers)) {
+            map.addLayer(GisState.layers[key]);
+        }
+
+        // Empty result set
+        this.resultsTarget.innerHTML = '';
+        this.containerTarget.classList.add('is-hidden');
+    }
+
+    #injectResults(results) {
+        /**
+         * @type {HTMLOutputElement} output
+         */
+        const output = this.resultsTarget;
+        output.innerHTML = '';
+
+        if (results.length === 0) {
+            output.innerHTML = html`
+                <p class="has-background-white-bis p-4 mt-0 has-text-centered">
+                    Nessun risultato trovato per i parametri di ricerca
+                </p>
+            `;
+        }
+
+        const sites = GisState.markers.sites;
+
+        for (const result of results) {
+            let coordinates = ''
+            for (let key of Object.keys(sites)) {
+                if (sites[key].options.data.label === result.label) {
+                    coordinates = key;
+                }
+            }
+
+            // TODO The group value should be dynamic!!
+            const item = html`
+                <tr>
+                    <td class="pt-4">${result.label}</td>
+                    <td>
+                        <button class="button is-link"
+                            data-controller="marker"
+                            data-action="marker#go"
+                            data-marker-coords-value="${coordinates}"
+                            data-marker-group-value="sites">
+                            Vai al sito
+                            <span class="ml-1 icon">
+                                <i class="fa fa-chevron-right"></i>
+                            </span>
+                        </button>
+                    </td>
+                </tr>
+            `;
+
+            output.innerHTML += item;
+        }
+    }
+    /**
+     * 
+     * @param {Array<Object>} results 
+     */
+    #filterMap(results) {
+        const map = GisState.map;
+        const labels = [];
+        results.forEach(r => labels.push(r.label));
+
+        // Remove all layer groups first
+        for (const key of Object.keys(GisState.layers)) {
+            map.removeLayer(GisState.layers[key]);
+        }
+
+        const sites = GisState.markers.sites;
+
+        for (let key of Object.keys(sites)) {
+            // If map has layers from previous search results...
+            map.removeLayer(sites[key]);
+            for (const label of labels) {
+                if (sites[key].options.data.label === label) {
+                    map.addLayer(sites[key]);
+                }
+            }
+        }
+    }
+}
diff --git a/webgis/js/index.js b/webgis/js/index.js
index 1eb9fbb..3ebc473 100644
--- a/webgis/js/index.js
+++ b/webgis/js/index.js
@@ -7,8 +7,7 @@ import MarkerController from './controllers/marker_controller.js';
 import BiblioController from './controllers/biblio_controller.js';
 import TabsController from './controllers/tabs_controller.js';
 import LayerController from './controllers/layer_controller.js';
-import FormController from './controllers/form_controller.js';
-import { GisState } from './state.js';
+import SearchController from './controllers/search_controller.js';
 
 document.addEventListener('DOMContentLoaded', async () => {
     // Register Stimulus controllers
@@ -28,12 +27,6 @@ document.addEventListener('DOMContentLoaded', async () => {
 
     UI.addCenterMapControl(map, GIS.CENTER_COORDS, GIS.INIT_ZOOM);
     UI.toggleBurger('navbar-burger');
-
-    // TEMP!!
-    const sites = GisState.markers.sites;
-    for (let key of Object.keys(sites)) {
-        //console.debug(sites[key].options.data);
-    }
 });
 
 function initStimulus() {
@@ -44,5 +37,5 @@ function initStimulus() {
     Stimulus.register("biblio", BiblioController);
     Stimulus.register("tabs", TabsController);
     Stimulus.register("layer", LayerController);
-    Stimulus.register("form", FormController);
+    Stimulus.register("search", SearchController);
 }
\ No newline at end of file