app/soc/views/user/profile.py
changeset 118 d2e61a490969
parent 99 8c38b546a3cf
child 124 051afb721c22
--- a/app/soc/views/user/profile.py	Thu Aug 28 22:23:11 2008 +0000
+++ b/app/soc/views/user/profile.py	Thu Aug 28 22:27:40 2008 +0000
@@ -14,7 +14,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-"""Views relevant to the User role.
+"""Views for editing and examining User profiles.
 """
 
 __authors__ = [
@@ -28,11 +28,13 @@
 from django import shortcuts
 from django import newforms as forms
 
+from soc.logic.site import id_user
 from soc.views.helpers import forms_helpers
 from soc.views.helpers import response_helpers
 
 import soc.models.user
 
+
 class UserForm(forms_helpers.DbModelForm):
   """Django form displayed when creating or editing a User.
   """
@@ -46,7 +48,6 @@
       )*        # zero or more of OR group
       $'''
   LINKNAME_REGEX = re.compile(LINKNAME_PATTERN)
-
   
   class Meta:
     """Inner Meta class that defines some behavior for the form.
@@ -59,10 +60,10 @@
   
   def clean_link_name(self):
     linkname = self.cleaned_data.get('link_name')
-    linkname_user = soc.models.user.User.getUserForLinkname(linkname)
-    current_id = users.get_current_user()
+    linkname_user = id_user.getUserFromLinkName(linkname)
+    id = users.get_current_user()
     # if linkname exist in datastore and doesn't belong to current user
-    if linkname_user and (linkname_user.id != current_id):
+    if linkname_user and (linkname_user.id != id):
       raise forms.ValidationError("This link name is already in use.")
     elif not self.LINKNAME_REGEX.match(linkname):
       raise forms.ValidationError("This link name is in wrong format.")
@@ -99,15 +100,15 @@
   elif not current_id and linkname:
     return public(request, linkname)
     
-  user = soc.models.user.User.getUserForId(current_id)
+  user = id_user.getUserFromId(current_id)
   
   #: Show custom 404 page when linkname doesn't exist in datastore
   #: or show public view for linkname user
   if linkname:
-    linkname_user = soc.models.user.User.getUserForLinkname(linkname)
+    linkname_user = id_user.getUserFromLinkName(linkname)
     if not linkname_user:
       return http.HttpResponseNotFound('No user exists with that link name "%s"' %
-                                linkname)
+                                       linkname)
     elif linkname_user and (linkname_user.id != current_id):
       return public(request, linkname)
 
@@ -117,8 +118,7 @@
                              'link_name': user.link_name})
     return response_helpers.respond(request,
         template, {'template': template, 
-                   'form': form, 
-                   'user': user})
+                   'form': form})
   
   #: POST method
   form = UserForm()
@@ -129,16 +129,15 @@
       linkname = form.cleaned_data.get('link_name')
       nickname = form.cleaned_data.get("nick_name")
       if not user:
-        user = soc.models.user.User(id = user,link_name = linkname,
-                                        nick_name = nickname)
+        user = soc.models.user.User(id=id,link_name=linkname,
+                                    nick_name=nickname)
       else:
         user.nick_name = nickname
         user.link_name = linkname
       user.put()
       return response_helpers.respond(request,
               template, {'template': template, 
-                         'form': form, 
-                         'user': user,
+                         'form': form,
                          'submit_message': 'Profile saved.'})
 
   return response_helpers.respond(request,
@@ -160,14 +159,14 @@
   #: If linkname is empty or not a valid linkname on the site, display
   #: "user does not exist", otherwise render public view for linkname user
   if linkname:
-    linkname_user = soc.models.user.User.getUserForLinkname(linkname)
+    linkname_user = id_user.getUserFromLinkName(linkname)
     if not linkname_user:
       return http.HttpResponseNotFound('No user exists with that link name "%s"' %
-                                linkname)
+                                       linkname)
     else:
       return response_helpers.respond(request, 
           template, {'template': template,
                      'user': linkname_user})
       
   return http.HttpResponseNotFound('No user exists with that link name "%s"' %
-                            linkname)
+                                   linkname)