From c0c8a2531c0229ce104d19d7a7687a91af03dd4e Mon Sep 17 00:00:00 2001
From: Gabriele Cimaschi <gabriele.cimaschi@gmail.com.com>
Date: Wed, 26 Aug 2026 14:23:20 +0200
Subject: [PATCH] fix: ora le ricerche lavorano nuovamente in cache. feat:
 aggiunto pulsante per eliminare rapidamente una ricerca testuale. feat: la
 ricerca in alto a destra rimanda alla ricerca spudorata ed il campo viene
 liberato.

---
 public/index.html                   |  9 ++-----
 public/js/app.js                    | 27 +++++++++++--------
 public/js/views/ricercaSpudorata.js | 42 +++++++++++++----------------
 public/js/views/ticketList.js       | 33 ++++++++++++++---------
 4 files changed, 57 insertions(+), 54 deletions(-)

diff --git a/public/index.html b/public/index.html
index 9c6ed50..ceb0989 100644
--- a/public/index.html
+++ b/public/index.html
@@ -197,13 +197,8 @@
             <input type="text" class="search-input" id="global-search"
               placeholder="Cerca ticket (numero, titolo o corpo)..." style="padding-right: 32px;" />
             <button id="global-search-clear"
-              style="position: absolute; right: 10px; top: 50%; transform: translateY(-50%); background: none; border: none; padding: 4px; cursor: pointer; color: var(--text-muted); display: none; align-items: center; justify-content: center;"
-              title="Cancella ricerca">
-              <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2"
-                style="width:14px;height:14px;display:block;">
-                <path d="M18 6L6 18M6 6l12 12" />
-              </svg>
-            </button>
+              style="position: absolute; right: 10px; top: 50%; transform: translateY(-50%); background: none; border: none; padding: 0; cursor: pointer; color: var(--text-muted); display: none; align-items: center; justify-content: center; font-size: 13px; line-height: 1;"
+              title="Cancella ricerca">✕</button>
           </div>
           <button class="btn btn-primary btn-sm" id="topbar-new-ticket" onclick="window.location.hash='#/tickets/new'">
             <svg viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" style="width:16px;height:16px;">
