Skip to content

Conversation

@Mopolo
Copy link
Contributor

@Mopolo Mopolo commented Nov 30, 2025

Cette PR met en place doctrine/orm avec pour objectif de migrer la majorité des accès à la base de données (sauf cas exceptionnels, besoin de perf ou autre qui utilisent/utiliseront dbal directement).

L'ORM permet de gérer 80% des cas qui sont du simple CRUD qui est très bien intégré à Symfony, tout en étant plus facile d'accès à la contribution.

Quelques partis pris :

  • les entités n'ont pas de getter/setter, plus besoin grâce aux propriétés typées
  • les DQL est interdit, il faut utiliser le query builder
  • les méthodes de base des repositories sont interdites en dehors des repositories (pour éviter par exemple l'utilisation du query builder dans un controller).

Reste à faire

  • règle PHPStan pour bloquer le DQL
  • règle PHPStan pour limiter les méthodes d'un repository
  • découper la PR en plusieurs pour faciliter la review

@Mopolo Mopolo self-assigned this Nov 30, 2025
@Mopolo Mopolo force-pushed the doctrine-orm branch 7 times, most recently from 51af96d to ca1dff3 Compare December 5, 2025 12:26
@Mopolo Mopolo force-pushed the doctrine-orm branch 9 times, most recently from 4e65122 to 47b00ce Compare December 15, 2025 11:42
@Mopolo Mopolo changed the title [WIP] Poc doctrine/orm Setup de doctrine/orm Dec 15, 2025
@Mopolo Mopolo marked this pull request as ready for review December 15, 2025 11:53
@Mopolo Mopolo force-pushed the doctrine-orm branch 3 times, most recently from 0eeccb6 to f5a4955 Compare December 17, 2025 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants