From 54e2e688e1cfbdb37dfa5dd3b3ffdf2af12b4423 Mon Sep 17 00:00:00 2001 From: Tim Graham Date: Fri, 20 Nov 2015 10:31:33 -0500 Subject: Fixed #25715 -- Fixed Model.refresh_from_db() with ForeignKey w/on_delete=SET_NULL. --- tests/basic/models.py | 1 + tests/basic/tests.py | 11 +++++++++++ 2 files changed, 12 insertions(+) (limited to 'tests/basic') diff --git a/tests/basic/models.py b/tests/basic/models.py index 0e85087d32..91fbe7d648 100644 --- a/tests/basic/models.py +++ b/tests/basic/models.py @@ -34,6 +34,7 @@ class SelfRef(models.Model): null=True, blank=True, related_name='+', ) + article = models.ForeignKey(Article, models.SET_NULL, null=True, blank=True) def __str__(self): # This method intentionally doesn't work for all cases - part diff --git a/tests/basic/tests.py b/tests/basic/tests.py index 84033da43e..c67f78926f 100644 --- a/tests/basic/tests.py +++ b/tests/basic/tests.py @@ -765,6 +765,17 @@ class ModelRefreshTests(TestCase): self.assertEqual(a2.pub_date, pub_date) self.assertEqual(a2._state.db, "default") + def test_refresh_fk_on_delete_set_null(self): + a = Article.objects.create( + headline='Parrot programs in Python', + pub_date=datetime(2005, 7, 28), + ) + s1 = SelfRef.objects.create(article=a) + a.delete() + s1.refresh_from_db() + self.assertIsNone(s1.article_id) + self.assertIsNone(s1.article) + def test_refresh_no_fields(self): a = Article.objects.create(pub_date=self._truncate_ms(datetime.now())) with self.assertNumQueries(0): -- cgit v1.3