diff options
| author | Tom Christie | 2014-09-05 13:57:34 +0100 | 
|---|---|---|
| committer | Tom Christie | 2014-09-05 13:57:34 +0100 | 
| commit | 2b47c6b700be530605bd30f0afe7214ea376bd13 (patch) | |
| tree | 35376b99d70fe8a73356676b1057b1ec021d6d87 /rest_framework | |
| parent | 4207d4453155105e69264d6e1b8cc7a3340d5e9d (diff) | |
| parent | cb3cc00edd4a7a7e8b583a9ba3a823fb6ca76346 (diff) | |
| download | django-rest-framework-2b47c6b700be530605bd30f0afe7214ea376bd13.tar.bz2 | |
Merge pull request #1834 from piotrjakimiak/master
Fix returning None when allow_none is True in CharField
Diffstat (limited to 'rest_framework')
| -rw-r--r-- | rest_framework/fields.py | 8 | 
1 files changed, 6 insertions, 2 deletions
| diff --git a/rest_framework/fields.py b/rest_framework/fields.py index 9d707c9b..8e15345d 100644 --- a/rest_framework/fields.py +++ b/rest_framework/fields.py @@ -475,8 +475,12 @@ class CharField(WritableField):          if isinstance(value, six.string_types):              return value -        if value is None and not self.allow_none: -            return '' +        if value is None: +            if not self.allow_none: +                return '' +            else: +                # Return None explicitly because smart_text(None) == 'None'. See #1834 for details +                return None          return smart_text(value) | 
