From: poltawski <poltawski>
Date: Thu, 27 Dec 2007 13:40:27 +0000 (+0000)
Subject: MDL-12345 - use consistent ordering on both sides of the assign group members
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=47d1ec41ce425a5b2773aecd472767c3a85500cd;p=moodle.git

MDL-12345 - use consistent ordering on both sides of the assign group members
pane, sorted by lastname like other screens
merged from MOODLE_19_STABLE
---

diff --git a/group/lib.php b/group/lib.php
index 84bee7ecc0..0f9a495fda 100644
--- a/group/lib.php
+++ b/group/lib.php
@@ -308,7 +308,7 @@ function groups_delete_groupings($courseid, $showfeedback=false) {
  * This function was changed to get_users_by_capability style
  * mostly because of the searchtext requirement
  */
-function groups_get_users_not_in_group($courseid, $groupid, $searchtext='') {
+function groups_get_users_not_in_group($courseid, $groupid, $searchtext='', $sort = 'u.lastname ASC') {
 
     global $CFG;
 
@@ -369,8 +369,9 @@ function groups_get_users_not_in_group($courseid, $groupid, $searchtext='') {
                                    WHERE groupid = $groupid)
                   $wheresearch";
     $groupby = " GROUP BY u.id, u.firstname, u.lastname ";
+    $orderby = " ORDER BY $sort";
 
-    return get_records_sql($select.$from.$where.$groupby);
+    return get_records_sql($select.$from.$where.$groupby.$orderby);
 }
 
 
@@ -490,4 +491,4 @@ function groups_unassign_grouping($groupingid, $groupid) {
     return delete_records('groupings_groups', 'groupingid', $groupingid, 'groupid', $groupid);
 }
 
-?>
\ No newline at end of file
+?>
diff --git a/group/members.php b/group/members.php
index d1cac6d362..51981eb136 100644
--- a/group/members.php
+++ b/group/members.php
@@ -116,7 +116,6 @@ if ($potentialmemberscount <=  MAX_USERS_PER_PAGE) {
         foreach ($potentialmembers as $userid => $user) {
             $nonmembers[$userid] = fullname($user)." (".@count($usergroups[$userid]).")";
         }
-        natcasesort($nonmembers);
 
         // Print out the HTML
         foreach($nonmembers as $id => $name) {