8000 Merge pull request #59 from twschiller/bugfix/58-display-none · 5monkeys/django-enumfield@ca5f334 · GitHub
[go: up one dir, main page]

Skip to content

Commit ca5f334

Browse files
authored
Merge pull request #59 from twschiller/bugfix/58-display-none
#58: fix display of None value
2 parents e532e0b + ead037a commit ca5f334

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

django_enumfield/db/fields.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ def contribute_to_class(
6464

6565
def _get_FIELD_display(self, cls):
6666
value = getattr(cls, self.attname)
67+
if value is None:
68+
return value
6769
return force_text(value.label, strings_only=True)
6870

6971
def get_prep_value(self, value):

django_enumfield/tests/test_enum.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,10 @@ class CustomPersonForm(forms.Form):
259259
self.assertTrue(form.is_valid(), form.errors)
260260
self.assertEqual(form.cleaned_data["status"], six.text_type())
261261

262+
def test_enum_display_none(self):
263+
beer = Beer(state=None)
264+
self.assertIsNone(beer.get_state_display())
265+
262266

263267
class EnumTest(TestCase):
264268
def test_label(self):

0 commit comments

Comments
 (0)
0