diff accounts/views.py @ 783:004b3a90de66

Update our code that expected LOGIN_REDIRECT_URL to be a url. It is now a view name.
author Brian Neal <bgneal@gmail.com>
date Tue, 13 May 2014 20:23:30 -0500
parents 4a49d4ac319f
children be233ba7ca31
line wrap: on
line diff
--- a/accounts/views.py	Tue May 13 19:44:48 2014 -0500
+++ b/accounts/views.py	Tue May 13 20:23:30 2014 -0500
@@ -4,9 +4,7 @@
 """
 import logging
 
-from django.shortcuts import render
-from django.http import HttpResponseRedirect
-from django.core.urlresolvers import reverse
+from django.shortcuts import render, redirect
 from django.conf import settings
 
 from accounts.models import PendingUser
@@ -22,13 +20,13 @@
 @log_auth_failures('Register')
 def register(request):
     if request.user.is_authenticated():
-        return HttpResponseRedirect(settings.LOGIN_REDIRECT_URL)
+        return redirect(settings.LOGIN_REDIRECT_URL)
 
     if request.method == 'POST':
         form = RegisterForm(request.POST, ip=request.META.get('REMOTE_ADDR', '?'))
         if form.is_valid():
             form.save()
-            return HttpResponseRedirect(reverse('accounts.views.register_thanks'))
+            return redirect('accounts.views.register_thanks')
     else:
         form = RegisterForm()
 
@@ -38,7 +36,7 @@
 
 def register_thanks(request):
     if request.user.is_authenticated():
-        return HttpResponseRedirect(settings.LOGIN_REDIRECT_URL)
+        return redirect(settings.LOGIN_REDIRECT_URL)
 
     return render(request, 'accounts/register_thanks.html')
 
@@ -46,7 +44,7 @@
 
 def register_confirm(request, username, key):
     if request.user.is_authenticated():
-        return HttpResponseRedirect(settings.LOGIN_REDIRECT_URL)
+        return redirect(settings.LOGIN_REDIRECT_URL)
 
     # purge expired users
 
@@ -78,13 +76,13 @@
 def username_query(request):
     """This view handles forgotten username queries."""
     if request.user.is_authenticated():
-        return HttpResponseRedirect(settings.LOGIN_REDIRECT_URL)
+        return redirect(settings.LOGIN_REDIRECT_URL)
 
     if request.method == 'POST':
         form = ForgotUsernameForm(data=request.POST)
         if form.is_valid():
             form.save()
-            return HttpResponseRedirect(reverse('accounts-username_sent'))
+            return redirect('accounts-username_sent')
     else:
         form = ForgotUsernameForm()