Project

General

Profile

Bug #5855

Affichage du LP (tool) en mode "Vue apprenant" (Suite de la demande 5593)

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

Status:
Bug resolved
Priority:
Normal
Category:
Learning paths / Lecciones
Target version:
Start date:
28/12/2012
Due date:
% Done:

100%

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

Description

En mode "Vue apprenant" lors de l'édition d'un cours et qu'on clique sur un icône de LP sur la page d'accueil du cours, on n'est pas dirigé sur la bonne page.

On clique sur l'icône du LP

Par contre, lorsque connecté en tant qu'apprenant, tout fonctionne bien.

Testé sur stable.chamilo.org le 28 décembre 2012

Je pense qu'il serait urgent de régler ce bug avant la sortie finale 1.9.4


Files

vue_prof_1.jpg (49.1 KB) vue_prof_1.jpg Alain Deschênes, 28/12/2012 15:24
vue_prof_2.jpg (86 KB) vue_prof_2.jpg Alain Deschênes, 28/12/2012 15:24
vue_prof_1x.jpg (62.4 KB) vue_prof_1x.jpg Alain Deschênes, 28/12/2012 15:27
vue_prof_2x.jpg (98.6 KB) vue_prof_2x.jpg Alain Deschênes, 28/12/2012 15:27
LP-01.jpg (40.9 KB) LP-01.jpg Alain Deschênes, 01/04/2013 23:07
learnpath.class.php (429 KB) learnpath.class.php Yoselyn Castillo, 02/04/2013 17:11
lp_controller.php (40.5 KB) lp_controller.php Yoselyn Castillo, 04/04/2013 16:38
LP-Work.jpg (29.5 KB) LP-Work.jpg Alain Deschênes, 18/05/2013 16:18
LP-Not-Work.jpg (27.7 KB) LP-Not-Work.jpg Alain Deschênes, 18/05/2013 16:18
lp_false.png (168 KB) lp_false.png Alain Deschênes, 21/05/2013 21:12

Related issues

Related to Chamilo LMS - Bug #6154: Learning path bugBug resolved20/05/2013

Actions
Related to Chamilo LMS - Bug #6157: Little quirks in learnpath list linksNew21/05/2013

Actions

Associated revisions

Revision a8f848db
Added by Yannick Warnier almost 7 years ago

Merge pull request #31 from ycastillo/yoselyn.lp1

Yoselyn.lp1 - refs #5855

Revision 023bd896 (diff)
Added by Yannick Warnier almost 7 years ago

Fix issue preventing the correct opening of the learning path in student view - refs #6154 refs #5855

History

#1

Updated by Alain Deschênes over 7 years ago

Voici les images avec commentaires:

#2

Updated by Alain Deschênes about 7 years ago

  • Target version set to 1.9.4
#3

Updated by Yannick Warnier about 7 years ago

  • Assignee deleted (Julio Montoya)
  • Priority changed from High to Normal
  • Target version changed from 1.9.4 to 1.9.6

Bonjour Alain,
Merci de ne pas assigner directement un développeur à une tâche. Ils s'assigneront eux-mêmes selon les disponibilités.
Pour ce qui est de la version, nous devrions normalement publier la 1.9.4 RC1 aujourd'hui même. Les nouveaux bugs seront réglés dans le but de les publier comme la version 1.9.6 (il faut mettre la limite quelque part, désolé). J'imagine que tu as confondu ici la "Target version" avec la "Version in which it happens" :-)

