diff --git a/common/lib/capa/capa/inputtypes.py b/common/lib/capa/capa/inputtypes.py index 31482214b3..8b3867be5b 100644 --- a/common/lib/capa/capa/inputtypes.py +++ b/common/lib/capa/capa/inputtypes.py @@ -313,14 +313,20 @@ def textbox(element, value, status, render_template, msg=''): size = element.get('size') rows = element.get('rows') or '30' cols = element.get('cols') or '80' - mode = element.get('mode') or 'python' # mode for CodeMirror, eg "python" or "xml" hidden = element.get('hidden', '') # if specified, then textline is hidden and id is stored in div of name given by hidden - linenumbers = element.get('linenumbers','true') # for CodeMirror + if not value: value = element.text # if no student input yet, then use the default input given by the problem + + # For CodeMirror + mode = element.get('mode') or 'python' # mode, eg "python" or "xml" + linenumbers = element.get('linenumbers','true') # for CodeMirror + tabsize = element.get('tabsize','4') + tabsize = int(tabsize) + context = {'id': eid, 'value': value, 'state': status, 'count': count, 'size': size, 'msg': msg, 'mode': mode, 'linenumbers': linenumbers, 'rows': rows, 'cols': cols, - 'hidden': hidden, + 'hidden': hidden, 'tabsize': tabsize, } html = render_template("textbox.html", context) try: diff --git a/common/lib/capa/capa/responsetypes.py b/common/lib/capa/capa/responsetypes.py index 944b84bf61..75ed92ab3e 100644 --- a/common/lib/capa/capa/responsetypes.py +++ b/common/lib/capa/capa/responsetypes.py @@ -811,7 +811,7 @@ class CodeResponse(LoncapaResponse): def setup_response(self): xml = self.xml self.url = xml.get('url', "http://107.20.215.194/xqueue/submit/") # FIXME -- hardcoded url - self.queue_name = xml.get('queuename', 'python') # TODO: Default queue_name should be course-specific + self.queue_name = xml.get('queuename', self.system.xqueue_default_queuename) answer = xml.find('answer') if answer is not None: @@ -905,7 +905,7 @@ class CodeResponse(LoncapaResponse): def _send_to_queue(self, extra_payload): # Prepare payload xmlstr = etree.tostring(self.xml, pretty_print=True) - header = {'return_url': self.system.xqueue_callback_url, + header = {'lms_callback_url': self.system.xqueue_callback_url, 'queue_name': self.queue_name, } @@ -914,7 +914,7 @@ class CodeResponse(LoncapaResponse): h.update(str(self.system.seed)) h.update(str(time.time())) queuekey = int(h.hexdigest(), 16) - header.update({'queuekey': queuekey}) + header.update({'lms_key': queuekey}) body = {'xml': xmlstr, 'edX_cmd': 'get_score', diff --git a/common/lib/capa/capa/templates/textbox.html b/common/lib/capa/capa/templates/textbox.html index d553ba16e5..d37eda7284 100644 --- a/common/lib/capa/capa/templates/textbox.html +++ b/common/lib/capa/capa/templates/textbox.html @@ -35,15 +35,20 @@ lineNumbers: true, % endif mode: "${mode}", - tabsize: 4, + matchBrackets: true, + lineWrapping: true, + indentUnit: "${tabsize}", + tabSize: "${tabsize}", + smartIndent: false }); }); diff --git a/common/lib/xmodule/xmodule/x_module.py b/common/lib/xmodule/xmodule/x_module.py index 3406bcb99c..7bb98dcdc5 100644 --- a/common/lib/xmodule/xmodule/x_module.py +++ b/common/lib/xmodule/xmodule/x_module.py @@ -587,7 +587,7 @@ class ModuleSystem(object): def __init__(self, ajax_url, track_function, get_module, render_template, replace_urls, user=None, filestore=None, debug=False, - xqueue_callback_url=None): + xqueue_callback_url=None, xqueue_default_queuename="null"): ''' Create a closure around the system environment. @@ -616,6 +616,7 @@ class ModuleSystem(object): ''' self.ajax_url = ajax_url self.xqueue_callback_url = xqueue_callback_url + self.xqueue_default_queuename = xqueue_default_queuename self.track_function = track_function self.filestore = filestore self.get_module = get_module diff --git a/lms/djangoapps/courseware/module_render.py b/lms/djangoapps/courseware/module_render.py index bf7d1f4c51..80a4ef90fc 100644 --- a/lms/djangoapps/courseware/module_render.py +++ b/lms/djangoapps/courseware/module_render.py @@ -140,8 +140,16 @@ def get_module(user, request, location, student_module_cache, position=None): # TODO (vshnayder): fix hardcoded urls (use reverse) # Setup system context for module instance ajax_url = settings.MITX_ROOT_URL + '/modx/' + descriptor.location.url() + '/' - xqueue_callback_url = (settings.MITX_ROOT_URL + '/xqueue/' + - str(user.id) + '/' + descriptor.location.url() + '/') + + # Fully qualified callback URL for external queueing system + xqueue_callback_url = (request.build_absolute_uri('/') + settings.MITX_ROOT_URL + + 'xqueue/' + str(user.id) + '/' + descriptor.location.url() + '/' + + 'score_update') + + # Default queuename is course-specific and is derived from the course that + # contains the current module. + # TODO: Queuename should be derived from 'course_settings.json' of each course + xqueue_default_queuename = descriptor.location.org + '-' + descriptor.location.course def _get_module(location): (module, _, _, _) = get_module(user, request, location, @@ -155,6 +163,7 @@ def get_module(user, request, location, student_module_cache, position=None): render_template=render_to_string, ajax_url=ajax_url, xqueue_callback_url=xqueue_callback_url, + xqueue_default_queuename=xqueue_default_queuename.replace(' ','_'), # TODO (cpennington): Figure out how to share info between systems filestore=descriptor.system.resources_fs, get_module=_get_module, @@ -203,13 +212,12 @@ def get_module(user, request, location, student_module_cache, position=None): return (module, instance_module, shared_module, descriptor.category) -# TODO: TEMPORARY BYPASS OF AUTH! @csrf_exempt def xqueue_callback(request, userid, id, dispatch): # Parse xqueue response get = request.POST.copy() try: - header = json.loads(get.pop('xqueue_header')[0]) # 'dict' + header = json.loads(get['xqueue_header']) except Exception as err: msg = "Error in xqueue_callback %s: Invalid return format" % err raise Exception(msg) @@ -230,7 +238,7 @@ def xqueue_callback(request, userid, id, dispatch): # Transfer 'queuekey' from xqueue response header to 'get'. This is required to # use the interface defined by 'handle_ajax' - get.update({'queuekey': header['queuekey']}) + get.update({'queuekey': header['lms_key']}) # We go through the "AJAX" path # So far, the only dispatch from xqueue will be 'score_update'