Skip to content

Commit 3686d7a

Browse files
committed
Fixing bug where first day of the week had to be part of recurrence
1 parent 8233ee5 commit 3686d7a

File tree

1 file changed

+8
-1
lines changed

1 file changed

+8
-1
lines changed

featuremanagement/_time_window_filter/_recurrence_validator.py

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,4 +154,11 @@ def _get_passed_week_days(current_day: int, first_day_of_week: int) -> int:
154154

155155
def _sort_days_of_week(days_of_week: List[int], first_day_of_week: int) -> List[int]:
156156
sorted_days = sorted(days_of_week)
157-
return sorted_days[sorted_days.index(first_day_of_week) :] + sorted_days[: sorted_days.index(first_day_of_week)]
157+
if first_day_of_week in sorted_days:
158+
return sorted_days[sorted_days.index(first_day_of_week) :] + sorted_days[: sorted_days.index(first_day_of_week)]
159+
next_closet_day = first_day_of_week
160+
for i in range(len(sorted_days)):
161+
if sorted_days[i] > first_day_of_week:
162+
next_closet_day = sorted_days[i]
163+
break
164+
return sorted_days[sorted_days.index(next_closet_day) :] + sorted_days[: sorted_days.index(next_closet_day)]

0 commit comments

Comments
 (0)