Project

General

Profile

Feature #5239

Réinitialisation d'un parcours / Reset learning path

Added by Alain Deschênes almost 7 years ago. Updated almost 7 years ago.

Status:
Feature implemented
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
23/07/2012
Due date:
% Done:

100%

Estimated time:
Spent time:
Complexity:
Normal
SCRUM pts - complexity:
?

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

Suivi_LP.pdf (378 KB) Suivi_LP.pdf Alain Deschênes, 26/07/2012 16:21

Associated revisions

Revision 6b5beaf2 (diff)
Added by Julio Montoya almost 7 years ago

Fixing validation in order to delete LP attempts see #5239

Revision 929fbdce (diff)
Added by Julio Montoya almost 7 years ago

Deleting oLP and lpobject when listing the LPs (code commented) see #5239

Revision bd616065 (diff)
Added by Julio Montoya almost 7 years ago

Deleting olp session when viewing the LP list + fixing latest view item see #5239

Revision de6e92d3 (diff)
Added by Julio Montoya almost 7 years ago

Undo delete LP session see #5239

History

#1

Updated by Yannick Warnier almost 7 years ago

  • Target version set to 1.9 RC1
#2

Updated by Yannick Warnier almost 7 years ago

  • Target version changed from 1.9 RC1 to 1.9 Stable
#3

Updated by Julio Montoya almost 7 years ago

  • Status changed from New to Assigned
  • Assignee set to Julio Montoya
#4

Updated by Julio Montoya almost 7 years ago

For some reason you're right :) testing ...

#5

Updated by Julio Montoya almost 7 years ago

je pense que la variable $_SESSION['olp'] n'est pas supprimé quand on va dans la list des LPs ...

#6

Updated by Julio Montoya almost 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)
  • % Done changed from 0 to 60

should work now

#7

Updated by Alain Deschênes almost 7 years ago

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

#8

Updated by Yannick Warnier almost 7 years ago

  • Status changed from Needs testing to Assigned
  • Assignee set to Julio Montoya

Il reste probablement encore un petit truc dans $_SESSION[]

#9

Updated by Julio Montoya almost 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

#10

Updated by Yannick Warnier almost 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.

#11

Updated by Yannick Warnier almost 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.

#12

Updated by Julio Montoya almost 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)

#13

Updated by Julio Montoya almost 7 years ago

This change is encore there:

http://support.chamilo.org/projects/chamilo-18/repository/revisions/19048/diff/main/newscorm/learnpath.class.php

I don't think this cause a slow behaviour, just saying

#14

Updated by Alain Deschênes almost 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.

#15

Updated by Yannick Warnier almost 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.

#16

Updated by Yannick Warnier almost 7 years ago

  • Tracker changed from Bug to Feature
#17

Updated by Julio Montoya almost 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:

http://code.google.com/p/chamilo/source/detail?r=d9819a8a6fd696cbe822f3d0f13ea4c69ab2540b&repo=classic

Also available in: Atom PDF