From 0903ba9c1f1ceaa1ffbb897f6672bf2e65e4c158 Mon Sep 17 00:00:00 2001
From: ichuang <ichuang@mit.edu>
Date: Thu, 19 Jul 2012 11:15:14 -0400
Subject: [PATCH] fix optioninput to preserve author's ordering of options

---
 common/lib/capa/capa/inputtypes.py                       | 3 ++-
 {lms => common/lib/capa/capa}/templates/optioninput.html | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)
 rename {lms => common/lib/capa/capa}/templates/optioninput.html (92%)

diff --git a/common/lib/capa/capa/inputtypes.py b/common/lib/capa/capa/inputtypes.py
index 66ac57c52d..c7a4da4ab4 100644
--- a/common/lib/capa/capa/inputtypes.py
+++ b/common/lib/capa/capa/inputtypes.py
@@ -132,7 +132,8 @@ def optioninput(element, value, status, render_template, msg=''):
     oset = [x[1:-1] for x  in list(oset)]
 
     # osetdict = dict([('option_%s_%s' % (eid,x),oset[x]) for x in range(len(oset)) ])	# make dict with IDs
-    osetdict = dict([(oset[x],oset[x]) for x in range(len(oset)) ])	# make dict with key,value same
+    osetdict = [(oset[x],oset[x]) for x in range(len(oset)) ]	# make ordered list with (key,value) same
+    # TODO: allow ordering to be randomized
     
     context={'id':eid,
              'value':value,
diff --git a/lms/templates/optioninput.html b/common/lib/capa/capa/templates/optioninput.html
similarity index 92%
rename from lms/templates/optioninput.html
rename to common/lib/capa/capa/templates/optioninput.html
index c23f433082..6ceba78c1c 100644
--- a/lms/templates/optioninput.html
+++ b/common/lib/capa/capa/templates/optioninput.html
@@ -2,7 +2,7 @@
   
    <select name="input_${id}" id="input_${id}" >
       <option value="option_${id}_dummy_default">  </option>
-      % for option_id, option_description in options.items():
+      % for option_id, option_description in options:
           <option value="${option_id}"
           % if (option_id==value):
         	selected="true"