From ed1f4bdd85f9c42370cf6644667d813b41eba9ee Mon Sep 17 00:00:00 2001 From: Victor Shnayder Date: Mon, 10 Dec 2012 14:10:19 -0500 Subject: [PATCH] Check for none. --- common/lib/capa/capa/responsetypes.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/common/lib/capa/capa/responsetypes.py b/common/lib/capa/capa/responsetypes.py index d6be8e041a..a3dd1f274b 100644 --- a/common/lib/capa/capa/responsetypes.py +++ b/common/lib/capa/capa/responsetypes.py @@ -1852,6 +1852,10 @@ class OpenEndedResponse(LoncapaResponse): oeparam = self.xml.find('openendedparam') prompt = self.xml.find('prompt') rubric = self.xml.find('openendedrubric') + + if not oeparam or not prompt or not rubric: + raise ValueError("openendedresponse missing required parameters.") + self._parse(oeparam, prompt, rubric) @staticmethod @@ -1861,7 +1865,7 @@ class OpenEndedResponse(LoncapaResponse): in order to avoid capa depending on xmodule (seems to be avoided in code) """ - parts=[node.text] + parts=[node.text if node.text is not None else ''] for p in node.getchildren(): parts.append(etree.tostring(p, with_tail=True, encoding='unicode'))