++ gruppi: le azioni del sorvegliante non finiscono piu' in 403; priorita' e filtri nel pacchetto; nome del sorvegliante

- LimitaSorvegliante: la rotta delle azioni di Livewire 3.7 si chiama "default.livewire.update"; con 'livewire.*' ogni clic era un 403. - Territorio::ordinaPerPriorita(): la regola di "Da assegnare" della Home in un metodo solo, usato dalla Home, dall'aggiunta al pacchetto e da "Il mio gruppo". - Aggiunta al pacchetto: ricerca, zona, tipologia, solo prioritari, "Seleziona tutti quelli mostrati", badge e giorni di fermo. - Il mio gruppo: da dare in ordine di priorita' poi i gia' dati; filtri stato, zona, tipologia, solo prioritari; sotto il titolo nome e cognome del sorvegliante. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
2026-10-20 13:58:53 +02:00
parent d2d6b1bdce
commit ce98165667
12 changed files with 371 additions and 110 deletions
--- a/app/Http/Middleware/LimitaSorvegliante.php
+++ b/app/Http/Middleware/LimitaSorvegliante.php
@@ -28,7 +28,10 @@ class LimitaSorvegliante
        'password.change.store',
        'logout',
        'verification.*',
-        'livewire.*',
+        // Le azioni Livewire: la rotta si chiama "default.livewire.update"
+        // (Livewire 3.7), non "livewire.update". Con 'livewire.*' ogni clic
+        // del sorvegliante finiva in 403.
+        '*livewire.*',
    ];

    public function handle(Request $request, Closure $next): Response
--- a/app/Livewire/Gruppi/GruppoShow.php
+++ b/app/Livewire/Gruppi/GruppoShow.php
@@ -32,6 +32,16 @@ class GruppoShow extends Component
    /** @var int[] territori da aggiungere al pacchetto */
    public array $nuoviTerritori = [];

+    // Filtri dei territori in reparto da aggiungere al pacchetto. L'ordine è
+    // quello di priorità (Territorio::ordinaPerPriorita), come nella Home.
+    public string $cercaTerritorio = '';
+
+    public string $filtroZona = '';
+
+    public string $filtroTipologia = '';
+
+    public bool $soloPrioritari = false;
+
    #[Locked]
    public ?string $passwordTemporanea = null;

@@ -91,6 +101,45 @@ class GruppoShow extends Component
        $this->nuoviTerritori = [];
    }

