chamilo / chamilo-lms

Chamilo is a learning management system focused on ease of use and accessibility
https://chamilo.org
GNU General Public License v3.0
798 stars 480 forks source link

Dans un scorm 1.2, le masteryscore semble ignoré #4331

Open noeldieschburg opened 2 years ago

noeldieschburg commented 2 years ago

Décrivez le problème Même si j'obtien un score > mastery_score dans un scorm mon status ect completed et non passed.

Comment le reproduire J'importe un scorm v1.2 avec un masteryscore de 80%

Comportement attendu Mon statut devrait être completed.

Captures d'écran pas utile

Bureau (à compléter au mieux): /

**Serveur /

Contexte additionnel En inspectant le code il se fait que le code gérant cela dans la fonction set_score (main/lp/learnpathItem.class.php) est commenté depuis 2012 (https://github.com/chamilo/chamilo-lms/commit/5b05845be124aa6151cb0a7bec3634aa08b66c23) Est-ce normal?

Cordialement,

Noel

NicoDucou commented 1 year ago

En effet cette modification avait été faite temporairement et il était prévu de rajouter une option pour l'activé ou la désactivé car certain logiciel d'édition de Scorm envoyé tout le temps un mastery score même si celui-ci n'était pas nécessaire et donc ça générait des erreurs. L'option n'a pas été mise en oeuvre et le code est resté ainsi depuis 10 ans sans que ça ne pose de problème. Il faudrait donc reprendre cette partie de code pour rajouter l'option de configuration de Chamilo et adapter les variables car ce code n'est pas à jour, Par exemple $master correspond à $masteryScore maintenant.

@ywarnier tu veux que l'on rajoute cette option et avec quel comportement par défaut ? Je pense que oui et que le comportement par défaut devrait être de prendre en compte le mastery score s'il est configuré.