From 6f76b9f58d3b34d71f31d088dc629d3337aef24e Mon Sep 17 00:00:00 2001 From: Luke Plant Date: Fri, 16 May 2008 22:53:39 +0000 Subject: [PATCH] Simplified control flow for change made in r7535 git-svn-id: http://code.djangoproject.com/svn/django/trunk@7536 bcc190cf-cafb-0310-a4f2-bffc1f526a37 --- django/contrib/admin/views/decorators.py | 17 ++++++----------- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/django/contrib/admin/views/decorators.py b/django/contrib/admin/views/decorators.py index 3039daadb1..c27b2e7427 100644 --- a/django/contrib/admin/views/decorators.py +++ b/django/contrib/admin/views/decorators.py @@ -84,18 +84,13 @@ def staff_member_required(view_func): message = ERROR_MESSAGE if '@' in username: # Mistakenly entered e-mail address instead of username? Look it up. - try: - users = list(User.objects.filter(email=username)) - if len(users) == 1: - user = users[0] - else: - # Either we cannot find the user, or if more than 1 - # we cannot guess which user is the correct one. - raise User.DoesNotExist() - except User.DoesNotExist: - message = _("Usernames cannot contain the '@' character.") + users = list(User.objects.filter(email=username)) + if len(users) == 1: + message = _("Your e-mail address is not your username. Try '%s' instead.") % users[0].username else: - message = _("Your e-mail address is not your username. Try '%s' instead.") % user.username + # Either we cannot find the user, or if more than 1 + # we cannot guess which user is the correct one. + message = _("Usernames cannot contain the '@' character.") return _display_login_form(request, message) # The user data is correct; log in the user in and continue.