# HG changeset patch # User Madhusudan.C.S # Date 1295044605 -19800 # Node ID edf514c6a820bd9b10ebef51c74169216f41a224 # Parent f11529f0e48215da987181bbf189aabeba1912d2 Used reverse URL matching instead of hard coded URLs. diff -r f11529f0e482 -r edf514c6a820 pytask/profile/views.py --- a/pytask/profile/views.py Sat Jan 15 04:06:30 2011 +0530 +++ b/pytask/profile/views.py Sat Jan 15 04:06:45 2011 +0530 @@ -1,5 +1,6 @@ from django.contrib.auth.decorators import login_required from django.core.context_processors import csrf +from django.core.urlresolvers import reverse from django.http import Http404 from django.shortcuts import redirect from django.shortcuts import render_to_response @@ -41,7 +42,7 @@ if form.is_valid(): form.save() - return redirect("/profile/view") + return redirect(reverse('view_profile')) else: context.update({"form":form}) return render_to_response("profile/edit.html", context) @@ -57,7 +58,8 @@ user = request.user - active_notifications = user.notification_sent_to.filter(is_deleted=False).order_by('sent_date').reverse() + active_notifications = user.notification_sent_to.filter( + is_deleted=False).order_by('-sent_date') context = {'user':user, 'notifications':active_notifications, @@ -104,18 +106,11 @@ notification.is_deleted = True notification.save() - context = {'user':user, - 'notification':notification, - 'newest':newest, - 'newer':newer, - 'older':older, - 'oldest':oldest, - } - if older: - redirect_url = "/profile/notf/view/nid=%s"%older.uniq_key + redirect_url = reverse('view_notification', + kwargs={'notification_id': older.id}) else: - redirect_url = "/profile/notf/browse" + redirect_url = reverse('browse_notifications') return redirect(redirect_url) @@ -134,18 +129,11 @@ notification.is_read = False notification.save() - context = {'user':user, - 'notification':notification, - 'newest':newest, - 'newer':newer, - 'older':older, - 'oldest':oldest, - } - if older: - redirect_url = "/profile/notf/view/nid=%s"%older.uniq_key + redirect_url = reverse('view_notification', + kwargs={'notification_id': older.id}) else: - redirect_url = "/profile/notf/browse" + redirect_url = reverse('browse_notifications') return redirect(redirect_url)