Mercurial Mercurial > py_tasks_melange / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | zip | gz | bz2 | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Fix some missing dots in access and club_admin modules.
authorPawel Solyga <Pawel.Solyga@gmail.com>
Sat, 24 Jan 2009 12:51:47 +0000
changeset 958 b4309e3cb899
parent 957 5b657063b23b
child 959 b6e8750c9407
Fix some missing dots in access and club_admin modules. Patch by: Pawel Solyga Review by: to-be-reviewed
app/soc/views/helper/access.py file | annotate | diff | comparison | revisions
app/soc/views/models/club_admin.py file | annotate | diff | comparison | revisions 
--- a/app/soc/views/helper/access.py	Sat Jan 24 12:50:59 2009 +0000
+++ b/app/soc/views/helper/access.py	Sat Jan 24 12:51:47 2009 +0000
@@ -109,7 +109,7 @@
 
 
 def allow(request, args, kwargs):
-  """Never raises an alternate HTTP response.  (an access no-op, basically)
+  """Never raises an alternate HTTP response.  (an access no-op, basically).
 
   Args:
     request: a Django HTTP request
@@ -257,9 +257,9 @@
     if user_entity.link_id != kwargs['link_id']:
       deny(request, args, kwargs)
 
-    fields = {'link_id' : kwargs['link_id'],
-        'scope_path' : kwargs['scope_path'],
-        'role' : role_name}
+    fields = {'link_id': kwargs['link_id'],
+        'scope_path': kwargs['scope_path'],
+        'role': role_name}
 
     request_entity = request_logic.getFromFieldsOr404(**fields)
 
@@ -283,9 +283,9 @@
     # not the current user's request
     return deny(request, args, kwargs)
 
-  fields = {'link_id' : kwargs['link_id'],
-            'scope_path' : kwargs['scope_path'],
-            'role' : kwargs['role']}
+  fields = {'link_id': kwargs['link_id'],
+            'scope_path': kwargs['scope_path'],
+            'role': kwargs['role']}
 
   request_entity = request_logic.getForFields(fields, unique=True)
 
@@ -325,8 +325,8 @@
   user = user_logic.getForFields({'account': users.get_current_user()},
                                  unique=True)
 
-  fields = {'user' : user,
-            'state' : 'active'}
+  fields = {'user': user,
+            'state': 'active'}
 
   host = host_logic.getForFields(fields, unique=True)
 
@@ -358,9 +358,9 @@
   user = user_logic.getForFields({'account': users.get_current_user()},
                                  unique=True)
 
-  fields = {'user' : user,
-            'scope_path' : kwargs['scope_path'],
-            'state' : 'active'}
+  fields = {'user': user,
+            'scope_path': kwargs['scope_path'],
+            'state': 'active'}
 
   host = host_logic.getForFields(fields, unique=True)
 
@@ -405,9 +405,9 @@
   else:
     scope_path = kwargs['link_id']
 
-  fields = {'user' : user,
-            'scope_path' : scope_path,
-            'state' : 'active'}
+  fields = {'user': user,
+            'scope_path': scope_path,
+            'state': 'active'}
 
   club_admin_entity = club_admin_logic.getForFields(fields, unique=True)
 
@@ -465,8 +465,8 @@
 
 
 def checkIsMyNotification(request, args, kwargs):
-  """Returns an alternate HTTP response if this request is for a Notification belonging
-     to the current user.
+  """Returns an alternate HTTP response if this request is for 
+     a Notification belonging to the current user.
 
   Args:
     request: a Django HTTP request
@@ -509,8 +509,8 @@
 
 
 def checkIsMyApplication(app_logic):
-  """Returns an alternate HTTP response if this request is for a Application belonging
-     to the current user.
+  """Returns an alternate HTTP response if this request is for 
+     a Application belonging to the current user.
 
   Args:
     request: a Django HTTP request
@@ -578,8 +578,8 @@
       # not my role
       deny(request, args, kwargs)
 
-    fields = {'link_id' : kwargs['link_id'],
-              'scope_path' : kwargs['scope_path']
+    fields = {'link_id': kwargs['link_id'],
+              'scope_path': kwargs['scope_path']
               }
 
     role_entity = role_logic.logic.getForFields(fields, unique=True)
@@ -641,7 +641,7 @@
 
 
 def checkHasPickGetArgs(request, arg, kwargs):
-  """Raises an alternate HTTP response if the request misses get args
+  """Raises an alternate HTTP response if the request misses get args.
 
   Args:
     request: a Django HTTP request
--- a/app/soc/views/models/club_admin.py	Sat Jan 24 12:50:59 2009 +0000
+++ b/app/soc/views/models/club_admin.py	Sat Jan 24 12:51:47 2009 +0000
@@ -100,7 +100,7 @@
     super(View, self)._editPost(request, entity, fields)
 
   def _acceptInvitePost(self, fields, request, context, params, **kwargs):
-    """Fills in the fields that were missing in the invited_created_form
+    """Fills in the fields that were missing in the invited_created_form.
     
     For params see base.View._acceptInvitePost()
     """
py_tasks_melange