--- loncom/interface/lonsearchcat.pm	2005/04/01 19:57:23	1.242
+++ loncom/interface/lonsearchcat.pm	2005/04/07 06:56:23	1.243
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # Search Catalog
 #
-# $Id: lonsearchcat.pm,v 1.242 2005/04/01 19:57:23 albertel Exp $
+# $Id: lonsearchcat.pm,v 1.243 2005/04/07 06:56:23 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -63,7 +63,7 @@ package Apache::lonsearchcat;
 
 use strict;
 use Apache::Constants qw(:common :http);
-use Apache::lonnet();
+use Apache::lonnet;
 use Apache::File();
 use CGI qw(:standard);
 use Text::Query;
@@ -144,54 +144,54 @@ sub handler {
     ##     printing the results.  We only need (theoretically) to do
     ##     this once, so the pause indicator is deleted
     ##
-    if (exists($ENV{'form.pause'})) {
+    if (exists($env{'form.pause'})) {
         sleep(1);
-        delete($ENV{'form.pause'});
+        delete($env{'form.pause'});
     }
     ##
     ## Initialize global variables
     ##
     my $domain  = $r->dir_config('lonDefDomain');
     $diropendb= "/home/httpd/perl/tmp/".
-        "$ENV{'user.domain'}_$ENV{'user.name'}_searchcat.db";
+        "$env{'user.domain'}_$env{'user.name'}_searchcat.db";
     #
     # set the name of the persistent database
-    #          $ENV{'form.persistent_db_id'} can only have digits in it.
-    if (! exists($ENV{'form.persistent_db_id'}) ||
-        ($ENV{'form.persistent_db_id'} =~ /\D/) ||
-        ($ENV{'form.launch'} eq '1')) {
-        $ENV{'form.persistent_db_id'} = time;
+    #          $env{'form.persistent_db_id'} can only have digits in it.
+    if (! exists($env{'form.persistent_db_id'}) ||
+        ($env{'form.persistent_db_id'} =~ /\D/) ||
+        ($env{'form.launch'} eq '1')) {
+        $env{'form.persistent_db_id'} = time;
     }
     $bodytag=&Apache::loncommon::bodytag(undef,undef,undef,1);
     my $persistent_db_file = "/home/httpd/perl/tmp/".
         &Apache::lonnet::escape($domain).
-            '_'.&Apache::lonnet::escape($ENV{'user.name'}).
-                '_'.$ENV{'form.persistent_db_id'}.'_persistent_search.db';
+            '_'.&Apache::lonnet::escape($env{'user.name'}).
+                '_'.$env{'form.persistent_db_id'}.'_persistent_search.db';
     ##
     &Apache::lonhtmlcommon::clear_breadcrumbs();
-    if (exists($ENV{'request.course.id'}) && $ENV{'request.course.id'} ne '') {
+    if (exists($env{'request.course.id'}) && $env{'request.course.id'} ne '') {
         &Apache::lonhtmlcommon::add_breadcrumb
              ({href=>'/adm/searchcat?'.
-                   'catalogmode='.$ENV{'form.catalogmode'}.
-                   '&launch='.$ENV{'form.launch'}.
-                   '&mode='.$ENV{'form.mode'},
+                   'catalogmode='.$env{'form.catalogmode'}.
+                   '&launch='.$env{'form.launch'}.
+                   '&mode='.$env{'form.mode'},
               text=>"Course and Catalog Search",
               target=>'_top',
               bug=>'Searching',});
     } else {
         &Apache::lonhtmlcommon::add_breadcrumb
              ({href=>'/adm/searchcat?'.
-                   'catalogmode='.$ENV{'form.catalogmode'}.
-                   '&launch='.$ENV{'form.launch'}.
-                   '&mode='.$ENV{'form.mode'},
+                   'catalogmode='.$env{'form.catalogmode'}.
+                   '&launch='.$env{'form.launch'}.
+                   '&mode='.$env{'form.mode'},
               text=>"Catalog Search",
               target=>'_top',
               bug=>'Searching',});
     }
     #
-    if ($ENV{'form.phase'} !~ m/(basic|adv|course)_search/) {
+    if ($env{'form.phase'} !~ m/(basic|adv|course)_search/) {
         if (! &get_persistent_form_data($persistent_db_file)) {
-            if ($ENV{'form.phase'} =~ /(run_search|results)/) {
+            if ($env{'form.phase'} =~ /(run_search|results)/) {
                 &Apache::lonnet::logthis('lonsearchcat:'.
                                          'Unable to recover data from '.
                                          $persistent_db_file);
@@ -216,13 +216,13 @@ END
     ## Clear out old values from groupsearch database
     ##
     untie %groupsearch_db if (tied(%groupsearch_db));
-    if (($ENV{'form.cleargroupsort'} eq '1') || 
-        (($ENV{'form.launch'} eq '1') && 
-         ($ENV{'form.catalogmode'} eq 'groupsearch'))) {
+    if (($env{'form.cleargroupsort'} eq '1') || 
+        (($env{'form.launch'} eq '1') && 
+         ($env{'form.catalogmode'} eq 'groupsearch'))) {
 	if (tie(%groupsearch_db,'GDBM_File',$diropendb,&GDBM_WRCREAT(),0640)) {
 	    &start_fresh_session();
 	    untie %groupsearch_db;
-            delete($ENV{'form.cleargroupsort'});
+            delete($env{'form.cleargroupsort'});
 	} else {
             # This is a stupid error to give to the user.  
             # It really tells them nothing.
@@ -236,36 +236,36 @@ END
     ## Configure hidden fields
     ##
     $hidden_fields = '<input type="hidden" name="persistent_db_id" value="'.
-        $ENV{'form.persistent_db_id'}.'" />'."\n";
-    if (exists($ENV{'form.catalogmode'})) {
+        $env{'form.persistent_db_id'}.'" />'."\n";
+    if (exists($env{'form.catalogmode'})) {
         $hidden_fields .= &hidden_field('catalogmode');
     }
-    if (exists($ENV{'form.form'})) {
+    if (exists($env{'form.form'})) {
         $hidden_fields .= &hidden_field('form');
     }
-    if (exists($ENV{'form.element'})) {
+    if (exists($env{'form.element'})) {
         $hidden_fields .= &hidden_field('element');
     }
-    if (exists($ENV{'form.titleelement'})) {
+    if (exists($env{'form.titleelement'})) {
         $hidden_fields .= &hidden_field('titleelement');
     }
-    if (exists($ENV{'form.mode'})) {
+    if (exists($env{'form.mode'})) {
         $hidden_fields .= &hidden_field('mode');
     }
     ##
     ## Configure dynamic components of interface
     ##
-    if ($ENV{'form.catalogmode'} eq 'interactive') {
+    if ($env{'form.catalogmode'} eq 'interactive') {
         $closebutton="<input type='button' name='close' value='CLOSE' ";
-        if ($ENV{'form.phase'} =~ /(results|run_search)/) {
+        if ($env{'form.phase'} =~ /(results|run_search)/) {
 	    $closebutton .="onClick='parent.close()'";
         } else {
             $closebutton .="onClick='self.close()'";
         }
         $closebutton .=">\n";
-    } elsif ($ENV{'form.catalogmode'} eq 'groupsearch') {
+    } elsif ($env{'form.catalogmode'} eq 'groupsearch') {
         $closebutton="<input type='button' name='close' value='CLOSE' ";
-        if ($ENV{'form.phase'} =~ /(results|run_search)/) {
+        if ($env{'form.phase'} =~ /(results|run_search)/) {
 	    $closebutton .="onClick='parent.close()'";
         } else {
             $closebutton .="onClick='self.close()'";
@@ -282,76 +282,76 @@ END
     ##
     ## Sanity checks on form elements
     ##
-    if (!defined($ENV{'form.viewselect'})) {
-        if (($ENV{'form.catalogmode'} eq 'groupsearch') ||
-            ($ENV{'form.catalogmode'} eq 'interactive')) {
-            $ENV{'form.viewselect'} ="Compact View";
+    if (!defined($env{'form.viewselect'})) {
+        if (($env{'form.catalogmode'} eq 'groupsearch') ||
+            ($env{'form.catalogmode'} eq 'interactive')) {
+            $env{'form.viewselect'} ="Compact View";
         } else {
-            $ENV{'form.viewselect'} ="Detailed Citation View";
+            $env{'form.viewselect'} ="Detailed Citation View";
         }
     }
-    $ENV{'form.phase'} = 'disp_basic' if (! exists($ENV{'form.phase'}));
-    $ENV{'form.show'} = 20 if (! exists($ENV{'form.show'}));
+    $env{'form.phase'} = 'disp_basic' if (! exists($env{'form.phase'}));
+    $env{'form.show'} = 20 if (! exists($env{'form.show'}));
     #
-    $ENV{'form.searchmode'} = 'basic' if (! exists($ENV{'form.searchmode'}));
-    if ($ENV{'form.phase'} eq 'adv_search' ||
-        $ENV{'form.phase'} eq 'disp_adv') {
-        $ENV{'form.searchmode'} = 'advanced';
-    } elsif ($ENV{'form.phase'} eq 'course_search') {
-        $ENV{'form.searchmode'} = 'course_search';
+    $env{'form.searchmode'} = 'basic' if (! exists($env{'form.searchmode'}));
+    if ($env{'form.phase'} eq 'adv_search' ||
+        $env{'form.phase'} eq 'disp_adv') {
+        $env{'form.searchmode'} = 'advanced';
+    } elsif ($env{'form.phase'} eq 'course_search') {
+        $env{'form.searchmode'} = 'course_search';
     }
     #
-    if ($ENV{'form.searchmode'} eq 'advanced') {
+    if ($env{'form.searchmode'} eq 'advanced') {
         &Apache::lonhtmlcommon::add_breadcrumb
             ({href=>'/adm/searchcat?phase=disp_adv&'.
-                  'catalogmode='.$ENV{'form.catalogmode'}.
-                  '&launch='.$ENV{'form.launch'}.
-                  '&mode='.$ENV{'form.mode'},
+                  'catalogmode='.$env{'form.catalogmode'}.
+                  '&launch='.$env{'form.launch'}.
+                  '&mode='.$env{'form.mode'},
                   text=>"Advanced Search",
                   bug=>'Searching',});
-    } elsif ($ENV{'form.searchmode'} eq 'course search') {
+    } elsif ($env{'form.searchmode'} eq 'course search') {
         &Apache::lonhtmlcommon::add_breadcrumb
             ({href=>'/adm/searchcat?phase=disp_adv&'.
-                  'catalogmode='.$ENV{'form.catalogmode'}.
-                  '&launch='.$ENV{'form.launch'}.
-                  '&mode='.$ENV{'form.mode'},
+                  'catalogmode='.$env{'form.catalogmode'}.
+                  '&launch='.$env{'form.launch'}.
+                  '&mode='.$env{'form.mode'},
                   text=>"Course Search",
                   bug=>'Searching',});
     }
     ##
     ## Switch on the phase
     ##
-    if ($ENV{'form.phase'} eq 'disp_basic') {
+    if ($env{'form.phase'} eq 'disp_basic') {
         &print_basic_search_form($r,$closebutton,$hidden_fields);
-    } elsif ($ENV{'form.phase'} eq 'disp_adv') {
+    } elsif ($env{'form.phase'} eq 'disp_adv') {
         &print_advanced_search_form($r,$closebutton,$hidden_fields);
-    } elsif ($ENV{'form.phase'} eq 'results') {
+    } elsif ($env{'form.phase'} eq 'results') {
         &display_results($r,$importbutton,$closebutton,$diropendb);
-    } elsif ($ENV{'form.phase'} =~ /^(sort|run_search)$/) {
+    } elsif ($env{'form.phase'} =~ /^(sort|run_search)$/) {
         my ($query,$customquery,$customshow,$libraries,$pretty_string) =
             &get_persistent_data($persistent_db_file,
                  ['query','customquery','customshow',
                   'libraries','pretty_string']);
-        if ($ENV{'form.phase'} eq 'sort') {
+        if ($env{'form.phase'} eq 'sort') {
             &print_sort_form($r,$pretty_string);
-        } elsif ($ENV{'form.phase'} eq 'run_search') {
+        } elsif ($env{'form.phase'} eq 'run_search') {
             &run_search($r,$query,$customquery,$customshow,
                         $libraries,$pretty_string);
         }
-    } elsif ($ENV{'form.phase'} eq 'course_search') {
+    } elsif ($env{'form.phase'} eq 'course_search') {
         &course_search($r);
-    } elsif(($ENV{'form.phase'} eq 'basic_search') ||
-            ($ENV{'form.phase'} eq 'adv_search')) {
+    } elsif(($env{'form.phase'} eq 'basic_search') ||
+            ($env{'form.phase'} eq 'adv_search')) {
         #
         # We are running a search, try to parse it
         my ($query,$customquery,$customshow,$libraries) = 
             (undef,undef,undef,undef);
         my $pretty_string;
-        if ($ENV{'form.phase'} eq 'basic_search') {
+        if ($env{'form.phase'} eq 'basic_search') {
             ($query,$pretty_string,$libraries) = 
                 &parse_basic_search($r,$closebutton,$hidden_fields);
             return OK if (! defined($query));
-            &make_persistent({ basicexp => $ENV{'form.basicexp'}},
+            &make_persistent({ basicexp => $env{'form.basicexp'}},
                              $persistent_db_file);
         } else {                      # Advanced search
             ($query,$customquery,$customshow,$libraries,$pretty_string) 
@@ -384,7 +384,7 @@ The search has been aborted.
 END
             return OK;
         }
-        delete($ENV{'form.launch'});
+        delete($env{'form.launch'});
         if (! &make_form_data_persistent($r,$persistent_db_file)) {
 	    my $html=&Apache::lonxml::xmlbegin();
 	    $r->print(<<END);
@@ -415,23 +415,23 @@ END
 # This routine sets non existant checkbox form elements to ''.
 #
 sub clean_up_environment {
-    if ($ENV{'form.phase'} eq 'basic_search') {
-        if (! exists($ENV{'form.related'})) {
-            $ENV{'form.related'} = '';
+    if ($env{'form.phase'} eq 'basic_search') {
+        if (! exists($env{'form.related'})) {
+            $env{'form.related'} = '';
         }
-        if (! exists($ENV{'form.domains'})) {
-            $ENV{'form.domains'} = '';
+        if (! exists($env{'form.domains'})) {
+            $env{'form.domains'} = '';
         }
-    } elsif ($ENV{'form.phase'} eq 'adv_search') {
+    } elsif ($env{'form.phase'} eq 'adv_search') {
         foreach my $field ('title','keywords','notes',
                            'abstract','standards','mime') {
-            if (! exists($ENV{'form.'.$field.'_related'})) {
-                $ENV{'form.'.$field.'_related'} = '';
+            if (! exists($env{'form.'.$field.'_related'})) {
+                $env{'form.'.$field.'_related'} = '';
             }
         }
-    } elsif ($ENV{'form.phase'} eq 'course_search') {
-        if (! exists($ENV{'form.crsrelated'})) {
-            $ENV{'form.crsrelated'} = '';
+    } elsif ($env{'form.phase'} eq 'course_search') {
+        if (! exists($env{'form.crsrelated'})) {
+            $env{'form.crsrelated'} = '';
         }
     }
 }
@@ -439,7 +439,7 @@ sub clean_up_environment {
 sub hidden_field {
     my ($name,$value) = @_;
     if (! defined($value)) {
-        $value = $ENV{'form.'.$name};
+        $value = $env{'form.'.$name};
     }
     return '<input type="hidden" name="'.$name.'" value="'.$value.'" />'.$/;
 }
@@ -470,20 +470,20 @@ sub make_symb {
 sub course_search {
     my $r=shift;
     my $bodytag=&Apache::loncommon::bodytag('Course Search');
-    my $pretty_search_string = '<b>'.$ENV{'form.courseexp'}.'</b>';
-    my $search_string = $ENV{'form.courseexp'};
+    my $pretty_search_string = '<b>'.$env{'form.courseexp'}.'</b>';
+    my $search_string = $env{'form.courseexp'};
     my @New_Words;
     undef(%alreadyseen);
-    if ($ENV{'form.crsrelated'}) {
-        ($search_string,@New_Words) = &related_version($ENV{'form.courseexp'});
+    if ($env{'form.crsrelated'}) {
+        ($search_string,@New_Words) = &related_version($env{'form.courseexp'});
         if (@New_Words) {
             $pretty_search_string .= ' '.&mt("with related words").": <b>@New_Words</b>.";
         } else {
             $pretty_search_string .= ' '.&mt('with no related words').".";
         }
     }
-    my $fulltext=$ENV{'form.crsfulltext'};
-    my $discuss=$ENV{'form.crsdiscuss'};
+    my $fulltext=$env{'form.crsfulltext'};
+    my $discuss=$env{'form.crsdiscuss'};
     my @allwords=($search_string,@New_Words);
     $totalfound=0;
     my $html=&Apache::lonxml::xmlbegin();
@@ -492,12 +492,12 @@ sub course_search {
     $r->rflush();
 # ======================================================= Go through the course
     my $c=$r->connection;
-    if (tie(%hash,'GDBM_File',$ENV{'request.course.fn'}.".db",
+    if (tie(%hash,'GDBM_File',$env{'request.course.fn'}.".db",
             &GDBM_READER(),0640)) {
         foreach (sort(keys(%hash))) {
             if ($c->aborted()) { last; }
             if (($_=~/^src\_(.+)$/)) {
-		if ($hash{'randomout_'.$1} & !$ENV{'request.role.adv'}) {
+		if ($hash{'randomout_'.$1} & !$env{'request.role.adv'}) {
 		    next; 
 		}
 		my $symb=&make_symb($1);
@@ -518,8 +518,8 @@ sub course_search {
         my $navmap = Apache::lonnavmaps::navmap->new();
         my @allres=$navmap->retrieveResources();
         my %discussiontime = &Apache::lonnet::dump('discussiontimes',
-                               $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 my $resource (@allres) {
             my $result = '';
             my $applies = 0;
@@ -532,9 +532,9 @@ sub course_search {
                 }
             }
             if (defined($discussiontime{$ressymb})) { 
-                my %contrib = &Apache::lonnet::restore($ressymb,$ENV{'request.course.id'},
-                     $ENV{'course.'.$ENV{'request.course.id'}.'.domain'},
-                     $ENV{'course.'.$ENV{'request.course.id'}.'.num'});
+                my %contrib = &Apache::lonnet::restore($ressymb,$env{'request.course.id'},
+                     $env{'course.'.$env{'request.course.id'}.'.domain'},
+                     $env{'course.'.$env{'request.course.id'}.'.num'});
                 if ($contrib{'version'}) {
                     for (my $id=1;$id<=$contrib{'version'};$id++) {
                         unless (($contrib{'hidden'}=~/\.$id\./) || ($contrib{'deleted'}=~/\.$id\./)) { 
@@ -598,7 +598,7 @@ sub checkonthis {
     $r->rflush();
     
     my $result=$title.' ';
-    if ($ENV{'request.role.adv'} || !$hash{'encrypted_'.$id}) {
+    if ($env{'request.role.adv'} || !$hash{'encrypted_'.$id}) {
 	$result.=&Apache::lonnet::metadata($url,'title').' '.
 	    &Apache::lonnet::metadata($url,'subject').' '.
 	    &Apache::lonnet::metadata($url,'abstract').' '.
@@ -619,7 +619,7 @@ sub checkonthis {
            $r->print('&nbsp;');
        }
        my $href=$url;
-       if ($hash{'encrypted_'.$id} && !$ENV{'request.role.adv'}) {
+       if ($hash{'encrypted_'.$id} && !$env{'request.role.adv'}) {
 	   $href=&Apache::lonenc::encrypted($href)
 	       .'?symb='.&Apache::lonenc::encrypted($symb);
        } else {
@@ -688,22 +688,22 @@ Prints the form for the basic search.  S
 ######################################################################
 sub print_basic_search_form {
     my ($r,$closebutton,$hidden_fields) = @_;
-    my $result = ($ENV{'form.catalogmode'} ne 'groupsearch');
+    my $result = ($env{'form.catalogmode'} ne 'groupsearch');
     my $bodytag=&Apache::loncommon::bodytag('Search').
         &Apache::lonhtmlcommon::breadcrumbs(undef,'Searching','Search_Basic',
                                    undef,undef,
-                                   $ENV{'form.catalogmode'} ne 'groupsearch');
+                                   $env{'form.catalogmode'} ne 'groupsearch');
     my $scrout = &search_html_header().$bodytag;
-    if (&Apache::lonnet::allowed('bre',$ENV{'request.role.domain'})) {
+    if (&Apache::lonnet::allowed('bre',$env{'request.role.domain'})) {
         # Define interface components
         my $userelatedwords= '<label>'.
             &mt('[_1] use related words',
                 &Apache::lonhtmlcommon::checkbox
-                ('related',$ENV{'form.related'},'related')).'</label>';
+                ('related',$env{'form.related'},'related')).'</label>';
         my $onlysearchdomain='<label>'.
             &mt('[_1] only search domain [_2]',
                 &Apache::lonhtmlcommon::checkbox('domains',
-                                                 $ENV{'form.domains'},
+                                                 $env{'form.domains'},
                                                  $r->dir_config('lonDefDomain')
                                                  ),
                 $r->dir_config('lonDefDomain')
@@ -711,9 +711,9 @@ sub print_basic_search_form {
         my $adv_search_link = 
             '<a href="/adm/searchcat?'.
             'phase=disp_adv&'.
-            'catalogmode='.$ENV{'form.catalogmode'}.
-            '&launch='.$ENV{'form.launch'}.
-            '&mode='.$ENV{'form.mode'}.
+            'catalogmode='.$env{'form.catalogmode'}.
+            '&launch='.$env{'form.launch'}.
+            '&mode='.$env{'form.mode'}.
             '">'.&mt('Advanced Search').'</a>';
         #
         $scrout.='<form name="loncapa_search" method="post" '.
@@ -722,7 +722,7 @@ sub print_basic_search_form {
             $hidden_fields;
         #
         $scrout .= '<center>'.$/;
-        if ($ENV{'request.course.id'}) {
+        if ($env{'request.course.id'}) {
             $scrout .= '<h1>'.&mt('LON-CAPA Catalog Search').'</h1>';
         } else {
             # No need to tell them they are searching
@@ -732,7 +732,7 @@ sub print_basic_search_form {
             '<tr><td align="center" valign="top">'.
             &Apache::lonhtmlcommon::textbox
             ('basicexp',
-             &HTML::Entities::encode($ENV{'form.basicexp'},'<>&"'),50
+             &HTML::Entities::encode($env{'form.basicexp'},'<>&"'),50
              ).
              '<br />'.
             '<font size="-1">'.&searchhelp().'</font>'.'</td>'.
@@ -753,7 +753,7 @@ sub print_basic_search_form {
             '</td></tr>'.$/;
         $scrout .= '</table>'.$/.'</center>'.'</form>';
     }
-    if ($ENV{'request.course.id'}) {
+    if ($env{'request.course.id'}) {
 	my %lt=&Apache::lonlocal::texthash('srch' => 'Search',
                                            'header' => 'Course Search',
 	 'note' => 'Enter terms or phrases, then press "Search" below',
@@ -777,16 +777,16 @@ $lt{'note'}.
 ENDCOURSESEARCH
         $scrout.='&nbsp;'.
             &Apache::lonhtmlcommon::textbox('courseexp',
-                                  $ENV{'form.courseexp'},40);
+                                  $env{'form.courseexp'},40);
         my $crscheckbox = 
             &Apache::lonhtmlcommon::checkbox('crsfulltext',
-                                   $ENV{'form.crsfulltext'});
+                                   $env{'form.crsfulltext'});
         my $relcheckbox = 
             &Apache::lonhtmlcommon::checkbox('crsrelated',
-				   $ENV{'form.crsrelated'});
+				   $env{'form.crsrelated'});
         my $discheckbox = 
             &Apache::lonhtmlcommon::checkbox('crsdiscuss',
-                                   $ENV{'form.crsrelated'});
+                                   $env{'form.crsrelated'});
         $scrout.=(<<ENDENDCOURSE);
 </td></tr>
 <tr><td><label>$relcheckbox $lt{'use'}</label></td><td></td></tr>
@@ -825,7 +825,7 @@ sub print_advanced_search_form{
         &Apache::lonhtmlcommon::breadcrumbs(undef,'Searching',
                                             'Search_Advanced',
                                             undef,undef,
-                                  $ENV{'form.catalogmode'} ne 'groupsearch');
+                                  $env{'form.catalogmode'} ne 'groupsearch');
     my %lt=&Apache::lonlocal::texthash('srch' => 'Search',
 				       'reset' => 'Reset',
 				       'help' => 'Help');
@@ -866,12 +866,12 @@ ENDHEADER
 	     'keywords','notes','abstract','standards','mime') {
 	$scrout.='<tr><td align="right">'.&titlefield($fields{$field}).'</td><td>'.
 	    &Apache::lonmeta::prettyinput($field,
-                                          $ENV{'form.'.$field},
+                                          $env{'form.'.$field},
                                           $field,
                                           'advsearch',
 					  $related_word_search{$field},
                                           '</td><td align="left">',
-                                          $ENV{'form.'.$field.'_related'},
+                                          $env{'form.'.$field.'_related'},
                                           50);
         if ($related_word_search{$field}) {
             $scrout .= 'related words';
@@ -885,7 +885,7 @@ ENDHEADER
             '<td align="right">'.&titlefield($fields{$field}).'</td>'.
             '<td colspan="2">'.
 	    &Apache::lonmeta::prettyinput($field,
-                                          $ENV{'form.'.$field},
+                                          $env{'form.'.$field},
                                           $field,
                                           'advsearch',
 					  0).
@@ -894,19 +894,19 @@ ENDHEADER
     $scrout.='<tr><td align="right">'.
 	&titlefield(&mt('MIME Type Category')).'</td><td colspan="2">'. 
 	    &Apache::loncommon::filecategoryselect('category',
-						   $ENV{'form.category'}).
+						   $env{'form.category'}).
 	    '</td></tr>'.$/;
     $scrout.='<tr><td align="right" valign="top">'.
 	&titlefield(&mt('Domains')).'</td><td colspan="2">'. 
 	    &Apache::loncommon::domain_select('domains',
-						   $ENV{'form.domains'},1).
+						   $env{'form.domains'},1).
 	    '</td></tr>'.$/;
     #
     # Misc metadata
     $scrout.='<tr><td align="right" valign="top">'.
 	&titlefield(&mt('Copyright/Distribution')).'</td><td colspan="2">'.
         &Apache::lonmeta::selectbox('copyright',
-                                    $ENV{'form.copyright'},
+                                    $env{'form.copyright'},
                                     \&Apache::loncommon::copyrightdescription,
                                     ( undef,
                                       &Apache::loncommon::copyrightids)
@@ -914,7 +914,7 @@ ENDHEADER
     $scrout.='<tr><td align="right" valign="top">'.
 	&titlefield(&mt('Language')).'</td><td colspan="2">'.
         &Apache::lonmeta::selectbox('language',
-                                    $ENV{'form.language'},
+                                    $env{'form.language'},
                                     \&Apache::loncommon::languagedescription,
                                     ('any',&Apache::loncommon::languageids)
                                     ).'</td></tr>';
@@ -1096,16 +1096,16 @@ Outputs: text for box with view options
 ######################################################################
 sub viewoptions {
     my $scrout;
-    if (! defined($ENV{'form.viewselect'})) { 
-        $ENV{'form.viewselect'}='detailed'; 
+    if (! defined($env{'form.viewselect'})) { 
+        $env{'form.viewselect'}='detailed'; 
     }
     $scrout.=&Apache::lonmeta::selectbox('viewselect',
-			$ENV{'form.viewselect'},
+			$env{'form.viewselect'},
 			\&viewoptiontext,
 			sort(keys(%Views)));
     $scrout.= '&nbsp;&nbsp;';
     my $countselect = &Apache::lonmeta::selectbox('show',
-                                                  $ENV{'form.show'},
+                                                  $env{'form.show'},
                                                   undef,
                                                   (10,20,50,100,1000,10000));
     $scrout .= ('&nbsp;'x2).&mt('[_1] Records per Page',$countselect).
@@ -1145,7 +1145,7 @@ Outputs: returns undef on database error
 
 This function is the reverse of &make_persistent() for form data.
 Retrieve persistent data from %persistent_db.  Retrieved items will have their
-values unescaped.  If a form value already exists in $ENV, it will not be
+values unescaped.  If a form value already exists in $env, it will not be
 overwritten.  Form values that are array references may have values appended
 to them.
 
@@ -1167,20 +1167,20 @@ sub get_persistent_form_data {
         next if ($name !~ /^form./);
         # Kludgification begins!
         if ($name eq 'form.domains' && 
-            $ENV{'form.searchmode'} eq 'basic' &&
-            $ENV{'form.phase'} ne 'disp_basic') {
+            $env{'form.searchmode'} eq 'basic' &&
+            $env{'form.phase'} ne 'disp_basic') {
             next;
         }
         # End kludge (hopefully)
-        next if (exists($ENV{$name}));
+        next if (exists($env{$name}));
         my @values = map { 
             &Apache::lonnet::unescape($_);
         } split(',',$persistent_db{$name});
         next if (@values <1);
         if ($arrays_allowed{$name}) {
-            $ENV{$name} = [@values];
+            $env{$name} = [@values];
         } else {
-            $ENV{$name} = $values[0] if ($values[0]);
+            $env{$name} = $values[0] if ($values[0]);
         }
     }
     untie (%persistent_db);
@@ -1285,9 +1285,9 @@ sub make_form_data_persistent {
     my $r = shift;
     my $filename = shift;
     my %save;
-    foreach (keys(%ENV)) {
+    foreach (keys(%env)) {
         next if (!/^form/ || /submit/);
-        $save{$_} = $ENV{$_};
+        $save{$_} = $env{$_};
     }
     return &make_persistent(\%save,$filename);
 }
@@ -1335,36 +1335,36 @@ sub parse_advanced_search {
 		   'lastrevisiondatestart_month','lastrevisiondatestart_day',
 		   'lastrevisiondatestart_year','lastrevisiondateend_month',
 		   'lastrevisiondateend_day','lastrevisiondateend_year') {
-	$ENV{'form.'.$field}=~s/[^\w\/\s\(\)\=\-\"\']//g;
+	$env{'form.'.$field}=~s/[^\w\/\s\(\)\=\-\"\']//g;
     }
     foreach ('mode','form','element') {
 	# is this required?  Hmmm.
-	next if (! exists($ENV{'form.'.$_}));
-	$ENV{'form.'.$_}=&Apache::lonnet::unescape($ENV{'form.'.$_});
-	$ENV{'form.'.$_}=~s/[^\w\/\s\(\)\=\-\"\']//g;
+	next if (! exists($env{'form.'.$_}));
+	$env{'form.'.$_}=&Apache::lonnet::unescape($env{'form.'.$_});
+	$env{'form.'.$_}=~s/[^\w\/\s\(\)\=\-\"\']//g;
     }
     # Preprocess the category form element.
-    $ENV{'form.category'} = 'any' if (! defined($ENV{'form.category'}) ||
-                                      ref($ENV{'form.category'}));
+    $env{'form.category'} = 'any' if (! defined($env{'form.category'}) ||
+                                      ref($env{'form.category'}));
     #
     # Check to see if enough information was filled in
     foreach my $field (@BasicFields) {
-	if (&filled($ENV{'form.'.$field})) {
+	if (&filled($env{'form.'.$field})) {
 	    $fillflag++;
 	}
     }
     foreach my $field (@StatsFields,@EvalFields) {
-        if (&filled($ENV{'form.'.$field.'_max'})) {
+        if (&filled($env{'form.'.$field.'_max'})) {
             $fillflag++;
         }
-        if (&filled($ENV{'form.'.$field.'_min'})) {
+        if (&filled($env{'form.'.$field.'_min'})) {
             $fillflag++;
         }
     }
 
     for my $field ('lowestgradelevel','highestgradelevel') {
-        if ( $ENV{'form.'.$field} =~ /^\d+$/ &&
-             $ENV{'form.'.$field} > 0) {
+        if ( $env{'form.'.$field} =~ /^\d+$/ &&
+             $env{'form.'.$field} > 0) {
             $fillflag++;
         }
     }
@@ -1379,21 +1379,21 @@ sub parse_advanced_search {
     my $font = '<font color="#800000" face="helvetica">';
     # Evaluate logical expression AND/OR/NOT phrase fields.
     foreach my $field (@BasicFields) {
-	next if (!defined($ENV{'form.'.$field}) || $ENV{'form.'.$field} eq '');
+	next if (!defined($env{'form.'.$field}) || $env{'form.'.$field} eq '');
         my ($error,$SQLQuery) = 
-            &process_phrase_input($ENV{'form.'.$field},
-                                  $ENV{'form.'.$field.'_related'},$field);
+            &process_phrase_input($env{'form.'.$field},
+                                  $env{'form.'.$field.'_related'},$field);
         if (defined($error)) {
             &output_unparsed_phrase_error($r,$closebutton,'phase=disp_adv',
                                          $hidden_fields,$field);
             return;
         } else {
             $pretty_search_string .= 
-                $font.$field.'</font>: '.$ENV{'form.'.$field};
-            if ($ENV{'form.'.$field.'_related'}) {
+                $font.$field.'</font>: '.$env{'form.'.$field};
+            if ($env{'form.'.$field.'_related'}) {
                 my @Words = 
                     &Apache::loncommon::get_related_words
-                    ($ENV{'form.'.$field});
+                    ($env{'form.'.$field});
                 if (@Words) {
                     $pretty_search_string.= ' with related words: '.
                         join(', ',@Words[0..4]);
@@ -1409,11 +1409,11 @@ sub parse_advanced_search {
     # Make the 'mime' from 'form.category' and 'form.extension'
     #
     my $searchphrase;
-    if (exists($ENV{'form.category'})    && 
-        $ENV{'form.category'} !~ /^\s*$/ &&
-        $ENV{'form.category'} ne 'any')     {
+    if (exists($env{'form.category'})    && 
+        $env{'form.category'} !~ /^\s*$/ &&
+        $env{'form.category'} ne 'any')     {
         my @extensions = &Apache::loncommon::filecategorytypes
-                                                   ($ENV{'form.category'});
+                                                   ($env{'form.category'});
         if (scalar(@extensions) > 0) {
             $searchphrase = join(' OR ',@extensions);
         }
@@ -1426,45 +1426,45 @@ sub parse_advanced_search {
     }
     #
     # Evaluate option lists
-    if ($ENV{'form.lowestgradelevel'}        &&
-        $ENV{'form.lowestgradelevel'} ne '0' &&
-        $ENV{'form.lowestgradelevel'} =~ /^\d+$/) {
+    if ($env{'form.lowestgradelevel'}        &&
+        $env{'form.lowestgradelevel'} ne '0' &&
+        $env{'form.lowestgradelevel'} =~ /^\d+$/) {
 	push(@queries,
-             '(lowestgradelevel>='.$ENV{'form.lowestgradelevel'}.')');
+             '(lowestgradelevel>='.$env{'form.lowestgradelevel'}.')');
         $pretty_search_string.="lowestgradelevel>=".
-            $ENV{'form.lowestgradelevel'}."<br />\n";
+            $env{'form.lowestgradelevel'}."<br />\n";
     }
-    if ($ENV{'form.highestgradelevel'}        &&
-        $ENV{'form.highestgradelevel'} ne '0' &&
-        $ENV{'form.highestgradelevel'} =~ /^\d+$/) {
+    if ($env{'form.highestgradelevel'}        &&
+        $env{'form.highestgradelevel'} ne '0' &&
+        $env{'form.highestgradelevel'} =~ /^\d+$/) {
 	push(@queries,
-             '(highestgradelevel<='.$ENV{'form.highestgradelevel'}.')');
+             '(highestgradelevel<='.$env{'form.highestgradelevel'}.')');
         $pretty_search_string.="highestgradelevel<=".
-            $ENV{'form.highestgradelevel'}."<br />\n";
+            $env{'form.highestgradelevel'}."<br />\n";
     }
-    if ($ENV{'form.language'} and $ENV{'form.language'} ne 'any') {
-	push @queries,"(language like \"$ENV{'form.language'}\")";
+    if ($env{'form.language'} and $env{'form.language'} ne 'any') {
+	push @queries,"(language like \"$env{'form.language'}\")";
         $pretty_search_string.=$font."language</font>= ".
-            &Apache::loncommon::languagedescription($ENV{'form.language'}).
+            &Apache::loncommon::languagedescription($env{'form.language'}).
                 "<br />\n";
     }
-    if ($ENV{'form.copyright'} and $ENV{'form.copyright'} ne 'any') {
-	push @queries,"(copyright like \"$ENV{'form.copyright'}\")";
+    if ($env{'form.copyright'} and $env{'form.copyright'} ne 'any') {
+	push @queries,"(copyright like \"$env{'form.copyright'}\")";
         $pretty_search_string.=$font."copyright</font> = ".
-            &Apache::loncommon::copyrightdescription($ENV{'form.copyright'}).
+            &Apache::loncommon::copyrightdescription($env{'form.copyright'}).
                 "<br />\n";
     }
     #
     # Statistics
     foreach my $field (@StatsFields,@EvalFields) {
         my ($min,$max);
-        if (exists($ENV{'form.'.$field.'_min'}) && 
-            $ENV{'form.'.$field.'_min'} ne '') {
-            $min = $ENV{'form.'.$field.'_min'};
-        }
-        if (exists($ENV{'form.'.$field.'_max'}) &&
-            $ENV{'form.'.$field.'_max'} ne '') {
-            $max = $ENV{'form.'.$field.'_max'};
+        if (exists($env{'form.'.$field.'_min'}) && 
+            $env{'form.'.$field.'_min'} ne '') {
+            $min = $env{'form.'.$field.'_min'};
+        }
+        if (exists($env{'form.'.$field.'_max'}) &&
+            $env{'form.'.$field.'_max'} ne '') {
+            $max = $env{'form.'.$field.'_max'};
         }
         next if (! defined($max) && ! defined($min));
         if (defined($min) && defined($max)) {
@@ -1517,17 +1517,17 @@ sub parse_advanced_search {
     ## because I was unable to figureout exactly how it worked and could
     ## not imagine people actually using it.  MH
     ##
-    # if ($ENV{'form.custommetadata'}) {
+    # if ($env{'form.custommetadata'}) {
     #    $pretty_search_string .=$font."Custom Metadata Search</font>: <b>".
-    #    $ENV{'form.custommetadata'}."</b><br />\n";
+    #    $env{'form.custommetadata'}."</b><br />\n";
     #    $customquery=&build_custommetadata_query('custommetadata',
-    #                                             $ENV{'form.custommetadata'});
+    #                                             $env{'form.custommetadata'});
     # }
     my $customshow=undef;
-    # if ($ENV{'form.customshow'}) {
+    # if ($env{'form.customshow'}) {
     # $pretty_search_string .=$font."Custom Metadata Display</font>: <b>".
-    #                         $ENV{'form.customshow'}."</b><br />\n";
-    #    $customshow=$ENV{'form.customshow'};
+    #                         $env{'form.customshow'}."</b><br />\n";
+    #    $customshow=$env{'form.customshow'};
     #    $customshow=~s/[^\w\s]//g;
     #    my @fields=split(/\s+/,$customshow);
     #    $customshow=join(" ",@fields);
@@ -1552,16 +1552,16 @@ sub parse_advanced_search {
 
 sub parse_domain_restrictions {
     my $libraries_to_query = undef;
-    # $ENV{'form.domains'} can be either a scalar or an array reference.
+    # $env{'form.domains'} can be either a scalar or an array reference.
     # We need an array.
-    if (! exists($ENV{'form.domains'}) || $ENV{'form.domains'} eq '') {
+    if (! exists($env{'form.domains'}) || $env{'form.domains'} eq '') {
         return (undef,'',undef);
     }
     my @allowed_domains;
-    if (ref($ENV{'form.domains'})) {
-        @allowed_domains =  @{$ENV{'form.domains'}};
+    if (ref($env{'form.domains'})) {
+        @allowed_domains =  @{$env{'form.domains'}};
     } else {
-        @allowed_domains = ($ENV{'form.domains'});
+        @allowed_domains = ($env{'form.domains'});
     }
     #
     my %domain_hash = ();
@@ -1613,19 +1613,19 @@ sub parse_basic_search {
     #
     # Clean up fields for safety
     for my $field ('basicexp') {
-	$ENV{"form.$field"}=~s/[^\w\s\'\"\!\(\)\-]//g;
+	$env{"form.$field"}=~s/[^\w\s\'\"\!\(\)\-]//g;
     }
     foreach ('mode','form','element') {
 	# is this required?  Hmmm.
-	next unless (exists($ENV{"form.$_"}));
-	$ENV{"form.$_"}=&Apache::lonnet::unescape($ENV{"form.$_"});
-	$ENV{"form.$_"}=~s/[^\w\/\s\(\)\=\-\"\']//g;
+	next unless (exists($env{"form.$_"}));
+	$env{"form.$_"}=&Apache::lonnet::unescape($env{"form.$_"});
+	$env{"form.$_"}=~s/[^\w\/\s\(\)\=\-\"\']//g;
     }
     my ($libraries_to_query,$pretty_domains_string,$domain_sql_restriction) = 
         &parse_domain_restrictions();
     #
     # Check to see if enough of a query is filled in
-    my $search_string = $ENV{'form.basicexp'};
+    my $search_string = $env{'form.basicexp'};
     if (! &filled($search_string)) {
 	&output_blank_field_error($r,$closebutton,'phase=disp_basic');
 	return OK;
@@ -1637,7 +1637,7 @@ sub parse_basic_search {
                                              'notes','abstract','keywords')
                                             ).')';
     my ($error,$SQLQuery) = &process_phrase_input($search_string,
-                                                    $ENV{'form.related'},
+                                                    $env{'form.related'},
                                                     $searchfield);
     if ($error) {
         &output_unparsed_phrase_error($r,$closebutton,'phase=disp_basic',
@@ -2012,13 +2012,13 @@ sub copyright_check {
                                               $Metadata->{'url'});
     # Check for priv
     if (($Metadata->{'copyright'} eq 'priv') && 
-        (($ENV{'user.name'} ne $resname) &&
-         ($ENV{'user.domain'} ne $resdom))) {
+        (($env{'user.name'} ne $resname) &&
+         ($env{'user.domain'} ne $resdom))) {
         return 0;
     }
     # Check for domain
     if (($Metadata->{'copyright'} eq 'domain') &&
-        ($ENV{'user.domain'} ne $resdom)) {
+        ($env{'user.domain'} ne $resdom)) {
         return 0;
     }
     return 1;
@@ -2099,7 +2099,7 @@ sub print_sort_form {
     my $bodytag=&Apache::loncommon::bodytag(undef,undef,undef,1).
         &Apache::lonhtmlcommon::breadcrumbs
         (undef,'Searching','Searching',undef,undef,
-         $ENV{'form.catalogmode'} ne 'groupsearch');
+         $env{'form.catalogmode'} ne 'groupsearch');
 
     ##
     my %SortableFields=&Apache::lonlocal::texthash( 
@@ -2118,7 +2118,7 @@ sub print_sort_form {
          lastrevisiondate => 'Revision Date'
      );
     ##
-    my $table = $ENV{'form.table'};
+    my $table = $env{'form.table'};
     return if (! &ensure_db_and_table($r,$table));
     ##
     ## Get the number of results 
@@ -2139,7 +2139,7 @@ $html
 <script>
     function change_sort() {
         var newloc = "/adm/searchcat?phase=results";
-        newloc += "&persistent_db_id=$ENV{'form.persistent_db_id'}";
+        newloc += "&persistent_db_id=$env{'form.persistent_db_id'}";
         newloc += "&sortby=";
         newloc += document.forms.statusform.elements.sortby.value;
         parent.resultsframe.location= newloc;
@@ -2154,10 +2154,10 @@ END
 
 #<h2>Sort Results</h2>
 #Sort by: <select size="1" name="sortby" onchange="javascript:change_sort();">
-#    $ENV{'form.sortby'} = 'id' if (! defined($ENV{'form.sortby'}));
+#    $env{'form.sortby'} = 'id' if (! defined($env{'form.sortby'}));
 #    foreach (keys(%SortableFields)) {
 #        $result.="<option name=\"$_\"";
-#        if ($_ eq $ENV{'form.sortby'}) {
+#        if ($_ eq $env{'form.sortby'}) {
 #            $result.=" selected ";
 #        }
 #        $result.=" >$SortableFields{$_}</option>\n";
@@ -2208,7 +2208,7 @@ my @Fullindicies;
 =item &create_results_table()
 
 Creates the table of search results by calling lonmysql.  Stores the
-table id in $ENV{'form.table'}
+table id in $env{'form.table'}
 
 Inputs: none.
 
@@ -2239,7 +2239,7 @@ sub create_results_table {
             FULLTEXT => [{'columns' => \@Fullindicies},],
         } );
     if (defined($table)) {
-        $ENV{'form.table'} = $table;
+        $env{'form.table'} = $table;
         return $table;
     } 
     return undef; # Error...
@@ -2340,9 +2340,9 @@ Returns: html string for a 'revise searc
 ######################################################################
 sub revise_button {
     my $revise_phase = 'disp_basic';
-    $revise_phase = 'disp_adv' if ($ENV{'form.searchmode'} eq 'advanced');
+    $revise_phase = 'disp_adv' if ($env{'form.searchmode'} eq 'advanced');
     my $newloc = '/adm/searchcat'.
-        '?persistent_db_id='.$ENV{'form.persistent_db_id'}.
+        '?persistent_db_id='.$env{'form.persistent_db_id'}.
             '&cleargroupsort=1'.
             '&phase='.$revise_phase;
     my $result = qq{<input type="button" value="Revise search" name="revise"} .
@@ -2369,7 +2369,7 @@ sub run_search {
     my $bodytag=&Apache::loncommon::bodytag(undef,undef,undef,1);
     $bodytag.=&Apache::lonhtmlcommon::breadcrumbs
         (undef,'Searching','Searching',undef,undef,
-         $ENV{'form.catalogmode'} ne 'groupsearch');
+         $env{'form.catalogmode'} ne 'groupsearch');
     my $connection = $r->connection;
     #
     # Print run_search header
@@ -2411,7 +2411,7 @@ END
     my %Server_status;
     #
     # Check on the mysql table we will use to store results.
-    my $table =$ENV{'form.table'};
+    my $table =$env{'form.table'};
     if (! defined($table) || $table eq '' || $table =~ /\D/ ) {
         $r->print("Unable to determine table id to store search results in.".
                   "The search has been aborted.</body></html>");
@@ -2421,7 +2421,7 @@ END
     if (! defined($table_status)) {
         $r->print("Unable to determine status of table.</body></html>");
         &Apache::lonnet::logthis("Bogus table id of $table for ".
-                                 "$ENV{'user.name'} @ $ENV{'user.domain'}");
+                                 "$env{'user.name'} @ $env{'user.domain'}");
         &Apache::lonnet::logthis("lonmysql error = ".
                                  &Apache::lonmysql::get_error());
         return;
@@ -2560,11 +2560,11 @@ END
     # results to get, so let the client know the top frame needs to be
     # loaded from /adm/searchcat
     $r->print("</body></html>");
-#    if ($ENV{'form.catalogmode'} ne 'groupsearch') {
+#    if ($env{'form.catalogmode'} ne 'groupsearch') {
         $r->print("<script>".
                       "window.location='/adm/searchcat?".
                       "phase=sort&".
-                      "persistent_db_id=$ENV{'form.persistent_db_id'}';".
+                      "persistent_db_id=$env{'form.persistent_db_id'}';".
                   "</script>");
 #    }
     return;
@@ -2619,7 +2619,7 @@ sub display_results {
     ##
     ## Set viewing function
     ##
-    my $viewfunction = $Views{$ENV{'form.viewselect'}};
+    my $viewfunction = $Views{$env{'form.viewselect'}};
     if (!defined($viewfunction)) {
         $r->print("Internal Error - Bad view selected.\n");
         $r->rflush();
@@ -2635,7 +2635,7 @@ sub display_results {
     my $action = "/adm/searchcat?phase=results";
     ##
     ## Deal with groupsearch by opening the groupsearch db file.
-    if ($ENV{'form.catalogmode'} eq 'groupsearch') {
+    if ($env{'form.catalogmode'} eq 'groupsearch') {
         if (! tie(%groupsearch_db,'GDBM_File',$diropendb,
                   &GDBM_WRCREAT(),0640)) {
             $r->print('Unable to store import results.</form></body></html>');
@@ -2645,7 +2645,7 @@ sub display_results {
     }
     ##
     ## Prepare the table for querying
-    my $table = $ENV{'form.table'};
+    my $table = $env{'form.table'};
     return if (! &ensure_db_and_table($r,$table));
     ##
     ## Get the number of results 
@@ -2659,21 +2659,21 @@ sub display_results {
     }
     ##
     ## Determine how many results we need to get
-    $ENV{'form.start'} = 1  if (! exists($ENV{'form.start'}));
-    $ENV{'form.show'}  = 20 if (! exists($ENV{'form.show'}));
-    if (exists($ENV{'form.prev'})) {
-        $ENV{'form.start'} -= $ENV{'form.show'};
-    } elsif (exists($ENV{'form.next'})) {
-        $ENV{'form.start'} += $ENV{'form.show'};
-    }
-    $ENV{'form.start'} = 1 if ($ENV{'form.start'}<1);
-    $ENV{'form.start'} = $total_results if ($ENV{'form.start'}>$total_results);
-    my $min = $ENV{'form.start'};
+    $env{'form.start'} = 1  if (! exists($env{'form.start'}));
+    $env{'form.show'}  = 20 if (! exists($env{'form.show'}));
+    if (exists($env{'form.prev'})) {
+        $env{'form.start'} -= $env{'form.show'};
+    } elsif (exists($env{'form.next'})) {
+        $env{'form.start'} += $env{'form.show'};
+    }
+    $env{'form.start'} = 1 if ($env{'form.start'}<1);
+    $env{'form.start'} = $total_results if ($env{'form.start'}>$total_results);
+    my $min = $env{'form.start'};
     my $max;
-    if ($ENV{'form.show'} eq 'all') {
+    if ($env{'form.show'} eq 'all') {
         $max = $total_results ;
     } else {
-        $max = $min + $ENV{'form.show'} - 1;
+        $max = $min + $env{'form.show'} - 1;
         $max = $total_results if ($max > $total_results);
     }
     ##
@@ -2685,6 +2685,66 @@ sub display_results {
               );
     #
     # Build sorting selector
+<<<<<<< lonsearchcat.pm
+    my @field_order =  ('default',
+                        'title',
+                        'author',
+                        'subject',
+                        'url',
+                        'keywords',
+                        'version',
+                        'language',
+                        'creationdate'=>,
+                        'lastrevisiondate',
+                        'owner',
+                        'copyright',
+                        'authorspace',
+                        'lowestgradeleve',
+                        'highestgradelevel',
+                        'standards',
+                        'count',
+                        'stdno',
+                        'avetries',
+                        'difficulty',
+                        'disc',
+                        'clear',
+                        'technical',
+                        'correct',
+                        'helpful',
+                        'depth',
+                        );                                              
+    my %sort_fields = ('default'     => 'Default',
+                       'title'       => 'Title',
+                       'author'      => 'Author',
+                       'subject'     => 'Subject',
+                       'url'         => 'URL',
+                       'keywords'    => 'Keywords',
+                       'version'     => 'Version',
+                       'language'    => 'Language',
+                       'creationdate'=> 'Creation Date',
+                       'lastrevisiondate' => 'Last Revision Date',
+                       'owner'       => 'Owner',
+                       'copyright'   => 'Copyright',
+                       'authorspace' => 'Authorspace',
+                       'lowestgradeleve' => 'Lowest Grade Level',
+                       'highestgradelevel' => 'Highest Grade Level',
+                       'standards'   => 'Standards',
+                       'count'       => 'Number of Accesses',
+                       'stdno'       => 'Students Attempting',
+                       'avetries'    => 'Average Number of Tries',
+                       'difficulty'  => 'Mean Degree of Difficulty',
+                       'disc'        => 'Mean Degree of Discrimination',
+                       'clear'       => 'Evaluation: Clear',
+                       'technical'   => 'Evaluation: Technically Correct',
+                       'correct'     => 'Evaluation: Material is Correct',
+                       'helpful'     => 'Evaluation: Material is Helpful',
+                       'depth'       => 'Evaluation: Material has Depth',
+                       'select_form_order' => \@field_order,
+                       );
+
+    my $sortform = &mt('Sort by [_1]',
+                       &Apache::loncommon::select_form($env{'form.sortfield'},
+=======
     my @fields = 
         (
          {key=>'default' },
@@ -2731,6 +2791,7 @@ sub display_results {
     $ENV{'form.sortorder'} = 'asc' if (! exists($ENV{'form.sortorder'}));
     my $sortform = &mt('Sort by [_1] [_2]',
                        &Apache::loncommon::select_form($ENV{'form.sortfield'},
+>>>>>>> 1.242
                                                       'sortfield',
                                                       %sort_fields),
                        &Apache::loncommon::select_form($ENV{'form.sortorder'},
@@ -2745,7 +2806,7 @@ sub display_results {
         ('<table width="100%"><tr><td width="25%" align="right">'.
          '<nobr>'.$sortform.'</nobr>'.
          '</td><td width="25%" align="right">'.
-         &prev_next_buttons($min,$ENV{'form.show'},$total_results).
+         &prev_next_buttons($min,$env{'form.show'},$total_results).
          '</td><td align="right">'.
          &viewoptions().'</td></tr></table>'
          );
@@ -2763,6 +2824,12 @@ sub display_results {
     ##
     ## Get results from MySQL table
     my $sort_command  = 'id>='.$min.' AND id<='.$max;
+<<<<<<< lonsearchcat.pm
+    if ($env{'form.sortfield'} ne 'default' && 
+        exists($sort_fields{$env{'form.sortfield'}})) {
+        $sort_command = $env{'form.sortfield'}.' IS NOT NULL '.
+            'ORDER BY '.$env{'form.sortfield'}.
+=======
     my $order;
     if (exists($ENV{'form.sortorder'})) {
         if ($ENV{'form.sortorder'} eq 'asc') {
@@ -2779,6 +2846,7 @@ sub display_results {
         exists($sort_fields{$ENV{'form.sortfield'}})) {
         $sort_command = $ENV{'form.sortfield'}.' IS NOT NULL '.
             'ORDER BY '.$ENV{'form.sortfield'}.' '.$order.
+>>>>>>> 1.242
             '  LIMIT '.($min-1).','.($max-$min);
     }
     my @Results = &Apache::lonmysql::get_rows($table,$sort_command);
@@ -2807,11 +2875,11 @@ sub display_results {
     } else {
         $r->print
             ('<center>'.
-             &prev_next_buttons($min,$ENV{'form.show'},$total_results,
-                                "table=".$ENV{'form.table'}.
+             &prev_next_buttons($min,$env{'form.show'},$total_results,
+                                "table=".$env{'form.table'}.
                                 "&phase=results".
                                 "&persistent_db_id=".
-                                $ENV{'form.persistent_db_id'})
+                                $env{'form.persistent_db_id'})
              ."</center>\n"
              );
     }
@@ -2829,7 +2897,7 @@ sub display_results {
 =item &catalogmode_output($title,$url,$fnum,$checkbox_num)
 
 Returns html needed for the various catalog modes.  Gets inputs from
-$ENV{'form.catalogmode'}.  Stores data in %groupsearch_db.
+$env{'form.catalogmode'}.  Stores data in %groupsearch_db.
 
 =cut
 
@@ -2838,16 +2906,16 @@ $ENV{'form.catalogmode'}.  Stores data i
 sub catalogmode_output {
     my $output = '';
     my ($title,$url,$fnum,$checkbox_num) = @_;
-    if ($ENV{'form.catalogmode'} eq 'interactive') {
+    if ($env{'form.catalogmode'} eq 'interactive') {
         $title=~ s/\'/\\\'/g;
-        if ($ENV{'form.catalogmode'} eq 'interactive') {
+        if ($env{'form.catalogmode'} eq 'interactive') {
             $output.=<<END 
 <font size='-1'><INPUT TYPE="button" NAME="returnvalues" VALUE="SELECT"
 onClick="javascript:select_data('$title','$url')">
 </font>
 END
         }
-    } elsif ($ENV{'form.catalogmode'} eq 'groupsearch') {
+    } elsif ($env{'form.catalogmode'} eq 'groupsearch') {
         $groupsearch_db{"pre_${fnum}_link"}=$url;
         $groupsearch_db{"pre_${fnum}_title"}=$title;
         $output.=<<END;
@@ -2937,8 +3005,8 @@ sub handle_custom_fields {
     my $customshow='';
     my $extrashow='';
     my @customfields;
-    if ($ENV{'form.customshow'}) {
-        $customshow=$ENV{'form.customshow'};
+    if ($env{'form.customshow'}) {
+        $customshow=$env{'form.customshow'};
         $customshow=~s/[^\w\s]//g;
         my @fields=map {
             "<font color=\"#008000\">$_:</font><!-- $_ -->";
@@ -2972,7 +3040,7 @@ sub handle_custom_fields {
 Output the proper html headers and javascript code to deal with different 
 calling modes.
 
-Takes most inputs directly from %ENV, except $mode.  
+Takes most inputs directly from %env, except $mode.  
 
 =over 4
 
@@ -3017,8 +3085,8 @@ sub search_results_header {
     # output beginning of search page
     # conditional output of script functions dependent on the mode in
     # which the search was invoked
-    if ($ENV{'form.catalogmode'} eq 'interactive'){
-	if (! exists($ENV{'form.mode'}) || $ENV{'form.mode'} ne 'edit') {
+    if ($env{'form.catalogmode'} eq 'interactive'){
+	if (! exists($env{'form.mode'}) || $env{'form.mode'} ne 'edit') {
             $result.=<<SCRIPT;
 <script type="text/javascript">
     function select_data(title,url) {
@@ -3038,10 +3106,10 @@ sub search_results_header {
     }
 </script>
 SCRIPT
-        } elsif ($ENV{'form.mode'} eq 'edit') {
-            my $form = $ENV{'form.form'};
-            my $element = $ENV{'form.element'};
-            my $titleelement = $ENV{'form.titleelement'};
+        } elsif ($env{'form.mode'} eq 'edit') {
+            my $form = $env{'form.form'};
+            my $element = $env{'form.element'};
+            my $titleelement = $env{'form.titleelement'};
 	    my $changetitle;
 	    if (!$titleelement) {
 		$changetitle='function changeTitle(val) {}';
@@ -3078,7 +3146,7 @@ function changeURL(val) {
 SCRIPT
         }
     }
-    $result.=<<SCRIPT if $ENV{'form.catalogmode'} eq 'groupsearch';
+    $result.=<<SCRIPT if $env{'form.catalogmode'} eq 'groupsearch';
 <script type="text/javascript">
     function queue(checkbox_num,val) {
         if (document.forms.results.returnvalues.length != "undefined" &&
@@ -3098,7 +3166,7 @@ SCRIPT
     }
     function select_group() {
 	parent.window.location=
-    "/adm/groupsort?mode=$ENV{'form.mode'}&catalogmode=groupsearch&acts="+
+    "/adm/groupsort?mode=$env{'form.mode'}&catalogmode=groupsearch&acts="+
 	    parent.statusframe.document.forms.statusform.elements.Queue.value;
     }
 </script>
@@ -3129,8 +3197,8 @@ ENDSTATUS
 }
 
 sub results_link {
-    my $basic_link   = "/adm/searchcat?"."&table=".$ENV{'form.table'}.
-        "&persistent_db_id=".$ENV{'form.persistent_db_id'};
+    my $basic_link   = "/adm/searchcat?"."&table=".$env{'form.table'}.
+        "&persistent_db_id=".$env{'form.persistent_db_id'};
     my $results_link = $basic_link."&phase=results".
         "&pause=1"."&start=1";
     return $results_link;
@@ -3140,8 +3208,8 @@ sub results_link {
 ######################################################################
 sub print_frames_interface {
     my $r = shift;
-    my $basic_link = "/adm/searchcat?"."&table=".$ENV{'form.table'}.
-        "&persistent_db_id=".$ENV{'form.persistent_db_id'};
+    my $basic_link = "/adm/searchcat?"."&table=".$env{'form.table'}.
+        "&persistent_db_id=".$env{'form.persistent_db_id'};
     my $run_search_link = $basic_link."&phase=run_search";
     my $results_link = &results_link();
     my $html=&Apache::lonxml::xmlbegin();
@@ -3551,9 +3619,9 @@ sub output_unparsed_phrase_error {
     my ($r,$closebutton,$parms,$hidden_fields,$field)=@_;
     my $errorstring;
     if ($field eq 'basicexp') {
-        $errorstring = &mt('Unable to understand the search phrase <i>[_1]</i>.  Please modify your search.',$ENV{'form.basicexp'});
+        $errorstring = &mt('Unable to understand the search phrase <i>[_1]</i>.  Please modify your search.',$env{'form.basicexp'});
     } else {
-        $errorstring = &mt('Unable to understand the search phrase <b>[_1]</b>:<i>[_2]</i>.',$field,$ENV{'form.'.$field});
+        $errorstring = &mt('Unable to understand the search phrase <b>[_1]</b>:<i>[_2]</i>.',$field,$env{'form.'.$field});
     }
     my $bodytag = &Apache::loncommon::bodytag('Search');
     my $heading = &mt('Unparsed Field');
@@ -3575,7 +3643,7 @@ $closebutton
 $errorstring
 </p>
 <p>
-<a href="/adm/searchcat?$parms&persistent_db_id=$ENV{'form.persistent_db_id'}">$revise</a>
+<a href="/adm/searchcat?$parms&persistent_db_id=$env{'form.persistent_db_id'}">$revise</a>
 </p>
 </body>
 </html>
@@ -3624,7 +3692,7 @@ $closebutton
 $errormsg
 </p>
 <p>
-<a href="/adm/searchcat?$parms&persistent_db_id=$ENV{'form.persistent_db_id'}">$revise</a>&nbsp;
+<a href="/adm/searchcat?$parms&persistent_db_id=$env{'form.persistent_db_id'}">$revise</a>&nbsp;
 </p>
 </body>
 </html>