Commit d36a0c3e4cee117120bbf68f54af1e6a9668345a

Authored by Miguel Barão
1 parent 4c4ed8e4
Exists in master and in 1 other branch dev

- review of tests is working, but images are not showing.

BUGS.md
1 1  
2 2 # BUGS
3 3  
  4 +- a revisao do teste não mostra as imagens.
4 5 - se aluno tem teste activo e é allowed uma segunda vez, deve manter o mesmo teste. adicionar opcao para eliminar um teste em curso.
5 6 - melhorar o botao de autorizar (desliga-se), usar antes um botao?
6 7 e.g. retornar None quando nao ha alteracoes relativamente à última vez.
... ... @@ -8,7 +9,6 @@ ou usar push (websockets?)
8 9 - pymips: nao pode executar syscalls do spim.
9 10 - perguntas checkbox [right,wrong] com pelo menos uma opção correcta.
10 11 - questions.py textarea has a abspath which does not make sense! why is it there? not working for perguntations, but seems to work for aprendizations
11   -- servir imagens das perguntas não funciona. Necessario passar a ref da pergunta no link para poder ajustar o path no FileHandler. Tem o problema do teste em JSON não conter a imagem.
12 12 - o eventloop está a bloquear. correção do teste é blocking. usar threadpoolexecutor?
13 13 - submissao faz um post ajax.
14 14 - eventos unfocus?
... ... @@ -61,6 +61,8 @@ ou usar push (websockets?)
61 61  
62 62 # FIXED
63 63  
  64 +- review nao esta a funcionar
  65 +- servir imagens das perguntas não funciona. Necessario passar a ref da pergunta no link para poder ajustar o path no FileHandler.
64 66 - a primeira coluna da tabela admin deveria estar sempre ordenada.
65 67 - abortar depois de testar todas as perguntas, caso haja algum erro.
66 68 - imagens jpg/png nas perguntas.
... ...
serve.py
... ... @@ -235,6 +235,7 @@ class ReviewHandler(BaseHandler):
235 235 raise tornado.web.HTTPError(404)
236 236  
237 237 test_id = self.get_query_argument('test_id', None)
  238 + logging.info(f'Review test {test_id}.')
238 239 try:
239 240 fname = self.testapp.get_json_filename_of_test(test_id)
240 241 except:
... ...
templates/review.html
... ... @@ -10,7 +10,7 @@
10 10 <script defer type="text/x-mathjax-config">
11 11 MathJax.Hub.Config({
12 12 tex2jax: {
13   - inlineMath: [["$$$","$$$"], ["$","$"], ["\(","\)"]]
  13 + inlineMath: [["$$$","$$$"], ["\(","\)"]]
14 14 }
15 15 });
16 16 </script>
... ... @@ -29,6 +29,8 @@
29 29 </head>
30 30 <!-- ===================================================================== -->
31 31 <body>
  32 +<!-- ===================================================================== -->
  33 +
32 34 <nav class="navbar navbar-expand-sm fixed-top navbar-dark bg-danger">
33 35 <a class="navbar-brand" href="#">
34 36 <img src="/static/logo_horizontal.png" height="30" alt="">
... ... @@ -103,9 +105,10 @@
103 105 </h3>
104 106 </div> <!-- jumbotron -->
105 107  
106   - {% for i, q in enumerate(t['questions']) %}
107   - {% module Template(templ[q['type']], i=i, q=q, md=md, t=t) %}
108   - {% end %}
  108 + {% for i, q in enumerate(t['questions']) %}
  109 + {% module Template(templ[q['type']], i=i, q=q, md=md(q['ref']), t=t) %}
  110 + {% end %}
  111 +
109 112 </div> <!-- container -->
110 113 </body>
111 114 </html>
... ...
templates/test.html
... ... @@ -36,6 +36,7 @@
36 36 </head>
37 37 <!-- ===================================================================== -->
38 38 <body>
  39 +<!-- ===================================================================== -->
39 40  
40 41 <nav class="navbar navbar-expand-sm fixed-top navbar-dark bg-dark">
41 42 <a class="navbar-brand" href="#">
... ... @@ -70,8 +71,6 @@
70 71 </div>
71 72 </nav>
72 73  
73   -
74   -
75 74 <!-- ===================================================================== -->
76 75 <div class="container">
77 76  
... ...