From f9089dcdeff9a08317c3f4788804d1cefad1218c Mon Sep 17 00:00:00 2001 From: Evgeny Fadeev Date: Thu, 16 Jun 2011 03:06:06 -0400 Subject: added redirect to the login page on views where authorized user is required --- askbot/tests/page_load_tests.py | 12 ++++++------ askbot/views/users.py | 3 ++- 2 files changed, 8 insertions(+), 7 deletions(-) diff --git a/askbot/tests/page_load_tests.py b/askbot/tests/page_load_tests.py index 9c8f685e..7dce9cf8 100644 --- a/askbot/tests/page_load_tests.py +++ b/askbot/tests/page_load_tests.py @@ -270,8 +270,8 @@ class PageLoadTests(PageLoadTestCase): 'user_profile', kwargs={'id': 2, 'slug': name_slug}, data={'sort':'inbox'}, - status_code=404, - template='404.html' + template='authopenid/signin.html', + follow=True ) self.try_url( 'user_profile', @@ -283,8 +283,8 @@ class PageLoadTests(PageLoadTestCase): 'user_profile', kwargs={'id': 2, 'slug': name_slug}, data={'sort':'votes'}, - status_code=404, - template='404.html' + template='authopenid/signin.html', + follow = True ) self.try_url( 'user_profile', @@ -296,8 +296,8 @@ class PageLoadTests(PageLoadTestCase): 'user_profile', kwargs={'id': 2, 'slug': name_slug}, data={'sort':'email_subscriptions'}, - status_code=404, - template='404.html' + template='authopenid/signin.html', + follow = True ) def test_user_urls_logged_in(self): diff --git a/askbot/views/users.py b/askbot/views/users.py index 324a4e8f..cbcbe185 100644 --- a/askbot/views/users.py +++ b/askbot/views/users.py @@ -60,7 +60,8 @@ def owner_or_moderator_required(f): elif request.user.is_authenticated() and request.user.can_moderate_user(profile_owner): pass else: - raise Http404 #todo: change to access forbidden? + params = '?next=%s' % request.path + return HttpResponseRedirect(reverse('user_signin') + params) return f(request, profile_owner, context) return wrapped_func -- cgit v1.2.3-1-g7c22