--- loncom/interface/loncoursegroups.pm	2006/03/19 22:31:41	1.9
+++ loncom/interface/loncoursegroups.pm	2006/04/10 19:59:53	1.13
@@ -464,7 +464,8 @@ sub group_administration {
         }
     }
 
-    if ($action eq 'create') {
+    if (($action eq 'create') || 
+        ($action eq 'modify' && $state eq 'pick_members')) {
         if (defined($env{'form.types'})) {
             @types=&Apache::loncommon::get_env_multiple('form.types');
         }
@@ -952,11 +953,13 @@ sub display_control {
 sub header {
     my ($bodytitle,$jscript,$action,$state,$page,$function,$loaditems) = @_;
     my $start_page=
-	&Apache::loncommon::start_page($bodytitle,$jscript,
-				       {'function' => $function,
+	&Apache::loncommon::start_page($bodytitle,
+				       '<script type="text/javascript">'.
+				       $jscript.'</script>',
+				       {'function'    => $function,
 					'add_entries' => $loaditems,});
     my $output = <<"END";
-$bodytag
+$start_page
 <form method="POST" name="$state">
 
 END
@@ -973,23 +976,23 @@ END
 
 sub onload_action {
     my ($action,$state) = @_;
-    my $loaditems;
+    my %loaditems;
     if ((defined($env{'form.origin'})) && ($action eq 'create') &&
                 ($state eq 'pick_name' || $state eq 'pick_members' || 
                  $state eq 'pick_privs')) {
         unless ($env{'form.origin'} eq '') {
-            $loaditems = 
-             'onload="javascript:setFormElements(document.'.$state.')"';
+	    $loaditems{'onload'} = 
+		'javascript:setFormElements(document.'.$state.')';
         }
     }
     if (($action eq 'modify') &&
                 ($state eq 'change_settings' || $state eq 'change_members' ||
                  $state eq 'change_privs' || $state eq 'add_members' ||
                  $state eq 'pick_members')) {
-            $loaditems =
-             'onload="javascript:setFormElements(document.'.$state.')"';
+	$loaditems{'onload'} = 
+	    'javascript:setFormElements(document.'.$state.')';
     }
-    return $loaditems;
+    return \%loaditems;
 }
 
 sub footer {