From: moodler <moodler>
Date: Sun, 1 Oct 2006 08:37:15 +0000 (+0000)
Subject: Tidied up groups interface to make it a bit more intuitive
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=5b1d59951450fccfeb7d1b3c4396819b789d4aba;p=moodle.git

Tidied up groups interface to make it a bit more intuitive
---

diff --git a/course/groups-edit.html b/course/groups-edit.html
index 05620d4443..ed0cf2a0e9 100755
--- a/course/groups-edit.html
+++ b/course/groups-edit.html
@@ -1,8 +1,8 @@
 <script type="text/javascript">
 <!-- Begin
 <?php 
+    $roleoptions = array(0 => get_string('all'));
     $roles = get_all_roles();
-    $roleoptions = array();
     foreach ($roles as $rolex) {
         $roleoptions[$rolex->id] = $rolex->name;
     }
@@ -80,6 +80,11 @@ function groupWindow(selectgroup) {
 // end hiding script -->
 </script>
 
+        <form name="rolesform1" action="groups.php" method="get">
+        <input type="hidden" name="id" value="<?php echo $courseid ?>">
+        <div align="center"><?php echo get_string('currentrole', 'role') ?>: 
+            <?php choose_from_menu ($roleoptions, 'roleid', $roleid, '', $script='rolesform1.submit()') ?>
+        </div></form>
 
   <table cellspacing="0" cellpadding="10" align="center" class="generaltable generalbox">
     <tr>
@@ -123,11 +128,6 @@ function groupWindow(selectgroup) {
                    onclick="return userWindow(document.form1['nonmembers[]']);" />
           </p>
         </form>
-        <form name="rolesform1" action="groups.php" method="get">
-        <input type="hidden" name="id" value="<?php echo $courseid ?>">
-        <div align="center"><?php echo get_string('selectrole', 'role') ?>: 
-            <?php choose_from_menu ($roleoptions, 'roleid', $roleid, 'choose', $script='rolesform1.submit()') ?>
-        </div></form>
       </td>
       <td class="generalboxcontent"><p>
         <form name="form2" id="form2" method="post" action="groups.php">
diff --git a/course/groups.php b/course/groups.php
index 5d7a8ef0d8..36ddd19043 100644
--- a/course/groups.php
+++ b/course/groups.php
@@ -30,7 +30,7 @@
 
     $courseid      = required_param('id', PARAM_INT);           // Course id
     $selectedgroup = optional_param('group', NULL, PARAM_INT);  // Current group id
-    $roleid        = optional_param('roleid', 0, PARAM_INT);  // Current group id
+    $roleid        = optional_param('roleid', 0, PARAM_INT);    // Current role id
 
     if (! $course = get_record('course', 'id', $courseid) ) {
         error("That's an invalid course id");