L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-10311 Group import broken - groups import hadn't been updated for new groups,

Browse Source 
also missing lang strings
This commit is contained in:
poltawski
2007-07-06 15:26:35 +00:00
parent ed200fd420
commit c278e8bc8f
1 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
course/import/groups/index.php
+7 -10
View File
@@ -4,6 +4,7 @@
require_once('../../../config.php');
require_once('../../lib.php');
require_once($CFG->dirroot . '/group/lib/basicgrouplib.php');
$mycourseid = required_param('id', PARAM_INT); // Course id
@@ -67,9 +68,9 @@
$optional = array("coursename" => 1,
"idnumber" =>1,
"description" => 1,
"password" => 1,
"enrolmentkey" => 1,
"theme" => 1,
"picture" => 1,
"picture" => 1,
"hidepicture" => 1, );
// --- get header (field names) ---
@@ -117,10 +118,6 @@
get_string('processingstops', 'error'),
'uploaduser.php?sesskey='.$USER->sesskey);
}
// password needs to be encrypted
else if ($name == "password") {
$newgroup->password = md5($value);
}
else if ($name == "groupname") {
$newgroup->name = addslashes($value);
}
@@ -133,7 +130,7 @@
//if idnumber is set, we use that.
//unset invalid courseid
if ($newgroup->idnumber){
if (isset($newgroup->idnumber)){
if (!$mycourse = get_record('course', 'idnumber',$newgroup->idnumber)){
notify(get_string('unknowncourseidnumber', 'error', $newgroup->idnumber));
unset($newgroup->courseid);//unset so 0 doesnt' get written to database
@@ -143,7 +140,7 @@
//else use course short name to look up
//unset invalid coursename (if no id)
else if ($newgroup->coursename){
else if (isset($newgroup->coursename)){
if (!$mycourse = get_record('course', 'shortname',$newgroup->coursename)){
notify(get_string('unknowncourse', 'error', $newgroup->coursename));
unset($newgroup->courseid);//unset so 0 doesnt' get written to database
@@ -167,11 +164,11 @@
if (!has_capability('moodle/course:managegroups', $newgrpcoursecontext)){
notify("$newgroup->name ".get_string('notaddedto').$newgroup->coursename.get_string('notinyourcapacity'));
} else {
if (get_record("groups","name",$groupname,"courseid",$newgroup->courseid) || !($newgroup->id = insert_record("groups", $newgroup))) {
if ( $group = groups_group_name_exists($newgroup->courseid, $groupname) || !($newgroup->id = groups_create_group($newgroup->courseid, $newgroup)) ) {
//Record not added - probably because group is already registered
//In this case, output groupname from previous registration
if ($group = get_record("groups","name",$groupname)) {
if ($group) {
notify("$newgroup->name ".get_string('groupexistforcourse', 'error', $groupname));
} else {
notify(get_string('groupnotaddederror', 'error', $groupname));