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/fields.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/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 336eac1e..33f25a00 100644 --- a/rest_framework/fields.py +++ b/rest_framework/fields.py @@ -161,7 +161,9 @@ class WritableField(Field): try: native = data[field_name] except KeyError: - return # TODO Consider validation behaviour, 'required' opt etc... + if self.required: + raise ValidationError(self.error_messages['required']) + return value = self.from_native(native) if self.source == '*': |