Jérôme Mouneyrac added a comment - 15/Mar/11 5:25 PM Not that the fix will have a very weak impact as the only place where $status argument is used is in this backup call: backup/moodle2/restore_stepslib.php /** * Create user enrolments * * This has to be called after creation of enrolment instances * and before adding of role assignments. * * @param mixed $data * @return void */ public function process_enrolment($data) { global $DB;   $data = (object)$data;   // Process only if parent instance have been mapped if ($enrolid = $this->get_new_parentid('enrol')) { if ($instance = $DB->get_record('enrol', array('id'=>$enrolid))) { // And only if user is a mapped one if ($userid = $this->get_mappingid('user', $data->userid)) { $enrol = enrol_get_plugin($instance->enrol); //TODO: do we need specify modifierid? $enrol->enrol_user($instance, $userid, null, $data->timestart, $data->timeend, $data->status); //note: roles are assigned in restore_ras_and_caps_structure_step::process_assignment() processing above } } } }

Jérôme Mouneyrac added a comment - 15/Mar/11 5:28 PM patch: https://github.com/mouneyrac/moodle/commit/d587f077ef87367dc9d69ecfee4320ef0a1bc090

Helen Foster added a comment - 24/Mar/11 4:01 AM Thanks Jerome