Cleaned up overly enthousiastic comments
From the styleguide:
On the other hand, never describe the code. Assume the person reading
the code knows Python (though not what you're trying to do) better
than you do.
Patch by: Sverre Rabbelier
--- a/app/soc/views/models/group.py Thu Jan 29 23:05:37 2009 +0000
+++ b/app/soc/views/models/group.py Thu Jan 29 23:07:04 2009 +0000
@@ -278,25 +278,14 @@
role_descriptions = {}
for role_name in role_views.keys():
- # get the view for a specific role
role_view = role_views[role_name]
-
- # get the params for this view
role_view_params = role_view.getParams()
-
- # get the logic for this view
role_logic = role_view_params['logic']
- # get all the active roles that this user has
roles = role_logic.getForFields(fields)
- # for each role that we find
for role in roles:
-
- # get the Key Name of the group
group_key_name = role.scope.key().name()
-
- # try to get an existing description
existing_role_descriptions = role_descriptions.get(group_key_name)
if existing_role_descriptions: