Fix text-regex questions due to upgrade to pyyaml 5.1.

Type declaration !regex is no longer required in questions.yaml files.

Fix text-regex questions due to upgrade to pyyaml 5.1.
Type declaration !regex is no longer required in questions.yaml files.
Miguel Barão
1 parent 5635b91a
Showing 4 changed files with 4 additions and 12 deletions Show diff stats
BUGS.md
aprendizations/knowledge.py
aprendizations/questions.py
demo/solar_system/questions.yaml
  
 # BUGS
  
-- testar se perguntas regex funcionam com yaml.safe_load.
 - safari as vezes envia dois gets no inicio do topico. nesses casos, a segunda pergunta não é actualizada no browser... o topico tem de ser gerado qd se escolhe o topico em main_topics. O get nao deve alterar o estado.
 - click numa opcao checkbox fora da checkbox+label não está a funcionar.
 - shift-enter não está a funcionar
@@ -71,13 +71,14 @@ class StudentKnowledge(object):
     # ------------------------------------------------------------------------
     # FIXME async mas nao tem awaits... do not allow restart same topic
     async def start_topic(self, topic):
-        logger.debug(f'StudentKnowledge.start_topic({topic})')
+        logger.debug('StudentKnowledge.start_topic()')
         if self.current_topic == topic:
+            logger.debug('  Restarting current topic is not allowed.')
             return False
  
         # do not allow locked topics
         if self.is_locked(topic):
-            logger.debug(f'Topic {topic} is locked')
+            logger.debug(f'  Topic {topic} is locked')
             return False
  
         # starting new topic
@@ -6,17 +6,9 @@ from os import path
 import logging
 import asyncio
  
-# user installed libraries
-import yaml
-
 # this project
 from aprendizations.tools import run_script
  
-
-# regular expressions in yaml files, e.g.   correct: !regex '[aA]zul'
-yaml.add_constructor('!regex', lambda l, n: re.compile(l.construct_scalar(n)))
-
-
 # setup logger for this module
 logger = logging.getLogger(__name__)
  
@@ -44,7 +44,7 @@
   ref: saturn
   title: Sistema solar
   text: O planeta do sistema solar conhecido por ter aneis é o planeta...
-  correct: !regex '[Ss]aturno'
+  correct: '[Ss]aturno'
   solution: |
     O planeta Saturno é famoso pelos seus anéis.
     É o segundo maior planeta do Sistema Solar, a seguir a Júpiter.
1	1
2	2	# BUGS
3	3
4		-- testar se perguntas regex funcionam com yaml.safe_load.
5	4	- safari as vezes envia dois gets no inicio do topico. nesses casos, a segunda pergunta não é actualizada no browser... o topico tem de ser gerado qd se escolhe o topico em main_topics. O get nao deve alterar o estado.
6	5	- click numa opcao checkbox fora da checkbox+label não está a funcionar.
7	6	- shift-enter não está a funcionar
...	...
...	...	@@ -71,13 +71,14 @@ class StudentKnowledge(object):
71	71	# ------------------------------------------------------------------------
72	72	# FIXME async mas nao tem awaits... do not allow restart same topic
73	73	async def start_topic(self, topic):
74		- logger.debug(f'StudentKnowledge.start_topic({topic})')
	74	+ logger.debug('StudentKnowledge.start_topic()')
75	75	if self.current_topic == topic:
	76	+ logger.debug(' Restarting current topic is not allowed.')
76	77	return False
77	78
78	79	# do not allow locked topics
79	80	if self.is_locked(topic):
80		- logger.debug(f'Topic {topic} is locked')
	81	+ logger.debug(f' Topic {topic} is locked')
81	82	return False
82	83
83	84	# starting new topic
...	...
...	...	@@ -6,17 +6,9 @@ from os import path
6	6	import logging
7	7	import asyncio
8	8
9		-# user installed libraries
10		-import yaml
11		-
12	9	# this project
13	10	from aprendizations.tools import run_script
14	11
15		-
16		-# regular expressions in yaml files, e.g. correct: !regex '[aA]zul'
17		-yaml.add_constructor('!regex', lambda l, n: re.compile(l.construct_scalar(n)))
18		-
19		-
20	12	# setup logger for this module
21	13	logger = logging.getLogger(__name__)
22	14
...	...
...	...	@@ -44,7 +44,7 @@
44	44	ref: saturn
45	45	title: Sistema solar
46	46	text: O planeta do sistema solar conhecido por ter aneis é o planeta...
47		- correct: !regex '[Ss]aturno'
	47	+ correct: '[Ss]aturno'
48	48	solution: \|
49	49	O planeta Saturno é famoso pelos seus anéis.
50	50	É o segundo maior planeta do Sistema Solar, a seguir a Júpiter.
...	...