Enfin, en ce qui concerne la priorité, nous (développeurs) décidons de ce qui est urgent ou pas. A priori, il est relativement naturel pour tout un chacun de considérer ses tâches comme plus importantes ou urgentes, ce pourquoi nous demandons (dans la pocédure d'enregistrement des erreurs) de ne pas:
  • assigner une personne en particulier (à moins que cette personne soit soi-même)
  • assigner de priorité (à moins que cela ne cause une indisponibilité complète de tous les serveurs Chamilo basés sur cette version - genre bugs de sécurité)
  • assigner de version particulière, la version étant la version "objectif" et non la version sur laquelle peut être notée l'erreur

Désolé pour la remarque, mais ça rend les choses vraiment compliquées à suivre si on ne suit pas ces quelques petites règles.

#4

Updated by Alain Deschênes about 7 years ago

Bonjour Yannick,

Je me sens très à l'aise avec les commentaires mentionnés ci-dessus et je compte bien les respecter. Cependant, je persiste à croire que ce bug devrait être résolu pour la version 1.9.4 puisqu'il peut générer de la confusion, surtout pour de nouveaux utilisateurs de Chamilo.

A des fins de vérification:
1 - Créer un LP et le rendre accessible via la page d'accueil du cours
2 - Y accéder en cliquant sur l'icône (On se retrouve automatiquement en mode apprenant et c'est correct)
3 - Revenir sur la page d'accueil du cours (On demeure en mode apprenant)
4 - On clique à nouveau sur le LP à partir de l'icône situé sur la page d'accueil et c'est là que ça se complique puisqu'on se retrouve en mode d'édition du LP tout en étant en mode Apprenant - Étrange

Note: Cette situation ne se produit que pour l'enseignant titulaire du cours - Du côté apprenant, tout se passe bien.

Voilà, à vous de voir.

Cordialement,

Alain

#5

Updated by Yannick Warnier about 7 years ago

Bonjour Alain,

OK, merci pour les commentaires. Pour ce qui est de régler ceci dans la 1.9.4, si un développeur de la communauté s'en charge ou que que nous avons du temps à dédier à cette tâche, nous nous en occuperons, bien entendu. Julio et Yoselyn sont bien des employés de BeezNest, cela dit, et ils ont des tâches (prioritaires) assignées qui justifient leurs salaires (je le dis sans animosité et au cas où tu ne le saurais pas). Il est donc prioritaire pour nous de porter assistance à nos clients (sans qui tout le projet Chamilo ne serait qu'une idée dans nos esprits).

Au moment de mettre des limites sur une version (déterminer quand il est sage de la publier), nous sommes conscients qu'il restera toujours une série de problèmes non-résolus et qui, selon les usages des différents utilisateurs, seront considérées prioritaires ou non. Il y a malheureusement une limite au budget qu'une entreprise peut apporter à un projet en tant que "support à la communauté". Je reste confiant du fait que cette quantité, telle qu'elle est investie par BeezNest aujourd'hui, est suffisante pour faire avancer le projet Chamilo dans la bonne direction et à la bonne vitesse, mais si la communauté veut voir le projet avancer plus rapidement, il est nécessaire que plus de personnes ou entreprises s'y investissent (objectif pour lequel plusieurs des volontaires de l'association, dont moi-même, dédient une partie considérable de leur temps, en espérant qu'il s'agisse d'un investissement intelligent). Les investissements ne doivent pas particulièrement être "financiers", et nous profitons largement des contributions de développeurs indépendants, comme Juan Carlos, Hubert, Juan Manuel, Laurent, Noël, Jan, etc, mais il

Enfin, publier une version corrective comme la 1.9.4 nous permet de dédier moins de temps au support des institutions et personnes rencontrant des problèmes dans la version antérieure (dans ce cas-ci 1.9.2) et d'avancer plus efficacement sur l'aspect développement. En effet, nous traitons tous les rapports d'erreur, et de multiples rapports sur le même problème dans une version antérieure du logiciel cause des pertes de temps qui serait plus utile dépensé d'une autre manière.

Ceci n'a donc rien à voir en particulier avec le rapport précis dont tu nous as fait part, mais bien avec un cycle qu'il nous est nécessaire de respecter pour pouvoir avancer de façon optimale dans Chamilo.

Chamilo RC1 (Release Candidate) a été terminé cette nuit. À moins d'obtenir quelques petites corrections de dernière minute (dont celle-ci pourrait faire partie si quelqu'un se porte volontaire), ce sera cette même version qui sera publiée dans 2-3 semaines officiellement sur le site web comme "Chamilo LMS 1.9.4".