+    /** Tutti i territori che i filtri mostrano finiscono tra quelli da aggiungere. */
+    public function selezionaMostrati(): void
+    {
+        $this->nuoviTerritori = collect($this->nuoviTerritori)
+            ->map(fn ($id) => (string) $id)
+            ->merge($this->candidatiFiltrati()->pluck('id')->map(fn ($id) => (string) $id))
+            ->unique()
+            ->values()
+            ->all();
+    }
+
+    public function deselezionaTutti(): void
+    {
+        $this->nuoviTerritori = [];
+    }
+
+    /** I territori in reparto, in ordine di priorità. */
+    private function candidati(): \Illuminate\Support\Collection
+    {
+        return Territorio::ordinaPerPriorita(
+            Territorio::inReparto()->with(['zona', 'tipologia', 'ultimaAssegnazione'])->get()
+        );
+    }
+
+    private function candidatiFiltrati(?\Illuminate\Support\Collection $candidati = null): \Illuminate\Support\Collection
+    {
+        $cerca = Str::lower(Str::ascii(trim($this->cercaTerritorio)));
+
+        return ($candidati ?? $this->candidati())
+            ->when($this->filtroZona !== '', fn ($c) => $c->where('zona_id', (int) $this->filtroZona))
+            ->when($this->filtroTipologia !== '', fn ($c) => $c->where('tipologia_id', (int) $this->filtroTipologia))
+            ->when($this->soloPrioritari, fn ($c) => $c->where('in_priorita', true))
+            ->when($cerca !== '', fn ($c) => $c->filter(fn (Territorio $t) => str_contains(
+                Str::lower(Str::ascii("{$t->numero} {$t->zona?->nome} {$t->tipologia?->nome}")),
+                $cerca
+            )))
+            ->values();
+    }
+
    public function rimanda(int $territorioId): void
    {
        $territorio = Territorio::findOrFail($territorioId);
@@ -135,13 +184,21 @@ class GruppoShow extends Component
        $membri = $this->gruppo->membri()->get()->sortBy(fn ($p) => mb_strtolower($p->cognome . ' ' . $p->nome))->values();
        $territori = $this->gruppo->territori()->with(['zona', 'assegnazioneCorrente.proclamatore'])->get()->sortBy('numero', SORT_NATURAL)->values();

+        // I territori in reparto (attivi, non assegnati, fuori da ogni
+        // pacchetto), in ordine di priorità; zone e tipologie dei filtri sono
+        // quelle che ci sono tra loro.
+        $candidati = $this->candidati();
+
        return view('livewire.gruppi.gruppo-show', [
            'membri' => $membri,
            'territori' => $territori,
            // Si possono aggiungere i proclamatori attivi senza gruppo...
            'candidatiMembri' => Proclamatore::attivi()->whereNull('gruppo_servizio_id')->get()->sortBy(fn ($p) => mb_strtolower($p->cognome . ' ' . $p->nome))->values(),
-            // ...e i territori in reparto (attivi, non assegnati, fuori da ogni pacchetto).
-            'candidatiTerritori' => Territorio::inReparto()->with('zona')->get()->sortBy('numero', SORT_NATURAL)->values(),
+            // ...e i territori in reparto.
+            'totaleCandidati' => $candidati->count(),
+            'candidatiTerritori' => $this->candidatiFiltrati($candidati),
+            'zoneCandidati' => $candidati->filter(fn ($t) => $t->zona)->mapWithKeys(fn ($t) => [$t->zona_id => $t->zona->nome])->sort(),
+            'tipologieCandidati' => $candidati->filter(fn ($t) => $t->tipologia)->mapWithKeys(fn ($t) => [$t->tipologia_id => $t->tipologia->nome])->sort(),
        ]);
    }
 }
--- a/app/Livewire/Gruppi/MioGruppo.php
+++ b/app/Livewire/Gruppi/MioGruppo.php
@@ -12,6 +12,7 @@ use App\Services\GruppiServizio;
 use Carbon\Carbon;
 use InvalidArgumentException;
 use Livewire\Attributes\Locked;
