--- loncom/interface/lonmsg.pm	2005/03/01 03:28:14	1.139
+++ loncom/interface/lonmsg.pm	2005/04/07 06:56:23	1.140
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Routines for messaging
 #
-# $Id: lonmsg.pm,v 1.139 2005/03/01 03:28:14 albertel Exp $
+# $Id: lonmsg.pm,v 1.140 2005/04/07 06:56:23 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -104,7 +104,7 @@ since lonmsg already implements that fun
 =cut
 
 use strict;
-use Apache::lonnet();
+use Apache::lonnet;
 use vars qw($msgcount);
 use HTML::TokeParser();
 use Apache::Constants qw(:common);
@@ -140,24 +140,24 @@ sub packagemsg {
     my $partsubj=$subject;
     $partsubj=&Apache::lonnet::escape($partsubj);
     my $msgid=&Apache::lonnet::escape(
-           $now.':'.$partsubj.':'.$ENV{'user.name'}.':'.
-           $ENV{'user.domain'}.':'.$msgcount.':'.$$);
-    my $result='<sendername>'.$ENV{'user.name'}.'</sendername>'.
-           '<senderdomain>'.$ENV{'user.domain'}.'</senderdomain>'.
+           $now.':'.$partsubj.':'.$env{'user.name'}.':'.
+           $env{'user.domain'}.':'.$msgcount.':'.$$);
+    my $result='<sendername>'.$env{'user.name'}.'</sendername>'.
+           '<senderdomain>'.$env{'user.domain'}.'</senderdomain>'.
            '<subject>'.$subject.'</subject>'.
 	   '<time>'.&Apache::lonlocal::locallocaltime($now).'</time>'.
 	   '<servername>'.$ENV{'SERVER_NAME'}.'</servername>'.
            '<host>'.$ENV{'HTTP_HOST'}.'</host>'.
 	   '<client>'.$ENV{'REMOTE_ADDR'}.'</client>'.
-	   '<browsertype>'.$ENV{'browser.type'}.'</browsertype>'.
-	   '<browseros>'.$ENV{'browser.os'}.'</browseros>'.
-	   '<browserversion>'.$ENV{'browser.version'}.'</browserversion>'.
-           '<browsermathml>'.$ENV{'browser.mathml'}.'</browsermathml>'.
+	   '<browsertype>'.$env{'browser.type'}.'</browsertype>'.
+	   '<browseros>'.$env{'browser.os'}.'</browseros>'.
+	   '<browserversion>'.$env{'browser.version'}.'</browserversion>'.
+           '<browsermathml>'.$env{'browser.mathml'}.'</browsermathml>'.
 	   '<browserraw>'.$ENV{'HTTP_USER_AGENT'}.'</browserraw>'.
-	   '<courseid>'.$ENV{'request.course.id'}.'</courseid>'.
-	   '<coursesec>'.$ENV{'request.course.sec'}.'</coursesec>'.
-	   '<role>'.$ENV{'request.role'}.'</role>'.
-	   '<resource>'.$ENV{'request.filename'}.'</resource>'.
+	   '<courseid>'.$env{'request.course.id'}.'</courseid>'.
+	   '<coursesec>'.$env{'request.course.sec'}.'</coursesec>'.
+	   '<role>'.$env{'request.role'}.'</role>'.
+	   '<resource>'.$env{'request.filename'}.'</resource>'.
            '<msgid>'.$msgid.'</msgid>'.
 	   '<recuser>'.$recuser.'</recuser>'.
 	   '<recdomain>'.$recdomain.'</recdomain>'.
