--- loncom/interface/lonevaluate.pm	2001/12/21 20:42:11	1.5
+++ loncom/interface/lonevaluate.pm	2003/07/05 10:07:11	1.12
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Evaluate
 #
-# $Id: lonevaluate.pm,v 1.5 2001/12/21 20:42:11 www Exp $
+# $Id: lonevaluate.pm,v 1.12 2003/07/05 10:07:11 www Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -33,13 +33,14 @@
 #
 # 3/1/1 Gerd Kortemeyer)
 #
-# 3/1,12/6,12/12,12/21 Gerd Kortemeyer
+# 3/1,12/6,12/12,12/21,12/22 Gerd Kortemeyer
 #
 package Apache::lonevaluate;
 
 use strict;
 use Apache::lonnet;
 use Apache::Constants qw(:common);
+use Apache::loncommon;
 
 sub handler {
     my $r = shift;
@@ -47,6 +48,7 @@ sub handler {
     $r->send_http_header;
     return OK if $r->header_only;
 
+    &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['postdata']);
   my $feedurl=$ENV{'form.postdata'};
   $feedurl=~s/^http\:\/\///;
   $feedurl=~s/^$ENV{'SERVER_NAME'}//;
@@ -57,7 +59,7 @@ sub handler {
 
     my ($rdomain,$rauth)=($feedurl=~/^(\w+)\/(\w+)\//);
 
-    my $prefix=$ENV{'user.name.'}.'@'.$ENV{'user.domain'}.'___'.
+    my $prefix=$ENV{'user.name'}.'@'.$ENV{'user.domain'}.'___'.
 	$feedurl.'___';
 
     my @items=('correct','helpful','depth','clear','technical','comments',
@@ -66,7 +68,7 @@ sub handler {
     my @retrieve=map{ $prefix.$_ } @items;
 
     my %currenteval=
-       &Apache::lonnet::get('resevaldata',\@retrieve,$rdomain,$rauth);
+       &Apache::lonnet::get('nohist_resevaldata',\@retrieve,$rdomain,$rauth);
 
     my $already=($currenteval{$prefix.'timestamp'} ne '');
 
@@ -74,6 +76,8 @@ sub handler {
 
     my $warning='';
 
+    my $bodytag=&Apache::loncommon::bodytag('Evaluate Resource');
+
     if ($ENV{'form.submiteval'} eq 'true') {
 # ------------------------------------------------ User is submitting something
         my $complete=1;
@@ -89,23 +93,32 @@ sub handler {
 <html>
 <head>
 <title>Evaluation submitted</title>
+<script>
+    function go() {
+	if ((window.name=='loncapaclient')) {
+            window.location='/res/$feedurl';
+        } else {
+            self.close();
+        }
+    }
+</script>
 <meta http-equiv="pragma" content="no-cache"></meta>
-<meta HTTP-EQUIV="Refresh" CONTENT="2; url=/res/$feedurl">
 </head>
-<body bgcolor="#FFFFFF">
+$bodytag
 <h1>Thank you for your input!</h1>
 ENDHEADER
        $r->print('Storing feedback: '.
-           &Apache::lonnet::put('resevaldata',\%currenteval,$rdomain,$rauth));
+           &Apache::lonnet::put('nohist_resevaldata',\%currenteval,$rdomain,$rauth));
            unless ($already) {
 	       $r->print('<br>Logging first evaluation');
            }
-           $r->print('</body></html>');
+           $r->print(
+            '<script>setTimeout("go()",2000);</script></body></html>');
            return OK;
         } else {
 	    $warning='Please fill out all fields below';
         }
-    
+   
     }
 
 # ---------------------------------------------------------- Dial in old values
@@ -114,7 +127,8 @@ ENDHEADER
 
     foreach my $item (@items) {
        unless ($item eq 'comments') {
-           unless ($item eq 'timestamp') {
+           unless (($item eq 'timestamp') || 
+                   (!defined($currenteval{$prefix.$item}))) {
               $dialold.=
                    'document.evaluation.'.$item.'.selectedIndex='.
                    $currenteval{$prefix.$item}.";\n";
@@ -139,8 +153,7 @@ ENDOPTIONS
 <head>
 <title>The LearningOnline Network with CAPA</title>
 </head>
-<body bgcolor="#FFFFFF">
-<h1>Evaluate Resource</h1>
+$bodytag
 <tt>$feedurl</tt><p>
 <h3><font color="red">$warning</font></h3>
 Please rank the following criteria: