Address comments to r844 and r845.
authorPawel Solyga <Pawel.Solyga@gmail.com>
Sun, 19 Oct 2008 17:33:44 +0000
changeset 391 849aa913e9c8
parent 390 d12c95ade374
child 392 76f130cc16b0
Address comments to r844 and r845. Patch by: Pawel Solyga Review by: to-be-reviewed
app/soc/models/base.py
app/soc/models/task.py
app/soc/views/user/profile.py
--- a/app/soc/models/base.py	Sun Oct 19 16:17:32 2008 +0000
+++ b/app/soc/models/base.py	Sun Oct 19 17:33:44 2008 +0000
@@ -27,7 +27,7 @@
 
 from google.appengine.ext import db
 
-from soc.views.helper import forms
+from soc.views.helper import forms as forms_helper
 
 
 class ModelWithFieldAttributes(db.Model):
@@ -69,7 +69,7 @@
       Property itself via the Model entity.
     """
     if not cls._fields_cache or (cls != cls._fields_cache.__class__.Meta.model):
-      class FieldsProxy(forms.DbModelForm):
+      class FieldsProxy(forms_helper.DbModelForm):
         """Form used as a proxy to access User model properties attributes.
         """
       
@@ -81,4 +81,4 @@
       
       cls._fields_cache = FieldsProxy()
 
-    return cls._fields_cache
+    return cls._fields_cache
\ No newline at end of file
--- a/app/soc/models/task.py	Sun Oct 19 16:17:32 2008 +0000
+++ b/app/soc/models/task.py	Sun Oct 19 17:33:44 2008 +0000
@@ -44,9 +44,9 @@
   """
  
   #: A required many:1 relationship with the Proposal on which the
-  #: Task is based.  A Task may be based on only a single Proposal, 
+  #: Task is based.  A Task may be based on only a single Proposal,
   #: but a Proposal can be the foundation for multiple Tasks. The
-  #: back-reference in the Proposal model is a Query named 'tasks'.  
+  #: back-reference in the Proposal model is a Query named 'tasks'.
   proposal = db.ReferenceProperty(reference_class=soc.models.proposal.Proposal,
                                   required=True,
                                   collection_name="tasks")
--- a/app/soc/views/user/profile.py	Sun Oct 19 16:17:32 2008 +0000
+++ b/app/soc/views/user/profile.py	Sun Oct 19 17:33:44 2008 +0000
@@ -107,10 +107,10 @@
     # not logged in, and no link name, so request that the user sign in 
     return simple.requestLogin(request, page, template, context,
         # TODO(tlarsen): /user/profile could be a link to a help page instead
-        login_message_fmt=ugettext_lazy('To create a new'
-                          ' <a href="/user/profile">User Profile</a>'
-                          ' or modify an existing one, you must first'
-                          ' <a href="%(sign_in)s">sign in</a>.'))
+        login_message_fmt=ugettext_lazy(
+            'To create a new <a href="/user/profile">User Profile</a>'
+            ' or modify an existing one, you must first'
+            ' <a href="%(sign_in)s">sign in</a>.'))
 
   if (not id) and link_name:
     # not logged in, so show read-only public profile for link_name user