Commit 4cf48b086b1098e6f79583fc3c3f990e2d8b67ea
1 parent
37932331
Exists in
master
and in
5 other branches
Fixed redirect bug in update category, user now gets redirected to the list of c…
…ategory, as happens in delete and create category
Showing
1 changed file
with
3 additions
and
2 deletions
Show diff stats
courses/views.py
| @@ -488,11 +488,12 @@ class UpdateCatView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): | @@ -488,11 +488,12 @@ class UpdateCatView(LoginRequiredMixin, HasRoleMixin, generic.UpdateView): | ||
| 488 | template_name = 'category/update.html' | 488 | template_name = 'category/update.html' |
| 489 | model = CourseCategory | 489 | model = CourseCategory |
| 490 | form_class = CategoryCourseForm | 490 | form_class = CategoryCourseForm |
| 491 | + success_url = reverse_lazy('course:manage_cat') | ||
| 491 | 492 | ||
| 492 | def get_success_url(self): | 493 | def get_success_url(self): |
| 493 | messages.success(self.request, _('Category updated successfully!')) | 494 | messages.success(self.request, _('Category updated successfully!')) |
| 494 | - return reverse_lazy('course:update_cat', kwargs={'slug' : self.object.slug}) | ||
| 495 | - | 495 | + #return reverse_lazy('course:update_cat', kwargs={'slug' : self.object.slug}) |
| 496 | + return reverse_lazy('course:manage_cat') | ||
| 496 | class DeleteCatView(LoginRequiredMixin, HasRoleMixin, generic.DeleteView): | 497 | class DeleteCatView(LoginRequiredMixin, HasRoleMixin, generic.DeleteView): |
| 497 | 498 | ||
| 498 | allowed_roles = ['professor', 'system_admin'] | 499 | allowed_roles = ['professor', 'system_admin'] |