--- loncom/interface/lonrequestcourse.pm	2009/08/05 12:46:44	1.3
+++ loncom/interface/lonrequestcourse.pm	2009/08/17 04:39:15	1.15
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Request a course
 #
-# $Id: lonrequestcourse.pm,v 1.3 2009/08/05 12:46:44 raeburn Exp $
+# $Id: lonrequestcourse.pm,v 1.15 2009/08/17 04:39:15 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -44,6 +44,62 @@ described at http://www.lon-capa.org.
 
 =item handler()
 
+=item header()
+
+=item form_elements()
+
+=item onload_action()
+
+=item check_can_request() 
+
+=item course_types()
+
+=item print_main_menu()
+
+=item request_administration()
+
+=item print_request_form()
+
+=item print_enrollment_menu()
+
+=item inst_section_selector()
+
+=item date_setting_table()
+
+=item print_personnel_menu()
+
+=item print_request_status()
+
+=item print_request_logs()
+
+=item print_review()
+
+=item dates_from_form()
+
+=item courseinfo_form()
+
+=item clone_form()
+
+=item clone_text()
+
+=item coursecode_form()
+
+=item get_course_dom()
+
+=item display_navbuttons()
+
+=item print_request_outcome()
+
+=item get_processtype()
+
+=item check_autolimit()
+
+=item build_batchcreatehash()
+
+=item retrieve_settings()
+
+=item get_request_settings()
+
 =back
 
 =cut
@@ -55,7 +111,8 @@ use Apache::Constants qw(:common :http);
 use Apache::lonnet;
 use Apache::loncommon;
 use Apache::lonlocal;
