--- rat/lonratparms.pm	2001/07/04 15:35:44	1.7
+++ rat/lonratparms.pm	2002/03/11 19:54:31	1.9
@@ -1,6 +1,30 @@
 # The LearningOnline Network with CAPA
 # Set parameters inside of the RAT
 #
+# $Id: lonratparms.pm,v 1.9 2002/03/11 19:54:31 matthew Exp $
+#
+# Copyright Michigan State University Board of Trustees
+#
+# This file is part of the LearningOnline Network with CAPA (LON-CAPA).
+#
+# LON-CAPA is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# (at your option) any later version.
+#
+# LON-CAPA is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with LON-CAPA; if not, write to the Free Software
+# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+#
+# /home/httpd/html/adm/gpl.txt
+#
+# http://www.lon-capa.org/
+#
 # (Search Catalog
 #
 # (Internal Server Error Handler
@@ -14,6 +38,7 @@
 # 3/1 Gerd Kortemeyer)
 #
 # 03/03/01,06/15,06/25,06/26,06/30,07/02,07/03,07/04 Gerd Kortemeyer
+# 03/11/02 Matthew Hall
 #
 package Apache::lonratparms;
 
@@ -28,7 +53,7 @@ sub handler {
 
 # Get query string for limited number of parameters
 
-    map {
+    foreach (split(/&/,$ENV{'QUERY_STRING'})) {
        my ($name, $value) = split(/=/,$_);
        $value =~ tr/+/ /;
        $value =~ s/%([a-fA-F0-9][a-fA-F0-9])/pack("C",hex($1))/eg;
@@ -38,7 +63,7 @@ sub handler {
               $ENV{'form.'.$name}=$value;
 	   }
        }
-    } (split(/&/,$ENV{'QUERY_STRING'}));
+    }
 
 
 # ------------------------------------------------------------------- Read file
@@ -50,21 +75,21 @@ sub handler {
   my %display;
   my %value;
   
-  map {
+  foreach (split(/\,/,&Apache::lonnet::metadata($uri,'keys'))) {
       if ($_=~/^parameter\_/) {
          $content{$_}=&Apache::lonnet::metadata($uri,$_);
          $type{$_}=&Apache::lonnet::metadata($uri,$_.'.type');
          $display{$_}=&Apache::lonnet::metadata($uri,$_.'.display');
       }
-  } split(/\,/,&Apache::lonnet::metadata($uri,'keys'));
+  }
 
 # ---------------------------------------------------------- Current Parameters
 
-  map {
+  foreach (split(/\:/,$ENV{'form.parms'})) {
       my ($ptype,$pname,$pvalue)=split(/\_\_\_/,$_);
       unless ($type{$pname}) { $type{$pname}=$ptype; }
       $value{$pname}=$pvalue;
-  } split(/\:/,$ENV{'form.parms'});
+  } 
   
 # --------------------------------------------------- Print input screen header
     $r->print(<<ENDHEADER);
@@ -119,21 +144,20 @@ ENDHEADER
 <table border=2>
 <tr><th>Parameter</th><th>Default</th><th>Value</th><th>Set?</th></tr>
 ENDDOCUMENT
-    map {
+    foreach (keys(%content)) {
         my $cur=$content{$_};
+        # Should if(defined($value{$_})) be if(exists($value{$_})) ?
         if (defined($value{$_})) { $cur=$value{$_}; };
-        $r->print('<tr><td><b>'.$display{$_}.'</b><br><tt>'.$_.
-        '</tt></td><td>&nbsp;'.$content{$_}.
-        '</td><td><input type="text" size="10" name="'.$_.'" value="'.$cur.
-        '">&nbsp;<a href='."'".'javascript:pjump("'.
-		  $type{$_}.'","'.
-                  $display{$_}.'",document.parameters.'.$_.
-                  '.value,"'.$_.'","parameters.pres","stpr");'."'>Enter</a>".
-        '</td><td><input type="checkbox" name="def_'.$_.
-        '"');
+        $r->print(<<"END");
+<tr><td><b>$display{$_}</b><br><tt>$_</tt></td>
+<td>&nbsp;$content{$_}</td>
+<td><input type="text" size="10" name="$_" value="$cur">&nbsp;
+<a href='javascript:pjump("$type{$_}","$display{$_}",document.parameters.$_.value,"$_","parameters.pres","stpr");'>Enter</a></td>
+<td><input type="checkbox" name="def_$_" 
+END
         if ($value{$_}) { $r->print(' checked'); }
         $r->print('></td></tr>');
-    } keys %content;
+    }
     $r->print(
         '</table><br><input type="submit" value="Set"></form></body></html>');
     return OK;