Skip to content

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

Merged
Korbeil merged 2 commits into
afup:masterfrom
Korbeil:afup-migration-entite-ting-badge
Oct 13, 2026
Merged

Korbeil merged 2 commits 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
@Korbeil
Korbeil force-pushed the afup-migration-entite-ting-badge branch 2 times, most recently from b18128c to a2d4f9d Compare October 6, 2026 08:34
Comment thread sources/AppBundle/Association/Model/Repository/UserRepository.php Outdated
Comment thread sources/AppBundle/Controller/Admin/Members/UserBadgeDeleteAction.php Outdated
Comment thread sources/AppBundle/Controller/Admin/Members/UserBadgeDeleteAction.php Outdated
Comment thread sources/AppBundle/Controller/Admin/Members/UserBadgeDeleteAction.php Outdated
@Korbeil
Korbeil marked this pull request as draft October 11, 2026 19:45
@Korbeil
Korbeil force-pushed the afup-migration-entite-ting-badge branch from 1b34b3a to 4e7bfff Compare October 12, 2026 08:37
@Korbeil
Korbeil requested a review from stakovicz October 12, 2026 08:38
@Korbeil
Korbeil marked this pull request as ready for review October 12, 2026 09:10
Comment thread sources/AppBundle/Event/Entity/UserBadge.php Outdated
Comment thread sources/AppBundle/Controller/Website/Badge/ImageAction.php
@Korbeil
Korbeil marked this pull request as draft October 12, 2026 18:55
Le referer peut être null; RedirectResponse attend une string. On retombe
sur l'URI courante (pattern déjà utilisé dans PrepareAction). Cela corrige
l'erreur PHPStan argument.type et retire les entrées de baseline devenues
inutiles pour UserBadgeNewAction et UserBadgeDeleteAction.
@Korbeil
Korbeil force-pushed the afup-migration-entite-ting-badge branch from 4e7bfff to 691e770 Compare October 12, 2026 19:23
@Korbeil
Korbeil requested a review from vgreb October 12, 2026 19:23
@Korbeil
Korbeil marked this pull request as ready for review October 12, 2026 19:45
@Korbeil
Korbeil merged commit c748c5c into afup:master Oct 13, 2026
7 checks passed
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.

3 participants