Ticket #16173: unittest.diff

tests/modeltests/many_to_one/tests.py

@@ -4 +4 @@
 from django.test import TestCase
 from django.core.exceptions import FieldError
 
-from models import Article, Reporter
+from models import Article, Reporter, ArticleWithDummyManager
+from models import SuccessfulExecutedException
 
 
 class ManyToOneTests(TestCase):
@@ -372 +373 @@
         # recursive don't cause recursion depth problems under deepcopy.
         self.r.cached_query = Article.objects.filter(reporter=self.r)
         self.assertEqual(repr(deepcopy(self.r)), "<Reporter: John Smith>")
+        
+    def test_reverse_single_related_manager(self):
+        # This unit test prooves if the dummy managers get methode will be 
+        # executed. Related to the ticket #16173.
+        r1 = Reporter.objects.create(first_name='John', last_name='Doh', email='john@doh.com')
+        a1 = ArticleWithDummyManager.objects.create(id=None, headline="Article with dummy manager",
+                                    pub_date=datetime(2011, 6, 9), reporter_id=r1.id)
+        self.assertRaises(a1.reporter, SuccessfulExecutedException)
+        
+ No newline at end of file

tests/modeltests/many_to_one/models.py

@@ -24 +24 @@
 
     class Meta:
         ordering = ('headline',)
+        
+class SuccessfulExecutedException(Exception):
+    pass
+
+class ArticleDummyManager(models.Manager):
+    use_for_related_fields = True
+    
+    def get(self, **kwargs):
+        raise SuccessfulExecutedException
+        
+class ArticleWithDummyManager(Article):
+    default_manager = ArticleDummyManager()

django/db/models/fields/related.py

@@ -311 +311 @@
             rel_mgr = self.field.rel.to._default_manager
             db = router.db_for_read(self.field.rel.to, instance=instance)
             if getattr(rel_mgr, 'use_for_related_fields', False):
-                rel_obj = rel_mgr.using(db).get(**params)
+                rel_obj = rel_mgr.db_manager(db).get(**params)
             else:
                 rel_obj = QuerySet(self.field.rel.to).using(db).get(**params)
             setattr(instance, cache_name, rel_obj)