From da942d13d28ed1a63725e1e1964db7b62c044b20 Mon Sep 17 00:00:00 2001 From: Andrew Nicols Date: Wed, 29 Jan 2014 00:12:55 +0800 Subject: [PATCH] MDL-43893 Javascript: Remove YUI2 uses in group members management JS --- group/clientlib.js | 23 ++++------------------- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/group/clientlib.js b/group/clientlib.js index fef76b1edf8..ca2249594ab 100644 --- a/group/clientlib.js +++ b/group/clientlib.js @@ -64,22 +64,6 @@ function UpdatableGroupsCombo(wwwRoot, courseId) { } }; - - // Add onchange event to groups list box. - // Okay, this is not working in IE. The onchange is never fired... - // I'm hard coding the onchange in ../index.php. Not ideal, but it works - // then. vyshane AT moodle DOT com. - /* - 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. - YUI().use('yui2-dom', function (Y) { - Y.YUI2.util.Dom.setStyle("updategroups", "display", "none"); - }); } @@ -131,9 +115,10 @@ function UpdatableMembersCombo(wwwRoot, courseId) { }; // Hide the updatemembers input since AJAX will take care of this. - YUI().use('yui2-dom', function (Y) { - Y.YUI2.util.Dom.setStyle("updatemembers", "display", "none"); - }); + var updatemembers = Y.one('#updatemembers'); + if (updatemembers) { + updatemembers.hide(); + } } /**