From c7a6c722197180ab318f6ef3df74e15fd3fd8a77 Mon Sep 17 00:00:00 2001
From: James Graham <J.Graham@software.ac.uk>
Date: Thu, 2 Apr 2020 17:15:27 +0100
Subject: [PATCH] fix: Fix broken activity signup

---
 activities/templates/activities/activity/detail.html | 2 ++
 activities/views.py                                  | 7 +++----
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/activities/templates/activities/activity/detail.html b/activities/templates/activities/activity/detail.html
index 5e1901d..40a3ceb 100644
--- a/activities/templates/activities/activity/detail.html
+++ b/activities/templates/activities/activity/detail.html
@@ -39,6 +39,8 @@
 
     <hr>
 
+    <h2>Attendance</h2>
+
     <table class="table table-borderless">
         <thead>
         <tr>
diff --git a/activities/views.py b/activities/views.py
index ec9b0e6..a2af88b 100644
--- a/activities/views.py
+++ b/activities/views.py
@@ -74,14 +74,13 @@ class ActivityAttendanceView(permissions.UserIsLinkedPersonMixin, SingleObjectMi
     def get_test_person(self) -> people_models.Person:
         data = json.loads(self.request.body)
 
-        self.person = people_models.Person.objects.get(pk=data['pk'])
-        return self.person
+        return people_models.Person.objects.get(pk=data['pk'])
 
     def post(self, request, *args, **kwargs):
         self.object = self.get_object()
 
         if request.is_ajax():
-            self.object.attendance_list.add(self.person)
+            self.object.attendance_list.add(self.get_test_person())
 
             return HttpResponse(status=204)
 
@@ -91,7 +90,7 @@ class ActivityAttendanceView(permissions.UserIsLinkedPersonMixin, SingleObjectMi
         self.object = self.get_object()
 
         if request.is_ajax():
-            self.object.attendance_list.remove(self.person)
+            self.object.attendance_list.remove(self.get_test_person())
 
             return HttpResponse(status=204)