diff --git a/group/group.php b/group/group.php index 260a3578e77..fdab5a6d8ca 100644 --- a/group/group.php +++ b/group/group.php @@ -76,9 +76,6 @@ if ($success) { */ $success = (bool)$groupid = groups_create_group($courseid); //$groupsettings); - if ($groupingid != GROUP_NOT_IN_GROUPING) { - $success = groups_add_group_to_grouping($groupid, $groupingid); - } } elseif ($groupingid != $newgrouping) { //OK, move group. // Allow groups to be created outside of groupings @@ -159,109 +156,112 @@ if ($success) { "-> $strgroups", '', '', true, '', user_login_string($course, $USER)); $usehtmleditor = false; -?> -
-'."\n"; - - $strgroups", '', '', true, '', user_login_string($course, $USER)); $usehtmleditor = false; - //TODO: eventually we'll implement all buttons, meantime hide the ones we haven't finised. - $shownotdone = false; + //TODO: eventually we'll implement all buttons, meantime hide the ones we haven't finished. + $shownotdone = false; + $disabled = 'disabled="disabled"'; + + // Pre-disable buttons based on URL variables + if (isset($groupingid) && $groupingid > -1) { + $showeditgroupsettingsform_disabled = ''; + $showeditgroupingsettingsform_disabled = ''; + $deletegroup_disabled = ''; + $deletegrouping_disabled = ''; + $printerfriendly_disabled = ''; + $showcreategroupform_disabled = ''; + } else { + $showeditgroupsettingsform_disabled = $disabled; + $showeditgroupingsettingsform_disabled = $disabled; + $deletegroup_disabled = $disabled; + $deletegrouping_disabled = $disabled; + $printerfriendly_disabled = $disabled; + $showcreategroupform_disabled = $disabled; + } + + if (isset($groupid)) { + $showaddmembersform_disabled = ''; + $showeditgroupsettingsform_disabled = ''; + $deletegroup_disabled = ''; + } else { + $deletegroup_disabled = $disabled; + $showeditgroupsettingsform_disabled = $disabled; + $showaddmembersform_disabled = $disabled; + } print_heading(format_string($course->shortname) .' '.$strgroups, 'center', 3); echo '