Update GenericForeignKey object_id to CharField/TextField

timgraham · timgraham · commit d86d77c1119e · 2024-10-21T11:00:05.000-04:00
MongoDB uses ObjectId rather than integer
diff --git a/tests/admin_filters/models.py b/tests/admin_filters/models.py
@@ -77,7 +77,7 @@ class TaggedItem(models.Model):
     content_type = models.ForeignKey(
         ContentType, models.CASCADE, related_name="tagged_items"
     )
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     content_object = GenericForeignKey("content_type", "object_id")
 
     def __str__(self):
diff --git a/tests/admin_inlines/models.py b/tests/admin_inlines/models.py
@@ -29,7 +29,7 @@ class Child(models.Model):
     teacher = models.ForeignKey(Teacher, models.CASCADE)
 
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     parent = GenericForeignKey()
 
     def __str__(self):
diff --git a/tests/admin_views/models.py b/tests/admin_views/models.py
@@ -545,7 +545,7 @@ class FunkyTag(models.Model):
     "Because we all know there's only one real use case for GFKs."
     name = models.CharField(max_length=25)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     content_object = GenericForeignKey("content_type", "object_id")
 
     def __str__(self):
@@ -1048,7 +1048,7 @@ class ImplicitlyGeneratedPK(models.Model):
 # Models for #25622
 class ReferencedByGenRel(models.Model):
     content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     content_object = GenericForeignKey("content_type", "object_id")
 
 
diff --git a/tests/contenttypes_tests/models.py b/tests/contenttypes_tests/models.py
@@ -77,7 +77,7 @@ class Question(models.Model):
 class Answer(models.Model):
     text = models.CharField(max_length=200)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     question = GenericForeignKey()
 
     class Meta:
@@ -89,7 +89,7 @@ class Post(models.Model):
 
     title = models.CharField(max_length=200)
     content_type = models.ForeignKey(ContentType, models.CASCADE, null=True)
-    object_id = models.PositiveIntegerField(null=True)
+    object_id = models.TextField(null=True)
     parent = GenericForeignKey()
     children = GenericRelation("Post")
 
diff --git a/tests/custom_managers/models.py b/tests/custom_managers/models.py
@@ -106,7 +106,7 @@ class Person(models.Model):
     favorite_thing_type = models.ForeignKey(
         "contenttypes.ContentType", models.SET_NULL, null=True
     )
-    favorite_thing_id = models.IntegerField(null=True)
+    favorite_thing_id = models.TextField()
     favorite_thing = GenericForeignKey("favorite_thing_type", "favorite_thing_id")
 
     objects = PersonManager()
@@ -134,7 +134,7 @@ class FunPerson(models.Model):
     favorite_thing_type = models.ForeignKey(
         "contenttypes.ContentType", models.SET_NULL, null=True
     )
-    favorite_thing_id = models.IntegerField(null=True)
+    favorite_thing_id = models.TextField()
     favorite_thing = GenericForeignKey("favorite_thing_type", "favorite_thing_id")
 
     objects = FunPeopleManager()
diff --git a/tests/delete/models.py b/tests/delete/models.py
@@ -219,21 +219,21 @@ class DeleteBottom(models.Model):
 
 class GenericB1(models.Model):
     content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     generic_delete_top = GenericForeignKey("content_type", "object_id")
 
 
 class GenericB2(models.Model):
     content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     generic_delete_top = GenericForeignKey("content_type", "object_id")
     generic_delete_bottom = GenericRelation("GenericDeleteBottom")
 
 
 class GenericDeleteBottom(models.Model):
     generic_b1 = models.ForeignKey(GenericB1, models.RESTRICT)
     content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     generic_b2 = GenericForeignKey()
 
 
diff --git a/tests/delete_regress/models.py b/tests/delete_regress/models.py
@@ -5,7 +5,7 @@
 
 class Award(models.Model):
     name = models.CharField(max_length=25)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
     content_object = GenericForeignKey()
 
diff --git a/tests/filtered_relation/models.py b/tests/filtered_relation/models.py
@@ -11,7 +11,7 @@ class Author(models.Model):
         related_query_name="preferred_by_authors",
     )
     content_type = models.ForeignKey(ContentType, models.CASCADE, null=True)
-    object_id = models.PositiveIntegerField(null=True)
+    object_id = models.TextField(null=True)
     content_object = GenericForeignKey()
 
 
