X-Git-Url: http://git.ithinksw.org/philo.git/blobdiff_plain/3f3fc1db168598439bc6db14aaae061e76d991f8..e9c5d3eb331486b87d7276d37b86933bd810200b:/contrib/waldo/models.py diff --git a/contrib/waldo/models.py b/contrib/waldo/models.py index 37e7a95..4f0cd8c 100644 --- a/contrib/waldo/models.py +++ b/contrib/waldo/models.py @@ -4,6 +4,7 @@ from django.contrib import messages from django.contrib.auth import authenticate, login, views as auth_views from django.contrib.auth.forms import PasswordResetForm, SetPasswordForm, PasswordChangeForm from django.contrib.auth.models import User +from django.contrib.auth.tokens import default_token_generator as password_token_generator from django.contrib.sites.models import Site from django.core.mail import send_mail from django.core.urlresolvers import reverse @@ -16,7 +17,8 @@ from django.views.decorators.cache import never_cache from django.views.decorators.csrf import csrf_protect from philo.models import MultiView, Page from philo.contrib.waldo.forms import LOGIN_FORM_KEY, LoginForm, RegistrationForm -from philo.contrib.waldo.tokens import default_token_generator +from philo.contrib.waldo.tokens import registration_token_generator, email_token_generator +import urlparse ERROR_MESSAGE = ugettext_lazy("Please enter a correct username and password. Note that both fields are case-sensitive.") @@ -37,6 +39,9 @@ class LoginMultiView(MultiView): """ login_page = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_login_related') password_reset_page = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_password_reset_related') + password_reset_confirmation_email = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_password_reset_confirmation_email_related') + password_set_page = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_password_set_related') + password_change_page = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_password_change_related', blank=True, null=True) register_page = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_register_related') register_confirmation_email = models.ForeignKey(Page, related_name='%(app_label)s_%(class)s_register_confirmation_email_related') @@ -44,18 +49,20 @@ class LoginMultiView(MultiView): def urlpatterns(self): urlpatterns = patterns('', url(r'^login/$', self.login, name='login'), - url(r'^logout/$', self.logout, name='logout') - ) - urlpatterns += patterns('', + url(r'^logout/$', self.logout, name='logout'), + url(r'^password/reset/$', csrf_protect(self.password_reset), name='password_reset'), - url(r'^password/reset/(?P\w+)/(?P[^/]+)/$', - self.password_reset_confirm, name='password_reset_confirm') - ) - urlpatterns += patterns('', + url(r'^password/reset/(?P\w+)/(?P[^/]+)/$', self.password_reset_confirm, name='password_reset_confirm'), + url(r'^register/$', csrf_protect(self.register), name='register'), - url(r'^register/(?P\w+)/(?P[^/]+)/$', - self.register_confirm, name='register_confirm') + url(r'^register/(?P\w+)/(?P[^/]+)/$', self.register_confirm, name='register_confirm') ) + + if self.password_change_page: + urlpatterns += patterns('', + url(r'^password/change/$', csrf_protect(self.login_required(self.password_change)), name='password_change'), + ) + return urlpatterns def get_context(self, extra_dict=None): @@ -66,7 +73,19 @@ class LoginMultiView(MultiView): def display_login_page(self, request, message, node=None, extra_context=None): request.session.set_test_cookie() - redirect = request.META.get('HTTP_REFERER', None) + referrer = request.META.get('HTTP_REFERER', None) + + if referrer is not None: + referrer = urlparse.urlparse(referrer) + host = referrer[1] + if host != request.get_host(): + referrer = None + else: + redirect = ''.join(referrer[2:]) + + if referrer is None: + redirect = node.get_absolute_url() + path = request.get_full_path() if redirect != path: if redirect is None: @@ -131,7 +150,10 @@ class LoginMultiView(MultiView): else: if user.is_active: login(request, user) - redirect = request.session.pop('redirect') + try: + redirect = request.session.pop('redirect') + except KeyError: + redirect = node.get_absolute_url() return HttpResponseRedirect(redirect) else: return self.display_login_page(request, ERROR_MESSAGE, node, context) @@ -154,13 +176,72 @@ class LoginMultiView(MultiView): from_email = 'noreply@%s' % Site.objects.get_current().domain send_mail(subject, message, from_email, [email]) - def password_reset(self, request, node=None, extra_context=None): - pass + def password_reset(self, request, node=None, extra_context=None, token_generator=password_token_generator): + if request.method == 'POST': + form = PasswordResetForm(request.POST) + if form.is_valid(): + for user in form.users_cache: + current_site = Site.objects.get_current() + token = token_generator.make_token(user) + link = 'http://%s/%s/%s/' % (current_site.domain, node.get_absolute_url().strip('/'), reverse('password_reset_confirm', urlconf=self, kwargs={'uidb36': int_to_base36(user.id), 'token': token}).strip('/')) + context = { + 'link': link, + 'username': user.username + } + self.send_confirmation_email('Confirm password reset for account at %s' % current_site.domain, user.email, self.password_reset_confirmation_email, context) + messages.add_message(request, messages.SUCCESS, "An email has been sent to the address you provided with details on resetting your password.", fail_silently=True) + return HttpResponseRedirect('') + else: + form = PasswordResetForm() + + context = self.get_context({'form': form}) + context.update(extra_context or {}) + return self.password_reset_page.render_to_response(node, request, extra_context=context) - def password_reset_confirm(self, request, node=None, extra_context=None): - pass + def password_reset_confirm(self, request, node=None, extra_context=None, uidb36=None, token=None, token_generator=password_token_generator): + """ + Checks that a given hash in a password reset link is valid. If so, + displays the password set form. + """ + assert uidb36 is not None and token is not None + try: + uid_int = base36_to_int(uidb36) + except: + raise Http404 + + user = get_object_or_404(User, id=uid_int) + + if token_generator.check_token(user, token): + if request.method == 'POST': + form = SetPasswordForm(user, request.POST) + + if form.is_valid(): + form.save() + messages.add_message(request, messages.SUCCESS, "Password reset successful.") + return HttpResponseRedirect('/%s/%s/' % (node.get_absolute_url().strip('/'), reverse('login', urlconf=self).strip('/'))) + else: + form = SetPasswordForm(user) + + context = self.get_context({'form': form}) + return self.password_set_page.render_to_response(node, request, extra_context=context) + + raise Http404 + + def password_change(self, request, node=None, extra_context=None): + if request.method == 'POST': + form = PasswordChangeForm(request.user, request.POST) + if form.is_valid(): + form.save() + messages.add_message(request, messages.SUCCESS, 'Password changed successfully.', fail_silently=True) + return HttpResponseRedirect('') + else: + form = PasswordChangeForm(request.user) + + context = self.get_context({'form': form}) + context.update(extra_context or {}) + return self.password_change_page.render_to_response(node, request, extra_context=context) - def register(self, request, node=None, extra_context=None, token_generator=default_token_generator): + def register(self, request, node=None, extra_context=None, token_generator=registration_token_generator): if request.user.is_authenticated(): return HttpResponseRedirect(node.get_absolute_url()) @@ -169,13 +250,13 @@ class LoginMultiView(MultiView): if form.is_valid(): user = form.save() current_site = Site.objects.get_current() - token = default_token_generator.make_token(user) + token = token_generator.make_token(user) link = 'http://%s/%s/%s/' % (current_site.domain, node.get_absolute_url().strip('/'), reverse('register_confirm', urlconf=self, kwargs={'uidb36': int_to_base36(user.id), 'token': token}).strip('/')) context = { 'link': link } self.send_confirmation_email('Confirm account creation at %s' % current_site.name, user.email, self.register_confirmation_email, context) - messages.add_message(request, messages.SUCCESS, 'An email has been sent to %s with details on activating your account.' % user.email) + messages.add_message(request, messages.SUCCESS, 'An email has been sent to %s with details on activating your account.' % user.email, fail_silently=True) return HttpResponseRedirect('') else: form = RegistrationForm() @@ -184,7 +265,7 @@ class LoginMultiView(MultiView): context.update(extra_context or {}) return self.register_page.render_to_response(node, request, extra_context=context) - def register_confirm(self, request, node=None, extra_context=None, uidb36=None, token=None): + def register_confirm(self, request, node=None, extra_context=None, uidb36=None, token=None, token_generator=registration_token_generator): """ Checks that a given hash in a registration link is valid and activates the given account. If so, log them in and redirect to @@ -197,7 +278,7 @@ class LoginMultiView(MultiView): raise Http404 user = get_object_or_404(User, id=uid_int) - if default_token_generator.check_token(user, token): + if token_generator.check_token(user, token): user.is_active = True true_password = user.password try: @@ -305,7 +386,7 @@ class AccountMultiView(LoginMultiView): def account_required(self, view): def inner(request, *args, **kwargs): if not self.has_valid_account(request.user): - messages.add_message(request, messages.ERROR, "You need to add some account information before you can post listings.") + messages.add_message(request, messages.ERROR, "You need to add some account information before you can post listings.", fail_silently=True) return self.account_view(request, *args, **kwargs) return view(request, *args, **kwargs) @@ -313,7 +394,7 @@ class AccountMultiView(LoginMultiView): return inner def post_register_confirm_redirect(self, request, node): - messages.add_message(request, messages.INFO, 'Welcome! Please fill in some more information.') + messages.add_message(request, messages.INFO, 'Welcome! Please fill in some more information.', fail_silently=True) return HttpResponseRedirect('/%s/%s/' % (node.get_absolute_url().strip('/'), reverse('account', urlconf=self).strip('/'))) class Meta: