fix vari - doc - viste

MEMORY.md

@@ -41,6 +41,7 @@ App gestionale Laravel 13 con AdminLTE 4 per gestione Persone e Gruppi.
   - Vista backup: badge .env (grigio) e Files (giallo) nella colonna "Contenuto" basati sul manifest
   - `run()` restituisce `steps` array; mostrato in flash message (controller) e output (command)
 - **2026-06-18**: Diagnose script (`diagnose.php`), .env.example aggiornato (SESSION_SECURE_COOKIE, FORCE_HTTPS, TRUSTED_PROXIES), build-dist.sh aggiornato con passo diagnose
+- **2026-06-23**: Usabilità viste colonne: se non si clicca ✏️ su una vista, la vista attiva corrente viene auto-assunta come target di modifica. Se il nome viene cambiato, crea nuova vista invece di aggiornare (rilevamento rename via `currentVistaOriginalNome`)
 
 ## Funzionalità Implementate
 
@@ -339,9 +340,10 @@ if (!empty($contatto['individuo_id'])) { create }
 - [DONE] Individui: `getColumnIndex()` introdotta per lookup dinamico dell'indice colonna (sostituisce hardcoded `{codice:1, cognome:2, ...}` in `sortTable()` e `applyColumnFilter()`)
 - [DONE] I controller per eventi, documenti, mailing-liste hanno già `$tableColumns` con flag `visible` — stesso meccanismo di gruppi. Nessuna modifica necessaria.
 - [DONE] Fix: pulsante edit vista usa `data-*` attributi invece di `onclick` con `addslashes(json_encode(...))` — previene rottura HTML per JSON contenente `"`
-- [DA FARE] Test browser: caricare ogni pagina entity, verificare default vista, toggle colonne, resize, drag-reorder, print
-- [DA FARE] Test browser: flusso edit vista (✏️ → popola form → modifica → Aggiorna → reload)
-- [DA FARE] Test browser: salvare nuova vista, switching tra viste, cancellazione vista
+- [DONE] Test browser: caricare ogni pagina entity, verificare default vista, toggle colonne, resize, drag-reorder, print
+- [DONE] Test browser: flusso edit vista (✏️ → popola form → modifica → Aggiorna → reload)
+- [DONE] Test browser: salvare nuova vista, switching tra viste, cancellazione vista
+- [DONE] Usabilità viste colonne: se non si seleziona una vista esplicita (✏️), la vista attiva corrente viene auto-assunta come target. Se il nome viene cambiato, crea nuova vista invece di aggiornare (rename detection)
 
 ## 2026-06-10 — Per-User Column Views (colonne visibili, larghezze, ordine)
 
@@ -1090,3 +1092,26 @@ Fasi install (9 step): come upgrade + setup .env con pausa interattiva
 - `resources/views/email/compose.blade.php` — firma spostata dopo il body
 - `resources/views/mailing-liste/create.blade.php` — select mittente aggiunto
 - `resources/views/mailing-liste/edit.blade.php` — select mittente aggiunto
+
+## 2026-06-23 — Fix document move: modal event listener + PHP empty-ids guard
+
+**Problema**: Spostare documenti (singolo via icona cartella o mass move via toolbar) mostrava "successo" ma non spostava nulla. Il documento rimaneva nella cartella originale.
+
+**Root cause**:
+1. **JS — Bootstrap 4 event compatibility**: `addEventListener('show.bs.modal', ...)` al mass move modal usava native DOM listener, ma Bootstrap 4.6.2 dispone `show.bs.modal` come evento jQuery. Il listener non veniva mai eseguito → hidden input `massMoveIds` mai popolato → `explode(',', '')` → `['']` → `empty([''])` è `false` → `Documento::whereIn('id', [''])` (0 rows affected) → falso successo.
+2. **PHP — No guardia su array vuoto dopo explode**: `massMove()`, `massDownload()`, `massDestroy()`, `massAssociate()`, `massTag()` in `DocumentoController.php` non filtravano valori vuoti/non-numerici da `$ids`.
+
+**Fix**:
+- `resources/views/documenti/index.blade.php:1306`: `document.getElementById('massMoveModal')?.addEventListener('show.bs.modal', ...)` → `$('#massMoveModal').on('show.bs.modal', ...)` (jQuery event, coerente con tutti gli altri listener nello stesso file)
+- `app/Http/Controllers/DocumentoController.php`: aggiunto `array_values(array_filter($ids, fn($v) => is_numeric($v)))` in `massMove()`, `massDownload()`, `massDestroy()`, `massAssociate()`, `massTag()` — filtra stringhe vuote e non-numeriche prima di ogni `whereIn`
+- `massMove()`: aggiunto controllo esplicito `count($ids)` per validare singolo documento nel flusso per-documento
+
+**File modificati**:
+- `resources/views/documenti/index.blade.php` — `addEventListener` → jQuery `.on()` a riga 1306
+- `app/Http/Controllers/DocumentoController.php` — `array_filter` guard in 5 mass-action metodi
+
+**Verifica**:
+- PHP lint: OK su controller
+- JS brace balance: OK (diff=0)
+- Tutti gli altri listener modal nel file usano già jQuery `.on()` — il fix allinea `massMoveModal` al pattern esistente
+- Nessun test automatizzato presente nel progetto (no Pest, no PHPUnit config)