diff --git a/common/lib/xmodule/xmodule/capa_module.py b/common/lib/xmodule/xmodule/capa_module.py index 386d76696a..d186bcc39c 100644 --- a/common/lib/xmodule/xmodule/capa_module.py +++ b/common/lib/xmodule/xmodule/capa_module.py @@ -202,10 +202,8 @@ class CapaModule(XModule): try: return Progress(score, total) except Exception as err: - # TODO (vshnayder): why is this still here? still needed? - if self.system.DEBUG: - return None - raise + log.exception("Got bad progress") + return None return None def get_html(self): @@ -322,16 +320,10 @@ class CapaModule(XModule): before = self.get_progress() d = handlers[dispatch](get) - try: - after = self.get_progress() - d.update({ - 'progress_changed': after != before, - 'progress_status': Progress.to_js_status_str(after), - }) - except ValueError: - d.update({ - 'progress_changed': False, - 'progress_status': Progress.to_js_status(before), + after = self.get_progress() + d.update({ + 'progress_changed': after != before, + 'progress_status': Progress.to_js_status_str(after), }) return json.dumps(d, cls=ComplexEncoder)