Je mets Hubert en copie puisqu'il combine un intérêt profond dans l'outil de parcours d'apprentissage et des talents de développeur. Si quelqu'un doit donner un coup de pelle à cette tâche avant la release finale, c'est probablement lui le plus approprié :-)

#6

Updated by Yannick Warnier about 7 years ago

  • Category set to Learning paths / Lecciones
  • Status changed from New to Assigned
  • Assignee set to Yoselyn Castillo
#7

Updated by Yoselyn Castillo about 7 years ago

Confirmed. Working..

#8

Updated by Alain Deschênes about 7 years ago

Sorry, I've just tested on stable.chamilo.org and it is still not working well ! (Tested with IE9 and Firefox 19)

When you're connected on Chamilo as a teacher and you click on LP (as a tool on the front page) depending if you click on image or on text, Chamilo react differently...

But, when you are connected as a student on Chamilo, everything is working well.

Kind regards,

Alain

#9

Updated by Yoselyn Castillo almost 7 years ago

Hello!
Try this 2 files in main/newscorm/. This fixes student view in LP.

#10

Updated by Yannick Warnier almost 7 years ago

Yoselyn, now that we use github, you can send these patches to the Chamilo repository straight away, then we will verify them.
It is important so that users can try it in stable.chamilo.org and we cut the time needed to gt your patches into the software (it's much faster to check the difference through git (or mercurial) than it is to download, check and upload.

#11

Updated by Alain Deschênes almost 7 years ago

Hi,

I've replace those files, and the problem is still there :-(

It bring me into the LP edit mode on learner view as shown upper.

Alain

#12

Updated by Yoselyn Castillo almost 7 years ago

Well it fixes when we click in general LearningPath icon, if student view is active but i will fix when we click in a specific LP published on training page. I will upload files later.
Yannick, the problem is my clone process is a little complicated due my internet. I am on it.

#13

Updated by Yoselyn Castillo almost 7 years ago

This is the file. please test it, i haven´t still my repository completely in order to uploading now, but this fixes the bug.

#14

Updated by Yoselyn Castillo almost 7 years ago

  • File deleted (lp_list.php)
#15

Updated by Yannick Warnier almost 7 years ago

  • Assignee changed from Julio Montoya to Yoselyn Castillo
#16

Updated by Yannick Warnier almost 7 years ago

Is this the file of the pending pull request?

#17

Updated by Alain Deschênes almost 7 years ago

Still not working well on stable.chamilo.org

Try it on any course, if you're loggin as a teacher and you go to the student view and click on LP's icon, and you will fall on edition mode !

#18

Updated by Yoselyn Castillo almost 7 years ago

Commit for the initial bug, but there is some php notice, that will be fixed later
https://github.com/ycastillo/chamilo-lms/commit/7de3b158a73e8d94d6030c05530b728a95a3b674

#19

Updated by Alain Deschênes almost 7 years ago

Hi Yoselyn,

On student view, with this correction, everything is working well now.

On teacher view, when you click on icon, it is working correctly, but, when you click on text below the icon, we go on edit mode.

But for now, I can deal with it.

Kind regards,

Alain

#20

Updated by Yoselyn Castillo almost 7 years ago

  • Assignee changed from Yoselyn Castillo to Julio Montoya

This pull fixes the initial bug
https://github.com/chamilo/chamilo-lms/pull/31

#21

Updated by Alain Deschênes almost 7 years ago

Hi Yoselin and Julio,

I've tried those modifications and when I modify the file "main/newscorm/learnpath.class.php" by adding + at the line 2142, it does not change anything.

And when I modify the file main/newscorm/lp_controller.php, Chamilo is working as I've indicated above.

Alain

#22

Updated by Yoselyn Castillo almost 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Julio Montoya)

Alain Deschênes wrote:

Hi Yoselyn,

