diff --git a/CHANGELOG.md b/CHANGELOG.md
index bd677e1a0c7ee99d9220e358b4343524315c8d4f..0619ce86dcbdf79a19b3b38f3e4b26888b0f77c7 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,11 +1,17 @@
# 19/02/19
-## Création du groupe
-
Le création d'un groupe Mahara n'envoie plus vers une page Mahara de type "groupe non trouvé".
Le bouton de création manuelle de tous les groupes qui y inscrit également l'enseignenant a maintenant une fenêtre de confirmation afin d'éviter des usages accidentels. Ce bouton était et reste réservé aux enseignants, et pas aux enseignants non éditeurs.
-## 22/05/2019
+# 22/05/2019
A cause d'un problème de performances sur l'API de Mahara qui récupère la liste des users, lors de l'instanciation du bloc (activation par l'enseignant), seul lui-même est inscrit dans le groupe.
+
+# 21/06/2019
+
+Mise en compatibilité avec la version 3.7 de Moodle.
+
+Correction de bugs mineurs.
+
+Une maquette d'une interface pour le management des groupes est disponible à ce chemin : `/blocks/mahara_iena/mahara_iena_manage.php?courseid=5` en adaptant d'id du cours. Le développement est en attente d'une solution pour le problème suivant : interroger la liste des utilisateurs de Mahara par l'API prends plusieurs minutes.
diff --git a/README.md b/README.md
index a0750b7a57d362e2400ac65f2de5d80660c48124..d2b15674b90441e02f7221118ebcae1b254ee833 100644
--- a/README.md
+++ b/README.md
@@ -1,8 +1,8 @@
# Bloc Connecteur Mahara iENA
-Ce plugin est un bloc de cours qui permet de lier un cours Moodle à un groupe sur une plateforme Mahara, avec les participants du cours.
+A partir d'un espace de cours MOODLE, ce plugin permet de créer un groupe dans une plateforme Mahara liée et d'y inscrire les participants du cours.
-This plugin is a block that provide a link between a Moodle course and a Mahara group, with course participants.
+This plugin create a group in Mahara from a Moodle course and set the members of the Mahara group from the Moodle course participants.
## Auteurs
@@ -12,13 +12,11 @@ This plugin is a block that provide a link between a Moodle course and a Mahara
## Compatibility
-MOODLE 3.5.4+
-
-Stabilité : expérimental.
+MOODLE 3.7
## Contribution
-Contributors are welcom ! Please contact <iena-contact@univ-lorraine.fr>.
+Contributors are welcome ! Please contact <iena-contact@univ-lorraine.fr>.
## Contact
@@ -30,23 +28,33 @@ Other : <iena-contact@univ-lorraine.fr>.
## Activation et paramétrage
-Application folder name have to be : `/blocks/mahara_iena`.
+Application folder path and name _have to be_ : `/blocks/mahara_iena`.
Le fonctionnement du plugin nécessite :
-1. D'avoir liées les plateformes MOODLE et MAHARA pour la création de compte et l'autentification.
+1. D'avoir liées les plateformes MOODLE et MAHARA pour la création de compte et l'autentification (en Mnet). Cf. documentation de Mahara.
2. D'avoir un utilisateur MAHARA disposant d'un token d'API lié aux services exploités par ce plugin.
Le second point spécifique à ce plugin est documenté ici : [Documentation bloc Mahara iENA](iena-mahara-link-block-documentation.pdf).
## Fonctionnalités
-L'enseignant peut activer le bloc et lier un cours à un groupe Mahara. Les étudiants pourront accéder au groupe Mahara en un clic. Si le cours Moodle comporte des groupes, alors un groupe Mahara peut être créé pour chaque groupe Moodle du cours. Attention, ce choix doit être fait à l'activation et est définitif. Il n'est pas possible de lier à un cours Moodle un groupe existant.
+L'enseignant peut activer le bloc et lier un cours à un groupe Mahara. Les étudiants pourront accéder au groupe Mahara en un clic. Si le cours Moodle comporte des groupes, alors un groupe Mahara peut être créé pour chaque groupe Moodle du cours. Attention, ce choix doit être fait à l'activation et est définitif. Il n'est pas possible de lier à un cours Moodle un groupe Mahara existant.
+
+La connexion, l'éventuelle création de compte utilisateur et la création de groupe sur Mahara est prise en charge par le plugin. Mais le lien entre les deux plateformes doit préalablement exister (paramétrage préalable).
-La connexion, l'éventuelle création de compte utilisateur et la création de groupe sur Mahara est prise en charge par le plugin. Le lien entre les deux plateformes doit préalablement exister (paramétrage préalable).
+L'enseignant éditeur peut s'inscrire dans tous les groupes Mahara liés à son cours (créés depuis son cours) en un clic.
Usecase : place 3 groups with 1 teacher and n students in a moodle course and create in one click 3 mahara groups where each teacher will be able to administrate his mahara group and each student will be able to access the right group in mahara, based on the moodle group.
+NB : les groupes et les inscription sur Mahara et dans les groupes Mahara ne sont pas créées immédiatement, mais au fur et à mesure des demandes d'accès. C'est au moment où un étudiant cliquera sur le bouton du block pour se rendre pour la première fois dans son groupe que le groupe sera créé au besoin, l'étudiant sera inscrit sur Mahara au besoin et dans le groupe au besoin.
+
+NB 2 : la connexion préalablement existante entre les deux plateforme doit être en Mnet.
+
+NB 3 : il est fortement conseillé de prendre garde aux méthodes d'authentification utilisées dans Mahara pour ne pas créer de duplicatats de comptes utilisateur, cf. documentation de Mahara.
+
+NB 4 : le nom du groupe Mahara généré contient le nom du cours et l'utilisateur de l'API y est inscrit. Ceci devrait permettre de supprimer des groupes inactifs automatiquement pour des opérations de nettoyage.
+
---
By clicking a button in this block, editing teachers can create one mahara group within every student enrolled in the moodle course will become members of the mahara group. Moodle course enrolled users with the role of "teacher" and "editingteacher" will have the role of "admin" in the mahara group, other will be simple members.
@@ -57,11 +65,10 @@ Alternatively, if they are user groups within the moodle cours, teacher can choo
Usecase : place 3 groups with 1 teacher and n students in a moodle course and create in one click 3 mahara groups where each teacher will be able to administrate his mahara group and each student will be able to access the right group in mahara, based on the moodle group.
-## Amélioration à effectuer
+## Roadmap
- Liaison d'un groupe Mahara existant au cours
- Pré-création de tous les groupes Mahara liés au cours en un clic sans inscription de l'enseignant dans tous ces groupes
-- Inscription de l'enseignant dans tous les groupes en un clic
- Management des groupes Mahara liés au cours (liaison d'un groupe Mahara existant à un groupe du cours ou au cours, suppression d'un groupe Mahara préalablement lié au cours)
- Passage possible du mode un groupe Mahara pour tout le cours Moodle à un groupe Mahara par groupe Moodle et vice-versa.
-- Gestion des groupements : actuellement il est incompatible avec les groupements.
\ No newline at end of file
+- Fonctionnement avec LTI au lien de Mnet (à l'étude) : help wanted !!!
diff --git a/block_mahara_iena.php b/block_mahara_iena.php
index 639e24cead1c5651cdba6558f607653d844d431b..1104c7900fbe0cfdfd0f6df76b52922321e18864 100644
--- a/block_mahara_iena.php
+++ b/block_mahara_iena.php
@@ -64,7 +64,7 @@ class block_mahara_iena extends block_base
if (count($result) == 0 && $is_mahara_user) {
// create one mahara group for the course
- $this->content->text .= '<a href="' . $CFG->wwwroot . '/blocks/mahara_iena/mahara_iena.php?courseid='
+ $this->content->text = '<a href="' . $CFG->wwwroot . '/blocks/mahara_iena/mahara_iena.php?courseid='
. $COURSE->id . '" class="btn btn-block btn-primary iena-btn-big">' . get_string('create_group', 'block_mahara_iena', $CFG->mahara_alias) . '</a><p class="text-muted">Créer un groupe dans '.$CFG->mahara_alias.'.</p>';
// create one mahara group for each moodle group (need groups)
@@ -141,10 +141,10 @@ class block_mahara_iena extends block_base
if ( count($ma_links) != 0 ) {
if ( count($ma_links) == 1 ) {
/* si il n'y a qu'un groupe, on affiche un bouton */
- $this->content->text .= "<a target='_blank' class='btn btn-block btn-primary iena-btn-big' href='$CFG->wwwroot/blocks/mahara_iena/mahara_iena_link.php?courseid=$COURSE->id&mahara_group=" . $ma_links[0]['ma_group_id'] . "&moodle_group=" . $ma_links[0]['mo_gr_id'] . "'>Aller à mon groupe sur ".$CFG->mahara_alias."</a><p class='alert alert-secondary'>Mon groupe : ".$ma_links[0]['mo_gr_name'].".</p>";
+ $this->content->text = "<a target='_blank' class='btn btn-block btn-primary iena-btn-big' href='$CFG->wwwroot/blocks/mahara_iena/mahara_iena_link.php?courseid=$COURSE->id&mahara_group=" . $ma_links[0]['ma_group_id'] . "&moodle_group=" . $ma_links[0]['mo_gr_id'] . "'>Aller à mon groupe sur ".$CFG->mahara_alias."</a><p class='alert alert-secondary'>Mon groupe : ".$ma_links[0]['mo_gr_name'].".</p>";
} elseif ( count($ma_links) > 1 && count($ma_links) <= 5 ) {
/* si il y a entre 2 et 5 groupes inclus, on affiche une liste */
- $this->content->text .= "<p style='border-left: 2px solid #1177d1;padding-left: 0.5rem;font-weight: 600;margin-bottom: 0.5rem;'>Accéder à mon groupe sur $CFG->mahara_alias : </p>
+ $this->content->text = "<p style='border-left: 2px solid #1177d1;padding-left: 0.5rem;font-weight: 600;margin-bottom: 0.5rem;'>Accéder à mon groupe sur $CFG->mahara_alias : </p>
<ul class='iena-dashed p-l-0'>";
foreach ($ma_links as $ma_link) {
$this->content->text .= "<li>Groupe : <a target='_blank' href='$CFG->wwwroot/blocks/mahara_iena/mahara_iena_link.php?courseid=$COURSE->id&mahara_group=" . $ma_link['ma_group_id'] . "&moodle_group=" . $ma_link['mo_gr_id'] . "'>" . $ma_link['mo_gr_name'] . "</a></li>";
@@ -170,7 +170,7 @@ class block_mahara_iena extends block_base
} else {
/* quand un seul groupe mahara pour tout le cours ($hasgroup à false) */
- $this->content->text .= "<a target='_blank' class='btn btn-block btn-primary iena-btn-big' href='$CFG->wwwroot/blocks/mahara_iena/mahara_iena_link.php?courseid=$COURSE->id&mahara_group=" . reset($result)->mahara_group_id . "&moodle_group=nogroups'>Aller au groupe sur ".$CFG->mahara_alias."</a>";
+ $this->content->text = "<a target='_blank' class='btn btn-block btn-primary iena-btn-big' href='$CFG->wwwroot/blocks/mahara_iena/mahara_iena_link.php?courseid=$COURSE->id&mahara_group=" . reset($result)->mahara_group_id . "&moodle_group=nogroups'>Aller au groupe sur ".$CFG->mahara_alias."</a>";
}
/* permet à l'enseignant de créer tous les groupes mahara en fonction des groupes moodle existant à un temps t. Ca n'inscrit pas les utilisateurs à part lui. */
if ( $has_groups && has_capability('moodle/course:update', $context = context_course::instance($COURSE->id), $USER->id) ) {
diff --git a/entity/block_mahara_iena_connexion.php b/entity/block_mahara_iena_connexion.php
index f59fc7b03a8c4fc16854e003d0321f3cd1dca3e0..ac34f74b204e64968a6fae796e83edf92001330a 100644
--- a/entity/block_mahara_iena_connexion.php
+++ b/entity/block_mahara_iena_connexion.php
@@ -88,7 +88,7 @@ class block_mahara_iena_connexion
$mahara_group_get_groups_by_id = "mahara_group_get_groups_by_id";
$url = $this->create_url($mahara_group_get_groups_by_id);
$res = json_decode($this->httpPost($params, $url), true);
- return $res[0]->members;
+ return $res[0]['members'];
}
public function get_mahara_user_by_mail($email) {
@@ -254,7 +254,7 @@ class block_mahara_iena_connexion
*/
public function ensure_user_exists_in_mahara($user) {
$is_a_mahara_user = $this->get_mahara_user_by_mail($user->email);
- if ( $is_a_mahara_user->error ) {
+ if ( isset($is_a_mahara_user->error) ) {
return false;
} else {
return true;
@@ -299,7 +299,7 @@ class block_mahara_iena_connexion
$params = http_build_query($params);
$url = $this->create_url("mahara_group_create_groups");
$res = json_decode($this->httpPost($params, $url));
- if ($res->error) {
+ if ( isset($res->error) ) {
return false;
} else {
$record = new stdClass();
@@ -358,7 +358,7 @@ class block_mahara_iena_connexion
$params = http_build_query($params);
$url = $this->create_url("mahara_group_create_groups");
$res = json_decode($this->httpPost($params, $url));
- if ($res->error) {
+ if ( isset($res->error) ) {
return false;
} else {
$record = new stdClass();
@@ -373,7 +373,7 @@ class block_mahara_iena_connexion
$ma_group_members = $this->get_mahara_users_in_mahara_group_by_id($ma_group_id);
$is_in_group = false;
foreach ($ma_group_members as $member) {
- if ( $mo_user->email == $this->get_mahara_user_by_id($member->id)[0]->email ) {
+ if ( $mo_user->email == $this->get_mahara_user_by_id($member['id'])[0]->email ) {
$is_in_group = true;
break;
}
diff --git a/mahara_iena.php b/mahara_iena.php
index 488cbb508bc03994d62f1498554aaa663a8b83da..efb9217f625cb66d870b56cf8e045b356d512767 100644
--- a/mahara_iena.php
+++ b/mahara_iena.php
@@ -65,15 +65,15 @@ if (count($students) == 0) {
// get mahara user by mail (teacher)
$mahara_user = $connexion->get_mahara_user_by_mail($USER->email);
// if user is not a mahara user, there will be an error : it's not supposed to be because mahara account is checked before to have a button to go in this page but... but.
- if ( !$mahara_user->error ) {
+ if ( isset($mahara_user->error) ) {
+ echo "<p style='color: red;'>You must have a Mahara account to do that.</p>";
+ die;
+ } else {
array_push($usersTab,array(
'id' => $mahara_user[0]->id,
'username' => $mahara_user[0]->username,
'role' => "admin",
));
- } else {
- echo "<p style='color: red;'>You must have a Mahara account to do that.</p>";
- die;
}
//All params used for mahara_group_create_groups her we can add/change some lines
diff --git a/mahara_iena_groups.php b/mahara_iena_groups.php
index d1c3e71d73d9a02d71aaef1735666a0f218cc79d..b780fae491c532a8668a32a45f58bf0f9a1eb8c0 100644
--- a/mahara_iena_groups.php
+++ b/mahara_iena_groups.php
@@ -56,15 +56,15 @@ if (count($students) == 0) {
// get mahara user by mail (teacher)
$mahara_user = $connexion->get_mahara_user_by_mail($USER->email);
// if user is not a mahara user, there will be an error : it's not supposed to be because mahara account is checked before to have a button to go in this page but... but.
- if ( !$mahara_user->error ) {
+ if ( isset($mahara_user->error) ) {
+ echo "<p style='color: red;'>You must have a Mahara account to do that.</p>";
+ die;
+ } else {
array_push($usersTab,array(
'id' => $mahara_user[0]->id,
'username' => $mahara_user[0]->username,
'role' => "admin",
));
- } else {
- echo "<p style='color: red;'>You must have a Mahara account to do that.</p>";
- die;
}
if ( count($usersTab) == 0 ) {
diff --git a/mahara_iena_link.php b/mahara_iena_link.php
index 59dba10b8df738566818c94042001bd925e16467..8fd2a30d7132d1361199824bd4f69dd2bd79108f 100644
--- a/mahara_iena_link.php
+++ b/mahara_iena_link.php
@@ -15,7 +15,7 @@ require_login($courseid, false, NULL);
$content = "";
-if ( $_GET['connect_mahara'] == true ) {
+if ( isset($_GET['connect_mahara']) && $_GET['connect_mahara'] == true ) {
// connect the user to mahara to create his account with moodle/mahara existing link
$mnetauth = get_auth_plugin('mnet');
$target_url = "/";
@@ -31,7 +31,7 @@ if ( $_GET['connect_mahara'] == true ) {
//ensure user is member of it's mahara group : true if already or succeed
// $all_right = $connexion->ensure_user_is_mahara_group_member($USER, $_GET['mahara_group']);
- if ( $_GET['connect_direct'] == true ) {
+ if ( isset($_GET['connect_direct']) && $_GET['connect_direct'] == true ) {
$all_right = true;
} else {
$all_right = $connexion->ensure_user_is_mahara_group_member_new($USER, $_GET['mahara_group'], $_GET['moodle_group']);
diff --git a/version.php b/version.php
index 0290ba5972ca3974bdf4ce3a9ae4eaf55d35b561..5529aaa4f7075e52937a67bf3ef17739a22ddd74 100644
--- a/version.php
+++ b/version.php
@@ -28,7 +28,7 @@
*/
defined('MOODLE_INTERNAL') || die();
-$plugin->version = 2019052201;
+$plugin->version = 2019062101;
$plugin->requires = 2014051200;
$plugin->component = 'block_mahara_iena';
$plugin->release = 'v1.0';