From d3cdd5a83890f8290085de8648ded0ebe12e4152 Mon Sep 17 00:00:00 2001 From: Thomas Steen Rasmussen Date: Mon, 16 Aug 2021 00:34:56 +0200 Subject: [PATCH] fix autoschedule apply success message when not doing similar schedule --- src/backoffice/views/program.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/backoffice/views/program.py b/src/backoffice/views/program.py index edded8c0..5387ad69 100644 --- a/src/backoffice/views/program.py +++ b/src/backoffice/views/program.py @@ -890,6 +890,7 @@ class AutoScheduleApplyView(CampViewMixin, ContentTeamPermissionMixin, FormView) autoschedule, diff = scheduler.calculate_similar_autoschedule() elif form.cleaned_data["schedule"] == "new": autoschedule = scheduler.calculate_autoschedule() + diff = None # check validity valid, violations = scheduler.is_valid(autoschedule, return_violations=True) @@ -898,8 +899,13 @@ class AutoScheduleApplyView(CampViewMixin, ContentTeamPermissionMixin, FormView) deleted, created = scheduler.apply(autoschedule) messages.success( 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: messages.error(self.request, "Schedule is NOT valid, cannot apply!") return redirect(