mirror_php/moodle
1
0
Fork 0
mirror of https://github.com/moodle/moodle.git synced 2025-01-18 05:58:34 +01:00
Code Issues Projects Releases Wiki Activity

Follow-up fix for Bug MDL-8638 "Can't move groups between groupings"

Browse Source
This commit is contained in:
nfreear 2007-02-26 10:55:45 +00:00
parent fb6e87f373
commit f677bf92cb
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
group/lib/groupinglib.php
View File

@ -65,7 +65,7 @@ function groups_get_groups_in_grouping_records($groupingid) {
}
$grouping_groups = get_records('groups_groupings_groups', 'groupingid ',
$groupingid, '', $fields='id, groupid, timeadded');
return $grouping_groups;
}
@ -231,12 +231,10 @@ function groups_user_is_in_multiple_groups($userid, $groupingid) {
array_push($groupinggroupids, $groupid);
}
}
if (count($groupinggroupids) > 1) {
$inmultiplegroups = true;
}
}
return $inmultiplegroups;
}
@ -409,7 +407,7 @@ function groups_add_group_to_grouping($groupid, $groupingid) {
} else {
$groupadded = true;
}
return $groupadded;
}
@ -451,6 +449,10 @@ function groups_set_grouping_for_coursemodule($groupingid, $coursemoduleid) {
* @return boolean True if the deletion was successful, false otherwise.
*/
function groups_remove_group_from_grouping($groupid, $groupingid) {
if (GROUP_NOT_IN_GROUPING == $groupingid) {
//Quietly ignore.
return true;
}
return groups_db_remove_group_from_grouping($groupid, $groupingid);
}
@ -461,6 +463,9 @@ function groups_remove_group_from_grouping($groupid, $groupingid) {
* @return boolean True if the deletion was successful, false otherwise.
*/
function groups_delete_grouping($groupingid) {
if (GROUP_NOT_IN_GROUPING == $groupingid) {
return false;
}
return groups_db_delete_grouping($groupingid);
}