Project

General

Profile

Bug #7349

Access to learning path is possible even if tool is hidden

Added by Andre Boivin about 5 years ago. Updated over 4 years ago.

Status:
Bug resolved
Priority:
Normal
Assignee:
Category:
Learning paths / Lecciones
Target version:
Start date:
24/10/2014
Due date:
03/11/2014
% Done:

100%

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

Description

When learnpath tool is made invisible, the student can still access a given learning path by selecting "My progress" > "Details".
Even if the eye is closed by teacher on main page
Learnpath accessibility is managed two places, causing confusion


Files

parcours.mp4 (1.38 MB) parcours.mp4 Andre Boivin, 30/10/2014 15:13
lp_controller.php (48.9 KB) lp_controller.php Andre Boivin, 09/11/2014 17:39
lp_list.php (22.6 KB) lp_list.php Andre Boivin, 09/11/2014 17:39

Associated revisions

Revision 19ea1ec7 (diff)
Added by Julio Montoya over 4 years ago

If lp is invisible then don't show in the progress see #7349

Revision 8af3788d (diff)
Added by Julio Montoya over 4 years ago

Just hide the link see #7349

History

#1

Updated by Yannick Warnier about 5 years ago

I think it might be described more precisely in French :-) (and with images). We need more details.

#2

Updated by Julio Montoya about 5 years ago

  • Status changed from New to Needs more info
  • Assignee set to Andre Boivin
#3

Updated by Andre Boivin about 5 years ago

  • Assignee changed from Andre Boivin to Julio Montoya

Même si l'oeil est fermé sous l'icone du parcours, à l'acceuil du cours, l'élève peut y avoir accès s'il passe par l'onglet Ma progression et Détails.

#4

Updated by Andre Boivin about 5 years ago

La vidéo ci-jointe sera plus utile.

#5

Updated by Yannick Warnier about 5 years ago

  • Subject changed from Access Learnpath close by selecting my progress to Access to learning path is possible even if tool is hidden
  • Description updated (diff)
  • Due date set to 03/11/2014
  • Category set to Learning paths / Lecciones
  • Assignee changed from Julio Montoya to Andre Boivin
André, that's the logic in Chamilo:
  • you can hide one specific content (a document, a learning path, an exercise) directly, and the student will not be able to access it
  • you can hide a tool from the homepage, but this does not prevent the student to access the contents inside of it. If he has the link, he can get to it.
Traduction:
La logique de Chamilo est la suivante:
  • tu peux cacher un contenu spécifique et dans ce cas les étudiants n'y ont plus accès
  • tu peux désactiver l'apparition d'un outil sur la page de garde d'un cours, mais cela ne désactive pas spécifiquement tous les contenus de l'outil: ça cache seulement l'icône de la page de garde

Ça a toujours été comme ça, et les conséquences possibles d'un changement de comportement à ce niveau sont... cosmiques.

Est-ce que ça répond à tes doutes?

#6

Updated by Andre Boivin about 5 years ago

  • Assignee changed from Andre Boivin to Yannick Warnier

je comprend mais à ce moment là si l'oeil est fermé dans la gesion des parcours, celui-ci ne devrait-il ne plus apparaitre sur l'acceuil du cours? Ça règlerait la confusion causée par 2 façons de rendre visible cet outil.

#7

Updated by Andre Boivin about 5 years ago

Une autre erreur fréquente est causée lorsque l'oeil est ouvert dans la gestion mais que la publication n'est pas activée. Ceci rend l'icone apparente sur l'acceuil du cours mais si l'élève clique dessus, il ne peut y avoir accès.
J'ai joint les 2 fonctions sur le même boutons, j'ai conservé l'oeil et j'ai supprimé le bouton publish et ça me semble plus logique comme fonctionnement. En fait quelle est l'utilité du bouton publish?

#8

Updated by Yannick Warnier about 5 years ago

  • Status changed from Needs more info to New
  • Assignee deleted (Yannick Warnier)
  • Target version changed from 1.9.10 to 1.10.0

Le bouton publish met le parcours sur la page d'accueil du cours. Sinon, le parcours en soi n'est visible que depuis la liste de parcours.

Je paraphrase donc le problème d'une autre manière:

Supprimmer le parcours de la page d'accueil du cours (où il a été placé antérieurement via "publish") quand le parcours est rendu invisible.

Cela implique de modifier la table c_tool lorsque l'on change le parcours à invisible.
Il est trop tard pour le mettre dans 1.9.10, mais je le programme déjà pour 1.9.12.

