iena issueshttps://gitlab.univ-lorraine.fr/groups/iena/-/issues2022-05-02T07:45:06Zhttps://gitlab.univ-lorraine.fr/iena/iena-path-block/-/issues/1Ajout d’une option pour ne pas afficher le bloc aux étudiant·e·s2022-05-02T07:45:06ZJean-François Lemoine (Paris Nanterre)Ajout d’une option pour ne pas afficher le bloc aux étudiant·e·sUne de nos enseignantes a exprimé le souhait de pouvoir masquer ce bloc aux étudiant·e·s.
Nous avons implémenté cette possibilité avec un réglage général pour le plugin, et une option par instance. Le défaut du réglage général correspon...Une de nos enseignantes a exprimé le souhait de pouvoir masquer ce bloc aux étudiant·e·s.
Nous avons implémenté cette possibilité avec un réglage général pour le plugin, et une option par instance. Le défaut du réglage général correspond à la situation _ante_, de sorte que les utilisateurices ne soient pas surpris·e·s.Jean-François Lemoine (Paris Nanterre)Jean-François Lemoine (Paris Nanterre)https://gitlab.univ-lorraine.fr/iena/iena-modale-activity-filter/-/issues/1L’activation du filtre fait planter certains contenus2022-05-02T07:43:47ZJean-François Lemoine (Paris Nanterre)L’activation du filtre fait planter certains contenusNous avons rencontré deux cas où l’activation du filtre faisait planter la page. La génération est très longue et lorsqu’elle dépasse les 30 secondes accordées par le serveur la page s’affiche, mais elle ne contient que les éléments d’in...Nous avons rencontré deux cas où l’activation du filtre faisait planter la page. La génération est très longue et lorsqu’elle dépasse les 30 secondes accordées par le serveur la page s’affiche, mais elle ne contient que les éléments d’interface (entête, barre de navigation, menus, etc.) et pas le contenu proprement-dit. Aucun message d’erreur n’est affiché, ni enregistré dans le journal.
Ce qui est très surprenant est que de très légère variation dans le code, comme de retirer le premier paragraphe du second exemple (qui contient 2 mots), font que la page s’affiche sans problème.
Ce sont des cas un peu extrêmes dans la mesure où ils contiennent des images embarquées.
> La critique est aisé, mais l’art est difficile. Je pense néanmoins que c’était une mauvaise idée de s’appuyer sur une regexp pour remplacer des urls plein-code. Il aurait été moins gourmand et moins périeux de s’appuyer sur une pseudo-syntaxe du type `[modale]https://url…[/modale]`.
> [error-prone-1.html](/uploads/76488bed12a22ba5b375d646862146ed/error-prone-1.html)
> [error-prone-2.html](/uploads/93638d5911f1584e3a0b5ab2a6bb9a08/error-prone-2.html)https://gitlab.univ-lorraine.fr/iena/course-format/-/issues/3Page figée après une permutation de sections2022-05-02T07:45:51ZJean-François Lemoine (Paris Nanterre)Page figée après une permutation de sectionsEn mode édition, il est possible de déplacer les sections par simple glisser-déposer. Lors de cette opération, certains contenus contenant du javascript peuvent faire planter la page, qui se fige alors. Un rafraîchissement de la page mon...En mode édition, il est possible de déplacer les sections par simple glisser-déposer. Lors de cette opération, certains contenus contenant du javascript peuvent faire planter la page, qui se fige alors. Un rafraîchissement de la page montre que les sections ont bien été déplacées.
Voici l’erreur obtenue :
> ```
> Uncaught TypeError: Cannot read property 'swap' of null
> at Object.M.course.format.swap_sections (format.js:2)
> ```
C’est une régression documentée dans Moodle 3.9 : [[MDL-68231] Dropping sections does not complete](https://tracker.moodle.org/browse/MDL-68231).Jean-François Lemoine (Paris Nanterre)Jean-François Lemoine (Paris Nanterre)https://gitlab.univ-lorraine.fr/iena/course-format/-/issues/2Bobologie 22021-12-01T13:02:23ZJean-François Lemoine (Paris Nanterre)Bobologie 2Bonjour,
Nous avons remarqué que l’entête des section est doublé (mais pas fermé) lorsque le mode édition est activé, perturbant les navigateurs vocaux.
```html
<div class="course-content">
<h2 class="accesshide">Aperçu des sections<...Bonjour,
Nous avons remarqué que l’entête des section est doublé (mais pas fermé) lorsque le mode édition est activé, perturbant les navigateurs vocaux.
```html
<div class="course-content">
<h2 class="accesshide">Aperçu des sections</h2>
<ul class="iena-editing">
<h2 class="accesshide">Aperçu des sections</h2>
<ul class="iena-editing">
<li id="section-0" class="section main clearfix"
```
Cette entête est généré par la fonction `start_section_list()` définie dans `renderer.php`. On trouve deux appels consécutifs à cette fonction dans la fonction `print_iena_section_pages()` :
> ```php
> public function print_iena_section_pages($course)
> {
> …
> echo $this->output->heading($this->page_title(), 2, 'accesshide');
> echo $this->course_activity_clipboard($course, 0);
> echo $this->start_section_list();
> …
> if ($PAGE->user_is_editing()) {
> // echo $completioninfo->display_help_icon();
> echo $this->output->heading($this->page_title(), 2, 'accesshide');
> echo $this->course_activity_clipboard($course, 0);
> echo $this->start_section_list();
> }
> ```
On devrait pouvoir supprimer le second bloc sans état d’âme.Jean-François Lemoine (Paris Nanterre)Jean-François Lemoine (Paris Nanterre)https://gitlab.univ-lorraine.fr/iena/course-format/-/issues/1Bobologie 12021-12-01T12:48:23ZJean-François Lemoine (Paris Nanterre)Bobologie 1Bonjour,
Quelques variables sont utilisées sans vérifier qu’elles existent, générant des avertissements.
`renderer.php`
> ```diff
> - $sectionIntro = $introSection[$i];
> + $sectionIntro = isset($introSection[$i]) ? $intro...Bonjour,
Quelques variables sont utilisées sans vérifier qu’elles existent, générant des avertissements.
`renderer.php`
> ```diff
> - $sectionIntro = $introSection[$i];
> + $sectionIntro = isset($introSection[$i]) ? $introSection[$i] : '';
> …
> - if ($course->hiddensections && !(int)$thissection->visible) {
> + if (isset($course->hiddensections) && $course->hiddensections && !(int)$thissection->visible) {
> ```Jean-François Lemoine (Paris Nanterre)Jean-François Lemoine (Paris Nanterre)https://gitlab.univ-lorraine.fr/iena/iena-competency/-/issues/1Test issue2019-01-15T12:28:33ZThomas FradetTest issueTest issue description.Test issue description.