Commit 96c37089 by Steve Strassmann

whitespace for pep8

parent c49b1f09
...@@ -23,8 +23,8 @@ from contentstore.utils import get_modulestore, get_lms_link_for_item, \ ...@@ -23,8 +23,8 @@ from contentstore.utils import get_modulestore, get_lms_link_for_item, \
from models.settings.course_grading import CourseGradingModel from models.settings.course_grading import CourseGradingModel
from requests import get_request_method, _xmodule_recurse from .requests import get_request_method, _xmodule_recurse
from access import has_access from .access import has_access
# TODO: should explicitly enumerate exports with __all__ # TODO: should explicitly enumerate exports with __all__
......
...@@ -309,7 +309,7 @@ def course_grader_updates(request, org, course, name, grader_index=None): ...@@ -309,7 +309,7 @@ def course_grader_updates(request, org, course, name, grader_index=None):
# ??? Should this return anything? Perhaps success fail? # ??? Should this return anything? Perhaps success fail?
CourseGradingModel.delete_grader(Location(location), grader_index) CourseGradingModel.delete_grader(Location(location), grader_index)
return HttpResponse() return HttpResponse()
elif request.method == 'POST': # post or put, doesn't matter. elif request.method == 'POST': # post or put, doesn't matter.
return HttpResponse(json.dumps(CourseGradingModel.update_grader_from_json(Location(location), request.POST)), return HttpResponse(json.dumps(CourseGradingModel.update_grader_from_json(Location(location), request.POST)),
mimetype="application/json") mimetype="application/json")
...@@ -347,24 +347,24 @@ def course_advanced_updates(request, org, course, name): ...@@ -347,24 +347,24 @@ def course_advanced_updates(request, org, course, name):
# to edit the combinedopenended or peergrading # to edit the combinedopenended or peergrading
# module, and to remove it if they have removed the open ended elements. # module, and to remove it if they have removed the open ended elements.
if ADVANCED_COMPONENT_POLICY_KEY in request_body: if ADVANCED_COMPONENT_POLICY_KEY in request_body:
#Check to see if the user instantiated any open ended components # Check to see if the user instantiated any open ended components
found_oe_type = False found_oe_type = False
#Get the course so that we can scrape current tabs # Get the course so that we can scrape current tabs
course_module = modulestore().get_item(location) course_module = modulestore().get_item(location)
for oe_type in OPEN_ENDED_COMPONENT_TYPES: for oe_type in OPEN_ENDED_COMPONENT_TYPES:
if oe_type in request_body[ADVANCED_COMPONENT_POLICY_KEY]: if oe_type in request_body[ADVANCED_COMPONENT_POLICY_KEY]:
#Add an open ended tab to the course if needed # Add an open ended tab to the course if needed
changed, new_tabs = add_open_ended_panel_tab(course_module) changed, new_tabs = add_open_ended_panel_tab(course_module)
# If a tab has been added to the course, then send the # If a tab has been added to the course, then send the
# metadata along to CourseMetadata.update_from_json # metadata along to CourseMetadata.update_from_json
if changed: if changed:
request_body.update({'tabs': new_tabs}) request_body.update({'tabs': new_tabs})
#Indicate that tabs should not be filtered out of the metadata # Indicate that tabs should not be filtered out of the metadata
filter_tabs = False filter_tabs = False
#Set this flag to avoid the open ended tab removal code below. # Set this flag to avoid the open ended tab removal code below.
found_oe_type = True found_oe_type = True
break break
#If we did not find an open ended module type in the advanced settings, # If we did not find an open ended module type in the advanced settings,
# we may need to remove the open ended tab from the course. # we may need to remove the open ended tab from the course.
if not found_oe_type: if not found_oe_type:
# Remove open ended tab to the course if needed # Remove open ended tab to the course if needed
......
...@@ -15,6 +15,7 @@ from auth.authz import get_user_by_email, add_user_to_course_group, remove_user_ ...@@ -15,6 +15,7 @@ from auth.authz import get_user_by_email, add_user_to_course_group, remove_user_
from .access import has_access from .access import has_access
from .requests import create_json_response from .requests import create_json_response
def user_author_string(user): def user_author_string(user):
'''Get an author string for commits by this user. Format: '''Get an author string for commits by this user. Format:
first last <email@email.com>. first last <email@email.com>.
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment