diff options
| author | Tom Christie | 2013-01-03 14:05:36 -0800 |
|---|---|---|
| committer | Tom Christie | 2013-01-03 14:05:36 -0800 |
| commit | 6da21fa79604ffa7dc57267effa3afb59c63e62b (patch) | |
| tree | 097158438ea23789c1dbe8efed7b735fa1c26e1e /rest_framework/fields.py | |
| parent | a122856d4610492670838298e40a2081f5ef9f75 (diff) | |
| parent | 3873bc8a857829a12b2bb6a34610ab5295887660 (diff) | |
| download | django-rest-framework-6da21fa79604ffa7dc57267effa3afb59c63e62b.tar.bz2 | |
Merge pull request #541 from yprez/default_and_partial_serializers2
Fix "default" values and partial serializers. Fixes #532.
Diffstat (limited to 'rest_framework/fields.py')
| -rw-r--r-- | rest_framework/fields.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/rest_framework/fields.py b/rest_framework/fields.py index a022fe09..c406a2f3 100644 --- a/rest_framework/fields.py +++ b/rest_framework/fields.py @@ -186,8 +186,10 @@ class WritableField(Field): else: native = data[field_name] except KeyError: - if self.default is not None: + if self.default is not None and not self.root.partial: native = self.default + # partial serializers shouldn't set the default field to avoid + # overriding the previously set value else: if self.required: raise ValidationError(self.error_messages['required']) |
