From b0c4c3e50982167ba1830f22e55b63d70c261035 Mon Sep 17 00:00:00 2001
From: Thomas Fradet <t.fradet8@gmail.com>
Date: Fri, 24 May 2019 17:35:31 +0200
Subject: [PATCH] bug envoi msg suivi
---
CHANGELOG.md | 27 +++++++++++++--------------
send_message_post.php | 18 +++++++++---------
view/view_send_message.php | 2 +-
3 files changed, 23 insertions(+), 24 deletions(-)
diff --git a/CHANGELOG.md b/CHANGELOG.md
index 45889a9..6a090b9 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -111,6 +111,10 @@ __MAJ en 3.7__
## __à tester__
+## __Bugs résolus__
+
+### Page principale du cours
+
### 2
Sur la page d'accueil du cours.
@@ -125,11 +129,7 @@ Warning: include(): Failed opening '../../../../config.php' for inclusion (inclu
Les fichiers ne sont plus inclus car ils le sont dans course/view.php qui devrait se charger d'inclure à son tout le reste.
-[ ] tester si les crons fonctionnent toujours et si oui, classer le bug.
-
-## __Bugs résolus__
-
-### Page principale du cours
+[x] tester si les crons fonctionnent toujours et si oui, classer le bug.
#### 3
@@ -298,6 +298,14 @@ Le nombre d'activité est vérifié pour afficher 0 s'il n'y a pas d'activité.
[x] Solved.
+#### 17
+
+Bug : L'envoi d'un message à plusieurs étudiants ne l'envoie qu'au premier le nombre de fois qu'il y avait d'étudiants sélectionnés.
+
+Instanciation du message pour chaque utilisateur, sans quoi la propriété du userto n'est pas surécrite. Modification de la boucle : ok.
+
+[x] Solved.
+
### Paramétrage d'une section (date, modalité, etc.)
#### 15
@@ -328,15 +336,6 @@ Variable courseid non définie remplacée par $COURSE->id plus sécurisée.
[x] Solved.
-
-
-
-
-
-
-
-
-
## Améliorations
### Suivi des étudiants
diff --git a/send_message_post.php b/send_message_post.php
index 7e63aa2..4d123af 100644
--- a/send_message_post.php
+++ b/send_message_post.php
@@ -59,23 +59,23 @@
$brutMessageContent = $course->fullname . " " . $_POST['summary']['text'];
$course_ctx = context_course::instance($courseID);
$students = get_enrolled_users($course_ctx);
- $message = new \core\message\message();
- $message->courseid = $course->id;
- $message->component = 'moodle';
- $message->name = 'instantmessage';
- $message->userfrom = $USER;
- $message->notification = '0';
- $message->fullmessageformat = FORMAT_HTML;
+
// sending a notification to each students
foreach ($usersid as $userID) {
foreach ($students as $student) {
- //var_dump($student);
if ($student->id == $userID) {
- $message->userto = $student;
+ $message = new \core\message\message();
+ $message->courseid = $course->id;
+ $message->component = 'moodle';
+ $message->name = 'instantmessage';
+ $message->userfrom = $USER;
+ $message->notification = '0';
+ $message->fullmessageformat = FORMAT_HTML;
$message->subject = "Rappel : " . $course->fullname;
$message->smallmessage = $messageContent;
$message->fullmessage = $brutMessageContent;
$message->fullmessagehtml = $messageContent;
+ $message->userto = $student;
$messageid = message_send($message);
break;
}
diff --git a/view/view_send_message.php b/view/view_send_message.php
index 4945c9c..5aff912 100644
--- a/view/view_send_message.php
+++ b/view/view_send_message.php
@@ -53,7 +53,7 @@
$content = "<h2>Envoi d'un message à ". $nb ." personnes</h2>
<p>Envoi d'un message à tous les étudiants précédemment sélectionnés. Les informations suivantes seront automatiquement ajoutées au message : nom du cours, lien vers le cours, nom du l'enseignant émetteur du message (vous). </p>
- <p>L'étudiant recevra le message dans le chat et sera notifié par mail s'il n'est pas connecté au moment où le message est envoyé.</p>";
+ <p>L'étudiant recevra le message dans le chat et pourra être notifié par mail s'il n'est pas connecté au moment où le message est envoyé.</p>";
$mform = $this->_form;
$description = "";
--
GitLab