Feature #5239
Réinitialisation d'un parcours / Reset learning path
Description
Le "Reset Learning Path (Réinitialisation du parcours)" ne fonctionne pas.
Bien qu'après avoir cliqué sur le petit balais et qu'il affiche un % d'avancement du parcours à 0%, lorsqu'on retourne dans le parcours, aucune réinitialisation ne s'est réalisée et Chamilo revient au % d'avancement d'avant le reset.
Testé sur Windows 7 - IE9 / Firefox 14 / Chrome 20 à partir de la version Nightly 22 juillet et sur Chamilodev.beeznest.com
Alain
Files
Associated revisions
Deleting oLP and lpobject when listing the LPs (code commented) see #5239
Deleting olp session when viewing the LP list + fixing latest view item see #5239
Undo delete LP session see #5239
History
Updated by
Updated by Julio Montoya over 7 years ago
Updated by - Status changed from New to Assigned
- Assignee set to Julio Montoya
Updated by Julio Montoya over 7 years ago
je pense que la variable $_SESSION['olp'] n'est pas supprimé quand on va dans la list des LPs ...
Updated by Julio Montoya over 7 years ago
- Status changed from Assigned to Needs testing
- Assignee deleted (
Julio Montoya) - % Done changed from 0 to 60
should work now
Updated by Alain Deschênes over 7 years ago
Updated by - File Suivi_LP.pdf Suivi_LP.pdf added
Fonctionne, mais si l'utilisateur reste dans Chamilo, il ne revient pas au début de son parcours mais plutôt à l'endroit où il était rendu.
Par contre, s'il se déconnecte et se reconnecte à Chamilo et que l'enseignant a fait un reset du LP entre temps, ça fonctionne bien.
Voir le document en pièce jointe : Suivi LP.pdf
Updated by Yannick Warnier over 7 years ago
- Status changed from Needs testing to Assigned
- Assignee set to Julio Montoya
Il reste probablement encore un petit truc dans $_SESSION[]
Updated by Julio Montoya over 7 years ago
- Status changed from Assigned to Needs testing
- Assignee deleted (
Julio Montoya) - % Done changed from 60 to 70
je viens de réparer ca, par contre il faut que l'étudiant entre dans la liste des LP (lp_list.php).
Si on veut que cela marche mieux il faudra que j'appelle toujours à la BD quand je rentre pour la première fois dans un LP et pas la session
Updated by Yannick Warnier over 7 years ago
Julio A. Montoya wrote:
[...] et pas la session [...]
Pour l'instant c'est appelé un fois par session? Pour moi c'est bon de l'appeler quand tu rentres dans le LP, mais j'ai peur que, du coup, il soit appelé à chaque item.
Updated by Yannick Warnier over 7 years ago
- Status changed from Needs testing to Assigned
- Assignee set to Julio Montoya
Got a report from André Boivin saying now the learning paths are dead slow. This is probably due to this change. Julio, I've assigned you the other task (in our private system) to check on that.
I guess my last suggestion to always clear the session wasn't that good after all.
Updated by Julio Montoya over 7 years ago
- Status changed from Assigned to Needs testing
- Assignee deleted (
Julio Montoya) - % Done changed from 70 to 80
ok, I removed those recent changes (Session:erase)
Updated by Julio Montoya over 7 years ago
This change is encore there:
I don't think this cause a slow behaviour, just saying
Updated by Alain Deschênes over 7 years ago
Pour ma part, pour l'instant je peux très bien vivre avec la fonctionnalité actuelle - Reset LP - logout - login de l'utilisateur visé, ce qui fonctionne bien.
D'autre part, je prèfère de beaucoup la version 1.9.0 qui nous ramène au même item où nous étions au moment de quitter le LP, contrairement à la version 1.8.8.4 qui nous ramenait à l'item suivant (Ce qui supposait que nous avions terminé l'item au moment de quitter, ce qui n'est pas nécessairement vrai).
Aucun problème de vitesse constaté dans l'accès aux parcours suite à des essais avec la version.
Testé sur la version nitghtly 28 juillet.
Updated by Yannick Warnier over 7 years ago
- Target version changed from 1.9 Stable to 1.9.2
I'm moving this task to 1.9.2. We have to figure out what caused the slowliness for one of our users, but we won't be able to review that before the release. The initial bug here was solved, anyway.
Updated by Julio Montoya over 7 years ago
- Status changed from Needs testing to Feature implemented
- % Done changed from 80 to 100
I think the slow LP was fixed in another task, adding 2 indexes resolve this issue
see:
Fixing validation in order to delete LP attempts see #5239