francesco/termanager2
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

++ proclamatori senza email e telefono; il link si copia o si condivide

Browse Source 
- Migrazione: tolte le colonne email e telefono dei proclamatori (vuote
  in produzione). Tolti da form, scheda, elenco, XML (un file vecchio si
  importa ignorandoli).
- Invio del link: solo "Copia link" e "Condividi…" (menu di condivisione
  del telefono, navigator.share): il destinatario lo sceglie chi usa
  l'app. Via WhatsApp diretto, canale email, notifica LinkTerritorio,
  componente x-whatsapp.
- Elenco assegnati: "Condividi" al posto di WhatsApp; a condivisione
  fatta il link risulta inviato.
- Informativa privacy, README, HANDOFF e test aggiornati.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
Francesco Picone
2026-10-19 16:57:23 +02:00
parent 3000ca9939
commit 7384a4b39d
25 changed files with 436 additions and 674 deletions
Download Patch File Download Diff File Expand all files Collapse all files

48
app/Livewire/Assegnazioni/Assegna.php
View File

@@ -10,10 +10,7 @@ use App\Models\Territorio;
use App\Models\Proclamatore;
use App\Models\Assegnazione;
use App\Models\AnnoTeocratico;
use App\Notifications\LinkTerritorio;
use App\Services\ConfigurazionePosta;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Notification;
class Assegna extends Component
{
@@ -22,10 +19,11 @@ class Assegna extends Component
// ─── Invio del link ─────────────────────────────────────────
//
// Dopo l'assegnazione non si torna subito alla pagina: si sceglie il
// canale (copia link sempre; WhatsApp ed email solo se il proclamatore ha
// numero / indirizzo), si dice se il link è partito, e solo allora si va
// avanti. Se la procedura si interrompe, l'assegnazione resta con
// link_da_inviare e lo si segnala dove compare.
// canale (copia link o Condividi: il destinatario lo sceglie chi usa
// l'app dai suoi contatti, i proclamatori non hanno numero né email),
// si dice se il link è partito, e solo allora si va avanti. Se la
// procedura si interrompe, l'assegnazione resta con link_da_inviare e lo
// si segnala dove compare.
/** L'assegnazione di cui si sta mandando il link. */
#[Locked]
@@ -162,15 +160,14 @@ class Assegna extends Component
$this->erroreInvio = '';
}
/** Copia link o WhatsApp: l'invio vero lo fa chi usa l'app, qui si annota il canale. */
/**
* Copia link o Condividi: l'invio vero lo fa chi usa l'app (il
* destinatario lo sceglie lui dai suoi contatti), qui si annota il canale.
*/
public function canale(string $canale): void
{
$assegnazione = $this->assegnazioneInInvio();
if (! $assegnazione || ! in_array($canale, ['copia', 'whatsapp'], true)) {
return;
}
if ($canale === 'whatsapp' && ! $assegnazione->proclamatore?->whatsappUrl()) {
if (! $assegnazione || ! in_array($canale, Assegnazione::CANALI_ATTIVI, true) || ! $assegnazione->shortPdfUrl()) {
return;
}
@@ -178,30 +175,6 @@ class Assegna extends Component
$this->erroreInvio = '';
}
/** Email: la manda l'applicazione, subito (vedi LinkTerritorio). */
public function inviaEmail(): void
{
$assegnazione = $this->assegnazioneInInvio();
$email = trim((string) $assegnazione?->proclamatore?->email);
$link = $assegnazione?->shortPdfUrl();
if (! $assegnazione || $email === '' || ! $link || ! app(ConfigurazionePosta::class)->funzionante()) {
return;
}
try {
Notification::route('mail', $email)->notifyNow(new LinkTerritorio($assegnazione, $link));
} catch (\Throwable $e) {
report($e);
$this->erroreInvio = "L'email non è partita: controlla il server di posta, oppure usa un altro canale.";
return;
}
$this->canaleUsato = 'email';
$this->erroreInvio = '';
}
/**
* La risposta a "Hai inviato il link?": la si registra e solo adesso si
* va avanti. "Sì" vuole un canale usato; "No" lascia il link da inviare.
@@ -284,7 +257,6 @@ class Assegna extends Component
return view('livewire.assegnazioni.assegna', [
'invio' => $invio,
'postaAttiva' => $invio ? app(ConfigurazionePosta::class)->funzionante() : false,
'opzioniTerritori' => $opzioniTerritori,
'opzioniProclamatori' => $opzioniProclamatori,
'anteprimaIniziale' => $this->territorio_id ? $territori->firstWhere('id', $this->territorio_id)?->miniaturaUrl() : null,

7
app/Livewire/Assegnazioni/ElencoAssegnati.php
View File

@@ -29,6 +29,13 @@ class ElencoAssegnati extends Component
$assegnazione->registraInvioLink(! $assegnazione->link_sent, $assegnazione->link_canale);
}
/** "Condividi" dall'elenco andato a buon fine: il link risulta inviato. */
public function segnaCondiviso(int $assegnazioneId): void
{
$this->authorize('territori.assign');
Assegnazione::aperte()->find($assegnazioneId)?->registraInvioLink(true, 'condividi');
}
public function placeholder()
{
return view('livewire.assegnazioni.elenco-assegnati-attesa');

8
app/Livewire/Privacy.php
View File

@@ -28,7 +28,7 @@ class Privacy extends Component
/** I permessi spiegati a chi legge l'informativa. */
public const PERMESSI = [
'settings.manage' => 'impostazioni, utenti, backup e import',
'proclamatori.manage' => 'anagrafica dei proclamatori, contatti compresi',
'proclamatori.manage' => 'anagrafica dei proclamatori',
'territori.manage' => 'gestione dei territori e delle piantine',
'campagne.manage' => 'campagne',
'registro.view' => 'consultazione del registro delle assegnazioni',
@@ -68,13 +68,11 @@ class Privacy extends Component
->map(fn ($p) => self::PERMESSI[$p] ?? $p)
->all(),
]),
// Quanti dati sono conservati adesso. I campi cifrati restano NULL
// quando sono vuoti, quindi si contano in SQL senza decifrare.
// Quanti dati sono conservati adesso: solo conteggi, contati in
// SQL, senza decifrare né mostrare nomi.
'conteggi' => [
'proclamatori' => Proclamatore::count(),
'proclamatoriAttivi' => Proclamatore::where('attivo', true)->count(),
'conEmail' => Proclamatore::whereNotNull('email')->count(),
'conTelefono' => Proclamatore::whereNotNull('telefono')->count(),
'anonimizzati' => Proclamatore::onlyTrashed()->count(),
'utenti' => User::count(),
'assegnazioni' => Assegnazione::count(),

15
app/Livewire/Proclamatori/ProclamatoreCreate.php
View File

@@ -13,8 +13,6 @@ class ProclamatoreCreate extends Component
public string $nome = '';
public string $cognome = '';
public bool $attivo = true;
public string $email = '';
public string $telefono = '';
protected function rules(): array
{
@@ -22,17 +20,6 @@ class ProclamatoreCreate extends Component
'nome' => 'required|string|max:100',
'cognome' => 'required|string|max:100',
'attivo' => 'boolean',
// Facoltativi: si controlla il formato solo se compilati.
'email' => 'nullable|email|max:255',
'telefono' => ['nullable', 'string', 'max:30', 'regex:/^\+?[0-9 ().\/-]{6,25}$/'],
];
}
protected function messages(): array
{
return [
'email.email' => "L'indirizzo email non è valido (puoi anche lasciarlo vuoto).",
'telefono.regex' => 'Il numero può contenere solo cifre, spazi e un "+" iniziale (puoi anche lasciarlo vuoto).',
];
}
@@ -46,8 +33,6 @@ class ProclamatoreCreate extends Component
'nome' => $this->nome,
'cognome' => $this->cognome,
'attivo' => $this->attivo,
'email' => trim($this->email) ?: null,
'telefono' => trim($this->telefono) ?: null,
]);
activity()->causedBy(auth()->user())

