summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJacob Kaplan-Moss <jacob@jacobian.org>2009-03-30 22:38:19 +0000
committerJacob Kaplan-Moss <jacob@jacobian.org>2009-03-30 22:38:19 +0000
commit02e56530c561e624bd2bee94a5d2ac0fabdfaa54 (patch)
treec8373aad8dbb6fb3d31bffbe62f924cdbe7ac191
parent657eefbf2551bc322c5d5cd8ea64ebe245964aed (diff)
Fixed #10157: correctly set the related objects pk when assigning a reverse OneToOne. Thanks, dgouldin.
git-svn-id: http://code.djangoproject.com/svn/django/trunk@10224 bcc190cf-cafb-0310-a4f2-bffc1f526a37
-rw-r--r--django/db/models/fields/related.py4
-rw-r--r--tests/modeltests/one_to_one/models.py2
2 files changed, 4 insertions, 2 deletions
diff --git a/django/db/models/fields/related.py b/django/db/models/fields/related.py
index f986eea75b..df0f6ef538 100644
--- a/django/db/models/fields/related.py
+++ b/django/db/models/fields/related.py
@@ -210,8 +210,8 @@ class SingleRelatedObjectDescriptor(object):
(value, instance._meta.object_name,
self.related.get_accessor_name(), self.related.opts.object_name))
- # Set the value of the related field
- setattr(value, self.related.field.rel.get_related_field().attname, instance)
+ # Set the value of the related field to the value of the related object's related field
+ setattr(value, self.related.field.attname, getattr(instance, self.related.field.rel.get_related_field().attname))
# Since we already know what the related object is, seed the related
# object caches now, too. This avoids another db hit if you get the
diff --git a/tests/modeltests/one_to_one/models.py b/tests/modeltests/one_to_one/models.py
index 20ac7a5573..e25e33bcc0 100644
--- a/tests/modeltests/one_to_one/models.py
+++ b/tests/modeltests/one_to_one/models.py
@@ -79,6 +79,8 @@ DoesNotExist: Restaurant matching query does not exist.
<Restaurant: Ace Hardware the restaurant>
>>> r.place
<Place: Ace Hardware the place>
+>>> p2.id
+2
# Set the place back again, using assignment in the reverse direction.
>>> p1.restaurant = r