Skip to content

Migre les entités Badge de Ting vers Doctrine - #2404

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-entite-ting-badge
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-entite-ting-badge

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Migre les badges des membres (tables afup_badge et afup_personnes_physiques_badge) de Ting vers Doctrine ORM, dans la continuité du chantier de migration #2383 et du pattern établi par la PR #2387. Le comportement fonctionnel est inchangé et aucune migration de schéma n'est nécessaire.

Changes

  • Les badges attribués aux membres sont désormais gérés par Doctrine : création de badges et attribution/suppression dans le back-office, affichage des badges sur les pages membres et service des images restent identiques.
  • L'association membre ↔ badge est une vraie relation Doctrine : le badge lié est hydraté en une seule requête lors du listing des badges d'un membre.
  • L'accès aux utilisateurs d'un badge dans le back-office est adapté à la nouvelle entité (passage par l'identifiant).
  • Nouveau test d'intégration couvrant le listing des badges d'un membre (hydratation, tri par date) et le CRUD via l'EntityManager.

How to test

  1. Back-office > Membres > Badges : créer un badge, l'attribuer à une personne physique depuis sa fiche, puis vérifier qu'il apparaît dans la liste des badges avec l'utilisateur associé et sur la page membre.
  2. Supprimer l'attribution depuis la fiche du membre et vérifier qu'elle disparaît de la liste.
  3. make test-integration-ci (inclut les nouveaux tests) puis make phpstan.

@Korbeil
Korbeil force-pushed the afup-migration-entite-ting-badge branch from 5f9fb14 to 2e9aa3a Compare September 21, 2026 19:15
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant