inizio sistemazione tag mailing list

2026-06-18 07:43:33 +02:00
parent 3b2ad1a0d5
commit d2084b9d07
9 changed files with 195 additions and 20 deletions
@@ -13,7 +13,7 @@ App gestionale Laravel 13 con AdminLTE 4 per gestione Persone e Gruppi.
 | **Email (IMAP/SMTP)** | ✅ Completo | Sync messaggi, firma, allegati, mittenti multipli, XOAUTH2 OAuth |
 | **Google OAuth 2.0** | ✅ Implementato | Unified (Email/Drive/Calendar), UI impostazioni, XOAUTH2 SMTP+IMAP |
 | **Repository Remoti** | ✅ Completo | WebDAV, Google Drive, OAuth |
-| **Calendario** | ✅ Completo | Google Calendar, CalDAV, sync bidirezionale, pulsante sync |
+| **Calendario** | ✅ Completo | Google Calendar, CalDAV, sync bidirezionale, pulsante sync, fix salvataggio campi sensibili |
 | **Report** | ✅ Completo | Custom report con colonne/filtri salvabili, tag filter |
 | **Help / Documentazione** | ✅ Completo | Tab Google Drive + Calendar in help page, PDF export |
 | **Distribuzione** | ✅ Completo | build-dist.sh, fix path/cache/permessi/symlink |
@@ -24,6 +24,7 @@ App gestionale Laravel 13 con AdminLTE 4 per gestione Persone e Gruppi.
 - **2026-06-09**: Tag completato (MailingList, Report, Ricerca), @stack scripts fix, storeCustom tag_filter fix, MailingList edit JS fix
 - **2026-06-10**: Per-User Column Views, ColumnManager JS, editVista data-* fix, 405 AJAX fix
 - **2026-06-17**: Google OAuth 2.0 unificato (revert Socialite, unified OAuth per Email/Drive/Calendar, XOAUTH2 SMTP+IMAP, UI impostazioni)
+- **2026-06-17**: Fix CalendarioConnessione — `encryptAndSetConfig()` perdeva campi sensibili in edit, `is_active` checkbox senza hidden fallback, query duplicata nella view

 ## Funzionalità Implementate

@@ -558,6 +559,33 @@ if (!empty($contatto['individuo_id'])) { create }
 **Fix** (`app/Http/Controllers/DocumentoController.php`):
 - Aggiunto `->whereNull('cartella_id')` alla query di root, così mostra solo documenti **senza cartella** (radice). Per vedere i documenti dentro una cartella bisogna navigarci dentro con `?folder_id=X`.

+## 2026-06-17 — Fix: CalendarioConnessione — salvataggio perdeva password/client_secret in edit + is_active sempre true
+
+**Problema**: Salvando una connessione calendario esistente (CalDAV o Google Calendar), i campi sensibili (password, client_secret) venivano sovrascritti con stringa vuota. Il test connessione e la sincronizzazione fallivano silenziosamente.
+
+**Root cause 1 — `encryptAndSetConfig()` sovrascrive campi vuoti**:
+- Il JS di edit (`editCalendario()`) azzera `config[password]` e `config[client_secret]` per sicurezza (valore `''`, placeholder `......`)
+- `encryptAndSetConfig()` in `CalendarioConnessione.php` controllava `$config[$field] !== ''` e saltava la crittografia, ma poi eseguiva `$this->config = $config` — la stringa vuota sovrascriveva il valore crittato precedente
+- `StorageRepository` NON aveva questo bug: `StorageRepositoryService::encryptSensitiveConfig()` preserva i campi esistenti con `} elseif (empty($config[$field]) && !empty($existingConfig[$field])) { $config[$field] = $existingConfig[$field]; }`
+
+**Root cause 2 — `is_active` checkbox senza hidden fallback**:
+- Stesso bug già fixato per email (2026-06-08): checkbox `is_active` senza `<input type="hidden" name="is_active" value="0">`
+- Quando non spuntato, il campo non veniva inviato → `$request->boolean('is_active', true)` restituiva sempre `true`
+- La connessione restava sempre attiva indipendentemente dallo switch
+
+**Root cause 3 — Query duplicata nella view**:
+- `resources/views/impostazioni/index.blade.php:773` ri-eseguiva `CalendarioConnessione::orderBy('ordine')->get()` nonostante il controller lo passasse già via `compact()`
+
+**Fix**:
+1. **`app/Models/CalendarioConnessione.php:56`** — `encryptAndSetConfig()` ora accetta `array $existingConfig = []` e preserva i valori crittati esistenti quando il campo submitted è vuoto (stesso pattern di `StorageRepositoryService::encryptSensitiveConfig()`)
+2. **`app/Http/Controllers/CalendarioConnessioneController.php:67`** — `update()` passa `$connessione->config` come secondo parametro a `encryptAndSetConfig()`
+3. **`resources/views/impostazioni/index.blade.php:1576`** — Aggiunto `<input type="hidden" name="is_active" value="0">` prima della checkbox
+4. **`resources/views/impostazioni/index.blade.php:773`** — Rimossa query duplicata `@php $calendarioConnessioni = ...`
+
+**Verifica**:
+- PHP lint: OK su entrambi i file modificati
+- JS brace balance: OK
+
 ## 2026-06-17 — Fix: Documenti mass tag "nessun documento selezionato"
 **Problema**: Stesso identico bug di Eventi — toolbar button chiamava `$('#massTagModal').modal('show')` e il listener `show.bs.modal` non sempre popolava i campi.

