Skip to content

Comments

Fix conditional check for quiz result validation#6745

Merged
ywarnier merged 1 commit intochamilo:1.11.xfrom
LudiscapeXApi:patch-33
Feb 24, 2026
Merged

Fix conditional check for quiz result validation#6745
ywarnier merged 1 commit intochamilo:1.11.xfrom
LudiscapeXApi:patch-33

Conversation

@LudiscapeXApi
Copy link
Contributor

Le contrôle de la valeur max n'a pas de sens si la valeur minimum est déjà atteinte.

Le contrôle de la valeur max n'a pas de sens si la valeur minimum est déjà atteinte.
@ywarnier
Copy link
Member

ywarnier commented Nov 9, 2025

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".

@LudiscapeXApi
Copy link
Contributor Author

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.

ywarnier added a commit to ywarnier/chamilo-lms that referenced this pull request Feb 23, 2026
… capping against exe_weighting, maxScore check is now null-safe and optional, all values cast to float before comparison) - refs chamilo#6745
@ywarnier
Copy link
Member

@LudiscapeXApi Juste pour être sûr de bien comprendre, tu parles d'un cas où, par exemple :

  • le test a un score maximum de 10
  • on met un pré-requis minimum de 7
  • on change ensuite le score maximum du test à 5
  • le pré-requis ne change pas automatiquement et quand l'apprenant obtient le max au test, à 5/5, il n'atteint pas le pré-requis donc ne peut pas continuer

C'est ça ?

Parce que dans le cas opposé :

  • le test a un score maximum de 10
  • on met un pré-requis minimum de 7
  • quelqu'un passe le test et obtient 8
  • on change ensuite le score maximum du test à 20
  • on change le pré-requis à 14
  • l'apprenant a toujours 7 et ne peut donc pas passer

... 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.

@LudiscapeXApi
Copy link
Contributor Author

LudiscapeXApi commented Feb 23, 2026

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.
En fait le score maximum est stocké mais inutil même au niveau de l'interface UI de l'édition de prérequis. Le meilleur exemple pour le e-learning est le "mastery score" du scorm : une valeur unique minimum de validation.

ywarnier added a commit to ywarnier/chamilo-lms that referenced this pull request Feb 24, 2026
ywarnier added a commit that referenced this pull request Feb 24, 2026
Learnpath: Remove validation of quiz score against max score (only min score matters) - refs #6745
@ywarnier ywarnier merged commit cf75d0f into chamilo:1.11.x Feb 24, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants