diff --git a/common/lib/xmodule/xmodule/capa_module.py b/common/lib/xmodule/xmodule/capa_module.py index a03c0f4160..792860e642 100644 --- a/common/lib/xmodule/xmodule/capa_module.py +++ b/common/lib/xmodule/xmodule/capa_module.py @@ -517,8 +517,14 @@ class CapaModule(CapaFields, XModule): return False def is_completed(self): - # used by conditional module - # return self.answer_available() + """ Used to decide to show or hide RESET or CHECK buttons. + + Actually means that student submitted problem and nothing more. + Problem can be completely wrong. + Pressing RESET button makes this function to return False. + Suggestion: rename it to is_submitted. + + # older comment: return self.answer_available()""" return self.lcp.done def is_attempted(self): diff --git a/common/lib/xmodule/xmodule/conditional_module.py b/common/lib/xmodule/xmodule/conditional_module.py index 9fda387ecb..080a7c48ea 100644 --- a/common/lib/xmodule/xmodule/conditional_module.py +++ b/common/lib/xmodule/xmodule/conditional_module.py @@ -70,8 +70,18 @@ class ConditionalModule(ConditionalFields, XModule): # value: conditions_map = { 'poll_answer': 'poll_answer', # poll_question attr + + # problem was submitted (it can be wrong) + # if student will press reset button after that, + # state will be reverted 'completed': 'is_completed', # capa_problem attr + + # if student attempted problem 'attempted': 'is_attempted', # capa_problem attr + + # if problem is full points + 'correct': 'is_correct', + 'voted': 'voted' # poll_question attr }