17
app/Livewire/Proclamatori/ProclamatoreEdit.php
View File

@@ -14,8 +14,6 @@ class ProclamatoreEdit extends Component
public string $nome = '';
public string $cognome = '';
public bool $attivo = true;
public string $email = '';
public string $telefono = '';
public function mount(Proclamatore $proclamatore)
{
@@ -23,8 +21,6 @@ class ProclamatoreEdit extends Component
$this->nome = $proclamatore->nome;
$this->cognome = $proclamatore->cognome;
$this->attivo = $proclamatore->attivo;
$this->email = (string) $proclamatore->email;
$this->telefono = (string) $proclamatore->telefono;
}
protected function rules(): array
@@ -33,17 +29,6 @@ class ProclamatoreEdit extends Component
'nome' => 'required|string|max:100',
'cognome' => 'required|string|max:100',
'attivo' => 'boolean',
// Facoltativi: si controlla il formato solo se compilati.
'email' => 'nullable|email|max:255',
'telefono' => ['nullable', 'string', 'max:30', 'regex:/^\+?[0-9 ().\/-]{6,25}$/'],
];
}
protected function messages(): array
{
return [
'email.email' => "L'indirizzo email non è valido (puoi anche lasciarlo vuoto).",
'telefono.regex' => 'Il numero può contenere solo cifre, spazi e un "+" iniziale (puoi anche lasciarlo vuoto).',
];
}
@@ -57,8 +42,6 @@ class ProclamatoreEdit extends Component
'nome' => $this->nome,
'cognome' => $this->cognome,
'attivo' => $this->attivo,
'email' => trim($this->email) ?: null,
'telefono' => trim($this->telefono) ?: null,
]);
session()->flash('success', "Proclamatore {$this->proclamatore->nome_completo} aggiornato.");

