mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-10-31 09:41:08 +00:00
Code Issues 32 Releases Wiki Activity

[1.9.x] Fixed #25981 -- Added need to update migrations to on_delete deprecation warning.

Browse Source 
Backport of a856555df2 from master
This commit is contained in:
Luis San Pablo
2015-12-23 19:52:24 -06:00
committed by Tim Graham
parent bc5eed8fb0
commit 4f2eb5fc5d
2 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
django/db/models/fields/related.py
View File

@@ -759,8 +759,9 @@ class ForeignKey(ForeignObject):
if on_delete is None: if on_delete is None:
warnings.warn( warnings.warn(
"on_delete will be a required arg for %s in Django 2.0. " "on_delete will be a required arg for %s in Django 2.0. Set "
"Set it to models.CASCADE if you want to maintain the current default behavior. " "it to models.CASCADE on models and in existing migrations "
"if you want to maintain the current default behavior. "
"See https://docs.djangoproject.com/en/%s/ref/models/fields/" "See https://docs.djangoproject.com/en/%s/ref/models/fields/"
"#django.db.models.ForeignKey.on_delete" % ( "#django.db.models.ForeignKey.on_delete" % (
self.__class__.__name__, self.__class__.__name__,
@@ -1001,8 +1002,9 @@ class OneToOneField(ForeignKey):
if on_delete is None: if on_delete is None:
warnings.warn( warnings.warn(
"on_delete will be a required arg for %s in Django 2.0. " "on_delete will be a required arg for %s in Django 2.0. Set "
"Set it to models.CASCADE if you want to maintain the current default behavior. " "it to models.CASCADE on models and in existing migrations "
"if you want to maintain the current default behavior. "
"See https://docs.djangoproject.com/en/%s/ref/models/fields/" "See https://docs.djangoproject.com/en/%s/ref/models/fields/"
"#django.db.models.ForeignKey.on_delete" % ( "#django.db.models.ForeignKey.on_delete" % (
self.__class__.__name__, self.__class__.__name__,

12
tests/invalid_models_tests/test_relative_fields.py
View File

@@ -53,9 +53,9 @@ class RelativeFieldTests(IsolatedModelsTestCase):
self.assertEqual( self.assertEqual(
str(warns[0].message), str(warns[0].message),
'on_delete will be a required arg for ForeignKey in Django ' 'on_delete will be a required arg for ForeignKey in Django '
'2.0. Set it to models.CASCADE if you want to maintain the ' '2.0. Set it to models.CASCADE on models and in existing '
'current default behavior. See ' 'migrations if you want to maintain the current default '
'https://docs.djangoproject.com/en/%s/ref/models/fields/' 'behavior. See https://docs.djangoproject.com/en/%s/ref/models/fields/'
'#django.db.models.ForeignKey.on_delete' % get_docs_version(), '#django.db.models.ForeignKey.on_delete' % get_docs_version(),
) )
@@ -90,9 +90,9 @@ class RelativeFieldTests(IsolatedModelsTestCase):
self.assertEqual( self.assertEqual(
str(warns[0].message), str(warns[0].message),
'on_delete will be a required arg for OneToOneField in Django ' 'on_delete will be a required arg for OneToOneField in Django '
'2.0. Set it to models.CASCADE if you want to maintain the ' '2.0. Set it to models.CASCADE on models and in existing '
'current default behavior. See ' 'migrations if you want to maintain the current default '
'https://docs.djangoproject.com/en/%s/ref/models/fields/' 'behavior. See https://docs.djangoproject.com/en/%s/ref/models/fields/'
'#django.db.models.ForeignKey.on_delete' % get_docs_version(), '#django.db.models.ForeignKey.on_delete' % get_docs_version(),
) )