@@ -602,3 +630,38 @@ Aggiunto `if (!Schema::hasTable('table_name')) { ... }` wrapper a 28 migration f

 ### Risultato
 `php artisan migrate --seed` è ora completamente idempotente. Zero errori su 64 migration + 6 seeder. Re-run è un no-op.
+
+## 2026-06-17 — Fix: MailingList tag management (mass tag + per-riga + attiva checkbox)
+
+**Problema**: Nella GUI non era possibile assegnare tag alle mailing list né singolarmente né via azione massiva.
+
+**Root cause**:
+1. **Nessuna route mass-tag per mailing-liste** — Eventi, Documenti, Individui, Gruppi avevano `POST /{entity}/mass-tag` ma MailingList no
+2. **Nessun metodo `massTag()` in MailingListController** — assente rispetto agli altri 4 controller
+3. **Nessun pulsante per-riga tag** nella index view (mancava `<button type="button" onclick="openSingleTag()">`)
+4. **Nessun pulsante toolbar "Tag"** nella index view (mancava il mass tag button)
+5. **Checkbox `attiva` senza hidden fallback** in create/edit — stesso bug ricorrente: deselezionando, il campo non veniva inviato
+
+**Fix**:
+1. **`app/Http/Controllers/MailingListController.php:189-214`** — Aggiunto metodo `massTag()` (stesso pattern di `EventoController@massTag`):
+   - Autorizzazione via `$this->authorizeWrite('mailing')`
+   - Validazione: `tags` required array, `mode` required in:assign,remove
+   - Processa in chunk(100) via `syncWithoutDetaching()` (assign) / `detach()` (remove)
+   - Restituisce `back()->with('success', ...)`
+2. **`routes/web.php:177`** — Aggiunta route `POST mailing-liste/mass-tag` → `MailingListController@massTag` (name: `mailing-liste.mass-tag`)
+3. **`resources/views/mailing-liste/index.blade.php`**:
+   - `select-all` checkbox: permesso esteso da `canDeleteMailing` a `canWriteMailing || canDeleteMailing`
+   - `row-checkbox`: stessa estensione permesso
+   - Toolbar: aggiunto pulsante "Tag" (btn-sm btn-info, `showMassTagModal()`) prima di "Elimina Selezionati"
+   - Azioni per-riga: aggiunto pulsante `fa-tags` con `openSingleTag(lista.id)` dopo il pulsante show
+   - Aggiunto `@if(session('error'))` alert
+   - Aggiunta modale `#massTagModal` (stesso pattern di eventi: titolo, counter, radio assign/remove, `_tag-selector`, hidden `ids`)
+   - Aggiunte funzioni JS: `showMassTagModal()`, `openSingleTag(listId)`
+4. **`resources/views/mailing-liste/create.blade.php:23`** — Aggiunto `<input type="hidden" name="attiva" value="0">` prima della checkbox
+5. **`resources/views/mailing-liste/edit.blade.php:23`** — Stesso hidden fallback
+6. **`resources/views/mailing-liste/edit.blade.php:11`** — Fix percorso hardcoded: `url('/mailing-liste/' . ...)` → `route('mailing-liste.update', ...)`
+
+**Verifica**:
+- PHP lint: OK su controller + routes
+- JS brace balance: OK su index view
+- Route list: `POST mailing-liste/mass-tag` → `MailingListController@massTag` (name: `mailing-liste.mass-tag`)
@@ -64,7 +64,7 @@ class CalendarioConnessioneController extends Controller
        $connessione->is_active = $request->boolean('is_active', true);

        if (!empty($data['config'])) {
-            $connessione->encryptAndSetConfig($data['config']);
+            $connessione->encryptAndSetConfig($data['config'], $connessione->config ?? []);
        }

        $connessione->save();
