Skip to content

Migre l'entité Ting EventTheme vers Doctrine - #2408

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrer-eventtheme-vers-doctrine
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migrer-eventtheme-vers-doctrine

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Déplace le thème d'évènement de Ting vers Doctrine ORM, dans la migration globale Ting vers Doctrine (#2383), en répliquant le pattern de la PR #2387. L'entité est fortement typée (ADR-002) et la requête SQL brut du repository est remplacée par un findBy. Aucun impact utilisateur visible : le comportement des pages back-office de thèmes et de la page programme publique est inchangé.

Changes

  • Le thème d'évènement est désormais géré par Doctrine : ajout, modification, tri et suppression des thèmes fonctionnent à l'identique dans l'admin.
  • La résolution du thème via l'URL d'édition passe par le resolver Doctrine natif de Symfony (plus de dépendance à l'attribut Ting sur ce contrôleur).
  • Un test d'intégration couvre la sauvegarde, le tri par priorité puis nom, le filtrage par évènement et la suppression.

How to test

  1. make phpstan puis make test-integration-ci (le nouveau test fait partie de la suite).
  2. make test (suite unitaire + php-cs-fixer).

@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 19:30
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