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:  | 
