summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorStephen Finucane <stephen@that.guru>2018-11-03 21:47:51 +0000
committerStephen Finucane <stephen@that.guru>2018-12-22 17:56:11 +0000
commitb1c2e80187008aff472c6509c67f44583cd3334c (patch)
tree54e8cde6d44f37206a62ca0487c737a288487ee1
parent08d1459a4a400083c92f8d7cf7738899b560bdf0 (diff)
downloadpatchwork-b1c2e80187008aff472c6509c67f44583cd3334c.tar
patchwork-b1c2e80187008aff472c6509c67f44583cd3334c.tar.gz
models: Ensure UserProfile.user is configured
Django Admin seems to be doing something funky with how it's handling the creation of a User's corresponding UserProfile instance when modelled as an inline field. Re-setting the UserProfile.user attribute seems to resolve the issue, so do just that. Signed-off-by: Stephen Finucane <stephen@that.guru> Closes: #110
-rw-r--r--patchwork/models.py1
-rw-r--r--releasenotes/notes/issue-110-a5bb3184bf831280.yaml5
2 files changed, 6 insertions, 0 deletions
diff --git a/patchwork/models.py b/patchwork/models.py
index a483dc6..a7eee4d 100644
--- a/patchwork/models.py
+++ b/patchwork/models.py
@@ -194,6 +194,7 @@ class UserProfile(models.Model):
def _user_saved_callback(sender, created, instance, **kwargs):
try:
profile = instance.profile
+ profile.user = instance
except UserProfile.DoesNotExist:
profile = UserProfile(user=instance)
profile.save()
diff --git a/releasenotes/notes/issue-110-a5bb3184bf831280.yaml b/releasenotes/notes/issue-110-a5bb3184bf831280.yaml
new file mode 100644
index 0000000..16a0fa5
--- /dev/null
+++ b/releasenotes/notes/issue-110-a5bb3184bf831280.yaml
@@ -0,0 +1,5 @@
+---
+fixes:
+ - |
+ Assigning maintained projects when creating a new user in the admin page
+ was causing an error. This is now resolved.