On student view, with this correction, everything is working well now.

On teacher view, when you click on icon, it is working correctly, but, when you click on text below the icon, we go on edit mode.

But for now, I can deal with it.

Kind regards,

Alain

Test now on stable.chamilo.org, Now in learner view, when you click on icon or link of a learning path, it redirects you to preview page, not edit mode.

#23

Updated by Alain Deschênes almost 7 years ago

Wow, everything are working well, except for one thing, and it should be the last one ;-)

When you publish the LP on the training homepage, one item is not working correctly when you are on Teacher view. Get a look on picture joined:

When you click on icon, it is working well:

When you click on text, it is not working correctly:

#24

Updated by Yannick Warnier almost 7 years ago

  • Status changed from Needs testing to Assigned
  • Assignee set to Yoselyn Castillo
#25

Updated by Yannick Warnier almost 7 years ago

Yoselyn Castillo wrote:

Commit for the initial bug, but there is some php notice, that will be fixed later
https://github.com/ycastillo/chamilo-lms/commit/7de3b158a73e8d94d6030c05530b728a95a3b674

This fix breaks something else (much more important), see #6154, so I am considering reverting it.

Yoselyn, could you explain the logic of this change?

#26

Updated by Yoselyn Castillo almost 7 years ago

The change was in order to asking if it as active student view, because lp was not taking into account when student view was active from course home, and it always showed edit mode.
The fix was simply change line 207
-if (isset($_GET['isStudentView']) && $_GET['isStudentView'] 'true')
+if (isset($_SESSION['studentview']) && $_SESSION['studentview']'studentview')
This was because when you logged as a teacher and you active student view, the LP is not calling using the parameter 'isStudentView', but indeed $_SESSION['studentview']='studentview'
What else is breaking this change? Please,do you have other idea for solving this bug?

#27

Updated by Yoselyn Castillo almost 7 years ago

I see the bug in 6154. But i don't know Why it causes that..

#28

Updated by Marko Kastelic almost 7 years ago

Yoselyn Castillo wrote:

The change was in order to asking if it as active student view, because lp was not taking into account when student view was active from course home, and it always showed edit mode.

as i remember, when you are a teacher, you cannot execute LP as teacher, but execution put you into learner mode (that's why GET method with param isStudentView is used). Teacher mode always put you into editing mode. Hope will help ..

#29

Updated by Yannick Warnier almost 7 years ago

Yoselyn says:

when you are logged as a teacher and you enable the student view, the LP is not called using the parameter $_GET['isStudentView'] but, instead $_SESSION['studentview']='studentview'

In fact, as I see it, this statement is not really focusing on the problem. The problem, as stated by Alain, is that the link to the learning path is not the same in the image and in the link. If the link is alright, the rest is already alright.

I've fixed all that (I think) with the following changes: https://github.com/chamilo/chamilo-lms/commit/023bd896f6595af4c36a75b6dca96582f3212b3b

I've also returned to the $_GET version in lp_controller.php.

I've tried all types of access (from course home as teacher, from lp list as teacher, from lp edition using the student view button, from course home as student and from lp list as student). All seemed to work perfectly.

#30

Updated by Yannick Warnier almost 7 years ago

  • Status changed from Assigned to Needs testing
  • Assignee deleted (Yoselyn Castillo)
#31

Updated by Alain Deschênes almost 7 years ago

Hi, it's me again ;-)

Everything I mentioned above works fine now.

One thing seems strange to me when we are in the LP's folder and you click on Learner view buton.

See the image below:

#32

Updated by Yannick Warnier almost 7 years ago

  • Status changed from Needs testing to Bug resolved
  • Assignee set to Yoselyn Castillo
  • % Done changed from 80 to 100

Nice catch. There is also another weird behaviour: if you click on an "eye" icon (to make visible/invisible), then click on the "Student view" button, you see the learning path for which you just changed the visibility.

I'll move those two bugs in a new task for 1.9.8. These are really minor ones...

Also available in: Atom PDF