Skip to content

refactor: migration de l'entité Planning de Ting vers Doctrine (#2383) - #2413

Open
Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-ting-planning-vers
Open

Korbeil wants to merge 1 commit into
afup:masterfrom
Korbeil:afup-migration-ting-planning-vers

Conversation

@Korbeil

@Korbeil Korbeil commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Migre l'entité Planning (programmation des sessions d'un forum) de Ting vers Doctrine, en suivant le pattern de la PR #2387. Le back-office, les pages publiques (programme, planning ICS/JSON) et le notifier vidéo utilisent désormais l'entité et le repository Doctrine.

Changes

  • Le planning des sessions est stocké et manipulé via une entité Doctrine fortement typée (propriétés publiques) avec un repository Doctrine ; l'ancien modèle Ting et son SQL brut sont supprimés.
  • Les sessions sans planning ou sans salle sont désormais omises du calendrier (ICS/JSON) et de la vue admin au lieu de provoquer une erreur 500.
  • L'édition d'un planning inexistant et la consultation d'une fiche de talk sans planning affichent une 404 au lieu d'une erreur 500.
  • Un test d'intégration couvre le CRUD et la recherche des plannings hors keynotes.

How to test

  1. Dans le back-office, ouvrir l'agenda d'un forum, créer/modifier/déplacer puis supprimer une session : les plannings s'enregistrent comme avant.
  2. Consulter la page programme d'un événement et les fichiers planning.ics / planning.json : le contenu est inchangé.
  3. Lancer make test, make test-integration et make phpstan.

@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch 2 times, most recently from 58a208f to c9ec7f6 Compare September 21, 2026 20:48
* Les événements de l'AFUP se déroulant en France, ils sont saisis et affichés
* dans cette timezone, quelle que soit celle du serveur ou du navigateur.
*/
public const string TIMEZONE = 'Europe/Paris';

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On devrait pas faire ça via un setting global dans Doctrine plutôt que ici individuellement ?

@Korbeil
Korbeil force-pushed the afup-migration-ting-planning-vers branch from c9ec7f6 to 6458a69 Compare September 21, 2026 21:09
@Korbeil
Korbeil marked this pull request as ready for review September 21, 2026 21: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