From 505f1173d0c5a20ea804cad3b503c4bf55cd04d8 Mon Sep 17 00:00:00 2001
From: Dustin Farris <dustin@dustinfarris.com>
Date: Wed, 26 Feb 2014 12:08:26 -0500
Subject: [PATCH 1/2] Demonstrate problem post/put'ing serialized data that
 contains 'None'

---
 rest_framework/tests/serializers.py          |  8 ++++++
 rest_framework/tests/test_nullable_fields.py | 30 ++++++++++++++++++++
 rest_framework/tests/views.py                |  8 ++++++
 3 files changed, 46 insertions(+)
 create mode 100644 rest_framework/tests/serializers.py
 create mode 100644 rest_framework/tests/test_nullable_fields.py
 create mode 100644 rest_framework/tests/views.py

diff --git a/rest_framework/tests/serializers.py b/rest_framework/tests/serializers.py
new file mode 100644
index 000000000..cc943c7d0
--- /dev/null
+++ b/rest_framework/tests/serializers.py
@@ -0,0 +1,8 @@
+from rest_framework import serializers
+
+from rest_framework.tests.models import NullableForeignKeySource
+
+
+class NullableFKSourceSerializer(serializers.ModelSerializer):
+    class Meta:
+        model = NullableForeignKeySource
diff --git a/rest_framework/tests/test_nullable_fields.py b/rest_framework/tests/test_nullable_fields.py
new file mode 100644
index 000000000..556543d99
--- /dev/null
+++ b/rest_framework/tests/test_nullable_fields.py
@@ -0,0 +1,30 @@
+from django.core.urlresolvers import reverse
+
+from rest_framework.compat import patterns, url
+from rest_framework.test import APITestCase
+from rest_framework.tests.models import NullableForeignKeySource
+from rest_framework.tests.serializers import NullableFKSourceSerializer
+from rest_framework.tests.views import NullableFKSourceDetail
+
+
+urlpatterns = patterns(
+    '',
+    url(r'^objects/(?P<pk>\d+)/$', NullableFKSourceDetail.as_view(), name='object-detail'),
+)
+
+
+class NullableForeignKeyTests(APITestCase):
+    """
+    DRF should be able to handle nullable fields when a TestClient
+    POST/PUT request is made with its own serialized object.
+    """
+    urls = 'rest_framework.tests.test_nullable_fields'
+
+    def test_updating_object_with_null_field_value(self):
+        obj = NullableForeignKeySource(name='example', target=None)
+        obj.save()
+        serialized_data = NullableFKSourceSerializer(obj).data
+
+        response = self.client.put(reverse('object-detail', args=[obj.pk]), serialized_data)
+
+        self.assertEqual(response.data, serialized_data)
diff --git a/rest_framework/tests/views.py b/rest_framework/tests/views.py
new file mode 100644
index 000000000..3917b74a9
--- /dev/null
+++ b/rest_framework/tests/views.py
@@ -0,0 +1,8 @@
+from rest_framework import generics
+from rest_framework.tests.models import NullableForeignKeySource
+from rest_framework.tests.serializers import NullableFKSourceSerializer
+
+
+class NullableFKSourceDetail(generics.RetrieveUpdateDestroyAPIView):
+    model = NullableForeignKeySource
+    model_serializer_class = NullableFKSourceSerializer

From f126856f65aa86de2c4cc1b1e3bb6a52cebb34b8 Mon Sep 17 00:00:00 2001
From: Dustin Farris <dustin@dustinfarris.com>
Date: Thu, 27 Feb 2014 12:17:32 -0500
Subject: [PATCH 2/2] Allow 'None' to pass as a null value in RelatedFields

---
 rest_framework/relations.py                  | 7 ++++---
 rest_framework/tests/test_nullable_fields.py | 6 +++---
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/rest_framework/relations.py b/rest_framework/relations.py
index 02185c2ff..163a89840 100644
--- a/rest_framework/relations.py
+++ b/rest_framework/relations.py
@@ -33,6 +33,7 @@ class RelatedField(WritableField):
     many_widget = widgets.SelectMultiple
     form_field_class = forms.ChoiceField
     many_form_field_class = forms.MultipleChoiceField
+    null_values = (None, '', 'None')
 
     cache_choices = False
     empty_label = None
@@ -168,9 +169,9 @@ class RelatedField(WritableField):
                 return
             value = [] if self.many else None
 
-        if value in (None, '') and self.required:
-            raise ValidationError(self.error_messages['required'])
-        elif value in (None, ''):
+        if value in self.null_values:
+            if self.required:
+                raise ValidationError(self.error_messages['required'])
             into[(self.source or field_name)] = None
         elif self.many:
             into[(self.source or field_name)] = [self.from_native(item) for item in value]
diff --git a/rest_framework/tests/test_nullable_fields.py b/rest_framework/tests/test_nullable_fields.py
index 556543d99..6ee55c005 100644
--- a/rest_framework/tests/test_nullable_fields.py
+++ b/rest_framework/tests/test_nullable_fields.py
@@ -15,12 +15,12 @@ urlpatterns = patterns(
 
 class NullableForeignKeyTests(APITestCase):
     """
-    DRF should be able to handle nullable fields when a TestClient
-    POST/PUT request is made with its own serialized object.
+    DRF should be able to handle nullable foreign keys when a test
+    Client POST/PUT request is made with its own serialized object.
     """
     urls = 'rest_framework.tests.test_nullable_fields'
 
-    def test_updating_object_with_null_field_value(self):
+    def test_updating_object_with_null_fk(self):
         obj = NullableForeignKeySource(name='example', target=None)
         obj.save()
         serialized_data = NullableFKSourceSerializer(obj).data