use writablefield style for serializer handling of self.default

craigds · craigds · commit 101fa26ebc09 · 2013-03-18T16:05:34.000+13:00
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py
@@ -358,11 +358,13 @@ def field_from_native(self, data, files, field_name, into):
         try:
             value = data[field_name]
         except KeyError:
-            if self.required:
-                raise ValidationError(self.error_messages['required'])
-            if self.default is None:
+            if self.default is not None and not self.partial:
+                # Note: partial updates shouldn't set defaults
+                value = copy.deepcopy(self.default)
+            else:
+                if self.required:
+                    raise ValidationError(self.error_messages['required'])
                 return
-            value = copy.deepcopy(self.default)
 
         # Set the serializer object if it exists
         obj = getattr(self.parent.object, field_name) if self.parent.object else None