diff --git a/tests/generic_inline_admin/models.py b/tests/generic_inline_admin/models.py
@@ -15,7 +15,7 @@ class Media(models.Model):
     """
 
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     content_object = GenericForeignKey()
     url = models.URLField()
     description = models.CharField(max_length=100, blank=True)
@@ -34,7 +34,7 @@ class Category(models.Model):
 
 class PhoneNumber(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     content_object = GenericForeignKey("content_type", "object_id")
     phone_number = models.CharField(max_length=30)
     category = models.ForeignKey(Category, models.SET_NULL, null=True, blank=True)
diff --git a/tests/generic_relations/models.py b/tests/generic_relations/models.py
@@ -19,7 +19,7 @@ class TaggedItem(models.Model):
 
     tag = models.SlugField()
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
 
     content_object = GenericForeignKey()
 
@@ -40,7 +40,7 @@ class AbstractComparison(models.Model):
     content_type1 = models.ForeignKey(
         ContentType, models.CASCADE, related_name="comparative1_set"
     )
-    object_id1 = models.PositiveIntegerField()
+    object_id1 = models.TextField()
 
     first_obj = GenericForeignKey(ct_field="content_type1", fk_field="object_id1")
 
@@ -54,7 +54,7 @@ class Comparison(AbstractComparison):
     content_type2 = models.ForeignKey(
         ContentType, models.CASCADE, related_name="comparative2_set"
     )
-    object_id2 = models.PositiveIntegerField()
+    object_id2 = models.TextField()
 
     other_obj = GenericForeignKey(ct_field="content_type2", fk_field="object_id2")
 
@@ -119,20 +119,20 @@ class ValuableRock(Mineral):
 
 
 class ManualPK(models.Model):
-    id = models.IntegerField(primary_key=True)
+    id = models.TextField(primary_key=True)
     tags = GenericRelation(TaggedItem, related_query_name="manualpk")
 
 
 class ForProxyModelModel(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     obj = GenericForeignKey(for_concrete_model=False)
     title = models.CharField(max_length=255, null=True)
 
 
 class ForConcreteModelModel(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.TextField()
     obj = GenericForeignKey()
 
 
diff --git a/tests/generic_relations/tests.py b/tests/generic_relations/tests.py
@@ -1,3 +1,5 @@
+from bson import ObjectId
+
 from django.contrib.contenttypes.models import ContentType
 from django.contrib.contenttypes.prefetch import GenericPrefetch
 from django.core.exceptions import FieldError
@@ -44,7 +46,7 @@ def setUpTestData(cls):
 
     def comp_func(self, obj):
         # Original list of tags:
-        return obj.tag, obj.content_type.model_class(), obj.object_id
+        return obj.tag, obj.content_type.model_class(), ObjectId(obj.object_id)
 
     async def test_generic_async_acreate(self):
         await self.bacon.tags.acreate(tag="orange")
@@ -258,10 +260,11 @@ def test_queries_content_type_restriction(self):
             Animal.objects.filter(tags__tag="fatty"),
             [self.platypus],
         )
-        self.assertSequenceEqual(
-            Animal.objects.exclude(tags__tag="fatty"),
-            [self.lion],
-        )
+        # Exists is not supported in MongoDB.
+        # self.assertSequenceEqual(
+        #     Animal.objects.exclude(tags__tag="fatty"),
+        #     [self.lion],
+        # )
 
     def test_object_deletion_with_generic_relation(self):
         """
@@ -639,13 +642,7 @@ def test_unsaved_generic_foreign_key_parent_bulk_create(self):
 
     def test_cache_invalidation_for_content_type_id(self):
         # Create a Vegetable and Mineral with the same id.
-        new_id = (
-            max(
-                Vegetable.objects.order_by("-id")[0].id,
-                Mineral.objects.order_by("-id")[0].id,
-            )
-            + 1
-        )
+        new_id = ObjectId()
         broccoli = Vegetable.objects.create(id=new_id, name="Broccoli")
         diamond = Mineral.objects.create(id=new_id, name="Diamond", hardness=7)
         tag = TaggedItem.objects.create(content_object=broccoli, tag="yummy")
