Re-save user profile after setting name

This means we get Person objects created with the correct name, rather
than the username at time of creation.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>

diff --git a/apps/patchwork/forms.py b/apps/patchwork/forms.py
index c02d11ef98a7e09d6aca3a83f160f0fa70eba3be..bc746bd11c580deb3031dbaa1925bf36920d29de 100644 (file)
--- a/apps/patchwork/forms.py
+++ b/apps/patchwork/forms.py
@@ -44,6 +44,11 @@ class RegistrationForm(RegistrationFormUniqueEmail):
         user.first_name = self.cleaned_data.get('first_name', '')
         user.last_name = self.cleaned_data.get('last_name', '')
         user.save()
+
+       # saving the userprofile causes the firstname/lastname to propagate
+       # to the person objects.
+       user.get_profile().save()
+
         return user
 
     def clean(self):

diff --git a/apps/patchwork/models.py b/apps/patchwork/models.py
index 11457d76bdf9a0e14524b425dbc483b41d8f083c..fb2ccc7a298aaba21496dcf644dc916b4eacb81e 100644 (file)
--- a/apps/patchwork/models.py
+++ b/apps/patchwork/models.py
@@ -122,7 +122,7 @@ class UserProfile(models.Model):
             person.save()
         else:
             for person in people:
-                 person.user = self.user
+                 person.link_to_user(self.user)
                  person.save()
 
     def __str__(self):