diff options
| author | Stephan Groß | 2013-01-23 07:53:54 +0100 |
|---|---|---|
| committer | Stephan Groß | 2013-01-23 07:53:54 +0100 |
| commit | 69e62457ef80b40398a6b137f10d5a0e05c3f07f (patch) | |
| tree | e363c37cf283b226063aa16d687e77d81ecb7b2b /rest_framework/serializers.py | |
| parent | f0071dbccd592ba3157738ced66809869f68b1cb (diff) | |
| download | django-rest-framework-69e62457ef80b40398a6b137f10d5a0e05c3f07f.tar.bz2 | |
Improve validate_<fieldname> fix
Diffstat (limited to 'rest_framework/serializers.py')
| -rw-r--r-- | rest_framework/serializers.py | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py index 0c60d17f..1450b9c7 100644 --- a/rest_framework/serializers.py +++ b/rest_framework/serializers.py @@ -227,14 +227,15 @@ class BaseSerializer(Field): Run `validate_<fieldname>()` and `validate()` methods on the serializer """ for field_name, field in self.fields.items(): - if field_name not in self._errors: - try: - validate_method = getattr(self, 'validate_%s' % field_name, None) - if validate_method: - source = field.source or field_name - attrs = validate_method(attrs, source) - except ValidationError as err: - self._errors[field_name] = self._errors.get(field_name, []) + list(err.messages) + if field_name in self._errors: + continue + try: + validate_method = getattr(self, 'validate_%s' % field_name, None) + if validate_method: + source = field.source or field_name + attrs = validate_method(attrs, source) + except ValidationError as err: + self._errors[field_name] = self._errors.get(field_name, []) + list(err.messages) # If there are already errors, we don't run .validate() because # field-validation failed and thus `attrs` may not be complete. |