@@ -226,23 +226,19 @@ sub sendemail {
     my $msg = new Mail::Send;
     $msg->to($to);
     $msg->subject('[LON-CAPA] '.$subject);
-    my %oldENV=%ENV;
-    undef(%ENV);
     if (my $fh = $msg->open()) {
 	print $fh $body;
 	$fh->close;
     }
-    %ENV=%oldENV;
-    undef(%oldENV);
 }
 
 # ==================================================== Send notification emails
 
 sub sendnotification {
     my ($to,$touname,$toudom,$subj,$crit,$text)=@_;
-    my $sender=$ENV{'environment.firstname'}.' '.$ENV{'environment.lastname'};
+    my $sender=$env{'environment.firstname'}.' '.$env{'environment.lastname'};
     unless ($sender=~/\w/) { 
-	$sender=$ENV{'user.name'}.'@'.$ENV{'user.domain'};
+	$sender=$env{'user.name'}.'@'.$env{'user.domain'};
     }
     my $critical=($crit?' critical':'');
     $text=~s/\&lt\;/\</gs;
@@ -271,7 +267,7 @@ ENDMSG
 # ============================================================= Check for email
 
 sub newmail {
-    if ((time-$ENV{'user.mailcheck.time'})>300) {
+    if ((time-$env{'user.mailcheck.time'})>300) {
         my %what=&Apache::lonnet::get('email_status',['recnewemail']);
         &Apache::lonnet::appenv('user.mailcheck.time'=>time);
         if ($what{'recnewemail'}>0) { return 1; }
@@ -370,10 +366,10 @@ sub user_crit_msg_raw {
            'put:'.$domain.':'.$user.':critical:'.
            &Apache::lonnet::escape($msgid).'='.
            &Apache::lonnet::escape($message),$homeserver);
-       if ($ENV{'request.course.id'}) {
+       if ($env{'request.course.id'}) {
           &user_normal_msg_raw(
-            $ENV{'course.'.$ENV{'request.course.id'}.'.num'},
-            $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
+            $env{'course.'.$env{'request.course.id'}.'.num'},
+            $env{'course.'.$env{'request.course.id'}.'.domain'},
             'Critical ['.$user.':'.$domain.']',
 	    $message);
        }
@@ -396,8 +392,8 @@ sub user_crit_msg_raw {
     &Apache::lonnet::logthis(
       'Sending critical email '.$msgid.
       ', log status: '.
-      &Apache::lonnet::log($ENV{'user.domain'},$ENV{'user.name'},
-                         $ENV{'user.home'},
+      &Apache::lonnet::log($env{'user.domain'},$env{'user.name'},
+                         $env{'user.home'},
       'Sending critical '.$msgid.' to '.$user.' at '.$domain.' with status: '
       .$status));
     return $status;
@@ -440,8 +436,8 @@ sub user_crit_received {
     my %contents=&unpackagemsg($message{$msgid},1);
     my $status='rec: '.($contents{'sendback'}?
      &user_normal_msg($contents{'sendername'},$contents{'senderdomain'},
-                     &mt('Receipt').': '.$ENV{'user.name'}.' '.&mt('at').' '.$ENV{'user.domain'}.', '.$contents{'subject'},
-                     &mt('User').' '.$ENV{'user.name'}.' '.&mt('at').' '.$ENV{'user.domain'}.
+                     &mt('Receipt').': '.$env{'user.name'}.' '.&mt('at').' '.$env{'user.domain'}.', '.$contents{'subject'},
+                     &mt('User').' '.$env{'user.name'}.' '.&mt('at').' '.$env{'user.domain'}.
                      ' acknowledged receipt of message'."\n".'   "'.
                      $contents{'subject'}.'"'."\n".&mt('dated').' '.
                      $contents{'time'}.".\n"
@@ -451,8 +447,8 @@ sub user_crit_received {
      'nohist_email',{$contents{'msgid'} => $message{$msgid}});
     $status.=' del: '.
      &Apache::lonnet::del('critical',[$contents{'msgid'}]);
-    &Apache::lonnet::log($ENV{'user.domain'},$ENV{'user.name'},
-                         $ENV{'user.home'},'Received critical message '.
+    &Apache::lonnet::log($env{'user.domain'},$env{'user.name'},
+                         $env{'user.home'},'Received critical message '.
                          $contents{'msgid'}.
                          ', '.$status);
     return $status;
@@ -482,10 +478,10 @@ sub user_normal_msg_raw {
                          ('email_status',{'recnewemail'=>time},$domain,$user);
 # Into sent-mail folder
        $status.=' '.&Apache::lonnet::critical(
-           'put:'.$ENV{'user.domain'}.':'.$ENV{'user.name'}.
+           'put:'.$env{'user.domain'}.':'.$env{'user.name'}.
 					      ':nohist_email_sent:'.
            &Apache::lonnet::escape($msgid).'='.
-           &Apache::lonnet::escape($message),$ENV{'user.home'});
+           &Apache::lonnet::escape($message),$env{'user.home'});
     } else {
        $status='no_host';
     }
@@ -501,8 +497,8 @@ sub user_normal_msg_raw {
       &sendnotification($userenv{'permanentemail'},$user,$domain,$subject,0,
 			$text);
     }
-    &Apache::lonnet::log($ENV{'user.domain'},$ENV{'user.name'},
-                         $ENV{'user.home'},
+    &Apache::lonnet::log($env{'user.domain'},$env{'user.name'},
+                         $env{'user.home'},
       'Sending '.$msgid.' to '.$user.' at '.$domain.' with status: '.$status);
     return $status;
 }
@@ -559,7 +555,7 @@ sub folderlist {
 	 ($_==$interdis?' selected="selected"':'').'>'.$_.'</option>' }
 				   (10,20,50,100,200)).'</select>'.	
    '<input type="submit" value="'.&mt('View Folder').'" /><br />'.
-    '<input type="hidden" name="sortedby" value="'.$ENV{'form.sortedby'}.'" />'.
+    '<input type="hidden" name="sortedby" value="'.$env{'form.sortedby'}.'" />'.
 			      ($folder=~/^(new|critical)/?'</form>':'');
 }
 
@@ -781,34 +777,34 @@ sub sortedmessages {
     }
     #default sort
     @temp = sort  {$a->[0] <=> $b->[0]} @temp;    
-    if ($ENV{'form.sortedby'} eq "date"){
+    if ($env{'form.sortedby'} eq "date"){
         @temp = sort  {$a->[0] <=> $b->[0]} @temp;    
     }
-    if ($ENV{'form.sortedby'} eq "revdate"){
+    if ($env{'form.sortedby'} eq "revdate"){
     	@temp = sort  {$b->[0] <=> $a->[0]} @temp; 
     }
-    if ($ENV{'form.sortedby'} eq "user"){
+    if ($env{'form.sortedby'} eq "user"){
 	@temp = sort  {lc($a->[2]) cmp lc($b->[2])} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "revuser"){
+    if ($env{'form.sortedby'} eq "revuser"){
 	@temp = sort  {lc($b->[2]) cmp lc($a->[2])} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "domain"){
+    if ($env{'form.sortedby'} eq "domain"){
         @temp = sort  {$a->[3] cmp $b->[3]} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "revdomain"){
+    if ($env{'form.sortedby'} eq "revdomain"){
         @temp = sort  {$b->[3] cmp $a->[3]} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "subject"){
+    if ($env{'form.sortedby'} eq "subject"){
         @temp = sort  {lc($a->[1]) cmp lc($b->[1])} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "revsubject"){
+    if ($env{'form.sortedby'} eq "revsubject"){
         @temp = sort  {lc($b->[1]) cmp lc($a->[1])} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "status"){
+    if ($env{'form.sortedby'} eq "status"){
         @temp = sort  {$a->[4] cmp $b->[4]} @temp;
     }
-    if ($ENV{'form.sortedby'} eq "revstatus"){
+    if ($env{'form.sortedby'} eq "revstatus"){
         @temp = sort  {$b->[4] cmp $a->[4]} @temp;
     }
     return @temp;
@@ -828,9 +824,9 @@ sub disnew {
 				       'do' => 'Domain'
 				       );
     my @msgids = sort split(/\&/,&Apache::lonnet::reply
-                            ('keys:'.$ENV{'user.domain'}.':'.
-                             $ENV{'user.name'}.':nohist_email',
-                             $ENV{'user.home'}));
+                            ('keys:'.$env{'user.domain'}.':'.
+                             $env{'user.name'}.':nohist_email',
+                             $env{'user.home'}));
     my @newmsgs;
     my %setters = ();
     my $startblock = 0;
@@ -962,31 +958,31 @@ ENDDISHEADER
     $r->print(&scrollbuttons($startdis,$number,$firstdis,$lastdis,$totalnumber));
     $r->print('<form method="post" name="disall" action="/adm/email">'.
 	      '<table border=2><tr><th colspan="3">&nbsp</th><th>');
-    if ($ENV{'form.sortedby'} eq "revdate") {
+    if ($env{'form.sortedby'} eq "revdate") {
 	$r->print('<a href = "?sortedby=date'.$fsqs.'">'.&mt('Date').'</a></th>');
     } else {
 	$r->print('<a href = "?sortedby=revdate'.$fsqs.'">'.&mt('Date').'</a></th>');
     }
     $r->print('<th>');
-    if ($ENV{'form.sortedby'} eq "revuser") {
+    if ($env{'form.sortedby'} eq "revuser") {
 	$r->print('<a href = "?sortedby=user'.$fsqs.'">'.&mt('Username').'</a>');
     } else {
 	$r->print('<a href = "?sortedby=revuser'.$fsqs.'">'.&mt('Username').'</a>');
     }
     $r->print('</th><th>');
-    if ($ENV{'form.sortedby'} eq "revdomain") {
+    if ($env{'form.sortedby'} eq "revdomain") {
 	$r->print('<a href = "?sortedby=domain'.$fsqs.'">'.&mt('Domain').'</a>');
     } else {
 	$r->print('<a href = "?sortedby=revdomain'.$fsqs.'">'.&mt('Domain').'</a>');
     }
     $r->print('</th><th>');
-    if ($ENV{'form.sortedby'} eq "revsubject") {
+    if ($env{'form.sortedby'} eq "revsubject") {
 	$r->print('<a href = "?sortedby=subject'.$fsqs.'">'.&mt('Subject').'</a>');
     } else {
     	$r->print('<a href = "?sortedby=revsubject'.$fsqs.'">'.&mt('Subject').'</a>');
     }
     $r->print('</th><th>');
-    if ($ENV{'form.sortedby'} eq "revstatus") {
+    if ($env{'form.sortedby'} eq "revstatus") {
 	$r->print('<a href = "?sortedby=status'.$fsqs.'">'.&mt('Status').'</a></th>');
     } else {
      	$r->print('<a href = "?sortedby=revstatus'.$fsqs.'">'.&mt('Status').'</a></th>');
@@ -1020,7 +1016,7 @@ ENDDISHEADER
     $r->print("</table>\n<p>".
   '<a href="javascript:checkall()">'.&mt('Check All').'</a>&nbsp;'.
   '<a href="javascript:uncheckall()">'.&mt('Uncheck All').'</a></p>'.
-  '<input type="hidden" name="sortedby" value="'.$ENV{'form.sortedby'}.'" />');
+  '<input type="hidden" name="sortedby" value="'.$env{'form.sortedby'}.'" />');
     if ($folder ne 'trash') {
 	$r->print(
 	      '<p><input type="submit" name="markeddel" value="'.&mt('Delete Checked').'" /></p>');
@@ -1033,7 +1029,7 @@ ENDDISHEADER
 			     ( map { $_ => $_ } @allfolders))
 	      );
     my $postedstartdis=$startdis+1;
-    $r->print('<input type="hidden" name="folder" value="'.$folder.'" /><input type="hidden" name="startdis" value="'.$postedstartdis.'" /><input type="hidden" name="interdis" value="'.$ENV{'form.interdis'}.'" /></form>');
+    $r->print('<input type="hidden" name="folder" value="'.$folder.'" /><input type="hidden" name="startdis" value="'.$postedstartdis.'" /><input type="hidden" name="interdis" value="'.$env{'form.interdis'}.'" /></form>');
     if ($numblocked > 0) {
         my $beginblock = &Apache::lonlocal::locallocaltime($startblock);
         my $finishblock = &Apache::lonlocal::locallocaltime($endblock);
@@ -1087,7 +1083,7 @@ sub compout {
 				       'ca' => 'Cancel',
 				       'ma' => 'Mail');
 
-    if (&Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) {
+    if (&Apache::lonnet::allowed('srm',$env{'request.course.id'})) {
 	 my $crithelp = Apache::loncommon::help_open_topic("Course_Critical_Message");
          $dispcrit=
  '<p><label><input type="checkbox" name="critmsg" /> '.&mt('Send as critical message').'</label> ' . $crithelp . 
@@ -1099,7 +1095,7 @@ sub compout {
      }
     my %message;
     my %content;
-    my $defdom=$ENV{'user.domain'};
+    my $defdom=$env{'user.domain'};
     if ($forwarding) {
 	%message=&Apache::lonnet::get('nohist_email'.$suffix,[$forwarding]);
 	%content=&unpackagemsg($message{$forwarding},$folder);
@@ -1128,7 +1124,7 @@ sub compout {
 	$dismsg=~s/\n+/\n\> /g;
 	if ($content{'baseurl'}) {
 	    $disbase='<input type="hidden" name="baseurl" value="'.&Apache::lonnet::escape($content{'baseurl'}).'" />';
-	    if ($ENV{'user.adv'}) {
+	    if ($env{'user.adv'}) {
 		$disbase.='<label><input type="checkbox" name="storebasecomment" />'.&mt('Store message for re-use').
 		    '</label> <a href="/adm/email?showcommentbaseurl='.
 		    &Apache::lonnet::escape($content{'baseurl'}).'" target="comments">'.
@@ -1137,7 +1133,7 @@ sub compout {
 	}
     }
     my $citation=&displayresource(%content);
-    if ($ENV{'form.recdom'}) { $defdom=$ENV{'form.recdom'}; }
+    if ($env{'form.recdom'}) { $defdom=$env{'form.recdom'}; }
       $r->print(
                 '<form action="/adm/email"  name="compemail" method="post"'.
                 ' enctype="multipart/form-data">'."\n".
@@ -1158,7 +1154,7 @@ sub compout {
 	    my $selectlink=&Apache::loncommon::selectstudent_link
 	    ('compemail','recuname','recdomain');
 	    $r->print(<<"ENDREC");
-<tr><td>$lt{'us'}:</td><td><input type="text" size="12" name="recuname" value="$ENV{'form.recname'}" /></td><td rowspan="2">$selectlink</td></tr>
+<tr><td>$lt{'us'}:</td><td><input type="text" size="12" name="recuname" value="$env{'form.recname'}" /></td><td rowspan="2">$selectlink</td></tr>
 <tr><td>$lt{'do'}:</td>
 <td>$domform</td></tr>
 ENDREC
@@ -1219,14 +1215,14 @@ ENDUPLOAD
 
 sub retrieve_instructor_comments {
     my ($user,$domain)=@_;
-    my $target=$ENV{'form.grade_target'};
-    if (! $ENV{'request.course.id'}) { return; }
-    if (! &Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) {
+    my $target=$env{'form.grade_target'};
+    if (! $env{'request.course.id'}) { return; }
+    if (! &Apache::lonnet::allowed('srm',$env{'request.course.id'})) {
 	return;
     }
     my %records=&Apache::lonnet::dump('nohist_email',
-			 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-			 $ENV{'course.'.$ENV{'request.course.id'}.'.num'},
+			 $env{'course.'.$env{'request.course.id'}.'.domain'},
+			 $env{'course.'.$env{'request.course.id'}.'.num'},
                          '%255b'.$user.'%253a'.$domain.'%255d');
     my $result='';
     foreach (sort(keys(%records))) {
@@ -1244,14 +1240,14 @@ sub retrieve_instructor_comments {
 
 sub disfacetoface {
     my ($r,$user,$domain)=@_;
-    my $target=$ENV{'form.grade_target'};
-    unless ($ENV{'request.course.id'}) { return; }
-    unless (&Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) {
+    my $target=$env{'form.grade_target'};
+    unless ($env{'request.course.id'}) { return; }
+    unless (&Apache::lonnet::allowed('srm',$env{'request.course.id'})) {
 	return;
     }
     my %records=&Apache::lonnet::dump('nohist_email',
-			 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-			 $ENV{'course.'.$ENV{'request.course.id'}.'.num'},
+			 $env{'course.'.$env{'request.course.id'}.'.domain'},
+			 $env{'course.'.$env{'request.course.id'}.'.num'},
                          '%255b'.$user.'%253a'.$domain.'%255d');
     my $result='';
     foreach (sort keys %records) {
@@ -1294,7 +1290,7 @@ $content{'sendername'}.'@'.
 
 sub facetoface {
     my ($r,$stage)=@_;
-    unless (&Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) {
+    unless (&Apache::lonnet::allowed('srm',$env{'request.course.id'})) {
 	return;
     }
     &printheader($r,
@@ -1302,12 +1298,12 @@ sub facetoface {
 		 "User Notes, Face-to-Face, Critical Messages, Broadcast Messages");
 # from query string
 
-    if ($ENV{'form.recname'}) { $ENV{'form.recuname'}=$ENV{'form.recname'}; }
-    if ($ENV{'form.recdom'}) { $ENV{'form.recdomain'}=$ENV{'form.recdom'}; }
+    if ($env{'form.recname'}) { $env{'form.recuname'}=$env{'form.recname'}; }
+    if ($env{'form.recdom'}) { $env{'form.recdomain'}=$env{'form.recdom'}; }
 
-    my $defdom=$ENV{'user.domain'};
+    my $defdom=$env{'user.domain'};
 # already filled in
-    if ($ENV{'form.recdomain'}) { $defdom=$ENV{'form.recdomain'}; }
+    if ($env{'form.recdomain'}) { $defdom=$env{'form.recdomain'}; }
 # generate output
     my $domform = &Apache::loncommon::select_dom_form($defdom,'recdomain');
     my $stdbrws = &Apache::loncommon::selectstudent_link
@@ -1323,7 +1319,7 @@ sub facetoface {
 <form method="post" action="/adm/email" name="stdselect">
 <input type="hidden" name="recordftf" value="retrieve" />
 <table>
-<tr><td>$lt{'user'}:</td><td><input type="text" size="12" name="recuname" value="$ENV{'form.recuname'}" /></td>
+<tr><td>$lt{'user'}:</td><td><input type="text" size="12" name="recuname" value="$env{'form.recuname'}" /></td>
 <td rowspan="2">
 $stdbrws
 <input type="submit" value="$lt{'subm'}" /></td>
@@ -1334,23 +1330,23 @@ $stdbrws
 </form>
 ENDTREC
     if (($stage ne 'query') &&
-        ($ENV{'form.recdomain'}) && ($ENV{'form.recuname'})) {
-        chomp($ENV{'form.newrecord'});
-        if ($ENV{'form.newrecord'}) {
+        ($env{'form.recdomain'}) && ($env{'form.recuname'})) {
+        chomp($env{'form.newrecord'});
+        if ($env{'form.newrecord'}) {
            &user_normal_msg_raw(
-            $ENV{'course.'.$ENV{'request.course.id'}.'.num'},
-            $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
+            $env{'course.'.$env{'request.course.id'}.'.num'},
+            $env{'course.'.$env{'request.course.id'}.'.domain'},
             &mt('Record').
-	     ' ['.$ENV{'form.recuname'}.':'.$ENV{'form.recdomain'}.']',
-	    $ENV{'form.newrecord'});
+	     ' ['.$env{'form.recuname'}.':'.$env{'form.recdomain'}.']',
+	    $env{'form.newrecord'});
         }
-        $r->print('<h3>'.&Apache::loncommon::plainname($ENV{'form.recuname'},
-				     $ENV{'form.recdomain'}).'</h3>');
-        &disfacetoface($r,$ENV{'form.recuname'},$ENV{'form.recdomain'});
+        $r->print('<h3>'.&Apache::loncommon::plainname($env{'form.recuname'},
+				     $env{'form.recdomain'}).'</h3>');
+        &disfacetoface($r,$env{'form.recuname'},$env{'form.recdomain'});
 	$r->print(<<ENDRHEAD);
 <form method="post" action="/adm/email">
-<input name="recdomain" value="$ENV{'form.recdomain'}" type="hidden" />
-<input name="recuname" value="$ENV{'form.recuname'}" type="hidden" />
+<input name="recdomain" value="$env{'form.recdomain'}" type="hidden" />
+<input name="recuname" value="$env{'form.recuname'}" type="hidden" />
 ENDRHEAD
         $r->print(<<ENDBFORM);
 <hr />$lt{'newr'}<br />
@@ -1367,8 +1363,8 @@ ENDBFORM
 
 sub examblock {
     my ($r,$action) = @_;
-    unless ($ENV{'request.course.id'}) { return;}
-    unless (&Apache::lonnet::allowed('srm',$ENV{'request.course.id'})) { $r->print('Not allowed'); }
+    unless ($env{'request.course.id'}) { return;}
+    unless (&Apache::lonnet::allowed('srm',$env{'request.course.id'})) { $r->print('Not allowed'); }
     my %lt=&Apache::lonlocal::texthash(
             'comb' => 'Communication Blocking',
             'cbds' => 'Communication blocking during scheduled exams',
@@ -1438,7 +1434,7 @@ sub blockstore {
     my $addtotal = 0;
     my %blocking = ();
     $r->print('<h3>'.$lt{'head'}.'</h3>');
-    foreach (keys %ENV) {
+    foreach (keys %env) {
         if ($_ =~ m/^form\.modify_(\w+)$/) {
             $adds{$1} = $1;
             $removals{$1} = $1;
@@ -1456,23 +1452,23 @@ sub blockstore {
     }
 
     foreach (keys %removals) {
-        my $hashkey = $ENV{'form.key_'.$_};
+        my $hashkey = $env{'form.key_'.$_};
         &Apache::lonnet::del('comm_block',["$hashkey"],
-                         $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-                         $ENV{'course.'.$ENV{'request.course.id'}.'.num'}
+                         $env{'course.'.$env{'request.course.id'}.'.domain'},
+                         $env{'course.'.$env{'request.course.id'}.'.num'}
                          );
     }
     foreach (keys %adds) {
         unless ( defined($cancels{$_}) ) {
             my ($newstart,$newend) = &get_dates_from_form($_);
             my $newkey = $newstart.'____'.$newend;
-            $blocking{$newkey} = $ENV{'user.name'}.'@'.$ENV{'user.domain'}.':'.$ENV{'form.title_'.$_};
+            $blocking{$newkey} = $env{'user.name'}.'@'.$env{'user.domain'}.':'.$env{'form.title_'.$_};
         }
     }
     if ($addtotal + $modtotal > 0) {
         &Apache::lonnet::put('comm_block',\%blocking,
-                     $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-                     $ENV{'course.'.$ENV{'request.course.id'}.'.num'}
+                     $env{'course.'.$env{'request.course.id'}.'.domain'},
+                     $env{'course.'.$env{'request.course.id'}.'.num'}
                      );
     }
     my $chgestotal = $canceltotal + $modtotal + $addtotal;
@@ -1506,8 +1502,8 @@ sub get_blockdates {
     my ($records,$blockcount) = @_;
     $$blockcount = 0;
     %{$records} = &Apache::lonnet::dump('comm_block',
-                         $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-                         $ENV{'course.'.$ENV{'request.course.id'}.'.num'}
+                         $env{'course.'.$env{'request.course.id'}.'.domain'},
+                         $env{'course.'.$env{'request.course.id'}.'.num'}
                          );
     $$blockcount = keys %{$records};
                                                                                                              
@@ -1525,7 +1521,7 @@ sub display_blocker_status {
     my @bgcols = ("#eeeeee","#dddddd");
     my $function = &Apache::loncommon::get_users_function();
     my $color = &Apache::loncommon::designparm($function.'.tabbg',
-                                                    $ENV{'user.domain'});
+                                                    $env{'user.domain'});
     my %lt = &Apache::lonlocal::texthash(
         'modi' => 'Modify',
         'canc' => 'Cancel',
@@ -1591,7 +1587,7 @@ sub display_addblocker_table {
     my $endform = &Apache::lonhtmlcommon::date_setter('blockform','enddate_'.$parmcount,$end,$onchange);
     my $function = &Apache::loncommon::get_users_function();
     my $color = &Apache::loncommon::designparm($function.'.tabbg',
-                                                    $ENV{'user.domain'});
+                                                    $env{'user.domain'});
     my %lt = &Apache::lonlocal::texthash(
         'addb' => 'Add block',
         'exam' => 'e.g., Exam 1',
@@ -1634,12 +1630,12 @@ sub blockcheck {
     my @staffcses = ();
     $$startblock = 0;
     $$endblock = 0;
-    foreach (keys %ENV) {
+    foreach (keys %env) {
         if ($_ =~ m-^user\.role\.(st|cc|in)\./(.+)$-) {
             my $role = $1;
             my $cse = $2;
             $cse =~ s|/|_|;
-            if ($ENV{$_} =~ m/^(\d*)\.(\d*)$/) {
+            if ($env{$_} =~ m/^(\d*)\.(\d*)$/) {
                 unless (($2 > 0 && $2 < time) || ($1 > time)) {
                     if ($role eq 'st') {
                         push @livecses, $cse;
@@ -1651,14 +1647,14 @@ sub blockcheck {
                 }
             }
         } elsif ($_ =~ m-user\.role\.cr/(\w+)/(\w+)/([^/]+)\./(.+)$- ) { 
-            my $rolepriv = $ENV{'user.role..rolesdef_'.$3};
+            my $rolepriv = $env{'user.role..rolesdef_'.$3};
         }
     }
     # Retrieve blocking times and identity of blocker for active courses for students.
     if (@livecses > 0) {
         foreach my $cse (@livecses) {
             my ($cdom,$crs) = split/_/,$cse;
-            if ( (grep/^$cse$/,@staffcses) && ($ENV{'request.role'} !~ m-^st\./$cdom/$crs$-) ) {
+            if ( (grep/^$cse$/,@staffcses) && ($env{'request.role'} !~ m-^st\./$cdom/$crs$-) ) {
                 next;
             } else {
                 %{$$setters{$cse}} = ();
@@ -1689,7 +1685,7 @@ sub build_block_table {
     my ($r,$startblock,$endblock,$setters) = @_;
     my $function = &Apache::loncommon::get_users_function();
     my $color = &Apache::loncommon::designparm($function.'.tabbg',
-                                                    $ENV{'user.domain'});
+                                                    $env{'user.domain'});
     my %lt = &Apache::lonlocal::texthash(
         'cacb' => 'Currently active communication blocks',
         'cour' => 'Course',
@@ -1776,7 +1772,7 @@ sub displaymessage {
 	      '<td><a href="/adm/email?markdel='.&Apache::lonnet::escape($msgid).$sqs.
 	      '"><b>Delete</b></a></td>'.
 	      '<td><a href="/adm/email?'.$sqs.
-	      ($ENV{'form.dismode'} eq 'new'?'&folder=new':'').
+	      ($env{'form.dismode'} eq 'new'?'&folder=new':'').
 	      '"><b>'.&mt('Back to Folder Display').'</b></a></td>');
     if ($counter > 0){
 	$r->print('<td><a href="/adm/email?display='.$messages[$counter-1]->[5].$sqs.
@@ -1818,7 +1814,7 @@ sub displayresource {
 # If the recipient is in the same course that the message was sent from and
 # has sufficient privileges, show "all details," else show citation
 #
-    if (($ENV{'request.course.id'} eq $content{'courseid'})
+    if (($env{'request.course.id'} eq $content{'courseid'})
      && (&Apache::lonnet::allowed('vgr',$content{'courseid'}))) {
 	my $symb=&Apache::lonnet::symbread($content{'baseurl'});
 # Could not get a symb, give up
@@ -1875,21 +1871,21 @@ sub printheader {
 
 sub storecomment {
     my ($r)=@_;
-    my $msgtxt=&Apache::lonfeedback::clear_out_html($ENV{'form.message'});
+    my $msgtxt=&Apache::lonfeedback::clear_out_html($env{'form.message'});
     my $cleanmsgtxt='';
     foreach (split(/[\n\r]/,$msgtxt)) {
 	unless ($_=~/^\s*(\>|\&gt\;)/) {
 	    $cleanmsgtxt.=$_."\n";
 	}
     }
-    my $key=&Apache::lonnet::escape($ENV{'form.baseurl'}).'___'.time;
+    my $key=&Apache::lonnet::escape($env{'form.baseurl'}).'___'.time;
     &Apache::lonnet::put('nohist_stored_comments',{ $key => $cleanmsgtxt });
 }
 
 sub storedcommentlisting {
     my ($r)=@_;
     my %msgs=&Apache::lonnet::dump('nohist_stored_comments',undef,undef,
-       '^'.&Apache::lonnet::escape(&Apache::lonnet::escape($ENV{'form.showcommentbaseurl'})));
+       '^'.&Apache::lonnet::escape(&Apache::lonnet::escape($env{'form.showcommentbaseurl'})));
     $r->print(&Apache::lonxml::xmlbegin().'<head>'.
 	      &Apache::lonxml::fontsettings().'</head><body>');
     if ((keys %msgs)[0]=~/^error\:/) {
@@ -1912,35 +1908,35 @@ sub sendoffmail {
     my ($r,$folder)=@_;
     my $suffix=&foldersuffix($folder);
     my $sendstatus='';
-    if ($ENV{'form.send'}) {
+    if ($env{'form.send'}) {
 	&printheader($r,'','Messages being sent.');
 	$r->rflush();
 	my %content=();
 	undef %content;
-	if ($ENV{'form.forwid'}) {
-	    my $msgid=$ENV{'form.forwid'};
+	if ($env{'form.forwid'}) {
+	    my $msgid=$env{'form.forwid'};
 	    my %message=&Apache::lonnet::get('nohist_email'.$suffix,[$msgid]);
 	    %content=&unpackagemsg($message{$msgid},1);
 	    &statuschange($msgid,'forwarded',$folder);
-	    $ENV{'form.message'}.="\n\n-- Forwarded message --\n\n".
+	    $env{'form.message'}.="\n\n-- Forwarded message --\n\n".
 		$content{'message'};
 	}
-	if ($ENV{'form.replyid'}) {
-	    my $msgid=$ENV{'form.replyid'};
+	if ($env{'form.replyid'}) {
+	    my $msgid=$env{'form.replyid'};
 	    my %message=&Apache::lonnet::get('nohist_email'.$suffix,[$msgid]);
 	    %content=&unpackagemsg($message{$msgid},1);
 	    &statuschange($msgid,'replied',$folder);
 	}
 	my %toaddr=();
 	undef %toaddr;
-	if ($ENV{'form.sendmode'} eq 'group') {
-	    foreach (keys %ENV) {
+	if ($env{'form.sendmode'} eq 'group') {
+	    foreach (keys %env) {
 		if ($_=~/^form\.send\_to\_\&\&\&[^\&]*\&\&\&\_(.+)$/) {
 		    $toaddr{$1}='';
 		}
 	    }
-	} elsif ($ENV{'form.sendmode'} eq 'upload') {
-	    foreach (split(/[\n\r\f]+/,$ENV{'form.upfile'})) {
+	} elsif ($env{'form.sendmode'} eq 'upload') {
+	    foreach (split(/[\n\r\f]+/,$env{'form.upfile'})) {
 		my ($rec,$txt)=split(/\s*\:\s*/,$_);
 		if ($txt) {
 		    $rec=~s/\@/\:/;
@@ -1948,10 +1944,10 @@ sub sendoffmail {
 		}
 	    }
 	} else {
-	    $toaddr{$ENV{'form.recuname'}.':'.$ENV{'form.recdomain'}}='';
+	    $toaddr{$env{'form.recuname'}.':'.$env{'form.recdomain'}}='';
 	}
-	if ($ENV{'form.additionalrec'}) {
-	    foreach (split(/\,/,$ENV{'form.additionalrec'})) {
+	if ($env{'form.additionalrec'}) {
+	    foreach (split(/\,/,$env{'form.additionalrec'})) {
 		my ($auname,$audom)=split(/\@/,$_);
 		$toaddr{$auname.':'.$audom}='';
 	    }
@@ -1960,31 +1956,31 @@ sub sendoffmail {
 	foreach (keys %toaddr) {
 	    my ($recuname,$recdomain)=split(/\:/,$_);
             my $msgtxt;
-            if ((($ENV{'form.critmsg'}) || ($ENV{'form.sendbck'})) &&
-                (&Apache::lonnet::allowed('srm',$ENV{'request.course.id'}))) {
-                $msgtxt=&Apache::lonfeedback::clear_out_html($ENV{'form.message'},1);
+            if ((($env{'form.critmsg'}) || ($env{'form.sendbck'})) &&
+                (&Apache::lonnet::allowed('srm',$env{'request.course.id'}))) {
+                $msgtxt=&Apache::lonfeedback::clear_out_html($env{'form.message'},1);
             } else {  
-	        $msgtxt=&Apache::lonfeedback::clear_out_html($ENV{'form.message'});
+	        $msgtxt=&Apache::lonfeedback::clear_out_html($env{'form.message'});
             }
 	    if ($toaddr{$_}) { $msgtxt.='<hr />'.$toaddr{$_}; }
 	    my $thismsg;    
-	    if ((($ENV{'form.critmsg'}) || ($ENV{'form.sendbck'})) && 
-		(&Apache::lonnet::allowed('srm',$ENV{'request.course.id'}))) {
+	    if ((($env{'form.critmsg'}) || ($env{'form.sendbck'})) && 
+		(&Apache::lonnet::allowed('srm',$env{'request.course.id'}))) {
 		$r->print(&mt('Sending critical message').' '.$recuname.'@'.$recdomain.': ');
 		$thismsg=&user_crit_msg($recuname,$recdomain,
-					&Apache::lonfeedback::clear_out_html($ENV{'form.subject'}),
+					&Apache::lonfeedback::clear_out_html($env{'form.subject'}),
 					$msgtxt,
-					$ENV{'form.sendbck'},$ENV{'form.permanent'});
+					$env{'form.sendbck'},$env{'form.permanent'});
 	    } else {
 		$r->print(&mt('Sending').' '.$recuname.'@'.$recdomain.': ');
 		$thismsg=&user_normal_msg($recuname,$recdomain,
-					  &Apache::lonfeedback::clear_out_html($ENV{'form.subject'}),
+					  &Apache::lonfeedback::clear_out_html($env{'form.subject'}),
 					  $msgtxt,
-					  $content{'citation'},undef,undef,$ENV{'form.permanent'});
-		if (($ENV{'request.course.id'}) && ($ENV{'form.sendmode'} eq 'group')) {
+					  $content{'citation'},undef,undef,$env{'form.permanent'});
+		if (($env{'request.course.id'}) && ($env{'form.sendmode'} eq 'group')) {
 		    &user_normal_msg_raw(
-					 $ENV{'course.'.$ENV{'request.course.id'}.'.num'},
-					 $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
+					 $env{'course.'.$env{'request.course.id'}.'.num'},
+					 $env{'course.'.$env{'request.course.id'}.'.domain'},
 					 'Broadcast ['.$recuname.':'.$recdomain.']',
 					 $msgtxt);
 		}
@@ -1997,7 +1993,7 @@ sub sendoffmail {
     }
     if ($sendstatus=~/^(\s*(?:ok|con_delayed)\s*)*$/) {
 	$r->print('<br /><font color="green">'.&mt('Completed.').'</font>');
-	if ($ENV{'form.displayedcrit'}) {
+	if ($env{'form.displayedcrit'}) {
 	    &discrit($r);
 	} else {
 	    &Apache::loncommunicate::menu($r);
@@ -2028,10 +2024,10 @@ sub handler {
          'sendreply','compose','sendmail','critical','recname','recdom',
          'recordftf','sortedby','block','folder','startdis','interdis',
 	 'showcommentbaseurl','dismode']);
-    $sqs='&sortedby='.$ENV{'form.sortedby'};
+    $sqs='&sortedby='.$env{'form.sortedby'};
 
 # ------------------------------------------------------ They checked for email
-    unless ($ENV{'form.block'}) {
+    unless ($env{'form.block'}) {
         &Apache::lonnet::put('email_status',{'recnewemail'=>0});
     }
 
@@ -2045,7 +2041,7 @@ sub handler {
 
 # ------------------------------------------------------------------ Get Folder
 
-    my $folder=$ENV{'form.folder'};
+    my $folder=$env{'form.folder'};
     unless ($folder) { 
 	$folder=''; 
     } else {
@@ -2054,24 +2050,24 @@ sub handler {
 
 # --------------------------------------------------------------------- Display
 
-    $startdis=$ENV{'form.startdis'};
+    $startdis=$env{'form.startdis'};
     $startdis--;
     unless ($startdis) { $startdis=0; }
 
-    $interdis=$ENV{'form.interdis'};
+    $interdis=$env{'form.interdis'};
     unless ($interdis) { $interdis=20; }
     $sqs.='&interdis='.$interdis;
 
-    if ($ENV{'form.firstview'}) {
+    if ($env{'form.firstview'}) {
 	$startdis=0;
     }
-    if ($ENV{'form.lastview'}) {
+    if ($env{'form.lastview'}) {
 	$startdis=-1;
     }
-    if ($ENV{'form.prevview'}) {
+    if ($env{'form.prevview'}) {
 	$startdis--;
     }
-    if ($ENV{'form.nextview'}) {
+    if ($env{'form.nextview'}) {
 	$startdis++;
     }
     my $postedstartdis=$startdis+1;
@@ -2079,13 +2075,13 @@ sub handler {
 
 # --------------------------------------------------------------- Render Output
 
-    if ($ENV{'form.display'}) {
-	&displaymessage($r,$ENV{'form.display'},$folder);
-    } elsif ($ENV{'form.replyto'}) {
-	&compout($r,'',$ENV{'form.replyto'},undef,undef,$folder);
-    } elsif ($ENV{'form.confirm'}) {
+    if ($env{'form.display'}) {
+	&displaymessage($r,$env{'form.display'},$folder);
+    } elsif ($env{'form.replyto'}) {
+	&compout($r,'',$env{'form.replyto'},undef,undef,$folder);
+    } elsif ($env{'form.confirm'}) {
 	&printheader($r,'','Confirmed Receipt');
-	foreach (keys %ENV) {
+	foreach (keys %env) {
 	    if ($_=~/^form\.rec\_(.*)$/) {
 		$r->print('<b>'.&mt('Confirming Receipt').':</b> '.
 			  &user_crit_received($1).'<br>');
@@ -2098,22 +2094,22 @@ sub handler {
 	    }
 	}
 	&discrit($r);
-    } elsif ($ENV{'form.critical'}) {
+    } elsif ($env{'form.critical'}) {
 	&printheader($r,'','Displaying Critical Messages');
 	&discrit($r);
-    } elsif ($ENV{'form.forward'}) {
-	&compout($r,$ENV{'form.forward'},undef,undef,undef,$folder);
-    } elsif ($ENV{'form.markdel'}) {
+    } elsif ($env{'form.forward'}) {
+	&compout($r,$env{'form.forward'},undef,undef,undef,$folder);
+    } elsif ($env{'form.markdel'}) {
 	&printheader($r,'','Deleted Message');
-	&statuschange($ENV{'form.markdel'},'deleted',$folder);
+	&statuschange($env{'form.markdel'},'deleted',$folder);
 	&Apache::loncommunicate::menu($r);
 	&disall($r,$folder);
-    } elsif ($ENV{'form.markedmove'}) {
+    } elsif ($env{'form.markedmove'}) {
 	my $total=0;
-	foreach (keys %ENV) {
+	foreach (keys %env) {
 	    if ($_=~/^form\.delmark_(.*)$/) {
 		&movemsg(&Apache::lonnet::unescape($1),$folder,
-			 $ENV{'form.movetofolder'});
+			 $env{'form.movetofolder'});
 		$total++;
 	    }
 	}
@@ -2121,9 +2117,9 @@ sub handler {
 	$r->print('Moved '.$total.' message(s)<p>');
 	&Apache::loncommunicate::menu($r);
 	&disall($r,$folder);
-    } elsif ($ENV{'form.markeddel'}) {
+    } elsif ($env{'form.markeddel'}) {
 	my $total=0;
-	foreach (keys %ENV) {
+	foreach (keys %env) {
 	    if ($_=~/^form\.delmark_(.*)$/) {
 		&statuschange(&Apache::lonnet::unescape($1),'deleted',$folder);
 		$total++;
@@ -2133,29 +2129,29 @@ sub handler {
 	$r->print('Deleted '.$total.' message(s)<p>');
 	&Apache::loncommunicate::menu($r);
 	&disall($r,$folder);
-    } elsif ($ENV{'form.markunread'}) {
+    } elsif ($env{'form.markunread'}) {
 	&printheader($r,'','Marked Message as Unread');
-	&statuschange($ENV{'form.markunread'},'new');
+	&statuschange($env{'form.markunread'},'new');
 	&Apache::loncommunicate::menu($r);
 	&disall($r,$folder);
-    } elsif ($ENV{'form.compose'}) {
-	&compout($r,'','',$ENV{'form.compose'});
-    } elsif ($ENV{'form.recordftf'}) {
-	&facetoface($r,$ENV{'form.recordftf'});
-    } elsif ($ENV{'form.block'}) {
-        &examblock($r,$ENV{'form.block'});
-    } elsif ($ENV{'form.sendmail'}) {
+    } elsif ($env{'form.compose'}) {
+	&compout($r,'','',$env{'form.compose'});
+    } elsif ($env{'form.recordftf'}) {
+	&facetoface($r,$env{'form.recordftf'});
+    } elsif ($env{'form.block'}) {
+        &examblock($r,$env{'form.block'});
+    } elsif ($env{'form.sendmail'}) {
 	&sendoffmail($r,$folder);
-	if ($ENV{'form.storebasecomment'}) {
+	if ($env{'form.storebasecomment'}) {
 	    &storecomment($r);
 	}
 	&disall($r,$folder);
-    } elsif ($ENV{'form.newfolder'}) {
+    } elsif ($env{'form.newfolder'}) {
 	&printheader($r,'','New Folder');
-	&makefolder($ENV{'form.newfolder'});
+	&makefolder($env{'form.newfolder'});
 	&Apache::loncommunicate::menu($r);
-	&disall($r,$ENV{'form.newfolder'});
-    } elsif ($ENV{'form.showcommentbaseurl'}) {
+	&disall($r,$env{'form.newfolder'});
+    } elsif ($env{'form.showcommentbaseurl'}) {
 	&storedcommentlisting($r);
     } else {
 	&printheader($r,'','Display All Messages');