From afa2b5ca9e29a5f2f3dfa8f94a5be2f9e612d719 Mon Sep 17 00:00:00 2001
From: jordan <jordan@0cfe37f9-358a-4d5e-be75-b63607b5c754>
Date: Thu, 15 Dec 2011 19:46:30 +0000
Subject: [PATCH] making the user edit url contain the user slug

git-svn-id: http://svn.osqa.net/svnroot/osqa/trunk@1212 0cfe37f9-358a-4d5e-be75-b63607b5c754
---
 forum/registry.py                             | 5 ++++-
 forum/skins/default/templates/users/edit.html | 2 +-
 forum/urls.py                                 | 2 +-
 forum/views/users.py                          | 2 +-
 4 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/forum/registry.py b/forum/registry.py
index 4c8df65..871f1e5 100644
--- a/forum/registry.py
+++ b/forum/registry.py
@@ -71,7 +71,10 @@ class SuperUserSwitchMenuItem(ui.UserMenuItem):
 ui.register(ui.USER_MENU,
             ui.UserMenuItem(
                 label=_("edit profile"),
-                url=lambda u, c: reverse('edit_user', kwargs={'id': c['user'].id}),
+                url=lambda u, c: reverse('edit_user', kwargs={
+                    'id': c['user'].id,
+                    'slug' : slugify(c['user'].username)
+                }),
                 span_attrs={'class': 'user-edit'},
                 weight=0,
                 name='EDIT_PROFILE'
diff --git a/forum/skins/default/templates/users/edit.html b/forum/skins/default/templates/users/edit.html
index 09292fe..ce54c77 100644
--- a/forum/skins/default/templates/users/edit.html
+++ b/forum/skins/default/templates/users/edit.html
@@ -33,7 +33,7 @@
     {{ user.username }} - {% trans "edit profile" %}
 </div>
 <div id="main-body" style="width:100%;padding-top:10px">
-    <form name="" action="{% url edit_user user.id %}" method="post">
+    <form name="" action="{% url edit_user user.id user.username|slugify %}" method="post">
         {% csrf_token %}
         <div id="left" style="float:left;width:180px">
             {% if user.email %}
diff --git a/forum/urls.py b/forum/urls.py
index 3d202c7..fd82f7f 100644
--- a/forum/urls.py
+++ b/forum/urls.py
@@ -100,7 +100,7 @@ core_urls = (
     url(r'^%s$' % _('users/'), app.users.users, name='users'),
     # url(r'^%s$' % _('online_users/'), app.users.online_users, name='online_users'),    
     
-    url(r'^%s(?P<id>\d+)/%s$' % (_('users/'), _('edit/')), app.users.edit_user, name='edit_user'),
+    url(r'^%s(?P<id>\d+)/(?P<slug>.*)/%s$' % (_('users/'), _('edit/')), app.users.edit_user, name='edit_user'),
     url(r'^%s(?P<id>\d+)/%s$' % (_('users/'), _('award/')), app.users.award_points, name='user_award_points'),
     url(r'^%s(?P<id>\d+)/%s$' % (_('users/'), _('suspend/')), app.users.suspend, name='user_suspend'),
     url(r'^%s(?P<id>\d+)/%s(?P<action>[a-z]+)/(?P<status>[a-z]+)/$' % (_('users/'), _('powers/')), app.users.user_powers, name='user_powers'),
diff --git a/forum/views/users.py b/forum/views/users.py
index bc0abc3..db65003 100644
--- a/forum/views/users.py
+++ b/forum/views/users.py
@@ -131,7 +131,7 @@ def online_users(request):
     })
 
 
-def edit_user(request, id):
+def edit_user(request, id, slug):
     user = get_object_or_404(User, id=id)
     if not (request.user.is_superuser or request.user == user):
         return HttpResponseUnauthorized(request)
-- 
2.39.5