- fixed finished topic was not being saved in the database

Miguel Barão
1 parent 45cffaba
Showing 3 changed files with 6 additions and 2 deletions Show diff stats
BUGS.md
knowledge.py
learnapp.py
@@ -34,6 +34,7 @@
 # FIXED
+- não está a guardar o resultado no final do topico
 - esta a permitir 2 logins em simultaneo do mesmo user. fica tudo baralhado se mxerem em simultaneo...
 - errar no ultimo topico nao mostra solucao?
 - quando a pergunta devolve comments, este é apresentado, mas fica persistente nas tentativas seguintes. devia ser limpo apos a segunda submissao.
@@ -177,6 +177,9 @@ class StudentKnowledge(object):
     # pure functions of the state (no side effects)
     # ========================================================================
+    def topic_has_finished(self):
+        return self.current_question is None
+
     # ------------------------------------------------------------------------
     # compute recommended sequence of topics  ['a', 'b', ...]
     # ------------------------------------------------------------------------
@@ -81,7 +81,7 @@ class LearnApp(object):
                 return False
         pw_ok = await check_password(try_pw, password)  # async bcrypt
-        
+
         if pw_ok:
             if uid in self.online:
                 logger.warning(f'User "{uid}" already logged in, overwriting')
@@ -153,7 +153,7 @@ class LearnApp(object):
                 topic_id=topic))
             logger.debug(f'Saved "{q["ref"]}" into database')
-        if action == 'finished_topic':
+        if knowledge.topic_has_finished():
             # finished topic, save into database
             logger.info(f'User "{uid}" finished "{topic}"')
             level = knowledge.get_topic_level(topic)
	@@ -34,6 +34,7 @@		@@ -34,6 +34,7 @@
34		34
35	# FIXED	35	# FIXED
36		36
		37	+- não está a guardar o resultado no final do topico
37	- esta a permitir 2 logins em simultaneo do mesmo user. fica tudo baralhado se mxerem em simultaneo...	38	- esta a permitir 2 logins em simultaneo do mesmo user. fica tudo baralhado se mxerem em simultaneo...
38	- errar no ultimo topico nao mostra solucao?	39	- errar no ultimo topico nao mostra solucao?
39	- quando a pergunta devolve comments, este é apresentado, mas fica persistente nas tentativas seguintes. devia ser limpo apos a segunda submissao.	40	- quando a pergunta devolve comments, este é apresentado, mas fica persistente nas tentativas seguintes. devia ser limpo apos a segunda submissao.
	@@ -81,7 +81,7 @@ class LearnApp(object):		@@ -81,7 +81,7 @@ class LearnApp(object):
81	return False	81	return False
82		82
83	pw_ok = await check_password(try_pw, password) # async bcrypt	83	pw_ok = await check_password(try_pw, password) # async bcrypt
84	-	84	+
85	if pw_ok:	85	if pw_ok:
86	if uid in self.online:	86	if uid in self.online:
87	logger.warning(f'User "{uid}" already logged in, overwriting')	87	logger.warning(f'User "{uid}" already logged in, overwriting')
	@@ -153,7 +153,7 @@ class LearnApp(object):		@@ -153,7 +153,7 @@ class LearnApp(object):
153	topic_id=topic))	153	topic_id=topic))
154	logger.debug(f'Saved "{q["ref"]}" into database')	154	logger.debug(f'Saved "{q["ref"]}" into database')
155		155
156	- if action == 'finished_topic':	156	+ if knowledge.topic_has_finished():
157	# finished topic, save into database	157	# finished topic, save into database
158	logger.info(f'User "{uid}" finished "{topic}"')	158	logger.info(f'User "{uid}" finished "{topic}"')
159	level = knowledge.get_topic_level(topic)	159	level = knowledge.get_topic_level(topic)