From c24f9265c22d68403b7734fe572a07388d7b1752 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Paris?= <postmaster@greg0ire.fr> Date: Sat, 3 Mar 2018 20:32:10 +0100 Subject: [PATCH] Recommendation on entity repository --- index.md | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/index.md b/index.md index 171005f..8f776da 100644 --- a/index.md +++ b/index.md @@ -241,8 +241,13 @@ final class DoctrineArticleRepository implements ArticleRepository ``` Note: -- `Repository`, c'est un pattern, et il vaut mieux définir les vôtres sous - forme d'interface. +- `Repository`, c'est un pattern, et c'est vous qui devriez en définir + l'interface. +- `EntityRepository` et ses méthodes magiques à éviter si vous voulez des type + hints de retour et donc de l'autocompletion. Il est maintenant possible d'en +faire des services: https://github.com/doctrine/DoctrineBundle/pull/727, mais +ça ne résout pas le problème. Si vous tenez à les utiliser, injectez les dans +vos repositories plutôt que de les étendre. - Plus l'interface est grosse, plus elle devient difficile à implémenter, et le code qui consomme l'API a rarement besoin de faire beaucoup d'appels, du coup… slide suivant -- GitLab