--- loncom/interface/Attic/londropadd.pm	2004/06/29 04:30:00	1.115
+++ loncom/interface/Attic/londropadd.pm	2004/10/16 00:46:58	1.119
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Handler to drop and add students in courses 
 #
-# $Id: londropadd.pm,v 1.115 2004/06/29 04:30:00 raeburn Exp $
+# $Id: londropadd.pm,v 1.119 2004/10/16 00:46:58 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -309,7 +309,7 @@ END
         return;
     }
     // alert('current.radiovalue = '+current.radiovalue);
-    if (current.radiovalue == null || current.radiovalue == 'nochange') {
+    if (current.radiovalue == null || current.radiovalue == '' || current.radiovalue == 'nochange') {
         // They did not check any of the login radiobuttons.
         alert('$alert{'authen'}');
         return;
@@ -924,6 +924,8 @@ sub print_enroll_single_student_form {
     #
     my $username = $ENV{'form.cuname'};
     my $domain   = $ENV{'form.cudomain'};
+    $username=~s/[\W|_]//gs;
+    $domain=~s/[\W|_]//gs;
     my $home = &Apache::lonnet::homeserver($username,$domain);
     # $new_user flags whether we are creating a new user or using an old one
     my $new_user = 1;
@@ -1375,7 +1377,7 @@ END
                     $cellentry = '<b>'.&mt('auto').'</b>&nbsp;<input type="checkbox" name="chgauto" value="'.$username.':'.$domain.'" />&nbsp;Change';
                     $autocount ++;
                 } else {
-                    $cellentry = '<table border="0" cellspacing="0"><tr><td rowspan="2"><b>'.&mt('manual').'</b></td><td><input type="checkbox" name="chgmanual" value="'.$username.':'.$domain.'" />&nbsp;Change</td></tr><tr><td>';
+                    $cellentry = '<table border="0" cellspacing="0"><tr><td rowspan="2"><b>'.&mt('manual').'</b></td><td><nobr><input type="checkbox" name="chgmanual" value="'.$username.':'.$domain.'" />&nbsp;Change</nobr></td></tr><tr><td><nobr>';
                     $manualcount ++;
                     if ($lockedtype) {
                         $cellentry .= '<input type="checkbox" name="unlockchg" value="'.$username.':'.$domain.'" />&nbsp;'.&mt('Unlock');
@@ -1384,7 +1386,7 @@ END
                         $cellentry .= '<input type="checkbox" name="lockchg" value="'.$username.':'.$domain.'" />&nbsp;'.&mt('Lock');
                         $lockcount ++;
                     }
-                    $cellentry .= '</td></tr></table>';
+                    $cellentry .= '</nobr></td></tr></table>';
                 }
                 $r->print("<td>$cellentry<td>\n    ");
             } else {
@@ -2289,7 +2291,7 @@ sub handler {
             ({href=>'/adm/dropadd?action=upload&state=',
               text=>"Upload Classlist"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs
-                  (undef,'Upload Classlist'));
+                  (undef,'Upload Classlist','Course_Create_Class_List'));
         if (! exists($ENV{'form.state'})) {
             &print_first_courselist_upload_form($r);            
         } elsif ($ENV{'form.state'} eq 'got_file') {
@@ -2308,7 +2310,7 @@ sub handler {
             ({href=>'/adm/dropadd?action=drop',
               text=>"Drop Students"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs
-                  (undef,'Drop Students'));
+                  (undef,'Drop Students','Course_Drop_Student'));
         if (! exists($ENV{'form.state'})) {
             &print_drop_menu($r);
         } elsif ($ENV{'form.state'} eq 'done') {
@@ -2321,7 +2323,7 @@ sub handler {
             ({href=>'/adm/dropadd?action=enrollstudent',
               text=>"Enroll Student"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs
-                  (undef,'Enroll Student'));
+                  (undef,'Enroll Student','Course_Add_Student'));
         if (! exists($ENV{'form.state'})) {
             &get_student_username_domain_form($r);
         } elsif ($ENV{'form.state'} eq 'gotusername') {
@@ -2336,7 +2338,7 @@ sub handler {
             ({href=>'/adm/dropadd?action=classlist',
               text=>"View Classlist"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs
-                  (undef,'View Classlist'));
+                  (undef,'View Classlist','Course_View_Class_List'));
         if (! exists($ENV{'form.state'})) {
             &print_html_classlist($r,undef);
         } elsif ($ENV{'form.state'} eq 'csv') {
@@ -2351,7 +2353,7 @@ sub handler {
             ({href=>'/adm/dropadd?action=modifystudent',
               text=>"Modify Student Data"});
         $r->print(&Apache::lonhtmlcommon::breadcrumbs
-                  (undef,'Modify Student Data'));
+                  (undef,'Modify Student Data','Course_Modify_Student_Data'));
         if (! exists($ENV{'form.state'})) {
             &print_html_classlist($r);
         } elsif ($ENV{'form.state'} eq 'selected') {