summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Graham <timograham@gmail.com>2015-02-14 08:01:37 -0500
committerTim Graham <timograham@gmail.com>2015-02-14 08:08:05 -0500
commit0af3822dc362b6253bda1c9699466dd0bbbf6066 (patch)
tree356e4d6e1c0cb2259f6e33c6cd753208db9c1a5e
parent8a21d250334f56845f255be5534b01d8c6eda314 (diff)
Fixed #24325 -- Documented change in ModelForm.save() foreign key access.
-rw-r--r--docs/releases/1.8.txt22
-rw-r--r--docs/spelling_wordlist1
2 files changed, 23 insertions, 0 deletions
diff --git a/docs/releases/1.8.txt b/docs/releases/1.8.txt
index 4c7076a51e..b3dd2f6381 100644
--- a/docs/releases/1.8.txt
+++ b/docs/releases/1.8.txt
@@ -711,6 +711,28 @@ Now, an error will be raised to prevent data loss::
...
ValueError: Cannot assign "<Author: John>": "Author" instance isn't saved in the database.
+Accessing foreign keys in ``ModelForm.save()``
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+
+In older versions, you could access unsaved foreign key objects in
+``ModelForm.save()`` when adding new objects. For example, given ``Book`` with
+a ``ForeignKey`` to ``Author``::
+
+ class BookForm(forms.ModelForm):
+ def save(self, *args, **kwargs):
+ book = super(BookForm, self).save(*args, **kwargs)
+ book.title = "%s by %s" % (book.title, book.author.name)
+ return book
+
+Now if the related instance hasn't been saved (for example, when adding an
+author and some inlined books in the admin), accessing the foreign key
+``book.author`` in the example) will raise ``RelatedObjectDoesNotExist``. This
+change was necessary to avoid assigning unsaved objects to relations (as
+described in the previous section).
+
+To adapt the example above, you could replace ``book.author.name`` with
+``self.cleaned_data['author'].name``.
+
Management commands that only accept positional arguments
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
diff --git a/docs/spelling_wordlist b/docs/spelling_wordlist
index 19c72fc543..cea6650b1c 100644
--- a/docs/spelling_wordlist
+++ b/docs/spelling_wordlist
@@ -293,6 +293,7 @@ ing
ini
init
inline
+inlined
inlines
inspectdb
Instagram