- fixed FileHandler

- fixed enter-submit issue

- fixed FileHandler
- fixed enter-submit issue
Miguel Barão
1 parent 68528695
Showing 4 changed files with 12 additions and 18 deletions Show diff stats
BUGS.md
knowledge.py
serve.py
templates/learn.html
 BUGS:
-- enter faz GET /question, que responde com json no ecran
+- aumentar espaço a seguir às tabelas no texto
+- guardar state cada vez que topico termina
 - tabs em textarea nao funcionam correctamente (insere 1 espaco em vez de 4)
 - reportar comentarios após submeter.
-- os topicos locked devem estar inactivos no sidebar.
 - textarea deve mostrar no html os valores iniciais de ans, se existir
 - detect questions in questions.yaml without ref -> error ou generate default.
 - error if demo.yaml has no topics
-- guardar state cada vez que topico termina
 - session management. close after inactive time.
 - implementar xsrf. Ver [http://www.tornadoweb.org/en/stable/guide/security.html#cross-site-request-forgery-protection]()
 - titulos das perguntas não suportam markdown
+- generators not working: bcrypt (ver blog)
 TODO:
 - pymips: activar/desactivar instruções
 - implementar servidor http com redirect para https.
 - usar codemirror no textarea
-- generators not working: bcrypt (ver blog)
 FIXED:
+- os topicos locked devem estar inactivos no sidebar.
+- enter faz GET /question, que responde com json no ecran. (solution: disabled enter)
 - topicos no sidebar devem ser links para iniciar um topico acessivel.
 - logs inicio de topico
 - indicar o topico actual no sidebar
@@ -88,12 +88,9 @@ class Knowledge(object):
         # new question if answer is correct
         if grade > 0.999:
             self.finished_questions.append(q)
-            print('questions: ', self.questions)
-            print('finished: ', self.finished_questions)
             try:
                 self.current_question = self.questions.pop(0) # FIXME empty?
             except IndexError:
-                print('no more questions!')
                 self.current_question = None
                 self.state[self.current_topic] = {
                     'level': 1.0,
@@ -103,10 +100,8 @@ class Knowledge(object):
             else:
                 self.current_question['start_time'] = datetime.now()
         else:
-            # FIXME debug this
             factory = self.depgraph.node[self.current_topic]['factory']
             self.questions.append(factory[q['ref']].generate())
-            print([q['ref'] for q in self.questions])
         return q
@@ -135,11 +130,6 @@ class Knowledge(object):
             if t in self.state:
                 ts.append((t, self.state[t]['level']))   # already done
             else:
-                # deps = self.depgraph.predecessors(t)
-                # # print(t, deps)
-                # if all(d in self.state for d in deps):
-                #     ts.append((t, 0.0))   # unlocked not yet done
-                # else:
                 ts.append((t, None))  # locked
         return ts
@@ -134,11 +134,14 @@ class FileHandler(BaseHandler):
     def get(self, filename):
         uid = self.current_user
         public_dir = self.learn.get_current_public_dir(uid)
+        filepath = os.path.expanduser(os.path.join(public_dir, filename))
         try:
-            with open(os.path.join(public_dir, filename), 'rb') as f:
-                self.write(f.read())
+            f = open(filepath, 'rb')
         except FileNotFoundError:
             raise tornado.web.HTTPError(404)
+        else:
+            self.write(f.read())
+            f.close()
 # ----------------------------------------------------------------------------
 # respond to AJAX to get a JSON question
@@ -165,9 +165,9 @@ function updateQuestion(response){
         // enable shift+enter to submit and tab to spaces conversion
         $("input:text, input:radio, input:checkbox").keydown(function (e) {
-            if (e.keyCode == 13 && e.shiftKey) {
+            if (e.keyCode == 13) {
                 e.preventDefault();
-                postQuestion();
+                if (e.shiftKey) postQuestion();
                 return false;
             }
         });
	@@ -165,9 +165,9 @@ function updateQuestion(response){		@@ -165,9 +165,9 @@ function updateQuestion(response){
165		165
166	// enable shift+enter to submit and tab to spaces conversion	166	// enable shift+enter to submit and tab to spaces conversion
167	$("input:text, input:radio, input:checkbox").keydown(function (e) {	167	$("input:text, input:radio, input:checkbox").keydown(function (e) {
168	- if (e.keyCode == 13 && e.shiftKey) {	168	+ if (e.keyCode == 13) {
169	e.preventDefault();	169	e.preventDefault();
170	- postQuestion();	170	+ if (e.shiftKey) postQuestion();
171	return false;	171	return false;
172	}	172	}
173	});	173	});