--- loncom/interface/lonuserutils.pm	2010/03/21 14:58:40	1.118
+++ loncom/interface/lonuserutils.pm	2010/05/22 01:11:53	1.119
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Utility functions for managing LON-CAPA user accounts
 #
-# $Id: lonuserutils.pm,v 1.118 2010/03/21 14:58:40 raeburn Exp $
+# $Id: lonuserutils.pm,v 1.119 2010/05/22 01:11:53 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -4395,7 +4395,6 @@ sub update_user_list {
     my ($r,$context,$setting,$choice,$crstype) = @_;
     my $now = time;
     my $count=0;
-    my $crstype;
     if ($context eq 'course') {
         $crstype = &Apache::loncommon::course_type();
     }
@@ -4936,7 +4935,7 @@ sub setsections_javascript {
                     accr => 'A course coordinator role will be added with access to all sections.',
                     acor => 'A coordinator role will be added with access to all sections',
                     inea => 'In each course, each user may only have one student role at a time.',
-                    inec => 'In each community, each user may only have on member role at a time.',
+                    inec => 'In each community, each user may only have one member role at a time.',
                     youh => 'You had selected ',
                     secs => 'sections.',
                     plmo => 'Please modify your selections so they include no more than one section.',
@@ -4974,7 +4973,7 @@ function setSections(formname,crstype) {
                     for (var j=0; j<formname.elements[num].length; j++) {
                         if (formname.elements[num].options[j].selected == true ) {
                             var addsec = formname.elements[num].options[j].value;
-                            if (addsec != "") {
+                            if ((addsec != "") && (addsec != null)) {
                                 fromexisting.push(addsec);
                                 if (numsec == 0) {
                                     sections = addsec;