Commit 1d1bd169 authored by Camille Tardy's avatar Camille Tardy
Browse files

Response to Moodle comments on plugin (CONTRIB-7218): fix naming in localib...


Response to Moodle comments on plugin (CONTRIB-7218): fix naming in localib according to comvention, Mention privacy in Readme, fix small coding warning in code analysis
Signed-off-by: Camille Tardy's avatarCamille Tardy <camille.tardy@unige.ch>
parent fdfd9a92
......@@ -114,7 +114,7 @@ class quiz_group_observer{
}else {
//ERROR : Grp attempt not in DB
//create grp_attempt if not in DB
create_grpattempt_from_attempt($attempt,$course_id);
create_groupattempt_from_attempt($attempt,$course_id);
}
}
......@@ -203,7 +203,7 @@ class quiz_group_observer{
public static function course_reset_ended($event) {
if (!empty(self::$resetinprogress)) {
if (!empty($event->other['reset_options']['reset_groups_remove'])) {
quiz_process_grp_deleted_in_course($event->courseid);
quiz_process_group_deleted_in_course($event->courseid);
}
}
......@@ -221,7 +221,7 @@ class quiz_group_observer{
// We will take care of that once the course reset ends.
return;
}
quiz_process_grp_deleted_in_course($event->courseid);
quiz_process_group_deleted_in_course($event->courseid);
}
}
\ No newline at end of file
......@@ -27,7 +27,7 @@ $string['quizgroup'] = 'Group quiz';
$string['activatebygroup'] = 'Do quiz for group of student?';
$string['no_grouping'] = 'Standard Quiz (no grouping)';
$string['info_bygroup'] = 'Only one student per group answers the quiz questions. The grade is copied to all the members of the group using the second form below. <br/> If you need to change any grade in the test, you must use the Responses view and not the Gradebook. Then you can apply again the group grade to all the members of the group.';
$string['warning_group'] = "Careful: <ul> <li>The group quiz uses a specific grouping. Make sure to give the appropriate access rule in the 'Restrict access' section of the quiz main settings page. </li><li/>If a student is in more than one groupe in the same grouping, the system will only consider his first affiliation.</li><li>Do not change the grouping if attempts exist for this quiz!</li></ul>";
$string['warning_group'] = "Careful: <ul> <li>The group quiz uses a specific grouping. Make sure to give the appropriate access rule in the 'Restrict access' section of the quiz main settings page. </li><li>If a student is in more than one groupe in the same grouping, the system will only consider his first affiliation.</li><li>Do not change the grouping if attempts exist for this quiz!</li></ul>";
$string['error_groupingquizsetting'] = 'The selected grouping for access and quiz must be the same.';
$string['savechanges'] = 'Save';
$string['settings_edited'] = 'The quiz has been edited with the following grouping: {$a}';
......
......@@ -28,7 +28,7 @@ $string['quizgroup'] = 'Test en groupe';
$string['activatebygroup'] = 'Faire passer le test par groupe ?';
$string['no_grouping'] = 'Test standard (pas de groupes)';
$string['info_bygroup'] = 'Seul un étudiant répond au test. La même note est distribuée à tous les membres du groupe en utilisant le formulaire ci dessous.<br/> Pour changer une note, il faut utiliser la vue des Réponses et non pas le Carnet de note. Vous pourrez alors copier la note à tous les membres du groupe.';
$string['warning_group'] = 'Attention : <ul> <li>Le test en groupe utilise un groupement spécifique. Assurez vous de donner les accès appropriés dans la section "Restreindre l\'acces" dans les paramètres du quiz.</li><li/>Si un étudiant est dans plusieurs groupes d\'un même groupement, le système ne considèrerea que sa première affiliation.</li><li>Ne pas changer le groupement si des tentatives sont déjà enregistrées pour ce test!</li></ul>';
$string['warning_group'] = 'Attention : <ul> <li>Le test en groupe utilise un groupement spécifique. Assurez vous de donner les accès appropriés dans la section "Restreindre l\'acces" dans les paramètres du quiz.</li><li>Si un étudiant est dans plusieurs groupes d\'un même groupement, le système ne considèrerea que sa première affiliation.</li><li>Ne pas changer le groupement si des tentatives sont déjà enregistrées pour ce test!</li></ul>';
$string['error_groupingquizsetting'] = 'Le groupement sélectionné pour l\'accès et le test doivent être identiques.';
$string['savechanges'] = 'Enregistrer';
$string['settings_edited'] = 'Le test a bien été modifié avec le groupement : {$a}';
......
......@@ -31,7 +31,7 @@ defined('MOODLE_INTERNAL') || die();
/**
* Return grouping used in Group quiz or false if not found
* @param $quizid
* @return $groupingid
* @return int $groupingid
*/
function get_groupquiz_groupingid($quizid){
global $DB;
......@@ -56,10 +56,11 @@ function get_groupquiz_groupingid($quizid){
* @param int $userid
* @param int $quizid
* @param int $courseid
* @param int $groupingid
*
* @return $grpid
* @return int $grpid
*/
function get_user_group_for_groupquiz($userid, $quizid, $courseid, $groupingID = null){
function get_user_group_for_groupquiz($userid, $quizid, $courseid, $groupingid = null){
// retreive all groups for user
$user_grpids = groups_get_user_groups($courseid, $userid);
//keep only grp ids
......@@ -74,12 +75,12 @@ function get_user_group_for_groupquiz($userid, $quizid, $courseid, $groupingID =
}
// retrieve grouping ID used in Quiz_group
if($groupingID == null ){
$groupingID = get_groupquiz_groupingid($quizid);
if($groupingid == null ){
$groupingid = get_groupquiz_groupingid($quizid);
}
// filter group from grouping.
$grpsingrouping = groups_get_all_groups(intval($courseid), null ,intval($groupingID));
$grpsingrouping = groups_get_all_groups(intval($courseid), null ,intval($groupingid));
$grps_in_ging = array();
//keep only grp ids
foreach ($grpsingrouping as $gp){
......@@ -103,7 +104,11 @@ function get_user_group_for_groupquiz($userid, $quizid, $courseid, $groupingID =
* Transform an attempt obj (event) in a group attempt object to save in DB
*
* @param quiz_attempt $attempt
* @return group_attempt
* @param int $quizid
* @param int $grpid
* @param int $groupingid
*
* @return object $grp_attempt
*/
function quiz_group_attempt_to_groupattempt_dbobject($attempt, $quizid, $grpid, $groupingid){
......@@ -136,7 +141,7 @@ function quiz_group_attempt_to_groupattempt_dbobject($attempt, $quizid, $grpid,
* @param $attempt
* @param $courseid
*/
function create_grpattempt_from_attempt($attempt,$courseid)
function create_groupattempt_from_attempt($attempt,$courseid)
{
global $DB;
......@@ -169,8 +174,9 @@ function create_grpattempt_from_attempt($attempt,$courseid)
/**
* Dispatch grade function.
* @param quizid $quizid
* @param groupingid $groupingID
*
* @param quiz $quiz
* @param int $groupingID
*
*/
function dispatch_grade($quiz, $groupingID) {
......@@ -194,7 +200,7 @@ function dispatch_grade($quiz, $groupingID) {
//if user not in correct grouping do not create
$grpid = get_user_group_for_groupquiz($att->userid, $quizid, $courseid);
if($grpid > 0){
create_grpattempt_from_attempt($att,$courseid);
create_groupattempt_from_attempt($att,$courseid);
}// if user not in grouping do not create grp_attempt
}
}
......@@ -282,7 +288,7 @@ function dispatch_grade($quiz, $groupingID) {
* @param int $courseid The course ID.
* @return void
*/
function quiz_process_grp_deleted_in_course($courseid) {
function quiz_process_group_deleted_in_course($courseid) {
global $DB;
......
......@@ -53,7 +53,7 @@ class quiz_group_report extends quiz_default_report{
*/
public function display($quiz, $cm, $course)
{
global $DB, $PAGE;
global $DB;
// global $OUTPUT;
$this->quiz = $quiz;
......
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