Fixed #21414 -- Removed RelatedObject and deprecated Field.related.

3 files changed, 171 insertions, 156 deletions

diff --git a/tests/basic/tests.py b/tests/basic/tests.py
index 31e8b724bc..74515047f5 100644
--- a/tests/basic/tests.py
+++ b/tests/basic/tests.py
@@ -2,11 +2,13 @@ from __future__ import unicode_literals
 
 from datetime import datetime, timedelta
 import threading
+import warnings
 
 from django.core.exceptions import ObjectDoesNotExist, MultipleObjectsReturned
 from django.db import connections, DEFAULT_DB_ALIAS
 from django.db import DatabaseError
 from django.db.models.fields import Field
+from django.db.models.fields.related import ForeignObjectRel
 from django.db.models.manager import BaseManager
 from django.db.models.query import QuerySet, EmptyQuerySet, ValuesListQuerySet, MAX_GET_RESULTS
 from django.test import TestCase, TransactionTestCase, skipIfDBFeature, skipUnlessDBFeature
@@ -770,3 +772,16 @@ class ModelRefreshTests(TestCase):
         a = Article.objects.create(pub_date=self._truncate_ms(datetime.now()))
         with self.assertNumQueries(0):
             a.refresh_from_db(fields=[])
+
+
+class TestRelatedObjectDeprecation(TestCase):
+    def test_field_related_deprecation(self):
+        field = SelfRef._meta.get_field_by_name('selfref')[0]
+        with warnings.catch_warnings(record=True) as warns:
+            warnings.simplefilter('always')
+            self.assertIsInstance(field.related, ForeignObjectRel)
+            self.assertEqual(len(warns), 1)
+            self.assertEqual(
+                str(warns.pop().message),
+                'Usage of field.related has been deprecated. Use field.rel instead.'
+            )
diff --git a/tests/model_meta/test.py b/tests/model_meta/test.py
index 9901631a70..835ce9c6d5 100644
--- a/tests/model_meta/test.py
+++ b/tests/model_meta/test.py
@@ -233,114 +233,114 @@ TEST_RESULTS = {
     },
     'get_all_related_objects_with_model_hidden': {
         BasePerson: (
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:person', None),
-            ('model_meta:relating_basepeople', None),
-            ('model_meta:relating_basepeople_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.person', None),
+            ('model_meta.relating_basepeople', None),
+            ('model_meta.relating_basepeople_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Person: (
-            ('model_meta:baseperson_friends_base', BasePerson),
-            ('model_meta:baseperson_friends_base', BasePerson),
-            ('model_meta:baseperson_m2m_base', BasePerson),
-            ('model_meta:baseperson_following_base', BasePerson),
-            ('model_meta:baseperson_following_base', BasePerson),
-            ('model_meta:baseperson_m2m_abstract', BasePerson),
-            ('model_meta:baseperson_friends_abstract', BasePerson),
-            ('model_meta:baseperson_friends_abstract', BasePerson),
-            ('model_meta:baseperson_following_abstract', BasePerson),
-            ('model_meta:baseperson_following_abstract', BasePerson),
-            ('model_meta:relating_basepeople', BasePerson),
-            ('model_meta:relating_basepeople_hidden', BasePerson),
-            ('model_meta:relating', BasePerson),
-            ('model_meta:relating', BasePerson),
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:relating_people', None),
-            ('model_meta:relating_people_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.baseperson_friends_base', BasePerson),
+            ('model_meta.baseperson_friends_base', BasePerson),
+            ('model_meta.baseperson_m2m_base', BasePerson),
+            ('model_meta.baseperson_following_base', BasePerson),
+            ('model_meta.baseperson_following_base', BasePerson),
+            ('model_meta.baseperson_m2m_abstract', BasePerson),
+            ('model_meta.baseperson_friends_abstract', BasePerson),
+            ('model_meta.baseperson_friends_abstract', BasePerson),
+            ('model_meta.baseperson_following_abstract', BasePerson),
+            ('model_meta.baseperson_following_abstract', BasePerson),
+            ('model_meta.relating_basepeople', BasePerson),
+            ('model_meta.relating_basepeople_hidden', BasePerson),
+            ('model_meta.relating', BasePerson),
+            ('model_meta.relating', BasePerson),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.relating_people', None),
+            ('model_meta.relating_people_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Relation: (
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
         ),
     },
     'get_all_related_objects_with_model_hidden_local': {
         BasePerson: (
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:person', None),
-            ('model_meta:relating_basepeople', None),
-            ('model_meta:relating_basepeople_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.person', None),
+            ('model_meta.relating_basepeople', None),
+            ('model_meta.relating_basepeople_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Person: (
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:relating_people', None),
-            ('model_meta:relating_people_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.relating_people', None),
+            ('model_meta.relating_people_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Relation: (
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
         ),
     },
     'get_all_related_objects_with_model_proxy': {
@@ -360,67 +360,67 @@ TEST_RESULTS = {
     },
     'get_all_related_objects_with_model_proxy_hidden': {
         BasePerson: (
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_friends_base', None),
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_following_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_friends_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:baseperson_following_abstract', None),
-            ('model_meta:person', None),
-            ('model_meta:relating_basepeople', None),
-            ('model_meta:relating_basepeople_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_friends_base', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_following_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_friends_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.baseperson_following_abstract', None),
+            ('model_meta.person', None),
+            ('model_meta.relating_basepeople', None),
+            ('model_meta.relating_basepeople_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Person: (
-            ('model_meta:baseperson_friends_base', BasePerson),
-            ('model_meta:baseperson_friends_base', BasePerson),
-            ('model_meta:baseperson_m2m_base', BasePerson),
-            ('model_meta:baseperson_following_base', BasePerson),
-            ('model_meta:baseperson_following_base', BasePerson),
-            ('model_meta:baseperson_m2m_abstract', BasePerson),
-            ('model_meta:baseperson_friends_abstract', BasePerson),
-            ('model_meta:baseperson_friends_abstract', BasePerson),
-            ('model_meta:baseperson_following_abstract', BasePerson),
-            ('model_meta:baseperson_following_abstract', BasePerson),
-            ('model_meta:relating_basepeople', BasePerson),
-            ('model_meta:relating_basepeople_hidden', BasePerson),
-            ('model_meta:relating', BasePerson),
-            ('model_meta:relating', BasePerson),
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_friends_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:person_following_inherited', None),
-            ('model_meta:relating_people', None),
-            ('model_meta:relating_people_hidden', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
-            ('model_meta:relating', None),
+            ('model_meta.baseperson_friends_base', BasePerson),
+            ('model_meta.baseperson_friends_base', BasePerson),
+            ('model_meta.baseperson_m2m_base', BasePerson),
+            ('model_meta.baseperson_following_base', BasePerson),
+            ('model_meta.baseperson_following_base', BasePerson),
+            ('model_meta.baseperson_m2m_abstract', BasePerson),
+            ('model_meta.baseperson_friends_abstract', BasePerson),
+            ('model_meta.baseperson_friends_abstract', BasePerson),
+            ('model_meta.baseperson_following_abstract', BasePerson),
+            ('model_meta.baseperson_following_abstract', BasePerson),
+            ('model_meta.relating_basepeople', BasePerson),
+            ('model_meta.relating_basepeople_hidden', BasePerson),
+            ('model_meta.relating', BasePerson),
+            ('model_meta.relating', BasePerson),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_friends_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.person_following_inherited', None),
+            ('model_meta.relating_people', None),
+            ('model_meta.relating_people_hidden', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
+            ('model_meta.relating', None),
         ),
         Relation: (
-            ('model_meta:baseperson_m2m_base', None),
-            ('model_meta:baseperson_m2m_abstract', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:baseperson', None),
-            ('model_meta:person_m2m_inherited', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:person', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
-            ('model_meta:proxyperson', None),
+            ('model_meta.baseperson_m2m_base', None),
+            ('model_meta.baseperson_m2m_abstract', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.baseperson', None),
+            ('model_meta.person_m2m_inherited', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.person', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
+            ('model_meta.proxyperson', None),
         ),
     },
     'get_all_related_many_to_many_with_model': {
@@ -643,12 +643,12 @@ class GetFieldByNameTests(OptionsBaseTests):
     def test_get_related_object(self):
         field_info = Person._meta.get_field_by_name('relating_baseperson')
         self.assertEqual(field_info[1:], (BasePerson, False, False))
-        self.assertIsInstance(field_info[0], related.RelatedObject)
+        self.assertIsInstance(field_info[0], related.ForeignObjectRel)
 
     def test_get_related_m2m(self):
         field_info = Person._meta.get_field_by_name('relating_people')
         self.assertEqual(field_info[1:], (None, False, True))
-        self.assertIsInstance(field_info[0], related.RelatedObject)
+        self.assertIsInstance(field_info[0], related.ForeignObjectRel)
 
     def test_get_generic_foreign_key(self):
         # For historic reasons generic foreign keys aren't available.
diff --git a/tests/one_to_one/tests.py b/tests/one_to_one/tests.py
index 72d279b83c..73d76d4a48 100644
--- a/tests/one_to_one/tests.py
+++ b/tests/one_to_one/tests.py
@@ -383,7 +383,7 @@ class OneToOneTests(TestCase):
         be added to the related model.
         """
         self.assertFalse(
-            hasattr(Target, HiddenPointer._meta.get_field('target').related.get_accessor_name())
+            hasattr(Target, HiddenPointer._meta.get_field('target').rel.get_accessor_name())
         )
 
     def test_related_object(self):