Used reverse URL matching instead of hard coded URLs.
authorMadhusudan.C.S <madhusudancs@gmail.com>
Sat, 15 Jan 2011 04:06:45 +0530
changeset 411 edf514c6a820
parent 410 f11529f0e482
child 412 ddd52b52dd9d
Used reverse URL matching instead of hard coded URLs.
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)