Fix conditional check for quiz result validation#6745
Fix conditional check for quiz result validation#6745ywarnier merged 1 commit intochamilo:1.11.xfrom
Conversation
Le contrôle de la valeur max n'a pas de sens si la valeur minimum est déjà atteinte.
|
Pas très utile, d'accord, mais dans quel cas est-ce que le score max obtenu peut être supérieur au score max prévu ? À mon sens, cela révèle un risque ailleurs, mais je suis d'accord que dans ce cas il vaudrait peut être mieux l'enregistrer comme un log d'événement inquiétant plutôt que de considérer que, du coup, l'exercice n'est pas complet. Il reste un risque, selon moi, que la donnée reçue ne soit pas un nombre mais soit finalement interprétée comme un nombre qui est supérieur au max. Dans ce cas, pas sûr que la valeur doive être acceptée comme "succès au test". |
|
Explication du cas : Les quiz intégrés dans les learning paths peuvent être modifiés au fil du temps grâce à une option disponible dans le fichier configuration.php. Ces modifications peuvent entraîner des variations du nombre de points attribués. Dans ce contexte, un score devenu supérieur peut provoquer un blocage au niveau des prérequis des LP. |
… capping against exe_weighting, maxScore check is now null-safe and optional, all values cast to float before comparison) - refs chamilo#6745
|
@LudiscapeXApi Juste pour être sûr de bien comprendre, tu parles d'un cas où, par exemple :
C'est ça ? Parce que dans le cas opposé :
... la procédure, dans ce cas-ci, devrait être de recalculer le score de l'apprenant une fois le test modifié, et il pourra passer à l'item suivant de parcours. |
|
Dans ce cas aussi, vérifier la valeur maximale n’a pas de sens : si un étudiant obtient 21/20, son test est validé, et s’il a 7/20, cette limite ne change rien. Cela reste vrai, même en attendant le recalcul par l’administrateur. |
Learnpath: Remove validation of quiz score against max score (only min score matters) - refs #6745
Le contrôle de la valeur max n'a pas de sens si la valeur minimum est déjà atteinte.