#9

Updated by Andre Boivin about 5 years ago

Sans modifier la table, en jumelant les fonctions de visibilité et de publication, ça semble bien fonctionner. J'ai modifié lp_controler.php et lp_list.php

#10

Updated by Yannick Warnier almost 5 years ago

  • Status changed from New to Needs more info
  • Assignee set to Andre Boivin

Bonjour André,

Nous ne parvenons pas à reproduire le problème sur la 1.10.x (stable.chamilo.org). Est-ce que tu pourrais nous confirmer que cela est résolu?

#11

Updated by Yannick Warnier almost 5 years ago

  • Status changed from Needs more info to Needs testing
  • % Done changed from 0 to 90

Apparemment ça a été réglé en collaboration avec l'Académie d'Aix-Marseilles dans une tâche privée (BT7688).
Si tu peux tester, André, et me faire signe si ça te convient, je pourrais fermer cette tâche (et la réassigner à la 1.9.10 où elle devrait être restée, du coup).

#12

Updated by Andre Boivin almost 5 years ago

le problème est encore présent. Sur stable.chamilo.org j'ai créé un user nommé bidon passw: bidon et si tu clique sur Ma progression, détails, tu verras le parcours nommé parcours1 et tu y auras accès.

#13

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs testing to Assigned
  • Assignee changed from Andre Boivin to Julio Montoya
  • % Done changed from 90 to 50
#14

Updated by Julio Montoya over 4 years ago

Tested in my branch it works now. 1.10.x.doctrine.
I have to do a merge soon.

#15

Updated by Yannick Warnier over 4 years ago

We have to update unstable.chamilo.org now (to use the new database structure)

#16

Updated by Julio Montoya over 4 years ago

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

Yannick Warnier wrote:

We have to update unstable.chamilo.org now (to use the new database structure)

I just updated unstable.chamilo.org today.

#17

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs more info to Needs testing
  • Assignee set to Andre Boivin
#18

Updated by Yannick Warnier over 4 years ago

Will be considered fixed in 5 days if no feedback is provided.

#19

Updated by Andre Boivin over 4 years ago

sur https://unstable.chamilo.org c'est le même problème, sur la main page, le parcours et la section de gestion des parcours ne sont pas visibles mais l'Élève y a accès en passant par son suivi.

#20

Updated by Andre Boivin over 4 years ago

  • Assignee changed from Andre Boivin to Julio Montoya

sur https://unstable.chamilo.org c'est le même problème, sur la main page, le parcours et la section de gestion des parcours ne sont pas visibles mais l'Élève y a accès en passant par son suivi.

#21

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs testing to Assigned

Julio, il ne faut pas enlever le progrès lui-même... seulement le lien qui permet d'accéder au parcours (je me réfère à ce commit: https://github.com/chamilo/chamilo-lms/commit/19ea1ec7645085114264ac70cca0695c24ab7a0d)

#22

Updated by Julio Montoya over 4 years ago

Yannick Warnier wrote:

Julio, il ne faut pas enlever le progrès lui-même... seulement le lien qui permet d'accéder au parcours (je me réfère à ce commit: https://github.com/chamilo/chamilo-lms/commit/19ea1ec7645085114264ac70cca0695c24ab7a0d)

Corrigé.

#23

Updated by Julio Montoya over 4 years ago

  • Status changed from Assigned to Needs testing
  • Assignee changed from Julio Montoya to Yannick Warnier
#24

Updated by Yannick Warnier over 4 years ago

  • Status changed from Needs testing to Assigned
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 50 to 70

Excellent. Tant que tu y es, est-ce que tu pourrais faire la même chose pour les exercices? (pour l'instant, ils disparaîssent aussi du reporting de l'étudiant quand ils sont cachés dans l'outil exercice).

#25

Updated by Julio Montoya over 4 years ago

  • Assignee changed from Julio Montoya to Yannick Warnier

Yannick Warnier wrote:

Excellent. Tant que tu y es, est-ce que tu pourrais faire la même chose pour les exercices? (pour l'instant, ils disparaîssent aussi du reporting de l'étudiant quand ils sont cachés dans l'outil exercice).

Je viens d'ajouter le changement.

#26

Updated by Yannick Warnier over 4 years ago

  • Status changed from Assigned to Bug resolved
  • Assignee changed from Yannick Warnier to Julio Montoya
  • % Done changed from 70 to 100

Excellent. Je confirme la résolution!

Also available in: Atom PDF