[LON-CAPA-cvs] cvs: loncom /interface loncreateuser.pm

raeburn raeburn@source.lon-capa.org
Wed, 18 Nov 2009 20:01:36 -0000


raeburn		Wed Nov 18 20:01:36 2009 EDT

  Modified files:              
    /loncom/interface	loncreateuser.pm 
  Log:
  - $role wasn't being interpolated
  - remove extraneous onchange attribute. 
  
  
Index: loncom/interface/loncreateuser.pm
diff -u loncom/interface/loncreateuser.pm:1.325 loncom/interface/loncreateuser.pm:1.326
--- loncom/interface/loncreateuser.pm:1.325	Thu Nov 12 16:09:10 2009
+++ loncom/interface/loncreateuser.pm	Wed Nov 18 20:01:36 2009
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Create a user
 #
-# $Id: loncreateuser.pm,v 1.325 2009/11/12 16:09:10 raeburn Exp $
+# $Id: loncreateuser.pm,v 1.326 2009/11/18 20:01:36 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -552,10 +552,10 @@
                           &mt('View/Modify existing role:').'</b></label></td>'.
                           '<td align="center"><br />'.
                           '<select name="rolename" onchange="setCustomAction('."'edit'".');">'.
-                          '<option value="" selected="selected" onchange="">'.
+                          '<option value="" selected="selected">'.
                           &mt('Select'));
                 foreach my $role (sort(keys(%existingroles))) {
-                    $r->print('<option value="$role">'.$role.'</option>');
+                    $r->print('<option value="'.$role.'">'.$role.'</option>');
                 }
                 $r->print('</select>'.
                           '</td>'.