Produits Entr'ouvert » Chrono

À traiter ici aussi, les exceptions jours fériés qui consomment également trop de requêtes (non couvert par les tests pour le moment), il faudrait un simple select_related('source') sur le qs des exceptions comme c'est déjà fait dans TimePeriodExceptionListView.

Sur l'essentiel je suis jaloux de pas avoir trouvé ça tout seul.

Je me dis peut-être avoir une implémentation par défaut de prefetched_exceptions, genre

@property
def prefetched_exceptions(self):
    if getattr(self, prefetched_exceptions_data, None) is None:
        self.prefetched_exception_data = TimePeriodException.objects.filter(Q(desk=desk) | Q(unavailability_calendar__desks=desk))
    return prefetched_exceptions_data

Avec du coup un desk.prefetched_exceptions_data = ... dans la vue, pour s'éviter les contorsions dans le test et pour filet de sécurité ?

J'aime bien l'idée que ça pète si prefetched_exceptions n'est pas défini, ça évite l'utilisation irréfléchie de la méthode (ie sans avoir anticipé le prefetch), et les pb de perf :)