From 2a63c0ac60b97682b2000b24278fdb81b3da8f7d Mon Sep 17 00:00:00 2001
From: Thomas Fradet <t.fradet8@gmail.com>
Date: Fri, 24 May 2019 15:28:51 +0200
Subject: [PATCH] bugs divers
---
CHANGELOG.md | 8 +++++++-
renderer.php | 3 ++-
view/view_course_header.php | 4 ++--
3 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 94cf18b..da812fc 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -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
+
+
diff --git a/renderer.php b/renderer.php
index 48ffcaa..96b27f0 100644
--- a/renderer.php
+++ b/renderer.php
@@ -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>";
}
diff --git a/view/view_course_header.php b/view/view_course_header.php
index 3901804..1b15ee7 100644
--- a/view/view_course_header.php
+++ b/view/view_course_header.php
@@ -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,
--
GitLab