app/soc/views/helper/access.py
changeset 958 b4309e3cb899
parent 950 8f59df49985b
child 960 129efa976d6d
--- 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