From: skodak <skodak>
Date: Mon, 20 Nov 2006 08:29:50 +0000 (+0000)
Subject: removed JS from admin/course/assign.php MDL-7492
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=89d30d02306a95fe6e5ba81931c04467ab32dc2f;p=moodle.git

removed JS from admin/course/assign.php MDL-7492
---

diff --git a/admin/roles/assign.php b/admin/roles/assign.php
index 8abfde2732..48ee603c52 100755
--- a/admin/roles/assign.php
+++ b/admin/roles/assign.php
@@ -214,16 +214,13 @@
                                             ORDER BY lastname ASC, firstname ASC');
 
         /// In the .html file below we loop through these results and exclude any in $contextusers
-    
-        echo '<form name="rolesform" action="assign.php" method="post">';
+
         echo '<div align="center">'.$strcurrentcontext.': '.print_context_name($context).'<br/>';
-        if ($userid) {
-            echo '<input type="hidden" name="userid" value="'.$userid.'" />';
-        }
-        echo '<input type="hidden" name="courseid" value="'.$courseid.'" />';
-        echo '<input type="hidden" name="contextid" value="'.$context->id.'" />'.$strroletoassign.': ';
-        choose_from_menu ($assignableroles, 'roleid', $roleid, get_string('listallroles', 'role').'...', $script='rolesform.submit()');
-        echo '</div></form>';
+        echo $strroletoassign.': ';
+        $assignableroles = array('0'=>get_string('listallroles', 'role').'...') + $assignableroles; 
+        popup_form("$CFG->wwwroot/$CFG->admin/roles/assign.php?userid=$userid&amp;courseid=$courseid&amp;contextid=$contextid&amp;roleid=",
+            $assignableroles, 'switchrole', $roleid, '');
+        echo '</div>';
 
         print_simple_box_start('center');
         include('assign.html');