9
app/Livewire/Settings/XmlExchange.php
View File

@@ -629,9 +629,8 @@ class XmlExchange extends Component
'nome' => (string) $proclamatoreNode->nome,
'cognome' => (string) $proclamatoreNode->cognome,
'attivo' => ((int) ($proclamatoreNode->attivo ?? 1)) === 1,
// Facoltativi, e assenti nei file di prima dell'ottobre 2026.
'email' => trim((string) ($proclamatoreNode->email ?? '')) ?: null,
'telefono' => trim((string) ($proclamatoreNode->telefono ?? '')) ?: null,
// <email> e <telefono>, presenti nei file del 13-19 ottobre
// 2026, si ignorano: i proclamatori non li hanno più.
]);
$this->applyRowMeta('proclamatori', $p->id, $proclamatoreNode, true);
$stats['proclamatori_importati']++;
@@ -830,7 +829,7 @@ class XmlExchange extends Component
])->all(),
'zones' => $this->righeEsportate(Zona::query()->orderBy('id'), ['id', 'nome', 'attivo', 'created_at', 'updated_at']),
'tipologie' => $this->righeEsportate(Tipologia::query()->orderBy('id'), ['id', 'nome', 'attivo', 'created_at', 'updated_at']),
'proclamatori' => $this->righeEsportate(Proclamatore::query()->withTrashed()->orderBy('id'), ['id', 'nome', 'cognome', 'email', 'telefono', 'attivo', 'deleted_at', 'created_at', 'updated_at']),
'proclamatori' => $this->righeEsportate(Proclamatore::query()->withTrashed()->orderBy('id'), ['id', 'nome', 'cognome', 'attivo', 'deleted_at', 'created_at', 'updated_at']),
'territori' => $this->righeEsportate(Territorio::query()->withTrashed()->orderBy('id'), ['id', 'numero', 'zona_id', 'tipologia_id', 'confini', 'note', 'attivo', 'prioritario', 'pdf_path', 'thumbnail_path', 'deleted_at', 'created_at', 'updated_at']),
'anni_teocratici' => $this->righeEsportate(AnnoTeocratico::query()->orderBy('id'), ['id', 'label', 'start_date', 'end_date', 'created_at', 'updated_at']),
'campagne' => $this->righeEsportate(Campagna::query()->orderBy('id'), ['id', 'descrizione', 'start_date', 'end_date', 'created_at', 'updated_at']),
@@ -1265,8 +1264,6 @@ class XmlExchange extends Component
$this->addLegacyIdAttribute($node, $proclamatore);
$this->addXmlText($node, 'nome', (string) ($proclamatore['nome'] ?? ''));
$this->addXmlText($node, 'cognome', (string) ($proclamatore['cognome'] ?? ''));
$this->addXmlText($node, 'email', (string) ($proclamatore['email'] ?? ''));
$this->addXmlText($node, 'telefono', (string) ($proclamatore['telefono'] ?? ''));
$node->addChild('attivo', (string) ((int) ($proclamatore['attivo'] ?? 1)));
$node->addChild('deleted_at', (string) ($proclamatore['deleted_at'] ?? ''));
$this->addTimestampNodes($node, $proclamatore);

