diff --git a/CHANGELOG.md b/CHANGELOG.md new file mode 100644 index 0000000000000000000000000000000000000000..f9050885fb4cc06cb5a8413e4b6a5c6ba8ba701a --- /dev/null +++ b/CHANGELOG.md @@ -0,0 +1,7 @@ +# 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. diff --git a/README.md b/README.md index bf50ab3fefb9d2796694b38e00e1f30ef850ae95..7916341f158c6f01ee4ce31020be0fdb7bad77cc 100644 --- a/README.md +++ b/README.md @@ -12,7 +12,7 @@ This plugin is a block that provide a link between a Moodle course and a Mahara ## Compatibility -MOODLE 3.5 +MOODLE 3.5.4+ Stabilité : expérimental. @@ -30,6 +30,8 @@ Other : . ## Activation et paramétrage +Application folder : `mahara_iena`. + __Documentation à préciser :__ - création du user mahara avec les bons droits pour l'api - paramétrage du plugin dans moodle diff --git a/block_mahara_iena.php b/block_mahara_iena.php index f450f05c8aaf835314fe9d2b0e1ee490003a8177..265b64e516866229211a49f1f1ce63fb978af411 100644 --- a/block_mahara_iena.php +++ b/block_mahara_iena.php @@ -171,7 +171,8 @@ class block_mahara_iena extends block_base } /* 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) ) { - $this->content->text .= 'Créer tous les groupes

Créer manuellement tous les groupes sur '.$CFG->mahara_alias.' liés à ce cours et s\'y inscrire en tant qu\'enseignant.

'; + $this->content->text .= 'Créer tous les groupes

Créer manuellement tous les groupes sur '.$CFG->mahara_alias.' liés à ce cours et s\'y inscrire en tant qu\'enseignant.

'; + $this->content->text .= ""; // $this->content->text .= 'S\'inscrire dans tous les groupes

S\'inscrire en tant qu\'enseignant dans tous les groupes '.$CFG->mahara_alias.' liés à ce cours.

'; /* POUR PLUS TARD : liste les groupes moodle liés à mahara qui n'existent plus dans moodle */ diff --git a/entity/block_mahara_iena_connexion.php b/entity/block_mahara_iena_connexion.php index 232ef1dca2d3ec5f72f2a7a1ce74bf5f5b20885e..5798180a9e8db068eefd59e6e316139687c03601 100644 --- a/entity/block_mahara_iena_connexion.php +++ b/entity/block_mahara_iena_connexion.php @@ -365,7 +365,7 @@ class block_mahara_iena_connexion $record->mahara_group_id = $res[0]->id; $record->moodle_group_id = $mo_user_group_id; $DB->insert_records('block_mahara_iena', array( 0 => $record ), false); - return true; + return $res[0]->id; } } else { diff --git a/mahara_iena_link.php b/mahara_iena_link.php index d6be9440576f2a35e925ed24e9db8a3a39fbe414..6b491f3394fe82bb77700b407299863f9c202c96 100644 --- a/mahara_iena_link.php +++ b/mahara_iena_link.php @@ -34,12 +34,20 @@ if ( $_GET['connect_mahara'] == true ) { $all_right = $connexion->ensure_user_is_mahara_group_member_new($USER, $_GET['mahara_group'], $_GET['moodle_group']); - if ( ! $all_right ) { + if ( $all_right === false ) { $content .= '
Erreur d\'ajout au groupe sur '.$CFG->base_mahara.'.
'; } else { /* connect to mahara and redirect to the mahara group */ $mnetauth = get_auth_plugin('mnet'); - $group_url = "/group/view.php?id=".$_GET['mahara_group']; + $ma_group = $_GET['mahara_group']; + if ( $ma_group == "" || $ma_group == NULL ) { + if ( gettype($all_right) == "integer" ) { + $ma_group = $all_right; + } else { + $content .= '
La redirection vers le groupe a échouée, veuillez réessayer.
'; + } + } + $group_url = "/group/view.php?id=".$ma_group; $url = $mnetauth->start_jump_session($CFG->iena_mahara, $group_url); if (empty($url)) { $content .= '
La connexion a échouée.
';