Style fixes plus removal of unused imports, add missing docstring and fix too long lines in soc.views.models.org_app module.
authorPawel Solyga <Pawel.Solyga@gmail.com>
Wed, 04 Mar 2009 19:00:44 +0000
changeset 1645 c7e05dafaede
parent 1644 64ccd9628f15
child 1646 42b8e294792e
Style fixes plus removal of unused imports, add missing docstring and fix too long lines in soc.views.models.org_app module. Patch by: Pawel Solyga Reviewed by: to-be-reviewed
app/soc/views/models/org_app.py
--- a/app/soc/views/models/org_app.py	Wed Mar 04 18:54:39 2009 +0000
+++ b/app/soc/views/models/org_app.py	Wed Mar 04 19:00:44 2009 +0000
@@ -43,8 +43,6 @@
 from soc.views.models import group_app
 from soc.views.models import program as program_view
 
-import soc.logic.dicts
-
 
 class View(group_app.View):
   """View methods for the Organization Application model.
@@ -140,11 +138,15 @@
   @ decorators.merge_params
   def reviewOverview(self, request, access_type,
                page_name=None, params=None, **kwargs):
+    """View that allows to see organization application review overview.
+
+       For Args see base.View.public().
+    """
 
     params['list_template'] = 'soc/org_app/review_overview.html'
     context = {
-        'bulk_accept_link': '/org_app/bulk_accept/%(scope_path)s' %(kwargs),
-        'bulk_reject_link': '/org_app/bulk_reject/%(scope_path)s' %(kwargs),}
+        'bulk_accept_link': '/org_app/bulk_accept/%(scope_path)s' % (kwargs),
+        'bulk_reject_link': '/org_app/bulk_reject/%(scope_path)s' % (kwargs),}
 
     return super(View, self).reviewOverview(request, access_type,
         page_name=page_name, params=params, context=context, **kwargs)
@@ -256,7 +258,8 @@
     # get the program entity from the keyname
     program_entity = program_logic.logic.getFromKeyName(program_keyname)
 
-    # get all the organization applications for the given program and from_status
+    # get all the organization applications for the 
+    # given program and from_status
     filter = {'scope': program_entity,
               'status': from_status}