Bug #7456
Wrong icon for failed quiz
Description
A failed HotPotatoes is indicated by a red cross icon but chamilo's quiz failed is indicated by a green check.
Files
Associated revisions
Fix exercises table - refs #7456
Fix blank page when using hotpotatoe + LP see #7456
If chamilo exercise added in LP has pass % then change status to passed/failed
See #7456
History
Updated by Yannick Warnier about 6 years ago
- Target version changed from 1.9.10 to 1.10.0
Moving to 1.10 because before 1.9 there was never a minimum score.
Updated by Yannick Warnier almost 6 years ago
- Category set to Learning paths / Lecciones
- Status changed from New to Assigned
- Assignee set to Angel Quiroz
Updated by Angel Quiroz almost 6 years ago
- Status changed from Assigned to Needs more info
- Assignee deleted (
Angel Quiroz) - % Done changed from 0 to 50
I've reviewed the code and reproduced the "error" in 1.9.x because in 1.10.x the CSS style still missing.
With a failed HotPotatoes, the item (in lesson item list) is indicated with a red cross. But, with a Chamilo exercise is indicated with a green cross. I guess this is because the exercise should be reviewed by the teacher, then the item is indicated with a green cross for completing the exercise regardless of it's right or wrong
Updated by Andre Boivin almost 6 years ago
Normalement quand un Hotpotatoes est réussit par rapport au pré requis, celui-ci devrait avoir une croix verte, sans intervention de l'enseignant. Sinon les hotpotatoes sont inutiles, autant utiliser un document Word...
(Normally when a HotPotatoes is a success compared to the prerequisites, it should have a green cross without teacher intervention. Otherwise HotPotatoes are useless, so use a Word document ...)
Updated by Yannick Warnier over 5 years ago
- Status changed from Needs more info to Assigned
- Assignee set to Angel Quiroz
Updated by Yannick Warnier over 5 years ago
- Assignee changed from Angel Quiroz to Andre Boivin
André, aurais-tu un petit hotpotatoes utile à tester? J'en ai un qui renvoie toujours 100% :-)
Updated by Andre Boivin over 5 years ago
- File hotpotatoes_test.htm hotpotatoes_test.htm added
- Assignee changed from Andre Boivin to Angel Quiroz
le voici, mais tu devrais aussi avoir 100% ;)
Updated by Yannick Warnier over 5 years ago
- Assignee changed from Angel Quiroz to Yannick Warnier
Le but est d'avoir un hotpotatoes qui me donne un score insuffisant pour pouvoir voir la croix rouge, non?
Updated by Andre Boivin over 5 years ago
- Assignee changed from Yannick Warnier to Angel Quiroz
Effectivement mais à la condition de mettre un prérequis sur l'élément suivant. Avec le hotpotatoes que je t'ai envoyé, met en prérequis l'élément suivant dans le parcours avec un seuil de 100% et lorsque l'élève complète le quiz avec une faute, un X rouge devrait s'afficher.
Updated by Yannick Warnier over 5 years ago
- Assignee changed from Angel Quiroz to Julio Montoya
Julio, je te réassigne la tâche. Elle demande une certaine concentration et beaucoup d'attention pour ne pas casser le reste de SCORM.
Updated by Julio Montoya over 5 years ago
- File Selection_011.png Selection_011.png added
- File Selection_012.png Selection_012.png added
- Status changed from Assigned to Needs more info
- Assignee changed from Julio Montoya to Andre Boivin
Je pense que le problème est dans le hotpotatoes (fichier html)
Quand j'ai un erreur (voir image) le fonction "Finish()" (voir fichier main/exercice/showinframes.php) n'est pas appelé:
Par contre quand je réponds correctement la fonction "Finish()" est appelé.
La fonction "Finish()" modifie le résultat, donc le statut (failed/passed).
Updated by Yannick Warnier over 5 years ago
- Status changed from Needs more info to Assigned
- Assignee changed from Andre Boivin to Julio Montoya
- al incluir el hotpotatoes en ejemplo dentro de una lección SCORM, ya no aparece para nada (página blanca pero sin error JS)
- el último comentario de André decía que cuando se incluye en una lección y el usuario falla, debería poner una marca roja. Para este segundo punto creo que tienes la razón al decir que no envía la señal finish, pero por ahora no lo puedo probar por el primer problema (que quizás esté relacionado con lo de https de Laurent?)
Updated by Julio Montoya over 5 years ago
Yannick Warnier wrote:
Julio, el problema ahora es doble:
- al incluir el hotpotatoes en ejemplo dentro de una lección SCORM, ya no aparece para nada (página blanca pero sin error JS)
Localmente si puedo ver el hotpotatoe. En unstable.chamilo aparece una pag en blanco.
- el último comentario de André decía que cuando se incluye en una lección y el usuario falla, debería poner una marca roja. Para este segundo punto creo que tienes la razón al decir que no envía la señal finish, pero por ahora no lo puedo probar por el primer problema (que quizás esté relacionado con lo de https de Laurent?)
Updated by Julio Montoya over 5 years ago
- Status changed from Assigned to Needs testing
- Assignee changed from Julio Montoya to Yannick Warnier
Julio Montoya wrote:
Yannick Warnier wrote:
Julio, el problema ahora es doble:
- al incluir el hotpotatoes en ejemplo dentro de una lección SCORM, ya no aparece para nada (página blanca pero sin error JS)
Localmente si puedo ver el hotpotatoe. En unstable.chamilo aparece una pag en blanco.
Acabo de corregir el problema de pag en blanco.
Updated by Yannick Warnier over 5 years ago
- Assignee changed from Yannick Warnier to Julio Montoya
- Priority changed from Normal to High
- que les hotpotatoes se montrent bien dans les parcours
- quand le hotpotatoe d'exemple est réussi, celui-ci renvoie le signal de fin heureuse et l'étape se marque comme complétée
- quand le hotpotatoe d'exemple est échoué, apparemment celui-ci ne renvoie aucun signal (il devrait renvoyer un truc du genre "failed"), et du coup on ne peut pas mettre l'étape en rouge
- par contre, la logique n'est pas bonne quand c'est un vrai quiz de Chamilo: s'il existe un c_quiz.pass_percentage au test et que celui-ci n'est pas atteint, Chamilo devrait automatiquement mettre le statut à scorm_failed. Or pour l'instant il le met à scorm_completed, même si c'est râté. Il doit y avoir du code dans main/exercices/ qui réagit quand c'est la fin du quiz (exercise_result.php?) et qui simule un message SCORM... pour une raison où l'autre, on dirait que ce message ne passe plus... Dans lp_view.php et lp_ajax_save_item.php, on devrait avoir une relation avec l'exercice.
- par ailleurs, quand on clique sur "Terminer" dans le test, la petite icône de disquette n'apparaît pas. On dirait que le chemin pour y accéder est mauvais dans le cas où on est dans un parcours (learnpath_id != 0 ?)
Julio, pourrais-tu t'occuper des deux points ci-dessus?
Updated by Julio Montoya over 5 years ago
- Assignee changed from Julio Montoya to Yannick Warnier
Yannick Warnier wrote:
Bien, donc comme résultat, on a:
- que les hotpotatoes se montrent bien dans les parcours
- quand le hotpotatoe d'exemple est réussi, celui-ci renvoie le signal de fin heureuse et l'étape se marque comme complétée
- quand le hotpotatoe d'exemple est échoué, apparemment celui-ci ne renvoie aucun signal (il devrait renvoyer un truc du genre "failed"), et du coup on ne peut pas mettre l'étape en rouge
- par contre, la logique n'est pas bonne quand c'est un vrai quiz de Chamilo: s'il existe un c_quiz.pass_percentage au test et que celui-ci n'est pas atteint, Chamilo devrait automatiquement mettre le statut à scorm_failed. Or pour l'instant il le met à scorm_completed, même si c'est râté. Il doit y avoir du code dans main/exercices/ qui réagit quand c'est la fin du quiz (exercise_result.php?) et qui simule un message SCORM... pour une raison où l'autre, on dirait que ce message ne passe plus... Dans lp_view.php et lp_ajax_save_item.php, on devrait avoir une relation avec l'exercice.
Corrigé.
- par ailleurs, quand on clique sur "Terminer" dans le test, la petite icône de disquette n'apparaît pas. On dirait que le chemin pour y accéder est mauvais dans le cas où on est dans un parcours (learnpath_id != 0 ?)
Je vois bien l'icône de disquette en local.
Updated by Yannick Warnier over 5 years ago
- Status changed from Needs testing to Bug resolved
- Assignee changed from Yannick Warnier to Julio Montoya
- % Done changed from 50 to 100
Ça marche pour moi!
Updated by Andre Boivin over 5 years ago
Est-ce possible de connaitre la liste des fichiers modifiés?
Updated by Yannick Warnier over 5 years ago
Toujours. C'est pour ça qu'on utilise des codes de référence entre nos changements et les tâches. Tu peux retrouver les codes juste en dessous debla description initiale de la tâche, alignés à droite. En suivant les liens, tu trouveras tous les fichiers modifîés.
Tu peux aussi les trouver directement sur github qui est plus rapide pour analyser le code (mais il faut chercher les commits indiqués ci-dessus)
Fix E_NOTICE - refs #7456