diff options
| author | Tom Christie | 2013-12-06 13:40:35 -0800 |
|---|---|---|
| committer | Tom Christie | 2013-12-06 13:40:35 -0800 |
| commit | e05fac685b41ccd33f3ffc0714f792eb0c8a9596 (patch) | |
| tree | aefd1b7a9f75e7a50c1fac28f6b9a1889eb99553 /rest_framework/serializers.py | |
| parent | e155534f4f11a3d17b64e7b2e4aad6f002eb271c (diff) | |
| parent | cf6c11bd4b7e7fdaa1de659d69792030e565412a (diff) | |
| download | django-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.py | 5 |
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: |