@@ -186,6 +186,41 @@ class MailingListController extends Controller
        return redirect()->route('mailing-liste.index')->with('success', 'Lista eliminata.');
    }

+    public function massTag(Request $request)
+    {
+        $this->authorizeWrite('mailing');
+        $idsInput = $request->input('ids', '');
+        $ids = is_array($idsInput) ? $idsInput : (is_string($idsInput) ? explode(',', $idsInput) : []);
+
+        if (empty($ids)) {
+            return back()->with('error', 'Nessuna mailing list selezionata.');
+        }
+
+        $data = $request->validate([
+            'tags' => 'required|array',
+            'tags.*' => 'exists:tags,id',
+            'mode' => 'required|in:assign,remove',
+        ]);
+
+        $tagIds = $data['tags'];
+        $mode = $data['mode'];
+        $count = 0;
+
+        MailingList::whereIn('id', $ids)->chunk(100, function ($lists) use ($tagIds, $mode, &$count) {
+            foreach ($lists as $list) {
+                if ($mode === 'assign') {
+                    $list->tags()->syncWithoutDetaching($tagIds);
+                } else {
+                    $list->tags()->detach($tagIds);
+                }
+                $count++;
+            }
+        });
+
+        $actionLabel = $mode === 'assign' ? 'assegnati' : 'rimossi';
+        return back()->with('success', "Tag $actionLabel per $count mailing list.");
+    }
+
    public function massElimina(Request $request)
    {
        $this->authorizeDelete('mailing');
@@ -53,11 +53,13 @@ class CalendarioConnessione extends Model
        return $config;
    }

-    public function encryptAndSetConfig(array $config): void
+    public function encryptAndSetConfig(array $config, array $existingConfig = []): void
    {
        foreach (['username', 'password', 'client_secret', 'refresh_token', 'api_key'] as $field) {
-            if (isset($config[$field]) && $config[$field] !== '' && !str_starts_with($config[$field], 'eyJpdiI')) {
+            if (!empty($config[$field]) && !str_starts_with($config[$field], 'eyJpdiI')) {
                $config[$field] = Crypt::encryptString($config[$field]);
+            } elseif (empty($config[$field]) && !empty($existingConfig[$field])) {
+                $config[$field] = $existingConfig[$field];
            }
        }
        $this->config = $config;
@@ -769,10 +769,6 @@
                            <div class="alert alert-danger">{{ session('error') }}</div>
                        @endif

-                        @php
-                            $calendarioConnessioni = \App\Models\CalendarioConnessione::orderBy('ordine')->get();
-                        @endphp
-
                        @if($calendarioConnessioni->count() > 0)
                        <div class="table-responsive">
                            <table class="table table-bordered table-hover table-sm">
@@ -1572,6 +1568,7 @@
                    </div>
                    <div class="form-group">
                        <div class="custom-control custom-switch">
+                            <input type="hidden" name="is_active" value="0">
                            <input type="checkbox" class="custom-control-input" id="cal_is_active" name="is_active" value="1" checked>
                            <label class="custom-control-label" for="cal_is_active">Connessione attiva</label>
                        </div>
@@ -19,10 +19,11 @@
                </div>
                <div class="col-md-6">
                    <div class="form-group">
-                        <div class="custom-control custom-checkbox mt-4">
-                            <input type="checkbox" class="custom-control-input" id="attiva" name="attiva" value="1" checked>
-                            <label class="custom-control-label" for="attiva">Lista attiva</label>
-                        </div>
+                            <div class="custom-control custom-checkbox mt-4">
+                                <input type="hidden" name="attiva" value="0">
+                                <input type="checkbox" class="custom-control-input" id="attiva" name="attiva" value="1" checked>
+                                <label class="custom-control-label" for="attiva">Lista attiva</label>
+                            </div>
                    </div>
                </div>
            </div>
@@ -8,7 +8,7 @@
        <h3 class="card-title"><i class="fas fa-edit mr-2"></i>Modifica Mailing List</h3>
    </div>
    <div class="card-body">
-        <form action="{{ url('/mailing-liste/' . $mailingList->id) }}" method="POST" id="ml-form">
+        <form action="{{ route('mailing-liste.update', $mailingList->id) }}" method="POST" id="ml-form">
            @csrf @method('PUT')
            <div class="row">
                <div class="col-md-6">
@@ -20,6 +20,7 @@
                <div class="col-md-6">
                    <div class="form-group">
                        <div class="custom-control custom-checkbox mt-4">
+                            <input type="hidden" name="attiva" value="0">
                            <input type="checkbox" class="custom-control-input" id="attiva" name="attiva" value="1" {{ $mailingList->attiva ? 'checked' : '' }}>
                            <label class="custom-control-label" for="attiva">Lista attiva</label>
                        </div>
@@ -18,6 +18,12 @@ $tableColumnsJson = json_encode($tableColumns ?? []);
    {{ session('success') }}
 </div>
@endif
+@if(session('error'))
+<div class="alert alert-danger alert-dismissible">
+    <button type="button" class="close" data-dismiss="alert">&times;</button>
+    {{ session('error') }}
+</div>
+@endif

 <div class="card">
    <div class="card-header">
@@ -26,12 +32,15 @@ $tableColumnsJson = json_encode($tableColumns ?? []);
            <button type="button" class="btn btn-sm btn-secondary mr-1" onclick="$('#table-settings-modal').modal('show')" title="Impostazioni tabella">
                <i class="fas fa-cog"></i>
            </button>
+            @if($canWriteMailing)
+            <button type="button" class="btn btn-sm btn-info mr-2" onclick="showMassTagModal()">
+                <i class="fas fa-tags mr-1"></i> Tag
+            </button>
+            @endif
            @if($canDeleteMailing)
-            <div class="btn-group mr-2">
-                <button type="button" class="btn btn-sm btn-danger" onclick="deleteSelected()">
-                    <i class="fas fa-trash mr-1"></i> Elimina Selezionati
-                </button>
-            </div>
+            <button type="button" class="btn btn-sm btn-danger" onclick="deleteSelected()">
+                <i class="fas fa-trash mr-1"></i> Elimina Selezionati
+            </button>
            @endif
            @if($canWriteMailing)
            <a href="{{ route('mailing-liste.create') }}" class="btn btn-success btn-sm">
@@ -47,7 +56,7 @@ $tableColumnsJson = json_encode($tableColumns ?? []);
            <thead class="thead-light">
                <tr>
                    <th style="width: 40px;" data-column="select">
-                        @if($canDeleteMailing)
+                        @if($canWriteMailing || $canDeleteMailing)
                        <div class="custom-control custom-checkbox">
                            <input type="checkbox" class="custom-control-input" id="select-all" onchange="toggleSelectAll(this)">
                            <label class="custom-control-label" for="select-all"></label>
@@ -79,7 +88,7 @@ $tableColumnsJson = json_encode($tableColumns ?? []);
                @foreach($mailingLists as $lista)
                <tr data-id="{{ $lista->id }}">
                    <td>
-                        @if($canDeleteMailing)
+                        @if($canWriteMailing || $canDeleteMailing)
                        <div class="custom-control custom-checkbox">
                            <input type="checkbox" class="custom-control-input row-checkbox" id="row-checkbox-{{ $lista->id }}" value="{{ $lista->id }}">
                            <label class="custom-control-label" for="row-checkbox-{{ $lista->id }}"></label>
@@ -128,6 +137,9 @@ $tableColumnsJson = json_encode($tableColumns ?? []);
                            <i class="fas fa-eye"></i>
                        </a>
                        @if($canWriteMailing)
+                        <button type="button" class="btn btn-xs btn-info" onclick="openSingleTag({{ $lista->id }})" title="Assegna tag">
+                            <i class="fas fa-tags"></i>
+                        </button>
                        <a href="{{ route('mailing-liste.edit', $lista->id) }}" class="btn btn-xs btn-warning" title="Modifica">
                            <i class="fas fa-edit"></i>
                        </a>
@@ -159,6 +171,51 @@ $tableColumnsJson = json_encode($tableColumns ?? []);

 <div id="vista-data" style="display:none;">{{ $vistaDefaultJson }}</div>

+<div class="modal fade" id="massTagModal" tabindex="-1" role="dialog">
+    @if($canWriteMailing)
+    <div class="modal-dialog modal-dialog-scrollable" role="document">
+        <div class="modal-content">
+            <div class="modal-header">
+                <h5 class="modal-title"><i class="fas fa-tags mr-2"></i>Gestione Tag</h5>
+                <button type="button" class="close" onclick="$('#massTagModal').modal('hide')">
+                    <span>&times;</span>
+                </button>
+            </div>
+            <form id="massTagForm" method="POST" action="{{ route('mailing-liste.mass-tag') }}">
+                @csrf
+                <div class="modal-body">
+                    <p class="mb-2">Operazione su <strong id="massTagCount">0</strong> mailing list selezionate.</p>
+
+                    <div class="form-group mb-3">
+                        <label class="font-weight-bold">Azione</label>
+                        <div class="d-flex" style="gap:1.5rem;">
+                            <div class="form-check">
+                                <input class="form-check-input" type="radio" name="mode" id="mtModeAssign" value="assign" checked>
+                                <label class="form-check-label" for="mtModeAssign">Assegna tag</label>
+                            </div>
+                            <div class="form-check">
+                                <input class="form-check-input" type="radio" name="mode" id="mtModeRemove" value="remove">
+                                <label class="form-check-label" for="mtModeRemove">Rimuovi tag</label>
+                            </div>
+                        </div>
+                    </div>
+
+                    @include('partials._tag-selector', ['label' => 'Seleziona tag'])
+
+                    <input type="hidden" name="ids" id="massTagIds" value="">
+                </div>
+                <div class="modal-footer">
+                    <button type="button" class="btn btn-secondary" onclick="$('#massTagModal').modal('hide')">Annulla</button>
+                    <button type="submit" class="btn btn-primary">
+                        <i class="fas fa-tags mr-1"></i> Applica
+                    </button>
+                </div>
+            </form>
+        </div>
+    </div>
+    @endif
+</div>
+
 <div class="modal fade" id="deleteModal" tabindex="-1" role="dialog">
    @if($canDeleteMailing)
    <div class="modal-dialog" role="document">
@@ -194,6 +251,24 @@ function getSelectedIds() {
    return Array.from(document.querySelectorAll('.row-checkbox:checked')).map(cb => cb.value);
 }

+function showMassTagModal() {
+    const ids = getSelectedIds();
+    if (ids.length === 0) {
+        alert('Seleziona almeno una mailing list');
+        return;
+    }
+    document.getElementById('massTagIds').value = ids.join(',');
+    document.getElementById('massTagCount').textContent = ids.length;
+    $('#massTagModal').modal('show');
+}
+
+function openSingleTag(listId) {
+    document.querySelectorAll('.row-checkbox').forEach(cb => cb.checked = false);
+    var cb = document.getElementById('row-checkbox-' + listId);
+    if (cb) cb.checked = true;
+    showMassTagModal();
+}
+
 function deleteSelected() {
    const selectedIds = getSelectedIds();
    if (selectedIds.length === 0) {
@@ -175,6 +175,7 @@ Route::post('mailing/invio/elabora', [MailingController::class, 'invioElabora'])
 Route::resource('mailing-liste', MailingListController::class)->middleware('auth');
 Route::post('mailing-liste/create-from-individui', [MailingListController::class, 'createFromIndividui'])->middleware('auth');
 Route::post('mailing-liste/mass-elimina', [MailingListController::class, 'massElimina'])->middleware('auth');
+Route::post('mailing-liste/mass-tag', [MailingListController::class, 'massTag'])->middleware('auth')->name('mailing-liste.mass-tag');

 Route::get('/impostazioni', [ImpostazioniController::class, 'index'])->middleware('auth')->name('impostazioni.index');
 Route::post('/impostazioni/tipologie', [ImpostazioniController::class, 'tipologieStore'])->middleware('auth')->name('impostazioni.tipologie.store');