# HG changeset patch # User Sverre Rabbelier # Date 1233270424 0 # Node ID 4fd9b11fba562187639170cb5e3fcc0bca09e8bd # Parent 94bc2a9ae103ce47220c9cd4b8940cc68dc372fb 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 diff -r 94bc2a9ae103 -r 4fd9b11fba56 app/soc/views/models/group.py --- 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: