diff --git a/training/templates/trainee_detail.html b/training/templates/trainee_detail.html
index 968773f2..cdb61d94 100644
--- a/training/templates/trainee_detail.html
+++ b/training/templates/trainee_detail.html
@@ -71,7 +71,7 @@
{% if forloop.last %}{%endif%}
{% empty %}
- No qualifications in any levels yet...did someone forget to fill out the paperwork?
+
No qualifications in any levels yet...did someone forget to fill out the paperwork?
{% endfor %}
{% for level in started_levels %}
diff --git a/training/views.py b/training/views.py
index 641c554f..62262705 100644
--- a/training/views.py
+++ b/training/views.py
@@ -22,7 +22,7 @@ class ItemList(generic.ListView):
return context
-class TraineeDetail(views.ProfileDetail, ModalURLMixin):
+class TraineeDetail(views.ProfileDetail):
template_name = "trainee_detail.html"
model = models.Trainee
@@ -41,9 +41,6 @@ class TraineeDetail(views.ProfileDetail, ModalURLMixin):
context[str(i)] = self.object.get_records_of_depth(i)
return context
- def get_success_url(self):
- return self.get_close_url('trainee_detail', 'trainee_detail')
-
class TraineeItemDetail(generic.ListView):
model = models.TrainingItemQualification
@@ -90,7 +87,7 @@ class SessionLog(generic.FormView):
return context
-class AddQualification(generic.CreateView):
+class AddQualification(generic.CreateView, ModalURLMixin):
template_name = "edit_training_record.html"
model = models.TrainingItemQualification
form_class = forms.QualificationForm
@@ -106,7 +103,7 @@ class AddQualification(generic.CreateView):
return context
def get_success_url(self):
- return reverse_lazy('trainee_detail', kwargs={"pk": self.object.pk})
+ return self.get_close_url('trainee_detail', 'trainee_detail')
def get_form_kwargs(self):
kwargs = super(AddQualification, self).get_form_kwargs()
@@ -114,7 +111,7 @@ class AddQualification(generic.CreateView):
return kwargs
-class AddLevelRequirement(generic.CreateView):
+class AddLevelRequirement(generic.CreateView, ModalURLMixin):
template_name = "edit_training_level.html"
model = models.TrainingLevelRequirement
form_class = forms.RequirementForm
@@ -130,7 +127,7 @@ class AddLevelRequirement(generic.CreateView):
return kwargs
def get_success_url(self):
- return reverse_lazy('level_detail', kwargs={"pk": self.kwargs['pk']})
+ return self.get_close_url('level_detail', 'level_detail')
@transaction.atomic()
@reversion.create_revision()