mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2024-12-23 01:25:58 +00:00
Code Issues 25 Releases Wiki Activity

Fixed #32807 -- Fixed JSONField crash when redisplaying None values.

Browse Source 
Thanks to Alex Hill for the initial patch.
This commit is contained in:
Dan Strokirk 2021-06-24 17:15:25 +02:00 committed by Mariusz Felisiak
parent 66af94d56e
commit f5ea9aa2f3
2 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
django/forms/fields.py
View File

@ -1251,6 +1251,8 @@ class JSONField(CharField):
def bound_data(self, data, initial):
if self.disabled:
return initial
if data is None:
return None
try:
return json.loads(data, cls=self.decoder)
except json.JSONDecodeError:

15
tests/forms_tests/field_tests/test_jsonfield.py
View File

@ -97,6 +97,21 @@ class JSONFieldTest(SimpleTestCase):
form = JSONForm({'json_field': '["bar"]'}, initial={'json_field': ['foo']})
self.assertIn('[&quot;foo&quot;]</textarea>', form.as_p())
def test_redisplay_none_input(self):
class JSONForm(Form):
json_field = JSONField(required=True)
tests = [
{},
{'json_field': None},
]
for data in tests:
with self.subTest(data=data):
form = JSONForm(data)
self.assertEqual(form['json_field'].value(), 'null')
self.assertIn('null</textarea>', form.as_p())
self.assertEqual(form.errors['json_field'], ['This field is required.'])
def test_redisplay_wrong_input(self):
"""
Displaying a bound form (typically due to invalid input). The form