entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 5 Releases Wiki Activity

api: use a for loop instead of itertools.group_by (#46145)

This commit is contained in:
Emmanuel Cazenave 2020-08-27 16:46:13 +02:00
parent 0c61adf2ed
commit 38df08daf0
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
chrono/api/views.py
View File

@ -219,13 +219,10 @@ def get_all_slots(base_agenda, meeting_type, resources=None, unique=False):
# skip time period that can't even hold a single meeting
continue
desks_by_min_max_datetime = {
datetime_range: list(desks)
for datetime_range, desks in itertools.groupby(
time_period.desks,
key=lambda desk: desk_min_max_datetime.get(desk, agenda_id_min_max_datetime[desk.agenda_id]),
)
}
desks_by_min_max_datetime = collections.defaultdict(list)
for desk in time_period.desks:
min_max = desk_min_max_datetime.get(desk, agenda_id_min_max_datetime[desk.agenda_id])
desks_by_min_max_datetime[min_max].append(desk)
# aggregate agendas based on their real min/max_datetime :
# the get_time_slots() result is dependant upon these values, so even