--- loncom/interface/groupsort.pm	2004/07/12 20:42:31	1.29
+++ loncom/interface/groupsort.pm	2005/04/07 06:56:22	1.31
@@ -2,7 +2,7 @@
 # The LON-CAPA group sort handler
 # Allows for sorting prior to import into RAT.
 #
-# $Id: groupsort.pm,v 1.29 2004/07/12 20:42:31 matthew Exp $
+# $Id: groupsort.pm,v 1.31 2005/04/07 06:56:22 albertel Exp $
 # 
 # Copyright Michigan State University Board of Trustees
 #
@@ -69,7 +69,7 @@ sub handler {
 
 # finish_import looks different for graphical or "simple" RAT
     my $finishimport='';
-    if ($ENV{'form.mode'} eq 'simple' || $ENV{'form.mode'} eq '') {
+    if ($env{'form.mode'} eq 'simple' || $env{'form.mode'} eq '') {
         $finishimport=(<<ENDSMP);
 function finish_import() {
     opener.document.forms.simpleedit.importdetail.value='';
@@ -105,9 +105,9 @@ ENDADV
     }
 
 # output start of web page
-
+    my $html=&Apache::lonxml::xmlbegin();
     $r->print(<<END);
-<html>
+$html
 <head>
 <title>The LearningOnline Network With CAPA Group Sorter</title>
 <script language='javascript'>
@@ -150,17 +150,17 @@ END
 
     my $diropendb;
 # ------------------------------ which file do we open? Easy if explictly given
-    if ($ENV{'form.catalogmode'} eq 'groupsearch') {
+    if ($env{'form.catalogmode'} eq 'groupsearch') {
 	$diropendb = 
-	    "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_searchcat.db";
+	    "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_searchcat.db";
     }
-    elsif ($ENV{'form.catalogmode'} eq 'groupimport') {
+    elsif ($env{'form.catalogmode'} eq 'groupimport') {
 	$diropendb = 
-	    "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_indexer.db";
+	    "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_indexer.db";
     }
-    elsif ($ENV{'form.catalogmode'} eq 'groupsec') {
+    elsif ($env{'form.catalogmode'} eq 'groupsec') {
 	$diropendb = 
-	    "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_groupsec.db";
+	    "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_groupsec.db";
     }
 # --------------------- not explicitly given, choose the one most recently used
     else { # choose last accessed
@@ -168,19 +168,19 @@ END
         my @dbst;
 
 	$dbfn[0] =
-	    "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_searchcat.db";
+	    "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_searchcat.db";
         $dbst[0]=-1;
 	if (-e $dbfn[0]) {
 	    $dbst[0]=(stat($dbfn[0]))[9];
 	}
 	$dbfn[1] =
-            "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_indexer.db";
+            "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_indexer.db";
         $dbst[1]=-1;
 	if (-e $dbfn[1]) {
             $dbst[1]=(stat($dbfn[1]))[9];
         }
 	$dbfn[2] =
-            "/home/httpd/perl/tmp/$ENV{'user.domain'}_$ENV{'user.name'}_groupsec.db";
+            "/home/httpd/perl/tmp/$env{'user.domain'}_$env{'user.name'}_groupsec.db";
         $dbst[2]=-1;
 	if (-e $dbfn[2]) {
             $dbst[2]=(stat($dbfn[2]))[9];
@@ -202,7 +202,7 @@ END
     }
 # ----------------------------- diropendb is now the filename of the db to open
     if (tie(%hash,'GDBM_File',$diropendb,&GDBM_WRCREAT(),0640)) {
-	my $acts = $ENV{'form.acts'};
+	my $acts = $env{'form.acts'};
 	my @Acts = split(/b/,$acts);
 	my %ahash;
 	my %achash;
@@ -242,7 +242,7 @@ END
 		}
 	    }
 	}
-	if ($ENV{'form.oldval'}) {
+	if ($env{'form.oldval'}) {
 	    my $newctr = 0;
 	    my %chash;
 	    foreach (sort {$shash{$a} <=> $shash{$b}} (keys %shash)) {
@@ -252,8 +252,8 @@ END
 		$hash{'storectr_'.$key} = $newctr;
 		$chash{$newctr} = $key;
 	    }
-	    my $oldval = $ENV{'form.oldval'};
-	    my $newval = $ENV{'form.newval'};
+	    my $oldval = $env{'form.oldval'};
+	    my $newval = $env{'form.newval'};
 	    if ($oldval != $newval) {
 		# when newval==0, then push down and delete
 		if ($newval!=0) {
@@ -314,18 +314,18 @@ END
 <input type="hidden" name="fnum" value="$clen" />
 <input type="hidden" name="oldval" value="" />
 <input type="hidden" name="newval" value="" />
-<input type="hidden" name="mode" value="$ENV{'form.mode'}" />
+<input type="hidden" name="mode" value="$env{'form.mode'}" />
 END
 
         # --- Expand here if "GO BACK" button desired
-        if ($ENV{'form.catalogmode'} eq 'groupimport') {
+        if ($env{'form.catalogmode'} eq 'groupimport') {
             my $resurl = &Apache::loncommon::lastresurl();
 	    $r->print(<<END);
 <input type="button" name="alter" value="$lt{'gb'}"
  onClick="window.location='$resurl?catalogmode=groupimport'" />&nbsp;
 END
         }
-	if ($ENV{'form.catalogmode'} eq 'groupsearch') {
+	if ($env{'form.catalogmode'} eq 'groupsearch') {
 	    $r->print(<<END);
 <input type="button" name="alter" value="$lt{'ns'}"
  onClick="window.location='/adm/searchcat?catalogmode=groupsearch&cleargroupsort=1'" />&nbsp;
@@ -351,7 +351,7 @@ END
 <input type="hidden" name="fnum" value="$clen" />
 <input type="hidden" name="oldval" value="" />
 <input type="hidden" name="newval" value="" />
-<input type="hidden" name="mode" value="$ENV{'form.mode'}" />
+<input type="hidden" name="mode" value="$env{'form.mode'}" />
 END
     }
     foreach (sort {$shash{$a}<=>$shash{$b}} (keys %shash)) {
