From 298481260e4d5cb7ff4ccd21b33cc2976c277f14 Mon Sep 17 00:00:00 2001
From: vyshane <vyshane>
Date: Mon, 22 Jan 2007 06:57:40 +0000
Subject: [PATCH] Fixed MDL-8183.

---
 group/index.php        |  6 +--
 group/lib/clientlib.js | 83 +++++++++++++++++++++++++-----------------
 2 files changed, 52 insertions(+), 37 deletions(-)

diff --git a/group/index.php b/group/index.php
index 336740dc99..7c1c2d1ec7 100644
--- a/group/index.php
+++ b/group/index.php
@@ -143,9 +143,9 @@ if ($success) {
         <tr>
             <td class="generalboxcontent">
                 <p><label for="groupings"><?php print_string('groupings', 'group'); ?></label></p>
-                <select name="grouping" id="groupings" size="15" class="select">
+                <select name="grouping" id="groupings" size="15" class="select"
+                        onchange="groupsCombo.refreshGroups(this.options[this.selectedIndex].value);">
 <?php
-//onchange="onGroupingChange()"
 
     $groupingids = groups_get_groupings($courseid);
     $groupingids[] = GROUP_NOT_IN_GROUPING;
@@ -165,7 +165,7 @@ if ($success) {
                 $select = ' selected="selected"';
                 $sel_groupingid = $id;
             }
-            echo "<option value=\"$id\"$select onclick=\"groupsCombo.refreshGroups($id);\">$name</option>\n";
+            echo "<option value=\"$id\"$select>$name</option>\n";
             $count++;
         }
     }
diff --git a/group/lib/clientlib.js b/group/lib/clientlib.js
index 66689840b0..8e20f0bc1b 100644
--- a/group/lib/clientlib.js
+++ b/group/lib/clientlib.js
@@ -27,32 +27,33 @@ function UpdatableGroupsCombo(wwwRoot, courseId) {
                     }
                 }
 
-        	    if (groupsComboEl) {
-        	        // Clear the groups combo box.
-                    while (groupsComboEl.firstChild) {
-                        groupsComboEl.removeChild(groupsComboEl.firstChild);
+        	    if (groupsComboEl && o.responseText) {
+        	        //var groups = eval("("+o.responseText+")");
+        	        var groups = eval(o.responseText);
+
+        	        // Populate the groups combo box.
+                    for (var i=0; i<groups.length; i++) {
+                        var optionEl = document.createElement("option");
+                        optionEl.setAttribute("value", groups[i].id);
+                        optionEl.innerHTML = groups[i].name;
+                        groupsComboEl.appendChild(optionEl);
                     }
-            	    if (o.responseText) {
-            	        //var groups = eval("("+o.responseText+")");
-            	        var groups = eval(o.responseText);
-
-            	        // Populate the groups combo box.
-                        for (var i=0; i<groups.length; i++) {
-                            var optionEl = document.createElement("option");
-                            optionEl.setAttribute("value", groups[i].id);
-                            optionEl.setAttribute("onclick",
-                                "membersCombo.refreshMembers("+groups[i].id+");");
-                            optionEl.innerHTML = groups[i].name;
-                            groupsComboEl.appendChild(optionEl);
-                        }
-            	    }
                 }
         	}
         	// Remove the loader gif image.
         	removeLoaderImgs("groupsloader", "groupslabel");
+        },
+
+        failure: function(o) {
+            removeLoaderImgs("membersloader", "memberslabel");
         }
     };
 
+    // Add onchange event to groups combo box.
+    groupsComboEl = document.getElementById("groups");
+    if (groupsComboEl) {
+        groupsComboEl.setAttribute("onchange", "membersCombo.refreshMembers(this.options[this.selectedIndex].value);");
+    }
     // Hide the updategroups input since AJAX will take care of this.
     var updateGroupsButton = document.getElementById("updategroups");
     updateGroupsButton.setAttribute("style", "display:none;");
@@ -64,7 +65,15 @@ function UpdatableGroupsCombo(wwwRoot, courseId) {
 UpdatableGroupsCombo.prototype.refreshGroups = function (groupingId) {
     // Add the loader gif image.
     createLoaderImg("groupsloader", "groupslabel", this.wwwRoot);
-    
+
+    // Clear the groups combo box.
+    var selectEl = document.getElementById("groups");
+    if (selectEl) {
+        while (selectEl.firstChild) {
+            selectEl.removeChild(selectEl.firstChild);
+        }
+    }
+
     var sUrl = this.wwwRoot+"/group/index.php?id="+this.courseId+"&grouping="+groupingId+"&act_ajax_getgroupsingrouping";
     YAHOO.util.Connect.asyncRequest('GET', sUrl, this.connectCallback, null);
 };
@@ -84,26 +93,24 @@ function UpdatableMembersCombo(wwwRoot, courseId) {
         	if (o.responseText !== undefined) {
                 var selectEl = document.getElementById("members");
 
-        	    if (selectEl) {
-        	        // Clear the members combo box.
-                    while (selectEl.firstChild) {
-                        selectEl.removeChild(selectEl.firstChild);
+        	    if (selectEl && o.responseText) {
+        	        var members = eval('('+o.responseText+')');
+
+        	        // Populate the members combo box.
+                    for (var i=0; i<members.length; i++) {
+                        var optionEl = document.createElement("option");
+                        optionEl.setAttribute("value", members[i].id);
+                        optionEl.innerHTML = members[i].firstname+" "+members[i].lastname;
+                        selectEl.appendChild(optionEl);
                     }
-            	    if (o.responseText) {
-            	        var members = eval('('+o.responseText+')');
-
-            	        // Populate the members combo box.
-                        for (var i=0; i<members.length; i++) {
-                            var optionEl = document.createElement("option");
-                            optionEl.setAttribute("value", members[i].id);
-                            optionEl.innerHTML = members[i].firstname+" "+members[i].lastname;
-                            selectEl.appendChild(optionEl);
-                        }
-            	    }
                 }
         	}
         	// Remove the loader gif image.
         	removeLoaderImgs("membersloader", "memberslabel");
+        },
+
+        failure: function(o) {
+            removeLoaderImgs("membersloader", "memberslabel");
         }
     };
 
@@ -119,6 +126,14 @@ UpdatableMembersCombo.prototype.refreshMembers = function (groupId) {
     // Add the loader gif image.
     createLoaderImg("membersloader", "memberslabel", this.wwwRoot);
 
+    // Clear the members combo box.
+    var selectEl = document.getElementById("members");
+    if (selectEl) {
+        while (selectEl.firstChild) {
+            selectEl.removeChild(selectEl.firstChild);
+        }
+    }
+
     var sUrl = this.wwwRoot+"/group/index.php?id="+this.courseId+"&group="+groupId+"&act_ajax_getmembersingroup";
     YAHOO.util.Connect.asyncRequest("GET", sUrl, this.connectCallback, null);
 };
-- 
2.39.5