diff --git a/common/lib/xmodule/xmodule/capa_module.py b/common/lib/xmodule/xmodule/capa_module.py index b7f8e68b5e..3cafcbde3d 100644 --- a/common/lib/xmodule/xmodule/capa_module.py +++ b/common/lib/xmodule/xmodule/capa_module.py @@ -141,7 +141,7 @@ class CapaModule(XModule): msg = '

%s

' % msg.replace('<', '<') msg += '

%s

' % traceback.format_exc().replace('<', '<') # create a dummy problem with error message instead of failing - problem_text = 'Problem file %s has an error:%s' % (self.filename, msg) + problem_text = 'Problem %s has an error:%s' % (self.location.url(), msg) self.lcp = LoncapaProblem(problem_text, self.location.html_id(), instance_state, seed=seed, system=self.system) else: raise @@ -194,7 +194,7 @@ class CapaModule(XModule): as necessary based on the problem config and state.''' html = self.lcp.get_html() - content = {'name': self.name, + content = {'name': self.metadata['display_name'], 'html': html, 'weight': self.weight, } @@ -368,7 +368,7 @@ class CapaModule(XModule): ''' event_info = dict() event_info['state'] = self.lcp.get_state() - event_info['filename'] = self.filename + event_info['problem_id'] = self.location.url() answers = self.make_dict_of_responses(get) @@ -394,13 +394,13 @@ class CapaModule(XModule): correct_map = self.lcp.grade_answers(answers) except StudentInputError as inst: # TODO (vshnayder): why is this line here? - self.lcp = LoncapaProblem(self.system.filestore.open(self.filename).read(), + self.lcp = LoncapaProblem(self.definition['data'], id=lcp_id, state=old_state, system=self.system) traceback.print_exc() return {'success': inst.message} except: # TODO: why is this line here? - self.lcp = LoncapaProblem(self.system.filestore.open(self.filename).read(), + self.lcp = LoncapaProblem(self.definition['data'], id=lcp_id, state=old_state, system=self.system) traceback.print_exc() raise Exception("error in capa_module") @@ -434,7 +434,7 @@ class CapaModule(XModule): ''' event_info = dict() event_info['state'] = self.lcp.get_state() - event_info['filename'] = self.filename + event_info['problem_id'] = self.location.url() answers = self.make_dict_of_responses(get) event_info['answers'] = answers @@ -468,7 +468,7 @@ class CapaModule(XModule): ''' event_info = dict() event_info['old_state'] = self.lcp.get_state() - event_info['filename'] = self.filename + event_info['problem_id'] = self.location.url() if self.closed(): event_info['failure'] = 'closed' @@ -485,7 +485,7 @@ class CapaModule(XModule): # reset random number generator seed (note the self.lcp.get_state() in next line) self.lcp.seed = None - self.lcp = LoncapaProblem(self.system.filestore.open(self.filename).read(), + self.lcp = LoncapaProblem(self.definition['data'], self.location.html_id(), self.lcp.get_state(), system=self.system) event_info['new_state'] = self.lcp.get_state() diff --git a/common/lib/xmodule/xmodule/modulestore/__init__.py b/common/lib/xmodule/xmodule/modulestore/__init__.py index f9bc675116..686ea22658 100644 --- a/common/lib/xmodule/xmodule/modulestore/__init__.py +++ b/common/lib/xmodule/xmodule/modulestore/__init__.py @@ -122,7 +122,7 @@ class Location(_LocationBase): return "-".join(str(v) for v in self.list() if v is not None).replace('.', '_') def dict(self): - return self.__dict__ + return self._asdict() def list(self): return list(self) diff --git a/lms/templates/problem.html b/lms/templates/problem.html index 3f79be3b3a..36fcb1bc66 100644 --- a/lms/templates/problem.html +++ b/lms/templates/problem.html @@ -1,7 +1,7 @@ <%namespace name='static' file='static_content.html'/>

${ problem['name'] } - % if problem['weight']: + % if problem['weight'] != 1: : ${ problem['weight'] } points % endif