Produits Entr'ouvert » Chrono

Actuellement les rendez-vous déjà bookés sont filtrés ainsi :

        booked_events = (
            Event.objects.filter(
                agenda__in=agenda_ids,
                start_datetime__gte=used_min_datetime
                start_datetime__lte=used_max_datetime + meeting_duration_td,
            )

i.e. pour un intervalle visé [used_min_datetime, used_max_datetime] et une durée de rendez-vous meeting_duration_td on va considérer les rendez-vous commençant dans l'intervalle [used_min_datetime, used_max_datetime + meeting_duration_td], or cela ignore les rendez-vous commençant avant pour les mêmes agendas mais qui intersecte l'intervalle visé. Ces rendez-vous peuvent être d'une durée maximale max(mt.duration for mt in agenda.iter_meetingtypes()), c'est donc elle qui devrait être prise en compte pour agrandir l'intervalle vers sont début, par contre il ne devrait pas être nécessaire d'augmenter l'intervalle vers sa fin, car si event.start_datetime est supérieur à used_max_datetime on est certain qu'il n'interfère pas.

Je propose donc d'introduire un Agenda.get_max_meeting_duration() et de définir l'intervalle de recherche des évènements déjà booké (par rapport à leur start_datetime) à

[used_min_datetime - max_meeting_duration_td, used_max_datetime]

Au passage je met en cache Agenda.iter_meetingtypes() pour éviter de le requêter plusieurs fois.

Le test introduit retourne 6 rendez-vous disponibles au lieu de 3 sans ce patch. J'ai aussi corrigé une typo timestamp -> timestamp() au passage (je peux en faire un autre ticket).

PS: problème observé en voulant ajouter du filtrage dans l'appel get_all_slots() du endpoint fillslot pour en diminuer le coût car actuellement on liste tous les slots possibles alors qu'on ne devrait chercher que les intersections avec l'intervalle des rendez-vous qu'on essaye de prendre. En ajoutant start/end_datetime à cet endroit ça a tout de suite cassé des tests; ce changement sera éventuellement proposé dans un autre ticket.