From 0b1f303d62f98f832a602e44dcb1ffd16aa809a7 Mon Sep 17 00:00:00 2001
From: James Graham <j.graham@software.ac.uk>
Date: Mon, 15 Feb 2021 12:00:54 +0000
Subject: [PATCH] fix: populate themes with prev val on person form

Resolves #66
---
 people/forms.py         | 2 ++
 people/models/person.py | 1 -
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/people/forms.py b/people/forms.py
index 2adc9d6..afa2207 100644
--- a/people/forms.py
+++ b/people/forms.py
@@ -107,6 +107,8 @@ class PersonAnswerSetForm(forms.ModelForm, DynamicAnswerSetBase):
         self.instance.person_id = self.initial['person_id']
         if commit:
             self.instance.save()
+            # Need to call same_m2m manually since we use commit=False above
+            self.save_m2m()
 
         if commit:
             # Save answers to relationship questions
diff --git a/people/models/person.py b/people/models/person.py
index 408e73c..03ab88b 100644
--- a/people/models/person.py
+++ b/people/models/person.py
@@ -214,7 +214,6 @@ class PersonAnswerSet(AnswerSet):
             'replaced_timestamp',
             'person_id',
             'question_answers',
-            'themes',
         }
 
         def field_value_repr(field):