diff --git a/public/js/app.js b/public/js/app.js
index ed32ced..6457ca5 100644
--- a/public/js/app.js
+++ b/public/js/app.js
@@ -163,12 +163,20 @@ const App = {
       searchInput.addEventListener('keydown', (e) => {
         if (e.key === 'Enter') {
           e.preventDefault();
-          const hash = window.location.hash;
-          if (!hash.startsWith('#/tickets') || hash.includes('/new') || hash.match(/#\/tickets\/\d+/)) {
-            window.location.hash = '#/tickets';
+          const query = searchInput.value.trim();
+          searchInput.value = '';
+          toggleClearBtn();
+          if (query) {
+            const newHash = `#/ricerca-spudorata?q=${encodeURIComponent(query)}`;
+            if (window.location.hash === newHash && typeof RicercaSpudorataView !== 'undefined') {
+              const input = document.getElementById('spudorata-search-input');
+              if (input) input.value = query;
+              RicercaSpudorataView.executeSearch(query);
+            } else {
+              window.location.hash = newHash;
+            }
           } else {
-            TicketListView.currentPage = 1;
-            TicketListView.render();
+            window.location.hash = '#/ricerca-spudorata';
           }
         }
       });
@@ -178,12 +186,9 @@ const App = {
           searchInput.value = '';
           searchClearBtn.style.display = 'none';
           searchInput.focus();
-          const hash = window.location.hash;
-          if (hash.startsWith('#/tickets') && !hash.includes('/new') && !hash.match(/#\/tickets\/\d+/)) {
-            TicketListView.currentPage = 1;
-            TicketListView.render();
-          } else {
-            window.location.hash = '#/tickets';
+          if (window.location.hash.startsWith('#/ricerca-spudorata') && typeof RicercaSpudorataView !== 'undefined') {
+            RicercaSpudorataView.invalidateCache();
+            RicercaSpudorataView.render();
           }
         });
       }
diff --git a/public/js/views/ricercaSpudorata.js b/public/js/views/ricercaSpudorata.js
index 6727649..dc853ac 100644
--- a/public/js/views/ricercaSpudorata.js
+++ b/public/js/views/ricercaSpudorata.js
@@ -24,36 +24,32 @@ const RicercaSpudorataView = {
     Filters.load();
 
     const container = document.getElementById('view-container');
-    container.innerHTML = `
-      <div class="loading-screen">
-        <div class="spinner"></div>
-        <p>Inizializzazione Ricerca Spudorata...</p>
-      </div>
-    `;
+    if (!container) return;
 
-    await App.ensureLookups();
+    const urlParams = new URLSearchParams((window.location.hash.split('?')[1]) || '');
+    const qParam = urlParams.get('q');
 
-    try {
-      const settings = await App.api('/api/dashboard/settings');
-      if (settings && settings.tickets_per_page) {
-        this.perPage = settings.tickets_per_page;
-      }
-    } catch (err) {
-      console.warn('Failed to load agent settings:', err);
+    if (qParam && qParam !== this.searchTerm) {
+      this.renderContainer(container);
+      this.bindEvents();
+      const input = document.getElementById('spudorata-search-input');
+      if (input) input.value = qParam;
+      this.executeSearch(qParam);
+      return;
+    }
+
+    // Instant render from memory cache if already searched
+    if (this.hasSearched && this.cachedTickets) {
+      this.renderContainer(container);
+      this.bindEvents();
+      this.renderLocalResults();
+      return;
     }
 
     this.renderContainer(container);
     this.bindEvents();
 
-    const urlParams = new URLSearchParams((window.location.hash.split('?')[1]) || '');
-    const qParam = urlParams.get('q');
-    if (qParam && !this.hasSearched) {
-      const input = document.getElementById('spudorata-search-input');
-      if (input) input.value = qParam;
-      this.executeSearch(qParam);
-    } else if (this.hasSearched && this.cachedTickets) {
-      this.renderLocalResults();
-    }
+    App.ensureLookups();
   },
 
   renderContainer(container) {
diff --git a/public/js/views/ticketList.js b/public/js/views/ticketList.js
index ffbbcf3..045f11a 100644
--- a/public/js/views/ticketList.js
+++ b/public/js/views/ticketList.js
@@ -9,15 +9,20 @@ const TicketListView = {
   sortDir: 'DESC',
   selectedIds: new Set(),
   selectedOrder: [],
-  rawTickets: null,
+  rawTicketsByMode: {
+    general: null,
+    my: null
+  },
+  lastSearchTermByMode: {
+    general: '',
+    my: ''
+  },
   otrsBaseUrl: null,
-  lastMode: null,
-  lastSearchTerm: '',
   needsRefresh: false,
 
   invalidateCache() {
-    this.rawTickets = null;
-    this.lastMode = null;
+    this.rawTicketsByMode.general = null;
+    this.rawTicketsByMode.my = null;
     this.needsRefresh = true;
   },
 
@@ -38,11 +43,13 @@ const TicketListView = {
     const container = document.getElementById('view-container');
     if (!container) return;
 
+    const cachedTickets = this.rawTicketsByMode[newMode];
+    const cachedSearchTerm = this.lastSearchTermByMode[newMode];
+
     const shouldFetch = force ||
       this.needsRefresh ||
-      !this.rawTickets ||
-      this.lastMode !== newMode ||
-      this.lastSearchTerm !== globalSearchVal;
+      !cachedTickets ||
+      cachedSearchTerm !== globalSearchVal;
 
     if (shouldFetch) {
       if (!container.querySelector('.ticket-table-wrapper') && !container.querySelector('.ticket-table')) {
@@ -78,10 +85,9 @@ const TicketListView = {
           return;
         }
 
-        this.rawTickets = data.tickets || [];
+        this.rawTicketsByMode[newMode] = data.tickets || [];
+        this.lastSearchTermByMode[newMode] = globalSearchVal;
         this.otrsBaseUrl = data.otrsBaseUrl || '';
-        this.lastMode = newMode;
-        this.lastSearchTerm = globalSearchVal;
         this.needsRefresh = false;
       } catch (err) {
         const currentHash = (window.location.hash || '#/dashboard').split('?')[0];
@@ -133,8 +139,9 @@ const TicketListView = {
   },
 
   getFilteredTickets() {
-    if (!this.rawTickets) return [];
-    let list = [...this.rawTickets];
+    const rawTickets = this.rawTicketsByMode[Filters.currentMode] || [];
+    if (!rawTickets || rawTickets.length === 0) return [];
+    let list = [...rawTickets];
     const s = Filters.state;
 
     // Sub-search text filter