diff options
| author | Tom Christie | 2014-11-28 15:31:51 +0000 |
|---|---|---|
| committer | Tom Christie | 2014-11-28 15:31:51 +0000 |
| commit | 6fbd23ab346e1a5b5401ba83e2ad2cd3474d2410 (patch) | |
| tree | 23b9d19d212ed67ef7c825b01ae2627d8dfaa630 /rest_framework | |
| parent | 8d989bb16dcc2f035ccc42710ca2714912a2ec40 (diff) | |
| parent | 67735687b297e66c7cfe61614040efcd9763f1f1 (diff) | |
| download | django-rest-framework-6fbd23ab346e1a5b5401ba83e2ad2cd3474d2410.tar.bz2 | |
Merge pull request #2086 from beck/doug/blow-up-with-bad-models
Ensure _resolve_model does not return None
Diffstat (limited to 'rest_framework')
| -rw-r--r-- | rest_framework/utils/model_meta.py | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/rest_framework/utils/model_meta.py b/rest_framework/utils/model_meta.py index 82361edf..54f9310d 100644 --- a/rest_framework/utils/model_meta.py +++ b/rest_framework/utils/model_meta.py @@ -43,7 +43,11 @@ def _resolve_model(obj): """ if isinstance(obj, six.string_types) and len(obj.split('.')) == 2: app_name, model_name = obj.split('.') - return models.get_model(app_name, model_name) + resolved_model = models.get_model(app_name, model_name) + if not resolved_model: + raise ValueError("Django did not return a model for " + "{0}.{1}".format(app_name, model_name)) + return resolved_model elif inspect.isclass(obj) and issubclass(obj, models.Model): return obj raise ValueError("{0} is not a Django model".format(obj)) |