23
app/Models/Assegnazione.php
View File

@@ -220,8 +220,17 @@ class Assegnazione extends Model
$this->registraInvioLink(true, $canale);
}
/** Canali con cui si manda il link al proclamatore. */
public const CANALI_LINK = ['copia' => 'Copia link', 'whatsapp' => 'WhatsApp', 'email' => 'Email'];
/**
* Canali con cui si manda il link al proclamatore. Dal 19 ottobre 2026 se
* ne usano due: copia link e Condividi (il menu di condivisione del
* telefono); il destinatario lo sceglie chi usa l'app. "whatsapp" ed
* "email" restano solo perché le assegnazioni già inviate così lo
* mostrino e l'XML lo accetti.
*/
public const CANALI_LINK = ['copia' => 'Copia link', 'condividi' => 'Condividi', 'whatsapp' => 'WhatsApp', 'email' => 'Email'];
/** I canali che si possono scegliere oggi nella finestra di invio. */
public const CANALI_ATTIVI = ['copia', 'condividi'];
/**
* Esito dell'invio del link: inviato (con il canale usato) oppure no. Un
@@ -243,10 +252,14 @@ class Assegnazione extends Model
return $this->link_da_inviare && ! $this->link_sent && $this->is_aperta;
}
/** Il messaggio con il link, uguale per WhatsApp ed email. */
public function messaggioLink(): ?string
/**
* Il messaggio con il link, per "Condividi": il menu di condivisione del
* telefono, da cui chi usa l'app sceglie app e contatto. L'app non vede
* quei contatti.
*/
public function messaggioLink(?string $link = null): ?string
{
$link = $this->shortPdfUrl();
$link ??= $this->shortPdfUrl();
if ($link === null) {
return null;
}

52
app/Models/Proclamatore.php
View File

@@ -13,16 +13,16 @@ class Proclamatore extends Model
protected $table = 'proclamatori';
protected $fillable = ['nome', 'cognome', 'email', 'telefono', 'attivo'];
// Niente email né telefono (tolti il 19 ottobre 2026): il link della
// piantina lo manda chi assegna, copiandolo o con "Condividi", e il
// destinatario lo sceglie dai propri contatti.
protected $fillable = ['nome', 'cognome', 'attivo'];
protected function casts(): array
{
return [
'nome' => 'encrypted',
'cognome' => 'encrypted',
// Facoltativi. Cifrati come nome e cognome: dati personali.
'email' => 'encrypted',
'telefono' => 'encrypted',
'attivo' => 'boolean',
];
}
@@ -30,7 +30,7 @@ class Proclamatore extends Model
public function getActivitylogOptions(): LogOptions
{
return LogOptions::defaults()
->logOnly(['attivo']) // Do NOT log nome/cognome/email/telefono in audit (encrypted, GDPR)
->logOnly(['attivo']) // Do NOT log nome/cognome in audit (encrypted, GDPR)
->logOnlyDirty()
->dontSubmitEmptyLogs();
}
@@ -43,46 +43,6 @@ class Proclamatore extends Model
return trim($this->cognome . ' ' . $this->nome);
}
/**
* Il numero come lo vuole WhatsApp: solo cifre, con il prefisso
* internazionale. Un numero senza prefisso si considera italiano: un
* cellulare scritto "333 1234567" diventa 393331234567. Null se non c'è un
* numero utilizzabile.
*/
public function telefonoInternazionale(): ?string
{
$grezzo = trim((string) $this->telefono);
if ($grezzo === '') {
return null;
}
$cifre = preg_replace('/\D+/', '', $grezzo);
if (str_starts_with($cifre, '00') && ! str_starts_with($grezzo, '+')) {
$cifre = substr($cifre, 2);
} elseif (! str_starts_with($grezzo, '+') && (! str_starts_with($cifre, '39') || strlen($cifre) <= 10)) {
// Senza prefisso: cellulari (3…) e fissi (0…) italiani. Un
// "39…" di 11 cifre o più ha già il prefisso, scritto senza "+".
$cifre = '39' . $cifre;
}
return strlen($cifre) >= 8 && strlen($cifre) <= 15 ? $cifre : null;
}
/**
* Il link che apre una chat WhatsApp con il proclamatore, con un messaggio
* già scritto se c'è. Null se non ha un numero utilizzabile.
*/
public function whatsappUrl(?string $messaggio = null): ?string
{
$numero = $this->telefonoInternazionale();
if ($numero === null) {
return null;
}
return 'https://wa.me/' . $numero . ($messaggio !== null && $messaggio !== '' ? '?text=' . rawurlencode($messaggio) : '');
}
/**
* L'opzione per il campo di scelta con ricerca (<x-scelta>): si cerca per
* cognome e nome. Nome e cognome sono cifrati nel database: la ricerca
@@ -134,8 +94,6 @@ class Proclamatore extends Model
{
$this->nome = 'Anonimo';
$this->cognome = 'Proclamatore #' . $this->id;
$this->email = null;
$this->telefono = null;
$this->attivo = false;
$this->save();
$this->delete(); // soft delete

45
app/Notifications/LinkTerritorio.php
View File

@@ -1,45 +0,0 @@
<?php
namespace App\Notifications;
use App\Models\Assegnazione;
use App\Models\Setting;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
/**
* Il link della piantina, mandato per email al proclamatore a cui è stato
* assegnato il territorio.
*
* Si invia con notifyNow(), mai in coda: in coda il job salverebbe in chiaro
* (Redis, failed_jobs) l'indirizzo che nel database è cifrato.
*/
class LinkTerritorio extends Notification
{
public function __construct(
private readonly Assegnazione $assegnazione,
private readonly string $link,
) {}
public function via(object $notifiable): array
{
return ['mail'];
}
public function toMail(object $notifiable): MailMessage
{
$numero = $this->assegnazione->territorio?->numero;
$congregazione = trim((string) Setting::getValue('congregazione_nome', ''));
// Stessa regola di AssignmentPdfController::linkScaduto().
$scadenza = $this->assegnazione->assigned_at->copy()
->addDays(max(1, (int) Setting::getValue('giorni_per_smarrito', 120)));
return (new MailMessage)
->subject("Territorio N° {$numero} — piantina")
->greeting('Ciao ' . $this->assegnazione->proclamatore?->nome . ',')
->line("ti è stato assegnato il territorio N° {$numero}.")
->action('Apri la piantina', $this->link)
->line("Il link funziona finché il territorio è assegnato a te, al massimo fino al {$scadenza->format('d/m/Y')}.")
->salutation($congregazione !== '' ? $congregazione : 'TerManager2');
}
}