Commit c9080116 authored by Thomas Fradet's avatar Thomas Fradet

bugs divers

parent ff3da1b2
...@@ -107,15 +107,11 @@ Correction du fait que le filtre affichait >=0 au lieu de >0. ...@@ -107,15 +107,11 @@ Correction du fait que le filtre affichait >=0 au lieu de >0.
__MAJ en 3.7__ __MAJ en 3.7__
__Bugs__ ## __à tester__
## 1 ### 2
NaN dans le tableau d'achèvement quand pas d'activité. Idée : buter le NaN définitivement avec si NaN alors "-" => fuck off Sur la page d'accueil du cours.
__à tester__
## 2
Warning: include(../../lib.php): failed to open stream: No such file or directory in /Applications/MAMP/htdocs/moodle37/course/format/iena/entity/course_format_iena_cron_action.php on line 37 Warning: include(../../lib.php): failed to open stream: No such file or directory in /Applications/MAMP/htdocs/moodle37/course/format/iena/entity/course_format_iena_cron_action.php on line 37
...@@ -131,7 +127,9 @@ Les fichiers ne sont plus inclus car ils le sont dans course/view.php qui devrai ...@@ -131,7 +127,9 @@ Les fichiers ne sont plus inclus car ils le sont dans course/view.php qui devrai
## __Bugs résolus__ ## __Bugs résolus__
## 3 ### Page principale du cours
#### 3
Clef numsection manquante sur l'objet cours. Suppression de l'appel à cette clef car dans le format de cours topic ils utilisent 0 pour ne pas recréer de sections. Question : à quoi ça sert ? Clef numsection manquante sur l'objet cours. Suppression de l'appel à cette clef car dans le format de cours topic ils utilisent 0 pour ne pas recréer de sections. Question : à quoi ça sert ?
...@@ -143,7 +141,7 @@ Après : course_create_sections_if_missing($course, 0); ...@@ -143,7 +141,7 @@ Après : course_create_sections_if_missing($course, 0);
[x] Solved. [x] Solved.
## 4 #### 4
Typo numsections au lieu de $numsections dans une condition dont l'intérieur n'était pas utilisé. Typo numsections au lieu de $numsections dans une condition dont l'intérieur n'était pas utilisé.
...@@ -151,7 +149,7 @@ La condition elle-même est commentée et le $ est ajouté. ...@@ -151,7 +149,7 @@ La condition elle-même est commentée et le $ est ajouté.
[x] Solved. [x] Solved.
## 5 #### 5
Notice: Undefined offset: 0 in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 788 Notice: Undefined offset: 0 in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 788
...@@ -169,7 +167,7 @@ Après : $introSection[$section] = $this->section_header($thissection, $course, ...@@ -169,7 +167,7 @@ Après : $introSection[$section] = $this->section_header($thissection, $course,
[x] Solved. [x] Solved.
## 6 #### 6
Notice: Undefined property: stdClass::$showdefaultsectionname in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 102 Notice: Undefined property: stdClass::$showdefaultsectionname in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 102
...@@ -177,7 +175,7 @@ Commenté : sert à rien et la property n'existe pas. Pas présent dans le code ...@@ -177,7 +175,7 @@ Commenté : sert à rien et la property n'existe pas. Pas présent dans le code
[x] Solved. [x] Solved.
## 7 #### 7
Notice: Undefined property: stdClass::$sectionposition in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 836 Notice: Undefined property: stdClass::$sectionposition in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 836
...@@ -187,7 +185,7 @@ La condition et l'echo qu'elle contient sont commentés. ...@@ -187,7 +185,7 @@ La condition et l'echo qu'elle contient sont commentés.
[x] Solved. [x] Solved.
## 8 #### 8
Notice: Undefined variable: completion_total in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_course_header.php on line 13 Notice: Undefined variable: completion_total in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_course_header.php on line 13
...@@ -197,7 +195,7 @@ La création de la propriété devenue inutile est commentée. ...@@ -197,7 +195,7 @@ La création de la propriété devenue inutile est commentée.
[x] Solved. [x] Solved.
## 9 #### 9
Notice: Undefined variable: USER in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_course_header.php on line 370 Notice: Undefined variable: USER in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_course_header.php on line 370
...@@ -205,7 +203,7 @@ Ajout de la global $USER dans la méthode. ...@@ -205,7 +203,7 @@ Ajout de la global $USER dans la méthode.
[x] Solved. [x] Solved.
## 10 #### 10
Notice: Undefined variable: view in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 413 Notice: Undefined variable: view in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 413
...@@ -213,7 +211,7 @@ Instanciation de la variable $view = ""; ...@@ -213,7 +211,7 @@ Instanciation de la variable $view = "";
[x] Solved. [x] Solved.
## 11 #### 11
Notice: Trying to get property 'presence' of non-object in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 511 Notice: Trying to get property 'presence' of non-object in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 511
...@@ -225,7 +223,7 @@ Ajout d'une condition que la variable contenant l'objet ne soit pas === false da ...@@ -225,7 +223,7 @@ Ajout d'une condition que la variable contenant l'objet ne soit pas === false da
[x] Solved. [x] Solved.
## 12 #### 12
Notice: Undefined variable: date in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 622 Notice: Undefined variable: date in /Applications/MAMP/htdocs/moodle37/course/format/iena/renderer.php on line 622
...@@ -235,3 +233,66 @@ Ajout d'un else à la condition pour que $date soit toujours instancié. ...@@ -235,3 +233,66 @@ Ajout d'un else à la condition pour que $date soit toujours instancié.
[x] Solved. [x] Solved.
### Page de suivi des étudiants
#### 13
Notice: Undefined variable: where in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 67
Warning: implode(): Invalid arguments passed in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 67
Notice: Undefined variable: where_params in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 68
Notice: Undefined variable: firstnamesort in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 70
Notice: Undefined variable: csv in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 71
Warning: Use of undefined constant COMPLETION_REPORT_PAGE - assumed 'COMPLETION_REPORT_PAGE' (this will throw an Error in a future version of PHP) in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 71
Notice: Undefined variable: csv in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 72
Notice: Undefined variable: start in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 72
Non-numeric limitnum parameter detected: 'COMPLETION_REPORT_PAGE', did you pass the correct arguments?
line 1050 of /lib/dml/moodle_database.php: call to debugging()
line 1231 of /lib/dml/mysqli_native_moodle_database.php: call to moodle_database->normalise_limit_from_num()
line 1200 of /lib/completionlib.php: call to mysqli_native_moodle_database->get_records_sql()
line 1232 of /lib/completionlib.php: call to completion_info->get_tracked_users()
line 73 of /course/format/iena/view/view_suivi_unit3.php: call to completion_info->get_progress_all()
line 102 of /course/format/iena/suivi_unit.php: call to require_once()
Problèmes relatifs à la fonction copiée dans le suivi d'achèvement natif de MOODLE dont les paramètres sont ici inexistants.
Remplacement des variables non instanciées par des valeur qui sont celles par défaut dans la fonction appellée :
- $where = ''
- $where_params = array()
- $pagesize = '',
- $start = ''
Utilisation du classement 'u.lastname ASC, u.firstname ASC' au lieu d'utiliser une variable booléenne (non instanciée) pour choisir entre prénom/nom ou nom/prénom.
[x] Solved.
#### 14
Notice: Undefined index: filter in /Applications/MAMP/htdocs/moodle37/course/format/iena/view/view_suivi_unit3.php on line 114
Modification de la prise en compte du paramètre get optionnel 'filter' pour 'all' si non isset et sinon la valeur du paramètre passé dans le but qu'il ne puisse jamais être NULL. La valeur est utilisée pour matcher la liste déroulante du filtre de % d'achèvement.
[x] Solved.
#### 1
NaN dans le tableau d'achèvement quand pas d'activité dans la section (division par 0).
Le nombre d'activité est vérifié pour afficher 0 s'il n'y a pas d'activité.
## Améliorations
### Suivi des étudiants
#### Message quand pas d'activité à suivre
[ ] Lorsqu'une section ne contient pas d'activité suivies, afficher un message au lieu de la liste des étudiants avec aucune activité en colonne et 0% pour chaque.
...@@ -64,12 +64,12 @@ foreach ($groups as $group) { ...@@ -64,12 +64,12 @@ foreach ($groups as $group) {
- informations personnelles (tous les utilisateurs du cours) - informations personnelles (tous les utilisateurs du cours)
- liste des activités avec état d'achèvement (si achevées, sinon vide) */ - liste des activités avec état d'achèvement (si achevées, sinon vide) */
$progress = $completion->get_progress_all( $progress = $completion->get_progress_all(
implode(' AND ', $where), '',
$where_params, array(),
$active_group_id, $active_group_id,
$firstnamesort ? 'u.firstname ASC, u.lastname ASC' : 'u.lastname ASC, u.firstname ASC', 'u.lastname ASC, u.firstname ASC',
$csv ? 0 : COMPLETION_REPORT_PAGE, '',
$csv ? 0 : $start, '',
$context $context
); );
...@@ -111,10 +111,10 @@ if ($active_section_id == NULL) { ...@@ -111,10 +111,10 @@ if ($active_section_id == NULL) {
// } // }
// Filtre actif si il existe (si la page est rechargée pour un filtre de groupe p.ex.) // Filtre actif si il existe (si la page est rechargée pour un filtre de groupe p.ex.)
$filter = $_GET['filter']; if (isset($_GET['filter'])) {
if ($filter == NULL) { $filter = $_GET['filter'];
// no param given for section : first section (index 1, not generalities section 0) } else {
$filter = '<100'; $filter = 'all';
} }
// echo "<pre>"; // echo "<pre>";
...@@ -306,6 +306,7 @@ th.th-rotate > div > span { ...@@ -306,6 +306,7 @@ th.th-rotate > div > span {
">25" => "&gt;=25%", ">25" => "&gt;=25%",
">0" => "&gt;0%", ">0" => "&gt;0%",
]; ];
var_dump($filter);
foreach ($filters as $value => $name) { foreach ($filters as $value => $name) {
if ( $value == $filter ) { if ( $value == $filter ) {
echo "<option selected='selected' value='$value'>$name</option>"; echo "<option selected='selected' value='$value'>$name</option>";
...@@ -405,7 +406,7 @@ var data = { ...@@ -405,7 +406,7 @@ var data = {
counter++; counter++;
} }
} }
console.log(counter); // console.log(counter);
if (counter == 0) { if (counter == 0) {
document.getElementById('suivi').style.display = "none"; document.getElementById('suivi').style.display = "none";
document.getElementById('no_result').style.display = ""; document.getElementById('no_result').style.display = "";
...@@ -635,7 +636,11 @@ var data = { ...@@ -635,7 +636,11 @@ var data = {
done++; done++;
} }
} }
if (modules_nbr > 0) {
var stud_perc = 100 * done / modules_nbr; var stud_perc = 100 * done / modules_nbr;
} else {
var stud_perc = 0;
}
rows[i].querySelector('span').innerHTML = Math.floor(stud_perc) + "% "; rows[i].querySelector('span').innerHTML = Math.floor(stud_perc) + "% ";
rows[i].dataset.percent = stud_perc; rows[i].dataset.percent = stud_perc;
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment