encode/django-rest-framework
1
1
Fork 0
mirror of https://github.com/encode/django-rest-framework.git synced 2025-08-08 06:14:47 +03:00
Code Issues Packages Projects Releases Wiki Activity

Add explicit on_delete=models.CASCADE to ForeignKey fields.

Browse Source
This commit is contained in:
Tom Christie 2016-06-02 13:04:56 +01:00
parent 252ccf1e6b
commit 642db43d5a
3 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/test_authentication.py
View File

@ -29,7 +29,7 @@ factory = APIRequestFactory()
class CustomToken(models.Model): class CustomToken(models.Model):
key = models.CharField(max_length=40, primary_key=True) key = models.CharField(max_length=40, primary_key=True)
user = models.OneToOneField(User) user = models.OneToOneField(User, on_delete=models.CASCADE)
class CustomTokenAuthentication(TokenAuthentication): class CustomTokenAuthentication(TokenAuthentication):

3
tests/test_filters.py
View File

@ -507,8 +507,7 @@ class OrderingFilterModel(models.Model):
class OrderingFilterRelatedModel(models.Model): class OrderingFilterRelatedModel(models.Model):
related_object = models.ForeignKey(OrderingFilterModel, related_object = models.ForeignKey(OrderingFilterModel, related_name="relateds", on_delete=models.CASCADE)
related_name="relateds")
class OrderingFilterSerializer(serializers.ModelSerializer): class OrderingFilterSerializer(serializers.ModelSerializer):

14
tests/test_model_serializer.py
View File

@ -403,20 +403,20 @@ class ThroughTargetModel(models.Model):
class Supplementary(models.Model): class Supplementary(models.Model):
extra = models.IntegerField() extra = models.IntegerField()
forwards = models.ForeignKey('ThroughTargetModel') forwards = models.ForeignKey('ThroughTargetModel', on_delete=models.CASCADE)
backwards = models.ForeignKey('RelationalModel') backwards = models.ForeignKey('RelationalModel', on_delete=models.CASCADE)
class RelationalModel(models.Model): class RelationalModel(models.Model):
foreign_key = models.ForeignKey(ForeignKeyTargetModel, related_name='reverse_foreign_key') foreign_key = models.ForeignKey(ForeignKeyTargetModel, related_name='reverse_foreign_key', on_delete=models.CASCADE)
many_to_many = models.ManyToManyField(ManyToManyTargetModel, related_name='reverse_many_to_many') many_to_many = models.ManyToManyField(ManyToManyTargetModel, related_name='reverse_many_to_many')
one_to_one = models.OneToOneField(OneToOneTargetModel, related_name='reverse_one_to_one') one_to_one = models.OneToOneField(OneToOneTargetModel, related_name='reverse_one_to_one', on_delete=models.CASCADE)
through = models.ManyToManyField(ThroughTargetModel, through=Supplementary, related_name='reverse_through') through = models.ManyToManyField(ThroughTargetModel, through=Supplementary, related_name='reverse_through')
class UniqueTogetherModel(models.Model): class UniqueTogetherModel(models.Model):
foreign_key = models.ForeignKey(ForeignKeyTargetModel, related_name='unique_foreign_key') foreign_key = models.ForeignKey(ForeignKeyTargetModel, related_name='unique_foreign_key', on_delete=models.CASCADE)
one_to_one = models.OneToOneField(OneToOneTargetModel, related_name='unique_one_to_one') one_to_one = models.OneToOneField(OneToOneTargetModel, related_name='unique_one_to_one', on_delete=models.CASCADE)
class Meta: class Meta:
unique_together = ("foreign_key", "one_to_one") unique_together = ("foreign_key", "one_to_one")
@ -598,7 +598,7 @@ class DisplayValueTargetModel(models.Model):
class DisplayValueModel(models.Model): class DisplayValueModel(models.Model):
color = models.ForeignKey(DisplayValueTargetModel) color = models.ForeignKey(DisplayValueTargetModel, on_delete=models.CASCADE)
class TestRelationalFieldDisplayValue(TestCase): class TestRelationalFieldDisplayValue(TestCase):