diff --git a/CHANGELOG.rst b/CHANGELOG.rst index 58be4f5ac47f1f0a92e5354440d357f3ec84f837..16113a46704c93e9c0bafb4e5fc40b50f16c5e0c 100644 --- a/CHANGELOG.rst +++ b/CHANGELOG.rst @@ -22,6 +22,11 @@ Added * Support for usage with new AlekSIS SPA. +Fixed +~~~~~ + +* In some cases, seating plan edit forms caused an error when trying to redirect after they were filled successfully. + `1.0.2`_ - 2022-11-04 --------------------- diff --git a/aleksis/apps/stoelindeling/views.py b/aleksis/apps/stoelindeling/views.py index 72ac1682e215a51fe909cb31b5ee51a6a0c2f1d1..f1367e921e53c17b452aa1e2e48b41fe95f15441 100644 --- a/aleksis/apps/stoelindeling/views.py +++ b/aleksis/apps/stoelindeling/views.py @@ -48,7 +48,7 @@ class SeatingPlanDetailView(PermissionRequiredMixin, DetailView): @method_decorator(never_cache, name="dispatch") -class SeatingPlanCreateView(PermissionRequiredMixin, SuccessNextMixin, AdvancedCreateView): +class SeatingPlanCreateView(PermissionRequiredMixin, AdvancedCreateView): """Create view for seating plans.""" model = SeatingPlan @@ -138,7 +138,7 @@ class SeatingPlanEditView(PermissionRequiredMixin, SuccessNextMixin, AdvancedEdi @method_decorator(never_cache, name="dispatch") -class SeatingPlanCopyView(PermissionRequiredMixin, SuccessNextMixin, AdvancedEditView): +class SeatingPlanCopyView(PermissionRequiredMixin, AdvancedEditView): """Copy view for seating plans.""" model = SeatingPlan @@ -147,7 +147,7 @@ class SeatingPlanCopyView(PermissionRequiredMixin, SuccessNextMixin, AdvancedEdi template_name = "stoelindeling/seating_plan/copy.html" def get_success_url(self): - return reverse("edit_seating_plan", args=[self.new_object.pk]) # FiXME NEXT URL + return reverse("edit_seating_plan", args=[self.new_object.pk]) def get_form_kwargs(self): kwargs = super().get_form_kwargs() @@ -176,7 +176,7 @@ class SeatingPlanCopyView(PermissionRequiredMixin, SuccessNextMixin, AdvancedEdi @method_decorator(never_cache, name="dispatch") class SeatingPlanDeleteView( - PermissionRequiredMixin, RevisionMixin, SuccessNextMixin, AdvancedDeleteView + PermissionRequiredMixin, RevisionMixin, AdvancedDeleteView ): """Delete view for seating plans."""