MDL-9038: Groups can now be created outside of any groupings. The number of groups...

diff --git a/group/group.php b/group/group.php
index 060f70a031886d54afb29ffc4e1a067e341b09dc..260a3578e7735aec3437ed645b4ef7588c5a6910 100644 (file)
--- a/group/group.php
+++ b/group/group.php
@@ -68,17 +68,29 @@ if ($success) {
         }
         elseif (isset($frm->update)) {
             if (! $groupid) { //OK, new group.
+                // Allow groups to be created outside of groupings
+                /*
                 if (GROUP_NOT_IN_GROUPING == $groupingid) {
                     print_error('errornotingrouping', 'group', groups_home_url($courseid), get_string('notingrouping', 'group'));
                 }
+                */
                 $success = (bool)$groupid = groups_create_group($courseid); //$groupsettings);
-                $success = groups_add_group_to_grouping($groupid, $groupingid);
+                
+                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
+                /*
                 if (GROUP_NOT_IN_GROUPING == $newgrouping) {
                     print_error('errornotingrouping', 'group', groups_home_url($courseid), get_string('notingrouping', 'group'));
                 }
-                $success = $success && groups_remove_group_from_grouping($groupid, $groupingid);
+                */
+                if ($groupingid != GROUP_NOT_IN_GROUPING) {
+                    $success = $success && groups_remove_group_from_grouping($groupid, $groupingid);
+                }
+
                 $success = $success && groups_add_group_to_grouping($groupid, $newgrouping);
             }
             if ($success) {

diff --git a/group/index.php b/group/index.php
index c47c597a36af53caa6deb85cd6ff1f6edb94477b..b9b416bcfdbed279f4d273f074815b12610fa505 100644 (file)
--- a/group/index.php
+++ b/group/index.php
@@ -115,9 +115,12 @@ if ($success) {
         case 'removegroup':
             break;
         case 'showcreategroupform':
+            // Allow groups to be created outside of groupings
+            /*
             if (GROUP_NOT_IN_GROUPING == $groupingid) {
                 print_error('errornotingrouping', 'group', groups_home_url($courseid), get_string('notingrouping', 'group'));
             }
+            */
             redirect(groups_group_edit_url($courseid, null, $groupingid, false));
             break;
         case 'addgroupstogroupingform':