diff --git a/CHANGELOG.md b/CHANGELOG.md index bd93de066b9e0c9291fcb526d177855d78c97cd0..3d94d45a755d6ed75162036b6a3cdf0bf6ed2b69 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -306,7 +306,20 @@ Suppression de la partie inutilisée du formulaire qui était utilisée pour cac #### 16 -Erreurs lors du post du formulaire d'enregistrement des paramètres d'une section. +Erreurs diverses lors du post du formulaire d'enregistrement des paramètres d'une section. + +Raison : index manquants dans le post lorsqu'ils ne sont pas définis dans le formulaire. + +Le paramètre hide ayant été supprimé, la valeur false est attribuée dans l'enregistrement. Par la suite, il faudra le supprimer des données en base. + +Les paramètres non définis sont testés avec un isset au lieu de ramener leur absence à false dans les conditions. + +Variable courseid non définie remplacée par $COURSE->id plus sécurisée. + +[x] Solved. + + + @@ -322,5 +335,8 @@ Erreurs lors du post du formulaire d'enregistrement des paramètres d'une sectio [ ] 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. +### Cron + +Changement des jours de `"*/1" à "*"` pour envoyer tous les jours. diff --git a/param_section.php b/param_section.php index e6025090f9ba73581c0f851709bbeb37812c5023..527a82e57d90ed210f9c8638763d42614c8f0141 100644 --- a/param_section.php +++ b/param_section.php @@ -53,31 +53,32 @@ $_POST["date_iena"]["hour"] . ":" . $_POST["date_iena"]["minute"] . ":00"; $dateUp = date_create($dateChaine); $data_iena = new stdClass(); - if ($_POST['date_render']) { + if (isset($_POST['date_render'])) { $dateChaine = ""; } $data_iena->date_rendu = $dateChaine; $data_iena->presence = $_POST['presence']; $data_iena->id_section = $sectionId; - if ($_POST['day_same']) { + if (isset($_POST['day_same'])) { $data_iena->day_same = $_POST['day_same']; } else { $data_iena->day_same = 0; } - if ($_POST['day_before']) { + if (isset($_POST['day_before'])) { $data_iena->day_before = $_POST['day_before']; } else { $data_iena->day_before = 0; } - if ($_POST['day_after']) { + if (isset($_POST['day_after'])) { $data_iena->day_after = $_POST['day_after']; } else { $data_iena->day_after = 0; } $data_iena->nb_days_before = $_POST['nb_days_before']; $data_iena->nb_days_after = $_POST['nb_days_after']; - $data_iena->hide = $_POST['hide']; + // $data_iena->hide = $_POST['hide']; + $data_iena->hide = false; $testIfsection = $DB->get_record('format_iena', array('id_section' => $sectionId), '*'); if ($testIfsection == false) { @@ -91,7 +92,7 @@ $data['name'] = $_POST['name']; $data['summary'] = $_POST['summary']['text']; $DB->update_record('course_sections', $data); - rebuild_course_cache($courseid, true); + rebuild_course_cache($COURSE->id, true); $link = $CFG->wwwroot . '/course/view.php?id=' . $courseID; header("Location: {$link}"); exit;