test0908
/
django
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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
This commit is contained in:
Jacob Kaplan-Moss 2009-03-30 22:38:19 +00:00
parent 657eefbf25
commit 02e56530c5
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
django/db/models/fields/related.py
View File

@ -210,8 +210,8 @@ class SingleRelatedObjectDescriptor(object):
(value, instance._meta.object_name, (value, instance._meta.object_name,
self.related.get_accessor_name(), self.related.opts.object_name)) self.related.get_accessor_name(), self.related.opts.object_name))
# Set the value of the related field # Set the value of the related field to the value of the related object's related field
setattr(value, self.related.field.rel.get_related_field().attname, instance) 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 # 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 # object caches now, too. This avoids another db hit if you get the

2
tests/modeltests/one_to_one/models.py
View File

@ -79,6 +79,8 @@ DoesNotExist: Restaurant matching query does not exist.
<Restaurant: Ace Hardware the restaurant> <Restaurant: Ace Hardware the restaurant>
>>> r.place >>> r.place
<Place: Ace Hardware the place> <Place: Ace Hardware the place>
>>> p2.id
2
# Set the place back again, using assignment in the reverse direction. # Set the place back again, using assignment in the reverse direction.
>>> p1.restaurant = r >>> p1.restaurant = r