reynir/bornhack-website
1
0
Fork 0
Code Issues Pull requests 1 Releases Wiki Activity

fix autoschedule apply success message when not doing similar schedule

Browse source
This commit is contained in:
Thomas Steen Rasmussen 2021-08-16 00:34:56 +02:00
parent 3a42b262f3
commit d3cdd5a838
1 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/backoffice/views/program.py
View file

@ -890,6 +890,7 @@ class AutoScheduleApplyView(CampViewMixin, ContentTeamPermissionMixin, FormView)
autoschedule, diff = scheduler.calculate_similar_autoschedule() autoschedule, diff = scheduler.calculate_similar_autoschedule()
elif form.cleaned_data["schedule"] == "new": elif form.cleaned_data["schedule"] == "new":
autoschedule = scheduler.calculate_autoschedule() autoschedule = scheduler.calculate_autoschedule()
diff = None
# check validity # check validity
valid, violations = scheduler.is_valid(autoschedule, return_violations=True) valid, violations = scheduler.is_valid(autoschedule, return_violations=True)
@ -898,8 +899,13 @@ class AutoScheduleApplyView(CampViewMixin, ContentTeamPermissionMixin, FormView)
deleted, created = scheduler.apply(autoschedule) deleted, created = scheduler.apply(autoschedule)
messages.success( messages.success(
self.request, self.request,
f"Schedule has been applied! {deleted} Events removed from schedule, {created} new Events scheduled. Differences to the previous schedule: {len(diff['event_diffs'])} Event diffs and {len(diff['slot_diffs'])} Slot diffs.", f"Schedule has been applied! {deleted} Events removed from schedule, {created} new Events scheduled.",
) )
if diff:
messages.success(
self.request,
"Differences to the previous schedule: {len(diff['event_diffs'])} Event diffs and {len(diff['slot_diffs'])} Slot diffs.",
)
else: else:
messages.error(self.request, "Schedule is NOT valid, cannot apply!") messages.error(self.request, "Schedule is NOT valid, cannot apply!")
return redirect( return redirect(