+use Livewire\Attributes\Url;
 use Livewire\Component;

 /**
@@ -35,6 +36,20 @@ class MioGruppo extends Component

    public string $dataAssegnazione = '';

+    // Filtri del pacchetto, nell'indirizzo: tornando dal rientro si ritrovano.
+    #[Url(except: '')]
+    public string $filtroZona = '';
+
+    #[Url(except: '')]
+    public string $filtroTipologia = '';
+
+    /** '' tutti, 'da_dare', 'dati' */
+    #[Url(except: '')]
+    public string $filtroStato = '';
+
+    #[Url(except: false)]
+    public bool $soloPrioritari = false;
+
    public function mount(): void
    {
        $this->dataAssegnazione = now()->format('Y-m-d');
@@ -170,11 +185,25 @@ class MioGruppo extends Component

    public function render()
    {
-        $gruppo = $this->gruppo();
+        $gruppo = $this->gruppo()?->load('sorvegliante');

-        $territori = $gruppo
-            ? $gruppo->territori()->with(['zona', 'assegnazioneCorrente.proclamatore'])->get()->sortBy('numero', SORT_NATURAL)->values()
+        // Prima quelli da dare, in ordine di priorità come "Da assegnare"
+        // nella Home (Territorio::ordinaPerPriorita); poi quelli già dati, dal
+        // più vecchio.
+        $pacchetto = $gruppo
+            ? $gruppo->territori()->with(['zona', 'tipologia', 'ultimaAssegnazione', 'assegnazioneCorrente.proclamatore'])->get()
            : collect();
+        [$dati, $daDare] = $pacchetto->partition(fn (Territorio $t) => $t->assegnazioneCorrente !== null);
+        $pacchetto = Territorio::ordinaPerPriorita($daDare)
+            ->concat($dati->sortBy(fn (Territorio $t) => $t->assegnazioneCorrente->assigned_at)->values());
+
+        $territori = $pacchetto
+            ->when($this->filtroZona !== '', fn ($c) => $c->where('zona_id', (int) $this->filtroZona))
+            ->when($this->filtroTipologia !== '', fn ($c) => $c->where('tipologia_id', (int) $this->filtroTipologia))
+            ->when($this->filtroStato === 'da_dare', fn ($c) => $c->whereNull('assegnazioneCorrente'))
+            ->when($this->filtroStato === 'dati', fn ($c) => $c->whereNotNull('assegnazioneCorrente'))
+            ->when($this->soloPrioritari, fn ($c) => $c->where('in_priorita', true))
+            ->values();

        $membri = $gruppo
            ? $gruppo->membri()->where('attivo', true)->get()->sortBy(fn ($p) => mb_strtolower($p->cognome . ' ' . $p->nome))->values()
@@ -182,7 +211,10 @@ class MioGruppo extends Component

        return view('livewire.gruppi.mio-gruppo', [
            'gruppo' => $gruppo,
+            'pacchetto' => $pacchetto,
            'territori' => $territori,
+            'zone' => $pacchetto->filter(fn ($t) => $t->zona)->mapWithKeys(fn ($t) => [$t->zona_id => $t->zona->nome])->sort(),
+            'tipologie' => $pacchetto->filter(fn ($t) => $t->tipologia)->mapWithKeys(fn ($t) => [$t->tipologia_id => $t->tipologia->nome])->sort(),
            'membri' => $membri,
            'opzioniMembri' => Proclamatore::opzioniScelta($membri, conGruppo: false),
            'invio' => $this->invioCorrente(),
--- a/app/Livewire/Home.php
+++ b/app/Livewire/Home.php
@@ -18,35 +18,10 @@ class Home extends Component
        $this->authorize('territori.assign');

        $settings = Setting::instance();
-        $priorityThreshold = (int) ($settings->giorni_giacenza_prioritari ?? 180);

-        $territori = Territorio::inReparto()
-            ->with('zona', 'tipologia', 'ultimaAssegnazione')
-            ->get()
-            ->map(function (Territorio $territorio) use ($priorityThreshold) {
-                $ultima = $territorio->ultimaAssegnazione;
-                if ($ultima && $ultima->returned_at) {
-                    $giorniGiacenza = $ultima->returned_at->startOfDay()->diffInDays(today());
-                } elseif (! $ultima) {
-                    $giorniGiacenza = $territorio->created_at->startOfDay()->diffInDays(today());
-                } else {
-                    $giorniGiacenza = 0;
-                }
-                $territorio->setAttribute('home_giorni_giacenza', $giorniGiacenza);
-                $territorio->setAttribute(
-                    'home_is_prioritario',
-                    (bool) $territorio->prioritario || $giorniGiacenza > $priorityThreshold
-                );
-                return $territorio;
-            })
-            ->sort(function (Territorio $left, Territorio $right) {
-                $p = (int) $right->home_is_prioritario <=> (int) $left->home_is_prioritario;
-                if ($p !== 0) return $p;
-                $g = $right->home_giorni_giacenza <=> $left->home_giorni_giacenza;
-                if ($g !== 0) return $g;
-                return strnatcasecmp((string) $left->numero, (string) $right->numero);
-            })
-            ->values();
+        $territori = Territorio::ordinaPerPriorita(
+            Territorio::inReparto()->with('zona', 'tipologia', 'ultimaAssegnazione')->get()
+        );

        $pdf = Pdf::loadView('pdf.territori-lista', [
            'titolo' => 'Territori da Assegnare',
@@ -90,7 +65,6 @@ class Home extends Component
    {
        $settings = Setting::instance();
        $homeLimit = max(1, (int) ($settings->home_limit_list ?? 10));
-        $priorityThreshold = (int) ($settings->giorni_giacenza_prioritari ?? 180);
        $annoCorrente = AnnoTeocratico::corrente();
        $campagnaAttiva = Campagna::attiva();

@@ -149,46 +123,11 @@ class Home extends Component
            ];
        }

-        // Quick lists
-        $territoriDaAssegnare = Territorio::inReparto()
-            ->with('zona', 'tipologia', 'ultimaAssegnazione')
-            ->get()
-            ->map(function (Territorio $territorio) use ($priorityThreshold) {
-                $ultima = $territorio->ultimaAssegnazione;
-
-                if ($ultima && $ultima->returned_at) {
-                    $giorniGiacenza = $ultima->returned_at->startOfDay()->diffInDays(today());
-                } elseif (! $ultima) {
-                    $giorniGiacenza = $territorio->created_at->startOfDay()->diffInDays(today());
-                } else {
-                    $giorniGiacenza = 0;
-                }
-
-                $territorio->setAttribute('home_giorni_giacenza', $giorniGiacenza);
-                $territorio->setAttribute(
-                    'home_is_prioritario',
-                    (bool) $territorio->prioritario || $giorniGiacenza > $priorityThreshold
-                );
-
-                return $territorio;
-            })
-            ->sort(function (Territorio $left, Territorio $right) {
-                $priorityComparison = (int) $right->home_is_prioritario <=> (int) $left->home_is_prioritario;
-
-                if ($priorityComparison !== 0) {
-                    return $priorityComparison;
-                }
-
-                $giacenzaComparison = $right->home_giorni_giacenza <=> $left->home_giorni_giacenza;
-
-                if ($giacenzaComparison !== 0) {
-                    return $giacenzaComparison;
-                }
-
-                return strnatcasecmp((string) $left->numero, (string) $right->numero);
-            })
-            ->take($homeLimit)
-            ->values();
+        // Quick lists: la logica di priorità sta in Territorio::ordinaPerPriorita
+        // (la usano anche il pacchetto dei gruppi di servizio e "Il mio gruppo").
+        $territoriDaAssegnare = Territorio::ordinaPerPriorita(
+            Territorio::inReparto()->with('zona', 'tipologia', 'ultimaAssegnazione')->get()
+        )->take($homeLimit)->values();

        $daRientrare = Territorio::daRientrare()
            ->with(['zona', 'assegnazioneCorrente.proclamatore'])
--- a/app/Models/Territorio.php
+++ b/app/Models/Territorio.php
@@ -169,6 +169,41 @@ class Territorio extends Model
        return false;
    }

+    /**
+     * La logica di priorità dei territori da distribuire, la stessa ovunque
+     * (Home "Da assegnare", pacchetto di un gruppo, "Il mio gruppo"): prima i
+     * prioritari — spunta manuale o fermi da più giorni della soglia
+     * "giorni_giacenza_prioritari" — poi chi è fermo da più tempo, poi il
+     * numero. Su ogni territorio imposta giacenza_giorni (dall'ultimo rientro,
+     * o dalla creazione se mai assegnato; 0 se è fuori) e in_priorita, da
+     * mostrare. Vuole 'ultimaAssegnazione' già caricata, per non fare una
+     * query a territorio.
+     */
+    public static function ordinaPerPriorita(iterable $territori): \Illuminate\Support\Collection
+    {
+        $soglia = (int) (Setting::instance()->giorni_giacenza_prioritari ?? 180);
+
+        return collect($territori)
+            ->each(function (self $territorio) use ($soglia) {
+                $ultima = $territorio->ultimaAssegnazione;
+                if ($ultima && $ultima->returned_at) {
+                    $giorni = (int) $ultima->returned_at->startOfDay()->diffInDays(today());
+                } elseif (! $ultima) {
+                    $giorni = (int) $territorio->created_at->startOfDay()->diffInDays(today());
+                } else {
+                    $giorni = 0;
+                }
+
+                $territorio->setAttribute('giacenza_giorni', $giorni);
+                $territorio->setAttribute('in_priorita', (bool) $territorio->prioritario || $giorni > $soglia);
+            })
+            ->sort(function (self $a, self $b) {
+                return [(int) $b->in_priorita, $b->giacenza_giorni] <=> [(int) $a->in_priorita, $a->giacenza_giorni]
+                    ?: strnatcasecmp((string) $a->numero, (string) $b->numero);
+            })
+            ->values();
+    }
+
    /**
     * L'opzione per il campo di scelta con ricerca (<x-scelta>): si cerca per
     * numero, zona e tipologia. Con $motivo compare cercando ma non si può