# HG changeset patch # User diegoe-guest # Date 1244055980 0 # Node ID 274a6d5b1926ba022b405f986666e11058f46d4b # Parent df0a699fac3195309d8ed791ce95abeb8aaf98ea views: PEP 8, reindentation, other fixes diff --git a/bts_webui/amancay/views.py b/bts_webui/amancay/views.py --- a/bts_webui/amancay/views.py +++ b/bts_webui/amancay/views.py @@ -12,7 +12,7 @@ from bts_webui.amancay.models import Package # Needed for AJAX -from django.utils import simplejson +from django.utils import simplejson # Needed for SOAP from bts_queries import SoapQueries @@ -21,37 +21,38 @@ from tables import process_post, selected_bugs from search import search -# The index page doesn't really do much. def index(request): - item_list = process_post(request) - # Check if it's AJAX or HTML - if (request.GET.has_key('xhr')): - return HttpResponse( simplejson.dumps({"item_list": item_list}), - mimetype='application/javascript' ) - else: - # TODO: choose which view to show. - return search(request) - - -# Package page -def package(request, package_name): - user = request.user - queries = SoapQueries() + """ + Our pretty useless index page. + """ + item_list = process_post(request) + # Check if it's AJAX or HTML + if request.GET.has_key('xhr'): + return HttpResponse(simplejson.dumps({"item_list": item_list}), + mimetype='application/javascript') + else: + # TODO: choose which view to show. + return search(request) - bugs = queries.get_packages_bugs(package_name) - bugs.sort(reverse=True) - bug_list = queries.get_bugs_status(bugs) +def package(request, package_name): + """ + Individual package page. + """ + user = request.user + queries = SoapQueries() + + bugs = queries.get_packages_bugs(package_name) + bugs.sort(reverse=True) + bug_list = queries.get_bugs_status(bugs) - # Check if it's AJAX or HTML - if (request.GET.has_key('xhr')): - return HttpResponse( simplejson.dumps({"package": - package_name, "bug_list": bug_list}), - mimetype='application/javascript' ) - else: - return render_to_response('package.html', - {'package': package_name, - 'bug_list': bug_list, - 'current_user': user} - ) - - + # Check if it's AJAX or HTML + if request.GET.has_key('xhr'): + return HttpResponse(simplejson.dumps({"package": package_name, + "bug_list": bug_list}), + mimetype='application/javascript') + else: + return render_to_response('package.html', + {'package': package_name, + 'bug_list': bug_list, + 'current_user': user} + )