--- loncom/auth/lonroles.pm	2012/12/13 21:11:17	1.269.2.4
+++ loncom/auth/lonroles.pm	2012/08/15 14:37:23	1.272
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # User Roles Screen
 #
-# $Id: lonroles.pm,v 1.269.2.4 2012/12/13 21:11:17 raeburn Exp $
+# $Id: lonroles.pm,v 1.272 2012/08/15 14:37:23 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -149,7 +149,6 @@ sub redirect_user {
     &Apache::loncommon::content_type($r,'text/html');
     &Apache::loncommon::no_cache($r);
     $r->send_http_header;
-    my $swinfo=&Apache::lonmenu::rawconfig();
 
     # Breadcrumbs
     my $brcrum = [{'href' => $url,
@@ -164,11 +163,6 @@ sub redirect_user {
     $url=~s/ /\%20/g;
     $r->print(<<ENDREDIR);
 $start_page
-<script type="text/javascript">
-// <![CDATA[
-$swinfo
-// ]]>
-</script>
 <p>$msg</p>
 $end_page
 ENDREDIR
@@ -217,17 +211,9 @@ sub handler {
         $update = $then;
     }
 
-    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'});
-
-# -------------------------------------------------- Check if setting hot list
-    my $hotlist;
-    if ($env{'form.action'} eq 'verify_and_change_rolespref') {
-        $hotlist = &Apache::lonpreferences::verify_and_change_rolespref($r);
-    }
-
 # -------------------------------------------------------- Check for new roles
     my $updateresult;
-    if ($env{'form.state'} eq 'doupdate') {
+    if ($env{'form.doupdate'}) {
         my $show_course=&Apache::loncommon::show_course();
         my $checkingtxt;
         if ($show_course) {
@@ -235,7 +221,7 @@ sub handler {
         } else {
             $checkingtxt = &mt('Checking for new roles ...');
         }
-        $updateresult = $checkingtxt;
+        $updateresult = '<span class="LC_info">'.$checkingtxt.'</span>';
         $updateresult .= &update_session_roles();
         &Apache::lonnet::appenv({'user.update.time'  => $now});
         $update = $now;
@@ -244,13 +230,14 @@ sub handler {
 
 # -------------------------------------------------- Check for author requests
     my $reqauthor;
-    if ($env{'form.state'} eq 'requestauthor') {
+    if ($env{'form.requestauthor'}) {
        $reqauthor = &Apache::loncoursequeueadmin::process_reqauthor(\$update);
     }
 
     my $envkey;
     my %dcroles = ();
     my $numdc = &check_fordc(\%dcroles,$update,$then);
+    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'});
     my $loncaparev = $Apache::lonnet::perlvar{'lonVersion'};
 
 # ================================================================== Roles Init
@@ -297,10 +284,8 @@ sub handler {
                 if (my ($ccrole,$domain,$coursenum) =
 		    ($envkey =~ m-^form\.(cc|co)\./($match_domain)/($match_courseid)$-)) {
                     if ($dcroles{$domain}) {
-                        if (&Apache::lonnet::check_adhoc_privs($domain,$coursenum,
-                                                           $update,$refresh,$now,$ccrole)) {
-                            &Apache::lonnet::appenv({"environment.internal.$domain.$coursenum.$ccrole.adhoc" => time});
-                        }
+                        &Apache::lonnet::check_adhoc_privs($domain,$coursenum,
+                                                           $update,$refresh,$now,$ccrole);
                     }
                     last;
                 }
@@ -523,12 +508,6 @@ ENDENTERKEY
                         my $msg;
 			my ($furl,$ferr)=
 			    &Apache::lonuserstate::readmap($cdom.'/'.$cnum);
-                        unless (($ferr) || ($env{'form.switchrole'}) ||
-                                ($env{"environment.internal.$cdom.$cnum.$role.adhoc"})) {
-                            &Apache::lonnet::put('nohist_crslastlogin',
-                                {$env{'user.name'}.':'.$env{'user.domain'}.
-                                ':'.$csec.':'.$role => $now},$cdom,$cnum);
-                        }
 			if (($env{'form.orgurl'}) && 
 			    ($env{'form.orgurl'}!~/^\/adm\/flip/)) {
 			    my $dest=$env{'form.orgurl'};
@@ -585,52 +564,11 @@ ENDENTERKEY
 				    $furl = "/adm/helper/course.initialization.helper";
 				    # Send the user to the course they selected
 				} elsif ($env{'request.course.id'}) {
-                                    my ($dest,$destsymb,$checkenc);
-                                    $dest = $env{'form.destinationurl'};
-                                    $destsymb = $env{'form.destsymb'};
-                                    if ($dest ne '') {
-                                        if ($env{'form.switchrole'}) {
-                                            if ($destsymb ne '') {
-                                                if ($destsymb !~ m{^/enc/}) {
-                                                    unless ($env{'request.role.adv'}) {
-                                                        $checkenc = 1;
-                                                    }
-                                                }
-                                            }
-                                            if ($dest =~ m{^/enc/}) {
-                                                if ($env{'request.role.adv'}) {
-                                                    $dest = &Apache::lonenc::unencrypted($dest);
-                                                    if ($destsymb eq '') {
-                                                        ($destsymb) = ($dest =~ /(?:\?|\&)symb=([^\&]*)/); 
-                                                        $destsymb = &unescape($destsymb);
-                                                    }
-                                                }
-                                            } else {
-                                                if ($destsymb eq '') {
-                                                    ($destsymb) = ($dest =~ /(?:\?|\&)symb=([^\&]+)/);
-                                                    $destsymb = &unescape($destsymb);
-                                                }
-                                                unless ($env{'request.role.adv'}) {
-                                                    $checkenc = 1;
-                                                }
-                                            }
-                                            if (($checkenc) && ($destsymb ne '')) {
-                                                my ($encstate,$unencsymb,$res);
-                                                my $unencsymb = &Apache::lonnet::symbclean($destsymb);
-                                                (undef,undef,$res) = &Apache::lonnet::decode_symb($unencsymb);
-                                                &Apache::lonnet::symbverify($unencsymb,$res,\$encstate);
-                                                if ($encstate) {
-                                                    if (($dest ne '') && ($dest !~ m{^/enc/})) {
-                                                        $dest=&Apache::lonenc::encrypted($dest);
-                                                    }
-                                                }
-                                            }
-                                        }
-                                        unless (($dest =~ m{^/enc/}) || ($dest =~ /(\?|\&)symb=.+___\d+___.+/)) { 
-                                            if (($destsymb ne '') && ($destsymb !~ m{^/enc/})) {
-                                                my $esc_symb = &escape($destsymb);
-                                                $dest .= '?symb='.$esc_symb;
-                                            }
+                                    if ($env{'form.destinationurl'}) {
+                                        my $dest = $env{'form.destinationurl'};
+                                        if ($env{'form.destsymb'} ne '') {
+                                            my $esc_symb = &HTML::Entities::encode($env{'form.destsymb'},'"<>&');
+                                            $dest .= '?symb='.$esc_symb;
                                         }
                                         &redirect_user($r, &mt('Entering [_1]',
                                                        $env{'course.'.$courseid.'.description'}),
@@ -716,20 +654,15 @@ ENDENTERKEY
         $recent = &mt('Recent Courses');
     }
     my $brcrum =[{href=>"/adm/roles",text=>$crumbtext}];
-
-    my %roles_in_env;
-    my $showcount = &roles_from_env(\%roles_in_env,$update); 
-
     my $swinfo=&Apache::lonmenu::rawconfig();
     my $start_page=&Apache::loncommon::start_page($pagetitle,undef,{bread_crumbs=>$brcrum});
-    my $funcs = &get_roles_functions($showcount);
     my $standby=&mt('Role selected. Please stand by.');
     $standby=~s/\n/\\n/g;
-    my $noscript='<br /><span class="LC_error">'.&mt('Use of LON-CAPA requires Javascript to be enabled in your web browser.').'<br />'.&mt('As this is not the case, most functionality in the system will be unavailable.').'</span><br />';
+    my $noscript='<span class="LC_error">'.&mt('Use of LON-CAPA requires Javascript to be enabled in your web browser.').'<br />'.&mt('As this is not the case, most functionality in the system will be unavailable.').'</span><br />';
 
     $r->print(<<ENDHEADER);
 $start_page
-$funcs
+<br />
 <noscript>
 $noscript
 </noscript>
@@ -752,17 +685,16 @@ function enterrole (thisform,rolecode,bu
     }
 }
 
-function rolesView (caller) {
-    if ((caller == 'showall') || (caller == 'noshowall')) {
-        document.rolechoice.display.value = caller;
-    } else {
-        if ((caller == 'doupdate') || (caller == 'requestauthor') ||
-            (caller == 'queued')) {
-            document.rolechoice.state.value = caller;
-        }
-    }
-    document.rolechoice.selectrole.value='';
-    document.rolechoice.submit();
+function setToUpdate(thisform) {
+    thisform.doupdate.value='1';
+    thisform.selectrole.value='';
+    thisform.submit();
+}
+
+function setToRequestAuthor(thisform) {
+    thisform.requestauthor.value='1';
+    thisform.selectrole.value='';
+    thisform.submit(); 
 }
 
 // ]]>
@@ -831,43 +763,115 @@ ENDHEADER
 		  &Apache::loncommon::end_page());
 	return OK;
     } else {
-        if ($updateresult || $reqauthor || $hotlist) {
-            my $showresult = '<div>';
-            if ($updateresult) {
-                $showresult .= &Apache::lonhtmlcommon::confirm_success($updateresult);
-            }
-            if ($reqauthor) {
-                $showresult .= &Apache::lonhtmlcommon::confirm_success($reqauthor);
-            }
-            if ($hotlist) {
-                $showresult .= $hotlist;
-            }
-            $showresult .= '</div>';
-            $r->print($showresult);
-        } elsif ($env{'form.state'} eq 'queued') {
-            $r->print(&get_queued());
+        if ($updateresult || $reqauthor) {
+            $r->print('<div>'.
+                      $updateresult.
+                      $reqauthor.
+                      '</div>');
         }
         if (($ENV{'REDIRECT_QUERY_STRING'}) && ($fn)) {
     	    $fn.='?'.$ENV{'REDIRECT_QUERY_STRING'};
         }
-        my $display = ($env{'form.display'} =~ /^(showall)$/);
         $r->print('<form method="post" name="rolechoice" action="'.(($fn)?$fn:$r->uri).'">');
         $r->print('<input type="hidden" name="orgurl" value="'.$fn.'" />');
         $r->print('<input type="hidden" name="selectrole" value="1" />');
         $r->print('<input type="hidden" name="newrole" value="" />');
-        $r->print('<input type="hidden" name="display" value="'.$display.'" />');
-        $r->print('<input type="hidden" name="state" value="" />');
     }
     $r->rflush();
 
     my (%roletext,%sortrole,%roleclass,%futureroles,%timezones);
     my ($countactive,$countfuture,$inrole,$possiblerole) = 
-        &gather_roles($update,$refresh,$now,$reinit,$nochoose,\%roles_in_env,\%roletext,
-                      \%sortrole,\%roleclass,\%futureroles,\%timezones,$loncaparev);
+        &gather_roles($update,$refresh,$now,$reinit,$nochoose,\%roletext,\%sortrole,\%roleclass,
+                      \%futureroles,\%timezones,$loncaparev);
     $refresh = $now;
     &Apache::lonnet::appenv({'user.refresh.time'  => $refresh});
-    unless ($env{'user.adv'}) {
+    my $updatebutton = &mt('Check for role changes');
+    my $show_course=&Apache::loncommon::show_course();
+    if ($show_course) {
+        $updatebutton = &mt('Check for new courses');
+    }
+    my $do_update;
+    unless (($env{'form.source'} eq 'login') || ($env{'form.doupdate'})) {
+        $do_update = '<input type="hidden" name="doupdate" value="" />'.
+                     '<input type="button" name="update" value="'.
+                     $updatebutton.'" onclick="javascript:setToUpdate(this.form)" />';
+    }
+    my ($requestauthor,$requestcrs);
+    unless ($reqauthor) {
+        if ($env{'environment.canrequest.author'}) {
+            unless (&is_active_author()) {
+                my ($status,$timestamp) = split(/:/,$env{'environment.requestauthorqueued'});
+                if ($status eq 'approval') {
+                    $requestauthor = &mt('A request for authoring space submitted on [_1] is awaiting approval',&Apache::lonlocal::locallocaltime($timestamp)); 
+                } elsif (($status eq 'approved') && ($do_update)) {
+                    my %roleshash = &Apache::lonnet::get_my_roles($env{'user.name'},$env{'user.domain'},'userroles',
+                                                                  ['active'],['au'],[$env{'user.domain'}]);
+                    if (keys(%roleshash)) {
+                        $requestauthor = &mt('Your request for an author role has been approved.').'<br />';
+                        if ($show_course) {
+                            $requestauthor .= &mt('Use the "Check for new courses" button to update your list of roles.');
+                        } else {
+                            $requestauthor .= &mt('Use the "Check for new roles" button to update your list of roles.'); 
+                        }
+                        $requestauthor = '<span class="LC_info">'.$requestauthor.'</span>';
+                    }
+                }
+                unless ($requestauthor) {
+                    $requestauthor = 
+                        '<input type="hidden" name="requestauthor" value="" />'.
+                        '<input type="button" name="reqauthor" value="'.
+                        &mt('Request author space').'" '. 
+                        'onclick="javascript:setToRequestAuthor(this.form)" />';
+                }
+            }
+        }
+    }
+    my $do_update;
+    unless (($env{'form.source'} eq 'login') || ($env{'form.doupdate'})) {
+        $do_update = '<input type="hidden" name="doupdate" value="" />'.
+                     '<input type="button" name="update" value="'.
+                     $updatebutton.'" onclick="javascript:setToUpdate(this.form)" />';
+    }
+    if ($env{'user.adv'}) {
+        my $showall = '<label><input type="checkbox" name="showall"';
+        if ($env{'form.showall'}) { 
+            $showall .= ' checked="checked" ';
+        }
+        $showall .= ' />'.&mt('Show all roles').'</label>&nbsp;'.
+                    '<input type="submit" value="'.&mt('Update display').'" />';
+        if ($do_update || $requestauthor) {
+            $r->print('<div class="LC_left_float"><fieldset>'.
+                      '<legend>'. &mt('Display').'</legend>'.
+                      $showall.'</fieldset></div>');
+            if ($do_update) {
+                $r->print('<div class="LC_left_float">'.
+                          '<fieldset><legend>'.&mt('Changes?').'</legend>'.
+                          $do_update.'</fieldset></div>');
+            }
+            if ($requestauthor) {
+                $r->print('<div class="LC_left_float">'.
+                          '<fieldset><legend>'.&mt('Add author role').'</legend>'.
+                          $requestauthor.'</fieldset>');
+            }
+            $r->print('</div><br clear="all" />');
+        } else {
+            $r->print($showall);
+        }
+    } else {
+        if ($do_update && $requestauthor) {
+            $r->print('<div class="LC_left_float">'.
+                      '<fieldset><legend>'.&mt('Changes?').'</legend>'.
+                      $do_update.'</fieldset></div>'.
+                      '<div class="LC_left_float">'.
+                      '<fieldset><legend>'.&mt('Add author role').'</legend>'.
+                      $requestauthor.'</fieldset></div><br clear="all" />');
+        } elsif ($do_update) {
+            $r->print('<p>'.$do_update.'</p>');
+        } elsif ($requestauthor) {
+            $r->print('<p>'.$requestauthor.'</p>');
+        }
         if ($countactive > 0) {
+            $r->print(&Apache::loncoursequeueadmin::queued_selfenrollment());
             my $domdesc = &Apache::lonnet::domain($env{'user.domain'},'description');
             my $esc_dom = &HTML::Entities::encode($env{'user.domain'},'"<>&'); 
             $r->print(
@@ -1045,38 +1049,46 @@ ENDHEADER
     return OK;
 }
 
-sub roles_from_env {
-    my ($roleshash,$update) = @_;
-    my $count = 0;
-    if (ref($roleshash) eq 'HASH') {
-        foreach my $envkey (keys(%env)) {
-            if ($envkey =~ m{^user\.role\.(\w+)[./]}) {
-                next if ($1 eq 'gr');
-                $roleshash->{$envkey} = $env{$envkey};
-                my ($start,$end) = split(/\./,$env{$envkey});
-                unless ($end && $end<$update) {
-                    $count ++;
-                }
-            }
+sub is_active_author {
+    if ($env{'user.role.au./'.$env{'user.domain'}.'/'} =~ /^(\d*)\.(\d*)$/) {
+        if ((!$1 || $1 < time) &&
+            (!$2 || $2 > time)) {
+            return 1;
+        }
+    }
+    return;
+}
+
+sub reqauthor_check {
+    my $queued = $env{'environment.requestauthorqueued'};
+    my %reqauthor = &Apache::lonnet::get('requestauthor',['author_status','author'],
+                                         $env{'user.domain'},$env{'user.name'});
+    my $reqstatus = $reqauthor{'author_status'};
+    if (($reqstatus eq '' && $queued ne '') || 
+        ($env{'environment.requestauthorqueued'} !~ /^\Q$reqstatus\E/)) {
+        if (ref($reqauthor{'author'}) eq 'HASH') {
+            $queued = $reqstatus.':'.$reqauthor{'author'}{'timestamp'};
+        } else {
+            undef($queued);
         }
+        &Apache::lonnet::appenv({'environment.requestauthorqueued' => $queued});
     }
-    return $count;
+    return $queued;
 }
 
 sub gather_roles {
-    my ($update,$refresh,$now,$reinit,$nochoose,$roles_in_env,$roletext,$sortrole,$roleclass,$futureroles,
-        $timezones,$loncaparev) = @_;
+    my ($update,$refresh,$now,$reinit,$nochoose,$roletext,$sortrole,$roleclass,$futureroles,$timezones,$loncaparev) = @_;
     my ($countactive,$countfuture,$inrole,$possiblerole) = (0,0,0,'');
     my $advanced = $env{'user.adv'};
     my $tryagain = $env{'form.tryagain'};
     my @ids = &Apache::lonnet::current_machine_ids();
-    if (ref($roles_in_env) eq 'HASH') {
-        foreach my $envkey (sort(keys(%{$roles_in_env}))) {
-            my $button = 1;
-            my $switchserver='';
-            my $switchwarning;
-            my ($role_text,$role_text_end,$sortkey,$role,$where,$trolecode,$tstart,
-                $tend,$tremark,$tstatus,$tpstart,$tpend);
+    foreach my $envkey (sort(keys(%env))) {
+        my $button = 1;
+        my $switchserver='';
+        my $switchwarning;
+        my ($role_text,$role_text_end,$sortkey);
+        if ($envkey=~/^user\.role\./) {
+            my ($role,$where,$trolecode,$tstart,$tend,$tremark,$tstatus,$tpstart,$tpend);
             &Apache::lonnet::role_status($envkey,$update,$refresh,$now,\$role,\$where,
                                          \$trolecode,\$tstatus,\$tstart,\$tend);
             next if (!defined($role) || $role eq '' || $role =~ /^gr/);
@@ -1090,7 +1102,7 @@ sub gather_roles {
             if (($tstatus eq 'is')
                 || ($tstatus eq 'selected')
                 || ($tstatus eq 'future')
-                || ($env{'form.display'} eq 'showall')) {
+                || ($env{'form.showall'})) {
                 my $timezone = &role_timezone($where,$timezones);
                 if ($tstart) {
                     $tpstart=&Apache::lonlocal::locallocaltime($tstart,$timezone);
@@ -2223,7 +2235,7 @@ sub update_session_roles {
     }
     if (($oldsec) && (@newsec > 0)) {
         if (@newsec > 1) {
-            $msg = '<p class="LC_warning">'.&mt('The section has changed for your current role. Log-out and log-in again to select a role for the new section.').'</p>';
+            $msg = '<div class="LC_warning">'.&mt('The section has changed for your current role. Log-out and log-in again to select a role for the new section.').'</div>';
         } else {
             my $newrole = $env{'request.role'};
             if ($newsec[0] eq 'none') {
@@ -2261,13 +2273,13 @@ sub update_session_roles {
             $msg .= '</form></p>';
         }
     } elsif ($currrole_expired) {
-        $msg .= '<p class="LC_warning">';
+        $msg .= '<div class="LC_warning">';
         if (&Apache::loncommon::show_course()) {
             $msg .= &mt('Your role in the current course has expired.');
         } else {
             $msg .= &mt('Your current role has expired.');
         }
-        $msg .= '<br />'.&mt('However you can continue to use this role until you logout, click the "Re-Select" button, or your session has been idle for more than 24 hours.').'</p>';
+        $msg .= '<br />'.&mt('However you can continue to use this role until you logout, click the "Re-Select" button, or your session has been idle for more than 24 hours.').'</div>';
     }
     if (!@changed_roles || !(keys(%changed_groups))) {
         my ($rolesmsg,$groupsmsg);
@@ -2549,131 +2561,6 @@ sub is_active_course {
     return $is_active;
 }
 
-sub get_roles_functions {
-    my ($rolescount) = @_;
-    my @links;
-    if ($env{'user.adv'}) {
-        if ($env{'form.display'} eq 'showall') {
-            push(@links,["javascript:rolesView('noshowall');",'edit-redo-22x22',&mt('Exclude expired roles')]);
-        } else {
-            push(@links,["javascript:rolesView('showall');",'edit-undo-22x22',&mt('Include expired roles')]);
-        }
-    }
-    push(@links,["javascript:rolesView('doupdate');",'start-here-22x22',&mt('Check for changes')]);
-    if ($env{'environment.canrequest.author'}) {
-        unless (&Apache::loncoursequeueadmin::is_active_author()) {
-            push(@links,["javascript:rolesView('requestauthor');",'list-add-22x22',&mt('Request author role')]);
-        }
-    }
-    if (&Apache::lonmenu::check_for_rcrs()) {
-        push(@links,['/adm/requestcourse','rcrs-22x22',&mt('Request course')]);
-    }
-    if ($env{'form.state'} eq 'queued') {
-        push(@links,["javascript:rolesView('noqueued');",'selfenrl-queue-22x22',&mt('Hide queued')]);
-    } else {
-        push(@links,["javascript:rolesView('queued');",'selfenrl-queue-22x22',&mt('Show queued')]);
-    }
-    if (($rolescount > 3) || ($env{'environment.recentroles'})) {
-        push(@links,['/adm/preferences?action=changerolespref&amp;returnurl=/adm/roles','role_hotlist-22x22',&mt('Hotlist')]);
-    }
-
-    my $funcs = &Apache::lonhtmlcommon::start_funclist();
-    foreach my $link (@links) {
-        $funcs .= &Apache::lonhtmlcommon::add_item_funclist(
-                      '<a href="'.$link->[0].'" class="LC_menubuttons_link">'.
-                      '<img src="/res/adm/pages/'.$link->[1].'.png" class="LC_icon" alt="'.$link->[2].'" />'.
-                      $link->[2].'</a>');
-    }
-    $funcs .= &Apache::lonhtmlcommon::end_funclist();
-    return &Apache::loncommon::head_subbox($funcs);
-}
-
-sub get_queued {
-    my ($output,%reqcrs);
-    my ($types,$typenames) = &Apache::loncommon::course_types();
-    my %statusinfo = &Apache::lonnet::dump('courserequests',$env{'user.domain'},
-                                           $env{'user.name'},'^status:');
-    foreach my $key (keys(%statusinfo)) {
-        next unless (($statusinfo{$key} eq 'approval') || ($statusinfo{$key} eq 'pending'));
-        (undef,my($cdom,$cnum)) = split(':',$key);
-        my $requestkey = $cdom.'_'.$cnum;
-        if ($requestkey =~ /^($match_domain)_($match_courseid)$/) {
-            my %history = &Apache::lonnet::restore($requestkey,'courserequests',
-                                                   $env{'user.domain'},$env{'user.name'});
-            next if ((exists($history{'status'})) && ($history{'status'} eq 'created'));
-            my $reqtime = $history{'reqtime'};
-            my $lastupdate = $history{'timestamp'};
-            my $showtype = $history{'crstype'};
-            if (defined($typenames->{$history{'crstype'}})) {
-                $showtype = $typenames->{$history{'crstype'}};
-            }
-            my $description;
-            if (ref($history{'details'}) eq 'HASH') {
-                $description = $history{details}{'cdescr'};
-            }
-            @{$reqcrs{$reqtime}} = ($description,$showtype);
-        }
-    }
-    my @sortedtimes = sort {$a <=> $b} (keys(%reqcrs));
-    if (@sortedtimes > 0) {
-        $output .= '<p><b>'.&mt('Course/Community requests').'</b><br />'.
-                   &Apache::loncommon::start_data_table().
-                   &Apache::loncommon::start_data_table_header_row().
-                   '<th>'.&mt('Date requested').'</th>'.
-                   '<th>'.&mt('Course title').'</th>'.
-                   '<th>'.&mt('Course type').'</th>';
-                   &Apache::loncommon::end_data_table_header_row();
-        foreach my $reqtime (@sortedtimes) {
-            next unless (ref($reqcrs{$reqtime}) eq 'ARRAY');
-            $output .= &Apache::loncommon::start_data_table_row().
-                       '<td>'.&Apache::lonlocal::locallocaltime($reqtime).'</td>'.
-                       '<td>'.join('</td><td>',@{$reqcrs{$reqtime}}).'</td>'.
-                       &Apache::loncommon::end_data_table_row();
-        }
-        $output .= &Apache::loncommon::end_data_table().
-                   '<br /></p>';
-    }
-    my $queuedselfenroll = &Apache::loncoursequeueadmin::queued_selfenrollment(1);
-    if ($queuedselfenroll) {
-        $output .= '<p><b>'.&mt('Enrollment requests').'</b><br />'.
-                   $queuedselfenroll.'<br /></p>';
-    }
-    if ($env{'environment.canrequest.author'}) {
-        unless (&Apache::loncoursequeueadmin::is_active_author()) {
-            my $requestauthor;
-            my ($status,$timestamp) = split(/:/,$env{'environment.requestauthorqueued'});
-            if (($status eq 'approval') || ($status eq 'approved')) {
-                $output .= '<p><b>'.&mt('Author role request').'</b><br />';
-                if ($status eq 'approval') {
-                    $output .= &mt('A request for authoring space submitted on [_1] is awaiting approval',
-                                  &Apache::lonlocal::locallocaltime($timestamp));
-                } elsif ($status eq 'approved') {
-                    my %roleshash =
-                        &Apache::lonnet::get_my_roles($env{'user.name'},$env{'user.domain'},'userroles',
-                                                      ['active'],['au'],[$env{'user.domain'}]);
-                    if (keys(%roleshash)) {
-                        $output .= '<span class="LC_info">'.
-                                   &mt('Your request for an author role has been approved.').'<br />'.
-                                   &mt('Use the "Check for changes" link to update your list of roles.').
-                                   '</span>';
-                    }
-                }
-                $output .= '</p>';
-            }
-        }
-    }
-    unless ($output) {
-        if ($env{'environment.canrequest.author'} || $env{'environment.canrequest.official'} ||
-            $env{'environment.canrequest.unofficial'} || $env{'environment.canrequest.community'}) {
-            $output = &mt('No requests for courses, communities or authoring currently queued');
-        } else {
-            $output = &mt('No enrollment requests currently queued awaiting approval');
-        }
-    }
-    return '<div class="LC_left_float"><fieldset><legend>'.&mt('Queued requests').'</legend>'.
-           $output.'</fieldset></div><br clear="all" />';
-}
-
 1;
 __END__