Compare commits

...

1 commit

Author SHA1 Message Date
Basile Clement
f59e868793 Ajoute get_queryset dans ModelSearch
Quand on utilise les vues d'auto-complétion, on a la possibilité de
configurer des filtres avec `get_queryset_filter`, mais pas d'autres
transformations sur le queryset.  Notamment, cela signifie que pour les
vues qui utilisent des `ForeignKey`, on ne peut pas spécifier de
`select_related` pour éviter des requêtes inutiles et répétées lors du
rendering des vues.

Ce patch ajoute une méthode `get_queryset`, en suivant la convention
habituelle de Django, permettant cette configuration, et l'utilise pour
appeler `select_related` de façon appropriée dans les vues qui en
bénéficient.

À priori, cela devrait permettre de compenser la potentielle perte de
performance de !469, qui a supprimé les `select_related` lors de la
sélection des participants.
2020-10-23 22:37:03 +02:00
3 changed files with 12 additions and 2 deletions

View file

@ -905,6 +905,9 @@ class ParticipantAutocomplete(Select2QuerySetView):
model = Participant model = Participant
search_fields = ("user__username", "user__first_name", "user__last_name") search_fields = ("user__username", "user__first_name", "user__last_name")
def get_queryset(self):
return super().get_queryset().select_related("user", "tirage")
participant_autocomplete = buro_required(ParticipantAutocomplete.as_view()) participant_autocomplete = buro_required(ParticipantAutocomplete.as_view())

View file

@ -18,6 +18,9 @@ class KfetAccountSearch(autocomplete.ModelSearch):
] ]
verbose_name = _("Comptes existants") verbose_name = _("Comptes existants")
def get_queryset(self):
return super().get_queryset().select_related("profile__account_kfet")
def get_queryset_filter(self, *args, **kwargs): def get_queryset_filter(self, *args, **kwargs):
qset_filter = super().get_queryset_filter(*args, **kwargs) qset_filter = super().get_queryset_filter(*args, **kwargs)
qset_filter &= Q(profile__account_kfet__isnull=False) qset_filter &= Q(profile__account_kfet__isnull=False)

View file

@ -98,7 +98,11 @@ class ModelSearch(SearchUnit):
def __init__(self): def __init__(self):
if self.verbose_name is None: if self.verbose_name is None:
self.verbose_name = "{} search".format(self.model._meta.verbose_name) model = self.get_queryset().model
self.verbose_name = "{} search".format(model._meta.verbose_name)
def get_queryset(self):
return self.model.objects.all()
def get_queryset_filter(self, keywords): def get_queryset_filter(self, keywords):
filter_q = Q() filter_q = Q()
@ -122,7 +126,7 @@ class ModelSearch(SearchUnit):
of the search fields. of the search fields.
""" """
return self.model.objects.filter(self.get_queryset_filter(keywords)) return self.get_queryset().filter(self.get_queryset_filter(keywords))
# --- # ---