Skip to content

les modifications de la mission ne sont pas appliquées à la volée aux rapports d'étudiants

Francois Gannaz requested to merge 34-remove-ld-from-mission into dev

Closes #34 (closed)

En l'état, cette branche ne corrige que la diffusion de la suppression d'un labdoc dans la mission, grâce au commit da164d9e. Je n'ai pas trouvé de doc sur le rôle de la table labdoc_status, donc j'ai un doute sur l'usage des champs deleted qui sont dans labdoc et labdoc_status et semblent redondants. Je me suis donc inspiré de deleteLD() côté étudiant pour corriger removeLD() côté enseignant. À terme, il faudrait un code commun.

J'ai testé l'ajout d'un labdoc dans la mission, ce qui ne se propage pas en AJAX dans le rapport de l'étudiant. Je ne sais pas quel est le processus pour les ajouts (on les passe dans le modified du XML ?), donc je préfère valider cette première étape avant de traiter les cas voisins.

Le rapport de bug signalait que le mécanisme ne fonctionnait plus, donc j'ai commencé par chercher dans les changements récent, mais au final je pense que ça n'a jamais fonctionné. Sinon, ce bug m'a forcé à manipuler plus LabBook côté enseignant, ce que j'avais peu fait. C'était donc très pédagogique pour moi, même si j'ai parfois pataugé pour comprendre l'interface web.

Edited by Francois Gannaz

Merge request reports