La tua risposta
diff --git a/src/lib/eventi-visibili.ts b/src/lib/eventi-visibili.ts
new file mode 100644
index 0000000..dccdaa5
--- /dev/null
+++ b/src/lib/eventi-visibili.ts
@@ -0,0 +1,14 @@
+import { eventoVisibile, useEventi } from "./eventi";
+import { usePuoGestireEventi } from "./ruoli";
+import { useGiocatoreId } from "./user-store";
+
+/** Gli eventi che l'utente corrente può vedere (calendario, home, elenco). */
+export function useEventiVisibili() {
+ const { eventi, ...resto } = useEventi();
+ const giocatoreId = useGiocatoreId();
+ const vedeTutto = usePuoGestireEventi();
+ return {
+ ...resto,
+ eventi: eventi.filter((e) => eventoVisibile(e, giocatoreId, vedeTutto)),
+ };
+}
diff --git a/src/lib/eventi.ts b/src/lib/eventi.ts
index a68d851..a0f1477 100644
--- a/src/lib/eventi.ts
+++ b/src/lib/eventi.ts
@@ -90,6 +90,25 @@ export function useEventi() {
return { ...query, eventi: query.data ?? [] };
}
+/** Il giocatore è tra i convocati (elenco vuoto = tutta la rosa). */
+export function eConvocato(evento: Pick, giocatoreId: string): boolean {
+ return evento.convocati.length === 0 || evento.convocati.includes(giocatoreId);
+}
+
+/**
+ * Chi vede un evento: i convocati; se `convocati` è vuoto, tutta la rosa. Chi gestisce gli
+ * eventi (admin e allenatori) li vede sempre, convocato o no.
+ */
+export function eventoVisibile(
+ evento: Pick,
+ giocatoreId: string | null,
+ vedeTutto: boolean,
+): boolean {
+ if (vedeTutto) return true;
+ if (evento.convocati.length === 0) return true;
+ return giocatoreId !== null && eConvocato(evento, giocatoreId);
+}
+
export function useEvento(id: string) {
const { eventi, ...resto } = useEventi();
return { ...resto, evento: eventi.find((e) => e.id === id) ?? null };
@@ -198,6 +217,11 @@ export function compleanniEventi(
/** Rosa convocata per un evento: se non specificata vale tutta la rosa. */
export function convocatiEvento(evento: Evento | null, rosa: Giocatore[]) {
- if (!evento || evento.convocati.length === 0) return rosa;
- return rosa.filter((g) => evento.convocati.includes(g.id));
+ return rosaConvocata(rosa, evento?.convocati ?? []);
+}
+
+/** Restringe una rosa ai convocati indicati: elenco vuoto = tutta la rosa. */
+export function rosaConvocata(rosa: T[], convocati: string[]): T[] {
+ if (convocati.length === 0) return rosa;
+ return rosa.filter((g) => convocati.includes(g.id));
}
diff --git a/src/routes/allenamento.$id.tsx b/src/routes/allenamento.$id.tsx
index 2a6e031..f58adc6 100644
--- a/src/routes/allenamento.$id.tsx
+++ b/src/routes/allenamento.$id.tsx
@@ -93,7 +93,12 @@ function AllenamentoDetail() {
);
diff --git a/src/routes/calendario.tsx b/src/routes/calendario.tsx
index 80b730b..31f046e 100644
--- a/src/routes/calendario.tsx
+++ b/src/routes/calendario.tsx
@@ -8,7 +8,8 @@ import { molla, proietta } from "@/lib/molla";
import { useMotoRidotto } from "@/lib/motion";
import { EventoCard, linkPerEvento } from "@/components/crapp/EventoCard";
import { Card, PageHeader, Section } from "@/components/crapp/ui-bits";
-import { compleanniEventi, useEventi, type Evento } from "@/lib/eventi";
+import { compleanniEventi, type Evento } from "@/lib/eventi";
+import { useEventiVisibili } from "@/lib/eventi-visibili";
import { useAnagraficaRosa } from "@/lib/rosa";
import { usePuoGestireEventi } from "@/lib/ruoli";
import { giorniDelMese, giorniIT, mesiIT, pad2, useMeseNav } from "@/lib/calendario";
@@ -56,7 +57,7 @@ function Calendario() {
const [giornoSelezionato, setGiornoSelezionato] = useState(null);
const [drawerAperto, setDrawerAperto] = useState(false);
const puoGestire = usePuoGestireEventi();
- const { eventi } = useEventi();
+ const { eventi } = useEventiVisibili();
const rosa = useAnagraficaRosa({ conAllenatori: true });
// `useMotoRidotto` copre anche i device deboli (RAM bassa), non solo
// `prefers-reduced-motion`: disattiva anche lo swipe orizzontale tra mesi.
diff --git a/src/routes/eventi.tsx b/src/routes/eventi.tsx
index 0bfd065..bc65a71 100644
--- a/src/routes/eventi.tsx
+++ b/src/routes/eventi.tsx
@@ -1,3 +1,4 @@
+import { useEventiVisibili } from "@/lib/eventi-visibili";
import { useEffect, useState } from "react";
import { createFileRoute } from "@tanstack/react-router";
import { CalendarPlus, ChevronLeft, ChevronRight, Loader2, Pencil, Trash2, X } from "lucide-react";
@@ -22,7 +23,6 @@ import {
daCategoria,
eventoVuoto,
useEliminaEvento,
- useEventi,
useSalvaEvento,
type CategoriaEvento,
type Evento,
@@ -65,7 +65,7 @@ function GestioneEventi() {
const io = useGiocatoreBase();
// Admin o allenatore (DD-034): la RLS di `eventi_app` (M21) dice lo stesso.
const puoGestire = usePuoGestireEventi();
- const { eventi, isPending, isError, error, refetch } = useEventi();
+ const { eventi, isPending, isError, error, refetch } = useEventiVisibili();
const { righe: squadra } = useGiocatoriSquadra();
const salva = useSalvaEvento();
const elimina = useEliminaEvento();
diff --git a/src/routes/evento.$id.tsx b/src/routes/evento.$id.tsx
index a3bf16a..9a5b4d3 100644
--- a/src/routes/evento.$id.tsx
+++ b/src/routes/evento.$id.tsx
@@ -88,7 +88,12 @@ function EventoDetail() {
);
diff --git a/src/routes/index.tsx b/src/routes/index.tsx
index 83cde7c..1838ff0 100644
--- a/src/routes/index.tsx
+++ b/src/routes/index.tsx
@@ -11,7 +11,8 @@ import { Barra } from "@/components/motion/Barra";
import { Numero } from "@/components/motion/Numero";
import { ObiettivoDrawer } from "@/components/crapp/ObiettivoDrawer";
import { microcopyObiettivo, progressoObiettivo } from "@/lib/obiettivi";
-import { useEventi, type Evento } from "@/lib/eventi";
+import { eConvocato, type Evento } from "@/lib/eventi";
+import { useEventiVisibili } from "@/lib/eventi-visibili";
import { useRispostePresenze } from "@/lib/presenze";
import { useIo, useObiettivi } from "@/lib/rosa";
import { isAllenatore, nomeCompleto } from "@/lib/giocatori-squadra";
@@ -47,14 +48,14 @@ function Index() {
const base = useGiocatoreBase();
const giocatore = useIo();
const allenatore = isAllenatore(base);
- const { eventi } = useEventi();
+ const { eventi } = useEventiVisibili();
const { presenze } = useRispostePresenze();
const oggi = new Date().toISOString().slice(0, 10);
const prossimi: Evento[] = eventi.filter((e) => e.data >= oggi).slice(0, 3);
const prossimo = prossimi[0] ?? null;
const daConfermare = prossimi
.slice(1)
- .filter((e) => !giocatore || !presenze[e.id]?.[giocatore.id]);
+ .filter((e) => !giocatore || (eConvocato(e, giocatore.id) && !presenze[e.id]?.[giocatore.id]));
const linkProssimo = prossimo ? linkPerEvento(prossimo) : null;
const { data: csi } = useCsi();
const noi = csi?.classifica.find((r) => isNostraSquadra(r.squadra));
diff --git a/src/routes/partita.$id.tsx b/src/routes/partita.$id.tsx
index b79972d..8f6c5e2 100644
--- a/src/routes/partita.$id.tsx
+++ b/src/routes/partita.$id.tsx
@@ -297,7 +297,12 @@ function PartitaDetail() {
) : null}
);
diff --git a/test/unit/eventi.test.ts b/test/unit/eventi.test.ts
index 46b12b2..20bad8d 100644
--- a/test/unit/eventi.test.ts
+++ b/test/unit/eventi.test.ts
@@ -7,6 +7,9 @@ import {
convocatiEvento,
daCategoria,
daRiga,
+ eConvocato,
+ eventoVisibile,
+ rosaConvocata,
eventoVuoto,
type RigaEvento,
} from "@/lib/eventi";
@@ -123,3 +126,49 @@ assert.match(bozza.id, /^e[a-z0-9]+$/, "id generato dal client");
assert.notEqual(bozza.id, "", "ogni bozza ha un id");
console.log("eventi: ok");
+
+// --- eventoVisibile: i non convocati non vedono l'evento, chi gestisce sì -----
+assert.equal(eventoVisibile({ convocati: ["g1"] }, "g1", false), true, "il convocato lo vede");
+assert.equal(eventoVisibile({ convocati: ["g1"] }, "g2", false), false, "il non convocato no");
+assert.equal(eventoVisibile({ convocati: ["g1"] }, null, false), false, "senza slot no");
+assert.equal(eventoVisibile({ convocati: ["g1"] }, "g2", true), true, "admin/allenatore sempre");
+assert.equal(eventoVisibile({ convocati: [] }, "g2", false), true, "vuoto = tutta la rosa");
+
+// --- eConvocato: elenco vuoto = tutti ----------------------------------------
+assert.equal(eConvocato({ convocati: ["g1"] }, "g1"), true, "convocato");
+assert.equal(eConvocato({ convocati: ["g1"] }, "g2"), false, "non convocato");
+assert.equal(eConvocato({ convocati: [] }, "g2"), true, "vuoto = tutti");
+
+// --- Copertura per numero di convocati: 1, 2, 3, tutti, nessuno, id sconosciuti -
+const squadra = ["g1", "g2", "g3", "g4"].map((id) => ({ id }));
+const ids = (l: Array<{ id: string }>) => l.map((g) => g.id);
+const casi: Array<{ nome: string; convocati: string[]; attesi: string[] }> = [
+ { nome: "nessuno indicato = tutta la rosa", convocati: [], attesi: ["g1", "g2", "g3", "g4"] },
+ { nome: "solo 1", convocati: ["g2"], attesi: ["g2"] },
+ { nome: "2", convocati: ["g1", "g4"], attesi: ["g1", "g4"] },
+ { nome: "3", convocati: ["g3", "g1", "g2"], attesi: ["g1", "g2", "g3"] },
+ { nome: "tutti indicati", convocati: ["g1", "g2", "g3", "g4"], attesi: ["g1", "g2", "g3", "g4"] },
+ { nome: "id sconosciuto ignorato", convocati: ["g2", "zz"], attesi: ["g2"] },
+ { nome: "solo ids sconosciuti: nessuno", convocati: ["zz"], attesi: [] },
+];
+for (const { nome, convocati, attesi } of casi) {
+ assert.deepEqual(ids(rosaConvocata(squadra, convocati)), attesi, `rosaConvocata: ${nome}`);
+ const conv = { convocati };
+ for (const { id } of squadra) {
+ const dentro = attesi.includes(id) || convocati.length === 0;
+ assert.equal(
+ eConvocato(conv, id),
+ convocati.length === 0 || convocati.includes(id),
+ `eConvocato ${id}: ${nome}`,
+ );
+ assert.equal(eventoVisibile(conv, id, false), dentro, `visibile a ${id}: ${nome}`);
+ assert.equal(
+ eventoVisibile(conv, id, true),
+ true,
+ `admin/allenatore vede sempre (${id}): ${nome}`,
+ );
+ }
+ assert.equal(eventoVisibile(conv, null, false), convocati.length === 0, `senza slot: ${nome}`);
+ assert.equal(eventoVisibile(conv, null, true), true, `admin senza slot: ${nome}`);
+}
+assert.equal(rosaConvocata([], ["g1"]).length, 0, "rosa vuota resta vuota");
diff --git a/test/unit/presenze.test.ts b/test/unit/presenze.test.ts
index 0ba147b..a6a1172 100644
--- a/test/unit/presenze.test.ts
+++ b/test/unit/presenze.test.ts
@@ -294,3 +294,35 @@ assert.equal(
);
console.log("presenze: ok");
+
+// --- Solleciti per numero di convocati: 1, 2, 3, tutti ------------------------
+{
+ const sq = (["g1", "g2", "g3", "g4"] as const).map((id) => ({
+ id,
+ attivo: true,
+ tipo: "giocatore" as const,
+ }));
+ const nessuna: Array<{ giocatore_id: string; stato: string }> = [];
+ assert.deepEqual(destinatariSollecito(sq, nessuna, ["g2"]), ["g2"], "solo 1 convocato");
+ assert.deepEqual(destinatariSollecito(sq, nessuna, ["g1", "g3"]), ["g1", "g3"], "2 convocati");
+ assert.deepEqual(
+ destinatariSollecito(sq, nessuna, ["g1", "g2", "g4"]),
+ ["g1", "g2", "g4"],
+ "3 convocati",
+ );
+ assert.deepEqual(
+ destinatariSollecito(sq, [{ giocatore_id: "g1", stato: "presente" }], ["g1", "g3"]),
+ ["g3"],
+ "il convocato che ha già risposto non va sollecitato",
+ );
+ assert.deepEqual(
+ destinatariSollecito(sq, [{ giocatore_id: "g4", stato: "presente" }], ["g1"]),
+ ["g1"],
+ "chi ha risposto ma non è convocato non cambia i destinatari",
+ );
+ assert.deepEqual(
+ destinatariSollecito(sq, nessuna, []),
+ ["g1", "g2", "g3", "g4"],
+ "nessuno = tutti",
+ );
+}