aboutsummaryrefslogtreecommitdiffstats
path: root/rest_framework
diff options
context:
space:
mode:
authorTom Christie2014-09-05 13:57:34 +0100
committerTom Christie2014-09-05 13:57:34 +0100
commit2b47c6b700be530605bd30f0afe7214ea376bd13 (patch)
tree35376b99d70fe8a73356676b1057b1ec021d6d87 /rest_framework
parent4207d4453155105e69264d6e1b8cc7a3340d5e9d (diff)
parentcb3cc00edd4a7a7e8b583a9ba3a823fb6ca76346 (diff)
downloaddjango-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.py8
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)