--- loncom/interface/lontest.pm	2008/11/14 21:26:54	1.20
+++ loncom/interface/lontest.pm	2014/12/15 00:59:40	1.23
@@ -1,7 +1,7 @@
 # The LearningOnline Network with CAPA
 # A debugging harness.
 #
-# $Id: lontest.pm,v 1.20 2008/11/14 21:26:54 jms Exp $
+# $Id: lontest.pm,v 1.23 2014/12/15 00:59:40 raeburn Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -27,48 +27,6 @@
 #
 #
 
-=head1 NAME
-
-Apache::lontest;
-
-=head1 SYNOPSIS
-
-Used for debugging and testing the LONCAPA
-system.
-
-This is part of the LearningOnline Network with CAPA project
-described at http://www.lon-capa.org.
-
-=head1 HANDLER SUBROUTINE
-
-handler()
-
-=head1 OTHER SUBROUTINES
-
-=over
-
-=item *
-
-section() : 
-
-section takes one env var name as input, and returns
-what section the given env var is in, which is the part
-of the env var before the first period.
-Returns the section, or blank string for 'no section',
-which is normal for the standard env vars like REQUEST_URI.
-
-=item *
-
-print_hash()
-
-=item *
-
-
-
-
-=back
-
-=cut
 
 package Apache::lontest;
 
@@ -142,8 +100,8 @@ sub handler {
 
 	if (tie(%hash,'GDBM_File',"$fn.db",&GDBM_READER(),0640)) {
 	    $r->print('<h2>Big Hash</h2>');
-	    foreach (sort keys %hash) {
-		$r->print("\n<br />".$_.': '.$hash{$_});
+	    foreach my $key (sort(keys(%hash))) {
+		$r->print("\n<br />".$key.': '.$hash{$key});
 	    }
 	    untie %hash;
 	} else {
@@ -153,8 +111,8 @@ sub handler {
 		$env{'request.course.fn'}.'_parms.db',
 		&GDBM_READER(),0640)) {
 	    $r->print('<h2>Parm Hash</h2>');
-	    foreach (sort keys %parmhash) {
-	        $r->print("\n<br />".$_.': '.$parmhash{$_});
+	    foreach my $param (sort(keys(%parmhash))) {
+	        $r->print("\n<br />".$param.': '.$parmhash{$param});
 	    }
 	    untie %parmhash;
 	} else {
@@ -162,8 +120,8 @@ sub handler {
 	}
 	if (tie(%symbhash,'GDBM_File',"$fn\_symb.db",&GDBM_READER(),0640)) {
             $r->print('<h2>Symb Hash</h2>');
-            foreach (sort keys %symbhash) {
-		$r->print("\n<br />".$_.': '.$symbhash{$_});
+            foreach my $symb (sort(keys(%symbhash))) {
+		$r->print("\n<br />".$symb.': '.$symbhash{$symb});
             }
             untie %symbhash;
 	} else {
@@ -176,8 +134,8 @@ sub handler {
 		my $fh=Apache::File->new($fn.'.state');
 		@conditions=<$fh>;
 	    }
-	    foreach (@conditions) {
-		$r->print('<tt>'.$_.'</tt><br />');
+	    foreach my $cond (@conditions) {
+		$r->print('<tt>'.$cond.'</tt><br />');
 	    }
 	}
     }
@@ -191,6 +149,41 @@ sub handler {
 1;
 __END__
 
+=pod
+
+=head1 NAME
+
+Apache::lontest;
+
+=head1 SYNOPSIS
+
+Used for debugging and testing the LONCAPA
+system.
+
+This is part of the LearningOnline Network with CAPA project
+described at http://www.lon-capa.org.
+
+=head1 HANDLER SUBROUTINE
+
+handler()
+
+=head1 OTHER SUBROUTINES
+
+=over
+
+=item  section() : 
+
+section takes one env var name as input, and returns
+what section the given env var is in, which is the part
+of the env var before the first period.
+Returns the section, or blank string for 'no section',
+which is normal for the standard env vars like REQUEST_URI.
+
+=item  print_hash()
+
+=back
+
+=cut