Skip to content

Migre l'entité CompanyMemberInvitation de Ting vers Doctrine - #2405

Open
Korbeil wants to merge 2 commits into
afup:masterfrom
Korbeil:afup-migrer-companymemberinvitation-vers-doctrine
Open

Korbeil wants to merge 2 commits into
afup:masterfrom
Korbeil:afup-migrer-companymemberinvitation-vers-doctrine

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Migre la persistance des invitations des personnes morales de Ting vers Doctrine ORM (ref #2383), en suivant le pattern des précédentes migrations : nouvelle entité et nouveau repository Doctrine, migration des 5 usages, suppression des classes Ting. Aucune migration de base de données nécessaire : la table existante est conservée telle quelle.

Changes

  • Nouvelle entité Doctrine CompanyMemberInvitation mappée sur afup_personnes_morales_invitations et son repository (requêtes via QueryBuilder, CRUD via l'EntityManager) — les invitations en attente sont désormais triées par date de soumission.
  • La recherche d'invitation par email passe par le repository (filtrée sur la société et le statut « en attente ») au lieu du filtre en mémoire.
  • Les 5 appelants basculés sur la nouvelle entité : adhésion entreprise, gestion des membres de l'entreprise, acceptation d'invitation, envoi du mail d'invitation et formulaires associés.
  • Nouveau test d'intégration (5 tests) couvrant le filtrage, le tri, la recherche par email et le cycle enregistrement/mise à jour.

How to test

  1. make test-integration-ci — lance la suite d'intégration (dont les 5 nouveaux tests).
  2. make phpstan — analyse statique niveau 10, 0 erreur.
  3. make test — suite unitaire (124 tests) + php-cs-fixer.

@Korbeil
Korbeil force-pushed the afup-migrer-companymemberinvitation-vers-doctrine branch from 7105d79 to 8715011 Compare September 21, 2026 19:27
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 20:25
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