Skip to content

Migre l'entité Ting TicketType vers Doctrine - #2411

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrate-ting-tickettype-to
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrate-ting-tickettype-to

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Migre l'entité des types de billets (« tarifs » des forums) de Ting vers Doctrine, dans la continuité de la migration globale suivie dans #2383 et selon le pattern des migrations précédentes. Aucun changement de schéma ni de comportement : les données restent servies depuis la même table, et les types de billets introuvables lèvent désormais une erreur explicite au lieu d'un échec ambigu.

Changements

  • Les types de billets sont désormais lus via Doctrine : liste ordonnée par identifiant pour les listes du back-office, lecture par identifiant pour les affichages.
  • Les statistiques d'inscriptions et les billetteries privées du back-office affichent les libellés de types de billets comme auparavant.
  • Un type de billet introuvable n'interrompt plus l'affichage des statistiques (dégradation propre du libellé).
  • Un test d'intégration couvre la lecture des types de billets : tri, hydratation complète des colonnes et cas introuvable.

Comment tester

  1. make test-integration-ci — la suite d'intégration passe, dont les 4 nouveaux tests.
  2. make phpstan — analyse statique sans erreur.
  3. make test — suite unitaire verte.

@Korbeil
Korbeil force-pushed the afup-migrate-ting-tickettype-to branch from efb6c85 to 2fe3d0b Compare September 21, 2026 19:32
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 20:00
@Korbeil
Korbeil force-pushed the afup-migrate-ting-tickettype-to branch from 2fe3d0b to 4a1342e Compare September 21, 2026 20:31
@Korbeil
Korbeil force-pushed the afup-migrate-ting-tickettype-to branch from 4a1342e to 6d01302 Compare September 21, 2026 21:24
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