diff options
| author | Evgeny Fadeev <evgeny.fadeev@gmail.com> | 2011-06-16 03:06:06 -0400 |
|---|---|---|
| committer | Evgeny Fadeev <evgeny.fadeev@gmail.com> | 2011-06-16 03:06:06 -0400 |
| commit | f9089dcdeff9a08317c3f4788804d1cefad1218c (patch) | |
| tree | fb404a818e8552a5a54228958c5212c585f8deca | |
| parent | 1bf853a2cbd28fa975ca767704192be963a53849 (diff) | |
| download | askbot-f9089dcdeff9a08317c3f4788804d1cefad1218c.tar.gz askbot-f9089dcdeff9a08317c3f4788804d1cefad1218c.tar.bz2 askbot-f9089dcdeff9a08317c3f4788804d1cefad1218c.zip | |
added redirect to the login page on views where authorized user is required
| -rw-r--r-- | askbot/tests/page_load_tests.py | 12 | ||||
| -rw-r--r-- | 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 |