-use LONCAPA;
+use Apache::loncoursequeueadmin;
+use LONCAPA qw(:DEFAULT :match);
 
 sub handler {
     my ($r) = @_;
@@ -113,13 +170,14 @@ sub handler {
     }
 
     my %trail = (
-                 crstype  => 'Pick Action',
-                 codepick => 'Pick Category',
-                 courseinfo => 'Information',
-                 enrollment => 'Enrollment',
-                 personnel => 'Personnel',
-                 review => 'Review',
-                 process => 'Result',
+                 crstype       => 'Course Request Action',
+                 codepick      => 'Category',
+                 courseinfo    => 'Description',
+                 enrollment    => 'Enrollment',
+                 personnel     => 'Personnel',
+                 review        => 'Review',
+                 process       => 'Result',
+                 pick_request  => 'Display Summary',
                 );
 
     my $page = 0;
@@ -178,7 +236,12 @@ sub handler {
                       '</div>'.&Apache::loncommon::end_page());
         }
     } elsif ($action eq 'view') {
-        &print_request_status($jscript,$loaditems,$crumb);
+        if ($state eq 'crstype') {
+           &print_main_menu($r,\%can_request,\%states,$dom,$jscript,'',$crumb);
+        } else {
+           &request_administration($r,$action,$state,$page,\%states,$dom,$jscript,
+                                   $loaditems,$crumb);
+        }
     } elsif ($action eq 'log') {
         &print_request_logs($jscript,$loaditems,$crumb);
     } else {
@@ -190,8 +253,9 @@ sub handler {
 sub header {
     my ($bodytitle,$jscript,$loaditems,$jsextra) = @_;
     if ($jscript) {
-        $jscript = '<script type="text/javascript">'.
-                   $jscript.'</script>'."\n";
+        $jscript = '<script type="text/javascript">'."\n".
+                   '// <![CDATA['."\n".
+                   $jscript."\n".'// ]]>'."\n".'</script>'."\n";
     }
     if ($loaditems) {
         $loaditems = {'add_entries' => $loaditems,};
@@ -212,30 +276,33 @@ sub form_elements {
             },
             courseinfo => {
                 cdescr           => 'text',
-                clonecourse      => 'text',
-                clonedomain      => 'selectbox',
+                clonecrs         => 'text',
+                clonedom         => 'selectbox',
                 datemode         => 'radio',
                 dateshift        => 'text',
             },
             enrollment  => {
-                startaccess_month  => 'selectbox',
-                startaccess_hour   => 'selectbox',
-                endaccess_month    => 'selectbox',
-                endaccess_hour     => 'selectbox',
-                startaccess_day    => 'text',
-                startaccess_year   => 'text',
-                startaccess_minute => 'text',
-                startaccess_second => 'text',
-                endaccess_day      => 'text',
-                endaccess_year     => 'text',
-                endaccess_minute   => 'text',
-                endaccess_second   => 'text',
+                accessstart_month  => 'selectbox',
+                accessstart_hour   => 'selectbox',
+                accessend_month    => 'selectbox',
+                accessend_hour     => 'selectbox',
+                accessstart_day    => 'text',
+                accessstart_year   => 'text',
+                accessstart_minute => 'text',
+                accessstart_second => 'text',
+                accessend_day      => 'text',
+                accessend_year     => 'text',
+                accessend_minute   => 'text',
+                accessend_second   => 'text',
                 no_end_date      => 'checkbox',
             },
             personnel => {
                 persontotal => 'hidden',
                 addperson   => 'checkbox', 
             },
+            review => {
+                cnum => 'hidden',
+            },
          },
          view => {
             crstype => {
@@ -244,6 +311,13 @@ sub form_elements {
             },
          },
     );
+    my %servers = &Apache::lonnet::get_servers($dom,'library');
+    my $numlib = keys(%servers);
+    if ($numlib > 1) {
+        $elements{'new'}{'courseinfo'}{'chome'} = 'selectbox';
+    } else {
+        $elements{'new'}{'courseinfo'}{'chome'} = 'hidden';
+    }
     my (@codetitles,%cat_titles,%cat_order,@code_order,$lastitem);
     &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
                                              \%cat_order,\@code_order);
@@ -260,18 +334,18 @@ sub form_elements {
     if (&Apache::lonnet::auto_run('',$dom)) {
         my %extras = (
                        sectotal           => 'hidden',
-                       startenroll_month  => 'selectbox',
-                       startenroll_hour   => 'selectbox',
-                       endenroll_month    => 'selectbox',
-                       endenroll_hour     => 'selectbox',
-                       startenroll_day    => 'text',
-                       startenroll_year   => 'text',
-                       startenroll_minute => 'text',
-                       startenroll_second => 'text',
-                       endenroll_day      => 'text',
-                       endenroll_year     => 'text',
-                       endenroll_minute   => 'text',
-                       endenroll_second   => 'text',
+                       enrollstart_month  => 'selectbox',
+                       enrollstart_hour   => 'selectbox',
+                       enrollend_month    => 'selectbox',
+                       enrollend_hour     => 'selectbox',
+                       enrollstart_day    => 'text',
+                       enrollstart_year   => 'text',
+                       enrollstart_minute => 'text',
+                       enrollstart_second => 'text',
+                       enrollend_day      => 'text',
+                       enrollend_year     => 'text',
+                       enrollend_minute   => 'text',
+                       enrollend_second   => 'text',
                        crosslisttotal     => 'hidden',
                        addcrosslist       => 'checkbox',
                        autoadds           => 'radio',
@@ -312,16 +386,15 @@ sub form_elements {
         $persontotal = 1;
     }
     for (my $i=0; $i<$persontotal; $i++) {
-        $people{'person_'.$i.'_uname'}    = 'text',
-        $people{'person_'.$i.'_dom'}      = 'selectbox',
-        $people{'person_'.$i.'_hidedom'}  = 'hidden',
-        $people{'person_'.$i.'_first'}    = 'text',
-        $people{'person_'.$i.'_last'}     = 'text',
-        $people{'person_'.$i.'_email'}    = 'text',
-        $people{'person_'.$i.'_role'}     = 'selectbox',
-        $people{'person_'.$i.'_sec'}      = 'selectbox',
-        $people{'person_'.$i.'_newsec'}   = 'text',
-        $people{'person_'.$i.'_sections'} = 'hidden',
+        $people{'person_'.$i.'_uname'}     = 'text',
+        $people{'person_'.$i.'_dom'}       = 'selectbox',
+        $people{'person_'.$i.'_hidedom'}   = 'hidden',
+        $people{'person_'.$i.'_firstname'} = 'text',
+        $people{'person_'.$i.'_lastname'}  = 'text',
+        $people{'person_'.$i.'_emailaddr'} = 'text',
+        $people{'person_'.$i.'_role'}      = 'selectbox',
+        $people{'person_'.$i.'_sec'}       = 'selectbox',
+        $people{'person_'.$i.'_newsec'}    = 'text',
     }
     my %personnelhash = (%{$elements{'new'}{'personnel'}},%people);
     %{$elements{'new'}{'personnel'}} = %personnelhash;
@@ -340,8 +413,9 @@ sub onload_action {
 sub check_can_request {
     my ($dom,$can_request) = @_;
     my $canreq = 0;
-    if (ref($can_request) eq 'HASH') {
-        foreach my $type ('official','unofficial','community') {
+    my ($types,$typename) = &course_types();
+    if ((ref($can_request) eq 'HASH') && (ref($types) eq 'ARRAY')) {
+        foreach my $type (@{$types}) {
             if (&Apache::lonnet::usertools_access($env{'user.name'},
                                                   $env{'user.domain'},
                                                   $type,undef,'requestcourses')) {
@@ -366,8 +440,20 @@ sub check_can_request {
     return $canreq;
 }
 
+sub course_types {
+    my @types = ('official','unofficial','community');
+    my %typename = (
+                         official   => 'Official course',
+                         unofficial => 'Unofficial course',
+                         community  => 'Community',
+                    );
+    return (\@types,\%typename);
+}
+
+
 sub print_main_menu {
     my ($r,$can_request,$states,$dom,$jscript,$loaditems,$crumb) = @_;
+    my ($types,$typename) = &course_types();
     my $onchange;
     unless ($env{'form.interface'} eq 'textual') {
         $onchange = 1;
@@ -455,7 +541,7 @@ END
               '<div>'.
               '<form name="domforcourse" method="post" action="/adm/requestcourse">'.
               &Apache::lonhtmlcommon::start_pick_box().
-              &Apache::lonhtmlcommon::row_title('Domain').
+              &Apache::lonhtmlcommon::row_title('Course Domain').
               &Apache::loncommon::select_dom_form($dom,'showdom','',1,$onchange));
     if (!$onchange) {
         $r->print('&nbsp;<input type="submit" name="godom" value="'.
@@ -466,7 +552,8 @@ END
 
     my $formname = 'requestcrs';
     my $nexttext = &mt('Next');
-    $r->print('<div><form name="'.$formname.'" method="post" action="/adm/requestcourse">'.
+    $r->print(
+              '<div><form name="'.$formname.'" method="post" action="/adm/requestcourse">'.
               &Apache::lonhtmlcommon::start_pick_box().
               &Apache::lonhtmlcommon::row_title('Action').'
 <input type="hidden" name="showdom" value="'.$dom.'" />
@@ -478,11 +565,18 @@ END
               &Apache::lonhtmlcommon::row_closure().
               &Apache::lonhtmlcommon::row_title('Course Type').'
 <select size="1" name="crstype">
- <option value="any">'.&mt('All types').'</option>
- <option value="official" selected="selected">'.&mt('Official course').'</option>
- <option value="unofficial">'.&mt('Unofficial course').'</option>
- <option value="community">'.&mt('Community').'</option>
-</select>
+ <option value="any">'.&mt('All types').'</option>');
+    if ((ref($types) eq 'ARRAY') && (ref($typename) eq 'HASH')) {
+        foreach my $type (@{$types}) {
+            my $selected = '';
+            if ($type eq 'official') {
+                $selected = ' selected="selected"';
+            }
+            $r->print('<option value="'.$type.'"'.$selected.'>'.$typename->{$type}.
+                      '</option>'."\n");
+        }
+    }
+    $r->print('</select>
 <input type="hidden" name="state" value="crstype" />'.
               &Apache::lonhtmlcommon::row_closure(1).
               &Apache::lonhtmlcommon::end_pick_box().'<br />
@@ -515,17 +609,14 @@ END
             $jsextra = "\n".&Apache::loncommon::coursebrowser_javascript($dom);
         }
         $r->print(&header('Request a course',$js.$jscript,$loaditems,$jsextra).$crumb);
-        if ($state eq 'review') {
-            &print_review($r,$state,$dom);
-        } elsif ($state eq 'process') {
-            &print_request_outcome($r,$state,$dom);
-        } else {
-            &print_request_form($r,$action,$state,$page,$states,$dom);
-        }
+        &print_request_form($r,$action,$state,$page,$states,$dom);
     } elsif ($action eq 'view') {
         $r->print(&header('Manage course requests',$js.$jscript,$loaditems).$crumb);
+        if ($state eq 'pick_request') {
+            $r->print(&print_request_status($dom));
+        }
     } elsif ($action eq 'log') {
-        $r->print(&coursereq_log('View request log',$jscript,$loaditems).$crumb);       
+        $r->print(&coursereq_log('View request log',$jscript,$loaditems).$crumb);
     }
     $r->print(&Apache::loncommon::end_page());
     return;
@@ -537,17 +628,23 @@ sub print_request_form {
     my ($next,$prev,$message,$output,$codepicker,$crstype);
     $prev = $states->{$action}[$page-1];
     $next = $states->{$action}[$page+1];
+    my %navtxt = &Apache::lonlocal::texthash (
+                                               prev => 'Back',
+                                               next => 'Next',
+                                             );
     $crstype = $env{'form.crstype'};
     $r->print('<form name="'.$formname.'" method="post" action="/adm/requestcourse">');
     my (@codetitles,%cat_titles,%cat_order,@code_order,$instcode,$code_chk);
     if ($crstype eq 'official') {
-        &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
-                                                 \%cat_order,\@code_order);
         if ($env{'form.instcode'} ne '') {
             $instcode = $env{'form.instcode'};
         }
     }
     if ($prev eq 'codepick') {
+        if ($crstype eq 'official') {
+            &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+                                                     \%cat_order,\@code_order);
+        }
         if (@code_order > 0) {
             my $message;
             if ($instcode eq '') {
@@ -581,6 +678,10 @@ sub print_request_form {
         }
     }
     if ($prev eq 'crstype') {
+        if ($crstype eq 'official') {
+            &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+                                                     \%cat_order,\@code_order);
+        }
         if (@code_order > 0) {
             $codepicker = &coursecode_form($dom,'instcode',\@codetitles,
                                            \%cat_titles,\%cat_order);
@@ -588,18 +689,35 @@ sub print_request_form {
                 $r->print('<div>'.&Apache::lonhtmlcommon::start_pick_box().$codepicker.
                           &Apache::lonhtmlcommon::end_pick_box().'</div>');
             } else {
-                $r->print(&courseinfo_form($dom,$formname,$crstype));
+                $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
             }
         } else {
-            $r->print(&courseinfo_form($dom,$formname,$crstype));
+            $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
         }
     } elsif ($prev eq 'codepick') {
-        $r->print(&courseinfo_form($dom,$formname,$crstype));
+        $r->print(&courseinfo_form($dom,$formname,$crstype,$next));
     } elsif ($state eq 'enrollment') {
+        if ($crstype eq 'official') {
+            &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+                                                     \%cat_order,\@code_order);
+        }
         $r->print(&print_enrollment_menu($formname,$instcode,$dom,\@codetitles,
                                          \%cat_titles,\%cat_order,\@code_order));
     } elsif ($state eq 'personnel') {
-        $r->print(&print_personnel_menu($dom,$formname));
+        $r->print(&print_personnel_menu($dom,$formname,$crstype));
+    } elsif ($state eq 'review') {
+        &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+                                                 \%cat_order,\@code_order);
+        $r->print(&print_review($formname,$dom,\@codetitles,\%cat_titles,\%cat_order,
+                                \@code_order));
+        $navtxt{'next'} = &mt('Submit course request');
+    }  elsif ($state eq 'process') {
+        if ($crstype eq 'official') {
+            &Apache::lonnet::auto_possible_instcodes($dom,\@codetitles,\%cat_titles,
+                                                     \%cat_order,\@code_order);
+        }
+        my $result = &print_request_outcome($dom,\@codetitles,\@code_order);
+        $r->print($result);
     }
     my @excluded = ('counter');
     my %elements = &form_elements($dom);
@@ -625,7 +743,7 @@ sub print_request_form {
         push(@excluded,'instcode');
     }
     $r->print(&Apache::lonhtmlcommon::echo_form_input(\@excluded).'</form>');
-    &display_navbuttons($r,$formname,$prev,'Previous',$next,'Next');
+    &display_navbuttons($r,$formname,$prev,$navtxt{'prev'},$next,$navtxt{'next'},$state);
     return;
 }
 
@@ -737,13 +855,13 @@ sub inst_section_selector {
 }
 
 sub date_setting_table {
-    my ($starttime,$endtime,$formname,$suffix,%datetitles) = @_;
+    my ($starttime,$endtime,$formname,$prefix,%datetitles) = @_;
     my ($perpetual,$table);
-    my $startform = &Apache::lonhtmlcommon::date_setter($formname,'start'.$suffix,
+    my $startform = &Apache::lonhtmlcommon::date_setter($formname,$prefix.'start',
                                                         $starttime,'','','',1,'','','',1);
-    my $endform = &Apache::lonhtmlcommon::date_setter($formname,'end'.$suffix,
+    my $endform = &Apache::lonhtmlcommon::date_setter($formname,$prefix.'end',
                                                       $endtime,'','','',1,'','','',1);
-    if ($suffix eq 'access') {
+    if ($prefix eq 'access') {
         $perpetual = ' <span class="LC_nobreak"><label>'.
                      '<input type="checkbox" name="no_end_date" />'.
                      &mt('No end date').'</label></span>';
@@ -758,7 +876,7 @@ sub date_setting_table {
 }
 
 sub print_personnel_menu {
-    my ($dom,$formname) = @_;
+    my ($dom,$formname,$crstype) = @_;
     my $output = '<div>'.&Apache::lonhtmlcommon::start_pick_box();
     my $persontotal = $env{'form.persontotal'};
     if (!defined($persontotal)) {
@@ -768,12 +886,16 @@ sub print_personnel_menu {
         $persontotal ++;
     }
     my $userlinktxt = &mt('Set User');
-    my @items = ('uname','dom','last','first','email','hidedom');
+    my @items = ('uname','dom','lastname','firstname','emailaddr','hidedom');
 
     my $roleoptions;
     my @roles = &Apache::lonuserutils::roles_by_context('course');
+    my $type = 'Course';
+    if ($crstype eq 'community') {
+        $type = 'Community';
+    }
     foreach my $role (@roles) {
-        my $plrole=&Apache::lonnet::plaintext($role);
+        my $plrole=&Apache::lonnet::plaintext($role,$type);
         $roleoptions .= '  <option value="'.$role.'">'.$plrole.'</option>'."\n";
     }
     my %customroles=&Apache::lonuserutils::my_custom_roles();
@@ -819,13 +941,13 @@ sub print_personnel_menu {
     for (my $i=0; $i<$persontotal; $i++) { 
         my @linkargs = map { 'person_'.$i.'_'.$_ } (@items);
         my $linkargstr = join("','",@linkargs);
-        my $userlink = &Apache::loncommon::selectuser_link($formname,@linkargs,$userlinktxt);
+        my $userlink = &Apache::loncommon::selectuser_link($formname,@linkargs,$dom,$userlinktxt);
         my $uname_form = '<input type="text" name="person_'.$i.'_uname" value=""'.
                          ' onFocus="this.blur();'.
-                         'openuserbrowser('."'$formname','$linkargstr'".');" />';
+                         'openuserbrowser('."'$formname','$linkargstr','$dom'".');" />';
         my $onchange = 'javascript:fix_domain('."'$formname','person_".$i."_dom',".
                        "'person_".$i."_hidedom'".');'.
-                       'openuserbrowser('."'$formname','$linkargstr'".');';
+                       'openuserbrowser('."'$formname','$linkargstr','$dom'".');';
         my $udom_form = &Apache::loncommon::select_dom_form($dom,'person_'.$i.'_dom','',
                                                             1,$onchange).
                         '<input type="hidden" name="person_'.$i.'_hidedom" value="'.$dom.'" />';
@@ -843,18 +965,16 @@ sub print_personnel_menu {
                                $existmult.'>'."\n".$existops.'</select>'.('&nbsp;' x3);
         }
         $sectionselector .= $newtitle.
-            '<input type="text" name="person_'.$i.'_newsec" size="15" value="" />'.
-            '<input type="hidden" name="person_'.$i.'_sections" value="" />'."\n";
-         
+            '<input type="text" name="person_'.$i.'_newsec" size="15" value="" />'."\n";
         $output .= 
             &Apache::lonhtmlcommon::row_title(&mt('Additional Personnel').'<br />'.
                                               '<span class="LC_nobreak">'.$userlink.
                                               '</span>').
             '<table><tr><td align="center" valign="top">'.&mt('Username').'<br />'.$uname_form.'</td>'."\n".
             '<td align="center" valign="top" colspan="2">'.&mt('Domain').'<br />'.$udom_form.'</td></tr><tr>'."\n".
-            '<td align="center" valign="top">'.&mt('First Name').'<br />'.$form_elems{'first'}.'</td>'."\n".
-            '<td align="center" valign="top">'.&mt('Last Name').'<br />'.$form_elems{'last'}.'</td>'."\n".
-            '<td align="center" valign="top">'.&mt('E-mail').'<br />'.$form_elems{email}.'</td></tr>'."\n".
+            '<td align="center" valign="top">'.&mt('First Name').'<br />'.$form_elems{'firstname'}.'</td>'."\n".
+            '<td align="center" valign="top">'.&mt('Last Name').'<br />'.$form_elems{'lastname'}.'</td>'."\n".
+            '<td align="center" valign="top">'.&mt('E-mail').'<br />'.$form_elems{'emailaddr'}.'</td></tr>'."\n".
             '<tr><td align="center" valign="top">'.&mt('Role').'<br />'.$roleselector.'</td>'."\n".
             '<td'.$seccolspan.' align="center" valign="top">'.&mt('Section(s)').'<br />'.$sectionselector.'</td>'."\n".
             '</tr></table>'.&Apache::lonhtmlcommon::row_closure();
@@ -869,26 +989,398 @@ sub print_personnel_menu {
 }
 
 sub print_request_status {
-    return;
+    my ($dom) = @_;
+    my %statusinfo = &Apache::lonnet::dump('courserequests',$env{'user.domain'},
+                                           $env{'user.name'},'^status:'.$dom);
+    my ($output,$formname,%queue_by_date,%typenames);
+    if ($env{'form.crstype'} eq 'any') {
+        %typenames = &Apache::lonlocal::texthash (
+                                                   official   => 'Official course',
+                                                   unofficial => 'Unofficial course',
+                                                   community  => 'Community',
+                                                 );
+    }
+    foreach my $key (keys(%statusinfo)) {
+        if (($statusinfo{$key} eq 'approval') || ($statusinfo{$key} eq 'pending')) { 
+            (undef,my($cdom,$cnum)) = split(':',$key);
+            next if ($cdom ne $dom);   
+            my $requestkey = $cdom.'_'.$cnum;
+            if ($requestkey =~ /^($match_domain)_($match_courseid)$/) {
+                my %history = &Apache::lonnet::restore($requestkey,'courserequests',
+                                                       $env{'user.domain'},$env{'user.name'});
+                my $entry;
+                my $timestamp = $history{'reqtime'};
+                my $crstype = $history{'crstype'};
+                my $disposition = $history{'disposition'};
+                next if ((exists($history{'status'})) && ($history{'status'} eq 'created'));
+                next unless (($env{'form.crstype'} eq 'any') || 
+                            ($env{'form.crstype'} eq $crstype));
+                next unless (($disposition eq 'approval') || 
+                             ($disposition eq 'pending'));
+                if (ref($history{'details'}) eq 'HASH') {
+                    $entry = $requestkey.':'.$crstype.':'.
+                             &escape($history{'details'}{'cdescr'});
+                    if ($crstype eq 'official') {
+                        $entry .= ':'.&escape($history{'details'}{'instcode'}); 
+                    }
+                }
+                if ($entry ne '') {
+                    if (exists($queue_by_date{$timestamp})) {
+                        if (ref($queue_by_date{$timestamp}) eq 'ARRAY') {
+                            push(@{$queue_by_date{$timestamp}},$entry);
+                        }
+                    } else {
+                        @{$queue_by_date{$timestamp}} = ($entry);
+                    }
+                }
+            }
+        }
+    }
+    $formname = 'requestcrs';
+    my @sortedtimes = sort {$a <=> $b} (keys(%queue_by_date));
+    $output = '<h3>'.&mt('Pending course requests').'</h3><div>'."\n".
+              '<form method="post" name="'.$formname.'" action="/adm/requestcourse" />'."\n".
+              '<input type="hidden" name="action" value="'.$env{'form.action'}.'" />'."\n".
+              '<input type="hidden" name="state" value="'.$env{'form.state'}.'" />'."\n".
+              '<input type="hidden" name="crstype" value="'.$env{'form.crstype'}.'" />'."\n";
+    if (@sortedtimes > 0) {
+        $output .= &Apache::loncommon::start_data_table().
+                   &Apache::loncommon::start_data_table_header_row().
+                   '<th>'.&mt('Action').'</th>'.
+                   '<th>'.&mt('Course Description').'</th>'.
+                   '<th>'.&mt('Domain').'</th>';
+        if ($env{'form.crstype'} eq 'any') {
+            $output .= '<th>'.&mt('Type').'</th>';
+        }
+        if (($env{'form.crstype'} eq 'any') || ($env{'form.crstype'} eq 'official')) {
+            $output .= '<th>'.&mt('Institutional Code').'</th>';
+        }
+        $output .= '<th>'.&mt('Date requested').'</th>'.
+                   &Apache::loncommon::end_data_table_header_row();
+        my $count = 0;
+        foreach my $item (@sortedtimes) {
+            my $showtime = &Apache::lonlocal::locallocaltime($item);
+            if (ref($queue_by_date{$item}) eq 'ARRAY') {
+                foreach my $request (sort(@{$queue_by_date{$item}})) {
+                    my ($key,$type,$desc,$instcode) = split(':',$request);
+                    my ($cdom,$cnum) = split('_',$key);
+                    $output .= &Apache::loncommon::start_data_table_row().
+     '<td><input type="button" value="'.&mt('Select').'" onclick="javascript:viewrequest('."'$cdom','$cnum'".')" /></td>'.
+     '<td>'.&unescape($desc).'</td>'.
+     '<td>'.$cdom.'</td>';
+                    if ($env{'form.crstype'} eq 'any') {
+                        my $typename = $typenames{$type};
+                        if ($typename eq '') {
+                            $typename = &mt('Unknown type');
+                        }
+                        $output .= '<td>'.$typename.'</td>';
+                    }
+                    if (($env{'form.crstype'} eq 'any') || 
+                        ($env{'form.crstype'} eq 'official')) {
+                        my $showinstcode;
+                        if ($type eq 'official') {
+                            $showinstcode = &unescape($instcode);
+                        } else {
+                            $showinstcode = &mt('Not applicable');
+                        }
+                        $output .= '<td>'.$showinstcode.'</td>';
+                    }
+                    $output .= '<td>'.$showtime.'</td>'.
+                               &Apache::loncommon::end_data_table_row();
+                }
+            }
+        }
+        $output .= &Apache::loncommon::end_data_table();
+    } else {
+        $output .= '<div>'.&mt('You have no matching course requests awaiting approval by a Domain Coordinator or held in a queue pending administrative action at your institution.').'</div>';
+    }
+    $output .= '
+<br /><input type="button" name="prev" value="'.&mt('Back').'" onclick="javascript:backPage(document.'.$formname.",'crstype'".')" />
+</form></div>';
+    return $output;
 }
 
 sub print_request_logs {
+    my ($jscript,$loaditems,$crumb) = @_;
     return;
 }
 
 sub print_review {
-    my ($r,$state,$dom) = @_;
-    return;
+    my ($formname,$dom,$codetitles,$cat_titles,$cat_order,$code_order) = @_;
+    my ($types,$typename) = &course_types();
+    my ($owner,$ownername,$owneremail);
+    $owner = $env{'user.name'}.':'.$env{'user.domain'};
+    $ownername = &Apache::loncommon::plainname($env{'user.name'},
+                                               $env{'user.domain'},'first');
+    my %emails = &Apache::loncommon::getemails();
+    foreach my $email ('permanentemail','critnotification','notification') {
+        $owneremail = $emails{$email};
+        last if ($owneremail ne '');
+    }
+    my ($inst_headers,$inst_values,$crstypename,$enroll_headers,$enroll_values,
+        $section_headers,$section_values,$personnel_headers,$personnel_values);
+
+    $crstypename = $env{'form.crstype'};
+    if (ref($typename) eq 'HASH') {
+        unless ($typename->{$env{'form.crstype'}} eq '') {
+            $crstypename = $typename->{$env{'form.crstype'}};
+        }
+    }
+
+    $inst_headers = '<th>'.&mt('Description').'</th><th>'.&mt('Type').'</th>';
+    $inst_values = '<td>'.$env{'form.cdescr'}.'</td><td>'.$crstypename.'</td>';
+
+    if ($env{'form.crstype'} eq 'official') {
+        if ((ref($codetitles) eq 'ARRAY') && (ref($cat_titles) eq 'HASH')) {
+            foreach my $title (@{$codetitles}) {
+                if ($env{'form.instcode_'.$title} ne '') {
+                    $inst_headers .= '<th>'.$title.'</th>';
+                    my $longitem = $env{'form.instcode_'.$title};
+                    if (ref($cat_titles->{$title}) eq 'HASH') {
+                        if ($cat_titles->{$title}{$env{'form.instcode_'.$title}} ne '') {
+                            $longitem = $cat_titles->{$title}{$env{'form.instcode_'.$title}};
+                        }
+                    }
+                    $inst_values .= '<td>'.$longitem.'</td>';
+                }
+            }
+        }
+        if (&Apache::lonnet::auto_run('',$dom)) {
+            $enroll_headers = '<th>'.&mt('Automatic Adds').'</th>'.
+                              '<th>'.&mt('Automatic Drops').'</th>'.
+                              '<th>'.&mt('Enrollment Starts').'</th>'.
+                              '<th>'.&mt('Enrollment Ends').'</th>';
+            $section_headers = '<th>'.&mt('Sections').'</th>'.
+                               '<th>'.&mt('Crosslistings').'</th>';
+
+            my ($enrollstart,$enrollend) = &dates_from_form('enrollstart','enrollend');
+            my @autoroster = (&mt('No'),&mt('Yes'));
+            $enroll_values = '<td>'.$autoroster[$env{'form.autoadds'}].'</td>'.
+                             '<td>'.$autoroster[$env{'form.autodrops'}].'</td>'.
+                             '<td>'.&Apache::lonlocal::locallocaltime($enrollstart).'</td>'.
+                             '<td>'.&Apache::lonlocal::locallocaltime($enrollend).'</td>';
+            $section_values = '<td><table class="LC_innerpickbox"><tr><th>'.
+                              &mt('Institutional section').'</th>'.
+                              '<th>'.&mt('LON-CAPA section').'</th></tr>';
+            my $secinfo;
+            if ($env{'form.sectotal'} > 0) {
+                for (my $i=0; $i<$env{'form.sectotal'}; $i++) {
+                    if ($env{'form.sec_'.$i}) {
+                        $secinfo .= '<tr><td>'.$env{'form.secnum_'.$i}.'</td><td>';
+                        if ($env{'form.loncapasec_'.$i} ne '') {
+                            $secinfo .= $env{'form.loncapasec_'.$i};
+                        } else {
+                            $secinfo .= &mt('None');
+                        }
+                        $secinfo .= '</td></tr>';
+                    }
+                }
+            }
+            if ($secinfo eq '') {
+                $secinfo = '<tr><td colspan="2">'.&mt('None').'</td></tr>';
+            }
+            $section_values .= $secinfo.'</table></td><td>'.
+                               '<table class="LC_innerpickbox"><tr><th>'.
+                               &mt('Institutional course/section').'</th>'.
+                               '<th>'.&mt('LON-CAPA section').'</th></tr>';
+            my $xlistinfo;
+            if ($env{'form.crosslisttotal'}) {
+                for (my $i=0; $i<$env{'form.crosslisttotal'}; $i++) {
+                    if ($env{'form.crosslist_'.$i}) {
+                        $xlistinfo .= '<tr><td>';
+                        if (ref($code_order) eq 'ARRAY') {
+                            if (@{$code_order} > 0) {
+                                foreach my $item (@{$code_order}) {
+                                    $xlistinfo .= $env{'form.crosslist_'.$i.'_'.$item};
+                                }
+                            }
+                        }
+                        $xlistinfo .= $env{'form.crosslist_'.$i.'_instsec'}.'</td><td>';
+                        if ($env{'form.crosslist_'.$i.'_lcsec'}) {
+                            $xlistinfo .= $env{'form.crosslist_'.$i.'_lcsec'};
+                        } else {
+                            $xlistinfo .= &mt('None');
+                        }
+                        $xlistinfo .= '</td></tr>';
+                    }
+                }
+            }
+            if ($xlistinfo eq '') {
+                $xlistinfo = '<tr><td colspan="2">'.&mt('None').'</td></tr>';
+            }
+            $section_values .= $xlistinfo.'</table></td>';
+        }
+    }
+
+    my %ctxt = &clone_text();
+    $inst_headers .= '<th>'.&mt('Clone From').'</th>';
+    if (($env{'form.clonecrs'} =~ /^$match_name$/) && 
+        ($env{'form.clonedom'} =~ /^$match_domain$/)) {
+        my $canclone = &Apache::loncoursequeueadmin::can_clone_course($env{'user.name'},
+                           $env{'user.domain'},$env{'form.clonecrs'}, $env{'form.clonedom'});
+        if ($canclone) {
+            my %courseenv = &Apache::lonnet::userenvironment($env{'form.clonedom'},
+                              $env{'form.clonecrs'},('description','internal.coursecode'));
+            if (keys(%courseenv) > 0) {
+                $inst_headers .= '<th>'.$ctxt{'dsh'}.'</th>';
+                $inst_values .= '<td>'.$courseenv{'description'}.'&nbsp;';
+                my $cloneinst = $courseenv{'internal.coursecode'};
+                if ($cloneinst ne '') {
+                    $inst_values .= &mt('([_1] in [_2])',$cloneinst,$env{'form.clonedom'});
+                } else {
+                    $inst_values .= &mt('(from [_1])',$env{'form.clonedom'});
+                }
+                $inst_values .= '</td><td>';
+                if ($env{'form.datemode'} eq 'preserve') {
+                    $inst_values .= $ctxt{'pcd'}; 
+                } elsif ($env{'form.datemode'} eq 'shift') {
+                    $inst_values .= &mt('Shift dates by [_1] days',$env{'form.dateshift'});
+                } else {
+                    $inst_values .= $ctxt{'ncd'};
+                }
+                $inst_values .= '</td>';
+             } else {
+                 $inst_values .= '<td>'.&mt('Unknown').'</td>';
+             }
+         } else {
+             $inst_values .= '<td>'.&mt('Not permitted'),'</td>';
+         }
+    } else {
+        $inst_values .= '<td>'.&mt('None').'</td>';
+    }
+    $enroll_headers .= '<th>'.&mt('Access Starts').'</th>'.
+                       '<th>'.&mt('Access Ends').'</th>';
+    my ($accessstart,$accessend) = &dates_from_form('accessstart','accessend');
+    $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($accessstart).'</td>';
+    if ($accessend == 0) {
+        $enroll_values .= '<td>'.&mt('No end date').'</td>';
+    } else {
+        $enroll_values .= '<td>'.&Apache::lonlocal::locallocaltime($accessend).'</td>';
+    }
+
+    my $container = 'Course';
+    if ($env{'form.crstype'} eq 'community') {
+        $container = 'Community';
+    }
+
+    $personnel_headers = '<th>'.&mt('Name').'</th><th>'.&mt('Username:Domain').
+                         '</th><th>'.&mt('Role').'</th><th>'.&mt('LON-CAPA Sections').
+                         '</th>';
+    $personnel_values .= '<tr><td>'.$ownername.'</td><td>'.$owner.'</td>'.
+                         '<td>'.&Apache::lonnet::plaintext('cc',$container).'</td>'.
+                         '<td>'.&mt('None').'</td></tr>';
+    for (my $i=0; $i<$env{'form.persontotal'}; $i++) {
+        if ($env{'form.person_'.$i.'_uname'} ne '') {
+            my @allsecs = &Apache::loncommon::get_env_multiple('form.person_'.$i.'_sec');
+            my $newsec = $env{'form.person_'.$i.'_newsec'};
+            $newsec =~ s/^\s+//;
+            $newsec =~s/\s+$//;
+            my @newsecs = split(/[\s,;]+/,$newsec);
+            foreach my $sec (@newsecs) {
+                next if ($sec =~ /\W/);
+                next if ($newsec eq 'none');
+                if ($sec ne '') {
+                    unless (grep(/^\Q$sec\E$/,@allsecs)) {
+                        push(@allsecs,$sec); 
+                    }
+                }
+            }
+            my $showsec = 'None';
+            if (@allsecs) {
+                $showsec = join(', ',@allsecs);
+            }
+            $personnel_values .= 
+                '<tr><td>'.$env{'form.person_'.$i.'_firstname'}.' '.
+                $env{'form.person_'.$i.'_lastname'}.'</td>'.
+                '<td>'.$env{'form.person_'.$i.'_uname'}.':'.
+                $env{'form.person_'.$i.'_dom'}.'</td>'.
+                '<td>'.&Apache::lonnet::plaintext($env{'form.person_'.$i.'_role'},
+                                                  $container).'</td>'.
+                '<td>'.$showsec.'</td></tr>';
+        }
+    }
+    my $output =  '<p>'.&mt('Review the details of the course request before submission.').'</p>'.  
+                  '<div>'.&Apache::lonhtmlcommon::start_pick_box().
+                  &Apache::lonhtmlcommon::row_title(&mt('Owner')).
+                  '<table class="LC_innerpickbox"><tr>'.
+                  '<th>'.&mt('Name').'</th>'.
+                  '<th>'.&mt('Username:Domain').'</th>'.
+                  '<th>'.&mt('E-mail address').'</th>'.
+                  '</tr><tr>'."\n".
+                  '<td>'.$ownername.'</td><td>'.$owner.'</td>'.
+                  '<td>'.$owneremail.'</td>'.
+                  '</tr></table>'."\n".
+                  &Apache::lonhtmlcommon::row_closure().
+                  &Apache::lonhtmlcommon::row_title(&mt('Description')).
+                  '<table class="LC_innerpickbox"><tr>'.$inst_headers.'</tr>'."\n".
+                  '<tr>'.$inst_values.'</tr></table>'."\n".
+                  &Apache::lonhtmlcommon::row_closure().
+                  &Apache::lonhtmlcommon::row_title(&mt('Enrollment')).
+                  '<table class="LC_innerpickbox"><tr>'.$enroll_headers.'</tr>'."\n".
+                  '<tr>'.$enroll_values.'</tr></table>'."\n".
+                  &Apache::lonhtmlcommon::row_closure();
+    if ($section_headers ne '') {
+        $output .= &Apache::lonhtmlcommon::row_title(&mt('Sections')).
+                   '<table class="LC_innerpickbox"><tr>'.$section_headers.'</tr>'."\n".
+                   '<tr>'.$section_values.'</tr></table>'."\n".
+                   &Apache::lonhtmlcommon::row_closure();
+    }
+    $output .= &Apache::lonhtmlcommon::row_title(&mt('Personnel')).
+               '<table class="LC_innerpickbox"><tr>'.$personnel_headers.'</tr>'."\n".
+               $personnel_values.'</table>'."\n".
+               &Apache::lonhtmlcommon::row_closure(1).
+               &Apache::lonhtmlcommon::end_pick_box();
+    my $cnum = &Apache::lonnet::generate_coursenum($dom);
+    $output .= '<input type="hidden" name="cnum" value="'.$cnum.'" />';
+    return $output;
+}
+
+sub dates_from_form {
+    my ($startname,$endname) = @_;
+    my $startdate = &Apache::lonhtmlcommon::get_date_from_form($startname);
+    my $enddate   = &Apache::lonhtmlcommon::get_date_from_form($endname);
+    if ($endname eq 'accessend') {
+        if (exists($env{'form.no_end_date'}) ) {
+            $enddate = 0;
+        }
+    }
+    return ($startdate,$enddate);
 }
 
 sub courseinfo_form {
-    my ($dom,$formname,$crstype) = @_;
-    my $output = '<div>'.&Apache::lonhtmlcommon::start_pick_box().
-                 &Apache::lonhtmlcommon::row_title('Course Description').
-                 '<input type="text" size="40" name="cdescr" />'.
-                 &Apache::lonhtmlcommon::row_closure(1).
-                 &Apache::lonhtmlcommon::end_pick_box().'</div>'.
-                 '<div>'.&clone_form($dom,$formname,$crstype).'</div>'."\n";
+    my ($dom,$formname,$crstype,$next) = @_;
+    my $nodescr = &mt('You must provide a (brief) course description.');
+    my $js_validate = <<"ENDJS";
+<script type="text/javascript">
+// <![CDATA['
+
+function validateForm() {
+    if ((document.$formname.cdescr.value == "")  || (document.$formname.cdescr.value == "undefined")) {
+        alert('$nodescr');
+        return;
+    }
+    nextPage(document.$formname,'$next');
+}
+// ]]
+</script>
+
+ENDJS
+
+    my $output .= $js_validate."\n".'<div>'.&Apache::lonhtmlcommon::start_pick_box().
+                  &Apache::lonhtmlcommon::row_title('Course Description').
+                 '<input type="text" size="40" name="cdescr" />';
+    my ($home_server_pick,$numlib) =
+        &Apache::loncommon::home_server_form_item($dom,'chome',
+                                                  'default','hide');
+    if ($numlib > 1) {
+        $output .= &Apache::lonhtmlcommon::row_closure().
+                   &Apache::lonhtmlcommon::row_title(
+                       &mt('Home Server for Course'));
+    }
+    $output .= $home_server_pick.
+               &Apache::lonhtmlcommon::row_closure(1).
+               &Apache::lonhtmlcommon::end_pick_box().'</div>'.
+               '<div>'.&clone_form($dom,$formname,$crstype).'</div>'."\n";
     return $output;
 }
 
@@ -898,21 +1390,14 @@ sub clone_form {
     if ($crstype eq 'community') {
         $type = 'Community';
     }
-    my $cloneform = &Apache::loncommon::select_dom_form($dom,'clonedomain').
-                    &Apache::loncommon::selectcourse_link($formname,'clonecourse','clonedomain','','','',$type);
-    my %lt = &Apache::lonlocal::texthash(
-                    'cid'  => 'Course ID',
-                    'dmn'  => 'Domain',
-                    'dsh'  => 'Date Shift',
-                    'ncd'  => 'Do not clone date parameters',
-                    'prd'  => 'Clone date parameters as-is',
-                    'shd'  => 'Shift date parameters by number of days',
-    );
+    my $cloneform = &Apache::loncommon::select_dom_form($dom,'clonedom').
+                    &Apache::loncommon::selectcourse_link($formname,'clonecrs','clonedom','','','',$type);
+    my %lt = &clone_text();
     my $output .= 
         &Apache::lonhtmlcommon::start_pick_box(). 
         &Apache::lonhtmlcommon::row_title($lt{'cid'}).'<label>'.
-        '<input type="text" size="25" name="clonecourse" value="" onfocus="this.blur();'.
-        'opencrsbrowser('."'$formname','clonecourse','clonedomain','','','','$type'".');" />'.
+        '<input type="text" size="25" name="clonecrs" value="" onfocus="this.blur();'.
+        'opencrsbrowser('."'$formname','clonecrs','clonedom','','','','$type'".');" />'.
         '</label>'.&Apache::lonhtmlcommon::row_closure(1).'<label>'.
         &Apache::lonhtmlcommon::row_title($lt{'dmn'}).'</label>'.
         $cloneform.'</label>'.&Apache::lonhtmlcommon::row_closure().
@@ -929,6 +1414,17 @@ sub clone_form {
     return $output;
 }
 
+sub clone_text { 
+    return &Apache::lonlocal::texthash(
+               'cid'  => 'Course ID',
+               'dmn'  => 'Domain',
+               'dsh'  => 'Date Shift',
+               'ncd'  => 'Do not clone date parameters',
+               'prd'  => 'Clone date parameters as-is',
+               'shd'  => 'Shift date parameters by number of days',
+        );
+}
+
 sub coursecode_form {
     my ($dom,$context,$codetitles,$cat_titles,$cat_order,$num) = @_;
     my $output;
@@ -1033,7 +1529,7 @@ sub get_course_dom {
 }
 
 sub display_navbuttons {
-    my ($r,$formname,$prev,$prevtext,$next,$nexttext) = @_;
+    my ($r,$formname,$prev,$prevtext,$next,$nexttext,$state) = @_;
     $r->print('<div class="LC_navbuttons">');
     if ($prev) {
         $r->print('
@@ -1046,7 +1542,10 @@ sub display_navbuttons {
     onclick="javascript:history.back()"/>
    &nbsp;&nbsp;&nbsp;');
     }
-    if ($next) {
+    if ($state eq 'courseinfo') {
+        $r->print('
+      <input type="button" name="next" value="'.$nexttext.'" onclick="javascript:validateForm();" />');
+    } elsif ($next) {
         $r->print('
       <input type="button" name="next" value="'.$nexttext.'"
  onclick="javascript:nextPage(document.'.$formname.','."'".$next."'".')" />');
@@ -1055,6 +1554,420 @@ sub display_navbuttons {
 }
 
 sub print_request_outcome {
+    my ($dom,$codetitles,$code_order) = @_;
+    my ($output,$cnum,$now,$req_notifylist,$crstype,$enrollstart,$enrollend,
+        %sections,%crosslistings,%personnel,@baduname,@missingdom,%domconfig,);
+    $cnum = $env{'form.cnum'};
+    unless ($cnum =~ /^$match_courseid$/) {
+        $output = &mt('Invalid LON-CAPA course number for the new course')."\n"; 
+        return $output;
+    }
+
+    %domconfig = &Apache::lonnet::get_dom('configuration',['requestcourses'],$dom);
+    if (ref($domconfig{'requestcourses'}) eq 'HASH') {
+        if (ref($domconfig{'requestcourses'}{'notify'}) eq 'HASH') {
+            $req_notifylist = $domconfig{'requestcourses'}{'notify'}{'approval'};
+        }
+    }
+    $now = time;
+    $crstype = $env{'form.crstype'};
+    if ($crstype eq 'official') {
+        if (&Apache::lonnet::auto_run('',$dom)) {
+            ($enrollstart,$enrollend)=&dates_from_form('enrollstart','enrollend');
+        }
+        for (my $i=0; $i<$env{'form.sectotal'}; $i++) {
+            if ($env{'form.sec_'.$i}) {
+                if ($env{'form.secnum_'.$i} ne '') {
+                    $sections{$i}{'inst'} = $env{'form.secnum_'.$i};
+                    $sections{$i}{'loncapa'} = $env{'form.loncapasec_'.$i};
+                }
+            }
+        }
+        for (my $i=0; $i<$env{'form.crosslisttotal'}; $i++) {
+            if ($env{'form.crosslist_'.$i}) {
+                my $xlistinfo = '';
+                if (ref($code_order) eq 'ARRAY') {
+                    if (@{$code_order} > 0) {
+                        foreach my $item (@{$code_order}) {
+                            $xlistinfo .= $env{'form.crosslist_'.$i.'_'.$item};
+                        }
+                    }
+                }
+                $xlistinfo .= $env{'form.crosslist_'.$i.'_instsec'};
+                $crosslistings{$i}{'inst'} = $xlistinfo;
+                $crosslistings{$i}{'loncapa'} = $env{'form.crosslist_'.$i.'_lcsec'};
+            }
+        }
+    } else {
+        $enrollstart = '';
+        $enrollend = '';
+    }
+    for (my $i=0; $i<$env{'form.persontotal'}; $i++) {
+        my $uname = $env{'form.person_'.$i.'_uname'};
+        my $udom = $env{'form.person_'.$i.'_uname'};
+        if (($uname =~ /^$match_username$/) && ($udom =~ /^$match_domain$/)) {
+            if (&Apache::lonnet::domain($udom) ne '') {
+                unless (ref($personnel{$uname.':'.$udom}) eq 'HASH') {
+                    $personnel{$uname.':'.$udom} = {
+                             firstname    => $env{'form.person_'.$i.'_firstname'},
+                             lastname     => $env{'form.person_'.$i.'_lastname'},
+                             emailaddr    => $env{'form.person_'.$i.'_emailaddr'},
+                                                   };
+                }
+                my $role = $env{'form.person_'.$i.'_role'};
+                unless ($role eq '') {
+                    if (ref(@{$personnel{$uname.':'.$udom}{'roles'}}) eq 'ARRAY') {
+                        my @curr_roles = @{$personnel{$uname.':'.$udom}{'roles'}};
+                        unless (grep(/^\Q$role\E$/,@curr_roles)) {
+                            push(@{$personnel{$uname.':'.$udom}{'roles'}},$role);
+                        }
+                    } else {
+                        @{$personnel{$uname.':'.$udom}{'roles'}} = ($role);
+                    }
+                    if ($role eq 'cc') {
+                        @{$personnel{$uname.':'.$udom}{$role}{'usec'}} = ();
+                    } else {
+                        my @currsec = &Apache::loncommon::get_env_multiple('form.person_'.$i.'_sec');
+                        my $newsec = $env{'form.person_'.$i.'_newsec'};
+                        $newsec =~ s/^\s+//;
+                        $newsec =~s/\s+$//;
+                        my @newsecs = split(/[\s,;]+/,$newsec);
+                        foreach my $sec (@newsecs) {
+                            next if ($sec =~ /\W/);
+                            next if ($newsec eq 'none');
+                            if ($sec ne '') {
+                                unless (grep(/^\Q$sec\E$/,@currsec)) {
+                                    push(@currsec,$sec);
+                                }
+                            }
+                        }
+                        @{$personnel{$uname.':'.$udom}{$role}{'usec'}} = @currsec;
+                    }
+                }
+            } else {
+                push(@missingdom,$uname.':'.$udom);
+            }
+        } else {
+            push(@baduname,$uname.':'.$udom);
+        }
+    }
+    my ($accessstart,$accessend) = &dates_from_form('accessstart','accessend');
+    my $autodrops = 0;
+    if ($env{'form.autodrops'}) {
+        $autodrops = $env{'form.autodrops'}; 
+    }
+    my $autoadds = 0;
+    if ($env{'form.autoadds'}) {
+        $autodrops = $env{'form.autoadds'};
+    }
+    my $sectotal = 0;
+    if ($env{'form.sectotal'}) {
+        $sectotal = $env{'form.sectotal'};
+    }
+    if ($env{'form.autoadds'}) {
+        $autodrops = $env{'form.autoadds'};
+    }
+    my $crosslisttotal = 0;
+    if ($env{'form.crosslisttotal'}) {
+        $crosslisttotal = $env{'form.crosslisttotal'};
+    }
+    my $instcode = '';
+    if (exists($env{'form.instcode'})) {
+        $instcode = $env{'form.instcode'};
+    }
+    my $clonecrs = '';
+    my $clonedom = '';
+    if (($env{'form.clonecrs'} =~ /^($match_courseid)$/) && 
+        ($env{'form.clonedom'} =~ /^($match_domain)$/)) {
+        my $clonehome = &Apache::lonnet::homeserver($env{'form.clonedom'},$env{'form.clonecrs'});
+        if ($clonehome ne 'no_host') {  
+            my $canclone = &can_clone_course($env{'user.name'},$env{'user.domain'},
+                                             $env{'form.clonecrs'}, $env{'form.clonedom'});
+            if ($canclone) {
+                $clonecrs = $env{'form.clonecrs'};
+                $clonedom = $env{'form.clonedom'};
+            }
+        }
+    }
+    my $details = {
+                    owner          => $env{'user.name'},
+                    domain         => $env{'user.domain'}, 
+                    cdom           => $dom,
+                    cnum           => $cnum,
+                    coursehome     => $env{'form.chome'},
+                    cdescr         => $env{'form.cdescr'},
+                    crstype        => $env{'form.crstype'},
+                    instcode       => $instcode,
+                    clonedom       => $clonedom,
+                    clonecrs       => $clonecrs,
+                    datemode       => $env{'form.datemode'},
+                    dateshift      => $env{'form.dateshift'},
+                    sectotal       => $sectotal,
+                    sections       => \%sections,
+                    crosslisttotal => $crosslisttotal,
+                    crosslists     => \%crosslistings,
+                    autoadds       => $autoadds,
+                    autodrops      => $autodrops,
+                    enrollstart    => $enrollstart,
+                    enrollend      => $enrollend,
+                    accessstart    => $accessstart,
+                    accessend      => $accessend,
+                    personnel      => \%personnel,
+                  };
+    my @inststatuses;
+    my $val = &get_processtype($dom,$crstype,\@inststatuses,\%domconfig);
+    if ($val eq '') {
+        if ($crstype eq 'official') {
+            $output = &mt('You are not permitted to request creation of official courses');
+        } elsif ($crstype eq 'unofficial') {
+            $output = &mt('You are not permitted to request creation of unofficial courses');
+        } elsif ($crstype eq 'community') {
+            $output = &mt('You are not permitted to request creation of communities');
+        } else {
+            $output = &mt('Unrecognized course type: [_1]',$crstype);
+        }
+    } else {
+        my ($disposition,$message,$reqstatus);
+        my %reqhash = (
+                        reqtime   => $now,
+                        crstype   => $crstype,
+                        details   => $details,
+                      );
+        my $requestkey = $dom.'_'.$cnum;
+        if ($val eq 'autolimit=') {
+            $disposition = 'process';
+        } elsif ($val =~ /^autolimit=(\d+)$/) {
+            my $limit = $1;
+            $disposition = &check_autolimit($env{'user.name'},$env{'user.domain'},
+                                            $dom,$crstype,$limit,\$message);
+        } elsif ($val eq 'validate') {
+            $disposition =
+                &Apache::lonnet::auto_courserequest_validation($dom,$details,
+                                                              \@inststatuses,\$message);
+        } else {
+            $disposition = 'approval';
+        }
+        $reqhash{'disposition'} = $disposition;
+        $reqstatus = $disposition;
+        if ($disposition eq 'rejected') {
+            $output = &mt('Your course request was rejected.');
+            if ($message) {
+                $output .= '<div class="LC_warning">'.$message.'</div>';
+            }
+        } elsif ($disposition eq 'process') {
+            my %domdefs = &Apache::lonnet::get_domain_defaults($dom);
+            my ($logmsg,$newusermsg,$addresult,$enrollcount,$response,$keysmsg,%longroles);
+            my @roles = &Apache::lonuserutils::roles_by_context('course');
+            my $type = 'Course';
+            if ($crstype eq 'community') {
+                $type = 'Community';
+            }
+            foreach my $role (@roles) {
+                $longroles{$role}=&Apache::lonnet::plaintext($role,$type);
+            }
+            my $result = &Apache::loncoursequeueadmin::course_creation($dom,$cnum,
+                                   'autocreate',$details,\$logmsg,\$newusermsg,\$addresult,
+                                   \$enrollcount,\$response,\$keysmsg,\%domdefs,\%longroles);
+            if ($result eq 'created') {
+                $disposition = 'created';
+                $reqstatus = 'created';
+                $output = &mt('Your course request has been processed and the course has been created.').
+                          '<br />'.
+                          &mt('You will need to logout and log-in again to be able to select a role in the course.');
+            } else {
+                $output = '<span class="LC_error">'.
+                          &mt('An error occurred when processing your course request.').
+                          '<br />'.
+                          &mt('You may want to review the request details and submit the request again.').
+                          '</span>';
+            }
+        } else {
+            my $requestid = $cnum.'_'.$disposition;
+            my $request = { 
+                            $requestid => {
+                                            timestamp   => $now,
+                                            crstype     => $crstype,
+                                            ownername   => $env{'user.name'},
+                                            ownerdom    => $env{'user.domain'},
+                                            description => $env{'form.cdescr'}, 
+                                          },
+                          };
+            my $putresult = &Apache::lonnet::newput_dom('courserequestqueue',$request,
+                                                        $dom);
+            if ($putresult eq 'ok') {
+                my %emails = &Apache::loncommon::getemails();
+                my $address;
+                if (($emails{'permanentemail'} ne '') || ($emails{'notification'} ne '')) {
+                    $address = $emails{'permanentemail'};
+                    if ($address eq '') {
+                        $address = $emails{'notification'};
+                    }
+                }
+                $output = &mt('Your course request has been recorded.').'<br />';
+                if ($disposition eq 'approval') {
+                    $output .= &mt('A message will be sent to your LON-CAPA account when a domain coordinator takes action on your request.').'<br />'.
+                               &mt('To access your LON-CAPA message, go to the Main Menu and click on "Send and Receive Messages".').'<br />';
+                    if ($address ne '') {     
+                        $output.= &mt('An e-mail will also be sent to: [_1] when this occurs.',$address).'<br />';
+                    }
+                    if ($req_notifylist) {
+                        my $fullname = &Apache::loncommon::plainname($env{'user.name'},
+                                                                     $env{'user.domain'});
+                        my $sender = $env{'user.name'}.':'.$env{'user.domain'};
+                        &Apache::loncoursequeueadmin::send_selfserve_notification($req_notifylist,"$fullname ($env{'user.name'}:$env{'user.domain'})",$cnum,$env{'form.cdescr'},$now,'coursereq',$sender);
+                    }
+                } else {
+                    $output .= '<div class="LC_info">'.
+&mt('Your request has been placed in a queue pending administrative action.').'<br />'.
+&mt("Usually this means that your institution's information systems do not list you among the instructional personnel for this course.").'<br />'.
+&mt('The list of instructional personnel for the course will be automatically checked daily, and once you are listed the request will be processed.').
+                               '</div>';
+                }
+            } else {
+                $reqstatus = 'domainerror';
+                $reqhash{'disposition'} = $disposition;
+                my $warning = &mt('An error occurred saving your request in the pending requests queue.');
+                $output = '<span class"LC_warning">'.$warning.'</span><br />';
+                
+            }
+        }
+        my ($storeresult,$statusresult);
+        if ($requestkey =~ /^($match_domain)_($match_courseid)$/) {
+            $storeresult = &Apache::lonnet::store_userdata(\%reqhash,$requestkey,
+                                                           'courserequests');
+            if ($storeresult eq 'ok') {
+                my %status = (
+                                 'status:'.$dom.':'.$cnum => $reqstatus,
+                             );
+                $statusresult = &Apache::lonnet::put('courserequests',\%status); 
+            }
+        } else {
+            $storeresult = 'error: invalid requestkey format'; 
+        }
+        if ($storeresult ne 'ok') {
+            $output .=  '<span class="LC_warning">'.&mt('An error occurred saving a record of the details of your request: [_1].',$storeresult).'</span><br />';
+            &Apache::lonnet::logthis("Error saving course request - $requestkey for $env{'user.name'}:$env{'user.domain'} - $storeresult");
+        } elsif ($statusresult ne 'ok') {
+             $output .=  '<span class="LC_warning">'.&mt('An error occurred saving a record of the status of your request: [_1].',$statusresult).'</span><br />';
+            &Apache::lonnet::logthis("Error saving course request status for  $requestkey (for $env{'user.name'}:$env{'user.domain'}) - $statusresult");
+        }
+    }
+    return $output;
+}
+
+sub get_processtype {
+    my ($dom,$crstype,$inststatuses,$domconfig) = @_;
+    return unless ((ref($inststatuses) eq 'ARRAY') && (ref($domconfig) eq 'HASH'));
+    my (%userenv,%settings,$val);
+    my @options = ('autolimit','validate','approve');
+    if ($dom eq $env{'user.domain'}) {
+        %userenv = 
+            &Apache::lonnet::userenvironment($env{'user.domain'},$env{'user.name'},
+                'requestcourses.'.$crstype,'inststatus');
+        if ($userenv{'requestcourses.'.$crstype}) {
+            $val = $userenv{'requestcourses.'.$crstype};
+            @{$inststatuses} = ('_custom_');
+        } else {
+            my ($task,%alltasks);
+            if (ref($domconfig->{'requestcourses'}) eq 'HASH') {
+                %settings = %{$domconfig->{'requestcourses'}};
+                if (ref($settings{$crstype}) eq 'HASH') {
+                    if (($env{'user.adv'}) && (exists($settings{$crstype}{'_LC_adv'}))) {
+                        $val = $settings{$crstype}{'_LC_adv'};
+                        @{$inststatuses} = ('_LC_adv_');
+                    } else {
+                        if ($userenv{'inststatus'} ne '') {
+                            @{$inststatuses} = split(',',$userenv{'inststatus'});
+                        } else {
+                            @{$inststatuses} = ('default');
+                        }
+                        foreach my $status (@{$inststatuses}) {
+                            if (exists($settings{$crstype}{$status})) {
+                                my $value = $settings{$crstype}{$status};
+                                next unless ($value); 
+                                unless (exists($alltasks{$value})) {
+                                    if (ref($alltasks{$value}) eq 'ARRAY') {
+                                        unless(grep(/^\Q$status\E$/,@{$alltasks{$value}})) {
+                                            push(@{$alltasks{$value}},$status); 
+                                        }
+                                    } else {
+                                        @{$alltasks{$value}} = ($status);
+                                    }
+                                }
+                            }
+                        }
+                        my $maxlimit = 0;
+                        
+                        foreach my $key (sort(keys(%alltasks))) {
+                            if ($key =~ /^autolimit=(\d*)$/) {
+                                if ($1 eq '') {
+                                    $val ='autolimit=';
+                                    last;
+                                } elsif ($1 > $maxlimit) {
+                                    $maxlimit = $1; 
+                                }
+                            }
+                        }
+                        if ($maxlimit) {
+                            $val = 'autolimit='.$maxlimit;
+                        } else {
+                            foreach my $option (@options) {
+                                if ($alltasks{$option}) {
+                                    $val = $option;
+                                    last;  
+                                }
+                            }
+                        }
+                    }
+                }
+            }
+        }
+    } else {
+        %userenv = &Apache::lonnet::userenvironment($env{'user.domain'},
+                      $env{'user.name'},'reqcrsotherdom.'.$env{'form.crstype'});
+        if ($userenv{'reqcrsotherdom'}) {
+            my @doms = split(',',$userenv{'reqcrsotherdom'});
+            my $optregex = join('|',@options); 
+            if (grep(/^\Q$dom\E:($optregex=?\d*)/,@doms)) {
+                $val = $1;
+            }
+            @{$inststatuses} = ('_external_');
+        }
+    }
+    return $val;
+}
+
+sub check_autolimit {
+    my ($uname,$udom,$dom,$crstype,$limit,$message) = @_;
+    my %crsroles = &Apache::lonnet::get_my_roles($env{'user.name'},$env{'user.domain'},
+                                        'userroles',['active','future'],['cc'],[$dom]);
+    my ($types,$typename) = &course_types();
+    my %requests = &Apache::lonnet::dumpstore('courserequests',$udom,$uname);
+    my %count;
+    if (ref($types) eq 'ARRAY') {
+        foreach my $type (@{$types}) {
+            $count{$type} = 0;
+        }
+    }
+    foreach my $key (keys(%requests)) {
+        my ($cdom,$cnum) = split('_',$key);
+        if (exists($crsroles{$cnum.':'.$cdom.':cc'})) {
+            if (ref($requests{$key}) eq 'HASH') {
+                my $type = $requests{$key}{'crstype'};
+                if ($type =~ /^official|unofficial|community$/) {
+                    $count{$type} ++;
+                }
+            }
+        }
+    }
+    if ($count{$crstype} < $limit) {
+        return 'process';
+    } else {
+        if (ref($typename) eq 'HASH') {
+            $$message = &mt('Your request has not been processed because you have reached the limit for the number of courses of this type.').'<br />'.&mt("Your $typename->{$crstype} limit is [_1].",$limit);
+        }
+        return 'rejected';
+    }
     return;
 }
 
@@ -1063,16 +1976,16 @@ sub retrieve_settings {
     my %reqinfo = &get_request_settings($request_id,$dom);
     my %stored;
     $stored{'cdescr'} = &unescape($reqinfo{'description'});
-    $stored{'startaccess'} = $reqinfo{'startaccess'};
-    $stored{'endaccess'} = $reqinfo{'endaccess'};
-    if ($stored{'endaccess'} == 0) {
+    $stored{'accessstart'} = $reqinfo{'accessstart'};
+    $stored{'accessend'} = $reqinfo{'accessend'};
+    if ($stored{'accessend'} == 0) {
         $stored{'no_end_date'} = 1;
     }
-    $stored{'startenroll'} = $reqinfo{'startenroll'};
-    $stored{'endenroll'} = $reqinfo{'endenroll'};
+    $stored{'enrollstart'} = $reqinfo{'enrollstart'};
+    $stored{'enrollend'} = $reqinfo{'enrollend'};
     $stored{'crosslist'} = $reqinfo{'crosslist'};
-    $stored{'clonecourse'} = $reqinfo{'clonecourse'};
-    $stored{'clonedomain'} = $reqinfo{'clonedomain'};
+    $stored{'clonecrs'} = $reqinfo{'clonecrs'};
+    $stored{'clonedom'} = $reqinfo{'clonedom'};
     $stored{'sections'} = $reqinfo{'sections'};
     $stored{'personnel'} = $reqinfo{'personnel'};