Commit 2a63c0ac authored by Thomas Fradet's avatar Thomas Fradet

bugs divers

parent 3fef2a61
......@@ -237,8 +237,9 @@ Ajout d'un else à la condition pour que $date soit toujours instancié.
Si l'achèvement d'activité n'est pas activé dans le cours.
Problème de variable nulle.
[x] Solved.
### Page de suivi des étudiants
......@@ -344,6 +345,11 @@ Variable courseid non définie remplacée par $COURSE->id plus sécurisée.
### Cron
#### Paramètres
Changement des jours de `"*/1" à "*"` pour envoyer tous les jours.
#### Performance
......@@ -408,6 +408,7 @@
$view = "";
require_once($CFG->dirroot . '/course/format/iena/entity/course_format_iena_completion.php');
// return false if completion disabled (site or course)
$cpl = course_format_iena_completion::get_completion_by_section($nameSection, $idSection);
/* Passer le calcul de complétion de la classe course_format_iena_completion à part à la vue qui fait le header permet de ne pas récupérer deux fois les mêmes informations. */
......@@ -542,7 +543,7 @@ foreach ($htmlsection as $section) {
<div class=\"card card_block\">
<div class=\"heading-iena set_height\">";
if ( count($cpl->sections[$i]->modules) > 0 ) {
if ( $cpl != false && count($cpl->sections[$i]->modules) > 0 ) {
$view .= "<div class='iena-percent set_height'>" . $cpl->sections[$i]->completion . "%</div>";
}
......
......@@ -4,7 +4,7 @@ class view_course_header {
private $_content = "";
public function __construct($section_names, $idSection, $course, $progress = null) {
public function __construct($section_names, $idSection, $course, $progress = false) {
global $CFG;
......@@ -19,7 +19,7 @@ class view_course_header {
// $this->get_progress_bis($this->section_names, $this->idSection, $this->course);
$prog = $progress != null ? $progress : $this->get_progress_bis($this->section_names, $this->idSection, $this->course);
$prog = $progress != false ? $progress : $this->get_progress_bis($this->section_names, $this->idSection, $this->course);
$infos = [
'progress' => $prog,
......
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