--- loncom/interface/lonevaluate.pm	2005/02/17 08:29:42	1.15
+++ loncom/interface/lonevaluate.pm	2006/03/19 22:22:49	1.19
@@ -1,7 +1,7 @@
 # The LearningOnline Network
 # Evaluate
 #
-# $Id: lonevaluate.pm,v 1.15 2005/02/17 08:29:42 albertel Exp $
+# $Id: lonevaluate.pm,v 1.19 2006/03/19 22:22:49 albertel Exp $
 #
 # Copyright Michigan State University Board of Trustees
 #
@@ -25,32 +25,25 @@
 #
 # http://www.lon-capa.org/
 #
-# (Internal Server Error Handler
-#
-# (Login Screen
-# 5/21/99,5/22,5/25,5/26,5/31,6/2,6/10,7/12,7/14,
-# 1/14/00,5/29,5/30,6/1,6/29,7/1,11/9 Gerd Kortemeyer)
-#
-# 3/1/1 Gerd Kortemeyer)
-#
-# 3/1,12/6,12/12,12/21,12/22 Gerd Kortemeyer
-#
+
 package Apache::lonevaluate;
 
 use strict;
-use Apache::lonnet();
+use Apache::lonnet;
 use Apache::Constants qw(:common);
 use Apache::loncommon();
 use Apache::lonenc();
 
 sub handler {
     my $r = shift;
-    $r->content_type('text/html');
+    &Apache::loncommon::content_type($r,'text/html');
+    &Apache::loncommon::no_cache($r);
+    
     $r->send_http_header;
     return OK if $r->header_only;
 
     &Apache::loncommon::get_unprocessed_cgi($ENV{'QUERY_STRING'},['postdata']);
-    my $feedurl=$ENV{'form.postdata'};
+    my $feedurl=$env{'form.postdata'};
     $feedurl=~s/^http\:\/\///;
     $feedurl=~s/^$ENV{'SERVER_NAME'}//;
     $feedurl=~s/^$ENV{'HTTP_HOST'}//;
@@ -62,7 +55,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',
@@ -75,30 +68,24 @@ sub handler {
 
     my $already=($currenteval{$prefix.'timestamp'} ne '');
 
-    $ENV{'form.timestamp'}=time;
+    $env{'form.timestamp'}=time;
 
     my $warning='';
 
-    my $bodytag=&Apache::loncommon::bodytag('Evaluate Resource');
-
-    if ($ENV{'form.submiteval'} eq 'true') {
+    if ($env{'form.submiteval'} eq 'true') {
 # ------------------------------------------------ User is submitting something
         my $complete=1;
 	foreach my $item (@items) {
-	    if ($ENV{'form.'.$item}) {
-		$currenteval{$prefix.$item}=$ENV{'form.'.$item};
+	    if ($env{'form.'.$item}) {
+		$currenteval{$prefix.$item}=$env{'form.'.$item};
             } elsif ($item ne 'comments') {
                 $complete=0;
             }
         }
         if ($complete) {
 	    my $showurl=&Apache::lonenc::check_encrypt(&Apache::lonnet::clutter($feedurl));
-	    my $html=&Apache::lonxml::xmlbegin();
-	    $r->print(<<ENDHEADER);
-$html
-<head>
-<title>Evaluation submitted</title>
-<script>
+	    my $js =<<ENDJS;
+<script type="text/javascript">
     function goback() {
 	if ((window.name=='loncapaclient')) {
             window.location='$showurl';
@@ -107,18 +94,21 @@ $html
         }
     }
 </script>
-<meta http-equiv="pragma" content="no-cache"></meta>
-</head>
-$bodytag
+ENDJS
+
+            my $start_page = 
+	        &Apache::loncommon::start_page('Evaluate Resource',$js);
+	    $r->print(<<ENDHEADER);
+$start_page
 <h1>Thank you for your input!</h1>
 ENDHEADER
        $r->print('Storing feedback: '.
            &Apache::lonnet::put('nohist_resevaldata',\%currenteval,$rdomain,$rauth));
            unless ($already) {
-	       $r->print('<br>Logging first evaluation');
+	       $r->print('<br />Logging first evaluation');
            }
            $r->print(
-            '<script>setTimeout("goback()",2000);</script></body></html>');
+            '<script type="text/javascript">setTimeout("goback()",2000);</script>'.&Apache::loncommon::end_page());
            return OK;
         } else {
 	    $warning='Please fill out all fields below';
@@ -144,60 +134,56 @@ ENDHEADER
     }
 # ------------------------------------------------------- Print out eval screen
     my $standardoptions=(<<ENDOPTIONS);
-<option value=0>-</option>
-<option value=1>Strongly Disagree</option>
-<option value=2>Disagree</option>
-<option value=3>Neutral</option>
-<option value=4>Agree</option>
-<option value=5>Strongly Agree</option>
+<option value="0">-</option>
+<option value="1">Strongly Disagree</option>
+<option value="2">Disagree</option>
+<option value="3">Neutral</option>
+<option value="4">Agree</option>
+<option value="5">Strongly Agree</option>
 ENDOPTIONS
     
-    my $html=&Apache::lonxml::xmlbegin();
+    my $start_page = &Apache::loncommon::start_page('Evaluate Resource');
+    my $end_page   = &Apache::loncommon::end_page();
     $r->print(<<ENDDOCUMENT);
-$html
-<head>
-<title>The LearningOnline Network with CAPA</title>
-</head>
-$bodytag
-<tt>$title</tt><p>
+$start_page
+<tt>$title</tt><br /><br />
 <h3><font color="red">$warning</font></h3>
 Please rank the following criteria:
-<form method=post name=evaluation>
-<input type=hidden name=submiteval value=true>
-<input type=hidden name=postdata value="$showurl">
+<form method="post" name="evaluation" action="/adm/evaluate">
+<input type="hidden" name="submiteval" value="true" />
+<input type="hidden" name="postdata" value="$showurl" />
 The material appears to be correct
-<br><select name=correct>
+<br /><select name="correct">
 $standardoptions
 </select>
-<hr>
+<hr />
 The material is helpful
-<br><select name=helpful>
+<br /><select name="helpful">
 $standardoptions
 </select>
-<hr>
+<hr />
 The material is covered with sufficient depth
-<br><select name=depth>
+<br /><select name="depth">
 $standardoptions
 </select>
-<hr>
+<hr />
 The material is presented in a clear way
-<br><select name=clear>
+<br /><select name="clear">
 $standardoptions
 </select>
-<hr>
+<hr />
 The resource is technically correct 
 (loads fast enough, does not produce errors, links work, etc) 
-<br><select name=technical>
+<br /><select name="technical">
 $standardoptions
 </select>
-<hr>
+<hr />
 Any comments?
-<br><textarea name=comments cols=40 rows=5>$oldcomments</textarea>
-<script>$dialold</script>
-<br><input type=submit value="Submit Evaluation">
+<br /><textarea name="comments" cols="40" rows="5">$oldcomments</textarea>
+<script type="text/javascript">$dialold</script>
+<br /><input type="submit" value="Submit Evaluation" />
 </form>
-</body>
-</html>
+$end_page
 ENDDOCUMENT
     return OK;
 }