diff options
| author | Tom Christie | 2012-10-08 17:53:18 +0100 |
|---|---|---|
| committer | Tom Christie | 2012-10-08 17:53:18 +0100 |
| commit | 65f592866c5cd5103e99ed453543807bcbdaa9da (patch) | |
| tree | 311282a5f372878b225ac183a387db3ba6334042 /rest_framework/serializers.py | |
| parent | 4a21b3557edb3b901b86d3a888c44f772e33b922 (diff) | |
| download | django-rest-framework-65f592866c5cd5103e99ed453543807bcbdaa9da.tar.bz2 | |
Fix issue where required fields were not being properly validated. Also make model fields with a default value be not required
Diffstat (limited to 'rest_framework/serializers.py')
| -rw-r--r-- | rest_framework/serializers.py | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py index 1770c4ce..2141619f 100644 --- a/rest_framework/serializers.py +++ b/rest_framework/serializers.py @@ -371,9 +371,14 @@ class ModelSerializer(Serializer): models.BooleanField: BooleanField, } try: - return field_mapping[model_field.__class__]() + ret = field_mapping[model_field.__class__]() except KeyError: - return ModelField(model_field=model_field) + ret = ModelField(model_field=model_field) + + if model_field.default: + ret.required = False + + return ret def restore_object(self, attrs, instance=None): """ |
