aboutsummaryrefslogtreecommitdiffstats
path: root/rest_framework/serializers.py
diff options
context:
space:
mode:
authorTom Christie2013-12-06 13:40:35 -0800
committerTom Christie2013-12-06 13:40:35 -0800
commite05fac685b41ccd33f3ffc0714f792eb0c8a9596 (patch)
treeaefd1b7a9f75e7a50c1fac28f6b9a1889eb99553 /rest_framework/serializers.py
parente155534f4f11a3d17b64e7b2e4aad6f002eb271c (diff)
parentcf6c11bd4b7e7fdaa1de659d69792030e565412a (diff)
downloaddjango-rest-framework-e05fac685b41ccd33f3ffc0714f792eb0c8a9596.tar.bz2
Merge pull request #1272 from chuckharmston/1158-partial_relatedfield_null
Raise appropriate error in serializer when making a partial update to set a required RelatedField to null (issue #1158)
Diffstat (limited to 'rest_framework/serializers.py')
-rw-r--r--rest_framework/serializers.py5
1 files changed, 4 insertions, 1 deletions
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py
index 163abf4f..44e4b04b 100644
--- a/rest_framework/serializers.py
+++ b/rest_framework/serializers.py
@@ -896,7 +896,10 @@ class ModelSerializer(Serializer):
# Update an existing instance...
if instance is not None:
for key, val in attrs.items():
- setattr(instance, key, val)
+ try:
+ setattr(instance, key, val)
+ except ValueError:
+ self._errors[key] = self.error_messages['required']
# ...or create a new instance
else: