minor

Miguel Barão
1 parent b73ff7c9
Showing 3 changed files with 6 additions and 7 deletions Show diff stats
aprendizations/learnapp.py
aprendizations/serve.py
aprendizations/student.py
@@ -20,6 +20,7 @@ from .questions import Question, QFactory, QDict
 from .student import StudentState
 from .tools import load_yaml
+
 # setup logger for this module
 logger = logging.getLogger(__name__)
@@ -277,7 +278,7 @@ class LearnApp(object):
             logger.warning(f'"{uid}" could not start course "{course}": {e}')
             raise
         else:
-            logger.info(f'"{uid}" started course "{course}"')
+            logger.info(f'User "{uid}" started course "{course}"')
     # ------------------------------------------------------------------------
     # Start new topic
@@ -19,6 +19,7 @@ from tornado.escape import to_unicode
 from .tools import md_to_html
 from . import APP_NAME
+
 # setup logger for this module
 logger = logging.getLogger(__name__)
@@ -326,15 +327,13 @@ class QuestionHandler(BaseHandler):
     # --- post answer, returns what to do next: shake, new_question, finished
     @tornado.web.authenticated
     async def post(self) -> None:
-        logger.debug('[QuestionHandler.post]')
         user = self.current_user
         answer = self.get_body_arguments('answer')  # list
-        logger.debug(f'user = {user}, answer = {answer}')
+        qid = self.get_body_arguments('qid')[0]
+        logger.debug(f'[QuestionHandler.post] {user}, {qid}, answer={answer}')
         # --- check if browser opened different questions simultaneously
-        answer_qid = self.get_body_arguments('qid')[0]
-        current_qid = self.learn.get_current_question_id(user)
-        if answer_qid != current_qid:
+        if qid != self.learn.get_current_question_id(user):
             logger.info(f'User {user} desynchronized questions')
             self.write({
                 'method': 'invalid',
@@ -141,7 +141,6 @@ class StudentState(object):
                 action = 'wrong'
                 if self.current_question['append_wrong']:
                     logger.debug('wrong answer, append new question')
-                    # self.questions.append(self.factory[q['ref']].generate())
                     new_question = await self.factory[q['ref']].gen_async()
                     self.questions.append(new_question)
                 self.next_question()