From fb1d28d26e09b64eb52a822936b467b4a94b8ab5 Mon Sep 17 00:00:00 2001 From: Eemeli Ranta Date: Thu, 19 Sep 2024 14:48:41 +0300 Subject: [PATCH] Django Admin: Fix removing empty timeslots --- reservation_units/admin/reservation_unit/form.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/reservation_units/admin/reservation_unit/form.py b/reservation_units/admin/reservation_unit/form.py index 8d7ceb2de..9839ecae6 100644 --- a/reservation_units/admin/reservation_unit/form.py +++ b/reservation_units/admin/reservation_unit/form.py @@ -11,10 +11,10 @@ def remove_empty_timeslots(timeslots: list[dict[str, str]]) -> None: - # Iterate in reverse order so that items can be deleted without affecting the loop - for i, timeslot in enumerate(reversed(timeslots)): - if timeslot == {"begin": "", "end": ""}: - del timeslots[i] + empty_timeslot = {"begin": "", "end": ""} + + while empty_timeslot in timeslots: + timeslots.remove(empty_timeslot) class TimeslotForm(forms.Form):