From 1d4990458e5248565f68e59ede55409c2b5a6f0a Mon Sep 17 00:00:00 2001 From: Victor Shnayder Date: Sat, 27 Oct 2012 20:09:02 -0400 Subject: [PATCH] add todos about would-be-nice refactors --- common/lib/capa/capa/inputtypes.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/common/lib/capa/capa/inputtypes.py b/common/lib/capa/capa/inputtypes.py index 0bbf54d153..08cd9ce413 100644 --- a/common/lib/capa/capa/inputtypes.py +++ b/common/lib/capa/capa/inputtypes.py @@ -25,6 +25,13 @@ graded status as'status' # problem ID for the input element, but it will turn into a dict containing both the # answer and any associated message for the problem ID for the input element. +# TODO: there is a lot of repetitive "grab these elements from xml attributes, with these defaults, +# put them in the context" code. Refactor so class just specifies required and optional attrs (with +# defaults for latter), and InputTypeBase does the right thing. + +# TODO: Quoting and unquoting is handled in a pretty ad-hoc way. Also something that could be done +# properly once in InputTypeBase. + import logging import re import shlex # for splitting quoted strings