diff options
| author | Jamie Matthews | 2012-10-24 11:27:01 +0100 |
|---|---|---|
| committer | Jamie Matthews | 2012-10-24 11:27:01 +0100 |
| commit | 388a807f64f60d84556288e2ade4f0fe57a8e66b (patch) | |
| tree | c7b376e48df0e781e195592b625e3fcd940c2e5f /rest_framework | |
| parent | 51fae73f3d565e2702c72ff9841cc072d6490804 (diff) | |
| download | django-rest-framework-388a807f64f60d84556288e2ade4f0fe57a8e66b.tar.bz2 | |
Switch from clean_<fieldname> to validate_<fieldname>, clarify documentation
Diffstat (limited to 'rest_framework')
| -rw-r--r-- | rest_framework/serializers.py | 4 | ||||
| -rw-r--r-- | rest_framework/tests/serializer.py | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/rest_framework/serializers.py b/rest_framework/serializers.py index c9c4faa3..802ca55f 100644 --- a/rest_framework/serializers.py +++ b/rest_framework/serializers.py @@ -210,13 +210,13 @@ class BaseSerializer(Field): def clean_fields(self, data): """ - Run clean_<fieldname> validators on the serializer + Run validate_<fieldname> methods on the serializer """ fields = self.get_fields(serialize=False, data=data, nested=self.opts.nested) for field_name, field in fields.items(): try: - clean_method = getattr(self, 'clean_%s' % field_name, None) + clean_method = getattr(self, 'validate_%s' % field_name, None) if clean_method: source = field.source or field_name data = clean_method(data, source) diff --git a/rest_framework/tests/serializer.py b/rest_framework/tests/serializer.py index 35908449..a32de80d 100644 --- a/rest_framework/tests/serializer.py +++ b/rest_framework/tests/serializer.py @@ -142,7 +142,7 @@ class ValidationTests(TestCase): class CommentSerializerWithFieldValidator(CommentSerializer): - def clean_content(self, attrs, source): + def validate_content(self, attrs, source): value = attrs[source] if "test" not in value: raise serializers.ValidationError("Test not in value") |
