diff options
| author | Ben Firshman | 2014-09-21 04:10:58 +0200 |
|---|---|---|
| committer | Ben Firshman | 2014-09-21 04:10:58 +0200 |
| commit | 576af8b1060d8a8881ef54b3b8bd7cdb6ecb0a47 (patch) | |
| tree | e3f413536a1ebdb870509282e5a31168c905b043 /src/shorturls/views.py | |
| parent | 61f4de67ed9cfdf09df4cee83d3c3c88775a6bf4 (diff) | |
| parent | 0158a679083f030980e5463b2e7af906423cf6b0 (diff) | |
| download | django-shorturls-576af8b1060d8a8881ef54b3b8bd7cdb6ecb0a47.tar.bz2 | |
Merge pull request #9 from bfirsh/django-1.7
Django 1.7
Diffstat (limited to 'src/shorturls/views.py')
| -rw-r--r-- | src/shorturls/views.py | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/shorturls/views.py b/src/shorturls/views.py index 1737472..d811fde 100644 --- a/src/shorturls/views.py +++ b/src/shorturls/views.py @@ -16,8 +16,12 @@ def redirect(request, prefix, tiny): # any of that stuff goes wrong. try: app_label, model_name = settings.SHORTEN_MODELS[prefix].split('.') - model = models.get_model(app_label, model_name) - if not model: raise ValueError + try: + model = models.get_model(app_label, model_name) + except LookupError: + raise Http404('Bad model.') + if not model: + raise ValueError id = base62.to_decimal(tiny) except (AttributeError, ValueError, KeyError): raise Http404('Bad prefix, model, SHORTEN_MODELS, or encoded ID.') @@ -50,4 +54,4 @@ def redirect(request, prefix, tiny): else: base = 'http://%s/' % RequestSite(request).domain - return HttpResponsePermanentRedirect(urlparse.urljoin(base, url))
\ No newline at end of file + return HttpResponsePermanentRedirect(urlparse.urljoin(base, url)) |
