diff options
| author | Tom Christie | 2014-12-13 14:58:04 +0000 |
|---|---|---|
| committer | Tom Christie | 2014-12-13 14:58:04 +0000 |
| commit | aa571abb2089aedb5902a8f1670e59b4df99f3e9 (patch) | |
| tree | 7992c626a96bafa1b9ef0e79315b3637e28f4b0d /rest_framework/fields.py | |
| parent | eb78da4b5441bec1758e7d6db86822b3ea890f5f (diff) | |
| download | django-rest-framework-aa571abb2089aedb5902a8f1670e59b4df99f3e9.tar.bz2 | |
Better errors when serializer has incorrectly named field.
Diffstat (limited to 'rest_framework/fields.py')
| -rw-r--r-- | rest_framework/fields.py | 18 |
1 files changed, 17 insertions, 1 deletions
diff --git a/rest_framework/fields.py b/rest_framework/fields.py index 25122e14..205efd2f 100644 --- a/rest_framework/fields.py +++ b/rest_framework/fields.py @@ -274,7 +274,23 @@ class Field(object): Given the *outgoing* object instance, return the primitive value that should be used for this field. """ - return get_attribute(instance, self.source_attrs) + try: + return get_attribute(instance, self.source_attrs) + except (KeyError, AttributeError) as exc: + msg = ( + 'Got {exc_type} when attempting to get a value for field ' + '`{field}` on serializer `{serializer}`.\nThe serializer ' + 'field might be named incorrectly and not match ' + 'any attribute or key on the `{instance}` instance.\n' + 'Original exception text was: {exc}.'.format( + exc_type=type(exc).__name__, + field=self.field_name, + serializer=self.parent.__class__.__name__, + instance=instance.__class__.__name__, + exc=exc + ) + ) + raise type(exc)(msg) def get_default(self): """ |
