diff options
| author | Chuck Harmston | 2013-12-06 14:00:23 -0600 |
|---|---|---|
| committer | Chuck Harmston | 2013-12-06 14:00:23 -0600 |
| commit | cf6c11bd4b7e7fdaa1de659d69792030e565412a (patch) | |
| tree | aefd1b7a9f75e7a50c1fac28f6b9a1889eb99553 /rest_framework/serializers.py | |
| parent | e155534f4f11a3d17b64e7b2e4aad6f002eb271c (diff) | |
| download | django-rest-framework-cf6c11bd4b7e7fdaa1de659d69792030e565412a.tar.bz2 | |
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: |