diff --git a/tests/generic_relations_regress/models.py b/tests/generic_relations_regress/models.py
@@ -21,7 +21,7 @@
 
 class Link(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey()
 
 
@@ -50,7 +50,7 @@ class Address(models.Model):
     state = models.CharField(max_length=2)
     zipcode = models.CharField(max_length=5)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey()
 
 
@@ -87,7 +87,7 @@ class OddRelation2(models.Model):
 # models for test_q_object_or:
 class Note(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey()
     note = models.TextField()
 
@@ -124,7 +124,7 @@ class Tag(models.Model):
     content_type = models.ForeignKey(
         ContentType, models.CASCADE, related_name="g_r_r_tags"
     )
-    object_id = models.CharField(max_length=15)
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey()
     label = models.CharField(max_length=15)
 
@@ -157,7 +157,7 @@ class HasLinkThing(HasLinks):
 class A(models.Model):
     flag = models.BooleanField(null=True)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey("content_type", "object_id")
 
 
@@ -187,7 +187,7 @@ class Meta:
 
 class Node(models.Model):
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content = GenericForeignKey("content_type", "object_id")
 
 
diff --git a/tests/generic_relations_regress/tests.py b/tests/generic_relations_regress/tests.py
@@ -184,48 +184,52 @@ def test_gfk_to_model_with_empty_pk(self):
     def test_ticket_20378(self):
         # Create a couple of extra HasLinkThing so that the autopk value
         # isn't the same for Link and HasLinkThing.
-        hs1 = HasLinkThing.objects.create()
-        hs2 = HasLinkThing.objects.create()
+        hs1 = HasLinkThing.objects.create()  # noqa: F841
+        hs2 = HasLinkThing.objects.create()  # noqa: F841
         hs3 = HasLinkThing.objects.create()
         hs4 = HasLinkThing.objects.create()
         l1 = Link.objects.create(content_object=hs3)
         l2 = Link.objects.create(content_object=hs4)
         self.assertSequenceEqual(HasLinkThing.objects.filter(links=l1), [hs3])
         self.assertSequenceEqual(HasLinkThing.objects.filter(links=l2), [hs4])
-        self.assertSequenceEqual(
-            HasLinkThing.objects.exclude(links=l2), [hs1, hs2, hs3]
-        )
-        self.assertSequenceEqual(
-            HasLinkThing.objects.exclude(links=l1), [hs1, hs2, hs4]
-        )
+        # Exists is not supported in MongoDB.
+        # self.assertSequenceEqual(
+        #     HasLinkThing.objects.exclude(links=l2), [hs1, hs2, hs3]
+        # )
+        # self.assertSequenceEqual(
+        #     HasLinkThing.objects.exclude(links=l1), [hs1, hs2, hs4]
+        # )
 
     def test_ticket_20564(self):
         b1 = B.objects.create()
         b2 = B.objects.create()
         b3 = B.objects.create()
         c1 = C.objects.create(b=b1)
-        c2 = C.objects.create(b=b2)
+        c2 = C.objects.create(b=b2)  # noqa: F841
         c3 = C.objects.create(b=b3)
         A.objects.create(flag=None, content_object=b1)
         A.objects.create(flag=True, content_object=b2)
         self.assertSequenceEqual(C.objects.filter(b__a__flag=None), [c1, c3])
-        self.assertSequenceEqual(C.objects.exclude(b__a__flag=None), [c2])
+        # Exists is not supported in MongoDB.
+        # self.assertSequenceEqual(C.objects.exclude(b__a__flag=None), [c2])
 
     def test_ticket_20564_nullable_fk(self):
         b1 = B.objects.create()
         b2 = B.objects.create()
         b3 = B.objects.create()
         d1 = D.objects.create(b=b1)
-        d2 = D.objects.create(b=b2)
+        d2 = D.objects.create(b=b2)  # noqa: F841
         d3 = D.objects.create(b=b3)
         d4 = D.objects.create()
         A.objects.create(flag=None, content_object=b1)
         A.objects.create(flag=True, content_object=b1)
         A.objects.create(flag=True, content_object=b2)
-        self.assertSequenceEqual(D.objects.exclude(b__a__flag=None), [d2])
+        # Exists is not supported in MongoDB.
+        # self.assertSequenceEqual(D.objects.exclude(b__a__flag=None), [d2])
         self.assertSequenceEqual(D.objects.filter(b__a__flag=None), [d1, d3, d4])
         self.assertSequenceEqual(B.objects.filter(a__flag=None), [b1, b3])
-        self.assertSequenceEqual(B.objects.exclude(a__flag=None), [b2])
+        # Exists is not supported in MongoDB.
+        # self.assertSequenceEqual(B.objects.exclude(a__flag=None), [b2])
 
     def test_extra_join_condition(self):
         # A crude check that content_type_id is taken in account in the
diff --git a/tests/managers_regress/models.py b/tests/managers_regress/models.py
@@ -131,7 +131,7 @@ class RelationModel(models.Model):
     m2m = models.ManyToManyField(RelatedModel, related_name="test_m2m")
 
     gfk_ctype = models.ForeignKey(ContentType, models.SET_NULL, null=True)
-    gfk_id = models.IntegerField(null=True)
+    gfk_id = models.TextField()
     gfk = GenericForeignKey(ct_field="gfk_ctype", fk_field="gfk_id")
 
     def __str__(self):
diff --git a/tests/multiple_database/models.py b/tests/multiple_database/models.py
@@ -7,7 +7,7 @@
 class Review(models.Model):
     source = models.CharField(max_length=100)
     content_type = models.ForeignKey(ContentType, models.CASCADE)
-    object_id = models.PositiveIntegerField()
+    object_id = models.CharField(max_length=24)
     content_object = GenericForeignKey()
 
     class Meta:

Original file line number	Diff line number	Diff line change
`@@ -77,7 +77,7 @@ class TaggedItem(models.Model):`
`77`	`77`	`content_type = models.ForeignKey(`
`78`	`78`	`ContentType, models.CASCADE, related_name="tagged_items"`
`79`	`79`	`)`
`80`		`- object_id = models.PositiveIntegerField()`
	`80`	`+ object_id = models.TextField()`
`81`	`81`	`content_object = GenericForeignKey("content_type", "object_id")`
`82`	`82`
`83`	`83`	`def __str__(self):`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@ class Author(models.Model):`
`11`	`11`	`related_query_name="preferred_by_authors",`
`12`	`12`	`)`
`13`	`13`	`content_type = models.ForeignKey(ContentType, models.CASCADE, null=True)`
`14`		`- object_id = models.PositiveIntegerField(null=True)`
	`14`	`+ object_id = models.TextField(null=True)`
`15`	`15`	`content_object = GenericForeignKey()`
`16`	`16`
`17`	`17`