Bénévole : l'email n'est plus unique (couple, famille)
Plusieurs bénévoles peuvent légitimement partager une même adresse email — l'architecture l'interdisait : l'import ignorait silencieusement le second (« duplicate_email_in_file »), et l'ajout manuel renvoyait un 409. Les deux sont corrigés. - Migration 0009 : retrait de la contrainte d'unicité (edition_id, email) sur `volunteer` (l'index sur `email` seul est conservé). - Import (`app/importing/service.py`) : l'upsert se fait désormais sur (email, nom + prénom), pas sur l'email seul. Un même email revu avec un nom différent dans le fichier crée un second bénévole (nouveau code `import.row.shared_email`, WARNING) au lieu d'être ignoré ; `duplicate_email_in_file` ne se déclenche plus que sur une ligne littéralement répétée (même email ET même nom). - API (`POST`/`PATCH /volunteers`) : retrait du rejet 409 `volunteer.duplicate_email`, devenu sans objet (le mock ne l'a d'ailleurs jamais appliqué à la création manuelle). - Mock frontend (`mock-client.ts`) : même correction de l'upsert par import, pour rester cohérent avec le backend. - CLAUDE.md à jour (modèle de domaine, règles d'import, journal). Contrepartie assumée (documentée) : un email re-importé avec un nom légèrement différent (faute de frappe corrigée) crée désormais un bénévole de plus signalé, plutôt que de renommer silencieusement l'existant — même philosophie que la détection sans email, qui ne fusionne jamais rien automatiquement. Vérifié : 179 tests backend, ruff, alembic check ; typecheck/lint/build/smoke frontend ; reseed de démonstration. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017uUsRg7XzopkCPWimwg9ZM
This commit is contained in:
AdamThomas
co-authored by
Claude Sonnet 5
Claude Sonnet 5
parent
fcc42c8d43
commit
897a76ca6d
@@ -1,9 +1,11 @@
|
||||
"""CRUD des bénévoles. La suppression est un effacement complet (RGPD)."""
|
||||
"""CRUD des bénévoles. La suppression est un effacement complet (RGPD).
|
||||
|
||||
L'email n'est pas unique : plusieurs bénévoles peuvent en partager un même
|
||||
(couple, famille) — voir `app/models/domain.py:Volunteer`."""
|
||||
|
||||
import uuid
|
||||
|
||||
from fastapi import APIRouter, Depends, HTTPException, Response, status
|
||||
from sqlalchemy.exc import IntegrityError
|
||||
from sqlmodel import Session, select
|
||||
|
||||
from app.api._common import owned_or_404, replace_links
|
||||
@@ -143,13 +145,7 @@ def create_volunteer(
|
||||
**payload.model_dump(exclude={"artist_ids", "wished_position_ids"}),
|
||||
)
|
||||
session.add(volunteer)
|
||||
try:
|
||||
session.flush()
|
||||
except IntegrityError as exc:
|
||||
session.rollback()
|
||||
raise HTTPException(
|
||||
status_code=409, detail={"code": "volunteer.duplicate_email"}
|
||||
) from exc
|
||||
session.flush()
|
||||
_set_artists(session, volunteer, payload.artist_ids)
|
||||
_set_wished_positions(session, volunteer, payload.wished_position_ids)
|
||||
session.commit()
|
||||
@@ -197,13 +193,7 @@ def update_volunteer(
|
||||
wished_position_ids = changes.pop("wished_position_ids", None)
|
||||
for field, value in changes.items():
|
||||
setattr(volunteer, field, value)
|
||||
try:
|
||||
session.flush()
|
||||
except IntegrityError as exc:
|
||||
session.rollback()
|
||||
raise HTTPException(
|
||||
status_code=409, detail={"code": "volunteer.duplicate_email"}
|
||||
) from exc
|
||||
session.flush()
|
||||
if artist_ids is not None:
|
||||
_set_artists(session, volunteer, artist_ids)
|
||||
if wished_position_ids is not None:
|
||||
|
||||
@@ -10,7 +10,6 @@
|
||||
"day_part.duplicate_code": "Un créneau type porte déjà ce code dans l'édition.",
|
||||
"shift.ends_not_after_starts": "La fin du créneau doit être postérieure à son début.",
|
||||
"shift.position_wrong_edition": "Le poste n'appartient pas à cette édition.",
|
||||
"volunteer.duplicate_email": "Un bénévole avec cet email existe déjà dans l'édition.",
|
||||
"artist.name_required": "Le nom de l'artiste ne peut pas être vide.",
|
||||
"artist.duplicate": "Un artiste porte déjà ce nom à cette date dans l'édition.",
|
||||
"artist.unknown": "Un des artistes choisis n'appartient pas à cette édition.",
|
||||
@@ -33,7 +32,8 @@
|
||||
"import.mapping.column_not_found": "La colonne « {column} » est absente du fichier.",
|
||||
"import.row.missing_required_field": "Champ obligatoire manquant : {field}.",
|
||||
"import.row.invalid_email": "Adresse email invalide : {value}.",
|
||||
"import.row.duplicate_email_in_file": "Email déjà présent à la ligne {first_line} du fichier.",
|
||||
"import.row.duplicate_email_in_file": "Même bénévole (nom et email) déjà présent à la ligne {first_line} du fichier.",
|
||||
"import.row.shared_email": "Email déjà utilisé par un autre bénévole à la ligne {first_line} du fichier.",
|
||||
"import.row.duplicate_name_in_file": "Nom et prénom déjà présents à la ligne {first_line} du fichier.",
|
||||
"import.row.updated_existing": "Bénévole existant mis à jour.",
|
||||
"import.row.possible_duplicate_name": "Un bénévole portant le même nom existe déjà dans l'édition.",
|
||||
|
||||
@@ -2,6 +2,12 @@
|
||||
|
||||
L'import ne porte que l'identité des bénévoles (nom, email, téléphone,
|
||||
commentaire). Les disponibilités se déclarent dans l'application.
|
||||
|
||||
Détection des doublons : upsert sur (email, nom + prénom) — l'email seul ne
|
||||
suffit pas, plusieurs bénévoles peuvent le partager (couple, famille) ; sans
|
||||
email, détection sur (nom, prénom) signalée sans upsert automatique. Un même
|
||||
email revu avec un nom différent crée un bénévole de plus (signalé), jamais
|
||||
ignoré.
|
||||
"""
|
||||
|
||||
import re
|
||||
@@ -104,14 +110,20 @@ def run_import(
|
||||
select(Volunteer).where(Volunteer.edition_id == edition.id)
|
||||
)
|
||||
)
|
||||
by_email = {v.email.lower(): v for v in existing if v.email}
|
||||
# Plusieurs bénévoles peuvent partager un même email (couple, famille) :
|
||||
# l'email seul ne suffit pas à identifier un bénévole existant, il faut
|
||||
# l'associer au nom. `by_name` ne sert qu'aux bénévoles sans email.
|
||||
by_email_and_name: dict[tuple[str, tuple[str, str]], Volunteer] = {}
|
||||
by_name: dict[tuple[str, str], Volunteer] = {}
|
||||
for volunteer in existing:
|
||||
if not volunteer.email:
|
||||
key = (normalize(volunteer.last_name), normalize(volunteer.first_name))
|
||||
key = (normalize(volunteer.last_name), normalize(volunteer.first_name))
|
||||
if volunteer.email:
|
||||
by_email_and_name[(volunteer.email.lower(), key)] = volunteer
|
||||
else:
|
||||
by_name.setdefault(key, volunteer)
|
||||
|
||||
report = ImportReport(dry_run=mapping.dry_run, total_rows=len(table.rows))
|
||||
row_seen: dict[tuple[str, tuple[str, str]], int] = {}
|
||||
email_seen: dict[str, int] = {}
|
||||
name_seen: dict[tuple[str, str], int] = {}
|
||||
|
||||
@@ -151,34 +163,50 @@ def run_import(
|
||||
report.skipped += 1
|
||||
continue
|
||||
|
||||
name_key = (normalize(last_name), normalize(first_name))
|
||||
if email:
|
||||
if email in email_seen:
|
||||
row_key = (email, name_key)
|
||||
if row_key in row_seen:
|
||||
# Même email ET même nom déjà vus : la ligne est répétée dans
|
||||
# le fichier (copier-coller), pas deux bénévoles distincts.
|
||||
entry.issues.append(
|
||||
Issue(
|
||||
code="import.row.duplicate_email_in_file",
|
||||
severity="ERROR",
|
||||
field="email",
|
||||
params={"value": email_raw, "first_line": email_seen[email]},
|
||||
params={"value": email_raw, "first_line": row_seen[row_key]},
|
||||
)
|
||||
)
|
||||
report.rows.append(entry)
|
||||
report.skipped += 1
|
||||
continue
|
||||
email_seen[email] = line
|
||||
if email in email_seen:
|
||||
# Même email, nom différent : deux bénévoles peuvent partager
|
||||
# une adresse (couple, famille) — on les crée tous les deux,
|
||||
# simplement signalé pour la coordination.
|
||||
entry.issues.append(
|
||||
Issue(
|
||||
code="import.row.shared_email",
|
||||
severity="WARNING",
|
||||
field="email",
|
||||
params={"value": email_raw, "first_line": email_seen[email]},
|
||||
)
|
||||
)
|
||||
row_seen[row_key] = line
|
||||
email_seen.setdefault(email, line)
|
||||
else:
|
||||
key = (normalize(last_name), normalize(first_name))
|
||||
if key in name_seen:
|
||||
if name_key in name_seen:
|
||||
entry.issues.append(
|
||||
Issue(
|
||||
code="import.row.duplicate_name_in_file",
|
||||
severity="WARNING",
|
||||
params={"first_line": name_seen[key]},
|
||||
params={"first_line": name_seen[name_key]},
|
||||
)
|
||||
)
|
||||
else:
|
||||
name_seen[key] = line
|
||||
name_seen[name_key] = line
|
||||
|
||||
volunteer = by_email.get(email) if email else None
|
||||
volunteer = by_email_and_name.get((email, name_key)) if email else None
|
||||
if volunteer is not None:
|
||||
entry.action = "UPDATED"
|
||||
entry.issues.append(
|
||||
@@ -190,7 +218,7 @@ def run_import(
|
||||
)
|
||||
else:
|
||||
if not email:
|
||||
match = by_name.get((normalize(last_name), normalize(first_name)))
|
||||
match = by_name.get(name_key)
|
||||
if match is not None:
|
||||
entry.issues.append(
|
||||
Issue(
|
||||
|
||||
@@ -0,0 +1,32 @@
|
||||
"""Bénévole : l'email n'est plus unique par édition
|
||||
|
||||
Revision ID: 0009_volunteer_email_not_unique
|
||||
Revises: 0008_artist_performance_time
|
||||
Create Date: 2026-09-15
|
||||
|
||||
Plusieurs bénévoles peuvent partager une même adresse (couple, famille) :
|
||||
l'import les traitait auparavant comme des doublons et ignorait le second.
|
||||
L'unicité (edition_id, email) est retirée ; l'index sur `email` seul est
|
||||
conservé pour les recherches.
|
||||
"""
|
||||
|
||||
from collections.abc import Sequence
|
||||
|
||||
from alembic import op
|
||||
|
||||
revision: str = "0009_volunteer_email_not_unique"
|
||||
down_revision: str | None = "0008_artist_performance_time"
|
||||
branch_labels: str | Sequence[str] | None = None
|
||||
depends_on: str | Sequence[str] | None = None
|
||||
|
||||
|
||||
def upgrade() -> None:
|
||||
op.drop_constraint(
|
||||
"uq_volunteer_edition_email", "volunteer", type_="unique"
|
||||
)
|
||||
|
||||
|
||||
def downgrade() -> None:
|
||||
op.create_unique_constraint(
|
||||
"uq_volunteer_edition_email", "volunteer", ["edition_id", "email"]
|
||||
)
|
||||
@@ -133,18 +133,14 @@ class Artist(TimestampMixin, table=True):
|
||||
|
||||
|
||||
class Volunteer(TimestampMixin, table=True):
|
||||
__table_args__ = (
|
||||
UniqueConstraint(
|
||||
"edition_id", "email", name="uq_volunteer_edition_email"
|
||||
),
|
||||
)
|
||||
|
||||
id: uuid.UUID = Field(default_factory=uuid.uuid4, primary_key=True)
|
||||
edition_id: uuid.UUID = Field(
|
||||
foreign_key="edition.id", index=True, ondelete="CASCADE"
|
||||
)
|
||||
# Identifiant fonctionnel unique par édition. Nullable : bénévole créé à la
|
||||
# main par la coordination, sans email.
|
||||
# Pas unique : plusieurs bénévoles peuvent partager une même adresse (ex.
|
||||
# un couple, une famille). L'import (app/importing/service.py) distingue
|
||||
# deux bénévoles d'un même email par leur nom ; la coordination reste
|
||||
# seule juge en cas d'ambiguïté réelle.
|
||||
email: str | None = Field(default=None, index=True)
|
||||
first_name: str
|
||||
last_name: str
|
||||
|
||||
Reference in New Issue
Block a user